@lmzhen/dsh-evolution-state-json 0.3.74 → 0.3.77

package/README.md

@@ -26,3 +26,5 @@ Independent of request-prefix construction. This package does not alter the asse
 - V25-11 (v25): the review-state table is likewise BOUNDED — `REVIEW_STATE_SESSION_CAP` (500, seam constant) rows keyed by session; on every save the least-recently-active sessions (provider-stamped `updatedAt`, stored on disk only) are pruned. The stamp is stripped on read, so the consumer-facing record shape is unchanged.
 - JSON provider serializes writers inside one process AND through the IO backend's cross-process transact lock (an internal transact wrapper — not public API, audit v10 S-03 — wraps every mutation, 0.3.20/0.3.27) — this provider is NOT limited to single-process safety. The caveat below is about the DSH storage-domain providers (`storage-json` documents no cross-process write locking) when the DOMAIN provider is used instead; multi-process deployments should route the evolution domain to a backend with cross-process semantics such as SQLite or remote storage.
 
+**Runtime invariant:** No companion is published. The platform auto-assembles nothing and the family mounts no `<pkg>/invariant` cordis row, so a companion here would never execute (v37 S2.1 / I-3).
+

package/lib/index.js

@@ -268,11 +268,9 @@ function apply(ctx, rawConfig = {}) {
 					for (const entry of rawBak) if (entry && typeof entry.id === "string") ids.add(entry.id);
 				}
 			} catch (bakError) {
-				if (bakError?.name === QUARANTINE_ERROR_NAME) throw bakError;
 				ctx.logger.warn(`evolution-state-json: pending archive .bak unreadable (${bakError instanceof Error ? bakError.message : String(bakError)}) — archived ids that live only in the .bak do not exclude their legacy twins until the sidecar is readable again`);
 			}
 		} catch (error) {
-			if (error?.name === QUARANTINE_ERROR_NAME) throw error;
 			ctx.logger.warn(`evolution-state-json: pending archive sidecars unreadable (${error instanceof Error ? error.message : String(error)}) — the V5-02 legacy ghost-twin filter runs WITHOUT the archived-id exclusion until the archive is readable again`);
 		}
 		return ids;
@@ -287,14 +285,38 @@ function apply(ctx, rawConfig = {}) {
 		}
 		return retired;
 	}
