nimmo/forgejo-auto-pr-review
Watch
1
0
Fork
You've already forked forgejo-auto-pr-review
 0
Code Issues 5 Pull requests Projects Releases Packages 1 Activity Actions

Support explicit comment-triggered re-reviews #21

New issue
Open
opened 2026-07-21 23:00:43 +01:00 by nimmo · 0 comments
nimmo commented 2026-07-21 23:00:43 +01:00
Owner
Copy link

Add an opt-in re-review path for pull requests where human discussion changes the review context but the code has not changed.

Scope:

  • Retain push-based reviews as the default.
  • Support an explicit human request, such as a /ai-review PR comment or a documented manual-dispatch equivalent.
  • Include human replies posted after the previous automated review in the new prompt.
  • Ignore bot-authored comments and prevent recursive/self-triggered runs.
  • Deduplicate requests for the same head commit and provide clear feedback when a request is ignored or already running.

Acceptance:

  • A human can request a re-review without pushing code.
  • Ordinary PR discussion does not automatically spend model time.
  • The re-review updates the existing single automated comment and remains safe for forks/tokens.
Add an opt-in re-review path for pull requests where human discussion changes the review context but the code has not changed. Scope: - Retain push-based reviews as the default. - Support an explicit human request, such as a `/ai-review` PR comment or a documented manual-dispatch equivalent. - Include human replies posted after the previous automated review in the new prompt. - Ignore bot-authored comments and prevent recursive/self-triggered runs. - Deduplicate requests for the same head commit and provide clear feedback when a request is ignored or already running. Acceptance: - A human can request a re-review without pushing code. - Ordinary PR discussion does not automatically spend model time. - The re-review updates the existing single automated comment and remains safe for forks/tokens.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
fix/action-yaml-parser
issue-4-normalize-analysis
No results found.
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
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.

Dependencies

No dependencies set

Reference
nimmo/forgejo-auto-pr-review#21
No description provided.