@lmzhen/dsh-evolution-state-domain 0.3.66 → 0.3.68

package/README.md

@@ -24,4 +24,5 @@ Independent of request-prefix construction. This package does not alter the asse
 
 - Requires the host-plane `storage-domain` facility. The bundle row stays dormant when it is absent.
 - P2-4 (v15): the live pending table is BOUNDED — resolved (approved/rejected) records are kept to the most recent `PENDING_RESOLVED_CAP` (200, seam constant in `evolution-state-storage`); the oldest by `resolvedAt` are deleted on resolve. The audit ARCHIVE sidecar that the json provider maintains beyond the cap is json-specific (the domain seam has no sidecar facility) — resolved records past the cap are gone, not archived.
+- 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`) are pruned (delete failures warn and retry on the next save). The stamp is stripped on read, so the consumer-facing record shape is unchanged.
 

package/lib/index.js

@@ -1,6 +1,6 @@
 import { z } from "zod";
 import { DomainError, defineDomain, domainTable } from "@deepseek-ai/dsh-storage-domain";
-import { CURATOR_STATE_KEY, CURATOR_STATE_TABLE, PENDING_RESOLVED_CAP, PENDING_TABLE, PROVIDER_DOMAIN, REVIEW_STATE_TABLE, assertCloneable, canClaimPending, canResolvePending, cloneRecord, recordIssue, releasedStatus } from "@lmzhen/dsh-evolution-state-storage";
+import { CURATOR_STATE_KEY, CURATOR_STATE_TABLE, PENDING_RESOLVED_CAP, PENDING_TABLE, PROVIDER_DOMAIN, REVIEW_STATE_SESSION_CAP, REVIEW_STATE_TABLE, assertCloneable, canClaimPending, canResolvePending, cloneRecord, recordIssue, releasedStatus, selectPendingOverflow, selectSessionOverflow } from "@lmzhen/dsh-evolution-state-storage";
 //#region lib/types/index.js
 /**
 * storage-domain provider for evolution state.
@@ -117,14 +117,36 @@ function apply(ctx) {
 		name: PROVIDER_DOMAIN,
 		async loadReviewState(sessionId) {
 			const record = (await ensure()).table(REVIEW_STATE_TABLE).get(sessionId);
-			return record === void 0 ? null : structuredClone(record);
+			if (record === void 0) return null;
+			const { updatedAt: _stamp, ...rest } = record;
+			return structuredClone(rest);
 		},
 		async saveReviewState(sessionId, record) {
 			const issue = recordIssue(REVIEW_STATE_TABLE, record) ?? assertCloneable(record);
 			if (issue !== null) throw new Error(`evolution-state-domain: refusing to persist an invalid review-state record: ${issue}`);
 			const parsed = reviewStateSchema.safeParse(record);
 			if (!parsed.success) throw new Error(`evolution-state-domain: refusing to persist an invalid review-state record: ${parsed.error.issues[0]?.message ?? "schema mismatch"}`);
-			await (await ensure()).table(REVIEW_STATE_TABLE).put(sessionId, cloneRecord(parsed.data));
+			const table = (await ensure()).table(REVIEW_STATE_TABLE);
+			const stamped = {
+				...cloneRecord(parsed.data),
+				updatedAt: Date.now()
+			};
+			await table.put(sessionId, stamped);
+			const entries = [...table.entries()].map(([key, row]) => ({
+				key,
+				updatedAt: row.updatedAt ?? 0
+			})).filter((entry) => entry.key !== sessionId);
+			const victims = new Set(selectSessionOverflow(entries, {
+				keyOf: (entry) => entry.key,
+				stampOf: (entry) => entry.updatedAt
+			}, REVIEW_STATE_SESSION_CAP));
+			for (const entry of entries.filter((candidate) => victims.has(candidate.key))) {
+				const current = table.get(entry.key);
+				if (current === void 0 || (current.updatedAt ?? 0) !== entry.updatedAt) continue;
+				await table.delete(entry.key).catch((error) => {
+					ctx.logger.warn(`evolution-state-domain: review-state session-cap eviction for "${entry.key}" failed (will retry on the next save): ${error instanceof Error ? error.message : String(error)}`);
+				});
+			}
 		},
 		async loadCuratorState() {
 			const record = (await ensure()).table(CURATOR_STATE_TABLE).get(CURATOR_STATE_KEY);
@@ -140,7 +162,7 @@ function apply(ctx) {
 		async transactCuratorState(task) {
 			const table = (await ensure()).table(CURATOR_STATE_TABLE);
 			const guarded = (current) => {
-				const next = task(current);
+				const next = task(current === null ? null : cloneRecord(current));
 				if (next === null) return null;
 				const issue = recordIssue(CURATOR_STATE_TABLE, next) ?? assertCloneable(next);
 				if (issue !== null) throw new Error(`evolution-state-domain: refusing to persist an invalid curator-state record: ${issue}`);
@@ -230,16 +252,11 @@ function apply(ctx) {
 					return resolved.record;
 				});
 				if (resolved.record !== null) {
-					const resolvedEntries = [...table.entries()].map(([key, record]) => ({
+					const evictedRecords = new Set(selectPendingOverflow([...table.entries()].map(([, record]) => record), PENDING_RESOLVED_CAP));
+					const evicted = [...table.entries()].filter(([, record]) => evictedRecords.has(record)).map(([key, record]) => ({
 						key,
 						record
-					})).filter((entry) => (entry.record.status === "approved" || entry.record.status === "rejected") && entry.record.kind !== "capability");
-					const resolvedAtMs = (record) => {
-						const parsed = Date.parse(record.resolvedAt ?? "");
-						return Number.isNaN(parsed) ? Number.MAX_SAFE_INTEGER : parsed;
-					};
-					resolvedEntries.sort((a, b) => resolvedAtMs(a.record) - resolvedAtMs(b.record));
-					const evicted = resolvedEntries.slice(0, Math.max(0, resolvedEntries.length - PENDING_RESOLVED_CAP));
+					}));
 					for (const entry of evicted) {
 						const current = table.get(entry.key);
 						if (!(current !== void 0 && (current.status === "approved" || current.status === "rejected") && current.resolvedAt === entry.record.resolvedAt)) continue;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-state-domain",
   "description": "storage-domain provider for evolution state (community build)",
-  "version": "0.3.66",
+  "version": "0.3.68",
   "publishConfig": {
     "access": "public"
   },
@@ -36,13 +36,13 @@
     "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2",
     "@deepseek-ai/cordis": "^4.0.1",
     "@deepseek-ai/dsh-storage-domain": "^0.1.1-rc.2",
-    "@lmzhen/dsh-evolution-state-storage": "^0.3.66"
+    "@lmzhen/dsh-evolution-state-storage": "^0.3.68"
   },
   "devDependencies": {
     "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2",
     "@deepseek-ai/dsh-storage-domain": "^0.1.1-rc.2",
     "@deepseek-ai/dsh-storage": "^0.1.1-rc.2",
     "@deepseek-ai/dsh-storage-json": "^0.1.1-rc.2",
-    "@lmzhen/dsh-evolution-state-storage": "^0.3.66"
+    "@lmzhen/dsh-evolution-state-storage": "^0.3.68"
   }
 }