feedback-resolutionlisted

Address PR review feedback through surgical fixes traceable to specific comments, apply the Boy Scout Rule only to already-modified files (separate `improve:` commits), recover context by code snippet rather than line number, and enforce pushback only when factually incorrect, test-breaking, or CLAUDE.md-violating. Use when resolving reviewer comments on a pull request. This skill MUST be consulted because every untraceable change is out-of-context, and pushback without evidence is just disagreement.
synaptiai/synapti-marketplace · ★ 6 · AI & Automation · score 77

Install: claude install-skill synaptiai/synapti-marketplace

# Feedback Resolution ## Contract Iron law: **every change traces to a specific review comment or passes the Boy Scout proximity test; anything else is out-of-context and does not belong in this round.** Invoked by `/flow:address` Phase 1 (collect and categorize feedback) through Phase 4 (verify, post the resolution comment, re-request review). Returns one commit per feedback item (`fix(scope): address review — {summary}`), separate `improve:` commits for Boy Scout fixes, a response for every Question or Pushback item, and the resolution comment from `templates/resolution-comment.md` posted before review is re-requested. Permitted skips: none — pushback replaces a fix only with `file:line` evidence, a named breaking test, or a quoted CLAUDE.md rule. ## Collect ```bash PR_NUM=$ARGUMENTS REPO=$(gh repo view --json nameWithOwner --jq '.nameWithOwner') gh api repos/$REPO/pulls/$PR_NUM/comments --jq '.[] | {id: .id, path: .path, line: .line, body: .body, author: .user.login}' gh pr view $PR_NUM --json reviews --jq '.reviews[] | {state: .state, body: .body, author: .author.login}' ``` ## Categorize | Category | Meaning | |----------|---------| | **P1 — Must fix** | Code bug, security issue, logic error | | **P2 — Should fix** | Convention violation, missing test, unclear code | | **P3 — Consider** | Style preference, optimization suggestion | | **Question** | Needs a response, not necessarily a code change | | **Resolved** | Already fixed or no longer relevant | ## Fix Each