surshar / claude-code
Claude Code is an agentic coding tool that lives in your terminal, understands your codebase, and helps you code faster by executing routine tasks, explaining complex code, and handling git workflows - all through natural language commands.
Built by Anthropic · github.com/anthropics
Add inline comments with suggestions to code-review plugin
| @@ -1,5 +1,5 @@ | ||
| 1 | 1 | --- |
| 2 | -allowed-tools: Bash(gh issue view:*), Bash(gh search:*), Bash(gh issue list:*), Bash(gh pr comment:*), Bash(gh pr diff:*), Bash(gh pr view:*), Bash(gh pr list:*) | |
| 2 | +allowed-tools: Bash(gh issue view:*), Bash(gh search:*), Bash(gh issue list:*), Bash(gh pr comment:*), Bash(gh pr diff:*), Bash(gh pr view:*), Bash(gh pr list:*), mcp__github_inline_comment__create_inline_comment | |
| 3 | 3 | description: Code review a pull request |
| 4 | 4 | --- |
| 5 | 5 | |
| @@ -11,7 +11,7 @@ | ||
| 11 | 11 | - The pull request is closed |
| 12 | 12 | - The pull request is a draft |
| 13 | 13 | - The pull request does not need code review (e.g. automated PR, trivial change that is obviously correct) |
| 14 | - - You have already submitted a code review on this pull request | |
| 14 | + - Claude has already commented on this PR (check `gh pr view <PR> --comments` for comments left by claude) | |
| 15 | 15 | |
| 16 | 16 | If any condition is true, stop and do not proceed. |
| 17 | 17 | |
| @@ -52,15 +52,34 @@ | ||
| 52 | 52 | |
| 53 | 53 | 6. Filter out any issues that were not validated in step 5. This step will give us our list of high signal issues for our review. |
| 54 | 54 | |
| 55 | -7. Finally, output the review. | |
| 56 | - - If the `--comment` argument is provided, post the review as a comment on the pull request using `gh pr comment` | |
| 57 | - - Otherwise (default), output the review directly to the terminal for local viewing | |
| 55 | +7. Post summary comment FIRST using `gh pr comment` (if `--comment` argument is provided): | |
| 56 | + - Total number of issues found | |
| 57 | + - Brief one-line summary of each issue (no "Bug:" prefix) | |
| 58 | + - Or if no issues: "No issues found. Checked for bugs and CLAUDE.md compliance." | |
| 59 | + | |
| 58 | 60 | When writing your comment, follow these guidelines: |
| 59 | 61 | a. Keep your output brief |
| 60 | 62 | b. Avoid emojis |
| 61 | 63 | c. Link and cite relevant code, files, and URLs for each issue |
| 62 | 64 | d. When citing CLAUDE.md violations, you MUST quote the exact text from CLAUDE.md that is being violated (e.g., CLAUDE.md says: "Use snake_case for variable names") |
| 63 | 65 | |
| 66 | +8. THEN post inline comments for each issue using `mcp__github_inline_comment__create_inline_comment`: | |
| 67 | + - `path`: the file path | |
| 68 | + - `line` (and `startLine` for ranges): select the buggy lines so the user sees them | |
| 69 | + - `body`: Brief description of the issue (no "Bug:" prefix). For small fixes (up to 5 lines changed), include a committable suggestion: | |
| 70 | + ```suggestion | |
| 71 | + corrected code here | |
| 72 | + ``` | |
| 73 | + For larger fixes (6+ lines or structural changes), do NOT use suggestion blocks. Instead: | |
| 74 | + 1. Describe what the issue is | |
| 75 | + 2. Explain the suggested fix at a high level | |
| 76 | + 3. Include a copyable prompt for Claude Code that the user can use to fix the issue, formatted as: | |
| 77 | + ``` | |
| 78 | + Fix [file:line]: [brief description of issue and suggested fix] | |
| 79 | + ``` | |
| 80 | + | |
| 81 | + **IMPORTANT: Only post ONE comment per unique issue. Do not post duplicate comments.** | |
| 82 | + | |
| 64 | 83 | Use this list when evaluating issues in Steps 4 and 5 (these are false positives, do NOT flag): |
| 65 | 84 | |
| 66 | 85 | - Pre-existing issues |