质量闸门与报告
跑真实 check → 对照质量目标 → 出可核对的报告 → 人签字放行。
这个命令的立身之本
一切判定来自客观数据:覆盖率数字来自覆盖率工具、E2E 覆盖来自 flow→test 映射 + 真跑结果、lint 来自退出码。 AI 只负责把这些数据整理成报告,不参与"达标与否"的判定,放行由人。
⛔ 绝不允许出现的行为:用"我看了一下代码,测试挺全的"这类判断替代真实数据; 用上一次的结果冒充本次;命令没跑通却按通过处理;覆盖率没测量却写一个数字。 量不到就如实写"未测量"——这与状态机里「无命令可跑 →
checks: {}」是同一条纪律。
前置:两份真源
| 文件 | 作用 | 缺失时 |
|---|---|---|
docs/00_standards/test-commands.yml |
怎么量(命令) | 中止,提示先跑 /pdlc-test-setup |
docs/00_standards/quality-targets.yml |
量到多少算达标 | 走 --init 交互创建(见下) |
docs/00_standards/e2e-flow-map.yml |
核心流 → E2E 测试 的映射 | 若 targets 里声明了 core_flows 则必须有,否则 E2E 判定无法机械化 |
--init:首次建立目标声明
- 读
templates/quality-targets-template.yml作骨架。 - 从 PRD 自动抽
core_flows草稿:扫docs/01_requirements/prd/,提取标记为 P0 / P1 的功能流程, 生成候选清单(含来源 PRD 路径)供人确认——降低首次声明的摩擦,但最终清单必须人确认,不自动落盘。 - 覆盖率达标线:默认与
test-commands.yml的 coverage 命令参数对齐;两者不一致要提示人对齐 (以命令参数为准——那才是真正的强制点)。 - 同时生成
e2e-flow-map.yml骨架(每条 flow 一个空tests列表待填)。
段一:跑真实 check
按 test-commands.yml 逐条真跑 coverage / e2e / lint,记录命令原文 + 退出码 + 关键输出。
退出码的三态语义、以及「命令跑不了 = yml 过期信号」按下面的规则处理:
补充两条本命令特有的:
- 命令为空字符串(项目未配置该项)→ 如实记为「留空,未测量」,不得因此判为通过
- 覆盖率数字从工具输出中摘取原文,不重新计算、不四舍五入到好看的数
段二:三项机械核对
2.1 覆盖率
拿实测数字对 quality-targets.yml 的达标线。真正的强制点是命令参数里的阈值(如 --cov-fail-under=85)——
退出码就是判定;yml 里的数字用于报告展示与趋势。两处不一致 → 报告里提示对齐。
2.2 E2E 覆盖矩阵(B2 的第一个地基)
对每条 core_flow,按 e2e-flow-map.yml 找到映射的测试标识,再到本次真跑的 E2E 结果里核对:
| 情况 | 判定 |
|---|---|
| 映射存在 且 对应测试本次通过 | ✅ |
| 映射存在 但 测试本次失败 | ❌ |
| 映射存在 但 该测试在本次结果里找不到 | ❌ 映射腐烂(指向了不存在的测试) |
core_flow 在映射文件里没有条目 |
❌ 缺映射 |
映射里有 core_flows 中不存在的 id |
⚠️ 黄:孤儿映射,建议清理 |
缺一条 = 红。 绝不用"我觉得这条流程被别的测试覆盖了"来补空缺——那正是要消灭的主观判断。
2.3 配置健康度(顺带检测,零额外成本)
段一已经把每条命令真跑了一遍,过期信号是白捡的。汇总成报告的一节:
| 状态 | 判据 | 报告里怎么写 |
|---|---|---|
| ✅ 健康 | 命令跑起来了(退出码 0 或非 0 皆可) | 正常 |
| ⚠️ 已失效 | 127 / command not found / 脚本不存在 |
「<项> 的命令已跑不通——yml 过期」+ 报错原文 |
| 💡 可收紧 | 该项当前留空,但本次探测到已可用的命令 | 「<项> 现在可以填了:<候选命令>(已验证退出码 <N>)」 |
| — 留空 | 该项留空且确无可用命令 | 「留空,未测量」(不得判为通过) |
本节只报告、不改 yml——改配置等于改"通过"的定义,要走 /pdlc-test-setup --refresh(严向自动、松向人确认)。
2.4 PRD ↔ core_flows 对账(B2 的第二个地基,防 false-green)⭐
清单靠"有人记得改"维护必然腐烂;腐烂的清单产出 false-green——新增的核心流没进清单,矩阵照样全绿, 把"我们不知道"伪装成"我们覆盖了",比没有闸门更坏。所以每次运行都强制对账:
- 扫
docs/01_requirements/prd/所有 PRD,提取 P0 / P1 流程。 - 与
quality-targets.yml的core_flows做 diff。 - 漂移即红灯,不是温柔提示:
- PRD 有、
core_flows无 → ❌「PRD 流程 X 未进 core_flows」 core_flows有、映射无 → ❌「core_flow Y 尚无映射的 E2E」- 映射有、
core_flows无 → ⚠️ 孤儿映射
- PRD 有、
⛔ "不可判"绝不能被当成"没问题"(对账自身的 false-green,真项目上实测踩到过): 若某份 PRD 不含任何 P0/P1 标记,它提取出的就是空集,于是不产生任何漂移条目—— 报告若就此显示"对账通过",等于宣称"这份 PRD 里的流程都覆盖了",而事实是它整份都没进闸门视野。 已上线的主链路最容易栽在这里(老 PRD 常只写"已上线/待开发",不标优先级)。
规则:统计"因无优先级标记而未参与对账"的 PRD,在报告里单列告警,并且 对账项不得判为 ✅——写成
⚠️ 无漂移,但另有 N 份 PRD 不可判。 处理建议:给这些 PRD 补优先级标记,或在quality-targets.yml里显式声明豁免(写明理由)。
这样清单维护就从"靠自觉"变成被产物纪律接管——PRD 本就被 pdlc 逼着落盘并保持最新, 让它当
core_flows的唯一上游真源,与「状态外化到磁盘」是同一个哲学。
段三:出报告(真源 .md + 视图 .html)
① 先出 .md(真源):按 templates/quality-report-template.md 生成
docs/07_reviews/quality/<YYYY-MM-DD>.md(ledger 型:一次一份,可 git diff、可看趋势;
同日重跑则覆盖当日文件)。必须包含:
- 结论红绿表 2. 实测证据(命令 + 退出码 + 关键输出) 3. E2E 覆盖矩阵
- 配置健康度(§2.3) 5. PRD 对账结果 6. 趋势(首次则写「无趋势基线」) 7. 人工确认签字栏
② 再出 .html(视图):按 templates/quality-report-template.html 生成同目录同名
<YYYY-MM-DD>.html——给人看的那一份,可直接双击打开、可打印签字、可发给同事。
.md是唯一真源:/pdlc-ship的发布闸门读它、git diff看它、趋势对比取它。.html只是同一份数据的另一种呈现,没有任何独立信息。⛔ 铁律:HTML 里的每个数字都从
.md抄,不得重新计算、不得另行判断。 双写最容易出的错就是「两份报告各说各话」——那比没有 HTML 更糟:读者不知道该信哪份, 而错的那份通常更好看。两处不一致时一律以.md为准并改正.html。填写要求:只替换
{{...}}占位符与各表格的<tbody>内容;不要改<style>(排版由模板负责,改了就失去一致性);删掉模板顶部的「填写说明」注释块; 保持自包含——不得引入任何外部 URL(css / js / 字体 / 图片一律不许外链: 报告含项目内部数据,一张远程图片就够把打开时间、IP、referer 送给第三方, 而且离线打不开)。确需图片内联成data:URI。三态 class 必须对号入座:
st-pass达标 /st-fail未达标 /st-warn无法判定 /st-na留空未测量。st-warn绝不能写成st-pass——这是 §6.5 那条反模式 (「把我判断不了当成没问题」)在报告上的最后一道落地面。页头的{{VERDICT_SHORT}}只填裸词pass/fail/warn,一处占位符同时驱动顶条 与徽章,顶条不会和总判定说两套话。
段四:自检(强制)
- 报告里每一条判定,都能追到本次真跑的退出码 / 工具输出 / 映射核对结果
- 没有任何一项是靠"读代码觉得"得出的
- 留空 / 无法执行的项,如实标注且未按通过处理
- 覆盖率数字是从工具输出摘的原文
- E2E 矩阵里每条
core_flow都有明确判定(含"映射腐烂"这种红) - PRD 对账已执行,漂移项按红灯列出
- 配置健康度已汇报:失效项 / 可收紧项都列了出来,且没有擅自改动 yml
- 报告落盘到
docs/07_reviews/quality/,含生成时间与 commit SHA - 未在报告里替人做 go/no-go 决定
-
.html已同步产出,且与.md逐项核对过关键数字(覆盖率、核心流 M/N、各 check 退出码、不可判份数、总判定)——不一致以.md为准并已改正 -
.html里无残留{{占位符、无任何外部 URL(含图片)、<style>未被改动 - 「无法判定」的项在 HTML 里用的是
st-warn而非st-pass;总判定未达标时页头顶条颜色已同步
段五:修复(单次,不递归)
可自动修复的(如 lint 可自动修的告警)→ 修完重跑该 check 并以重跑结果为准,报告里注明"已自动修复后重测"。 不可自动修复 → 如实留红,写进报告。
怎么"自动化运行"(不依赖 CI)
- pre-push 钩子:push 前本地跑一遍质量闸,不达标就拦或告警——日常自动且零 CI 成本
- 按需:随时
/pdlc-quality出全量报告 - 发布挂钩:
/pdlc-ship会读最近一份质量报告,未达标不让发(除非人显式 override 并写明理由) - 要"每天一份":用本机 launchd / cron 跑,报告进 git;绝不用 GitHub Actions
schedule
段六:交接
本命令的 handoff 输出:
📊 质量报告:docs/07_reviews/quality/<YYYY-MM-DD>.md
覆盖率 : <实测> / 目标 <目标> ✅|❌
E2E 核心流 : <M>/<N> 条已覆盖 ✅|❌
Lint : 退出码 <N> ✅|❌
PRD 对账 : <无漂移 | N 项漂移 | 另有 N 份不可判> ✅|⚠️|❌
配置健康度 : <全部健康 | N 项已失效 | N 项可收紧>
🧾 总判定:<达标 | 未达标>
✍️ 待人工签字:报告最后一节(go/no-go 由你拍,本命令不代劳)
👉 未达标项处理:<每条给出具体下一步>
诚实边界
- 达标不等于质量好:覆盖率线挡的是"几乎没测",不保证用例有效;矩阵证明"每条核心流有测试跑过", 不证明"测得对"。报告要如实呈现这层含义,不要把"全绿"说成"质量有保障"。
- 保障强度取决于
core_flows维护得多勤——这正是 §2.4 强制对账存在的原因,但对账只能发现 "PRD 里有而清单里没有",PRD 本身漏掉的核心流谁也发现不了。这条限制要让用户知道。
参数: $ARGUMENTS