# 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`。

```bash
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` 的处理方式。