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

Harden PR context collection and manual reruns #6

New issue
Closed
opened 2026-07-20 15:53:56 +01:00 by nimmo · 0 comments
nimmo commented 2026-07-20 15:53:56 +01:00
Owner
Copy link

Close the remaining Phase 1 correctness gaps in review context.

Scope:

  • Make workflow_dispatch check out the selected PR head before deterministic checks run.
  • Collect PR commit messages and safely available existing automated-review context.
  • Complete documented guidance lookup, including docs/ and optionally scoped guidance near changed files.
  • Validate event/manual inputs and report unavailable context explicitly.

Acceptance:

  • A manual rerun reviews the same commit and working tree as the selected PR.
  • Context coverage and limitations are visible in the final comment.

Completed

Delivered in merged PR #9:

  • Manual dispatch checks out the selected PR head.
  • Reviews include base/head metadata, commit messages, and bounded prior automated-review context.
  • Guidance lookup includes docs/ before README; per-file scoped guidance was optional and is deferred.
  • Positive pull-request input validation and review metadata make context coverage visible.

Live validation: PR #10 ran successfully against Nova and updated its marker-based review comment after a subsequent push.

Close the remaining Phase 1 correctness gaps in review context. Scope: - Make workflow_dispatch check out the selected PR head before deterministic checks run. - Collect PR commit messages and safely available existing automated-review context. - Complete documented guidance lookup, including docs/ and optionally scoped guidance near changed files. - Validate event/manual inputs and report unavailable context explicitly. Acceptance: - A manual rerun reviews the same commit and working tree as the selected PR. - Context coverage and limitations are visible in the final comment. ## Completed Delivered in merged PR #9: - Manual dispatch checks out the selected PR head. - Reviews include base/head metadata, commit messages, and bounded prior automated-review context. - Guidance lookup includes docs/ before README; per-file scoped guidance was optional and is deferred. - Positive pull-request input validation and review metadata make context coverage visible. Live validation: PR #10 ran successfully against Nova and updated its marker-based review comment after a subsequent push.
nimmo closed this issue 2026-07-20 16:58:59 +01:00
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#6
No description provided.