spec-code-review
用多 persona、confidence gate、合成去重和 safe_auto 修复做结构化代码评审。
ASCII 链路图
text
Codebase -> Spec -> Plan -> Tasks -> Code -> Review -> Knowledge
|
v
spec-code-review
entry: spec-code-review
type: workflow command基本信息
| 字段 | 内容 |
|---|---|
| Skill | spec-code-review |
| 类型 | workflow command |
| 研发链路位置 | Review |
| 当前入口 | spec-code-review |
| 兼容别名 | /spec:code-review、$spec-code-review |
| Source | skills/spec-code-review/SKILL.md |
| Governance | src/cli/contracts/dual-host-governance/skills-governance.json(supported-host skills governance) |
描述
Structured code review for bugs, regressions, tests, and standards. Report-only by default; apply fixes only when the current user or upstream caller explicitly requests review-and-fix. mode:agent is always report-only.
Source 契约摘要
该 source skill 未声明标准 Workflow Contract Summary 小节;本页以 frontmatter、supported-host governance 和维护者补充元数据为准。
何时使用
- 提交 PR 前。
- 完成实现后需要质量门。
- 需要围绕 diff 查逻辑、测试、安全、性能、契约和栈专项问题。
工作步骤
- 读取 diff、PR、文档、计划、task pack 或专项审查输入。
- 按风险域选择 always-on 与 conditional reviewer personas。
- 合并、去重和校准 findings,区分 safe_auto、gated_auto、manual 和 advisory。
- 输出修复、残留风险或下一步处理建议。
输出
- review findings、safe_auto fixes、residual risks、audit report 或 release checklist。
- 当 reviewer dispatch 不可用时,输出明确的 report-only fallback 覆盖说明。
Agent / Persona 列表
- 这是 persona agents 的主要使用层:correctness、testing、security、performance、scope、design 等按证据触发。
- review workflow 自身负责任务边界、合成和最终判断。
本 skill 在 source 中挂载了 17 个 skill-local 专家资产(agent 0 · persona 17),路径为 skills/spec-code-review/references/{agents,personas}/。跨 skill 复用时,详情页按文件名去重。
| 名称 | 类型 | 摘要 |
|---|---|---|
spec-adversarial-reviewer | persona | You are a chaos engineer who reads code by trying to break it. |
spec-agent-native-reviewer | persona | You review code to ensure agents are first-class citizens with the same capabilities as users -- not bolt-on features. |
spec-api-contract-reviewer | persona | You are an API design and contract stability expert who evaluates changes through the lens of every consumer that depends on the current interface. |
spec-correctness-reviewer | persona | You are a logic and behavioral correctness expert who reads code by mentally executing it -- tracing inputs through branches, tracking state across calls, an... |
spec-data-migration-reviewer | persona | You are a data migration and schema-change reviewer. |
spec-deployment-verification-agent | persona | You are a Deployment Verification Agent. |
spec-frontend-quality-reviewer | persona | 你审查当前 diff 中用户可见的 Web 交互质量。关注用户能否理解状态、用键盘完成任务、在窄屏和不同视觉条件下继续操作,以及 presentation/data 边界是否让这些行为可维护。只报告当前 source/diff 可以支持的缺陷;不把 visual polish、browser field outc... |
spec-julik-frontend-races-reviewer | persona | You are Julik, a seasoned full-stack developer reviewing frontend code through the lens of timing, cleanup, and UI feel. |
spec-learnings-researcher | persona | You are a domain-agnostic institutional knowledge researcher. |
spec-maintainability-reviewer | persona | You are a structural code-quality reviewer. |
spec-performance-reviewer | persona | You are a runtime performance and scalability expert who reads code through the lens of "what happens when this runs 10,000 times" or "what happens when this... |
spec-previous-comments-reviewer | persona | You verify that prior review feedback on this PR has been addressed. |
spec-project-standards-reviewer | persona | You audit code changes against the project's own standards files -- CLAUDE.md, AGENTS.md, and any directory-scoped equivalents. |
spec-reliability-reviewer | persona | You are a production reliability and failure mode expert who reads code by asking "what happens when this dependency is down?" You think about partial failur... |
spec-security-reviewer | persona | You are an application security expert who thinks like an attacker looking for the one exploitable path through the code. |
spec-swift-ios-reviewer | persona | You are a senior iOS engineer who has shipped production SwiftUI and UIKit apps at scale. |
spec-testing-reviewer | persona | You are a test architecture and coverage expert who evaluates whether the tests in a diff actually prove the code works -- not just that they exist. |
agent 详情页入口仍在
/reference/agents/*;完整按 skill 归属索引见 Agents 参考。persona 目前以 skill-local prompt 形式挂在所属 skill 下,不单独生成全局详情页。
边界
- report-only 模式不编辑。
- safe_auto 之外的问题需要用户或下游 resolver 决策。
源码校准
- 本页由
website/scripts/generate-skill-detail-pages.js从上游 source skill 生成。 - 当前入口、兼容别名与宿主投递 metadata 来自
src/cli/contracts/dual-host-governance/skills-governance.json。 - 当 source skill 或 governance contract 变化后,应重新运行
npm run reference:pages并执行npm run content:audit。
