lix/mnote
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
Files
main
mnote/bugs/05-editor-mainline/done/2026-05-21-workbench-filetree-office-slash-bugs-review-v1.md
T
lix-2026 eba1010191 fix local office resource editing 
- add local-folder OnlyOffice sign/callback writeback and edit-tab handling

- align main resource tabs, attachment edit menu, slash isolation, and filetree context behavior

- record Sidex/Hermes gap reviews and Reasonix task checklists
2026-05-21 05:40:06 +08:00

7.4 KiB
Raw Permalink Blame History

Workbench Filetree / Office / Slash Bug Review V1

日期：2026-05-21

范围

本轮覆盖用户截图 23-26 中的 6 个 P0/P1 缺陷：

  1. 文件树删除页面后，页面进入资源垃圾箱，页面垃圾箱没有使用。
  2. 主编辑区 tab 标题与实际打开页面标题不一致。
  3. 上传 docx 后页面卡死，docx 打不开。
  4. 不应自动出现“初始化的新页面”；空白工作区应保持空白。
  5. 文件树右键菜单大量动作不可用，需对照 Sidex / VSCode 文件资源管理器收敛到可用动作。
  6. 主栏打开 slash 菜单时，侧栏也出现 slash 菜单。

当前证据

  • rust/crates/mnote-web/src/routes/local_folder_source.rs
    • ensure_default_workspace_home_page 会创建 初始化的新页面/初始化的新页面.md。
    • create_default_local_workspace_for_actor_at_base_creates_manifest_and_root_only 仍断言该默认页面存在。
    • trash_local_entry 先解析 local-dir:*，再解析 local-file:*，最后才走 trash_local_markdown_page。
    • 文件树中页面 bundle 目录在 scan_directory 中是 rowKind=folder，因此删除会走 trash_local_directory，resourceKind=local_directory。
  • rust/crates/mnote-web/src/routes/web_shell.rs
    • replacePaneDocument / updatePaneChrome 会更新正文、面包屑、document.title，但没有同步 [data-mnote-main-tab="page"] .mnote-main-tab-title。
    • openUnsupportedSideTarget 会替换 secondary root 为不支持占位；positionSlashMenuForRoot 仍允许 fallback 到 document.querySelector('[data-testid="mnote-leptos-tiptap-slash-menu"]')，存在跨 pane 定位/展示风险。
    • openResourceAsSideTarget 对 markdown/text/code/office 全部进入 unsupported，占位会污染侧栏；这与用户期望“不可用动作不要误打开侧栏”冲突。
  • rust/crates/mnote-web/src/ssr/pages/layout.rs
    • 文件树右键菜单直接展示 open-right / new-file / new-folder / paste / refresh / collapse / reveal / duplicate / rename / delete，其中部分 disabled 或仅 dispatch event，无真实实现。
    • openConvexAssetFromFileTree 对 local office 可走主 tab 的 iframe，但 Convex office 仍直接 window.open(buildOnlyOfficeOpenUrl(...))。
    • 附件上传 insertUploadedAssetIntoEditor 和 enhanceEditorAttachmentLink 已尝试统一附件 UI；docx 卡死需要先确认是上传阶段阻塞、OnlyOffice iframe 回源失败，还是 active-tab iframe 加载导致主页面阻塞。
  • Sidex / VSCode 参考：
    • Sidex 保留 VSCode 的 MenuId.ExplorerContext 菜单体系，文件树动作来源在 reference-code/sidex-main/src/vs/workbench/contrib/files/browser/fileActions.contribution.ts。
    • 可对照的基础动作是 Open to the Side、New File、New Folder、Reveal、Copy Path、Rename、Delete；动作是否显示/启用由 context key 决定，不是把所有潜在动作硬塞进菜单。
    • Sidex workspace 层有 FileClipboard、InlineInputKind::{Rename, NewFile, NewFolder} 等状态，说明新建/重命名应走内联输入或明确命令，而不是菜单项只发事件。

