@lmzhen/dsh-evolution-approval 0.32.3 → 0.32.4

package/lib/index.js

@@ -408,7 +408,9 @@ var EvolutionApproval = class extends Service {
 		for (const record of await this.state().listPending("pending")) {
 			const stagedAt = Date.parse(record.createdAt);
 			if (!Number.isFinite(stagedAt) || stagedAt > cutoff) continue;
-			if ((await this.state().tryResolvePending(record.id, "rejected")).applied) expired.push(record.id);
+			const claimId = this.newClaimId();
+			if (!await this.state().claimPending(record.id, claimId)) continue;
+			if ((await this.state().tryResolvePending(record.id, "rejected", claimId)).applied) expired.push(record.id);
 		}
 		if (expired.length > 0) this.ctx.logger.warn(`evolution-approval: ${expired.length} staged write(s) passed pendingTtlMs=${ttl} and were closed as rejected WITHOUT executing: ${expired.join(", ")} — stage them again if they are still wanted`);
 		return expired;
@@ -418,6 +420,7 @@ var EvolutionApproval = class extends Service {
 		return await this.state().listPending(status);
 	}
 	async approve(id, audit) {
+		await this.expireStale();
 		return await this.dedupe(`approve:${id}`, () => this.doApprove(id, audit));
 	}
 	/** S2-P2-22 (0.3.80): return an ORPHANED executing record to the pending
@@ -641,11 +644,6 @@ var EvolutionApproval = class extends Service {
 			message: `已批准写入「${id}」已重放，但了结记录返回空记录：请核验该写入。`
 		};
 		await this.recordDecision(resolution.record, audit);
-		this.ctx.emit("evolution/approval-decided", {
-			record: resolution.record,
-			action: "approve",
-			ok: true
-		});
 		return {
 			ok: true,
 			message: `已批准 ${record.kind} 写入「${id}」（${record.summary}）。`

package/lib/types/index.d.ts

@@ -18,7 +18,6 @@
  */
 import { Context, Service } from '@deepseek-ai/cordis';
 import type { Volatile } from '@deepseek-ai/cordis';
-import { type WriteConfirmMode } from '@lmzhen/dsh-evolution-core';
 import z from '@deepseek-ai/schemastery';
 import type { PendingKind, PendingRecord, PendingStatus } from '@lmzhen/dsh-evolution-state-storage';
 import type { DecisionAudit } from '@lmzhen/dsh-evolution-core/http-routes';
@@ -157,15 +156,6 @@ declare module '@deepseek-ai/cordis' {
         evolutionApproval: EvolutionApproval;
     }
     interface Events {
-        /**
-         * Emitted after an approval decision has been durably recorded.
-         * @mode emit
-         */
-        'evolution/approval-decided'(payload: {
-            readonly record: PendingRecord;
-            readonly action: 'approve' | 'reject';
-            readonly ok: boolean;
-        }): void;
     }
 }
 export interface Config {
@@ -184,12 +174,12 @@ export interface Config {
      */
     pendingTtlMs?: Volatile<number>;
     /**
-     * U-6: what the confirmation before an APPROVE does. The same three words the skill-write gate uses
-     * (`WriteConfirmMode`, whose home is `evolution-core/src/write-authority.ts` - moved there with the
-     * authority that carries it; the re-export from `write-gates.ts` was removed in G1/1.8) - and now the
-     * IMPORT below is what makes that true: without it the docblock claimed an import the file did not have.
+     * U-6: what the confirmation before an APPROVE does: auto (execute immediately) or ask (wait for a
+     * human answer). Legacy `timeout` is accepted only so an old value can migrate to `ask`; it is not
+     * exposed and is owned by ControlPlane when that lands.
      */
-    approvalConfirm?: Volatile<WriteConfirmMode>;
+    /** Legacy `timeout` is accepted only to migrate to `ask`; it is no longer exposed or implemented. */
+    approvalConfirm?: Volatile<'auto' | 'ask' | 'timeout'>;
 }
 export declare class EvolutionApproval extends Service {
     static inject: string[];

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.32.3",
+  "version": "0.32.4",
   "publishConfig": {
     "access": "public"
   },
@@ -27,18 +27,18 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.5",
-    "@lmzhen/dsh-evolution-core": "^0.32.3"
+    "@lmzhen/dsh-evolution-core": "^0.32.4"
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.5",
-    "@lmzhen/dsh-evolution-core": "^0.32.3",
-    "@lmzhen/dsh-evolution-state": "^0.32.3",
-    "@lmzhen/dsh-evolution-state-storage": "^0.32.3"
+    "@lmzhen/dsh-evolution-core": "^0.32.4",
+    "@lmzhen/dsh-evolution-state": "^0.32.4",
+    "@lmzhen/dsh-evolution-state-storage": "^0.32.4"
   },
   "devDependencies": {
-    "@lmzhen/dsh-evolution-core": "^0.32.3",
-    "@lmzhen/dsh-evolution-state": "^0.32.3",
-    "@lmzhen/dsh-evolution-state-storage": "^0.32.3"
+    "@lmzhen/dsh-evolution-core": "^0.32.4",
+    "@lmzhen/dsh-evolution-state": "^0.32.4",
+    "@lmzhen/dsh-evolution-state-storage": "^0.32.4"
   },
   "engines": {
     "dsh": ">=0.2.1-rc.2 <0.2.2-0"