@hydraharness/harness-client-ui-layout 0.1.1-rc.8 → 0.1.1-rc.9

package/README.md

@@ -14,9 +14,11 @@ AppFrame always mounts the conversation and details columns; a connected Session
 
 In Electron, `Ctrl+P` opens the Files tab and focuses its filter for the current registered Workspace; without one, Files stays empty. The Explorer features a resizable splitter (160px–480px) and three collapsible sections: Open Editors with dirty indicators and Save All/Close All actions, Workspace Files with folders first in natural name order, color-coded extension icons, search filter, inline creation, and drag-and-drop moves, and an Outline section extracting code symbols (functions, classes, interfaces, types, headings) with click-to-jump line navigation.
 
-Selecting an eligible UTF-8 text file opens the in-panel editor with multi-document tabs and breadcrumbs path navigation. The editor provides a line numbers gutter with active line highlighting, Word Wrap toggling (`Alt+Z`), Go to Line (`Ctrl+G`), Diff with disk view, Prettier formatting (`Shift+Alt+F`), atomic version-checked saves (`Ctrl+S`), and a status bar displaying cursor coordinates, selection count, line count, indentation, encoding, line endings, and language mode. Drafts above 50,000 characters stay plain to keep editing responsive, and opening another file, closing a tab, or switching Workspace guards unsaved changes.
+Selecting an eligible UTF-8 text file opens the in-panel editor with multi-document tabs and breadcrumbs path navigation. The editor provides a line numbers gutter with active line highlighting, Word Wrap toggling (`Alt+Z`), Go to Line (`Ctrl+G`), Diff with disk view, Prettier formatting (`Shift+Alt+F`), atomic version-checked saves (`Ctrl+S`), and a status bar displaying cursor coordinates, selection count, line count, indentation, encoding, line endings, and language mode. Drafts above 50,000 characters stay plain to keep editing responsive, and opening another file, closing a tab, or switching Workspace guards unsaved changes. Save All acknowledges each successful file separately, preserves newer typing and closed tabs, and stops on the first failure with remaining drafts intact.
 
-The renderer sends only an opaque Workspace id. Electron re-resolves that id through the Host, validates the IPC sender, canonicalizes and confines every list, search, create, read, format, and save path to the registered root, excludes symlink traversal, accepts only one validated leaf name for creation, and creates without overwriting an existing entry. Editable content must be valid non-binary UTF-8 and no larger than 1 MB. Confinement and version checks reject stable outside paths and stale states; they are not an operating-system sandbox or an atomic compare-and-swap against an external process that replaces a path during the final filesystem operation. This remains a bounded textarea editor: it does not provide Monaco, per-file editor tabs, language diagnostics, or persisted edit buffers.
+Context-menu creation uses the addressed folder. Renaming or moving an entry updates descendant editor paths, expanded folders, and the creation target; deleting an entry removes its cached subtree and returns selection to its parent. A relocated file advances the editor's disk version only when it matches the version immediately before relocation; external edits still cause a save conflict. Rename, move, and delete run one at a time and wait for saving, formatting, and creation to finish. Responses from a previous Workspace cannot update the current tree or errors. A successful rename, move, or delete clears the file filter to display the updated tree.
+
+The renderer sends only an opaque Workspace id. Electron re-resolves that id through the Host, validates the IPC sender, canonicalizes and confines every list, search, create, rename, move, delete, read, format, and save path to the registered root, excludes symlink traversal, accepts only one validated leaf name for creation, and creates without overwriting an existing entry. Editable content must be valid non-binary UTF-8 and no larger than 1 MB. Confinement and version checks reject stable outside paths and stale states; they are not an operating-system sandbox or an atomic compare-and-swap against an external process that replaces a path during the final filesystem operation. This remains a bounded textarea editor: it does not provide Monaco, language diagnostics, or persisted edit buffers.
 
 The `/client` exports are the plugin body (`apply`/`inject`), `LayoutController`, and the four owner-share interfaces. AppFrame, the panel store, and the concession solver remain package-internal.
 

package/lib/client.js