修改策略

  • Bug1 / Bug4 属于本地文件夹 tree/source 口径问题，优先修 Rust kernel-facing 路径：
    • 默认工作区不再创建“初始化的新页面”。
    • bundle 页面删除必须以 Markdown 页面生命周期入 trash，页面垃圾箱可识别，不应按 local_directory 资源归类。
    • 空 root page tree 可返回空列表；UI 不渲染占位页面。
  • Bug2 属于主编辑区 tab chrome 同步：
    • updatePaneChrome(primary) 同步 page tab 标题、data-document-id 和 active 状态。
    • 页面标题保存事件也应更新 page tab。
  • Bug3 属于 Office 打开链路：
    • 上传本地 docx 后，主 tab 打开应使用 /onlyoffice?...fileUrl=/api/local-folder/files/open...，并避免在主线程同步读取/解析大文件。
    • 如果 OnlyOffice DocumentServer 不可连，应显示非阻塞错误，不应卡死整个页面。
  • Bug5 属于菜单可用性：
    • 菜单只显示/启用已有真实实现。
    • “在右侧边栏打开”只对已支持侧栏打开的页面显示；资源侧栏未支持时不要打开 unsupported placeholder。
    • Copy Path 对 local folder 应复制真实 relative path，不是复制标题。
    • New Folder 在 local_folder 下应接入 create_folder，否则隐藏。
  • Bug6 属于 editor host 隔离：
    • Slash 菜单定位只能操作当前 root 内的菜单，不能 fallback 到别的 pane 的菜单。
    • 不支持侧栏资源时应清理 secondary editor/slash overlay，或完全不打开侧栏。

不做事项

  • 不回滚当前工作区已有 ACP/Hermes 改动。
  • 不把资源 tab、side pane、OnlyOffice 做成全新架构；只修当前 P0 可见缺陷。
  • 不复活旧 React / BlockNote 运行时。
  • 不把 Sidex 菜单实现照搬成通用命令注册系统；本轮只借鉴“按可用能力显示动作”的原则。

验收

  • Rust targeted tests 至少覆盖：
    • local workspace 默认初始化不创建“初始化的新页面”。
    • 删除页面 bundle 进入 page/document trash，而不是 local_directory resource trash。
    • 主 page tab 随 active primary document 标题更新。
    • local docx onlyoffice URL / content-type / proxy 回源可达。
    • 文件树菜单不可用动作隐藏或禁用，已显示动作有真实 handler。
    • slash menu 不跨 primary/secondary/resource pane 显示。
  • UI 类最终必须做真实浏览器或 smoke 验证，并保留截图/输出。

本轮执行结果

  • 已完成并移动到 done/：
    • bugs/04-tree-domain/done/4-45-local-folder-page-trash-and-empty-workspace-v1.md
    • bugs/05-editor-mainline/done/5-28-main-page-tab-title-active-sync-v1.md
    • bugs/05-editor-mainline/done/5-29-onlyoffice-docx-upload-open-freeze-v1.md
    • bugs/05-editor-mainline/done/5-30-filetree-context-menu-sidex-alignment-v1.md
    • bugs/05-editor-mainline/done/5-31-side-pane-slash-overlay-isolation-v1.md
  • Rust 验证：
    • cargo test -p mnote-web -- --test-threads=1：481 passed。
    • 新增 local_tree_command_delete_page_bundle_directory_uses_page_trash，锁定 local-dir:* 页面 bundle 删除进入 markdown_bundle 页面垃圾箱。
  • Smoke / 浏览器验证：
    • node scripts/task463-onlyoffice-resolver-smoke.js：通过。
    • node scripts/task462-resource-tab-error-placeholder-smoke.js：通过。
    • node scripts/task455-local-folder-mindmap-clean-smoke.js：通过。
    • node scripts/task458-local-create-page-no-conflict-smoke.js：通过。
    • node scripts/task464-local-folder-resource-trash-ui-smoke.js：通过。
    • node scripts/task472-side-target-secondary-pane-smoke.js：通过。
  • 真实浏览器证据：
    • tmp/bug-20260521-workbench-browser-verify/result-title-docx-menu.json
    • tmp/bug-20260521-workbench-browser-verify/result-slash.json
    • tmp/bug-20260521-workbench-browser-verify/result-trash.json
    • 截图覆盖：主 tab 标题同步、真实 docx resource tab、资源右键菜单、primary/secondary slash 隔离、页面 bundle 垃圾箱分类。
  • 历史 smoke 注意：
    • node scripts/task432-filetree-trash-page-dual-browser-no-refresh-smoke.js 当前失败在 convex-source 双浏览器 live delta 未插入新建子页；失败点早于本轮页面删除/垃圾箱断言，不代表本地页面 bundle 垃圾箱修复失败。后续应按实时同步历史 smoke 单独修复或退役。
Reference in New Issue View Git Blame Copy Permalink