code-review-mastery 是一个用于本地 Git 代码审查的技能,仅在用户明确调用或发送该技能时启用,不会根据上下文或关键词主动触发。
用途:该技能用于审查用户本地的 Git 更改,包括已暂存(staged)和未暂存(unstaged)的差异,适用于提交前自检或准备发起 PR 前的代码审查场景。
主要能力:
- 检测变更:运行 git diff --stat 和 git diff --cached --stat,判断存在哪类变更;若两者都有更改,会询问用户审查哪一组(staged / unstaged / both);若无更改则提示“No local changes to review.”并停止。
- 收集项目上下文:读取 CLAUDE.md、AGENT.md、README.md 等说明文件,读取 ESLint、Prettier、Ruff、tsconfig 等 lint 与格式配置,并抽样查看变更文件同目录下的现有文件,以识别命名、导入和错误处理等约定。上下文仅用于校准审查,不直接输出给用户。
- 多轮分析:按安全、正确性、性能、设计、可读性、约定、测试等层面检查差异内容,对超过 500 行的差异会提示用户并建议仅聚焦 [MAJOR] 问题,或逐文件处理。
- 结构化报告:所有发现分为 [MAJOR](必须修复,如 bug、安全漏洞、数据丢失风险、关键路径缺少错误处理、违反明确项目规则、新行为缺少测试)和 [MINOR](建议改进,如命名、可读性、轻微性能、风格不一致、文档缺失等)。输出格式包含文件与行号、类别标签、一行描述和一行建议修复方式,以复选框形式呈现,便于逐项跟踪。
- 交互式修复:报告结束后会询问用户是否需要修复某些条目,可回复具体条目或“fix all MAJOR” / “fix all”。
使用方式:用户需明确调用该技能,技能首次响应会以 🧢 emoji 开头。随后按 DETECT、CONTEXT、ANALYZE、REPORT 四个阶段依次执行。审查仅针对差异中出现的代码行,不扩展到未变更文件;会跳过二进制文件、锁文件、生成文件等,并注明跳过项;遇到合并冲突会标记为 [MAJOR] 并建议先解决冲突。
适用场景:提交前对本地更改做一次结构化自查;团队 PR 前进行预审查;需要按影响优先级排列问题并逐项修复的场景。
限制与注意事项:该技能不会主动触发,必须由用户明确调用;不编造项目规则,只有在发现明确配置或规则时才将约定类问题标为 [MAJOR];不会重复报告 linter 已能捕获的问题;避免严重性膨胀,存疑时降级为 [MINOR];对超过 500 行的差异默认建议仅关注 [MAJOR] 问题。详细的分项检查清单与上下文检测指南分别位于 references/review-checklist.md 和 references/context-detection.md,但具体文件内容暂未提供。