lix/mnote
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: split tree shell renderer

Browse Source
This commit is contained in:
lix-2026
2026-05-26 03:51:06 +08:00
parent 0f0e37ed54
commit c65135f7ed
6 changed files with 1126 additions and 916 deletions
Download Patch File Download Diff File Expand all files Collapse all files
design/10-review/process/16-mnote-web-runtime-module-maintainability-checklist-v1.md
+1 -1
View File
@@ -210,7 +210,7 @@ cargo test --manifest-path rust/Cargo.toml -p mnote-web --lib ssr::pages::layout
- [x] D4. `tree-shell-picker-runtime.js`：picker search/focus/pick root。
- [x] D5. `tree-shell-dom-runtime.js`：DOM patch/render helpers。
- [x] D6. `tree-shell-icons-runtime.js`：icon templates 和 resource kind badge。
- [ ] D7. entrypoint 少于 2,500 行。
- [x] D7. entrypoint 少于 2,500 行。
验收命令：
rust/crates/mnote-web/browser/tree-shell-render-runtime.js
+1002
View File
File diff suppressed because it is too large Load Diff
rust/crates/mnote-web/browser/tree-shell-runtime.js
+93 -908
View File
File diff suppressed because it is too large Load Diff
rust/crates/mnote-web/src/routes/mod.rs
+5
View File
@@ -166,6 +166,10 @@ pub fn build_router(state: AppState) -> Router {
"/api/mnote-browser-runtime/tree-shell-runtime.js",
get(web_shell::tree_shell_runtime_asset),
)
.route(
"/api/mnote-browser-runtime/tree-shell-render-runtime.js",
get(web_shell::tree_shell_render_runtime_asset),
)
.route(
"/api/mnote-browser-runtime/tree-shell-page-runtime.js",
get(web_shell::tree_shell_page_runtime_asset),
@@ -649,6 +653,7 @@ mod tests {
"/api/mnote-browser-runtime/sidebar-tree-runtime.js",
"/api/mnote-browser-runtime/tree-live-controller.js",
"/api/mnote-browser-runtime/tree-shell-runtime.js",
"/api/mnote-browser-runtime/tree-shell-render-runtime.js",
"/api/mnote-browser-runtime/tree-shell-page-runtime.js",
"/api/mnote-browser-runtime/tree-shell-state-runtime.js",
"/api/mnote-browser-runtime/tree-shell-icons-runtime.js",
rust/crates/mnote-web/src/routes/tree.rs
+11 -7
View File
@@ -2393,6 +2393,8 @@ pub async fn reduce_tree_shell_runtime(
#[cfg(test)]
mod tests {
const TREE_SHELL_RUNTIME_JS: &str = include_str!("../../browser/tree-shell-runtime.js");
const TREE_SHELL_RENDER_RUNTIME_JS: &str =
include_str!("../../browser/tree-shell-render-runtime.js");
const TREE_SHELL_FILETREE_RUNTIME_JS: &str =
include_str!("../../browser/tree-shell-filetree-runtime.js");
const TREE_SHELL_FILETREE_MENU_RUNTIME_JS: &str =
@@ -2542,7 +2544,7 @@ mod tests {
assert!(html.contains("tree-create-root"));
assert!(TREE_SHELL_RUNTIME_JS.contains("tree.ready"));
assert!(html.contains("test-shell"));
assert!(TREE_SHELL_RUNTIME_JS.contains("tree-action-menu"));
assert!(TREE_SHELL_RENDER_RUNTIME_JS.contains("tree-action-menu"));
assert!(TREE_SHELL_RUNTIME_JS.contains("tree.page.context-menu"));
assert!(TREE_SHELL_RUNTIME_JS.contains("tree.page.expand.changed"));
assert!(TREE_SHELL_RUNTIME_JS.contains("tree.page.focus.changed"));
@@ -2675,13 +2677,15 @@ mod tests {
assert!(TREE_SHELL_RUNTIME_JS.contains("tree.filetree.selection.changed"));
assert!(html.contains("\"contractName\":\"rust_filetree_selection_reducer_v1\""));
assert!(TREE_SHELL_RUNTIME_JS.contains("applyFileTreeSelectionAction"));
assert!(TREE_SHELL_RUNTIME_JS.contains("tree.filetree.internal-drop"));
assert!(TREE_SHELL_RUNTIME_JS.contains("tree.filetree.external-drop"));
assert!(TREE_SHELL_RENDER_RUNTIME_JS.contains("tree.filetree.internal-drop"));
assert!(TREE_SHELL_RENDER_RUNTIME_JS.contains("tree.filetree.external-drop"));
assert!(html.contains("\"rowKind\":\"asset_folder\""));
assert!(html.contains("\"resourceMeta\""));
assert!(TREE_SHELL_RUNTIME_JS.contains("tree-shell-render-runtime.js"));
assert!(TREE_SHELL_RUNTIME_JS.contains("tree-shell-filetree-runtime.js"));
assert!(TREE_SHELL_RUNTIME_JS.contains("tree-shell-filetree-menu-runtime.js"));
assert!(TREE_SHELL_RUNTIME_JS.contains("tree-shell-filetree-dnd-runtime.js"));
assert!(TREE_SHELL_RENDER_RUNTIME_JS.contains("function createTreeShellRenderer(context)"));
assert!(
TREE_SHELL_FILETREE_RUNTIME_JS.contains("function getFileTreeRowOwnerDocumentId(item)")
);
@@ -2689,14 +2693,14 @@ mod tests {
.contains("function buildFileTreeMenuTarget(context"));
assert!(TREE_SHELL_FILETREE_DND_RUNTIME_JS
.contains("function createTreeShellFileTreeDndRuntime(context)"));
assert!(TREE_SHELL_RUNTIME_JS
assert!(TREE_SHELL_RENDER_RUNTIME_JS
.contains("row.dataset.ownerDocumentId = ownerDocumentId || \"\";"));
assert!(
TREE_SHELL_RUNTIME_JS.contains("if (rowId && getFileTreeRowDocumentId(renameItem))")
);
assert!(TREE_SHELL_RUNTIME_JS.contains("if (getFileTreeRowDocumentId(item))"));
assert!(TREE_SHELL_RUNTIME_JS.contains("documentId: ownerDocumentId || null"));
assert!(TREE_SHELL_RUNTIME_JS.contains("dragover"));
assert!(TREE_SHELL_RENDER_RUNTIME_JS.contains("if (getFileTreeRowDocumentId(item))"));
assert!(TREE_SHELL_RENDER_RUNTIME_JS.contains("documentId: ownerDocumentId || null"));
assert!(TREE_SHELL_RENDER_RUNTIME_JS.contains("dragover"));
assert!(TREE_SHELL_RUNTIME_JS.contains("hydrateInitialFileTree"));
assert!(TREE_SHELL_RUNTIME_JS
.contains("const usedRustInitialRenderer = hydrateInitialRenderer();"));
rust/crates/mnote-web/src/routes/web_shell.rs
+14
View File
@@ -938,6 +938,20 @@ pub async fn tree_shell_runtime_asset() -> Response {
.unwrap_or_else(|_| Response::new(Body::empty()))
}
pub async fn tree_shell_render_runtime_asset() -> Response {
const JS: &str = include_str!("../../browser/tree-shell-render-runtime.js");
Response::builder()
.status(StatusCode::OK)
.header(
header::CONTENT_TYPE,
"application/javascript; charset=utf-8",
)
.header(header::CACHE_CONTROL, "no-store")
.header(HEADER_MNOTE_WEB_OWNER, "mnote-web")
.body(Body::from(JS))
.unwrap_or_else(|_| Response::new(Body::empty()))
}
pub async fn tree_shell_page_runtime_asset() -> Response {
const JS: &str = include_str!("../../browser/tree-shell-page-runtime.js");
Response::builder()