/product-feature-tech-implement

name: product-feature-tech-implement
description: 基于功能设计文档(由 product-feature-tech-design 产出或等价输入)、源码目录、以及测试所依赖的环境信息(例如 PostgreSQL 插件开发需要的 PG 源码/实例、第三方依赖),编排「开发者 ↔ 评审者 ↔ 测试者」三方隔离的实施循环,直至评审通过、且所有测试(新增 + 已有)通过。三方互不可见,测试者绝对不能看到开发者写的代码。完成后输出 markdown 总结报告到当前项目的 `markdown/` 目录。适用于:实施功能设计文档、按设计稿开发并自验证、闭环式开发流程、PostgreSQL 插件/扩展开发、需要 build + 部署 + 跑测的工程项目、TDD 式实施循环。

Product Feature Tech Implement

核心约束(决定一切)

1. **测试者看不到代码**。这是与 `product-feature-tech-design` 一脉相承的隔离原则 — 测试者只读设计文档,绝不能读开发者写的代码、commit message 中的代码 diff、或评审意见里贴的代码片段。 2. **循环 A 必须有终止条件**。评审通过才能退出循环 A;测试通过(且已有测试不退化)才能结束任务。 3. **三方各持独立 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 时,**必须**: 1. 加载对应的 references 模板作为 subagent 的角色契约 2. 在 prompt 开头**显式**告知 subagent:

• 它是什么角色
• 它能读什么、不能读什么
• 它的输入路径(设计文档、上轮报告等)
• 它的输出格式

3. 显式约束:不要让 subagent 之间互看对方的产出 4. **subagent 完成后,只把它的最终输出带回主对话**,不要把它的内部 reasoning 全量带回

详细 prompt 见:

• `references/developer-prompt.md`
• `references/reviewer-prompt.md`
• `references/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 结论、测试结论)
• 最终增量代码清单(文件级)
• 最终测试覆盖(新增用例数、已有用例回归结果)
• 遗留风险与后续建议
• 关键决策记录(为什么这么做)

质检清单(交付前自检)

• [ ] 测试者 subagent 启动时 prompt 中显式写了"绝对不能看代码目录"
• [ ] 每轮 review 报告有明确的 verdict(pass / fail + 原因)
• [ ] 每轮测试报告有明确的 verdict(pass / fail + 失败用例)
• [ ] 已有测试无退化(失败数 0 或与基线一致)
• [ ] 总结报告保存到 `markdown/` 目录
• [ ] 三个 subagent 已被关闭(没有遗留子任务)
• [ ] 流程中**没有**把开发者代码泄漏给测试者
• [ ] 防死循环保护(同问题 3 轮未解决)未触发 / 已与用户确认

参考资料

• `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 是交付终点