Address PR Feedback

Step 1 — Fetch Copilot feedback

./.claude/skills/address-pr-feedback/get-copilot-feedback.sh [pr-number]

If no PR number is given, it uses the current branch's PR.

Copilot reports findings in two different places, and the script reads both:

1. Inline review threads — the classic form. Hidden comments (resolved threads, minimized comments) are skipped automatically. Outdated threads are still shown, tagged [OUTDATED].
2. Review bodies — as of 2026-09 Copilot frequently posts Comments generated: 0 and puts its real findings in the review body instead, under ### Suppressed comments. These never appear as threads.

An empty "Inline review threads" section does NOT mean the review was clean. That is the trap this skill exists to avoid: a thread-only query returns nothing and the PR looks unreviewed when Copilot in fact filed a finding in the body. Always read both sections before concluding there is no feedback. If both are empty, check that Copilot has actually reviewed (gh pr view <n> --json reviews) rather than assuming it found nothing.

The script deliberately strips Copilot's per-file summary table — it restates the diff and carries no findings. It keeps the headline verdict, the one-line concern beneath it, and the suppressed-comments block.

Step 2 — Evaluate each comment

For each finding:

1. Verify — read the current file and confirm the issue actually exists. Comments may already be fixed by later commits. For [OUTDATED] ones this matters most: the diff position moved, but the underlying issue often still exists — find the corresponding spot and evaluate there. A body finding cites path:line from an older push; re-locate it by symbol, not line number.
2. Assess — is it technically correct for this codebase? Push back with reasoning if not. Copilot's verdict line ("Needs a closer look") is not itself a finding; the substance is in the concern sentence and the suppressed comments.
3. Fix — edit the file if the feedback is valid, and mutation-check any test you add.

Do not post replies to GitHub comment threads. Just fix the code.

Hidden = ignored

A comment is hidden only when its thread is resolved or minimized on GitHub. Those are skipped automatically and need no action. Outdated is not hidden — evaluate those normally.