@lmzhen/dsh-evolution-skill-history 0.25.8 → 0.26.15

package/README.md

@@ -76,3 +76,9 @@ This package adds **no model-visible content**: no prompt section, no tool schem
 - **The six routes are loopback-only by design.** The fence is the family's shared loopback check, authoritative on the socket (`X-Forwarded-For` is never trusted), so a LAN-facing host that answers the platform's `/api` normally returns **403** here for any non-loopback client; reach the host over loopback (a local browser, or an SSH tunnel). Following the platform's `trustedHosts` instead is an open decision (v46 `决策 3`; the mechanism lives in `evolution-core`'s `http-routes.ts`).
 - **Support-file bytes are history only.** They share the index with the body, so they are listed and labelled, but `undo` refuses them by name — the bytes stay in the blob store for a hand copy.
 - **No occupancy row.** `.history` growth is documented in the evolution-core README; showing the size in the panel would promise a cleanup path the family has not designed (a whole-library reference count belongs to the curator).
+
+## 单元与撤销（本包负责的面）
+
+- 面板行的 undoable 是**单元属性**：同 unitId 的记录同值；路标（有 skill、无 moves）返回 undefined ⇒ 路由侧**省略该键**，绝不为 false；聚合时跳过路标，避免它的 hash 污染兄弟片（src/unit-undoable.ts）。
+- 面板不拥有规则：哪条记录持正文、哪条已是当前，由 core 的 partitionVersions 与该行的哈希决定；本包只把结论送上 wire。
+- 面向读者的完整说明见仓库 docs/units-and-undo.md；撤销的决策层在 evolution-curator（src/unit-undo-plan.ts）。

package/lib/index.js

@@ -1,4 +1,39 @@
-import { contentHash, displayBodyOf, elapsedSince, errorText, isLoopbackRequest, latestVersionAt, partitionVersions, readJsonObject, skillRowCells, textDiffFacts, versionActionKind, versionRowCells, writeJson } from "@lmzhen/dsh-evolution-core";
+import { contentHash, displayBodyOf, elapsedSince, errorText, isLandmark, isLoopbackRequest, latestVersionAt, partitionVersions, readJsonObject, skillRowCells, textDiffFacts, versionActionKind, versionRowCells, writeJson } from "@lmzhen/dsh-evolution-core";
+//#region lib/types/unit-undoable.js
+/**
+* G3b: undoability is a property of the UNIT, not of one entry.
+*
+* Every entry of a unit shares the same live body, so "is this undoable" must answer the same for all
+* of them; computing it per entry made the body row say `false` (already current) while a support row
+* of the SAME transaction said `true` — two entrances to one unit in one panel.
+*
+* Entries without a unit id keep the per-entry answer (they are their own singleton unit).
+*/
+/** @param members - every entry that shares the unit (content and support chains together).
+* @param liveHash - hash of the live body, or null when the skill is not readable.
+* @returns one flag per unit id; unknown ids fall back to the per-entry rule. */
+function unitUndoableFlags(members, liveHash) {
+	const flags = /* @__PURE__ */ new Map();
+	for (const entry of members) {
+		if (entry.unitId === void 0) continue;
+		if (isLandmark(entry)) continue;
+		const differs = entry.hash !== liveHash;
+		flags.set(entry.unitId, (flags.get(entry.unitId) ?? false) || differs);
+	}
+	return flags;
+}
+/** @param entry - the entry being rendered. @param flags - the unit flags from {@link unitUndoableFlags}.
+* @param liveHash - hash of the live body. @returns the row's `undoable` value. */
+function undoableFor(entry, flags, liveHash) {
+	if (isLandmark(entry)) return void 0;
+	const unit = entry.unitId;
+	if (unit !== void 0) {
+		const flagged = flags.get(unit);
+		if (flagged !== void 0) return flagged;
+	}
+	return entry.hash !== liveHash;
+}
+//#endregion
 //#region lib/types/routes.js
 /**
 * The skill-history HTTP surface: six loopback routes over the curator's read/write seam (five reads plus the one write).
@@ -125,13 +160,15 @@ function makeSkillHistoryRoutes(services) {
 							...!withDelta || previous === void 0 ? {} : { charsDelta: entry.chars - previous.chars }
 						};
 					};
+					const unitFlags = unitUndoableFlags([...groups.content, ...groups.support], liveHash);
 					writeJson(res, 200, {
 						ok: true,
 						data: {
 							content: groups.content.map((entry, index) => {
+								const undoable = undoableFor(entry, unitFlags, liveHash);
 								const row = {
 									...decorate(entry, index, groups.content, true),
-									undoable: entry.hash !== liveHash
+									...undoable === void 0 ? {} : { undoable }
 								};
 								return {
 									...row,

package/lib/types/unit-undoable.d.ts

@@ -0,0 +1,18 @@
+/**
+ * G3b: undoability is a property of the UNIT, not of one entry.
+ *
+ * Every entry of a unit shares the same live body, so "is this undoable" must answer the same for all
+ * of them; computing it per entry made the body row say `false` (already current) while a support row
+ * of the SAME transaction said `true` — two entrances to one unit in one panel.
+ *
+ * Entries without a unit id keep the per-entry answer (they are their own singleton unit).
+ */
+import { type SkillVersion } from '@lmzhen/dsh-evolution-core';
+/** @param members - every entry that shares the unit (content and support chains together).
+ * @param liveHash - hash of the live body, or null when the skill is not readable.
+ * @returns one flag per unit id; unknown ids fall back to the per-entry rule. */
+export declare function unitUndoableFlags(members: readonly SkillVersion[], liveHash: string | null): Map<string, boolean>;
+/** @param entry - the entry being rendered. @param flags - the unit flags from {@link unitUndoableFlags}.
+ * @param liveHash - hash of the live body. @returns the row's `undoable` value. */
+export declare function undoableFor(entry: SkillVersion, flags: Map<string, boolean>, liveHash: string | null): boolean | undefined;
+//# sourceMappingURL=unit-undoable.d.ts.map

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-skill-history",
   "description": "Web surface for skill content history: loopback host routes plus the sidebar panel (community build)",
-  "version": "0.25.8",
+  "version": "0.26.15",
   "publishConfig": {
     "access": "public"
   },
@@ -30,23 +30,23 @@
   ],
   "license": "MIT",
   "dependencies": {
-    "@lmzhen/dsh-evolution-core": "^0.25.8"
+    "@lmzhen/dsh-evolution-core": "^0.26.15"
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
     "@deepseek-ai/dsh-host-webserver": "^0.2.0-rc.1",
     "react": "^18.2.0",
-    "@lmzhen/dsh-evolution-curator": "^0.25.8"
+    "@lmzhen/dsh-evolution-curator": "^0.26.15"
   },
   "devDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
     "@deepseek-ai/dsh-host-webserver": "^0.2.0-rc.1",
     "@types/react": "~18.3.1",
     "react": "^18.2.0",
-    "@lmzhen/dsh-evolution-core": "^0.25.8",
-    "@lmzhen/dsh-evolution-curator": "^0.25.8",
-    "@lmzhen/dsh-evolution-io": "^0.25.8",
-    "@lmzhen/dsh-evolution-io-node": "^0.25.8"
+    "@lmzhen/dsh-evolution-core": "^0.26.15",
+    "@lmzhen/dsh-evolution-curator": "^0.26.15",
+    "@lmzhen/dsh-evolution-io": "^0.26.15",
+    "@lmzhen/dsh-evolution-io-node": "^0.26.15"
   },
   "dsh": {
     "client": {