docs: Phase 1 landed (ea19c7d) - move design docs to done/, reconcile Phase 2 SDD

Phase 1 (planner_reviewer review gate + P0 resume-bug fix) shipped as ea19c7d, matching org-graph-unification-phase1-review-gate-design.md exactly (new task.status value, org role, _REQUIRED_ARTIFACTS narrowing, both bugs fixed). Moving that design doc and the P0 bug doc it subsumed to docs/done/, updating their status headers, and fixing the doc-path references left pointing at their old docs/todo/ location across agents/planner_reviewer/{SOUL,AGENTS}.md, src/schema.py migration docstrings, and the other Phase 2/deblackbox docs. Full suite green (731 tests) both before and after. Also ran Phase 2 SDD's §9 interface-reconciliation checklist against the real Phase 1 implementation now that it exists for real: all three items check out (run_id is included in plan_and_review_documents()'s return shape, the new task.status value is exactly reviewing_planner_output as assumed, and planner_reviewer's role.json doesn't pre-reserve a skill slot for Phase 2's new skill, which is fine -- a one-line addition when Phase 2c lands, not an incompatibility). Updated §9 to reflect this. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

docs: Phase 1 landed (ea19c7d) - move design docs to done/, reconcile Phase 2 SDD
Phase 1 (planner_reviewer review gate + P0 resume-bug fix) shipped as ea19c7d, matching org-graph-unification-phase1-review-gate-design.md exactly (new task.status value, org role, _REQUIRED_ARTIFACTS narrowing, both bugs fixed). Moving that design doc and the P0 bug doc it subsumed to docs/done/, updating their status headers, and fixing the doc-path references left pointing at their old docs/todo/ location across agents/planner_reviewer/{SOUL,AGENTS}.md, src/schema.py migration docstrings, and the other Phase 2/deblackbox docs. Full suite green (731 tests) both before and after. Also ran Phase 2 SDD's §9 interface-reconciliation checklist against the real Phase 1 implementation now that it exists for real: all three items check out (run_id is included in plan_and_review_documents()'s return shape, the new task.status value is exactly reviewing_planner_output as assumed, and planner_reviewer's role.json doesn't pre-reserve a skill slot for Phase 2's new skill, which is fine -- a one-line addition when Phase 2c lands, not an incompatibility). Updated §9 to reflect this. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
bboonnee
1 parent 0f0b8d8f
Showing 9 changed files with 18 additions and 18 deletions
agents/planner_reviewer/AGENTS.md
agents/planner_reviewer/SOUL.md
docs/todo/org-graph-unification-phase1-review-gate-design.md → docs/done/org-graph-unification-phase1-review-gate-design.md
docs/todo/task-execution-resume-assumes-planner-artifacts-exist.md → docs/done/task-execution-resume-assumes-planner-artifacts-exist.md
docs/todo/org-graph-unification-and-universal-self-planning-requirements.md
docs/todo/org-graph-unification-phase2-universal-self-planning-design.md
docs/todo/validator-test-script-narrative-output-issue.md
docs/todo/work-graph-node-execution-deblackbox-requirements.md
src/schema.py
 <!-- New 2026-08-08 (Org Graph unification Phase 1,
-     docs/todo/org-graph-unification-phase1-review-gate-design.md §8.3):
+     docs/done/org-graph-unification-phase1-review-gate-design.md §8.3):
      adapted from agents/validator/AGENTS.md's运行契约结构, 去掉与代码执行/测试
      相关的全部内容(不需要 Bash, 不跑测试, 不做代码审计)——这个角色只读文档内容
      并给出 VERDICT。 -->
 <!-- New 2026-08-08 (Org Graph unification Phase 1,
-     docs/todo/org-graph-unification-phase1-review-gate-design.md §8.3):
+     docs/done/org-graph-unification-phase1-review-gate-design.md §8.3):
      adapted from agents/validator/SOUL.md's judgment framework (PASS/FAIL/
      INCONCLUSIVE, independent re-derivation, no rubber-stamping) -- the
      object being judged changes from "Coder's final code implementation"
