dsh-vscode-workbench 0.1.1 → 0.1.2

package/README.en.md

@@ -34,13 +34,6 @@ The workbench mounts through the official `sidebar.footer.action` and `shell.ove
 
 ## Installation
 
-Install the aggregate package for the complete workbench with terminal integration:
-
-```bash
-dsh plugin --profile web add dsh-vscode-plugin
-```
-
-Install only the workbench and its extension API:
 
 ```bash
 dsh plugin --profile web add dsh-vscode-workbench
@@ -58,9 +51,6 @@ dsh plugin --profile web add dsh-vscode-workbench
 
 Terminal shortcuts are registered by `dsh-vscode-plugin`.
 
-## Extension API
-
-The browser exposes a `vscodeWorkbench` service for registering bottom panels and commands and controlling workbench visibility. `dsh-vscode-plugin` uses this API to mount `@xterm/xterm`.
 
 ## Security
 

package/README.md

@@ -33,14 +33,6 @@
 
 ## 安装
 
-推荐安装包含终端适配器的聚合包：
-
-```bash
-dsh plugin --profile web add dsh-vscode-plugin
-```
-
-仅使用工作台和扩展 API：
-
 ```bash
 dsh plugin --profile web add dsh-vscode-workbench
 ```
@@ -57,9 +49,6 @@ dsh plugin --profile web add dsh-vscode-workbench
 
 终端快捷键由 `dsh-vscode-plugin` 注册。
 
-## 扩展
-
-浏览器端提供 `vscodeWorkbench` 服务，可注册底部面板和命令，并控制工作台与面板显隐。`dsh-vscode-plugin` 使用该接口挂载 `@xterm/xterm`。
 
 ## 安全
 

package/lib/client.js

@@ -4191,6 +4191,17 @@ Please report this to https://github.com/markedjs/marked.`, e) {
 		//#endregion
 		//#region src/client/MonacoEditor.tsx
 		let loading;
+		/** Monaco chrome used by both editors so only the narrow native scrollbar remains. */
+		const WORKBENCH_EDITOR_CHROME = {
+			minimap: { enabled: false },
+			renderOverviewRuler: false,
+			overviewRulerLanes: 0,
+			hideCursorInOverviewRuler: true,
+			scrollbar: {
+				verticalScrollbarSize: 10,
+				horizontalScrollbarSize: 10
+			}
+		};
 		function loadMonaco() {
 			loading ??= new Promise((resolve, reject) => {
 				const start = () => {
@@ -4233,9 +4244,8 @@ Please report this to https://github.com/markedjs/marked.`, e) {
 						renderSideBySide: props.mode === "split",
 						theme: props.theme === "dark" ? "vs-dark" : "vs",
 						fontSize: 14,
-						minimap: { enabled: false },
+						...WORKBENCH_EDITOR_CHROME,
 						originalEditable: false,
-						renderOverviewRuler: true,
 						ignoreTrimWhitespace: false
 					});
 					instance.setModel({
@@ -4300,7 +4310,7 @@ Please report this to https://github.com/markedjs/marked.`, e) {
 						automaticLayout: true,
 						theme: props.theme === "dark" ? "vs-dark" : "vs",
 						fontSize: 14,
-						minimap: { enabled: true },
+						...WORKBENCH_EDITOR_CHROME,
 						wordWrap: "off"
 					});
 					editor.current = instance;
@@ -4427,6 +4437,37 @@ Please report this to https://github.com/markedjs/marked.`, e) {
 			});
 		}
 		//#endregion
