| name | product-feature-tech-implement |
| description | 基于功能设计文档(由 product-feature-tech-design 产出或等价输入)、源码目录、以及测试所依赖的环境信息(例如 PostgreSQL 插件开发需要的 PG 源码/实例、第三方依赖),编排「开发者 ↔ 评审者 ↔ 测试者」三方隔离的实施循环,直至评审通过、且所有测试(新增 + 已有)通过。三方互不可见,测试者绝对不能看到开发者写的代码。完成后输出 markdown 总结报告到当前项目的 `markdown/` 目录。适用于:实施功能设计文档、按设计稿开发并自验证、闭环式开发流程、PostgreSQL 插件/扩展开发、需要 build + 部署 + 跑测的工程项目、TDD 式实施循环。 |
Product Feature Tech Implement
核心约束(决定一切)
- 测试者看不到代码。这是与
product-feature-tech-design 一脉相承的隔离原则 — 测试者只读设计文档,绝不能读开发者写的代码、commit message 中的代码 diff、或评审意见里贴的代码片段。
- 循环 A 必须有终止条件。评审通过才能退出循环 A;测试通过(且已有测试不退化)才能结束任务。
- 三方各持独立 prompt。下方 references 里固化的 subagent prompt 是隔离的物理边界,不要在主对话里临场改写。
输入
| 输入 | 必填 | 用途 |
|---|
| 功能设计文档路径 | 是 | 所有角色的唯一共同依据 |
| 源码目录路径 | 是 | 开发者写入、评审者读取、测试者不读 |
| 已有测试目录 | 是 | 测试者运行回归 |
| 测试环境信息 | 否(按需) | 例如:PG 源码路径、PG 实例初始化命令、插件加载方式、构建系统(Make/CMake/Cargo/...) |
| 增量代码范围 | 是(从第 2 轮起) | 仅给评审者,标识本次新增/修改的文件 |
| 上轮评审意见 | 否(从第 2 轮起) | 给开发者修复用 |
| 上轮测试报告 | 否(从第 2 轮起) | 给开发者修复用 |
如果输入不全,先停下来问用户补齐,不要凭空白开始。
角色契约
角色 1:开发者(Subagent 1)
| 项 | 约束 |
|---|
| 输入 | 设计文档 + 上轮 review 意见(若有) + 上轮测试报告(若有) + 源码目录 |
| 输出 | 增量代码(diff 或文件列表)+ 开发者自检说明 |
| 可以看 | 设计文档全部、源码目录全部、已有测试目录的文件名(不是内容)、build/部署/运行日志 |
| 不可以看 | 测试者写的测试代码 |
| 隔离实现 | 由 references/developer-prompt.md 物理屏蔽 |
角色 2:评审者(Subagent 2)
| 项 | 约束 |
|---|
| 输入 | 设计文档 + 增量代码(本轮新增/修改的文件)+ 上一轮 review 报告(若有) |
| 输出 | review 报告(通过 / 不通过 + 问题清单 + 严重度) |
| 可以看 | 设计文档全部、增量代码 |
| 不可以看 | 测试代码、上轮测试报告(避免被「能跑通」污染 review 视角) |
| 通过条件 | 所有 P0 问题已修复,无新增 P0/P1 问题 |
角色 3:测试者(Subagent 3)
| 项 | 约束 |
|---|
| 输入 | 仅功能设计文档 + 已有测试目录路径(运行用,不是阅读用) + 测试环境信息 |
| 输出 | 测试报告(新增测试用例 + 执行结果 + 已有测试执行结果) |
| 可以看 | 设计文档全部、运行 build/部署/跑测的命令、命令输出 |
| 绝对不可以看 | 开发者写的源码、commit message 中的 diff、review 报告里贴的代码片段 |
| 隔离实现 | 由 references/tester-prompt.md 强红线 |
编排流程
[读取输入]
↓
loop A:
① 启动 subagent 1(开发者)
- 首轮:基于设计文档实现
- 后续轮:基于设计文档 + review 意见 + 测试报告修复
② 启动 subagent 2(评审者)
- 基于设计文档 + 本轮增量代码 review
③ review 通过?
- 否 → 回到 loop A 开始(开发者按 review 修)
- 是 → 退出 loop A
④ 启动 subagent 3(测试者)
- 基于设计文档写新增测试 + 跑全量测试
⑤ 测试通过?
- 否 → 回到 loop A 开始(开发者按 review + 测试报告修)
- 是 → 退出整体流程
⑥ 关闭三个 subagent
⑦ 生成总结报告
循环退出判定
loop A 退出(进入测试):
- review 报告明确写
verdict: pass
- 没有任何 P0/P1 残留
整体流程退出:
- review pass 且 测试报告
verdict: pass
- 测试报告必须包含:新增测试用例数、通过/失败/跳过统计、已有测试全部通过(无退化)
- 性能/安全/兼容性的"必测项"覆盖
防死循环:
- 同一 review 意见反复未修复 3 轮 → 暂停,询问用户是否调整设计文档
- 同一测试用例反复失败 3 轮 → 暂停,询问用户是设计问题还是实现问题
subagent 启动规范
每次启动 subagent 时,必须:
- 加载对应的 references 模板作为 subagent 的角色契约
- 在 prompt 开头显式告知 subagent:
- 它是什么角色
- 它能读什么、不能读什么
- 它的输入路径(设计文档、上轮报告等)
- 它的输出格式
- 显式约束:不要让 subagent 之间互看对方的产出
- subagent 完成后,只把它的最终输出带回主对话,不要把它的内部 reasoning 全量带回
详细 prompt 见:
references/developer-prompt.mdreferences/reviewer-prompt.mdreferences/tester-prompt.md
报告文件位置
| 报告 | 路径 | 说明 |
|---|
| Review 报告 | markdown/<topic>-review-<round>-<YYYYMMDD>.md | 每轮 1 份 |
| 测试报告 | markdown/<topic>-test-<round>-<YYYYMMDD>.md | 每轮 1 份 |
| 总结报告 | markdown/<topic>-implement-summary-<YYYYMMDD>.md | 最终 1 份 |
<topic> 与功能设计文档保持一致。
总结报告(见 references/summary-template.md)
至少包含:
- 任务概览(设计文档 + 源码目录 + 测试环境)
- 实施时间线(每轮做了什么、review 结论、测试结论)
- 最终增量代码清单(文件级)
- 最终测试覆盖(新增用例数、已有用例回归结果)
- 遗留风险与后续建议
- 关键决策记录(为什么这么做)
质检清单(交付前自检)
参考资料
references/developer-prompt.md — 开发者 subagent 角色契约references/reviewer-prompt.md — 评审者 subagent 角色契约references/tester-prompt.md — 测试者 subagent 角色契约(强隔离)references/summary-template.md — 总结报告模板
上下游 skill
- 上游:
product-feature-tech-design — 输入是它的产物(功能设计文档)
- 本 skill — 实施 + review + test 闭环
- 下游:无需,本 skill 是交付终点