@@ -3,7 +3,7 @@
 **日期**: 2026-08-08
 **性质**: 详细设计，对应 `docs/todo/org-graph-unification-and-universal-self-planning-requirements.md` 的 Phase 1（FR-1，含 `task-execution-resume-assumes-planner-artifacts-exist.md` 的修复）。给出可直接实施的 schema/接口/算法/测试方案。
 **依赖代码**：`src/org_node_lifecycle.py`（`_PRE_STATE`/`_POST_STATE`/`_REQUIRED_ARTIFACTS`/`trigger_org_node_full`）、`src/e2e_pipeline.py`（`run_task_lifecycle`/`plan_documents`/`_parse_verdict_line`/`_VALIDATOR_VERDICT_PROMPT` 先例）、`src/task_execution.py`（`run_or_resume_task_lifecycle`/`_RESUMABLE_STATUSES`）、`src/scheduler.py`（`_BLOCKED_REASON_FOR_EXCEPTION`）、`src/schema.py`（`migrate_add_loop_back_event_type` 的 create-copy-drop-rename 先例）、`src/agent_profile.py`（`_discover_role_ids`，角色自动发现）、`roles/validator/role.json`（新角色的模板）。
-**状态**: 设计草案，待实施。
+**状态**: ✅ 已实现（commit `ea19c7d`，2026-08-08）。实现与本设计逐条吻合（新角色 `planner_reviewer`、`reviewing_planner_output` 状态值、`_REQUIRED_ARTIFACTS["coder"]` 收窄到 `(sdd,)`、P0 resume bug 与 fresh-run KeyError 一并修复，均与本文档 §3.1/§4.1/§7 一致）；实现过程中额外发现并修复了一个本设计未预见的问题：`agent_profile` 表的 `migrated_from_profile` 列原是 `NOT NULL`，`planner_reviewer` 这种没有对应 Hermes 遗留 profile 的新角色一注册就违反约束，随本次提交一并放开。`python3 -m unittest discover -s src` 全绿（731 tests，2026-08-12 复核）。移入 `docs/done/` 前保留在 `docs/todo/` 是 CLAUDE.md 记录过的已知文档滞后，本次一并订正。
 ---
@@ -2,7 +2,7 @@
 **日期**: 2026-08-08
 **性质**: 问题记录（真实复现，根因已定位，未修复）。发现于 `docs/done/work-graph-self-planning-and-loop-back-requirements.md`（Planner 自规划）上线后的第一次真实端到端验证。
-**状态**: 待修复，未立项。**2026-08-08 更新**：已折入 `docs/todo/org-graph-unification-and-universal-self-planning-requirements.md` 的 Phase 1（FR-1）——插入"Planner 产出复核"新阶段本来就要重写这段 `run_or_resume_task_lifecycle()` 分支逻辑，这个 bug 跟那次改动是同一段代码路径，不再单独排期，随 Phase 1 一并修。**详细设计已发布**：`docs/todo/org-graph-unification-phase1-review-gate-design.md` §6-7——修复方式是把 sdd/prd/benchmark 的存在性探测下沉到新增的 `_review_planner_output_with_retries()` 内部（有就用，没有就重新规划），不再在 resume 分支顶部无条件硬读；同一次调研还发现 fresh-run 路径（`run_task_lifecycle()`）有同类但更早触发的 `KeyError`，一并修复。
+**状态**: ✅ 已修复（commit `ea19c7d`，2026-08-08，随 Phase 1 一并落地）。修复方式与 §4"可能的修复方向"一致：sdd/prd/benchmark 的存在性探测下沉到新增的 `_review_planner_output_with_retries()` 内部（有就用，没有就重新规划），不再在 resume 分支顶部无条件硬读；同一次实现还修复了 fresh-run 路径（`run_task_lifecycle()`）里同类但更早触发的 `KeyError`。详细设计见 `docs/done/org-graph-unification-phase1-review-gate-design.md` §6-7，`python3 -m unittest discover -s src` 全绿（2026-08-12 复核）。
 ## 1. 现象（真实复现）