+		//#region src/client/workspaceState.ts
+		/** Remove editor state that belongs to the previous workspace. */
+		function resetWorkspaceDocuments() {
+			return {
+				tabs: [],
+				active: void 0,
+				error: void 0
+			};
+		}
+		/** Identifies responses that still belong to the active workspace request. */
+		var WorkspaceRequestScope = class {
+			epoch = 0;
+			/** Begin loading another workspace and invalidate earlier work. */
+			begin() {
+				this.epoch += 1;
+				return this.epoch;
+			}
+			/** Return the active workspace request identifier. */
+			current() {
+				return this.epoch;
+			}
+			/** Invalidate an active request while preserving any newer request. */
+			invalidate(epoch) {
+				if (this.epoch === epoch) this.epoch += 1;
+			}
+			/** Return whether a response may update the current workspace. */
+			isCurrent(epoch) {
+				return epoch === this.epoch;
+			}
+		};
+		//#endregion
 		//#region src/client/Workbench.tsx
 		/** Return the basename shown in a tab while preserving its full internal path. */
 		function tabLabel(tab) {
@@ -5234,6 +5275,9 @@ Please report this to https://github.com/markedjs/marked.`, e) {
 			const [chatWidth, setChatWidth] = (0, react.useState)(() => Math.max(400, Math.min(720, Math.round(Number(localStorage.getItem("dvw:dsh-chat-width")) || 420))));
 			const [chatVisible, setChatVisible] = (0, react.useState)(() => localStorage.getItem("dvw:dsh-chat-visible") !== "closed");
 			const [outlineVisible, setOutlineVisible] = (0, react.useState)(() => localStorage.getItem("dvw:outline") !== "closed");
+			const tabsRef = (0, react.useRef)(tabs);
+			tabsRef.current = tabs;
+			const workspaceRequests = (0, react.useRef)(new WorkspaceRequestScope());
 			const dshDark = useDshDarkTheme();
 			const [theme, setTheme] = (0, react.useState)(() => {
 				const saved = localStorage.getItem("dvw:theme");
@@ -5256,9 +5300,69 @@ Please report this to https://github.com/markedjs/marked.`, e) {
 				window.addEventListener("keydown", onKey, true);
 				return () => window.removeEventListener("keydown", onKey, true);
 			}, [snapshot.visible]);
+			const openForWorkspace = async (targetSessionId, epoch, path, line, column) => {
+				if (!workspaceRequests.current.isCurrent(epoch)) return;
+				if (tabsRef.current.find((tab) => tab.path === path)) {
+					setActive(path);
+					setTabs((rows) => rows.map((tab) => tab.path === path ? {
+						...tab,
+						line,
+						column
+					} : tab));
+					return;
+				}
+				if (/\.(?:avif|bmp|gif|ico|jpe?g|png|svg|webp)$/i.test(path)) {
+					const imageUrl = `/dsh-vscode/file?sessionId=${encodeURIComponent(targetSessionId)}&path=${encodeURIComponent(path)}`;
+					if (!workspaceRequests.current.isCurrent(epoch)) return;
+					setTabs((rows) => workspaceRequests.current.isCurrent(epoch) ? [...rows.filter((tab) => tab.path !== path), {
+						path,
+						content: "",
+						draft: "",
+						revision: "",
+						size: 0,
+						binary: false,
+						tooLarge: false,
+						dirty: false,
+						imageUrl
+					}] : rows);
+					setActive(path);
+					return;
+				}
+				try {
+					const doc = await api("fs.read", {
+						sessionId: targetSessionId,
+						path
+					});
+					if (!workspaceRequests.current.isCurrent(epoch)) return;
+					if (doc.binary || doc.tooLarge) {
+						setError(doc.binary ? "二进制文件不能在文本编辑器中打开" : `文件超过 5 MiB：${path}`);
+						return;
+					}
+					setTabs((rows) => workspaceRequests.current.isCurrent(epoch) ? [...rows.filter((tab) => tab.path !== path), {
+						...doc,
+						draft: doc.content,
+						dirty: false,
+						line,
+						column,
+						preview: /\.md$/i.test(path)
+					}] : rows);
+					setActive(path);
+				} catch (error) {
+					if (workspaceRequests.current.isCurrent(epoch)) setError(String(error));
+				}
+			};
 			(0, react.useEffect)(() => {
 				if (!snapshot.visible || sessionId === void 0) return;
-				api("workspace", { sessionId }).then((value) => {
+				const epoch = workspaceRequests.current.begin();
+				const cleared = resetWorkspaceDocuments();
+				setWorkspace(void 0);
+				setTabs(cleared.tabs);
+				tabsRef.current = cleared.tabs;
+				setActive(cleared.active);
+				setError(cleared.error);
+				const controller = new AbortController();
+				api("workspace", { sessionId }, controller.signal).then((value) => {
+					if (!workspaceRequests.current.isCurrent(epoch)) return;
 					setWorkspace(value);
 					const saved = localStorage.getItem(`dvw:${value.cwd}`);
 					if (saved) try {
@@ -5266,9 +5370,15 @@ Please report this to https://github.com/markedjs/marked.`, e) {
 						if (parsed.view) service.show(parsed.view);
 						if (typeof parsed.panelHeight === "number") setPanelHeight(parsed.panelHeight);
 						if (typeof parsed.sideWidth === "number") setSideWidth(Math.max(210, Math.min(640, Math.round(parsed.sideWidth))));
-						for (const path of parsed.paths ?? []) open(path);
+						for (const path of parsed.paths ?? []) openForWorkspace(sessionId, epoch, path);
 					} catch {}
-				}).catch((error) => setError(String(error)));
+				}).catch((error) => {
+					if (!controller.signal.aborted && workspaceRequests.current.isCurrent(epoch)) setError(String(error));
+				});
+				return () => {
+					controller.abort();
+					workspaceRequests.current.invalidate(epoch);
+				};
 			}, [snapshot.visible, sessionId]);
 			(0, react.useEffect)(() => {
 				if (workspace) localStorage.setItem(`dvw:${workspace.cwd}`, JSON.stringify({
@@ -5299,52 +5409,7 @@ Please report this to https://github.com/markedjs/marked.`, e) {
 			}, [snapshot.visible, sessionId]);
 			const open = async (path, line, column) => {
 				if (sessionId === void 0) return;
-				if (tabs.find((t) => t.path === path)) {
-					setActive(path);
-					setTabs((rows) => rows.map((t) => t.path === path ? {
-						...t,
-						line,
-						column
-					} : t));
-					return;
-				}
-				if (/\.(?:avif|bmp|gif|ico|jpe?g|png|svg|webp)$/i.test(path)) {
-					const imageUrl = `/dsh-vscode/file?sessionId=${encodeURIComponent(sessionId)}&path=${encodeURIComponent(path)}`;
-					setTabs((rows) => [...rows, {
-						path,
-						content: "",
-						draft: "",
-						revision: "",
-						size: 0,
-						binary: false,
-						tooLarge: false,
-						dirty: false,
-						imageUrl
-					}]);
-					setActive(path);
-					return;
-				}
-				try {
-					const doc = await api("fs.read", {
-						sessionId,
-						path
-					});
-					if (doc.binary || doc.tooLarge) {
-						setError(doc.binary ? "二进制文件不能在文本编辑器中打开" : `文件超过 5 MiB：${path}`);
-						return;
-					}
-					setTabs((rows) => [...rows, {
-						...doc,
-						draft: doc.content,
-						dirty: false,
-						line,
-						column,
-						preview: /\.md$/i.test(path)
-					}]);
-					setActive(path);
-				} catch (error) {
-					setError(String(error));
-				}
+				await openForWorkspace(sessionId, workspaceRequests.current.current(), path, line, column);
 			};
 			(0, react.useEffect)(() => {
 				const listener = (event) => void open(event.detail);
@@ -5964,10 +6029,13 @@ body[data-ds-dark-theme] .dvw-overlay,.dvw-overlay[data-dvw-theme=dark]{color-sc
 		}
 		//#endregion
 		//#region src/client/index.tsx
-		const Overlay = ({ service, useSessions }) => /* @__PURE__ */ (0, react_jsx_runtime.jsx)(Workbench, {
-			service,
-			useSessions
-		});
+		const Overlay = ({ service, useSessions }) => {
+			const sessionId = useSessions((state) => state.current);
+			return /* @__PURE__ */ (0, react_jsx_runtime.jsx)(Workbench, {
+				service,
+				useSessions
+			}, sessionId ?? "no-session");
+		};
 		const Footer = ({ service, wide }) => {
 			const marker = (0, react.useRef)(null);
 			const [target, setTarget] = (0, react.useState)();