@lmzhen/dsh-evolution-learning-graph 0.3.67 → 0.3.69

package/lib/index.js

@@ -13,6 +13,9 @@ import { SKILL_NAME_RE, SkillLibrary, contentHash, evolutionIoAdapter, relatedSk
 * edit/delete can reject a stale index (E-21).
 * @module @lmzhen/dsh-evolution-learning-graph
 */
+/** V27 INS-04: how many skill files one `/graph` batch reads concurrently.
+* Bounded so a large library cannot open every file at once (EMFILE). */
+const SKILL_READ_BATCH = 16;
 function graphDensity(graph) {
 	const linked = /* @__PURE__ */ new Set();
 	let relatedEdges = 0;
@@ -220,12 +223,12 @@ function apply(ctx, rawConfig = {}) {
 	const graphSkillsRoot = resolveSkillsRoot(rawConfig);
 	ctx.inject(["commands"], (commandCtx) => {
 		const commands = commandCtx.commands;
-		commandCtx.effect(() => commands.register({
+		const graphCommand = {
 			name: "graph",
 			description: "Show the learning graph, or act on a node: graph [detail|edit|delete] <nodeId>",
 			recordInput: false,
 			input: { hint: "[detail|edit|delete] <nodeId> [text]" },
-			handler: async (invocation) => {
+			run: async (invocation) => {
 				const ok = (text) => ({
 					kind: "success",
 					text
@@ -241,24 +244,49 @@ function apply(ctx, rawConfig = {}) {
 				const usage = usageService;
 				const memory = memoryService;
 				const io = ioService;
-				const session = invocation.agent?.session;
-				const input = (invocation.rawInput ?? "").trim();
+				const session = invocation.agent.session;
+				const input = invocation.rawInput.trim();
 				const detail = /^detail\s+(\S+)$/.exec(input);
 				if (detail && detail[1]) return await nodeDetail(detail[1]);
 				const edit = /^edit\s+(\S+)\s+([\s\S]+)$/.exec(input);
-				if (edit && edit[1] && edit[2] !== void 0) return await nodeEdit(edit[1], edit[2]);
+				if (edit && edit[1] && edit[2] !== void 0) {
+					if (!edit[1].startsWith("memory:")) {
+						if (await withSkills().read(edit[1]).catch(() => null) === null) return err(`Skill "${edit[1]}" not found in the live skill library — nothing to edit. (The graph may be showing a stale node.)`);
+					}
+					return await nodeEdit(edit[1], edit[2]);
+				}
 				const remove = /^delete\s+(\S+)$/.exec(input);
-				if (remove && remove[1]) return await nodeDelete(remove[1]);
+				if (remove && remove[1]) {
+					if (!remove[1].startsWith("memory:")) {
+						if (await withSkills().read(remove[1]).catch(() => null) === null) return err(`Skill "${remove[1]}" not found in the live skill library — nothing to delete.`);
+					}
+					return await nodeDelete(remove[1]);
+				}
 				const directory = await renderGraph();
 				if (input !== "") return err(`Unknown graph subcommand "${input.split(" ")[0]}". ${directory}`);
 				return ok(directory);
 				async function renderGraph() {
-					const usageMap = await usage.report();
+					const usageReport = await usage.report();
+					const usageMap = new Map([...usageReport].filter(([, record]) => record.state !== "archived"));
 					const memoryEntries = await memory.read("memory");
 					const userEntries = await memory.read("user");
 					const skills = withSkills();
 					const names = [...usageMap.keys()];
-					const contents = await Promise.all(names.map((name) => skills.read(name)));
+					const contents = [];
+					let unreadable = 0;
+					for (let offset = 0; offset < names.length; offset += SKILL_READ_BATCH) {
+						const batch = names.slice(offset, offset + SKILL_READ_BATCH);
+						const batchContents = await Promise.all(batch.map(async (name) => {
+							try {
+								return await skills.read(name);
+							} catch {
+								unreadable += 1;
+								return null;
+							}
+						}));
+						contents.push(...batchContents);
+					}
+					if (unreadable > 0) ctx.logger.warn(`evolution-learning-graph: ${unreadable} skill file(s) could not be read; their related_skills edges are missing from this graph`);
 					const related = /* @__PURE__ */ new Map();
 					names.forEach((name, index) => {
 						const content = contents[index];
@@ -291,7 +319,7 @@ function apply(ctx, rawConfig = {}) {
 					if (parsed === null) return err(`Invalid node id "${id}". Skill names or memory:<source>:<index> expected.`);
 					if (parsed.kind === "skill") {
 						const approval = ctx.get("evolutionApproval");
-						const origins = resolveOrigins(session?.header?.origin);
+						const origins = resolveOrigins(session.header.origin);
 						if (approval) {
 							const sessionPolicyEdit = effectiveSessionPolicy(ctx, session);
 							const stagesForeground = approval.stageForeground !== false;
@@ -312,8 +340,8 @@ function apply(ctx, rawConfig = {}) {
 									libraryOrigin: origins.library
 								},
 								origin: origins.approval,
-								...session?.id ? { sessionId: session.id } : {},
-								...session ? { session } : {},
+								...session.id ? { sessionId: session.id } : {},
+								session,
 								...sessionPolicy !== void 0 ? { sessionPolicy } : {}
 							});
 							if (decision.action === "staged") return ok(decision.message);
@@ -326,7 +354,7 @@ function apply(ctx, rawConfig = {}) {
 					if (!check.ok) return err(check.message ?? "Memory index check failed.");
 					const memoryApproval = ctx.get("evolutionApproval");
 					if (memoryApproval) {
-						const originsM = resolveOrigins(session?.header?.origin);
+						const originsM = resolveOrigins(session.header.origin);
 						const sessionPolicyM = effectiveSessionPolicy(ctx, session);
 						if (memoryApproval.isEnabled !== false && sessionPolicyM !== "never" && (originsM.approval === "background_review" || memoryApproval.stageForeground !== false) && !memoryApproval.hasRunner("memory")) return err("Graph memory write cannot be staged: no memory replay runner is registered — mount the tool-memory row (evolution-all bundle, or the evolution-preset overlay; the host bundle does not carry it) or disable evolution-approval.");
 						const decision = await memoryApproval.request({
@@ -341,8 +369,8 @@ function apply(ctx, rawConfig = {}) {
 								}]
 							},
 							origin: originsM.approval,
-							...session?.id ? { sessionId: session.id } : {},
-							...session ? { session } : {},
+							...session.id ? { sessionId: session.id } : {},
+							session,
 							...sessionPolicyM !== void 0 ? { sessionPolicy: sessionPolicyM } : {}
 						});
 						if (decision.action === "staged") return ok(decision.message);
@@ -360,7 +388,7 @@ function apply(ctx, rawConfig = {}) {
 					if (parsed.kind === "skill") {
 						const approval = ctx.get("evolutionApproval");
 						if (approval) {
-							const origins = resolveOrigins(session?.header?.origin);
+							const origins = resolveOrigins(session.header.origin);
 							const sessionPolicy = effectiveSessionPolicy(ctx, session);
 							if (approval.isEnabled !== false && sessionPolicy !== "never" && (origins.approval === "background_review" || approval.stageForeground !== false) && !approval.hasRunner("skill")) return err("Graph skill delete cannot be staged: no skill replay runner is registered — mount the tool-skill-manage row (evolution-agent preset) or disable evolution-approval.");
 							const decision = await approval.request({
@@ -375,8 +403,8 @@ function apply(ctx, rawConfig = {}) {
 									libraryOrigin: origins.library
 								},
 								origin: origins.approval,
-								...session?.id ? { sessionId: session.id } : {},
-								...session ? { session } : {},
+								...session.id ? { sessionId: session.id } : {},
+								session,
 								...sessionPolicy !== void 0 ? { sessionPolicy } : {}
 							});
 							if (decision.action === "staged") return ok(decision.message);
@@ -389,7 +417,7 @@ function apply(ctx, rawConfig = {}) {
 					if (!check.ok) return err(check.message ?? "Memory index check failed.");
 					const memoryApproval = ctx.get("evolutionApproval");
 					if (memoryApproval) {
-						const origins = resolveOrigins(session?.header?.origin);
+						const origins = resolveOrigins(session.header.origin);
 						const sessionPolicy = effectiveSessionPolicy(ctx, session);
 						if (memoryApproval.isEnabled !== false && sessionPolicy !== "never" && (origins.approval === "background_review" || memoryApproval.stageForeground !== false) && !memoryApproval.hasRunner("memory")) return err("Graph memory delete cannot be staged: no memory replay runner is registered — mount the tool-memory row (evolution-all bundle, or the evolution-preset overlay; the host bundle does not carry it) or disable evolution-approval.");
 						const decision = await memoryApproval.request({
@@ -403,8 +431,8 @@ function apply(ctx, rawConfig = {}) {
 								}]
 							},
 							origin: origins.approval,
-							...session?.id ? { sessionId: session.id } : {},
-							...session ? { session } : {},
+							...session.id ? { sessionId: session.id } : {},
+							session,
 							...sessionPolicy !== void 0 ? { sessionPolicy } : {}
 						});
 						if (decision.action === "staged") return ok(decision.message);
@@ -415,8 +443,13 @@ function apply(ctx, rawConfig = {}) {
 					}]);
 					return result.ok ? ok(result.message) : err(result.message);
 				}
-			}
-		}), "evolution-learning-graph.command");
+			},
+			handler: (invocation) => graphCommand.run(invocation).catch((error) => ({
+				kind: "error",
+				text: `graph: command failed: ${error instanceof Error ? error.message : String(error)}\nCheck the node id and the skills/memory files under the evolution root, then retry.`
+			}))
+		};
+		commandCtx.effect(() => commands.register(graphCommand), "evolution-learning-graph.command");
 	});
 }
 //#endregion

package/lib/types/index.d.ts

@@ -37,24 +37,6 @@ export interface GraphDensity {
     edgesPerNode: number;
     isolatedPct: number;
 }
-/**
- * The command invocation contract (N1, v12): the platform command handler
- * freezes the invoking agent onto the invocation object — evolution-commands
- * `/evolution learn` already reads `invocation.agent` — so the graph can pass
- * `agent.session` to the approval service (tool-skill-manage pattern) instead
- * of deriving every approval as foreground.
- */
-export interface GraphInvocation {
-    rawInput?: string;
-    agent?: {
-        session?: {
-            id?: string;
-            header?: {
-                origin?: string;
-            };
-        };
-    };
-}
 export declare function graphDensity(graph: LearningGraph): GraphDensity;
 /**
  * Render one node's display line for `/evolution graph`.

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-learning-graph",
   "description": "Learning graph over skills and memory (community build)",
-  "version": "0.3.67",
+  "version": "0.3.69",
   "publishConfig": {
     "access": "public"
   },
@@ -31,14 +31,16 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.1",
-    "@lmzhen/dsh-evolution-approval": "^0.3.67",
-    "@lmzhen/dsh-evolution-core": "^0.3.67"
+    "@lmzhen/dsh-evolution-approval": "^0.3.69",
+    "@lmzhen/dsh-evolution-core": "^0.3.69"
   },
   "peerDependencies": {
+    "@deepseek-ai/dsh-commands": "^0.1.1-rc.2",
     "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2",
     "@deepseek-ai/cordis": "^4.0.1"
   },
   "devDependencies": {
+    "@deepseek-ai/dsh-commands": "^0.1.1-rc.2",
     "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2"
   }
 }