@@ -3,7 +3,7 @@
 **日期**: 2026-08-08
 **性质**: 需求文档（BRD/PRD 层面），不含详细技术设计。融合并统领以下既有 `docs/todo/` 文档的相关内容：
 - `docs/todo/org-graph-as-work-graph-unified-loop-requirements.md`（已删除，内容 100% 并入本文档）——本文档是它的展开与拍板；该文档提出的"跨 Org Agent 精确 loop_back"诉求是本文档 FR-4 + §4 关键架构决策的正文。
-- `docs/todo/task-execution-resume-assumes-planner-artifacts-exist.md`——它记录的 bug 与本文档 FR-1 改动的是**同一段** `run_or_resume_task_lifecycle()` 分支逻辑（见 §2.1 调研结论），折入 Phase 1 一并修，不再单独排期。
+- `docs/done/task-execution-resume-assumes-planner-artifacts-exist.md`——它记录的 bug 与本文档 FR-1 改动的是**同一段** `run_or_resume_task_lifecycle()` 分支逻辑（见 §2.1 调研结论），折入 Phase 1 一并修，不再单独排期。
 - `docs/todo/validator-test-script-narrative-output-issue.md` 第 8 节提到的"另一份尚未撰写、关于给 Coder agent 加入 test_script 模块的文档"，即本文档 Phase 2（FR-2/FR-3）。
 **触发场景**：任务 `2084590a`（修复 `work_node_run.py` 的 `execution_log_file` bug）里，Planner 自规划出的 `benchmark` 节点连续 4 次产出的都是"叙述我做了什么"的散文（跟 `validator-test-script-narrative-output-issue.md` 同一类问题，这次发生在 benchmark 上），LLM judge 的 `on_fail=retry_self` 循环耗尽 `max_attempts=3` 后任务直接 `blocked_pending_human`——暴露出 Validator 从来没有机会在 Coder 开始实现之前复核 Planner 自己的产出。用户随后 2026-08-08 提出四点架构改动设想（见 §1），与当天早些时候整理的 `docs/todo/` 待办优先级建议（P0~P3）高度交叉，要求重新盘点、融合、给出 phase 规划。
@@ -110,8 +110,8 @@
 ## 6. Phase 规划
-### Phase 1 — Org 主流程改造 + resume bug 一并修复
-- **详细设计**：`docs/todo/org-graph-unification-phase1-review-gate-design.md`（2026-08-08，可直接实施）。
+### Phase 1 — Org 主流程改造 + resume bug 一并修复 ✅ 已实现（commit `ea19c7d`，2026-08-08）
+- **详细设计**：`docs/done/org-graph-unification-phase1-review-gate-design.md`（已随实现移入 `docs/done/`，与实现逐条吻合，见该文档更新后的状态行）。
 - **范围**：FR-1（含 `task-execution-resume-assumes-planner-artifacts-exist.md` 的修复）。
 - **产出**：`task.status`/`execution_event` 两处 schema 迁移、新增独立 org role `planner_reviewer`、`org_node_lifecycle.py` 的 `_PRE_STATE`/`_POST_STATE`/`_REQUIRED_ARTIFACTS` 扩展、`e2e_pipeline.py` 新增复核+重试循环、`run_task_lifecycle()`/`run_or_resume_task_lifecycle()` 新分支。复核失败的重试是任务内部的粗粒度循环（完整重新规划，不接 `org_level_loop.py`），不是"打回整个任务重来"。
 - **额外发现**（设计阶段调研中定位，一并修复）：`e2e_pipeline.py::run_task_lifecycle()` 对 `artifact_ids["prd"]`/`["benchmark"]` 的无条件下标访问，在自规划图省略这两个类型时会 `KeyError`——这个崩溃在 fresh-run 首次执行就会触发，不需要等到 resume，是比 P0 bug 更早触发的同类问题。
