code-review

用途：审查已写好的代码，覆盖架构、代码质量、测试与性能四个维度；属于实现后审查，不做计划层审查（计划审查另有 /plan-review）。\n\n定位审查范围（Step 0）：按参数判断——文件路径则读取指定文件；PR 编号（如 #123）用 gh pr diff 123 获取变更；diff 时用 git diff 与 git diff --staged 获取所有未提交变更；空白默认同 diff；分支名用 git diff main...<branch> 获取差异。读取后先摘要变更范围并确认审查对象，再读取变更文件完整内容及被 import 或依赖的关键文件。随后用 AskUserQuestion 选择深度：深度审查每段最多 4 个议题，快速审查每段最多 1 个议题。\n\n四段审查（Step 1-4）：架构关注组件边界、依赖方向与循环依赖、数据流、安全（认证、授权、输入验证、API 边界）；代码质量关注 DRY 重复、错误处理、边界案例、命名可读性、过度工程；测试关注新增逻辑是否有对应测试、是否验证行为而非实现细节、边界与失败路径覆盖、可维护性；性能关注 N+1 查询、不必要的计算、内存与资源、缓存机会、算法复杂度。每段结束后用 AskUserQuestion 确认决策再进入下一段。\n\n议题呈现格式：每个议题含问题描述（指明 file:line 或 file:SymbolName）、2-3 个选项（含「不处理」，标明实现成本、风险、影响范围、维护负担）与建议及理由。\n\n审查摘要（Step 5）：以表格列出维度、议题、决策，并给出待修项清单；若所有议题都选择不处理，直接输出 LGTM 并说明原因。\n\n审查原则：积极标记重复、测试优先、工程适度、多想边界案例、明确优于聪明、只审查变更（除非变更暴露既有问题）。资料未说明该技能的安装方式、收费与运行环境，暂未提供。