← ClaudeAtlas

review-roundlisted

Work a PR review round in this repo - reading bot findings with the head SHA, deciding what to act on and where the reason for a decline goes, spotting findings that accumulate on one seam, and mutation-verifying a new rule against the mutation it is for. Use when handling review comments on a windbg-mcp or dbgscope PR, or before calling a review done.
glslang/windbg-mcp · ★ 9 · Code & Development · score 74
Install: claude install-skill glslang/windbg-mcp
# Working a review round **Both review bots comment per commit**, and a round of findings can land *after* a reply to the previous round. Before calling a review done, re-check with the head SHA: `gh api --paginate repos/<owner>/<repo>/pulls/<n>/comments --jq '.[] | select(.original_commit_id=="<sha>")'` — with `--paginate`, since a busy PR's comments span pages and the first page is exactly where the older rounds are. **They also circle the same topic, and contradict each other and themselves across rounds.** A bot reviews *this diff* without the argument that produced it, so the same seam comes back round after round from a different angle — and a finding framed as "fresh evidence relative to the prior comment" may be the same claim, or may be genuinely new. Three shapes seen across the four PRs behind `FOLLOWUPS.md` item 34 (#189 to #192), all from the same reviewer: - **Against code that no longer exists.** One round argued about a teardown task the *previous* commit had deleted. Check which commit a comment is anchored to before acting on it. - **Round-tripping a decision.** Successive rounds drove a check out of `Lease::admit` and then asked for it back. Both were right about different properties, and only reading the code settled which — the review text alone could not. - **Right about the fact, wrong about the remedy.** "The SCM will not deliver a control code to a `StartPending` service" was correct (`ERROR_SERVICE_CANNOT_ACCEPT_CTRL` — measured, by holdin