@lmzhen/dsh-evolution-curator 0.24.0 → 0.25.0

package/lib/index.js

@@ -4,7 +4,7 @@ import { join } from "node:path";
 import { BlockAssembler, createUserMessage } from "@deepseek-ai/dsh-llm";
 import z from "@deepseek-ai/schemastery";
 import { preferProvider } from "@lmzhen/dsh-evolution-io";
-import { CURATOR_DRY_RUN_BANNER, CURATOR_PROMPT, DEFAULT_ARCHIVE_AFTER_DAYS, DEFAULT_CURATOR_BOOT_GRACE_SECONDS, DEFAULT_CURATOR_INTERVAL_HOURS, DEFAULT_CURATOR_MODEL, DEFAULT_CURATOR_REVIEW_MAX_TOKENS, DEFAULT_HEALTH_THRESHOLDS, DEFAULT_MIN_IDLE_HOURS, DEFAULT_REVIEW_TIMEOUT_MS, DEFAULT_SKILL_LIMITS, DEFAULT_STALE_AFTER_DAYS, EvolutionGateSet, INSTANCE_KEYS, MAX_TIMER_DELAY_MS, PROMPT_BUNDLE, SKILL_NAME_RE, buildCuratorRunReport, claimInstance, clampOnce, clampedNumber, computeDedupGroups, computeLifecycleTransitions, computePrefixClusters, computeQualityScores, computeScopeView, contentHash, emptyRecord, entryTarget, evolutionHome, evolutionIoAdapter, foldCuratorFields, isPresent, isUnknown, liveBool, loadSuppressedNames, loadUsage, markerEntryName, mutateUsage, newSkillLibrary, paramRowId, parseCuratorNominations, parseFrontmatter, pickWithPolicy, policyStageLimits, probeList, probeText, readNumberParam, relatedSkillNames, releaseInstance, renderCuratorReportMarkdown, reportTime, reportsSweepLockTarget, sessionLastEventTime, sweepReports, transactIo, unwrapLive, updateSuppressedNames, usageObserved, userSetKeys, verifyPromptBundle } from "@lmzhen/dsh-evolution-core";
+import { CURATOR_DRY_RUN_BANNER, CURATOR_PROMPT, DEFAULT_ARCHIVE_AFTER_DAYS, DEFAULT_CURATOR_BOOT_GRACE_SECONDS, DEFAULT_CURATOR_INTERVAL_HOURS, DEFAULT_CURATOR_MODEL, DEFAULT_CURATOR_REVIEW_MAX_TOKENS, DEFAULT_HEALTH_THRESHOLDS, DEFAULT_MIN_IDLE_HOURS, DEFAULT_REVIEW_TIMEOUT_MS, DEFAULT_SKILL_LIMITS, DEFAULT_STALE_AFTER_DAYS, EvolutionGateSet, INSTANCE_KEYS, MAX_TIMER_DELAY_MS, PROMPT_BUNDLE, SKILL_NAME_RE, buildCuratorRunReport, claimInstance, clampOnce, clampedNumber, computeDedupGroups, computeLifecycleTransitions, computePrefixClusters, computeQualityScores, computeScopeView, contentHash, emptyRecord, entryTarget, evolutionHome, evolutionIoAdapter, foldCuratorFields, isPresent, isUnknown, lintSkill, liveBool, loadSuppressedNames, loadUsage, markerEntryName, mutateUsage, newSkillLibrary, paramRowId, parseCuratorNominations, parseFrontmatter, pickWithPolicy, policyStageLimits, probeList, probeText, readNumberParam, relatedSkillNames, releaseInstance, renderCuratorReportMarkdown, reportTime, reportsSweepLockTarget, sessionLastEventTime, sweepReports, transactIo, unwrapLive, updateSuppressedNames, usageObserved, userSetKeys, verifyPromptBundle } from "@lmzhen/dsh-evolution-core";
 //#region lib/types/index.js
 /**
 * Deterministic skill lifecycle curator with interval gate and archive.
@@ -33,6 +33,24 @@ function gateConsolidations(consolidations, gates) {
 	const gateSet = gates instanceof EvolutionGateSet ? gates : new EvolutionGateSet(gates);
 	return consolidations.filter((n) => !gateSet.isBlocked(n.from) && !gateSet.isBlocked(n.into));
 }
+/**
+* G5 (in-use protection): which skills are POINTED AT by others, and by whom.
+*
+* Pure: derived from the bodies already in hand, using `relatedSkillNames` — the single reader of
+* that frontmatter field — so this map can never disagree with the catalog or the audit. The value
+* is a Set of SOURCES, not a boolean, because a refusal has to be able to name the referrers.
+* @param contents - skill name to its whole SKILL.md text.
+* @returns target name to the set of skills naming it; targets nobody names are absent.
+*/
+function referencedSkillNames(contents) {
+	const referencedBy = /* @__PURE__ */ new Map();
+	for (const [owner, text] of contents) for (const target of relatedSkillNames(text, owner)) {
+		const sources = referencedBy.get(target) ?? /* @__PURE__ */ new Set();
+		sources.add(owner);
+		referencedBy.set(target, sources);
+	}
+	return referencedBy;
+}
 var EvolutionCurator = class extends Service {
 	static inject = ["evolutionIo"];
 	static Config = z.object({
@@ -691,8 +709,9 @@ var EvolutionCurator = class extends Service {
 			const text = await this.skills.read(name);
 			if (text) contents.set(name, text);
 		}
+		const referencedBy = referencedSkillNames(contents);
 		const protectedNames = await this.protectedNameMap();
-		const dedupMembers = [...new Set(computeDedupGroups({ contents }).groups.filter((group) => group.length >= 2).flat())].filter((name) => !protectedNames.has(name));
+		const dedupMembers = [...new Set(computeDedupGroups({ contents }).groups.filter((group) => group.length >= 2).flat())].filter((name) => !protectedNames.has(name) && !referencedBy.has(name));
 		await this.scoreTree(usage, treeNames, contents);
 		const result = computeLifecycleTransitions(usage, {
 			staleAfterDays: lifecycle.staleAfterDays,
@@ -1219,10 +1238,47 @@ var EvolutionCurator = class extends Service {
 				patchCount: record.patch_count,
 				readCount: record.view_count
 			} : void 0);
-			if (assessment && assessment.verdict !== "healthy") rows.push({
+			if (assessment !== null) {
+				rows.push({
+					name: summary.name,
+					verdict: assessment.verdict,
+					dims: assessment.dims,
+					findings: assessment.findings
+				});
+				continue;
+			}
+			const findings = lintSkill({
+				skillName: summary.name,
+				dims: {
+					bodyChars: null,
+					bodyTokens: null,
+					stampDensityPerKb: null,
+					stampHits: null,
+					supportGroups: 0,
+					supportFiles: null,
+					churnPatches: null,
+					churnReads: null
+				},
+				softLineChars: thresholds.softBodyChars,
+				overSoftLine: false,
+				overTwiceSoftLine: false,
+				stampLimitPerKb: thresholds.stampDensityPerKb,
+				churnMinPatches: thresholds.churnMinPatches,
+				readable: false
+			});
+			rows.push({
 				name: summary.name,
-				verdict: assessment.verdict,
-				reasons: assessment.reasons
+				verdict: "warn",
+				dims: {
+					bodyChars: null,
+					bodyTokens: null,
+					stampDensityPerKb: null,
+					supportGroups: 0,
+					supportFiles: null,
+					churnPatches: null,
+					churnReads: null
+				},
+				findings
 			});
 		}
 		return rows;
@@ -1426,4 +1482,4 @@ var EvolutionCurator = class extends Service {
 	}
 };
 //#endregion
-export { EvolutionCurator, EvolutionCurator as default, gateConsolidations, validateCuratorSettings };
+export { EvolutionCurator, EvolutionCurator as default, gateConsolidations, referencedSkillNames, validateCuratorSettings };

package/lib/types/index.d.ts

@@ -148,6 +148,16 @@ export declare function gateConsolidations(consolidations: CuratorConsolidation[
     referenced?: ReadonlySet<string>;
     suppressed?: ReadonlySet<string>;
 }): CuratorConsolidation[];
+/**
+ * G5 (in-use protection): which skills are POINTED AT by others, and by whom.
+ *
+ * Pure: derived from the bodies already in hand, using `relatedSkillNames` — the single reader of
+ * that frontmatter field — so this map can never disagree with the catalog or the audit. The value
+ * is a Set of SOURCES, not a boolean, because a refusal has to be able to name the referrers.
+ * @param contents - skill name to its whole SKILL.md text.
+ * @returns target name to the set of skills naming it; targets nobody names are absent.
+ */
+export declare function referencedSkillNames(contents: ReadonlyMap<string, string>): Map<string, Set<string>>;
 export declare class EvolutionCurator extends Service {
     static inject: string[];
     static Config: z<Schemastery.ObjectS<NoInfer<{
@@ -495,7 +505,21 @@ export declare class EvolutionCurator extends Service {
     healthView(): Promise<Array<{
         name: string;
         verdict: SkillHealthVerdict;
-        reasons: string[];
+        dims: {
+            bodyChars: number | null;
+            bodyTokens: number | null;
+            stampDensityPerKb: number | null;
+            supportGroups: number;
+            supportFiles: number | null;
+            churnPatches: number | null;
+            churnReads: number | null;
+        };
+        findings: Array<{
+            severity: 'warn' | 'error';
+            rule: string;
+            message: string;
+            count: number;
+        }>;
     }>>;
     /** Whether the library has ANY observed read evidence (C observation-window gate for churn signals). */
     usageObserved(): Promise<boolean>;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-curator",
   "description": "Deterministic skill lifecycle and recovery (community build)",
-  "version": "0.24.0",
+  "version": "0.25.0",
   "publishConfig": {
     "access": "public"
   },
@@ -27,20 +27,20 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.4",
-    "@lmzhen/dsh-evolution-core": "^0.24.0"
+    "@lmzhen/dsh-evolution-core": "^0.25.0"
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
     "@deepseek-ai/dsh-llm": "^0.2.0-rc.1",
     "@deepseek-ai/dsh-session": "^0.2.0-rc.1",
-    "@lmzhen/dsh-evolution-io": "^0.24.0",
-    "@lmzhen/dsh-evolution-state": "^0.24.0"
+    "@lmzhen/dsh-evolution-io": "^0.25.0",
+    "@lmzhen/dsh-evolution-state": "^0.25.0"
   },
   "devDependencies": {
     "@deepseek-ai/dsh-llm": "^0.2.0-rc.1",
     "@deepseek-ai/dsh-session": "^0.2.0-rc.1",
-    "@lmzhen/dsh-evolution-core": "^0.24.0",
-    "@lmzhen/dsh-evolution-io": "^0.24.0",
-    "@lmzhen/dsh-evolution-state": "^0.24.0"
+    "@lmzhen/dsh-evolution-core": "^0.25.0",
+    "@lmzhen/dsh-evolution-io": "^0.25.0",
+    "@lmzhen/dsh-evolution-state": "^0.25.0"
   }
 }