name: inline-pr-comments description: 在特定行上发布内联PR审查评论。使用此技能以交付代码审查反馈作为内联评论,而不是单一摘要。
内联PR评论技能
使用此技能在PR中的特定行上发布代码审查反馈作为内联评论。
何时使用
- 完成代码审查后(与 /claude-review、/claude-security-review、/claude-tob-review 一起使用)
- 当您有具体的逐行反馈要交付时
- 为了使审查反馈更具可操作性
指令
使用MCP内联评论工具,而不是 gh api:
- 对于更改行上的每个问题,调用:
mcp__github_inline_comment__create_inline_comment
- 必需参数:
pathbodyline(单行),或startLine+line(多行)side: "RIGHT"除非有意评论旧代码
- 发布内联评论后,更新Claude粘性摘要评论:
mcp__github_comment__update_claude_comment
- 请 不要 在此处使用
gh api;操作工具权限通常阻止非git Bash命令。
评论字段(create_inline_comment)
path- 文件路径相对于仓库根目录line- 文件中新版本的行号(差异的右侧)startLine+line- 对于跨多行的评论side-RIGHT(新代码)或LEFT(旧代码),默认为RIGHTbody- Markdown格式的反馈
限制
- 只能评论出现在差异中的行(更改/添加的行)
- 在未更改行上的评论将失败,错误为“行无法解析”
处理非差异发现
当您发现代码中未被PR更改的问题时:
- 包含在摘要体中 - 始终在
"body"字段中报告 - 清晰格式化 - 使用专用部分“## 本PR之外的观察”
- 具可操作性 - 包括文件:行引用,以便作者跟进
- 不要阻止 - 这些是信息性的;不要对非差异问题使用
REQUEST_CHANGES
示例:更新粘性评论以包含非差异发现:
tool: mcp__github_comment__update_claude_comment
args:
body: |
## 审查摘要
[内联反馈摘要]
## 本PR之外的观察
在审查时,我注意到:
- `src/utils/foo.ts:142`:预先存在的空检查缺失
- `src/core/bar.ts:78-82`:与行45问题类似的模式 - 考虑去重
反馈指南
| 反馈类型 | 在差异中? | 放在哪里 |
|---|---|---|
| 具体代码问题 | ✅ 是 | 在该行的内联评论上 |
| 跨文件重复的模式 | ✅ 是 | 在首次出现处内联 + 备注“相同问题在X, Y, Z” |
| 相关发现的问题 | ❌ 否 | 摘要体下“本PR之外的观察” |
| 预先存在的bug发现 | ❌ 否 | 摘要体(如果关键,考虑单独问题) |
| 整体架构关注 | N/A | 摘要体 |
| 批准PR | N/A | 超出此技能范围;将反馈保留在粘性评论中 |
| 请求更改 | N/A | 粘性评论;永不使用 REQUEST_CHANGES |
简洁明了。将次要样式问题分组。