Reviews for D-08

tasks/reviews/D-08-r1.md

@@ -0,0 +1,3 @@
+FAIL
+
+crates/ely-ui/src/commands.rs:213 — 命令面板仍可绕过打印的平台禁用检查。`shell/root.rs:313-322` 将 `file.print` 放入命令面板，选择后直接调用 `commands::run`；这里直接分发 `Print`，文档视图的处理器先接收动作，外层 `Root::attempt` 的 `blocked()` 检查来不及拦截。具体场景：Linux/Windows 打开文档，打开命令面板，搜索并执行“打印…”；Doc/Show 仍进入打印范围流程，Table/PDF/Mark 仍生成打印副本，最后才收到平台未构建错误，违反 D-08 与 X-13 打印入口禁用要求。复现证据：在独立源码副本中仅将 commands.rs 的两处 `BUILT` 引用设为 false，经真实 GPUI 命令面板键入 `Print` 并回车，菜单已禁用、打印快捷键未绑定，文档打印处理器仍被调用 1 次；加入分发前检查后为 0 次，恢复后再次为 1 次（`/tmp/ely-d08-r1-probe.log`、`/tmp/ely-d08-r1-guard-control.log`）。原有平台/UI 单元测试 84 项通过，实际 Windows 打印分支独立编译通过；完整 Windows 平台库检查因缺少 Windows C 头文件停在 ring。修复方向：命令面板执行打印前统一检查平台能力并显示禁用原因，补充 `BUILT=false` 时从命令面板执行打印的回归测试。探针源码与其构建目录已删除。

tasks/reviews/D-08-r2.md

@@ -0,0 +1,3 @@
+FAIL
+
+crates/ely-ui/src/shell/root.rs:375 — 回归测试的 Save 计数存在假通过：紧接着的 `tool_root` 会在 `build` 中把焦点移到 Root，计数视图随即失焦。因此，`a_blocked_palette_pick_never_reaches_the_view` 的零次 Save 断言无法验证拦截。具体复现：在独立副本中把 `run_from_palette` 改为先调用 `self.refused(id, cx)` 显示提示，再无条件调用 `commands::run(id, window, cx)`；该错误实现仍通过现有回归测试（Save 为 0、提示为 1）。仅将测试中的 `window.focus(&view.0, cx)` 移到 `tool_root` 创建之后，同一错误实现即收到 1 次 Save，并在动作计数断言处失败。直接移除检查时，原测试仅因提示数量为 0 而失败。证据：`/tmp/ely-d08-r2-dispatch-despite-toast.log`、`/tmp/ely-d08-r2-focus-control.log`、`/tmp/ely-d08-r2-guard-removed-a_blocked_palette_pick.log`。修复方向：创建 tool root 后重新聚焦计数视图，并用保留提示但继续分发的变异验证 Save 断言确实失败。当前实际命令面板的打印拦截已通过独立复现，ely-ui 的 35 项单测及平台测试、注释检查通过；探针源码与独立构建目录已删除。

tasks/reviews/D-08-r3.md

@@ -0,0 +1 @@
+PASS