hf-tasks-review

HF Tasks Review

评审任务计划，判断任务是否可执行、可验证、正确排序、忠实覆盖已批准规格/设计。确保计划能支撑 router 稳定重选下一任务。

关键区别：通过时设 needs_human_confirmation=true（"任务真人确认" approval step），任务计划获批后才能进入实现。

Methodology

本 skill 融合以下已验证方法。每个方法在 Workflow 中有对应的落地步骤。

方法	核心原则	来源	落地步骤
INVEST Validation	检查每个任务是否满足 Independent/Negotiable/Valuable/Estimable/Small/Testable	Bill Wake, 2003；敏捷用户故事实践	步骤 2 — 多维评分；步骤 3.1 — 可执行性
Dependency Graph Validation	校验任务间依赖关系的正确性和无环性	项目化实践（任务计划评审通用方法）	步骤 2 — 多维评分；步骤 3.3 — 依赖/顺序
Traceability Matrix	检查任务是否忠实覆盖规格/设计的每一项关键决策	ISO/IEC/IEEE 29148 需求追溯实践	步骤 2 — 多维评分；步骤 3.4 — 追溯覆盖
Structured Walkthrough	多维度评分量化判断，任一关键维度低于阈值不得通过	项目化实践（评审通用方法）	步骤 2 — 多维评分；步骤 4 — verdict

When to Use

适用：

• 任务计划草稿需正式 review verdict
• hf-tasks 返回 需修改 或 阻塞，需复审修订后的计划
• 用户显式要求 review tasks plan

不适用 → 改用：

• 写/修任务计划 → hf-tasks
• 阶段不清 → hf-workflow-router
• 评审规格/设计 → 对应 review skill

Direct invoke 信号："review 任务计划"、"tasks review"、"帮我审一下这个计划"。

Hard Gates

• tasks review 通过并完成 approval step 前，不得进入 hf-test-driven-dev
• 输入工件不足不得开始
• reviewer 不替用户决定优先级或拆分方案

Workflow

1. 建立证据基线

读任务计划、已批准规格、已批准设计（默认 features/<active>/tasks.md / spec.md / design.md）、项目级约定、feature progress.md（默认 features/<active>/progress.md）。

1.5 Precheck：能否合法进入 review

检查：是否存在稳定可定位的任务计划、上游规格 / 设计 approval evidence 是否可回读、route/stage/profile 是否一致。

• route/stage/证据冲突 → 写最小 blocked precheck record，reroute_via_router=true
• route 明确但缺稳定任务计划 → 写最小 blocked record，下一步 hf-tasks
• precheck 通过 → 继续正式审查

2. 多维评分与挑战式审查

双层评分（v0.6 起）：

(a) 既有 6 维 0-10 评分（保留向后兼容）：任务可执行性、任务合同完整性（含 UI Implementation Contract 承接）、验证与测试设计种子（含 runtime / UI conformance evidence tier）、依赖/顺序正确性、追溯覆盖度、router 重选就绪度。任一关键维度 < 6 不得通过。

(b) Momus 4 维 boolean cliff（v0.6 新增，与 N=3 rewrite loop 配套）：按 references/momus-rubric.md 跑 4 维（Clarity 100% / Verification 90% / Context 80% / Big Picture 100%）+ 1 个 Zero-tolerance 行（0% critical red flag）。任一维不达阈值即 verdict: rejected-rewrite。

按 references/review-checklist.md 做正式 6 维审查；按 references/momus-rubric.md 做 4 维 boolean cliff 评分。两层均通过才算 通过。

每条 finding 必须带：

• severity（critical / important / minor）
• classification（USER-INPUT / LLM-FIXABLE）
• rule_id（如 TR2、TR5、TA3）

默认分类：

• USER-INPUT：优先级冲突、上游规格/设计裁决仍未稳定、任务边界一旦重排就会改变已确认范围或发布顺序
• LLM-FIXABLE：缺少 Acceptance / Files / Verify / test seed、任务过大、依赖链缺口、queue projection 表达不清

3. 正式 checklist 审查

3.1 可执行性：每个任务是否冷启动可执行？是否有"实现某模块"式模糊任务？ 3.2 任务合同完整性：关键任务是否显式具备 Acceptance、Files、Verify、完成条件？ 3.3 验证与测试设计种子：测试设计种子是否足够支持后续 fail-first 实现？是否只写了“补测试”这类空话？UI / API / full-stack 任务是否声明了应收集的 runtime evidence tier（API contract、browser runtime、full-stack smoke 等）、启动/健康检查入口和允许降级条件？UI 任务是否声明了 UI conformance evidence（截图路由/viewport、DOM anchors、console/network、token/visual drift checks）？ 3.4 依赖/顺序：依赖关系是否正确？关键路径是否合理？是否有循环依赖？ 3.5 追溯覆盖：任务是否忠实覆盖规格/设计？是否有任务无法追溯到上游依据？ 3.6 Router 重选就绪度：Current Active Task 选择规则是否唯一？queue projection 是否稳定？

