DevFlow/docs/待审查.md

待审查队列

职责分离:

• 主会话(编码合并后 / 巡检)→ 【写入】"当前队列"区,登记待审查项(范围 + 维度 + commit)。
• 审查 agent(独立会话 / workflow)→ 【读取】"当前队列",执行审查,完成回填结论 + 移到"已审归档"。

这里只放审查登记,不放推进待办(推进待办见 todo.md)。 主会话不在这里做审查,只登记 + 读 todo 推进。审查发现的【待修代码项】回流 todo 作推进待办。

审查规则(审查 agent 遵守)

• 只读审查,不改代码。
• 独立 grep/read 核验源码,不信文档 / 会话描述 / 设计声明的结论(防上下文污染)。
• 每条 finding 须 file:line 源码佐证 + evidence。
• 严重度:high(破坏性 / 数据错 / 回归) / med(逻辑瑕疵 / 边界漏) / low(风格 / 可读)。
• 完成回填:总体 verdict(PASS / ISSUES / WATCH)+ high/med/low 清单 + 待修项回流 todo。

状态图例

🟡 待审 / 🔵 审查中 / ✅ 已审(结论) / 🔴 待修(有 high 问题)

---

当前队列

CR-260616-01-fixes 审查待修项修复批次 — 🟡 待审(低优先,主代理已核查)

• 状态: 🟡 待审(低优先 — 小修+主代理已独立核查双绿,独立审查 agent 可选核)
• 来源: CR-260616-01 审查待修项①类7项(cr01-fixes workflow wu4r8ejcp)
• 改动文件:
  • src/views/TaskDetail.vue(.status-testing/.status-blocked CSS + advancing 接文案)
  • crates/df-storage/src/crud.rs(CAS SQL 加 AND deleted_at IS NULL 两分支 + 白名单注释)
  • crates/df-nodes/src/task_advance_node.rs(ALL_STATES.join DRY + 测试改名)
  • src-tauri/src/state.rs(build_registry 补 TaskAdvanceNode 未注册注释)
• 主代理核查: cargo check --workspace exit 0(仅 pre-existing warning)+ vue-tsc exit 0 + git diff 边界合规(4 文件符合 agent 领地)
• 审查关注点:①CR-01-B CAS 补 deleted_at 语义正确性(回收站任务必败)②CR-01-A CSS 配色跨视图一致 ③CR-01-C DRY 无运行时文案变更
• 未做: CR-01-D(错误枚举区分)/CR-01-G(i18n 分级)②类留 todo

---

已审归档

CR-260616-01 任务推进链(batch28) — ✅ 已审

• commit: d2cb38c 新增: 任务推进链(7态状态机+advance_task CAS原子写)+软删除+前后端7态对齐
• 审查时间: 2026-06-16(5 路并行多角度·防代入独立核验)
• 决策依据: D-260616-01(前端7态对齐) / D-02(任务软删除) / D-03(advance_task 走 df-nodes Node trait) / D-04(阶段1先行)
• 总体结论: 🔴0 🟡6 ⚪4 — 质量优
• 8 点 checklist 全过:
  • 状态机矩阵 ✅(7态15转换,前后端单一真相源)
  • CAS 原子 ✅(WHERE id AND status=expected + affected=0 报错)
  • 退回累加原子 ✅(同 UPDATE)
  • V15 迁移幂等 ✅(版本门控 + column_exists 双层,八版零事故)
  • IPC 真 thin ✅(D-03 守住,task.rs 仅 3 行转发)
  • i18n 成对零遗漏 ✅(28 key)
  • create_task 不暴露 rounds ✅
  • 测试 25 个 ✅
• F-03 status 双写 = 已知延后(阶段3 三件同落地,非新缺陷)
• 范围外澄清: df-ai/lib.rs(mod retry) + script_node.rs(shell_type) = 同 commit 无关连带改,零风险误报
• 待修项回流 todo(CR-01-A~I,见 todo.md "审查发现待修项"区):
  • [P1] CR-01-A TaskDetail.vue scoped style 补 .status-testing/.status-blocked(D-01 前端7态遗漏,testing/blocked 徽章裸渲染丢样式;同查 Tasks.vue/Dashboard)
  • [P1] CR-01-B crud.rs advance_status_atomic CAS SQL 加 AND deleted_at IS NULL(get_by_id 宏无软删过滤→回收站任务可被 advance,D-02 软删对齐遗漏,语义错乱)
  • [P2] CR-01-C task_advance_node.rs:46 错误7态硬编码改 ALL_STATES.join("/")(DRY 单一事实源)
  • [P2] CR-01-D task_advance_node.rs:62 同态拒绝(from==to)/非法转换区分(现合并 InvalidState,前端无法分辨)
  • [P2] CR-01-E state.rs build_registry 注释补「task_advance 未注册(阶段2联动时加)」
  • [P2] CR-01-F crud.rs:325 白名单注释补 review_rounds 专用路径
  • [P3] CR-01-G TaskDetail.vue:218 advance 失败错误走 i18n 分级(en locale 也中文)
  • [P3] CR-01-H task_advance_node.rs:297 测试改名(实际未测 CAS 失败)
  • [P3] CR-01-I i18n taskDetail.advancing 冗余 key 处理