@lmzhen/dsh-evolution-approval 0.3.62 → 0.3.64

package/README.md

@@ -12,7 +12,8 @@ replayable only when the corresponding tool package is composed.
 Run the companion invariant and tests:
 
 ```sh
-node node_modules/vitest/vitest.mjs run packages/evolution/evolution-approval/tests
+node node_modules/vitest/vitest.mjs run packages/evolution/evolution-approval/tests   # overlay layout
+# flat mirror: node node_modules/vitest/vitest.mjs run packages/evolution-approval/tests
 ```
 
 ## Model Experience

package/lib/index.js

@@ -31,13 +31,19 @@ var EvolutionApproval = class extends Service {
 		stageForeground: z.boolean().default(true)
 	});
 	enabled;
-	stageForeground;
+	stageForegroundConfig;
+	/** P3 (v16): public read for staging pre-checks (learning-graph refuses a
+	* stage that no runner could replay only when foreground writes stage at
+	* all). Mirrors `this.stageForeground`. */
+	get stageForeground() {
+		return this.stageForegroundConfig;
+	}
 	runners = /* @__PURE__ */ new Map();
 	inFlight = /* @__PURE__ */ new Map();
 	constructor(ctx, config = {}) {
 		super(ctx, "evolutionApproval");
 		this.enabled = config.enabled ?? false;
-		this.stageForeground = config.stageForeground ?? true;
+		this.stageForegroundConfig = config.stageForeground ?? true;
 	}
 	state() {
 		return this.ctx.evolutionState;
@@ -139,7 +145,7 @@ var EvolutionApproval = class extends Service {
 					message: `Pending write "${id}" is not in the pending window (rotated or resolved).`
 				};
 			}
-			const resolution = await this.state().tryResolvePending(id, "rejected");
+			const resolution = await this.state().tryResolvePending(id, "rejected", claimId);
 			if (!resolution.applied || !resolution.record) {
 				await this.state().releasePendingClaim(id, claimId);
 				return {
@@ -195,10 +201,16 @@ var EvolutionApproval = class extends Service {
 		const runner = this.runners.get(record.kind);
 		if (!runner) {
 			if (record.kind === "capability") {
-				if (!(await this.state().tryResolvePending(id, "approved")).applied) return {
-					ok: false,
-					message: `Pending write "${id}" was already resolved.`
-				};
+				if (!(await this.state().tryResolvePending(id, "approved", claimId)).applied) {
+					if ((await this.state().listPending("rejected")).find((item) => item.id === id)) return {
+						ok: false,
+						message: `Capability "${id}" was approved, but the record was resolved to "rejected" concurrently — no code ran and nothing needs replaying; verify before re-submitting.`
+					};
+					return {
+						ok: false,
+						message: `Pending write "${id}" was already resolved by another writer.`
+					};
+				}
 				return {
 					ok: true,
 					message: "Capability approved for manual activation in Creator mode (no code was executed)."
@@ -253,7 +265,7 @@ function normalizeSummary(input) {
 			return `memory ${targetLabel === "memory" ? "" : `${targetLabel} `}batch of ${candidate.operations.length} operations`;
 		}
 	}
-	if (input.kind === "skill" && /(?:^|\s)delete\s/.test(trimmed)) return `${trimmed} (warning: archive)`;
+	if (input.kind === "skill" && /^(?:skill|graph)\s+delete\s/.test(trimmed)) return `${trimmed} (warning: archive)`;
 	return trimmed;
 }
 //#endregion

package/lib/types/index.d.ts

@@ -66,6 +66,11 @@ export type ApprovalLike = {
     }>;
     hasRunner(kind: PendingKind): boolean;
     isEnabled?: boolean;
+    /** P3 (v16): whether foreground-origin writes stage (learning-graph's
+     * hasRunner pre-check reads it so it only refuses writes that would
+     * actually be staged). Optional: absent means "unknown" and callers must
+     * not pre-refuse on it. */
+    stageForeground?: boolean;
     registerRunner(kind: PendingKind, runner: WriteRunner): () => void;
     list(status?: PendingStatus): Promise<PendingRecord[]>;
     approve(id: string): Promise<{
@@ -106,7 +111,11 @@ export declare class EvolutionApproval extends Service {
     static inject: string[];
     static Config: Schema<Config>;
     private readonly enabled;
-    private readonly stageForeground;
+    private readonly stageForegroundConfig;
+    /** P3 (v16): public read for staging pre-checks (learning-graph refuses a
+     * stage that no runner could replay only when foreground writes stage at
+     * all). Mirrors `this.stageForeground`. */
+    get stageForeground(): boolean;
     private readonly runners;
     private readonly inFlight;
     constructor(ctx: Context, config?: Config);

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.62",
+  "version": "0.3.64",
   "publishConfig": {
     "access": "public"
   },
@@ -35,15 +35,15 @@
   "peerDependencies": {
     "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2",
     "@deepseek-ai/cordis": "^4.0.1",
-    "@lmzhen/dsh-evolution-state-storage": "^0.3.62",
-    "@lmzhen/dsh-evolution-state": "^0.3.62"
+    "@lmzhen/dsh-evolution-state-storage": "^0.3.64",
+    "@lmzhen/dsh-evolution-state": "^0.3.64"
   },
   "devDependencies": {
     "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2",
-    "@lmzhen/dsh-evolution-state-storage": "^0.3.62",
-    "@lmzhen/dsh-evolution-state": "^0.3.62",
-    "@lmzhen/dsh-evolution-io": "^0.3.62",
-    "@lmzhen/dsh-evolution-io-node": "^0.3.62",
-    "@lmzhen/dsh-evolution-state-json": "^0.3.62"
+    "@lmzhen/dsh-evolution-state-storage": "^0.3.64",
+    "@lmzhen/dsh-evolution-state": "^0.3.64",
+    "@lmzhen/dsh-evolution-io": "^0.3.64",
+    "@lmzhen/dsh-evolution-io-node": "^0.3.64",
+    "@lmzhen/dsh-evolution-state-json": "^0.3.64"
   }
 }