review

Identify Target PR

Parse $ARGUMENTS to extract a PR number or URL. If not provided, detect from the current branch:

gh pr view --json number --jq '.number' 2>/dev/null

If no PR is found, ask the user to specify one.

Step 1: Collect Review Comments

Fetch all review comments on the PR (both human and bot/AI):

gh api repos/{owner}/{repo}/pulls/{number}/comments --jq '.[] | {id: .id, path: .path, line: .line, body: .body, user: .user.login, created_at: .created_at}'

Also fetch PR-level review summaries:

gh api repos/{owner}/{repo}/pulls/{number}/reviews --jq '.[] | {id: .id, user: .user.login, state: .state, body: .body}'

Also fetch issue-level comments (used by AI reviewers like CodeRabbit):

gh api repos/{owner}/{repo}/issues/{number}/comments --jq '.[] | {id: .id, body: .body, user: .user.login, html_url: .html_url, created_at: .created_at}'

Fetch review thread IDs for resolving conversations later (GraphQL):

gh api graphql -f query='
  query($owner: String!, $repo: String!, $number: Int!) {
    repository(owner: $owner, name: $repo) {
      pullRequest(number: $number) {
        reviewThreads(first: 100) {
          nodes {
            id
            isResolved
            comments(first: 1) {
              nodes { databaseId }
            }
          }
        }
      }
    }
  }
' -f owner='{owner}' -f repo='{repo}' -F number={number}

Map each databaseId (= REST comment ID) to its GraphQL threadId for the resolve step.

Include all reviewers — human teammates and AI bots alike. Do NOT filter by user type. Track each comment's source type (review comment vs. issue comment) for the reply and resolve steps.

Step 2: Analyze Each Suggestion

For each review comment:

1. Read the referenced file and surrounding code context (at least 20 lines around the mentioned line).
2. Understand the reviewer's suggestion in the context of the actual code.
3. Evaluate the suggestion's validity:

Category	Criteria
Valid	Real bug, security issue, or meaningful improvement
Debatable	Stylistic preference or trade-off that could go either way
Can Safely Ignore	Misunderstands the code, already handled elsewhere, or not applicable

Step 3: Present Findings

Group findings by category and present to the user:

## Review Analysis for PR #{number}

### ✅ Valid Suggestions (recommended to apply)

1. **[file.py:42]** {summary}
   - Reviewer said: {brief quote}
   - Assessment: {why this is valid, what the fix should be}

### 🤔 Debatable

1. **[file.py:78]** {summary}
   - Reviewer said: {brief quote}
   - Pros: {benefits of applying}
   - Cons: {reasons to skip}

### ❌ Can Safely Ignore

1. **[file.py:15]** {summary}
   - Reviewer said: {brief quote}
   - Why: {reason this is not applicable — e.g., misunderstood context, project convention differs}

Step 4: Discuss and Apply

For each Valid or Debatable suggestion:

1. Ask the user whether to apply the change.
2. If approved, make the code change using Edit.
3. For items NOT applied, determine the disposition:
   • Won't Fix: Not applicable, intentionally not addressing, or disagree with the suggestion. Record reason in reply only.
   • Follow-up: Valid but out of scope for this PR. Will be tracked as an issue.
4. After all approved changes are applied, suggest a commit message following the project's commit convention (e.g., fix: 리뷰 피드백 반영).

Step 5: Reply to Review Comments

After the commit is created, reply to each review comment on GitHub to record the resolution.

For each comment that was discussed (Valid, Debatable, or Ignored), reply using the appropriate API based on the comment's source type:

For review comments (inline file comments):

gh api repos/{owner}/{repo}/pulls/{number}/comments/{comment_id}/replies -f body="{reply_body}"

For issue comments (CodeRabbit, etc.):

gh api repos/{owner}/{repo}/issues/{number}/comments -f body="{reply_body}"

For issue comments only, prepend a quote line before the structured format: > Re: @{user} [{comment_url}] This quote line is part of the format for issue comments — it does not apply to review comments.

Reply format

Replies MUST follow the exact structured format below. Do NOT use free-form prose or unstructured sentences.

Write the reply body in the language configured in the project's CLAUDE.md. If no language is configured, follow the user's conversational language. Examples below are in Korean:

If applied:

✅ **반영 완료**

- **커밋**: [{short_sha}](https://github.com/{owner}/{repo}/commit/{short_sha})
- **변경**: {1-line summary of what was changed}

If Won't Fix:

⏭️ **미반영 (Won't Fix)**

- **사유**: {concise reason — e.g., 프로젝트 컨벤션과 상충, 이미 다른 방식으로 처리됨}

If Follow-up:

🔜 **후속 작업 예정**

- **사유**: {concise reason — e.g., 현재 PR 범위 밖, 별도 작업으로 진행 예정}
- **이슈**: #{issue_number}

If no blanket approval was given, present all planned replies for approval before posting. A blanket instruction (e.g., "모두 반영", "다 적용해", "apply all") covers all remaining steps — code changes, replies, and thread resolution — so do not re-ask per step.

Resolve concluded threads

After posting replies, resolve the conversation thread using the GraphQL API:

gh api graphql -f query='mutation($id: ID!) { resolveReviewThread(input: {threadId: $id}) { thread { isResolved } } }' -f id='{thread_id}'

Use the databaseId → threadId mapping collected in Step 1.

Resolve rules

Disposition	Resolve?	Reason
✅ Applied	Yes	Work is complete
⏭️ Won't Fix	Yes	Decision is final, no further action
🔜 Follow-up	No	Open work remains — issue tracks it

Resolve MUST happen immediately after posting the reply for each concluded thread. Do NOT skip this step. Issue comments (CodeRabbit, etc.) do not have resolvable threads — skip the resolve step for those.

Create follow-up issue for deferred items

If there are Follow-up items from Step 4, group related ones and create a consolidated issue. Do NOT create issues for Won't Fix items — their reasons are already recorded in the PR reply.

gh issue create --title "[Review] PR #{number} 후속 작업" --body "$(cat <<'EOF'
PR #{number} 리뷰에서 확인된 후속 작업 항목.

### 1. {summary}
- 원본: {comment_url}
- 사유: {why deferred}

### 2. {summary}
- 원본: {comment_url}
- 사유: {why deferred}
EOF
)"

• Group by relevance — do NOT create one issue per comment.
• If all follow-up items are closely related, create a single issue.
• If there are clearly distinct groups, create one issue per group (max 2-3).
• Skip issue creation if there are no follow-up items.

Important:

• Do NOT apply any code changes without explicit user approval for each item.
• Do NOT post reply comments without explicit user approval.
• A blanket instruction (e.g., "모두 반영해줘", "apply all") counts as explicit approval for all steps. Do not ask again per-step.
• Read the actual code context before judging — do not rely solely on the review comment.
• Consider the project's existing patterns, conventions, and CLAUDE.md instructions.
• Be honest when a review catches a genuine issue — do not dismiss valid feedback.
• If no review comments are found, inform the user.