@@ -142,7 +142,7 @@
 ## 7. 待确认的问题（讨论后逐条勾掉，参照 `graph-engineering-open-items.md` 的状态图例）
 ✅ §4 关键架构决策：方案 A（合图）。2026-08-08 用户拍板。
-✅ FR-1 新增的 `task.status` 枚举值定为 `reviewing_planner_output`。2026-08-08，随 Phase 1 SDD（`docs/todo/org-graph-unification-phase1-review-gate-design.md`）一起拍板，理由见该文档 §3.1。
+✅ FR-1 新增的 `task.status` 枚举值定为 `reviewing_planner_output`。2026-08-08，随 Phase 1 SDD（`docs/done/org-graph-unification-phase1-review-gate-design.md`）一起拍板，理由见该文档 §3.1。
 ✅ FR-1 的"复核 Validator"用新增独立 org role `planner_reviewer`，不复用 `validator`。2026-08-08 用户拍板（`AskUserQuestion` 确认），理由见 Phase 1 SDD §2.1——`_PRE_STATE`/`_POST_STATE` 是按 org_node 名字一对一映射，同一个 key 没法挂两组状态转移。
 ✅ FR-4 的节点级验证 skill：新开一个 skill，不在 `qa-task-validation` 基础上改造。2026-08-08 用户拍板。理由：`qa-task-validation` 整份内容绑定飞书多维表格语义（硬编码 `tblXXX` 表 ID、状态机是"待测试→已完成"这类 Base 记录写回），判定粒度是整个任务的 PASS/FAIL/INCONCLUSIVE；FR-4 要的是单个 Work Graph 节点的 `judgment.kind="llm"` 回调，输入是一个节点的输出，输出要精确指向 `loop_back` 的目标 `node_id`，和 Base 完全无关，两种用途混进同一份 skill 容易互相污染。新 skill 可以借鉴其判定原则（三态结论、独立复核、真实路径优先）写进 prose，但不复用文件本身。
 ✅ Phase 2 的 Coder/Validator 自规划：加一份建议清单（类似 `_PLANNER_NODE_CATALOG`），不是完全自由。2026-08-08 用户拍板——推翻了 `work_graph_plan.py:161-170` 2026-08-06 注释里"Coder/Validator's sub-tasks have no such fixed small vocabulary of node types"的原判断。Phase 2 设计阶段需要给 Coder/Validator 各自定义一份"常见节点类型"参考目录（如 impl/unit_test/integration_check），每项可以像 `_PLANNER_NODE_CATALOG` 一样标注 `suggested_judgment`，但不强制——沿用 Planner 目录"建议不强制"的既有约定，而不是 Planner 文档链那种固定顺序的强约束。
@@ -2,8 +2,8 @@
 **æ—¥æœŸ**: 2026-08-08
 **æ€§è´¨**: è¯¦ç»†è®¾è®¡ï¼Œå¯¹åº” `docs/todo/org-graph-unification-and-universal-self-planning-requirements.md` çš„ Phase 2ï¼ˆFR-2/FR-3/FR-4ï¼‰ï¼ŒæŒ‰ Â§4/Â§4.1 å·²æ‹æ¿çš„åˆå›¾ï¼ˆæ–¹æ¡ˆ Aï¼‰è·¯å¾„è®¾è®¡ã€‚
