edit | blame | history | raw

2026-07-10 REVIEW-EDC-06

用户目标

对 MD-EDC-06-IMPL 做正式代码复核，Review Gate ID 固定为 crg-7d105de89062。本轮只审目标 worktree /Users/ar/Projects/Mindraw-worktrees/md-main-better-exp1 的声明 diff，不改源码、不改 m4c plugin、不执行 QA、不替代 PMO Gate / 用户验收 / 发布判断。

已完成事项

  • 读取项目规则、INDEX.md、CODEGRAPH.md、release / Apple 发布边界规则、MEXP15 台账、assignment、dispatch brief、review request、PO / UI / ARCH 输入和目标 worktree diff。
  • 只读复核目标分支 codex/md-main-better-exp1 上 MD-EDC-06-IMPL 的 11 个声明文件。
  • 复跑 coder 自查核心命令：前端测试、build、Rust check / tests、diff hygiene 和敏感扫描。
  • 回写 02-P/plan-progress.markdown 与正式 review 记录 02-P/MEXP15-SignedNotarizedDMG/review/review-edc-06-result-crg-7d105de89062.md。

关键决策

  • 代码复核结论：CHANGES_REQUESTED。
  • QA-EDC-06-EXEC 不允许启动，直到 crg-7d105de89062 或 PMO 指定的后续等效代码复核 gate 通过。
  • 本轮不触碰当前 CM1-OWNER-EDC-05 clean recheck 候选，不替换 1.2.4 / Build202607071930，不影响 0.1.3+codex.20260708220005 m4c plugin 复验结论。

核心事实 / 变更快照

  • Review Gate：crg-7d105de89062。
  • 变更对象：MD-EDC-06-IMPL App 新建文档默认行为增强。
  • Worktree：/Users/ar/Projects/Mindraw-worktrees/md-main-better-exp1。
  • Branch：codex/md-main-better-exp1。
  • HEAD / Base：0e329860f39577bdc0ab8236f5bf442fc163a22b。
  • 结论前：needs-code-review，QA-EDC-06-EXEC 不得启动。
  • 结论后：CHANGES_REQUESTED，仍不得启动 QA-EDC-06-EXEC。
  • 必须修复项：
  • .mindraw package create / rename / delete 路径操作跟随 symlink 后执行，可能突破 projectRoot / current folder guard。
  • create_blank_mindraw_package 未传 fileName 时默认 Untitled.mindraw，不符合 YY.MMDD-NN.mindraw。
  • 修复回包需明确 CODEGRAPH / INDEX / CHANGELOG 处理结论。

文件变更

  • 02-P/plan-progress.markdown：回写 REVIEW-EDC-06 / crg-7d105de89062=CHANGES_REQUESTED，追加 Progress Log。
  • 02-P/MEXP15-SignedNotarizedDMG/review/review-edc-06-result-crg-7d105de89062.md：新增正式 review 结论。
  • .codex-sessionhistory/COMMANDS.md：新增本轮 review 命令索引。
  • .codex-sessionhistory/2026-07-10-REVIEW-EDC-06.md：保存本轮会话记录。

可复用命令 / Runbook

用途：复核 MD-EDC-06-IMPL 的代码 diff、验证命令和安全扫描。

前置条件：目标 worktree 为 /Users/ar/Projects/Mindraw-worktrees/md-main-better-exp1，branch 为 codex/md-main-better-exp1。

git -C /Users/ar/Projects/Mindraw-worktrees/md-main-better-exp1 status --short --branch --untracked-files=all
git -C /Users/ar/Projects/Mindraw-worktrees/md-main-better-exp1 diff --stat -- src/lib/newDocument.ts src/lib/newDocument.test.ts src/types/index.ts src/store/useStore.ts src/hooks/useKeyboardShortcuts.ts src/hooks/useMenuHandler.ts src/components/Sidebar.tsx src/components/TreeView.tsx src/components/ExcalidrawEditor.tsx src-tauri/src/lib.rs src-tauri/src/menu.rs
git -C /Users/ar/Projects/Mindraw-worktrees/md-main-better-exp1 diff --check
npm run test:run -- src/lib/newDocument.test.ts
npm run test:run
npm run build
cargo check --manifest-path src-tauri/Cargo.toml
cargo test --manifest-path src-tauri/Cargo.toml mindraw_new_document_tests
cargo test --manifest-path src-tauri/Cargo.toml
git -C /Users/ar/Projects/Mindraw-worktrees/md-main-better-exp1 diff -- src src-tauri | rg -n "(SECRET|TOKEN|PASSWORD|PRIVATE_KEY|BEGIN RSA|AWS_|AKIA|sk-|xoxb-|ghp_|-----BEGIN)"

验证结果：git diff --check PASS；npm run test:run -- src/lib/newDocument.test.ts PASS，7 tests；npm run test:run PASS，12 tests；npm run build PASS，仅既有 warnings；cargo check PASS；cargo test ... mindraw_new_document_tests PASS，4 tests；cargo test PASS，4 tests；敏感扫描无命中。

搜索关键词：REVIEW-EDC-06、crg-7d105de89062、CHANGES_REQUESTED、MindrawCanvas、symlink、Untitled.mindraw。

未决问题

  • coder2@mindraw 需修复 review 记录中的 P1 / P2 问题并补测试。
  • PMO 需消费 PM_NOTIFY_READY REVIEW-EDC-06 REVIEW_GATE=crg-7d105de89062 RESULT=CHANGES_REQUESTED。
  • QA-EDC-06-EXEC 必须继续等待代码复核通过。

后续建议

  • 修复时优先在 Rust 层补 symlink / package boundary 测试，不要只在前端限制。
  • 后端默认文件名应与前端 formatMindrawDocumentFileName() 规则对齐，避免 App UI 和外部 command 合约分叉。
  • PMO 可要求修复回包同时明确目标 worktree 缺少 CODEGRAPH.md 的处理方式。