@lmzhen/dsh-evolution-approval 0.3.68 → 0.3.70

package/lib/index.js

@@ -233,7 +233,7 @@ var EvolutionApproval = class extends Service {
 				await this.state().releasePendingClaim(id, claimId);
 				return {
 					ok: false,
-					message: result.message
+					message: /changed since this plan|no entry matching|not found|no longer exists/i.test(result.message) ? `${result.message} — a sibling write from the same plan may have already changed this target, so repeated approves will keep failing. Reject this record unless the target changed again.` : result.message
 				};
 			}
 		} catch (error) {
@@ -244,7 +244,15 @@ var EvolutionApproval = class extends Service {
 				message: "Replay runner failed; the pending write remains pending."
 			};
 		}
-		if (!(await this.state().tryResolvePending(id, "approved", claimId)).applied) {
+		const resolution = await this.state().tryResolvePending(id, "approved", claimId).catch((error) => {
+			this.ctx.logger.warn(error);
+			return `failed: ${error instanceof Error ? error.message : String(error)}`;
+		});
+		if (typeof resolution === "string") return {
+			ok: false,
+			message: `Approved write "${id}" was replayed, but resolving the record failed (${resolution.slice(8)}) — the effect has LANDED while the audit row stays "executing". Verify the write manually; approve will not re-run it.`
+		};
+		if (!resolution.applied) {
 			if ((await this.state().listPending("rejected")).find((item) => item.id === id)) return {
 				ok: false,
 				message: `Approved write "${id}" was replayed, but the record was resolved to "rejected" concurrently — the effect has landed while the audit reads rejected. Verify the write and do NOT replay it.`

package/lib/types/index.d.ts

@@ -42,16 +42,18 @@ export interface ApprovalRequest {
      * (`overrideOf` → deployment `config.policy` → 'ask') and ignores this value
      * entirely; it is read only in assemblies WITHOUT that service. A caller
      * holding the session object must pass it as `session`: a bare `sessionId`
-     * cannot be probed (the platform reads `session.events` off the object).
+     * cannot be probed (the platform resolves the override from the object's
+     * session log view).
      */
     sessionPolicy?: 'ask' | 'never';
     /** 0.3.17 (E-25): the requesting session id, kept on the record for
      * audit attribution (staged AND resolved history). */
     sessionId?: string;
     /** 0.3.40 (V6-27 fix): the requesting SESSION OBJECT. The platform
-     * `approval.overrideOf(session: Session)` reads `session.events` — a bare id
-     * string throws a TypeError in the real implementation (`.events` on a
-     * string is undefined → `events.length`). Callers that hold the session
+     * `approval.overrideOf(session: Session)` needs the session's log view —
+     * `session.events` before 0.1.5, `session.snapshotEvents()` from 0.1.5 on —
+     * so a bare id string throws in the real implementation (a missing log view
+     * reads as `undefined.length`). Callers that hold the session
      * (model tools, review pipeline) MUST pass it so the session-level override
      * (`approval/policy` event) is honored; absent falls back to the id probe
      * (undefined for platform lookups) + the config chain. */

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-approval",
   "description": "Stage/pending approval service for Hermes-style self-evolution writes (community build)",
-  "version": "0.3.68",
+  "version": "0.3.70",
   "publishConfig": {
     "access": "public"
   },
@@ -33,17 +33,17 @@
     "@deepseek-ai/schemastery": "^3.18.1"
   },
   "peerDependencies": {
-    "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2",
+    "@deepseek-ai/dsh-invariants": "^0.1.5-rc.2",
     "@deepseek-ai/cordis": "^4.0.1",
-    "@lmzhen/dsh-evolution-state-storage": "^0.3.68",
-    "@lmzhen/dsh-evolution-state": "^0.3.68"
+    "@lmzhen/dsh-evolution-state-storage": "^0.3.70",
+    "@lmzhen/dsh-evolution-state": "^0.3.70"
   },
   "devDependencies": {
-    "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2",
-    "@lmzhen/dsh-evolution-state-storage": "^0.3.68",
-    "@lmzhen/dsh-evolution-state": "^0.3.68",
-    "@lmzhen/dsh-evolution-io": "^0.3.68",
-    "@lmzhen/dsh-evolution-io-node": "^0.3.68",
-    "@lmzhen/dsh-evolution-state-json": "^0.3.68"
+    "@deepseek-ai/dsh-invariants": "^0.1.5-rc.2",
+    "@lmzhen/dsh-evolution-state-storage": "^0.3.70",
+    "@lmzhen/dsh-evolution-state": "^0.3.70",
+    "@lmzhen/dsh-evolution-io": "^0.3.70",
+    "@lmzhen/dsh-evolution-io-node": "^0.3.70",
+    "@lmzhen/dsh-evolution-state-json": "^0.3.70"
   }
 }