@michengai/dsh-archive-manager 0.1.1 → 0.1.2

package/lib/client.js

@@ -492,9 +492,9 @@ window.__ModuleLoader__.load({
 		* except workspace Rename/Delete and session Rename/Fork/Archive; the session
 		* and workspace hover cards are suppressed while a menu is open.
 		*/
-		/** Row display title: blank rows show the localized New Session label. */
-		function displayTitle(node, t) {
-			return node.blank ? t("session.new") : node.title;
+		/** Row display title: blank rows show the localized New Session label. */
+		function displayTitle(node, t) {
+			return node.blank ? t("session.new") : node.title ?? node.displayTitle ?? "";
 		}
 		/** Localized compact relative time ("刚刚"/"5分钟" in zh, "now"/"5min" in en). */
 		function timeLabel(updatedAt, now, t) {
@@ -2885,8 +2885,9 @@ window.__ModuleLoader__.load({
 		}
 		//#endregion
 		/** Pure derivation surface for dsh-archive-manager self-tests (no-op for the runtime). */
-		exports.__test = {
-			sessionVisible,
+		exports.__test = {
+			displayTitle,
+			sessionVisible,
 			deriveGroups,
 			deriveFlat,
 			deriveSearchResults,
@@ -2899,6 +2900,4 @@ window.__ModuleLoader__.load({
 		exports.inject = inject;
 		return module.exports;
 	}
-});
-
-//# sourceMappingURL=client.js.map
+});

package/lib/workspace.js

@@ -7,51 +7,18 @@ import {
 import { bindTypertRemote, Remote } from "@deepseek-ai/dsh-typert-protocol";
 import { sessionDir } from "@deepseek-ai/dsh-spill-local";
 //#region lib/types/index.js
-/**
- * dsh-archive-manager host half — archive session management.
- *
- * `ArchiveWorkspaceRegistry` extends the shipped `WorkspaceRegistry`
- * (service name `workspaceRegistry`, one domain open, same accounting
- * invariants) and adds two durable operations:
- *
- * - `unarchiveSession(sessionId)` — removes the id from the registry-global
- *   `archivedSessionIds` set. Archiving never moves the accounting seat, so
- *   unarchiving restores the session at its original workspace position.
- *   Idempotent: a session that is not archived resolves without writing.
- *
- * - `deleteSession(sessionId)` — PERMANENT deletion, no trace left on disk:
- *   1. validate the session is known (live, header-indexed, or persisted);
- *   2. if live: flush buffered events to disk, then detach the session from
- *      the sessions store (`session/disposed` is emitted, so the browser
- *      receives `host/session-removed` and greys the composer);
- *   3. wait for the projection cache's dispose write-behind (`whenIdle`) so
- *      the final cache row is durable BEFORE it is deleted;
- *   4. remove the transcript directory (the whole `session-<id>` dir under
- *      the persistence root, located through `sessionPersistence.locate`);
- *   5. remove the id from `archivedSessionIds` (durable `setState`);
- *   6. remove the id from every workspace record's `sessionIds` and refresh
- *      the in-memory entity snapshots (durable `table.update`, so the host
- *      stream pushes `host/workspace-changed` to every browser);
- *   7. delete the projection-cache row (`sessionProjectionCache.delete`);
- *   8. best-effort (failures only warn): cascade-delete SUBAGENT child
- *      sessions of the deleted session (`header.origin === "subagent"` ONLY —
- *      a fork branch carries `parentSession` too but is an independent user
- *      session and is never cascade-deleted) and clear its spill data.
- *
- * Core steps throw on failure (no half-delete states: every failing step is
- * idempotent and re-runnable, so a retry heals); cascade/spill are
- * best-effort by design.
- *
- * The two new methods are also exported as Typert Remote endpoints
- * (`workspaceRegistry/unarchiveSession`, `workspaceRegistry/deleteSession`)
- * through the service's `typertRemote` binding + `Remote` markers — the
- * browser reaches them through the standard typert gateway SRC path, keeping
- * the legacy `/api/workspace.*` gateway untouched.
- *
- * The default export is a Service subclass (same shape as the shipped
- * `@deepseek-ai/dsh-workspace` package), so the profile patch can substitute
- * this package for the `workspace` row with no other wiring change.
- */
+/**
+* dsh-archive-manager 宿主侧归档会话管理。
+*
+* `deleteSession(sessionId)` 的顺序即语义：校验已知会话；实时会话先
+* flush 再 detach；等待投影缓存写入完成；移除归档标记和工作区记账；
+* 删除投影缓存，级联删除 SUBAGENT 子会话并清理 spill；最后才删除转录
+* 目录。fork 分支虽有 `parentSession`，但属于独立用户会话，不参与级联。
+*
+* 所有可失败的持久化清理都先于不可逆的物理删除。此前任一步失败时，
+* 转录仍保留，可重试删除；物理删除失败时记录残留路径供人工排查。
+* `unarchiveSession` 与 `deleteSession` 通过 Typert Remote 暴露给浏览器。
+*/
 function markRemoteMethod(instance, method) {
 	// Simulate the TS decorator pipeline `@Remote(method)` for one method:
 	// `Remote` returns a standard method decorator; we hand it a decorator
@@ -128,21 +95,22 @@ var ArchiveWorkspaceRegistry = class extends WorkspaceRegistry {
 			sessions.detachEntered(entry);
 		} else if (sessions !== void 0) await this.publishColdSessionRemoval(sessionId, sessions);
 		const projCache = this.ctx.get("sessionProjectionCache");
-		// The dispose write-behind must land BEFORE the cache row is deleted,
-		// otherwise the row is written back after deletion (resurrected).
-		await projCache?.whenIdle?.();
-		await this.removeTranscriptDirectory(sessionId);
-		const state = this.requireState();
+		// The dispose write-behind must land BEFORE the cache row is deleted,
+		// otherwise the row is written back after deletion (resurrected).
+		await projCache?.whenIdle?.();
+		// 物理删除不可逆：先完成所有可失败的持久化清理，失败时仍可重试。
+		const state = this.requireState();
 		if (state.archivedSessionIds.includes(sessionId)) {
 			await this.setState({
 				...state,
 				archivedSessionIds: state.archivedSessionIds.filter((id) => id !== sessionId)
 			});
-		}
-		await this.removeFromWorkspaceAccounts(sessionId);
-		if (projCache !== void 0) await projCache.delete(sessionId);
-		await this.deleteDescendants(sessionId);
-		await this.cleanSpill(sessionId);
+		}
+		await this.removeFromWorkspaceAccounts(sessionId);
+		if (projCache !== void 0) await projCache.delete(sessionId);
+		await this.deleteDescendants(sessionId);
+		await this.cleanSpill(sessionId);
+		await this.removeTranscriptDirectory(sessionId);
 		return { deleted: true };
 	}
 	/** 为未处于实时状态的持久化会话发布相同的移除事件。 */
@@ -162,17 +130,26 @@ var ArchiveWorkspaceRegistry = class extends WorkspaceRegistry {
 			this.ctx.logger.warn(`archive-manager: could not publish removal for stored session "${sessionId}": ${String(error)}`);
 		}
 	}
-	/** Remove the session's whole transcript directory (throw on failure —
-	* the irreversible step runs before any accounting mutation). */
-	async removeTranscriptDirectory(sessionId) {
+	/** 删除会话的转录目录；仅在所有记账清理完成后调用。 */
+	async removeTranscriptDirectory(sessionId) {
 		const persistence = this.ctx.get("sessionPersistence");
 		if (persistence === void 0 || typeof persistence.locate !== "function") {
 			throw new Error(`cannot delete session "${sessionId}": the session persistence backend does not expose locate() to resolve its transcript directory`);
 		}
-		const header = await this.readSessionHeader(sessionId);
-		const location = persistence.locate(header);
-		await rm(dirname(location.path), { recursive: true, force: true });
-	}
+		const header = await this.readSessionHeader(sessionId);
+		const location = persistence.locate(header);
+		if (location === void 0 || typeof location.path !== "string") {
+			throw new Error(`cannot delete session "${sessionId}": the session persistence backend could not resolve its transcript directory`);
+		}
+		const transcriptDir = dirname(location.path);
+		try {
+			await rm(transcriptDir, { recursive: true, force: true });
+		} catch (error) {
+			const message = `cannot delete session "${sessionId}": transcript directory "${transcriptDir}" remains after bookkeeping cleanup`;
+			this.ctx.logger.warn(`archive-manager: ${message}: ${String(error)}`);
+			throw new Error(`${message}: ${String(error)}`, { cause: error });
+		}
+	}
 	/** Drop the id from every workspace record and refresh entity snapshots. */
 	async removeFromWorkspaceAccounts(sessionId) {
 		const table = this.requireTable();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@michengai/dsh-archive-manager",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "description": "NPM-installable DSH Web plugin for managing archived sessions.",
   "type": "module",
   "packageManager": "pnpm@11.21.0",
@@ -28,7 +28,21 @@
     "test": "node --test test/*.test.mjs",
     "pack:check": "pnpm pack --dry-run",
     "prepublishOnly": "pnpm build && pnpm test"
-  },
+  },
+  "devDependencies": {
+    "@deepseek-ai/cordis": "4.0.1",
+    "@deepseek-ai/dsh-api-gateway": "0.1.0-rc.6",
+    "@deepseek-ai/dsh-client-runtime": "0.1.0-rc.6",
+    "@deepseek-ai/dsh-client-ui-slots": "0.1.0-rc.6",
+    "@deepseek-ai/dsh-client-web-react": "0.1.0-rc.6",
+    "@deepseek-ai/dsh-session-projection-cache": "0.1.0-rc.6",
+    "@deepseek-ai/dsh-spill-local": "0.1.0-rc.6",
+    "@deepseek-ai/dsh-typert-protocol": "0.1.0-rc.6",
+    "@deepseek-ai/dsh-typert-registry": "0.1.0-rc.6",
+    "@deepseek-ai/dsh-workspace": "0.1.0-rc.6",
+    "react": "18.3.1",
+    "react-dom": "18.3.1"
+  },
   "license": "Apache-2.0",
   "dsh": {
     "bundle": {