@lmzhen/dsh-evolution-state-domain 0.3.63 → 0.3.65

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, canClaimPending, canResolvePending, releasedStatus } from "@lmzhen/dsh-evolution-state-storage";
+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";
 //#region lib/types/index.js
 /**
 * storage-domain provider for evolution state.
@@ -23,7 +23,7 @@ const reviewStateSchema = z.object({
 	turnsSinceMemory: z.number().int().nonnegative(),
 	turnsSinceSkill: z.number().int().nonnegative(),
 	lastTurn: z.number().int().nonnegative()
-});
+}).loose();
 /**
 * Curator-state record schema of the storage-domain table.
 * @internal Referenced only by this package's own tests and EVOLUTION_DOMAIN
@@ -36,7 +36,7 @@ const curatorStateSchema = z.object({
 	runCount: z.number().int().nonnegative(),
 	lastSummary: z.string(),
 	paused: z.boolean()
-});
+}).loose();
 /**
 * Pending record schema of the storage-domain table.
 * @internal Referenced only by this package's own tests and EVOLUTION_DOMAIN
@@ -63,7 +63,7 @@ const pendingSchema = z.object({
 	claimedAt: z.string().optional(),
 	origin: z.string().optional(),
 	sessionId: z.string().optional()
-});
+}).loose();
 /**
 * The evolution storage-domain spec: three schema-validated KV tables.
 * @internal Referenced only by this package's own tests and `open()` — not
@@ -116,40 +116,64 @@ function apply(ctx) {
 	const provider = {
 		name: PROVIDER_DOMAIN,
 		async loadReviewState(sessionId) {
-			return (await ensure()).table(REVIEW_STATE_TABLE).get(sessionId) ?? null;
+			const record = (await ensure()).table(REVIEW_STATE_TABLE).get(sessionId);
+			return record === void 0 ? null : structuredClone(record);
 		},
 		async saveReviewState(sessionId, record) {
-			await (await ensure()).table(REVIEW_STATE_TABLE).put(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));
 		},
 		async loadCuratorState() {
-			return (await ensure()).table(CURATOR_STATE_TABLE).get(CURATOR_STATE_KEY) ?? null;
+			const record = (await ensure()).table(CURATOR_STATE_TABLE).get(CURATOR_STATE_KEY);
+			return record === void 0 ? null : structuredClone(record);
 		},
 		async saveCuratorState(record) {
-			await (await ensure()).table(CURATOR_STATE_TABLE).put(CURATOR_STATE_KEY, record);
+			const issue = recordIssue(CURATOR_STATE_TABLE, record) ?? assertCloneable(record);
+			if (issue !== null) throw new Error(`evolution-state-domain: refusing to persist an invalid curator-state record: ${issue}`);
+			const parsed = curatorStateSchema.safeParse(record);
+			if (!parsed.success) throw new Error(`evolution-state-domain: refusing to persist an invalid curator-state record: ${parsed.error.issues[0]?.message ?? "schema mismatch"}`);
+			await (await ensure()).table(CURATOR_STATE_TABLE).put(CURATOR_STATE_KEY, cloneRecord(parsed.data));
 		},
 		async transactCuratorState(task) {
 			const table = (await ensure()).table(CURATOR_STATE_TABLE);
+			const guarded = (current) => {
+				const next = task(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}`);
+				return cloneRecord(next);
+			};
 			try {
-				await table.update(CURATOR_STATE_KEY, (current) => task(current) ?? current);
+				await table.update(CURATOR_STATE_KEY, (current) => guarded(current) ?? current);
 				return;
 			} catch (error) {
 				if (!(error instanceof DomainError && error.code === "missing-key")) throw error;
 			}
 			try {
-				await table.update(CURATOR_STATE_KEY, (current) => task(current) ?? current);
+				await table.update(CURATOR_STATE_KEY, (current) => guarded(current) ?? current);
 				return;
 			} catch (error) {
 				if (!(error instanceof DomainError && error.code === "missing-key")) throw error;
 			}
-			const next = task(null);
+			const next = guarded(null);
 			if (next !== null) await table.put(CURATOR_STATE_KEY, next);
 			else await table.delete(CURATOR_STATE_KEY);
 		},
 		async listPending(status = "pending") {
-			return [...(await ensure()).table(PENDING_TABLE).entries()].map(([, value]) => value).filter((record) => record.status === status);
+			return [...(await ensure()).table(PENDING_TABLE).entries()].filter(([, value]) => value.status === status).map(([, value]) => structuredClone(value));
 		},
 		async savePending(record) {
-			await (await ensure()).table(PENDING_TABLE).put(record.id, record);
+			const issue = recordIssue(PENDING_TABLE, record) ?? assertCloneable(record);
+			if (issue !== null) throw new Error(`evolution-state-domain: refusing to persist an invalid pending record: ${issue}`);
+			const parsed = pendingSchema.safeParse({
+				...record,
+				args: record.args
+			});
+			if (!parsed.success) throw new Error(`evolution-state-domain: refusing to persist an invalid pending record: ${parsed.error.issues[0]?.message ?? "schema mismatch"}`);
+			await (await ensure()).table(PENDING_TABLE).put(record.id, cloneRecord(parsed.data));
 		},
 		async claimPending(id, claimId) {
 			const table = (await ensure()).table(PENDING_TABLE);
@@ -166,7 +190,7 @@ function apply(ctx) {
 					};
 					return slot.record;
 				});
-				return slot.record === null ? null : { ...slot.record };
+				return slot.record === null ? null : structuredClone(slot.record);
 			} catch (error) {
 				if (error instanceof DomainError && error.code === "missing-key") return null;
 				throw error;
@@ -216,17 +240,21 @@ function apply(ctx) {
 					};
 					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) await table.delete(entry.key).catch((error) => {
-						ctx.logger.warn(`evolution-state-domain: pending-cap eviction for "${entry.key}" failed (will retry on the next resolve): ${error instanceof Error ? error.message : String(error)}`);
-					});
+					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;
+						await table.delete(entry.key).catch((error) => {
+							ctx.logger.warn(`evolution-state-domain: pending-cap eviction for "${entry.key}" failed (will retry on the next resolve): ${error instanceof Error ? error.message : String(error)}`);
+						});
+					}
 				}
 				const rawRecord = record;
 				if (resolved.record === null) return {
-					record: rawRecord === null ? null : { ...rawRecord },
+					record: rawRecord === null ? null : structuredClone(rawRecord),
 					applied: false
 				};
 				return {
-					record: { ...resolved.record },
+					record: structuredClone(resolved.record),
 					applied: true
 				};
 			} catch (error) {

package/lib/types/index.d.ts

@@ -22,7 +22,7 @@ export declare const reviewStateSchema: z.ZodObject<{
     turnsSinceMemory: z.ZodNumber;
     turnsSinceSkill: z.ZodNumber;
     lastTurn: z.ZodNumber;
-}, z.core.$strip>;
+}, z.core.$loose>;
 /**
  * Curator-state record schema of the storage-domain table.
  * @internal Referenced only by this package's own tests and EVOLUTION_DOMAIN
@@ -34,7 +34,7 @@ export declare const curatorStateSchema: z.ZodObject<{
     runCount: z.ZodNumber;
     lastSummary: z.ZodString;
     paused: z.ZodBoolean;
-}, z.core.$strip>;
+}, z.core.$loose>;
 /**
  * Pending record schema of the storage-domain table.
  * @internal Referenced only by this package's own tests and EVOLUTION_DOMAIN
@@ -52,7 +52,7 @@ export declare const pendingSchema: z.ZodObject<{
     claimedAt: z.ZodOptional<z.ZodString>;
     origin: z.ZodOptional<z.ZodString>;
     sessionId: z.ZodOptional<z.ZodString>;
-}, z.core.$strip>;
+}, z.core.$loose>;
 /**
  * The evolution storage-domain spec: three schema-validated KV tables.
  * @internal Referenced only by this package's own tests and `open()` — not

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.63",
+  "version": "0.3.65",
   "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.63"
+    "@lmzhen/dsh-evolution-state-storage": "^0.3.65"
   },
   "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.63"
+    "@lmzhen/dsh-evolution-state-storage": "^0.3.65"
   }
 }