@lmzhen/dsh-evolution-review 0.1.0-rc.32 → 0.1.0-rc.34

package/lib/index.js

@@ -173,6 +173,7 @@ function apply(ctx, rawConfig) {
 					}
 				}
 			});
+			const childReads = run.localAgent ? collectReadSkillNames(run.localAgent.session) : /* @__PURE__ */ new Set();
 			const result = await run.result;
 			await run.dispose();
 			if (!result.structured) return true;
@@ -188,7 +189,7 @@ function apply(ctx, rawConfig) {
 				maxSkillContentChars: snapshot?.skillContentChars ?? DEFAULT_SKILL_CONTENT_CHARS
 			});
 			const acceptedSkillOps = validation.accepted.skillOps ?? [];
-			const skippedUnread = filterUnreadSkillOps(acceptedSkillOps, collectReadSkillNames(session));
+			const skippedUnread = filterUnreadSkillOps(acceptedSkillOps, new Set([...collectReadSkillNames(session), ...childReads]));
 			validation.accepted.skillOps = acceptedSkillOps;
 			const actions = await executePlan(validation.accepted);
 			const evidenceQuotes = [...validation.accepted.memoryOps ?? [], ...acceptedSkillOps].reduce((total, op) => total + (Array.isArray(op.evidence) ? op.evidence.length : 0), 0);
@@ -258,8 +259,25 @@ function apply(ctx, rawConfig) {
 				ok: false,
 				message: decision.message
 			};
+			if (approval.isEnabled === false) return await runnerDirect(kind, runnerArgs);
 			return await approval.run(kind, runnerArgs);
 		}
+		/** Direct execution for the approval-disabled case (parallel to executeSkillDirect). */
+		async function runnerDirect(kind, args) {
+			if (kind === "memory") {
+				const memory = ctx.get("memory");
+				const op = args;
+				if (!memory?.applyBatch) return void 0;
+				return await memory.applyBatch(op.target === "user" ? "user" : "memory", [{
+					action: op.action ?? "add",
+					facts: op.facts ?? op.content,
+					old_text: op.old_text
+				}]);
+			}
+			const wrapped = args ?? {};
+			if (!wrapped.operation) return void 0;
+			return await executeSkillDirect(wrapped.operation);
+		}
 		/**
 		* Approval-disabled path: execute the skill op through SkillLibrary with an
 		* EXPLICIT background_review origin. Going through ctx.tools.execute would
@@ -280,7 +298,14 @@ function apply(ctx, rawConfig) {
 			if (op.action === "create") return await library.create(name, op.content ?? "", origin);
 			if (op.action === "edit" || op.action === "update") return await library.update(name, op.content ?? "", origin);
 			if (op.action === "patch") return await library.patch(name, op.old_string ?? "", op.new_string ?? "", op.file_path ?? "", false, origin);
-			if (op.action === "delete") return await library.archive(name, op.absorbed_into ? { absorbedInto: op.absorbed_into } : {});
+			if (op.action === "delete") {
+				const into = (op.absorbed_into ?? "").trim();
+				if (!into || !await library.read(into)) return {
+					ok: false,
+					message: "delete requires an existing absorbed_into target"
+				};
+				return await library.archive(name, { absorbedInto: into });
+			}
 			if (op.action === "write_file") return await library.writeSupportFile(name, op.file_path ?? "", op.file_content ?? op.content ?? "", origin);
 			if (op.action === "remove_file") return await library.removeSupportFile(name, op.file_path ?? "", origin);
 			return {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-review",
   "description": "Background review orchestration (community build)",
-  "version": "0.1.0-rc.32",
+  "version": "0.1.0-rc.34",
   "publishConfig": {
     "access": "public"
   },
@@ -33,8 +33,8 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.1",
-    "@lmzhen/dsh-evolution-core": "^0.1.0-rc.32",
-    "@lmzhen/dsh-evolution-plan-validator": "^0.1.0-rc.32"
+    "@lmzhen/dsh-evolution-core": "^0.1.0-rc.34",
+    "@lmzhen/dsh-evolution-plan-validator": "^0.1.0-rc.34"
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
@@ -43,7 +43,7 @@
     "@deepseek-ai/dsh-llm": "^0.1.0-rc.6",
     "@deepseek-ai/dsh-session": "^0.1.0-rc.6",
     "@deepseek-ai/dsh-tools": "^0.1.0-rc.6",
-    "@lmzhen/dsh-evolution-state": "^0.1.0-rc.32"
+    "@lmzhen/dsh-evolution-state": "^0.1.0-rc.34"
   },
   "devDependencies": {
     "@deepseek-ai/dsh-agent": "^0.1.0-rc.6",
@@ -52,8 +52,8 @@
     "@deepseek-ai/dsh-llm": "^0.1.0-rc.6",
     "@deepseek-ai/dsh-session": "^0.1.0-rc.6",
     "@deepseek-ai/dsh-tools": "^0.1.0-rc.6",
-    "@lmzhen/dsh-evolution-core": "^0.1.0-rc.32",
-    "@lmzhen/dsh-evolution-plan-validator": "^0.1.0-rc.32",
-    "@lmzhen/dsh-evolution-state": "^0.1.0-rc.32"
+    "@lmzhen/dsh-evolution-core": "^0.1.0-rc.34",
+    "@lmzhen/dsh-evolution-plan-validator": "^0.1.0-rc.34",
+    "@lmzhen/dsh-evolution-state": "^0.1.0-rc.34"
   }
 }