4. 形成 verdict

verdict 取值（v0.6 起，与 N=3 rewrite loop 配套）：

• 通过：6 维 0-10 全 ≥ 6 + 4 维 boolean cliff（Clarity 100 / Verification 90 / Context 80 / Big Picture 100）全过 + 0 Zero-tolerance hit；可进入 approval step → next_action_or_recommended_skill=任务真人确认，needs_human_confirmation=true
• rejected-rewrite（v0.6 新名，旧名 需修改 仍兼容）：findings 可定向修订（包括 4 维 boolean cliff 任一不达标）→ next_action_or_recommended_skill=hf-tasks，needs_human_confirmation=false；标注当前 round 号到 review record
• 阻塞：核心任务结构有问题 / findings 无法定向回修 / 第 4 次 rejected-rewrite 仍未通过（N=3 上限触发，与 hf-ultrawork fast lane escape #5 对齐）→ next_action_or_recommended_skill=hf-tasks，needs_human_confirmation=false；若问题本质是 route/stage/profile/上游证据冲突 → next_action_or_recommended_skill=hf-workflow-router，reroute_via_router=true

N=3 Rewrite Loop（references/momus-rubric.md "N=3 Rewrite Loop" 段详）：Round 1/2/3 都允许 rejected-rewrite；第 4 次仍未通过自动转 阻塞 升级架构师，让架构师在 (a) 接受 Round 3 / (b) 重写 design / (c) 降级 Acceptance / (d) deferred backlog 之间拍板。

5. 写 review 记录

保存到 项目声明的 review record 路径；若无项目覆写，默认使用 features/<active>/reviews/tasks-review-YYYY-MM-DD.md。若项目无专用格式，默认使用 references/review-record-template.md。

回传结构化摘要时遵循当前 skill pack 中 hf-workflow-router/references/reviewer-return-contract.md：next_action_or_recommended_skill 只写一个 canonical 值；通过 时设 needs_human_confirmation=true；workflow blocker 必须显式写 reroute_via_router=true。

Output Contract

完成时产出：

• Review 记录（保存到 项目声明的 review record 路径；若无项目覆写，默认使用 features/<active>/reviews/tasks-review-YYYY-MM-DD.md）
• 结构化摘要含 record_path、next_action_or_recommended_skill、needs_human_confirmation（通过时 = true）
• workflow blocker 时显式写明 reroute_via_router=true

Reference Guide

文件	用途
references/review-checklist.md	正式 tasks review 6 维 0-10 checklist 与 rule IDs（既有）
references/momus-rubric.md	v0.6 新增 4 维 boolean cliff rubric（Clarity 100 / Verification 90 / Context 80 / Big Picture 100 / 0 Zero-tolerance）+ N=3 rewrite loop + fast lane escape #5 对齐
references/review-record-template.md	tasks review 记录模板与结构化返回契约（含 v0.6 起 rejected-rewrite verdict + round 号字段）
hf-workflow-router/references/reviewer-return-contract.md	当前 skill pack 共享的 reviewer 返回契约

Red Flags

• 不读规格/设计就审任务
• "看起来差不多"就通过
• 忽略任务间依赖错误
• 返回多个候选下一步

Common Rationalizations

借口	反驳 / Hard rule
"INVEST 大体满足就 pass。"	Hard Gates: rubric 中 INVEST 6 维必须逐项判定，"大体满足"非合规结论。
"DoD 没写但任务粒度小，我给过。"	Hard Gates: DoD 缺位 → finding；不可由 reviewer 默认 DoD。
"依赖图缺 1-2 条边没关系。"	Workflow stop rule: 依赖图缺边会让 hf-test-driven-dev 在错误任务上启动；任何缺失都必须落 finding。
"momus 阈值太严，第 3 轮还差 1% 就通过吧。"	Workflow stop rule（v0.6 momus）: 阈值是 boolean cliff，差 1% 也是不达标；自动转 阻塞 升级架构师，由架构师按 4 选项（接受 Round 3 / 重写 design / 降级 Acceptance / deferred backlog）拍板，reviewer 不替架构师做这个取舍。
"Clarity 100% 太理想化，99 个 task 缺 1 字段也算过吧。"	Workflow stop rule（v0.6 momus）: Clarity 是任务字段齐全度，缺字段的 task 直接让 hf-test-driven-dev 启动错误；"99/100" 不是合格结论。落 finding [important][LLM-FIXABLE][momus-Clarity] 让 author 补齐。

Verification

• review record 已落盘
• 给出明确结论、findings 和唯一下一步
• findings 已标明 severity / classification / rule_id
• precheck blocked 时已写明 workflow blocker 和 reroute_via_router
• 通过时已设 needs_human_confirmation=true
• 结构化摘要含 record_path 和 next_action_or_recommended_skill
• workflow blocker 时已显式写明 reroute_via_router