-**ä¾èµ–**: ç¡¬ä¾èµ– Phase 1ï¼ˆ`docs/todo/org-graph-unification-phase1-review-gate-design.md`ï¼‰å…ˆè½åœ°â€”â€”Phase 2 æ˜¯åœ¨ Phase 1 å»ºç«‹çš„ `planner_reviewer` é˜¶æ®µåŸºç¡€ä¸Šç»§ç»åˆå›¾ï¼ŒPhase 1 æœªå®Œæˆå‰æœ¬æ–‡æ¡£çš„è°ƒç”¨ç‚¹æ”¹åŠ¨æ— æ³•æŽ¥å…¥ã€‚Phase 1 ç›®å‰ç”±å¦ä¸€ä¸ªè¿›ç¨‹åœ¨å®žæ–½ä¸ï¼Œæœ¬æ–‡æ¡£çš„å…·ä½“ä»£ç ä½ç½®å¼•ç”¨ä»¥ Phase 1 SDD çš„è®¾è®¡ä¸ºå‡†ï¼ŒPhase 1 çœŸæ£è½åœ°åŽéœ€è¦ä¸€æ¬¡å¿«é€Ÿæ ¸å¯¹ï¼ˆè§ Â§9ï¼‰ã€‚
-**çŠ¶æ€**: è®¾è®¡è‰æ¡ˆã€‚**2026-08-08 ä¿®è®¢**ï¼šå†™ä½œè¿‡ç¨‹ä¸ï¼Œéœ€æ±‚æ–‡æ¡£ Â§7 çš„å››ä¸ªå¼€æ”¾é—®é¢˜å·²è¢«ç”¨æˆ·ç›´æŽ¥æ‹æ¿ï¼ˆè¯¦è§è¯¥æ–‡æ¡£ Â§7 å¯¹åº”æ¡ç›®ï¼‰ï¼Œå…¶ä¸ä¸¤æ¡æŽ¨ç¿»äº†æœ¬æ–‡æ¡£æœ€åˆçš„è®¾è®¡åˆ¤æ–â€”â€”Coder/Validator è‡ªè§„åˆ’æ”¹ä¸º"å»ºè®®èŠ‚ç‚¹æ¸…å•"è€Œéžå®Œå…¨è‡ªç”±ï¼ˆÂ§3ï¼‰ï¼Œ`work_graph_run.org_node` æ”¹ä¸ºå½»åº•ç§»é™¤è€Œéžä¿ç•™ï¼ˆÂ§2.1ï¼‰â€”â€”æœ¬ç‰ˆæœ¬å·²æŒ‰æ‹æ¿ç»“æžœæ”¹å†™ï¼Œä¸æ˜¯åŽŸå§‹è‰ç¨¿ã€‚
+**ä¾èµ–**: ç¡¬ä¾èµ– Phase 1ï¼ˆ`docs/done/org-graph-unification-phase1-review-gate-design.md`ï¼‰â€”â€”å·²å®žçŽ°ï¼ˆcommit `ea19c7d`ï¼Œ2026-08-08ï¼‰ã€‚Â§9 æŽ¥å£æ ¸å¯¹æ¸…å•å·²äºŽ 2026-08-12 é€æ¡æ ¸å¯¹å®Œæˆï¼šä¸‰é¡¹å…¨éƒ¨å»åˆï¼ˆ`plan_and_review_documents()` ç» `_review_planner_output_with_retries()` é€ä¼  `plan_documents()` çš„ `{"run_id", "artifact_ids"}` è¿”å›žå½¢çŠ¶ï¼Œå« `run_id`ï¼›`task.status` æ–°å€¼ç¡®è®¤æ˜¯ `reviewing_planner_output`ï¼›`roles/planner_reviewer/role.json` ç›®å‰ `skills: ["planner-output-review", "unattended-agent-operations"]`ï¼Œæœªé¢„ç•™ Â§7.1 æ–° skill ä½ç½®ï¼ŒPhase 2 å®žæ–½æ—¶æ–°å¢žä¸€è¡Œå³å¯ï¼Œä¸æ˜¯ä¸å…¼å®¹ï¼‰ã€‚
+**çŠ¶æ€**: è®¾è®¡è‰æ¡ˆï¼ŒPhase 1 å‰ç½®æ¡ä»¶å·²æ»¡è¶³ï¼Œå¯ä»¥å¼€å§‹å®žæ–½ã€‚**2026-08-08 ä¿®è®¢**ï¼šå†™ä½œè¿‡ç¨‹ä¸ï¼Œéœ€æ±‚æ–‡æ¡£ Â§7 çš„å››ä¸ªå¼€æ”¾é—®é¢˜å·²è¢«ç”¨æˆ·ç›´æŽ¥æ‹æ¿ï¼ˆè¯¦è§è¯¥æ–‡æ¡£ Â§7 å¯¹åº”æ¡ç›®ï¼‰ï¼Œå…¶ä¸ä¸¤æ¡æŽ¨ç¿»äº†æœ¬æ–‡æ¡£æœ€åˆçš„è®¾è®¡åˆ¤æ–â€”â€”Coder/Validator è‡ªè§„åˆ’æ”¹ä¸º"å»ºè®®èŠ‚ç‚¹æ¸…å•"è€Œéžå®Œå…¨è‡ªç”±ï¼ˆÂ§3ï¼‰ï¼Œ`work_graph_run.org_node` æ”¹ä¸ºå½»åº•ç§»é™¤è€Œéžä¿ç•™ï¼ˆÂ§2.1ï¼‰â€”â€”æœ¬ç‰ˆæœ¬å·²æŒ‰æ‹æ¿ç»“æžœæ”¹å†™ï¼Œä¸æ˜¯åŽŸå§‹è‰ç¨¿ã€‚**2026-08-08 è¿½åŠ **ï¼ˆåŒæ—¥ï¼Œè§éœ€æ±‚æ–‡æ¡£æ–°å¢ž Â§3.1ï¼‰ï¼šFR-4 çš„ç›®æ ‡è¢«æ˜Žç¡®ä¸º"å¤±è´¥å³è¿›åŒ–"å®Œæ•´æ–¹æ³•è®ºï¼ˆåˆ¤æ–å¤±è´¥ â†’ Reflection-review åŠ¨æ€è¯Šæ– â†’ ç³»ç»Ÿæ€§ç¼ºé™·åˆ™å…ˆ Fix-Skill â†’ è·³å›žæ‰§è¡Œï¼‰ï¼Œæœ¬æ–‡æ¡£ Â§5 è®¾è®¡çš„ `dynamic_loop_back` åªè¦†ç›–å…¶ä¸ç¬¬ 1/2/4 æ¥ï¼Œç¬¬ 3 æ¥ï¼ˆFix-Skillï¼‰æ˜¯å·²çŸ¥ç¼ºå£ï¼Œè§ Â§5 å¼€å¤´è¯´æ˜Žå’Œ Â§8 ç¬¬ 5 æ¡ã€‚
 ---
