caleb-brown/temper
1
0
Fork 0
Code Issues 36 Pull requests Projects Releases Packages Wiki Activity Actions

Add manual, skip, and fork-review controls #19

New issue
Open
opened 2026-07-14 22:32:24 -04:00 by caleb-brown · 0 comments
caleb-brown commented 2026-07-14 22:32:24 -04:00
Owner
Copy link

What to build

Authorized maintainers can control review behavior from Forgejo using explicit commands and labels, while untrusted contributors cannot spend review budget or authorize later fork revisions.

Acceptance criteria

  • The documented manual review and rerun commands create intentional work for the current revision.
  • Skip pauses automatic review according to policy while leaving authorized explicit review available.
  • Profile-selection controls affect the intended request without silently selecting an invalid profile.
  • Authorization is checked live through Forgejo permissions rather than trusting webhook payload role fields.
  • Fork pull requests never auto-review and require explicit maintainer authorization for each exact head SHA.
  • An authorization attached to an older fork revision cannot authorize a later push.
  • Tests cover authorized, unauthorized, stale-authorization, malformed-command, skip, and profile-selection scenarios.

Blocked by

  • #12 — Install a repository and prove webhook delivery
  • #18 — Keep feedback correct across retries and revisions
## What to build Authorized maintainers can control review behavior from Forgejo using explicit commands and labels, while untrusted contributors cannot spend review budget or authorize later fork revisions. ## Acceptance criteria - [ ] The documented manual review and rerun commands create intentional work for the current revision. - [ ] Skip pauses automatic review according to policy while leaving authorized explicit review available. - [ ] Profile-selection controls affect the intended request without silently selecting an invalid profile. - [ ] Authorization is checked live through Forgejo permissions rather than trusting webhook payload role fields. - [ ] Fork pull requests never auto-review and require explicit maintainer authorization for each exact head SHA. - [ ] An authorization attached to an older fork revision cannot authorize a later push. - [ ] Tests cover authorized, unauthorized, stale-authorization, malformed-command, skip, and profile-selection scenarios. ## Blocked by - [#12](https://git.caleb-brown.dev/caleb-brown/temper/issues/12) — Install a repository and prove webhook delivery - [#18](https://git.caleb-brown.dev/caleb-brown/temper/issues/18) — Keep feedback correct across retries and revisions
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
dev
t3code/hygiene-review
t3code/configurable-llm-timeout
fix/openai-non-streaming-completions
t3code/test-incorrect-changes
No results found.
Labels
Clear labels   
ready-for-agent
Fully specified and ready for an implementation agent

  
wayfinder:grilling
Human-in-the-loop product or domain decision

  
wayfinder:map
Canonical Wayfinder map

  
wayfinder:prototype
Human-in-the-loop concrete prototype

  
wayfinder:research
Agent-driven primary-source research

  
wayfinder:task
Task that unblocks a Wayfinder decision

No labels
ready-for-agent
wayfinder:grilling
wayfinder:map
wayfinder:prototype
wayfinder:research
wayfinder:task
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Blocks
#26 Establish Forgejo compatibility and review-quality gates
caleb-brown/temper
Depends on
#12 Install a repository and prove webhook delivery
caleb-brown/temper
#18 Keep feedback correct across retries and revisions
caleb-brown/temper
Reference
caleb-brown/temper#19
No description provided.