@@ -420,7 +420,7 @@ window.__ModuleLoader__.load({
 			const savingRef = (0, react.useRef)(false);
 			const formattingRef = (0, react.useRef)(false);
 			const creatingRef = (0, react.useRef)(false);
-			const movingRef = (0, react.useRef)(false);
+			const mutatingRef = (0, react.useRef)(false);
 			const mountedRef = (0, react.useRef)(true);
 			const rootWorkspaceRef = (0, react.useRef)();
 			const documentRef = (0, react.useRef)();
@@ -475,11 +475,19 @@ window.__ModuleLoader__.load({
 			const [creating, setCreating] = (0, react.useState)(false);
 			const [saving, setSaving] = (0, react.useState)(false);
 			const [formatting, setFormatting] = (0, react.useState)(false);
-			const [moving, setMoving] = (0, react.useState)(false);
+			const [mutating, setMutating] = (0, react.useState)(false);
 			const [draggedPath, setDraggedPath] = (0, react.useState)();
 			const [dropTargetPath, setDropTargetPath] = (0, react.useState)();
 			const deferredDraft = (0, react.useDeferredValue)(document?.draft);
 			const dirty = documentIsDirty(document);
+			const publishDocuments = (0, react.useCallback)((nextList) => {
+				openDocumentsRef.current = nextList;
+				setOpenDocuments(nextList);
+				const nextActiveDoc = nextList.find((d) => d.path === activePathRef.current);
+				documentRef.current = nextActiveDoc;
+				setDocument(nextActiveDoc);
+				props.onDirtyChange?.(nextList.some((d) => documentIsDirty(d)));
+			}, [props.onDirtyChange]);
 			const updateDocument = (0, react.useCallback)((update) => {
 				if (!mountedRef.current) return;
 				const next = update(openDocumentsRef.current.find((d) => d.path === activePathRef.current));
@@ -498,13 +506,32 @@ window.__ModuleLoader__.load({
 					activePathRef.current = next.path;
 					setActivePath(next.path);
 				}
-				openDocumentsRef.current = nextList;
-				setOpenDocuments(nextList);
-				const nextActiveDoc = nextList.find((d) => d.path === activePathRef.current);
-				documentRef.current = nextActiveDoc;
-				setDocument(nextActiveDoc);
-				props.onDirtyChange?.(nextList.some((d) => documentIsDirty(d)));
-			}, [props.onDirtyChange]);
+				publishDocuments(nextList);
+			}, [publishDocuments]);
+			const remapEntryPaths = (oldPath, newPath, version) => {
+				const remap = (path) => samePath(path, oldPath) || pathContains(oldPath, path) ? `${newPath}${path.slice(oldPath.length)}` : path;
+				if (activePathRef.current !== void 0) {
+					activePathRef.current = remap(activePathRef.current);
+					setActivePath(activePathRef.current);
+				}
+				publishDocuments(openDocumentsRef.current.map((d) => ({
+					...d,
+					path: remap(d.path),
+					version: samePath(d.path, oldPath) && version !== void 0 && d.version === version.before ? version.after : d.version
+				})));
+				setChildren((current) => Object.fromEntries(Object.entries(current).map(([path, entries]) => [remap(path), entries.map((entry) => ({
+					...entry,
+					path: remap(entry.path),
+					name: fileName(remap(entry.path))
+				}))])));
+				setExpanded((current) => new Set([...current].map(remap)));
+				setSelectedDirectory((current) => current === void 0 ? current : remap(current));
+				setCreateDraft((current) => current === void 0 ? current : {
+					...current,
+					parent: remap(current.parent)
+				});
+				setQuery("");
+			};
 			const loadDirectory = (0, react.useCallback)(async (path) => {
 				if (api === void 0 || rootPath === void 0 || workspaceId === void 0 || rootWorkspaceRef.current !== workspaceId || !mountedRef.current) return;
 				const generation = listGeneration.current;
@@ -668,7 +695,7 @@ window.__ModuleLoader__.load({
 				if (opening && children[path] === void 0) loadDirectory(path);
 			};
 			const openFile = (0, react.useCallback)(async (path) => {
-				if (api === void 0 || rootPath === void 0 || workspaceId === void 0 || savingRef.current || !mountedRef.current) return;
+				if (api === void 0 || rootPath === void 0 || workspaceId === void 0 || savingRef.current || mutatingRef.current || !mountedRef.current) return;
 				const previous = documentRef.current;
 				if (previous?.path === path) return;
 				if (documentIsDirty(previous) && !window.confirm("Discard unsaved changes and open another file?")) return;
@@ -748,18 +775,18 @@ window.__ModuleLoader__.load({
 				document,
 				updateDocument
 			]);
-			const startCreate = (kind) => {
-				if (rootPath === void 0 || creatingRef.current) return;
+			const startCreate = (kind, parent = selectedDirectory ?? rootPath) => {
+				if (parent === void 0 || creatingRef.current || mutatingRef.current) return;
 				setQuery("");
 				setError(void 0);
 				setCreateName("");
 				setCreateDraft({
-					parent: selectedDirectory ?? rootPath,
+					parent,
 					kind
 				});
 			};
 			const submitCreate = async () => {
-				if (api === void 0 || createDraft === void 0 || workspaceId === void 0 || createName.trim() === "" || creatingRef.current) return;
+				if (api === void 0 || createDraft === void 0 || workspaceId === void 0 || createName.trim() === "" || creatingRef.current || mutatingRef.current) return;
 				const draft = createDraft;
 				const name = createName.trim();
 				const generation = listGeneration.current;
@@ -786,103 +813,89 @@ window.__ModuleLoader__.load({
 				}
 			};
 			const startRename = (entry) => {
+				if (mutatingRef.current || savingRef.current || formattingRef.current || creatingRef.current) return;
 				setContextMenu(void 0);
 				setRenamingPath(entry.path);
 				setRenameName(entry.name);
 			};
 			const submitRename = async (entry) => {
+				if (mutatingRef.current || savingRef.current || formattingRef.current || creatingRef.current) return;
 				if (api?.rename === void 0 || workspaceId === void 0 || rootPath === void 0 || renameName.trim() === "" || renameName.trim() === entry.name) {
 					setRenamingPath(void 0);
 					return;
 				}
 				const newName = renameName.trim();
+				const generation = listGeneration.current;
+				mutatingRef.current = true;
+				setMutating(true);
+				setError(void 0);
+				readRequest.current += 1;
+				setReading(false);
 				try {
 					const result = await api.rename(entry.path, newName, workspaceId);
-					const nextList = openDocumentsRef.current.map((d) => {
-						if (d.path === entry.path) return {
-							...d,
-							path: result.path
-						};
-						if (entry.directory && (d.path.startsWith(`${entry.path}/`) || d.path.startsWith(`${entry.path}\\`))) {
-							const sub = d.path.slice(entry.path.length);
-							return {
-								...d,
-								path: `${result.path}${sub}`
-							};
-						}
-						return d;
-					});
-					openDocumentsRef.current = nextList;
-					setOpenDocuments(nextList);
-					if (activePathRef.current === entry.path) {
-						activePathRef.current = result.path;
-						setActivePath(result.path);
-						const nextDoc = nextList.find((d) => d.path === result.path);
-						documentRef.current = nextDoc;
-						setDocument(nextDoc);
-					}
+					if (!mountedRef.current || listGeneration.current !== generation) return;
+					remapEntryPaths(entry.path, result.path, result.version);
 					await loadDirectory(parentDirectory(rootPath, entry.path));
 				} catch (reason) {
-					setError(String(reason));
+					if (mountedRef.current && listGeneration.current === generation) setError(String(reason));
 				} finally {
-					setRenamingPath(void 0);
+					mutatingRef.current = false;
+					if (mountedRef.current) {
+						setMutating(false);
+						if (listGeneration.current === generation) setRenamingPath(void 0);
+					}
 				}
 			};
 			const handleMove = async (sourcePath, destinationPath) => {
 				setDropTargetPath(void 0);
 				setDraggedPath(void 0);
-				if (api?.move === void 0 || workspaceId === void 0 || rootPath === void 0 || movingRef.current || samePath(sourcePath, destinationPath) || pathContains(sourcePath, destinationPath)) return;
-				movingRef.current = true;
-				setMoving(true);
+				if (api?.move === void 0 || workspaceId === void 0 || rootPath === void 0 || mutatingRef.current || savingRef.current || formattingRef.current || creatingRef.current || samePath(parentDirectory(rootPath, sourcePath), destinationPath) || samePath(sourcePath, destinationPath) || pathContains(sourcePath, destinationPath)) return;
+				const generation = listGeneration.current;
+				mutatingRef.current = true;
+				setMutating(true);
 				setError(void 0);
+				readRequest.current += 1;
+				setReading(false);
 				try {
 					const result = await api.move(sourcePath, destinationPath, workspaceId);
-					const activePath = activePathRef.current;
-					const activeNextPath = activePath !== void 0 && (samePath(activePath, sourcePath) || pathContains(sourcePath, activePath)) ? `${result.path}${activePath.slice(sourcePath.length)}` : activePath;
-					const nextList = openDocumentsRef.current.map((d) => {
-						if (d.path === sourcePath) return {
-							...d,
-							path: result.path
-						};
-						if (pathContains(sourcePath, d.path)) {
-							const suffix = d.path.slice(sourcePath.length);
-							return {
-								...d,
-								path: `${result.path}${suffix}`
-							};
-						}
-						return d;
-					});
-					openDocumentsRef.current = nextList;
-					setOpenDocuments(nextList);
-					if (activeNextPath !== activePath) {
-						activePathRef.current = activeNextPath;
-						setActivePath(activeNextPath);
-						const nextDocument = nextList.find((d) => d.path === activeNextPath);
-						documentRef.current = nextDocument;
-						setDocument(nextDocument);
-					}
+					if (!mountedRef.current || listGeneration.current !== generation) return;
+					remapEntryPaths(sourcePath, result.path, result.version);
 					await Promise.all([loadDirectory(parentDirectory(rootPath, sourcePath)), loadDirectory(destinationPath)]);
 				} catch (reason) {
-					setError(String(reason));
+					if (mountedRef.current && listGeneration.current === generation) setError(String(reason));
 				} finally {
-					movingRef.current = false;
-					if (mountedRef.current) setMoving(false);
+					mutatingRef.current = false;
+					if (mountedRef.current) setMutating(false);
 				}
 			};
 			const handleDelete = async (entry) => {
 				setContextMenu(void 0);
-				if (api?.delete === void 0 || workspaceId === void 0 || rootPath === void 0) return;
+				if (api?.delete === void 0 || workspaceId === void 0 || rootPath === void 0 || mutatingRef.current || savingRef.current || formattingRef.current || creatingRef.current) return;
 				if (!window.confirm(`Are you sure you want to delete "${entry.name}"?`)) return;
+				const generation = listGeneration.current;
+				mutatingRef.current = true;
+				setMutating(true);
+				setError(void 0);
+				readRequest.current += 1;
+				setReading(false);
 				try {
 					await api.delete(entry.path, workspaceId);
+					if (!mountedRef.current || listGeneration.current !== generation) return;
 					if (entry.directory) {
 						const inside = openDocumentsRef.current.filter((d) => d.path.startsWith(`${entry.path}/`) || d.path.startsWith(`${entry.path}\\`));
 						for (const d of inside) closeTab(d.path, true);
 					} else closeTab(entry.path, true);
-					await loadDirectory(parentDirectory(rootPath, entry.path));
+					const parent = parentDirectory(rootPath, entry.path);
+					setSelectedDirectory((current) => current !== void 0 && (samePath(current, entry.path) || pathContains(entry.path, current)) ? parent : current);
+					setExpanded((current) => new Set([...current].filter((path) => !samePath(path, entry.path) && !pathContains(entry.path, path))));
+					setChildren((current) => Object.fromEntries(Object.entries(current).filter(([path]) => !samePath(path, entry.path) && !pathContains(entry.path, path))));
+					setQuery("");
+					await loadDirectory(parent);
 				} catch (reason) {
-					setError(String(reason));
+					if (mountedRef.current && listGeneration.current === generation) setError(String(reason));
+				} finally {
+					mutatingRef.current = false;
+					if (mountedRef.current) setMutating(false);
 				}
 			};
 			const handleReveal = async (path) => {
@@ -982,33 +995,28 @@ window.__ModuleLoader__.load({
 				props.onDirtyChange?.(false);
 			}, [props.onDirtyChange]);
 			const saveAllFiles = (0, react.useCallback)(async () => {
-				if (api === void 0 || workspaceId === void 0 || savingRef.current) return;
-				if (openDocumentsRef.current.filter((d) => documentIsDirty(d)).length === 0) return;
+				if (api === void 0 || workspaceId === void 0 || savingRef.current || formattingRef.current || mutatingRef.current) return;
+				const dirtyDocs = openDocumentsRef.current.filter((d) => documentIsDirty(d));
+				if (dirtyDocs.length === 0) return;
+				const generation = listGeneration.current;
 				savingRef.current = true;
 				setSaving(true);
 				props.onSavingChange?.(true);
 				setError(void 0);
 				try {
-					const updatedList = [...openDocumentsRef.current];
-					for (let i = 0; i < updatedList.length; i += 1) {
-						const doc = updatedList[i];
-						if (doc && documentIsDirty(doc)) {
-							const saved = await api.save(doc.path, doc.draft, doc.version, workspaceId);
-							updatedList[i] = {
-								...doc,
-								savedContent: doc.draft,
-								version: saved.version
-							};
-						}
+					for (const doc of dirtyDocs) {
+						if (!mountedRef.current || listGeneration.current !== generation) return;
+						if (!openDocumentsRef.current.some((current) => current.path === doc.path)) continue;
+						const saved = await api.save(doc.path, doc.draft, doc.version, workspaceId);
+						if (!mountedRef.current || listGeneration.current !== generation) return;
+						publishDocuments(openDocumentsRef.current.map((current) => current.path === doc.path ? {
+							...current,
+							savedContent: doc.draft,
+							version: saved.version
+						} : current));
 					}
-					openDocumentsRef.current = updatedList;
-					setOpenDocuments(updatedList);
-					const currentActive = updatedList.find((d) => d.path === activePathRef.current);
-					documentRef.current = currentActive;
-					setDocument(currentActive);
-					props.onDirtyChange?.(false);
 				} catch (reason) {
-					if (mountedRef.current) setError(String(reason));
+					if (mountedRef.current && listGeneration.current === generation) setError(String(reason));
 				} finally {
 					savingRef.current = false;
 					if (mountedRef.current) {
@@ -1018,26 +1026,28 @@ window.__ModuleLoader__.load({
 				}
 			}, [
 				api,
-				props.onDirtyChange,
+				publishDocuments,
 				props.onSavingChange,
 				workspaceId
 			]);
 			const saveFile = (0, react.useCallback)(async () => {
-				if (api === void 0 || document === void 0 || workspaceId === void 0 || !dirty || savingRef.current || formattingRef.current) return;
+				if (api === void 0 || document === void 0 || workspaceId === void 0 || !dirty || savingRef.current || formattingRef.current || mutatingRef.current) return;
 				const snapshot = document;
+				const generation = listGeneration.current;
 				savingRef.current = true;
 				setSaving(true);
 				props.onSavingChange?.(true);
 				setError(void 0);
 				try {
 					const saved = await api.save(snapshot.path, snapshot.draft, snapshot.version, workspaceId);
-					updateDocument((current) => current?.path === snapshot.path ? {
+					if (!mountedRef.current || listGeneration.current !== generation) return;
+					publishDocuments(openDocumentsRef.current.map((current) => current.path === snapshot.path ? {
 						...current,
 						savedContent: snapshot.draft,
 						version: saved.version
-					} : current);
+					} : current));
 				} catch (reason) {
-					if (mountedRef.current && documentRef.current?.path === snapshot.path) setError(String(reason));
+					if (mountedRef.current && listGeneration.current === generation) setError(String(reason));
 				} finally {
 					savingRef.current = false;
 					if (mountedRef.current) {
@@ -1050,11 +1060,11 @@ window.__ModuleLoader__.load({
 				dirty,
 				document,
 				props.onSavingChange,
-				updateDocument,
+				publishDocuments,
 				workspaceId
 			]);
 			const formatFile = (0, react.useCallback)(async () => {
-				if (api === void 0 || document === void 0 || workspaceId === void 0 || formattingRef.current || savingRef.current) return;
+				if (api === void 0 || document === void 0 || workspaceId === void 0 || formattingRef.current || savingRef.current || mutatingRef.current) return;
 				const snapshot = document;
 				formattingRef.current = true;
 				setFormatting(true);
@@ -1220,6 +1230,7 @@ window.__ModuleLoader__.load({
 									autoFocus: true,
 									className: DesktopFilesPanel_module_css_default.renameInput,
 									"aria-label": `Rename ${entry.name}`,
+									disabled: mutating,
 									value: renameName,
 									onChange: (e) => {
 										setRenameName(e.currentTarget.value);
@@ -1232,11 +1243,13 @@ window.__ModuleLoader__.load({
 									type: "submit",
 									"aria-label": "Confirm rename",
 									className: DesktopFilesPanel_module_css_default.renameBtn,
+									disabled: mutating,
 									children: (0, react_jsx_runtime.jsx)(_hydraharness_harness_client_ui_primitives.IconCheckOutline14, {})
 								}),
 								(0, react_jsx_runtime.jsx)("button", {
 									type: "button",
 									"aria-label": "Cancel rename",
+									disabled: mutating,
 									className: DesktopFilesPanel_module_css_default.renameBtn,
 									onClick: () => {
 										setRenamingPath(void 0);
@@ -1247,7 +1260,7 @@ window.__ModuleLoader__.load({
 						}) : (0, react_jsx_runtime.jsxs)("button", {
 							type: "button",
 							className: DesktopFilesPanel_module_css_default.row,
-							draggable: !moving,
+							draggable: !mutating && !saving && !formatting && !creating,
 							"data-drop-target": dropTargetPath === entry.path ? "true" : void 0,
 							style: { paddingLeft: 8 + depth * 14 },
 							"data-selected": entry.directory ? selectedDirectory === entry.path : document?.path === entry.path || void 0,
@@ -1265,7 +1278,7 @@ window.__ModuleLoader__.load({
 								});
 							},
 							onDragStart: (event) => {
-								if (moving) return;
+								if (mutating || saving || formatting || creating) return;
 								setDraggedPath(entry.path);
 								event.dataTransfer.effectAllowed = "move";
 								event.dataTransfer.setData("text/plain", entry.path);
@@ -1275,7 +1288,7 @@ window.__ModuleLoader__.load({
 								setDropTargetPath(void 0);
 							},
 							onDragOver: (event) => {
-								if (!entry.directory || draggedPath === void 0 || samePath(draggedPath, entry.path) || pathContains(draggedPath, entry.path) || moving) return;
+								if (!entry.directory || draggedPath === void 0 || samePath(draggedPath, entry.path) || pathContains(draggedPath, entry.path) || mutating) return;
 								event.preventDefault();
 								event.dataTransfer.dropEffect = "move";
 								setDropTargetPath(entry.path);
@@ -1309,7 +1322,7 @@ window.__ModuleLoader__.load({
 				role: error === void 0 ? "status" : "alert",
 				children: error ?? (workspaceId === void 0 ? "Open a workspace to browse its files." : rootLoading ? "Loading workspace files…" : "Workspace files are unavailable.")
 			});
-			const status = saving ? "Saving…" : formatting ? "Formatting…" : moving ? "Moving…" : creating ? "Creating…" : searching ? "Searching…" : loading || reading ? "Loading…" : void 0;
+			const status = saving ? "Saving…" : formatting ? "Formatting…" : mutating ? "Updating files…" : creating ? "Creating…" : searching ? "Searching…" : loading || reading ? "Loading…" : void 0;
 			const highlightedDraft = document !== void 0 && deferredDraft === document.draft && document.draft.length <= HIGHLIGHT_MAX_CHARS ? deferredDraft : void 0;
 			const lineCount = document ? document.draft.split("\n").length : 1;
 			return (0, react_jsx_runtime.jsxs)("div", {
@@ -1356,7 +1369,7 @@ window.__ModuleLoader__.load({
 												className: DesktopFilesPanel_module_css_default.sectionActionBtn,
 												title: "Save all files",
 												"aria-label": "Save all files",
-												disabled: saving || !openDocuments.some((d) => documentIsDirty(d)),
+												disabled: saving || formatting || mutating || !openDocuments.some((d) => documentIsDirty(d)),
 												onClick: () => {
 													saveAllFiles();
 												},
@@ -1464,7 +1477,7 @@ window.__ModuleLoader__.load({
 													className: DesktopFilesPanel_module_css_default.sectionActionBtn,
 													"aria-label": "New file",
 													title: "New file",
-													disabled: creating,
+													disabled: creating || mutating,
 													onClick: () => {
 														startCreate("file");
 													},
@@ -1475,7 +1488,7 @@ window.__ModuleLoader__.load({
 													className: DesktopFilesPanel_module_css_default.sectionActionBtn,
 													"aria-label": "New folder",
 													title: "New folder",
-													disabled: creating,
+													disabled: creating || mutating,
 													onClick: () => {
 														startCreate("directory");
 													},
@@ -1496,7 +1509,7 @@ window.__ModuleLoader__.load({
 													className: DesktopFilesPanel_module_css_default.sectionActionBtn,
 													"aria-label": "Refresh files",
 													title: "Refresh files",
-													disabled: creating,
+													disabled: creating || mutating,
 													onClick: () => {
 														loadDirectory(rootPath);
 													},
@@ -1523,10 +1536,10 @@ window.__ModuleLoader__.load({
 									className: DesktopFilesPanel_module_css_default.tree,
 									role: "tree",
 									"aria-label": "Workspace files",
-									"aria-busy": loading || searching || creating || moving || void 0,
+									"aria-busy": loading || searching || creating || mutating || void 0,
 									"data-drop-target": dropTargetPath === rootPath ? "true" : void 0,
 									onDragOver: (event) => {
-										if (event.target !== event.currentTarget || draggedPath === void 0 || moving) return;
+										if (event.target !== event.currentTarget || draggedPath === void 0 || mutating) return;
 										event.preventDefault();
 										event.dataTransfer.dropEffect = "move";
 										setDropTargetPath(rootPath);
@@ -1826,7 +1839,7 @@ window.__ModuleLoader__.load({
 											className: DesktopFilesPanel_module_css_default.editorActionBtn,
 											"aria-label": "Format",
 											"aria-keyshortcuts": "Shift+Alt+F",
-											disabled: formatting || saving,
+											disabled: formatting || saving || mutating,
 											title: "Format document (Shift+Alt+F)",
 											onMouseDown: (event) => {
 												event.preventDefault();
@@ -1841,7 +1854,7 @@ window.__ModuleLoader__.load({
 											className: `${DesktopFilesPanel_module_css_default.editorActionBtn} ${dirty ? DesktopFilesPanel_module_css_default.editorActionBtnDirty : ""}`,
 											"aria-label": "Save",
 											"aria-keyshortcuts": "Control+S",
-											disabled: !dirty || saving || formatting,
+											disabled: !dirty || saving || formatting || mutating,
 											title: "Save (Ctrl+S)",
 											onMouseDown: (event) => {
 												event.preventDefault();
@@ -2199,7 +2212,7 @@ window.__ModuleLoader__.load({
 									onClick: () => {
 										setContextMenu(void 0);
 										setSelectedDirectory(contextMenu.entry.path);
-										startCreate("file");
+										startCreate("file", contextMenu.entry.path);
 									},
 									children: [(0, react_jsx_runtime.jsx)(_hydraharness_harness_client_ui_primitives.IconPlusOutline16, { size: 13 }), "New File"]
 								}),
@@ -2210,7 +2223,7 @@ window.__ModuleLoader__.load({
 									onClick: () => {
 										setContextMenu(void 0);
 										setSelectedDirectory(contextMenu.entry.path);
-										startCreate("directory");
+										startCreate("directory", contextMenu.entry.path);
 									},
 									children: [(0, react_jsx_runtime.jsx)(_hydraharness_harness_client_ui_primitives.IconProjectAddOutline16, { size: 13 }), "New Folder"]
 								}),
@@ -2220,6 +2233,7 @@ window.__ModuleLoader__.load({
 								type: "button",
 								className: DesktopFilesPanel_module_css_default.contextMenuItem,
 								role: "menuitem",
+								disabled: api?.rename === void 0 || mutating || saving || formatting || creating,
 								onClick: () => {
 									startRename(contextMenu.entry);
 								},
@@ -2229,6 +2243,7 @@ window.__ModuleLoader__.load({
 								type: "button",
 								className: DesktopFilesPanel_module_css_default.contextMenuItem,
 								role: "menuitem",
+								disabled: api?.delete === void 0 || mutating || saving || formatting || creating,
 								onClick: () => {
 									handleDelete(contextMenu.entry);
 								},

package/lib/types/client/DesktopFilesPanel.d.ts

@@ -18,12 +18,20 @@ export interface DesktopFilesApi {
         path: string;
         name: string;
         directory?: boolean;
+        version?: {
+            before: string;
+            after: string;
+        };
     }>;
     move?(path: string, destinationDirectory: string, workspaceId: string): Promise<{
         oldPath: string;
         path: string;
         name: string;
         directory?: boolean;
+        version?: {
+            before: string;
+            after: string;
+        };
     }>;
     delete?(path: string, workspaceId: string): Promise<{
         path: string;

package/package.json

@@ -1,13 +1,13 @@
 {
   "name": "@hydraharness/harness-client-ui-layout",
   "description": "Shell plugin: three-column AppFrame with drag handles, ctx.layout viewing-state service (navigation + panels)",
-  "version": "0.1.1-rc.8",
+  "version": "0.1.1-rc.9",
   "publishConfig": {
     "access": "public"
   },
   "repository": {
     "type": "git",
-    "url": "git+https://github.com/MaiHongPhong1902/Hydra-Harness.git",
+    "url": "git+https://github.com/MaiHongPhong2010/Hydra-Harness.git",
     "directory": "packages/client/ui-layout"
   },
   "type": "module",
@@ -49,25 +49,25 @@
     "@xterm/xterm": "^6.0.0"
   },
   "peerDependencies": {
-    "@hydraharness/harness-client-locale": "^0.1.1-rc.8",
-    "@hydraharness/harness-client-ui-settings": "^0.1.1-rc.8",
-    "@hydraharness/harness-client-ui-theme": "^0.1.1-rc.8",
-    "@hydraharness/harness-invariants": "^0.1.1-rc.8",
-    "@hydraharness/cordis": "^4.0.2",
-    "@hydraharness/harness-client-runtime": "^0.1.1-rc.8"
+    "@hydraharness/harness-client-locale": "^0.1.1-rc.9",
+    "@hydraharness/harness-client-ui-settings": "^0.1.1-rc.9",
+    "@hydraharness/harness-client-runtime": "^0.1.1-rc.9",
+    "@hydraharness/harness-client-ui-theme": "^0.1.1-rc.9",
+    "@hydraharness/harness-invariants": "^0.1.1-rc.9",
+    "@hydraharness/cordis": "^4.0.2"
   },
   "devDependencies": {
     "@types/react": "~18.3.1",
     "react": "^18.2.0",
-    "@hydraharness/harness-client-locale": "^0.1.1-rc.8",
-    "@hydraharness/harness-client-test-runtime": "^0.1.1-rc.8",
-    "@hydraharness/harness-client-ui-slots": "^0.1.1-rc.8",
-    "@hydraharness/harness-client-ui-settings": "^0.1.1-rc.8",
-    "@hydraharness/harness-client-ui-theme": "^0.1.1-rc.8",
-    "@hydraharness/harness-client-runtime": "^0.1.1-rc.8",
-    "@hydraharness/cordis": "^4.0.2",
-    "@hydraharness/harness-client-ui-primitives": "^0.1.1-rc.8",
-    "@hydraharness/harness-invariants": "^0.1.1-rc.8"
+    "@hydraharness/harness-client-test-runtime": "^0.1.1-rc.9",
+    "@hydraharness/harness-client-ui-settings": "^0.1.1-rc.9",
+    "@hydraharness/harness-client-locale": "^0.1.1-rc.9",
+    "@hydraharness/harness-client-ui-slots": "^0.1.1-rc.9",
+    "@hydraharness/harness-client-ui-theme": "^0.1.1-rc.9",
+    "@hydraharness/harness-client-runtime": "^0.1.1-rc.9",
+    "@hydraharness/harness-invariants": "^0.1.1-rc.9",
+    "@hydraharness/harness-client-ui-primitives": "^0.1.1-rc.9",
+    "@hydraharness/cordis": "^4.0.2"
   },
   "files": [
     "lib/index.js",