@lmzhen/dsh-evolution-state-json 0.3.21 → 0.3.22

package/lib/index.js

@@ -13,6 +13,23 @@ import { join } from "node:path";
 const name = "evolution-state-json";
 const inject = ["evolutionStateStorage", "evolutionIo"];
 const Config = z.object({ root: z.string().default("") });
+/** 0.3.22 (F-336): resolved (approved/rejected) audit records are capped in
+* the LIVE pending map so a long-running deployment never grows it without
+* bound; the oldest over the cap are archived (made package-private so the
+* archive sidecar and the provider enforce one number). */
+const PENDING_RESOLVED_CAP = 200;
+/** 0.3.22 (F-215): a record-map state file must parse to a non-null plain
+* object (a map of records) — valid JSON that is `null`/array/scalar is a
+* corrupt map that used to read as "empty" and was silently overwritten by
+* the next save. Only these four files are record maps; the archive sidecar
+* is a top-level ARRAY and must NOT be gated by this predicate. */
+const isPlainRecord = (value) => value !== null && typeof value === "object" && !Array.isArray(value);
+const RECORD_MAP_FILES = new Set([
+	"review-state.json",
+	"curator-state.json",
+	"pending-state.json",
+	"pending.json"
+]);
 function apply(ctx, rawConfig) {
 	const root = rawConfig.root || evolutionHome();
 	const io = () => ctx.evolutionIo.provider();
@@ -32,7 +49,9 @@ function apply(ctx, rawConfig) {
 		const raw = await io().readText(pathOf(file));
 		if (raw === null) return null;
 		try {
-			return JSON.parse(raw);
+			const parsed = JSON.parse(raw);
+			if (RECORD_MAP_FILES.has(file) && !isPlainRecord(parsed)) return await quarantine(file, raw, `expected a plain JSON object (map of records), got ${Array.isArray(parsed) ? "an array" : parsed === null ? "null" : typeof parsed}`);
+			return parsed;
 		} catch (error) {
 			return await quarantine(file, raw, error instanceof Error ? error.message : String(error));
 		}
@@ -49,6 +68,7 @@ function apply(ctx, rawConfig) {
 			let parsed = null;
 			if (current !== null) try {
 				parsed = JSON.parse(current);
+				if (RECORD_MAP_FILES.has(file) && !isPlainRecord(parsed)) return await quarantine(file, current, `expected a plain JSON object (map of records), got ${Array.isArray(parsed) ? "an array" : parsed === null ? "null" : typeof parsed}`);
 			} catch (error) {
 				return await quarantine(file, current, error instanceof Error ? error.message : String(error));
 			}
@@ -64,6 +84,47 @@ function apply(ctx, rawConfig) {
 			...current ?? {}
 		};
 	}
+	/** 0.3.22 (F-336): when the live pending map holds more than
+	* `PENDING_RESOLVED_CAP` resolved records, drop the oldest (by resolvedAt,
+	* then insertion order on ties) from the map and return them for archiving.
+	* Only approved/rejected records are candidates — pending/executing are
+	* live work and are never trimmed. Returns the pruned map (rather than
+	* mutating in place) plus the evicted records. */
+	function enforceResolvedCap(map) {
+		const resolved = Object.values(map).filter((record) => record.status === "approved" || record.status === "rejected");
+		if (resolved.length <= PENDING_RESOLVED_CAP) return {
+			map,
+			evicted: []
+		};
+		const overflow = resolved.length - PENDING_RESOLVED_CAP;
+		const oldest = resolved.sort((a, b) => {
+			return (a.resolvedAt ? Date.parse(a.resolvedAt) : Number.MAX_SAFE_INTEGER) - (b.resolvedAt ? Date.parse(b.resolvedAt) : Number.MAX_SAFE_INTEGER);
+		}).slice(0, overflow);
+		const evictIds = new Set(oldest.map((record) => record.id));
+		const kept = {};
+		for (const [key, value] of Object.entries(map)) if (!evictIds.has(key)) kept[key] = value;
+		return {
+			map: kept,
+			evicted: oldest
+		};
+	}
+	/** 0.3.22 (F-336): append evicted resolved records to an audit sidecar
+	* (top-level array, oldest-first). This is a best-effort audit aid: a
+	* corrupt/unreadable archive is skipped and an archive write failure must
+	* NEVER fail the resolve that triggered it — the live map is already
+	* trimmed, so the audit copy is allowed to fall behind. */
+	async function appendArchive(records) {
+		try {
+			await transactIo(io(), pathOf("pending-state-archive.json"), (current) => {
+				let archive = [];
+				if (current !== null) try {
+					const parsed = JSON.parse(current);
+					if (Array.isArray(parsed)) archive = parsed;
+				} catch {}
+				return JSON.stringify([...archive, ...records], null, 2);
+			});
+		} catch {}
+	}
 	const provider = {
 		name: "json",
 		async loadReviewState(sessionId) {
@@ -96,10 +157,7 @@ function apply(ctx, rawConfig) {
 			await mutate(async () => {
 				await jsonTransact("curator-state.json", (current) => {
 					const next = task(current?.primary ?? null);
-					if (next === null) {
-						if (current !== null) delete current.primary;
-						return current ?? {};
-					}
+					if (next === null) return current;
 					return {
 						...current ?? {},
 						primary: next
@@ -171,6 +229,7 @@ function apply(ctx, rawConfig) {
 					record: null,
 					applied: false
 				};
+				let evicted = [];
 				await jsonTransact("pending-state.json", async (current) => {
 					const map = {
 						...await readJson("pending.json") ?? {},
@@ -194,8 +253,11 @@ function apply(ctx, rawConfig) {
 						record: resolved,
 						applied: true
 					};
-					return map;
+					const pruned = enforceResolvedCap(map);
+					evicted = pruned.evicted;
+					return pruned.map;
 				});
+				if (evicted.length > 0) await appendArchive(evicted);
 				return result;
 			});
 		}

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.21",
+  "version": "0.3.22",
   "publishConfig": {
     "access": "public"
   },
@@ -31,17 +31,17 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.1",
-    "@lmzhen/dsh-evolution-core": "^0.3.21"
+    "@lmzhen/dsh-evolution-core": "^0.3.22"
   },
   "peerDependencies": {
     "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2",
     "@deepseek-ai/cordis": "^4.0.1",
-    "@lmzhen/dsh-evolution-io": "^0.3.21",
-    "@lmzhen/dsh-evolution-state-storage": "^0.3.21"
+    "@lmzhen/dsh-evolution-io": "^0.3.22",
+    "@lmzhen/dsh-evolution-state-storage": "^0.3.22"
   },
   "devDependencies": {
     "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2",
-    "@lmzhen/dsh-evolution-io": "^0.3.21",
-    "@lmzhen/dsh-evolution-state-storage": "^0.3.21"
+    "@lmzhen/dsh-evolution-io": "^0.3.22",
+    "@lmzhen/dsh-evolution-state-storage": "^0.3.22"
   }
 }