refactor: extract drop indicator geometry

2026-05-25 05:06:28 +08:00
parent 63d1ea7e47
commit dddf9eb9c1
5 changed files with 61 additions and 12 deletions
@@ -57,6 +57,7 @@ UI / 浏览器可见项必须有真实浏览器截图或结构化 smoke 证据

 - Reasonix 结果必须先查 Hindsight `reasonix` recall，再读 `process-handoff.md/json`、`result.json`、diff 和验证证据。
 - 只采纳限定文件范围内的最小 diff；越界重构、回滚用户改动、修改 git 状态的一律拒绝。
+- 实现 worker 必须在独立 worktree 产出候选 patch；Codex 主工作区只维护事实源、复核证据和最终选择性合入。
 - browser smoke worker 必须 `modified_files=[]`。
 - 本文件只记录真实完成项；未验证项保持未勾选。

@@ -144,6 +145,7 @@ UI / 浏览器可见项必须有真实浏览器截图或结构化 smoke 证据
 - 2026-05-25：Batch J Codex 复核结论：Worker C / D 对应边界已在当前 HEAD 中具备，未重复合入新 diff。Worker C 已验证 `node --check rust/crates/mnote-web/browser/filetree-runtime.js`、`cargo check --manifest-path rust/Cargo.toml -p mnote-web`、`cargo test --manifest-path rust/Cargo.toml -p mnote-web filetree_runtime_contains_row_accessor_helpers -- --test-threads=1`；Worker D 已验证 `node --check rust/crates/mnote-web/browser/document-conflict-panel-runtime.js`、`cargo check --manifest-path rust/Cargo.toml -p mnote-web`、`cargo test --manifest-path rust/Cargo.toml -p mnote-web document_conflict_panel_runtime_contains_dom_helpers -- --test-threads=1`。
  - 主工作区 smoke：`NODE_PATH=/mnt/Data1T/mnote/node_modules node scripts/task479-local-folder-markdown-resource-lifecycle-smoke.js` 通过，截图证据 `/tmp/mnote-task479-lifecycle-xlAZrs/screenshots-1779650673787/final-state.png`；`NODE_PATH=/mnt/Data1T/mnote/node_modules node scripts/task451-local-markdown-conflict-resolution-ui-smoke.js` 通过，结果 `/mnt/Data1T/mnote/tmp/task451-local-markdown-conflict-resolution-ui-smoke/result.json`。
 - 2026-05-25：当前 goal 明确为每批最多 2 个 worker；后续 3-20 派发以 2 个为硬上限，历史记录中超过 2 个 worker 的批次保留为异常记录。
+- 2026-05-25：用户纠正 B/C/D 等实现任务也应在新的 worktree 中修改后再由 Codex 合并；主控已采用该口径：实现 worker 一律独立 worktree，主工作区不再直接扩大实现 diff，只保留事实源、复核和最终采纳。
 - 2026-05-25：Batch L Worker B `reasonix-2026-05-24T19-30-30-509Z-18f62949` 只读审计 `refreshLocalFolderSidebarSnapshot`；Codex 已读取 Hindsight recall、`process-handoff.md/json`、`result.json`，并核对 worktree diff 为空。
  - 审计结论：`refreshLocalFolderSidebarSnapshot` 仍是 shell 编排，包含 workspace/root/document 状态、两条 projection fetch、render 后副作用和 watcher 标记，不应整体迁入 `filetree-runtime.js`。
  - 下一刀建议：优先迁投影数据读取器层（如 `readProjection`、`readDatasetProjection`、`projectionItems`、row/id/title/file accessor、`groupRowsByParent`），或更保守地先迁未使用的 `replaceSidebarTreeFromDocument`；Codex 本轮不直接实现 3-20 代码，只记录候选。
@@ -56,7 +56,9 @@ UI / 浏览器可见项必须有真实浏览器截图或结构化 smoke 证据
 ## 5. 采纳规则

 - Reasonix 结果必须先查 Hindsight `reasonix` recall，再读 `process-handoff.md/json`、`result.json`、diff 和验证证据。
- 实现 worker 只能改允许范围；只读和 smoke worker 必须没有源码 diff。
+- 实现 worker 必须在独立 worktree 产出候选 patch；只读和 smoke worker 必须没有源码 diff。
+- Codex 主工作区只维护事实源、复核证据和最终选择性合入，不直接承接 B/C/D 等实现 worker 的原始修改过程。
+- 实现 worker 只能改允许范围。
 - `lib.rs` 不能新增附件/history/DOM bridge 大块逻辑。
 - 本文件只记录真实完成项；未验证项保持未勾选。