@@ -397,11 +397,11 @@ Phase 1 é˜¶æ®µ&quot;å¤æ ¸å¤±è´¥å…ˆç”¨çŽ°æœ‰ `org_level_loop.py` ç²—ç²’åº¦è·¯ç”±å…œåº
 ---
-## 9. è·Ÿæ£åœ¨å®žæ–½çš„ Phase 1 çš„æŽ¥å£æ ¸å¯¹æ¸…å•ï¼ˆPhase 1 è½åœ°åŽç«‹å³æ‰§è¡Œï¼‰
+## 9. è·Ÿ Phase 1 çš„æŽ¥å£æ ¸å¯¹æ¸…å•â€”â€”âœ… å…¨éƒ¨æ ¸å¯¹å®Œæˆï¼ˆ2026-08-12ï¼‰
-- [ ] `plan_and_review_documents()` çš„è¿”å›žå½¢çŠ¶æ˜¯å¦åŒ…å« `run_id`ï¼ˆÂ§6.2 ä¾èµ–ï¼‰ã€‚
-- [ ] Phase 1 å®žé™…å¼•å…¥çš„æ–° `task.status` å€¼æ˜¯ä¸æ˜¯æœ¬æ–‡æ¡£å‡è®¾çš„åå—ï¼ˆPhase 1 SDD å®šçš„æ˜¯ `reviewing_planner_output`ï¼ŒPhase 1 ç”±å¦ä¸€è¿›ç¨‹å®žæ–½ï¼Œå®žé™…è½åœ°å€¼éœ€è¦æ ¸å¯¹ä¸€æ¬¡ï¼‰ã€‚
-- [ ] `roles/planner_reviewer/role.json` çš„ `skills` åˆ—è¡¨æ˜¯å¦å·²ç»é¢„ç•™ä½ç½®ç»™ Â§7.1 æ–°å¢žçš„ `work-graph-node-review`ï¼ˆå¦‚æžœ Phase 1 å®žæ–½æ—¶å·²ç»å†™äº†åˆ«çš„å ä½ skill åå—ï¼Œéœ€è¦ç»Ÿä¸€ï¼‰ã€‚
+- [x] `plan_and_review_documents()` çš„è¿”å›žå½¢çŠ¶æ˜¯å¦åŒ…å« `run_id`â€”â€”**æ˜¯**ï¼š`plan_and_review_documents()` â†’ `_review_planner_output_with_retries()` â†’ é€ä¼ æ‹¿åˆ° PASS é‚£ä¸€è½® `plan_documents()` çš„åŽŸå§‹è¿”å›žå€¼ `{"run_id", "artifact_ids"}`ï¼Œ`run_id` é½å…¨ï¼ŒÂ§6.2 çš„å‡è®¾æˆç«‹ï¼Œ`implement_code()`/`validate_implementation_with_own_graph()` å¯ä»¥ç›´æŽ¥æ‹¿è¿™ä¸ª `run_id` ç”¨ä½œ `append_to_run_id`ã€‚
+- [x] Phase 1 å®žé™…å¼•å…¥çš„æ–° `task.status` å€¼æ˜¯ä¸æ˜¯ `reviewing_planner_output`â€”â€”**æ˜¯**ï¼Œ`src/schema.py`/`src/org_node_lifecycle.py` å‡æŒ‰æ¤å®žçŽ°ï¼Œä¸Žæœ¬æ–‡æ¡£å‡è®¾å®Œå…¨ä¸€è‡´ã€‚
+- [x] `roles/planner_reviewer/role.json` çš„ `skills` åˆ—è¡¨æ˜¯å¦å·²ç»é¢„ç•™ä½ç½®ç»™ Â§7.1 æ–°å¢žçš„ `work-graph-node-review`â€”â€”**æ²¡æœ‰é¢„ç•™**ï¼Œå½“å‰æ˜¯ `["planner-output-review", "unattended-agent-operations"]`ã€‚ä¸æ˜¯é—®é¢˜ï¼šPhase 2 å®žæ–½ Â§7.1 æ—¶ç›´æŽ¥è¿½åŠ ä¸€è¡Œ `"work-graph-node-review"` å³å¯ï¼Œ`roles/coder`/`roles/validator` çš„ `role.json` åŒç†éœ€è¦å„è¿½åŠ ä¸€è¡Œâ€”â€”è¿™æ˜¯ Phase 2c å®žæ–½æ¸…å•é‡Œçš„ä¸€æ¥ï¼Œä¸æ˜¯éœ€è¦æå‰åè°ƒçš„æŽ¥å£ä¸å…¼å®¹ã€‚
 ---
