@lmzhen/dsh-tool-skill-manage 0.3.73 → 0.3.76

package/README.md

@@ -32,3 +32,5 @@ Mutations (create/edit/update/patch/delete/write_file/remove_file/restructure) p
 
 
 - No known durable consumer gaps at this time. Runtime contracts are covered by package and boundary tests.
+
+**Runtime invariant:** No companion is published. The platform auto-assembles nothing and the family mounts no `<pkg>/invariant` cordis row, so a companion here would never execute (v37 S2.1 / I-3).

package/lib/index.js

@@ -1,7 +1,7 @@
 import { effectiveSessionPolicy } from "@lmzhen/dsh-evolution-approval";
 import z from "@deepseek-ai/schemastery";
 import { defineTool } from "@deepseek-ai/dsh-tools";
-import { DEFAULT_SKILL_LIMITS, DSH_AUTHORING_STANDARDS, SKILLS_GUIDANCE, SKILLS_GUIDANCE_SECTION_ORDER, SKILL_ACTION_REQUIRED_FIELDS, SkillLibrary, authoringFeedback, clampedNumber, computeDedupGroups, contentHash, evolutionIoAdapter, parseFrontmatter, resolveOrigins, resolveSkillsRoot } from "@lmzhen/dsh-evolution-core";
+import { DEFAULT_SKILL_LIMITS, DSH_AUTHORING_STANDARDS, SKILLS_GUIDANCE, SKILLS_GUIDANCE_SECTION_ORDER, SKILL_ACTION_REQUIRED_FIELDS, authoringFeedback, callingScope, clampedNumber, computeDedupGroups, contentHash, evolutionIoAdapter, isPresent, isReviewChannelSession, isUnknown, newSkillLibrary, parseFrontmatter, probePresent, probeUnknown, resolveOrigins } from "@lmzhen/dsh-evolution-core";
 //#region lib/types/index.js
 /**
 * Model-facing skill_manage tool over ctx.evolutionIo + ctx.skillUsage.
@@ -58,23 +58,30 @@ function apply(ctx, rawConfig = {}) {
 		if (value !== void 0 && result !== value) numericClamped.push(name);
 		return result;
 	};
-	const libraryOptions = {
-		maxNameLength: limit("maxSkillNameLength", rawConfig.maxSkillNameLength, DEFAULT_SKILL_LIMITS.maxNameLength),
-		maxDescriptionLength: limit("maxDescriptionLength", rawConfig.maxDescriptionLength, DEFAULT_SKILL_LIMITS.maxDescriptionLength),
-		maxSkillContentChars: limit("maxSkillContentChars", rawConfig.maxSkillContentChars, DEFAULT_SKILL_LIMITS.maxSkillContentChars),
-		maxSkillFileBytes: limit("maxSkillFileBytes", rawConfig.maxSkillFileBytes, DEFAULT_SKILL_LIMITS.maxSkillFileBytes)
-	};
-	const library = new SkillLibrary(resolveSkillsRoot(rawConfig), io, libraryOptions, (event) => {
-		ctx.emit("evolution/skill-mutated", event);
-	}, void 0, [...rawConfig.threatExemptLabels ?? []]);
+	const library = newSkillLibrary({
+		config: rawConfig,
+		io,
+		limits: {
+			maxNameLength: limit("maxSkillNameLength", rawConfig.maxSkillNameLength, DEFAULT_SKILL_LIMITS.maxNameLength),
+			maxDescriptionLength: limit("maxDescriptionLength", rawConfig.maxDescriptionLength, DEFAULT_SKILL_LIMITS.maxDescriptionLength),
+			maxSkillContentChars: limit("maxSkillContentChars", rawConfig.maxSkillContentChars, DEFAULT_SKILL_LIMITS.maxSkillContentChars),
+			maxSkillFileBytes: limit("maxSkillFileBytes", rawConfig.maxSkillFileBytes, DEFAULT_SKILL_LIMITS.maxSkillFileBytes)
+		},
+		ctx,
+		threatExemptLabels: rawConfig.threatExemptLabels
+	});
 	if (numericClamped.length > 0) ctx.logger.warn(`tool-skill-manage: ${numericClamped.join(", ")} provided an invalid value; falling back to the default`);
+	let crossSourceViewWarned = false;
 	const catalogWinner = async (name) => {
 		const catalog = ctx.get("skills");
-		if (catalog === void 0) return void 0;
+		if (catalog === void 0) return probeUnknown("the skills service is not mounted");
+		const scope = callingScope(ctx);
 		try {
-			return (await catalog.list()).find((summary) => summary.name === name);
-		} catch {
-			return;
+			const summaries = await catalog.list(scope === void 0 ? void 0 : { scope });
+			if (summaries.length === 0) return probeUnknown(`the catalog view is empty in the ${scope === void 0 ? "global" : "calling"} scope`);
+			return probePresent(summaries.find((summary) => summary.name === name));
+		} catch (error) {
+			return probeUnknown(`the catalog lookup failed (${error instanceof Error ? error.message : String(error)})`);
 		}
 	};
 	async function executeCore(args, origin = "foreground") {
@@ -298,13 +305,24 @@ function apply(ctx, rawConfig = {}) {
 		isConcurrencySafe: () => false,
 		async execute(args, exec) {
 			let operation = args;
-			const origins = resolveOrigins(exec.agent?.session?.header.origin);
+			const origins = resolveOrigins(exec.agent?.session?.header.origin, isReviewChannelSession(exec.agent?.session?.id));
 			const reviewOrigin = origins.approval;
 			const libraryOrigin = origins.library;
 			const sessionPolicy = effectiveSessionPolicy(ctx, exec.agent?.session);
 			if (typeof args.name === "string" && args.name !== "" && args.action !== "list" && args.action !== "review" && args.action !== "pin" && args.action !== "unpin") {
-				const winner = await catalogWinner(args.name);
-				if (winner !== void 0 && winner.provider !== "dsh-evolution") {
+				const probe = await catalogWinner(args.name);
+				if (isUnknown(probe)) {
+					if (rawConfig.strictCrossSource === true) return {
+						ok: false,
+						message: `Refused: the cross-source check could not be performed — ${probe.reason}. Fix the catalog view (or drop strictCrossSource) before writing "${args.name}".`,
+						skills: []
+					};
+					if (!crossSourceViewWarned) {
+						crossSourceViewWarned = true;
+						ctx.logger.warn(`skill_manage: cross-source check skipped for this session — ${probe.reason}`);
+					}
+				} else if (isPresent(probe) && probe.value !== void 0 && probe.value.provider !== "dsh-evolution") {
+					const winner = probe.value;
 					const message = `skill "${args.name}" resolves to a higher-priority "${winner.source}" skill (provider "${winner.provider}"); this write lands on the family copy, which the catalog does NOT serve — edit the "${winner.source}" copy or rename.`;
 					if (rawConfig.strictCrossSource === true) return {
 						ok: false,
@@ -317,10 +335,10 @@ function apply(ctx, rawConfig = {}) {
 			const approval = ctx.get("evolutionApproval");
 			if (approval && args.action !== "list" && args.action !== "review" && args.action !== "pin" && args.action !== "unpin") {
 				if ((args.action === "update" || args.action === "edit") && typeof args.name === "string" && args.name !== "") {
-					const stageCurrent = await library.read(args.name).catch(() => null);
-					if (stageCurrent !== null) operation = {
+					const stageCurrent = await library.read(args.name).catch(() => void 0);
+					if (stageCurrent !== void 0) operation = {
 						...args,
-						staged_from_sha256: contentHash(stageCurrent)
+						staged_from_sha256: stageCurrent === null ? "absent" : contentHash(stageCurrent)
 					};
 				}
 				if ((args.action === "write_file" || args.action === "remove_file") && typeof args.name === "string" && args.name !== "" && typeof args.file_path === "string" && args.file_path !== "") {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-tool-skill-manage",
   "description": "Model-facing skill_manage tool (community build)",
-  "version": "0.3.73",
+  "version": "0.3.76",
   "publishConfig": {
     "access": "public"
   },
@@ -18,10 +18,6 @@
       "types": "./lib/types/index.d.ts",
       "default": "./lib/index.js"
     },
-    "./invariant": {
-      "types": "./lib/types/invariant.d.ts",
-      "default": "./lib/invariant.js"
-    },
     "./package.json": "./package.json"
   },
   "files": [
@@ -31,26 +27,24 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.1",
-    "@lmzhen/dsh-evolution-approval": "^0.3.73",
-    "@lmzhen/dsh-evolution-core": "^0.3.73"
+    "@lmzhen/dsh-evolution-approval": "^0.3.76",
+    "@lmzhen/dsh-evolution-core": "^0.3.76"
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
-    "@deepseek-ai/dsh-invariants": "^0.1.5-rc.2",
     "@deepseek-ai/dsh-skill": "^0.1.5-rc.2",
     "@deepseek-ai/dsh-system-prompt": "^0.1.5-rc.2",
     "@deepseek-ai/dsh-tools": "^0.1.5-rc.2",
-    "@lmzhen/dsh-evolution-io": "^0.3.73",
-    "@lmzhen/dsh-skill-usage": "^0.3.73"
+    "@lmzhen/dsh-evolution-io": "^0.3.76",
+    "@lmzhen/dsh-skill-usage": "^0.3.76"
   },
   "devDependencies": {
     "@deepseek-ai/dsh-agent-loop-testkit": "^0.1.5-rc.2",
-    "@deepseek-ai/dsh-invariants": "^0.1.5-rc.2",
     "@deepseek-ai/dsh-skill": "^0.1.5-rc.2",
     "@deepseek-ai/dsh-system-prompt": "^0.1.5-rc.2",
     "@deepseek-ai/dsh-tools": "^0.1.5-rc.2",
-    "@lmzhen/dsh-evolution-core": "^0.3.73",
-    "@lmzhen/dsh-evolution-io": "^0.3.73",
-    "@lmzhen/dsh-skill-usage": "^0.3.73"
+    "@lmzhen/dsh-evolution-core": "^0.3.76",
+    "@lmzhen/dsh-evolution-io": "^0.3.76",
+    "@lmzhen/dsh-skill-usage": "^0.3.76"
   }
 }

package/lib/invariant.js

@@ -1,8 +0,0 @@
-//#region lib/types/invariant.js
-const PACKAGE_NAME = "@lmzhen/dsh-tool-skill-manage";
-const name = "tool-skill-manage-invariant";
-const inject = ["invariants"];
-const install = () => {};
-const apply = (ctx) => Promise.resolve(ctx.invariants.register(PACKAGE_NAME, install));
-//#endregion
-export { apply, inject, name };

package/lib/types/invariant.d.ts

@@ -1,5 +0,0 @@
-import type { Context } from '@deepseek-ai/cordis';
-export declare const name = "tool-skill-manage-invariant";
-export declare const inject: string[];
-export declare const apply: (ctx: Context) => Promise<() => void>;
-//# sourceMappingURL=invariant.d.ts.map