@lmzhen/dsh-evolution-skill-catalog 0.3.71 → 0.3.73

package/lib/index.js

@@ -76,6 +76,40 @@ function apply(ctx, rawConfig = {}) {
 	let summariesStamp = null;
 	let summariesEpoch = 0;
 	let lastScanWarn = "";
+	let invocationCache = null;
+	const warnedFrontmatter = /* @__PURE__ */ new Set();
+	const warnFrontmatterOnce = (name, key, detail) => {
+		const slot = `${name}:${key}`;
+		if (warnedFrontmatter.has(slot)) return;
+		warnedFrontmatter.add(slot);
+		ctx.logger.warn(`evolution-skill-catalog: "${name}" frontmatter ${detail}`);
+	};
+	const frontmatterBool = (name, data, key) => {
+		if (!Object.hasOwn(data, key)) return void 0;
+		const value = data[key];
+		if (typeof value === "boolean") return value;
+		if (value === 1 || value === "1") return true;
+		if (value === 0 || value === "0") return false;
+		if (typeof value === "string") {
+			const lowered = value.toLowerCase();
+			if (lowered === "true" || lowered === "yes" || lowered === "on") return true;
+			if (lowered === "false" || lowered === "no" || lowered === "off") return false;
+		}
+		warnFrontmatterOnce(name, key, `"${key}" must be a boolean; the row-level default applies until fixed`);
+	};
+	const invocationFromFrontmatter = (name, data) => {
+		for (const legacy of [
+			"disableModelInvocation",
+			"modelInvocable",
+			"userInvocable"
+		]) if (Object.hasOwn(data, legacy)) warnFrontmatterOnce(name, legacy, `field "${legacy}" is not accepted by the upstream registry; use the canonical kebab key`);
+		const disableModel = frontmatterBool(name, data, "disable-model-invocation");
+		const user = frontmatterBool(name, data, "user-invocable");
+		return {
+			modelInvocable: disableModel !== void 0 ? !disableModel : invocation.modelInvocable,
+			userInvocable: user !== void 0 ? user : invocation.userInvocable
+		};
+	};
 	async function summaries() {
 		const stamp = await io.mtime?.(library.root) ?? null;
 		if (summariesCache !== null && (stamp === null || summariesStamp === stamp)) return summariesCache;
@@ -96,6 +130,15 @@ function apply(ctx, rawConfig = {}) {
 			control?.invalidate();
 			lastScanWarn = "";
 		}
+		const invocationMap = /* @__PURE__ */ new Map();
+		for (const summary of scanned) {
+			const raw = await io.readText(join(summary.path, "SKILL.md")).catch(() => null);
+			if (raw === null) continue;
+			const parsed = parseFrontmatter(raw);
+			if (!parsed) continue;
+			invocationMap.set(summary.name, invocationFromFrontmatter(summary.name, parsed.frontmatter));
+		}
+		invocationCache = invocationMap;
 		if (epochAtScanStart === summariesEpoch) {
 			summariesCache = scanned;
 			summariesStamp = stamp;
@@ -105,8 +148,10 @@ function apply(ctx, rawConfig = {}) {
 	const dropSummariesCache = () => {
 		summariesEpoch += 1;
 		summariesCache = null;
+		invocationCache = null;
 		control?.invalidate();
 	};
+	const invocationFor = (name) => invocationCache?.get(name) ?? invocation;
 	const provider = {
 		name: "dsh-evolution",
 		async list(options) {
@@ -124,7 +169,7 @@ function apply(ctx, rawConfig = {}) {
 				name: summary.name,
 				description: summary.description,
 				...summary.whenToUse !== void 0 ? { whenToUse: summary.whenToUse } : {},
-				invocation,
+				invocation: invocationFor(summary.name),
 				source: "user-dsh",
 				provider: "dsh-evolution",
 				rank: EVOLUTION_SKILL_RANK,
@@ -153,7 +198,7 @@ function apply(ctx, rawConfig = {}) {
 				name,
 				description: summary.description,
 				...summary.whenToUse !== void 0 ? { whenToUse: summary.whenToUse } : {},
-				invocation,
+				invocation: invocationFor(name),
 				source: "user-dsh",
 				provider: "dsh-evolution",
 				resourceBase: {

package/lib/types/index.d.ts

@@ -13,9 +13,14 @@ export declare const name = "evolution-skill-catalog";
 export declare const inject: string[];
 export interface Config {
     root?: string;
-    /** Whether catalog skills are advertised/loadable by the model. */
+    /** Row-level DEFAULT invocation policy. OPT-10 (2026-09): a skill's own
+     * SKILL.md frontmatter (`disable-model-invocation` / `user-invocable`) now
+     * overrides this per skill — before, the row default was stamped onto every
+     * candidate and a user's `disable-model-invocation: true` in the shared
+     * tree was silently re-advertised as model-invocable. */
     modelInvocable?: boolean;
-    /** Whether catalog skills are advertised/loadable from user surfaces. */
+    /** Whether catalog skills are advertised/loadable from user surfaces
+     * (per-skill `user-invocable` frontmatter overrides — see modelInvocable). */
     userInvocable?: boolean;
     /** Restrict the published catalog to these skill names (empty = all). */
     includeSkillNames?: string[];

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-skill-catalog",
   "description": "Native ctx.skills provider for the evolution-managed skill tree (community build)",
-  "version": "0.3.71",
+  "version": "0.3.73",
   "publishConfig": {
     "access": "public"
   },
@@ -31,18 +31,18 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.1",
-    "@lmzhen/dsh-evolution-core": "^0.3.71"
+    "@lmzhen/dsh-evolution-core": "^0.3.73"
   },
   "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",
-    "@lmzhen/dsh-evolution-io": "^0.3.71"
+    "@lmzhen/dsh-evolution-io": "^0.3.73"
   },
   "devDependencies": {
     "@deepseek-ai/dsh-invariants": "^0.1.5-rc.2",
     "@deepseek-ai/dsh-skill": "^0.1.5-rc.2",
-    "@lmzhen/dsh-evolution-core": "^0.3.71",
-    "@lmzhen/dsh-evolution-io": "^0.3.71"
+    "@lmzhen/dsh-evolution-core": "^0.3.73",
+    "@lmzhen/dsh-evolution-io": "^0.3.73"
   }
 }