Review Lessons 是 agent-skills 集合中一个按案例组织的经验教训库,来源于过去构建-审查循环(build-review loops)的积累。它本身不是一个执行阶段,而是供开发者在审查或修复轮次遇到特定情况时按需查阅的参考集合。
用途
该技能用于在代码审查或修复轮次中遇到特定案例时,快速定位对应的经验文件并应用。资料明确说明:找到下方对应案例,只读那一个文件,将其应用到眼前的发现上;不要预加载,因为大多数教训是针对具体项目的。
主要内容分类
资料将教训按案例分为多组,每组列出案例与对应文件:
- 裁决发现与修复轮次:包括审查者 BLOCKING 与派发规格或契约测试矛盾、延迟发现作为设计问题(references/review-findings-adjudication.md);错误层级修复、红测试分诊、双侧修复、实时验证(references/pitfalls-fix-round-verification.md);构建者自报告、修复引入的回归、构建中用户干预、简报竞争(references/loop-pitfalls.md);确认循环探针:精确复现形态、空测试、对修复做变异测试、竞争探针、内核状态(references/confirm-loop-lessons.md);双审查者并行及发现综合(references/parallel-dual-review.md、references/cross-model-parallel-review.md);审查者通道宕机、f-string 生成的 YAML frontmatter(references/generated-frontmatter-and-reviewer-outages.md);插件发布门禁与修复轮次契约(references/plugin-gates-and-fix-rounds.md)。
- 测试质量:隔离/范围/角色表面的性质测试、惰性测试、豁免路径(references/security-property-tests.md);进程内范围性质测试(references/scope-test-in-process-execution.md);功能有测试但无生产调用者(references/production-reachability-review.md);循环中架构变化时需要更新的测试(references/architecture-refactor-test-updates.md);升级路径测试(references/upgrade-path-test-pattern.md);交付用前后证据对(references/evidence-pairs.md)。
- 数据与存储:不变量、外键、幂等性、金额以 TEXT/Decimal 存储、迁移夹具(references/data-integrity-principles.md);SQLite 表重建迁移(references/sqlite-migration-pitfalls.md);SQLite WAL 迁移审查发现(references/sqlite-wal-review-findings.md);用单模块替换 N 个模块(调用者清单,references/full-module-replacement-scoping.md);对账逻辑(references/reconciliation-review-findings.md)。
- 表面与领域:REST API:路由器、授权、SQL 模式(references/rest-api-review-checklist.md);MCP 工具表面与设计审查(references/mcp-category-tool-implementation.md、references/mcp-design-review-patterns.md);应用与外部真源同步(Sheets、SaaS 镜像,references/sync-pipeline-review-patterns.md);Android APK / WebView 验证(references/mobile-app-verification.md);填充模板文档抽取(references/extraction-from-filled-docs.md);针对已发布压缩包的样式(references/minified-bundle-selector-research.md)。
- 机制与流程历史:CLI 标志、git 机制、契约测试机制、推送卫生(references/tool-mechanics-pitfalls.md);派发、轮询、并行构建、简报竞争(references/pitfalls-dispatch-semantics.md);CI 与仓库卫生(references/ci-and-repo-hygiene.md);按依赖与风险分批处理阶段(references/multi-batch-sequencing.md);发布加固型周期(references/beta-gate-release-hardening.md);整合操作者的现场简报(references/field-briefing-integration.md);端口说明:delegate-skills、planning-with-files(references/delegate-skills-adaptation.md、references/planning-with-files-adaptation.md)。
与协议的关系
资料说明:dev-pipeline 的流程(轮次预算、严重性下限和停止条件)位于 review-commit/references/confirm-round-protocol.md,模型名称位于 orchestrator/references/lanes.md。若此处教训与那些文件冲突,以那些文件为准:教训是带日期的证据,协议是当前规则。文件中的旧名称(如 "build-review-loop"、"codex-confirm-loop"、"subagent-driven-development")指今天的 review-commit 和 build 阶段;模型名称是当时运行的,不是当前固定的。带编号的引用("pitfall #N"、"rule #N")指向已退役的 v0.1 阶段 SKILL.md 编号,应视为历史上下文,而非需要阅读的指针。
添加教训
只有当某个循环教会了未来循环在没有它时会出错的内容时,才添加教训。每个案例一个文件;标题中包含日期和项目类型;在上方添加一行。如果教训改变了循环的运行方式(上限、停止规则、谁来审查),应改为修改 confirm-round-protocol.md,而不是作为教训归档,否则会成为第二份矛盾的副本。
使用方式与适用场景
该技能适用于开发者在代码审查、修复轮次、测试编写、数据迁移、API 设计等场景中遇到具体问题时按案例查阅。使用方式是定位案例、阅读对应文件、应用到当前问题。限制:多数教训是项目特定的,不应预加载;教训是历史证据,与当前协议冲突时以协议为准。