.agents/skills/code-changes/references/pr-review-comments.md
Entry point when the task is "handle the review comments on PR #n". This replaces Phase 1's issue analysis; Phases 2–6 apply unchanged, including the stop gate, the merged-state Verify run, and Deliver's push policy — this special case does not opt out of any of them.
For each valid comment, the classification below (valid/invalid, with the code evidence) stands in
for the full analysis-report artifact from artifacts.md: root_cause /
proposed_change is the valid-comment list with its code evidence, out_of_scope is the invalid
comments (named explicitly, not silently dropped), repro_status is the code-path confirmation
used to classify each comment, and open_questions is empty once every thread is classified. Plan
still pins a spec from this and the stop gate still applies before any fixup is written, even when
the fix itself is small.
gh pr view <n> --comments, or the review threads via
gh api).git commit --fixup <sha> on top of the branch. The fixup commit is the working state; it is
not yet the delivered change.git rebase --autosquash to fold each fixup into the commit it targets.git diff between old and new tip). A fixup that changes anything else went to the wrong
commit.--force-with-lease when pushing a rewritten branch. Handling PR review
comments does not by itself imply a push — confirm the user wants the branch updated remotely.Each thread gets a reply describing what was done and how it was verified. Leave resolving the threads to the humans.