@@ -15,7 +15,7 @@
 | 5 | 08-08 | **真实复现，推翻第 3 轮判断**：任务 `2084590a` 的 `benchmark` 节点（不是 test_script）连续 4 次输出叙述文字（详见 §1 末尾），LLM judge 第 3 次已精确诊断出问题但未能强制拦住，第 4 次原样复现，`loop_back` 耗尽后 `blocked_pending_human`。触发条件改判为"这次任务需不需要模型先探索真实代码"，不是"节点类型是不是 test_script"——benchmark 也没有类似 `check_script_compiles` 的机械拦截网，只能靠 LLM judge 自己识别，识别到不等于能强制拦住。 |
 | 6 | 08-08 | 结构性修复的落脚点确定为 `org-graph-unification-and-universal-self-planning-requirements.md` Phase 2——该文档非目标 §5.2 已记录这条已知风险，要求 Phase 2 落地时专门验证是否复现。 |
-后续讨论方向记在 `docs/todo/org-graph-unification-and-universal-self-planning-requirements.md`（原草案 `org-graph-as-work-graph-unified-loop-requirements.md` 已并入此文档并删除）——包括要不要给 benchmark 也加一个类似的启发式机械检查、以及要不要在 Coder 开始实现之前对 Planner 产出多一道独立复核（后者已在该文档 Phase 1 SDD `docs/todo/org-graph-unification-phase1-review-gate-design.md` 落地为具体设计）。
+后续讨论方向记在 `docs/todo/org-graph-unification-and-universal-self-planning-requirements.md`（原草案 `org-graph-as-work-graph-unified-loop-requirements.md` 已并入此文档并删除）——包括要不要给 benchmark 也加一个类似的启发式机械检查、以及要不要在 Coder 开始实现之前对 Planner 产出多一道独立复核（后者已在该文档 Phase 1 SDD `docs/done/org-graph-unification-phase1-review-gate-design.md` 落地为具体设计）。
 ## 1. 现象