+	let pendingKeyWarned = false;
+	function keyPendingById(map) {
+		const keyed = {};
+		if (map === null) return keyed;
+		let drifted = 0;
+		let dropped = 0;
+		for (const [key, record] of Object.entries(map)) {
+			const id = typeof record.id === "string" ? record.id : key;
+			if (key !== id) {
+				if (keyed[id] !== void 0) {
+					dropped += 1;
+					continue;
+				}
+				drifted += 1;
+			}
+			keyed[id] = record;
+		}
+		if ((drifted > 0 || dropped > 0) && !pendingKeyWarned) {
+			pendingKeyWarned = true;
+			ctx.logger.warn(`evolution-state-json: ${drifted} pending table entr(ies) were keyed by something other than record.id (${dropped} same-id residue row(s) dropped) — re-keyed by id, so a drifted row is reachable by claim/resolve again`);
+		}
+		return keyed;
+	}
 	async function retireLegacyOnce(legacy) {
 		if (legacyMigrated) return {};
+		const keyedLegacy = keyPendingById(legacy);
 		try {
 			const retired = {};
 			await jsonTransact(ctx, io, root, PENDING_STATE_FILE, async (fresh) => {
-				const merged = { ...fresh ?? {} };
+				const merged = keyPendingById(fresh);
 				const archivedIds = await readArchivedIds();
-				for (const [id, record] of Object.entries(legacy)) {
+				for (const [id, record] of Object.entries(keyedLegacy)) {
 					if (id in merged) continue;
 					if (archivedIds.has(id)) continue;
 					if (gateDroppedCurrent.has(id)) continue;
@@ -314,11 +336,12 @@ function apply(ctx, rawConfig = {}) {
 	}
 	async function loadPendingMap() {
 		const [current, legacy] = await Promise.all([readJson(PENDING_STATE_FILE), readJson(PENDING_LEGACY_FILE)]);
+		const keyed = keyPendingById(current);
 		if (legacy !== null) return {
 			...await retireLegacyOnce(legacy),
-			...current ?? {}
+			...keyed
 		};
-		return { ...current ?? {} };
+		return { ...keyed };
 	}
 	/** V6-01 (0.3.34): single-sourced legacy merge for the four mutation paths —
 	* the SAME exclusion as the retirement read path, so a mutation that runs
@@ -326,10 +349,12 @@ function apply(ctx, rawConfig = {}) {
 	* (the archive id set is read fresh on every merge — V11-B2; current-wins
 	* covers stale misses). */
 	async function mergedWithFilteredLegacy(legacy, current) {
-		if (legacy === null) return current;
+		const keyed = keyPendingById(current);
+		if (legacy === null) return keyed;
+		const archivedIds = await readArchivedIds();
 		return {
-			...filterLegacy(legacy, current, await readArchivedIds()),
-			...current
+			...filterLegacy(keyPendingById(legacy), keyed, archivedIds),
+			...keyed
 		};
 	}
 	/** 0.3.22 (F-336): when the live pending map holds more than

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-state-json",
   "description": "JSON-file evolution state provider over the IO seam (community build)",
-  "version": "0.3.74",
+  "version": "0.3.77",
   "publishConfig": {
     "access": "public"
   },
@@ -18,10 +18,6 @@
       "types": "./lib/types/index.d.ts",
       "default": "./lib/index.js"
     },
-    "./invariant": {
-      "types": "./lib/types/invariant.d.ts",
-      "default": "./lib/invariant.js"
-    },
     "./package.json": "./package.json"
   },
   "files": [
@@ -31,18 +27,16 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.1",
-    "@lmzhen/dsh-evolution-core": "^0.3.74"
+    "@lmzhen/dsh-evolution-core": "^0.3.77"
   },
   "peerDependencies": {
-    "@deepseek-ai/dsh-invariants": "^0.1.5-rc.2",
     "@deepseek-ai/cordis": "^4.0.1",
-    "@lmzhen/dsh-evolution-io": "^0.3.74",
-    "@lmzhen/dsh-evolution-state-storage": "^0.3.74"
+    "@lmzhen/dsh-evolution-io": "^0.3.77",
+    "@lmzhen/dsh-evolution-state-storage": "^0.3.77"
   },
   "devDependencies": {
-    "@deepseek-ai/dsh-invariants": "^0.1.5-rc.2",
-    "@lmzhen/dsh-evolution-io": "^0.3.74",
-    "@lmzhen/dsh-evolution-state-storage": "^0.3.74",
-    "@lmzhen/dsh-evolution-io-node": "^0.3.74"
+    "@lmzhen/dsh-evolution-io": "^0.3.77",
+    "@lmzhen/dsh-evolution-state-storage": "^0.3.77",
+    "@lmzhen/dsh-evolution-io-node": "^0.3.77"
   }
 }

package/lib/invariant.js

@@ -1,8 +0,0 @@
-//#region lib/types/invariant.js
-const PACKAGE_NAME = "@lmzhen/dsh-evolution-state-json";
-const name = "evolution-state-json-invariant";
-const inject = ["invariants"];
-const install = () => {};
-const apply = (ctx) => Promise.resolve(ctx.invariants.register(PACKAGE_NAME, install));
-//#endregion
-export { apply, inject, name };

package/lib/types/invariant.d.ts

@@ -1,5 +0,0 @@
-import type { Context } from '@deepseek-ai/cordis';
-export declare const name = "evolution-state-json-invariant";
-export declare const inject: string[];
-export declare const apply: (ctx: Context) => Promise<() => void>;
-//# sourceMappingURL=invariant.d.ts.map