@lmzhen/dsh-evolution-approval 0.3.23 → 0.3.25

package/README.md

@@ -35,3 +35,5 @@ Independent of request-prefix construction. This package does not alter the asse
 
 
 - `approve()` deduplicates concurrent approvals inside one process, and state providers resolve the pending record atomically. However, the replay runner executes **before** that atomic resolution, so two OS processes approving the same id can each perform the write once while only one process wins the audit transition. Run approvals from a single writer process, or make replay runners idempotent when multi-process approval is required.
+- **Concurrent approve + reject on the same id (F-204).** Inside one process the dedupe keys are `approve:<id>` / `reject:<id>`, so the two paths are not serialized against each other. When an approve runner is slow, a reject resolves the still-executing record to `rejected` **without holding a claim**; the runner may then complete and the write can still land while the audit history reads `rejected` — the write effect, not the audit verdict, is what actually persists (`写效果以实际为准`). `reject` on an executing record reports this and asks you to verify the write state manually. Only reject a write after confirming no approve is in flight, or verify the write effect manually afterwards.
+- **No staged-content freshness re-validation (F-328).** The staged record stores the `args` snapshot captured at request time and replays exactly those args, but does not record a content hash, so `approve()` does **not** re-check whether the on-disk skill/memory the write targets changed since staging. The write is applied as staged regardless. The pending surface (`/evolution pending --detail`) exposes the staged `args` so you can review what will actually be replayed before approving; there is no automatic drift warning if the target changed in the meantime.

package/lib/index.js

@@ -129,7 +129,7 @@ var EvolutionApproval = class extends Service {
 			if (!await this.state().claimPending(id, claimId)) {
 				if ((await this.state().listPending("executing")).find((item) => item.id === id)) return (await this.state().tryResolvePending(id, "rejected")).applied ? {
 					ok: true,
-					message: `Rejected executing write "${id}" (crashed approve cleaned up; verify the write state manually).`
+					message: `Rejected executing write "${id}" (no claim held — this may race an in-flight approve runner; if the write already landed its effect stands as-is. Verify the write state manually).`
 				} : {
 					ok: false,
 					message: `Pending write "${id}" could not be rejected: it resolved concurrently.`
@@ -238,7 +238,10 @@ function normalizeSummary(input) {
 	const trimmed = input.summary.length > 120 ? `${input.summary.slice(0, 117)}...` : input.summary;
 	if (input.kind === "memory") {
 		const candidate = input.args;
-		if (Array.isArray(candidate?.operations) && candidate.operations.length > 1) return `memory ${candidate.target ?? "memory"} batch of ${candidate.operations.length} operations`;
+		if (Array.isArray(candidate?.operations) && candidate.operations.length > 1) {
+			const targetLabel = candidate.target ?? "memory";
+			return `memory ${targetLabel === "memory" ? "" : `${targetLabel} `}batch of ${candidate.operations.length} operations`;
+		}
 	}
 	if (input.kind === "skill" && /^skill delete /.test(trimmed)) return `${trimmed} (warning: archive)`;
 	return trimmed;

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.23",
+  "version": "0.3.25",
   "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.23",
-    "@lmzhen/dsh-evolution-state": "^0.3.23"
+    "@lmzhen/dsh-evolution-state-storage": "^0.3.25",
+    "@lmzhen/dsh-evolution-state": "^0.3.25"
   },
   "devDependencies": {
     "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2",
-    "@lmzhen/dsh-evolution-state-storage": "^0.3.23",
-    "@lmzhen/dsh-evolution-state": "^0.3.23",
-    "@lmzhen/dsh-evolution-io": "^0.3.23",
-    "@lmzhen/dsh-evolution-io-node": "^0.3.23",
-    "@lmzhen/dsh-evolution-state-json": "^0.3.23"
+    "@lmzhen/dsh-evolution-state-storage": "^0.3.25",
+    "@lmzhen/dsh-evolution-state": "^0.3.25",
+    "@lmzhen/dsh-evolution-io": "^0.3.25",
+    "@lmzhen/dsh-evolution-io-node": "^0.3.25",
+    "@lmzhen/dsh-evolution-state-json": "^0.3.25"
   }
 }