arinhub-submit-code-review
Original:🇺🇸 English
Translated
Submit a completed code review with line-specific comments and suggestions to a GitHub PR. Use when asked to "ah submit PR
1installs
Sourcearinhubcom/arinhub
Added on
NPX Install
npx skill4agent add arinhubcom/arinhub arinhub-submit-code-reviewTags
Translated version includes tags in frontmatterSKILL.md Content
View Translation Comparison →Submit Code Review
Submit a structured code review with line-specific comments to a GitHub pull request. Identifies issues in the current chat session or review file, checks for duplicate comments, and submits the review.
Input
- PR number or URL (required): The pull request identifier. Accepts:
- Number:
123 - Hash-prefixed:
#123 - Full URL:
https://github.com/owner/repo/pull/123
- Number:
- Review file path (optional): Path to a review file produced by (e.g.,
arinhub-code-reviewer). If provided, issues are extracted from this file instead of the current chat session.~/.agents/arinhub/code-reviews/pr-code-review-my-app-123.md
Procedure
1. Resolve PR Identifier
Extract the PR number from the user input. Strip any prefix or parse the number from a URL.
#PR_NUMBER=<extracted number>2. Fetch PR Metadata
Gather PR details:
bash
gh pr view $PR_NUMBER --json number,title,body,baseRefName,headRefName,files,url3. Fetch Existing Review Comments
Retrieve all existing review comments to prevent duplication:
bash
gh api repos/{owner}/{repo}/pulls/$PR_NUMBER/comments --paginate --jq '.[] | {id, path, line, body, user: .user.login}'Also fetch top-level review bodies:
bash
gh api repos/{owner}/{repo}/pulls/$PR_NUMBER/reviews --paginate --jq '.[] | {id, body, state, user: .user.login}'4. Get Issue List
Get a list of issues from one of these sources (in priority order):
- Review file: If a review file path is provided (e.g., from orchestrator), read the file and extract all issues from the
arinhub-code-reviewersection.## Issues - Current chat session: If no review file is specified, collect issues identified during the code review in the current chat session.
For each issue found, record:
- : The relative file path
path - : The specific line number in the new version of the file (must be within the diff hunk). For multi-line issues, this is the last line of the range.
line - (optional): The first line of a multi-line range. Only set when the issue spans more than one line.
start_line - : A concise, actionable comment explaining the issue
body - (optional): The replacement code that should replace the line(s) from
suggestion(orstart_line) throughline. Include this whenever you can propose a concrete fix. The suggestion content is the exact code that will replace the selected lines -- do not includelinefences here, they are added automatically in Step 7.```suggestion
5. Deduplicate Comments
For each issue identified in Step 4, compare against existing comments from Step 3:
- Skip if an existing comment on the same and
path(or nearby range +/- 3 lines) already addresses the same concernline - Skip if the issue is already mentioned in any top-level review body
- Use semantic comparison, not exact string matching -- if the existing comment covers the same problem, even with different wording, skip the new comment
6. Decision Gate
- If no new issues remain after deduplication: Do not submit a review. Inform the user that no new issues were found beyond existing review comments.
- If new issues exist: Proceed to Step 7.
7. Submit the Review
Submit a single review via the GitHub API. The review consists of one main comment () with individual inline comments () that appear as conversation threads anchored to specific lines in the diff.
bodycommentsPreflight validation checklist (run before submission):
- Validate JSON payload before sending (e.g., pipe the heredoc through to check syntax)
jq . >/dev/null - Ensure each comment has valid ,
path, andlineside: "RIGHT" - For multi-line comments, include and
start_linetogether withstart_side: "RIGHT"line - Confirm (and
linefor ranges) is inside the PR diff hunk for that filestart_line - If a suggestion is included, append it in using
bodyfences (do not pre-wrap suggestion content in fences earlier)```suggestion - Ensure suggestion replacement code preserves indentation and exact intended final content
- Use an empty suggestion block () only when the intent is to delete selected lines
```suggestion\n```
bash
gh api repos/{owner}/{repo}/pulls/$PR_NUMBER/reviews \
--method POST \
--input - <<'EOF'
{
"event": "APPROVE or COMMENT",
"body": "<main-review-comment>",
"comments": [
{
"path": "<file-path>",
"line": <line-number>,
"side": "RIGHT",
"body": "<thread-comment>\n\n```suggestion\n<replacement-code>\n```"
}
]
}
EOFFor multi-line suggestions, add and :
start_linestart_sidejson
{
"path": "<file-path>",
"start_line": <first-line>,
"line": <last-line>,
"start_side": "RIGHT",
"side": "RIGHT",
"body": "<thread-comment>\n\n```suggestion\n<replacement-code>\n```"
}If a comment has no suggestion (pure observation), omit the block from the body. Still include so that all comments are anchored to the new version of the file.
```suggestion ```"side": "RIGHT"Main review comment (): See main-review-comment.md for the full template and examples.
bodyThread comments (): See thread-comment.md for the full template and examples.
comments[].body8. Report Result
After submission, confirm to the user:
- Number of review comments submitted
- The PR URL for reference
- Brief list of issues flagged
If no review was submitted (Step 6), explain that no new issues were found beyond existing review comments.
9. Extract Requirements Coverage
Look for a Requirements Coverage section in the same source used in Step 4:
- Review file: If a review file was used, look for a section and extract its full content.
## Requirements Coverage - Current chat session: If no review file was used, look for any Requirements Coverage report or coverage summary produced during the current chat session.
If no Requirements Coverage is found, skip to the end -- this step is optional.
10. Post Requirements Coverage Comment
This step runs only if Requirements Coverage was found in Step 9. It must be the very last action -- execute it after all other steps (including the review submission and result report) are complete.
Post the coverage report as a standalone PR comment:
bash
gh pr comment $PR_NUMBER --body "$(cat <<'EOF'
<coverage-content>
EOF
)"- Use the Requirements Coverage content exactly as found -- do not modify, summarize, or reformat it
- This comment is independent of the review; post it even if no review was submitted in Step 6
- This must be the very last API call in the entire procedure to ensure the coverage comment appears at the bottom of the PR conversation
Important Notes
- Use when no High Priority issues are found, otherwise use
APPROVE. Never useCOMMENTunless the user explicitly asksREQUEST_CHANGES - The field in review comments must reference a line that appears in the diff -- comments on unchanged lines will be rejected by the API
line - For multi-line suggestions, use and
start_linetogether to define the range being replaced; both must be within the diff hunkline - An empty suggestion block () means "delete these lines"
```suggestion\n``` - The content inside replaces the selected line(s) verbatim -- ensure correct indentation and formatting
```suggestion ``` - Never fabricate issues -- only flag genuine concerns backed by evidence in the code