@lmzhen/dsh-evolution-state-json 0.3.25 → 0.3.27

package/lib/index.js

@@ -18,6 +18,16 @@ const Config = z.object({ root: z.string().default("") });
 * 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.27 (V4-01): the audit sidecar (pending-state-archive.json) is bounded
+* at this many resolved records. Past it the oldest history rotates to a
+* `.bak` sidecar, so the file — and the full-array rewrite on every append —
+* never grows without bound. */
+const ARCHIVE_RESOLVED_CAP = 5e3;
+/** 0.3.27 (V4-01): an archive entry's dedupe identity. The same audit record
+* (id + status + resolvedAt) must never appear twice; the read-only legacy
+* `pending.json` merge used to re-introduce an evicted record on the next
+* resolve and archive it again, growing the sidecar without bound. */
+const pendingArchiveKey = (record) => `${record.id}\u0000${record.status}\u0000${record.resolvedAt ?? ""}`;
 /** 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
@@ -31,7 +41,7 @@ const RECORD_MAP_FILES = new Set([
 	"pending.json"
 ]);
 function apply(ctx, rawConfig) {
-	const root = rawConfig.root || evolutionHome();
+	const root = (rawConfig.root ?? "").trim() || evolutionHome();
 	const io = () => ctx.evolutionIo.provider();
 	const pathOf = (file) => join(root, file);
 	/** 0.3.17 (E-9): a malformed state file used to parse to `null` and was then
@@ -48,13 +58,14 @@ function apply(ctx, rawConfig) {
 	async function readJson(file) {
 		const raw = await io().readText(pathOf(file));
 		if (raw === null) return null;
+		let parsed;
 		try {
-			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;
+			parsed = JSON.parse(raw);
 		} catch (error) {
 			return await quarantine(file, raw, error instanceof Error ? error.message : String(error));
 		}
+		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;
 	}
 	/**
 	* Cross-process JSON-file RMW (v3-audit M-8): every read-modify-write state
@@ -66,11 +77,13 @@ function apply(ctx, rawConfig) {
 	async function jsonTransact(file, task) {
 		await transactIo(ctx.evolutionIo.provider(), pathOf(file), async (current) => {
 			let parsed = null;
-			if (current !== null) try {
-				parsed = JSON.parse(current);
+			if (current !== null) {
+				try {
+					parsed = JSON.parse(current);
+				} catch (error) {
+					return await quarantine(file, current, error instanceof Error ? error.message : String(error));
+				}
 				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));
 			}
 			const next = await task(parsed);
 			return next === null ? null : JSON.stringify(next, null, 2);
@@ -112,16 +125,29 @@ function apply(ctx, rawConfig) {
 	* (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. */
+	* trimmed, so the audit copy is allowed to fall behind.
+	* 0.3.27 (V4-01): dedupe by id+status+resolvedAt before appending (the
+	* read-only legacy `pending.json` re-introduces an evicted record on the
+	* next resolve) and rotate the sidecar to `.bak` past ARCHIVE_RESOLVED_CAP
+	* so neither the file nor the per-append full-array rewrite grows without
+	* bound. */
 	async function appendArchive(records) {
 		try {
-			await transactIo(io(), pathOf("pending-state-archive.json"), (current) => {
+			await transactIo(io(), pathOf("pending-state-archive.json"), async (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);
+				const seen = new Set(archive.map(pendingArchiveKey));
+				const fresh = records.filter((record) => !seen.has(pendingArchiveKey(record)));
+				if (fresh.length === 0) return current;
+				const next = [...archive, ...fresh];
+				if (next.length > ARCHIVE_RESOLVED_CAP) {
+					await io().writeText(pathOf("pending-state-archive.json.bak"), JSON.stringify(archive, null, 2)).catch(() => {});
+					return JSON.stringify(fresh, null, 2);
+				}
+				return JSON.stringify(next, null, 2);
 			});
 		} catch {}
 	}

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