Inline PR Comments Skill
Use this skill to post code review feedback as inline comments on specific lines in a PR.
When to Use
- •After completing a code review (use with /claude-review)
- •When you have specific line-by-line feedback to deliver
- •To make review feedback more actionable
Instructions
Post a review with inline comments using gh api:
bash
gh api repos/{owner}/{repo}/pulls/{pr_number}/reviews --input - << 'EOF'
{
"event": "COMMENT",
"body": "Optional summary of overall findings",
"comments": [
{
"path": "path/to/file.mdx",
"line": 42,
"body": "Issue description and suggested fix"
},
{
"path": "another/file.mdx",
"start_line": 10,
"line": 15,
"body": "Multi-line comment spanning lines 10-15"
}
]
}
EOF
Comment Fields
- •
path- File path relative to repo root - •
line- Line number in the NEW version of the file (right side of diff) - •
start_line+line- For comments spanning multiple lines - •
body- Markdown-formatted feedback
Limitations
- •Can only comment on lines that appear in the diff (changed/added lines)
- •Comments on unchanged lines will fail with "Line could not be resolved"
Handling Non-Diff Findings
When you discover issues in code NOT changed by the PR:
- •Include in summary body - Always report in the
"body"field - •Format clearly - Use a dedicated section "## Observations Outside This PR"
- •Be actionable - Include file:line references so author can follow up
- •Don't block - These are informational; don't use
REQUEST_CHANGESfor non-diff issues
Example structure:
json
{
"event": "COMMENT",
"body": "## Review Summary\n[inline feedback summary]\n\n## Observations Outside This PR\nWhile reviewing, I noticed:\n- `docs/guides/foo.mdx:142`: Outdated link\n- `docs/reference/bar.mdx:78-82`: Similar issue - consider updating",
"comments": [
// Only lines IN the diff
]
}
Feedback Guidelines
| Feedback Type | In Diff? | Where to Put It |
|---|---|---|
| Specific code issue | Yes | Inline comment on that line |
| Pattern repeated across files | Yes | Inline on first occurrence + note "same issue in X, Y, Z" |
| Related issue found | No | Summary body under "Observations Outside This PR" |
| Pre-existing bug discovered | No | Summary body (consider separate issue if critical) |
| Overall architecture concern | N/A | Summary body |
| Approval/changes requested | N/A | Use event: "APPROVE" or event: "REQUEST_CHANGES" |
Be concise. Group minor style issues together.