@lmzhen/dsh-evolution-state-domain 0.3.79 → 0.3.81

package/lib/index.js

@@ -90,13 +90,19 @@ const OPEN_MAX_ATTEMPTS = 3;
 const OPEN_RETRY_BASE_MS = 100;
 /** Retry cannot heal these — the stored record or the medium fails its own
 * schema/version, or the backend cannot serve the domain at all. Both storage
-* vocabularies (DomainError, StorageError) carry the code as stable API. */
+* vocabularies (DomainError, StorageError) carry the code as stable API.
+* S1-D3: `already-open` (DomainError, upstream open()'s single-open constraint)
+* is deterministic too — a competing consumer of the `evolution` domain never
+* goes away on retry, so burning the full backoff budget (3× loadAll) and then
+* leaving `opening` cleared only re-runs the same futile budget on every later
+* call. */
 const DETERMINISTIC_OPEN_CODES = new Set([
 	"invalid-record",
 	"malformed-medium",
 	"version-mismatch",
 	"facet-unsupported",
 	"backend-not-found",
+	"already-open",
 	"closed"
 ]);
 function isTransientOpenFailure(error) {
@@ -130,6 +136,7 @@ function apply(ctx) {
 		domain = await opening;
 		return domain;
 	}
+	let pendingKeyWarned = false;
 	const provider = {
 		name: PROVIDER_DOMAIN,
 		async loadReviewState(sessionId) {
@@ -202,7 +209,33 @@ function apply(ctx) {
 			else await table.delete(CURATOR_STATE_KEY);
 		},
 		async listPending(status = "pending") {
-			return [...(await ensure()).table(PENDING_TABLE).entries()].filter(([, value]) => value.status === status).map(([, value]) => structuredClone(value));
+			const table = (await ensure()).table(PENDING_TABLE);
+			const entries = [...table.entries()];
+			const drifted = [];
+			for (const [key, record] of entries) if (key !== record.id && typeof record.id === "string" && record.id !== "") drifted.push({
+				key,
+				record
+			});
+			if (drifted.length > 0) {
+				if (!pendingKeyWarned) {
+					pendingKeyWarned = true;
+					ctx.logger.warn(`evolution-state-domain: ${drifted.length} pending table entr(ies) were keyed by something other than record.id — re-keyed by id, so a drifted row is reachable by claim/resolve again`);
+				}
+				for (const { key, record } of drifted) try {
+					await table.put(record.id, structuredClone(record));
+					await table.delete(key);
+				} catch (error) {
+					ctx.logger.warn(`evolution-state-domain: pending re-key for "${key}" failed (will retry on the next list): ${error instanceof Error ? error.message : String(error)}`);
+					break;
+				}
+			}
+			const final = drifted.length > 0 ? [...table.entries()] : entries;
+			const seen = /* @__PURE__ */ new Set();
+			return final.filter(([, value]) => {
+				if (seen.has(value.id)) return false;
+				seen.add(value.id);
+				return value.status === status;
+			}).map(([, value]) => structuredClone(value));
 		},
 		async savePending(record) {
 			const issue = recordIssue(PENDING_TABLE, record) ?? assertCloneable(record);

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.79",
+  "version": "0.3.81",
   "publishConfig": {
     "access": "public"
   },
@@ -31,12 +31,12 @@
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
     "@deepseek-ai/dsh-storage-domain": "^0.1.5-rc.2",
-    "@lmzhen/dsh-evolution-state-storage": "^0.3.79"
+    "@lmzhen/dsh-evolution-state-storage": "^0.3.81"
   },
   "devDependencies": {
     "@deepseek-ai/dsh-storage-domain": "^0.1.5-rc.2",
     "@deepseek-ai/dsh-storage": "^0.1.5-rc.2",
     "@deepseek-ai/dsh-storage-json": "^0.1.5-rc.2",
-    "@lmzhen/dsh-evolution-state-storage": "^0.3.79"
+    "@lmzhen/dsh-evolution-state-storage": "^0.3.81"
   }
 }