@@ -135,6 +137,7 @@ UI / 浏览器可见项必须有真实浏览器截图或结构化 smoke 证据
  - 已验证：在独立 worktree 中运行 `cargo fmt --manifest-path rust/Cargo.toml --all --check`、`cargo check --manifest-path rust/spikes/leptos-tiptap-spike/Cargo.toml`、wasm release build、`wasm-bindgen`；主工作区复跑 `NODE_PATH=/mnt/Data1T/mnote/node_modules node scripts/task489-block-menu-delete-undo-smoke.js`、`NODE_PATH=/mnt/Data1T/mnote/node_modules node scripts/task488-local-attachment-link-delete-undo-smoke.js` 均通过，结果分别位于 `/mnt/Data1T/mnote/tmp/task489-block-menu-delete-undo-smoke/result.json`、`/mnt/Data1T/mnote/tmp/task488-local-attachment-link-delete-undo-smoke/result.json`。
  - 未完成：`try_sync_editor_overlay_state`、`on_selection_change` overlay 同步、block menu signal 编排仍未迁出；本轮未扩大迁移边界。
 - 2026-05-25：当前 goal 明确为每批最多 2 个 worker；后续 Batch 计划和 Reasonix 派发以 2 个为硬上限，历史记录中超过 2 个 worker 的批次保留为异常记录。
+- 2026-05-25：用户纠正 B/C/D 等实现任务也应在新的 worktree 中修改后再由 Codex 合并；主控已采用该口径：实现 worker 一律独立 worktree，主工作区不再直接扩大实现 diff，只保留事实源、复核和最终采纳。
 - 2026-05-25：Batch K 只派发 1 个只读 Reasonix worker 审计 `close_editor_floating_overlays` 迁出边界，使用独立 worktree `/mnt/Data1T/mnote-worktrees/0525-k-5-29-close-overlays-readonly`，run id `reasonix-2026-05-24T19-18-51-566Z-90f11977`，并显式使用 `--no-memory-recall` 降低任务串线风险。
  - Codex 已读取 Hindsight recall、`process-handoff.md/json`、`result.json`，并核对 worktree `git status --short` / `git diff --stat` 为空。
  - 审计结论：先把 `ImageToolbarAnchor`、`TableOverlayAnchor`、`TableSelectionKind`、`TableSelectionOverlayState` 迁入 `editor_runtime/overlays.rs`，再迁出纯 WriteSignal setter `close_editor_floating_overlays`；`close_editor_floating_overlays_if_escape`、`open_slash_menu_overlay`、`open_image_toolbar_overlay`、`open_block_menu_overlay`、`try_sync_editor_overlay_state` 和 `on_selection_change` overlay 同步继续留在 `lib.rs`。
@@ -172,3 +175,11 @@ UI / 浏览器可见项必须有真实浏览器截图或结构化 smoke 证据
  - 已验证：`cargo fmt --manifest-path rust/Cargo.toml --all --check`、`cargo check --manifest-path rust/spikes/leptos-tiptap-spike/Cargo.toml`、wasm release build、`wasm-bindgen`、`MNOTE_WEB_SMOKE_BASE_URL=http://127.0.0.1:3001 NODE_PATH=/mnt/Data1T/mnote/node_modules node scripts/task489-block-menu-delete-undo-smoke.js`、`MNOTE_WEB_SMOKE_BASE_URL=http://127.0.0.1:3001 NODE_PATH=/mnt/Data1T/mnote/node_modules node scripts/task488-local-attachment-link-delete-undo-smoke.js`、`MNOTE_WEB_SMOKE_BASE_URL=http://127.0.0.1:3001 NODE_PATH=/mnt/Data1T/mnote/node_modules node scripts/task479-local-folder-markdown-resource-lifecycle-smoke.js`。
  - 环境阻塞：`MNOTE_UI_BASE_URL=http://127.0.0.1:3001 NODE_PATH=/mnt/Data1T/mnote/node_modules node scripts/task158-e30-menu-state-smoke.js` 在创建 cloud 临时文档时返回 `503 convex_unavailable`，当前 3001 服务未配置 Convex mutation；本地文件夹型 editor smoke 已通过。
  - 未迁：`on_selection_change` overlay 同步、`drop_indicator_from_target`、拖拽 listener、block reorder 行为和 selection bridge 仍留在 `lib.rs`。
