review-pr-comments

Review PR Comments

Review all open PR comments, present a summary with your position on each, iterate with the user until agreed, then post responses individually to each comment.

Rules

• PR target: if no PR number is given, auto-detect from the current branch. If none exists, ask.

• Scope: only active comments (both inline and general). Ignore resolved and outdated ones.

• Bias: defend the existing code. Only accept a change with a concrete technical reason (bug, violated principle, demonstrable readability gain). Reject style preferences without objective justification.

• Context: before evaluating, read the full files affected plus related files (tests, types, direct dependencies).

• Summary format: after reviewing every comment, present all decisions grouped by severity, one compact block per comment. Do NOT quote the full body verbatim — the user has GitHub open and can click through. Rephrase the claim in one sentence in your own words.

  Extract severity from the comment's prefix when present ([SECURITY — CRITICAL], [CODE QUALITY — HIGH], [TESTS — LOW], [SUGGESTION], etc.). If no prefix, infer: correctness/security bugs → HIGH; performance, tests, or missing error paths → MEDIUM; style, docstrings, naming, "for symmetry", "for consistency" nits → LOW.

  Block format per comment:

  N. [ACCEPT|REJECT] {path}:{line or "general"} @{user} — {your one-sentence rephrase of the claim}
     Reason: {1-2 lines, technical}
     Fix: {concrete action you will take}          # only if ACCEPT
     Alternative: {what you propose instead}       # only if REJECT
     Link: {comment.html_url}
  

  Present them in this order:

  HIGH — decision required:
    <blocks>
  
  MEDIUM — decision required:
    <blocks>
  
  LOW / nits — proposed REJECT in batch (unless you rescue any):
    <blocks, but with just the one-line rephrase + link; no Reason/Alternative>
  

  Then ask ONE question: "Confirm my ACCEPT/REJECT on HIGH+MEDIUM and I discard the LOW batch — or do you want to rescue any LOW?"