@@ -195,4 +195,4 @@ def run_fn(cwd, model=None):
 ## 7. 跟 Org-Graph-Unification 系列文档的关系
-正交但有先后关系：这次改动只影响**单节点执行机制**，不影响 Org Graph 流程形状或自规划范围，`docs/todo/org-graph-unification-phase1-review-gate-design.md`（Phase 1）不依赖这次改动，可以并行/先后独立推进。但 `docs/todo/org-graph-unification-and-universal-self-planning-requirements.md` 的 **Phase 2**（FR-4，节点级验证 skill）会大量新增 `judgment.kind="llm"` 判断调用——如果这次去黑盒化工作能在 Phase 2 动工前完成，Phase 2 就不需要在两套机制（Workflow 和新的 flat 调用）之间选一套来复用，直接基于 `trigger_org_node_structured()` 设计，减少后续返工。建议顺序：这份文档 → Phase 2，不是必须但值得。
+正交但有先后关系：这次改动只影响**单节点执行机制**，不影响 Org Graph 流程形状或自规划范围，`docs/done/org-graph-unification-phase1-review-gate-design.md`（Phase 1）不依赖这次改动，可以并行/先后独立推进。但 `docs/todo/org-graph-unification-and-universal-self-planning-requirements.md` 的 **Phase 2**（FR-4，节点级验证 skill）会大量新增 `judgment.kind="llm"` 判断调用——如果这次去黑盒化工作能在 Phase 2 动工前完成，Phase 2 就不需要在两套机制（Workflow 和新的 flat 调用）之间选一套来复用，直接基于 `trigger_org_node_structured()` 设计，减少后续返工。建议顺序：这份文档 → Phase 2，不是必须但值得。
@@ -697,7 +697,7 @@ def migrate_add_loop_back_event_type(conn):
 def migrate_add_reviewing_planner_output_status(conn):
     """2026-08-08 (Org Graph unification Phase 1,
-    docs/todo/org-graph-unification-phase1-review-gate-design.md §3.1): adds
+    docs/done/org-graph-unification-phase1-review-gate-design.md §3.1): adds
     'reviewing_planner_output' to task.status's CHECK enum on an
     ALREADY-POPULATED database -- same create-copy-drop-rename dance as
     migrate_expand_artifact_types/migrate_add_loop_back_event_type. Idempotent
@@ -741,7 +741,7 @@ def migrate_add_reviewing_planner_output_status(conn):
 def migrate_add_planner_review_verdict_event_type(conn):
     """2026-08-08 (Org Graph unification Phase 1,
-    docs/todo/org-graph-unification-phase1-review-gate-design.md §3.2): adds
+    docs/done/org-graph-unification-phase1-review-gate-design.md §3.2): adds
     'planner_review_verdict' to execution_event.event_type's CHECK enum on an
     ALREADY-POPULATED database -- same create-copy-drop-rename dance as
     migrate_add_loop_back_event_type. Idempotent and safe to call