+- 2026-05-25：Batch T 只派发 1 个只读 Reasonix worker 审计 `drop_indicator_from_target` 边界，使用独立 worktree `/mnt/Data1T/mnote-worktrees/0525-t-worker-a-5-29-drop-indicator-readonly`，run id `reasonix-2026-05-24T20-56-16-647Z-35de7d21`，并显式使用 `--no-memory-recall`。
+  - Codex 已读取 Hindsight recall、`process-handoff.md/json`、`result.json`、`final.md`，并核对 worktree `git status --short` / `git diff --stat` 为空；Reasonix 未修改源码。runner 完成后残留父进程未自行退出，主控已终止。
+  - 审计结论经 Codex 复核后采纳：`drop_indicator_from_target` 不能整体迁入 `editor_runtime/block_dnd.rs`，因为仍依赖 `hovered_block_from_target`、`target_element`、`direct_block_from_element`、`top_level_block_index`、`block_state_from_index`、`HANDLE_SHELL_SELECTOR` 等 `lib.rs` DOM -> block 状态解析链。
+  - Codex 主控完成最小几何切片：`block_dnd.rs` 新增 `drop_indicator_from_geometry(block_index, block_top, block_height, stage_top, client_y)` 和单元测试；`lib.rs::drop_indicator_from_target` 仍负责 DOM 查询与 hovered block 解析，只委托新 helper 计算 `DropIndicatorState`。
+  - 修改：`rust/spikes/leptos-tiptap-spike/src/editor_runtime/block_dnd.rs`、`rust/spikes/leptos-tiptap-spike/src/lib.rs`、generated island wasm。
+  - 已验证：`cargo fmt --manifest-path rust/Cargo.toml --all --check`、`cargo check --manifest-path rust/spikes/leptos-tiptap-spike/Cargo.toml`、wasm release build、`wasm-bindgen`、`MNOTE_WEB_SMOKE_BASE_URL=http://127.0.0.1:3001 NODE_PATH=/mnt/Data1T/mnote/node_modules node scripts/task489-block-menu-delete-undo-smoke.js`、`MNOTE_WEB_SMOKE_BASE_URL=http://127.0.0.1:3001 NODE_PATH=/mnt/Data1T/mnote/node_modules node scripts/task488-local-attachment-link-delete-undo-smoke.js`、`MNOTE_WEB_SMOKE_BASE_URL=http://127.0.0.1:3001 NODE_PATH=/mnt/Data1T/mnote/node_modules node scripts/task479-local-folder-markdown-resource-lifecycle-smoke.js`。
+  - 单元测试阻塞：`cargo test --manifest-path rust/spikes/leptos-tiptap-spike/Cargo.toml block_dnd -- --test-threads=1` 被既有 `STYLE` 测试编译错误阻断，未进入本次新增 helper 测试；该阻断在此前 `block_dnd` 批次已存在。
+  - 未迁：`hovered_block_from_target`、`drop_indicator_from_target`、`drop_indicator_from_point`、拖拽 listener、block reorder 行为和 DOM -> block 状态解析链仍留在 `lib.rs`。
@@ -3,7 +3,7 @@
 //! 本模块只承接拖拽命中所需的窄 helper，不处理 Leptos signal，
 //! 也不分发 editor command；调用方负责保留编辑器状态编排。

-use crate::editor_runtime::block_hover_state::DropPlacement;
+use crate::editor_runtime::block_hover_state::{DropIndicatorState, DropPlacement};
 use web_sys::{window, EventTarget};

 pub(crate) struct HandleCorridorGeometry {
@@ -57,6 +57,23 @@ pub(crate) fn drop_indicator_top(
    }
 }

+pub(crate) fn drop_indicator_from_geometry(
+    block_index: usize,
+    block_top: f64,
+    block_height: f64,
+    stage_top: f64,
+    client_y: i32,
+) -> DropIndicatorState {
+    let placement = drop_placement_from_block_point(client_y, block_top, block_height);
+    let block_bottom = block_top + block_height;
+    let top = drop_indicator_top(stage_top, block_top, block_bottom, placement);
+    DropIndicatorState {
+        index: block_index,
+        top,
+        placement,
+    }
+}
+
 pub(crate) fn event_target_from_point(client_x: i32, client_y: i32) -> Option<EventTarget> {
    window()
        .and_then(|win| win.document())
@@ -97,4 +114,24 @@ mod tests {
            DropPlacement::After
        );
    }
+
+    #[test]
+    fn drop_indicator_geometry_returns_stage_relative_top() {
+        assert_eq!(
+            drop_indicator_from_geometry(3, 140.0, 40.0, 100.0, 150),
+            DropIndicatorState {
+                index: 3,
+                top: 40.0,
+                placement: DropPlacement::Before,
+            }
+        );
+        assert_eq!(
+            drop_indicator_from_geometry(3, 140.0, 40.0, 100.0, 161),
+            DropIndicatorState {
+                index: 3,
+                top: 80.0,
+                placement: DropPlacement::After,
+            }
+        );
+    }
 }
@@ -22,8 +22,8 @@ use web_sys::{

 use editor_runtime::attachment_upload::dispatch_editor_upload_request;
 use editor_runtime::block_dnd::{
-    drop_indicator_top, drop_placement_from_block_point, event_target_from_point,
-    pointer_in_handle_corridor_geometry, HandleCorridorGeometry,
+    drop_indicator_from_geometry, event_target_from_point, pointer_in_handle_corridor_geometry,
+    HandleCorridorGeometry,
 };
 use editor_runtime::block_menu_overlay;
 use editor_runtime::block_menu_legacy_html::{
@@ -5419,14 +5419,13 @@ fn drop_indicator_from_target(
    let stage = editor_stage_element()?;
    let block_rect = block.get_bounding_client_rect();
    let stage_rect = stage.get_bounding_client_rect();
-    let placement = drop_placement_from_block_point(client_y, block_rect.top(), block_rect.height());
-    let top = drop_indicator_top(stage_rect.top(), block_rect.top(), block_rect.bottom(), placement);
-
-    Some(DropIndicatorState {
-        index: hovered.index,
-        top,
-        placement,
-    })
+    Some(drop_indicator_from_geometry(
+        hovered.index,
+        block_rect.top(),
+        block_rect.height(),
+        stage_rect.top(),
+        client_y,
+    ))
 }

 fn drop_indicator_from_point(client_x: i32, client_y: i32) -> Option<DropIndicatorState> {