@lmzhen/dsh-evolution-core 0.3.35 → 0.3.36

package/lib/index.js

@@ -746,7 +746,8 @@ function buildCuratorRunReport(input) {
 		failed: [...input.failed],
 		...input.consolidated === void 0 ? {} : { consolidated: [...input.consolidated] },
 		...input.snapshotPath === void 0 ? {} : { snapshotPath: input.snapshotPath },
-		...input.llmReviewEnabled === void 0 ? {} : { llmReviewEnabled: input.llmReviewEnabled }
+		...input.llmReviewEnabled === void 0 ? {} : { llmReviewEnabled: input.llmReviewEnabled },
+		...input.nominationsWarnings === void 0 ? {} : { nominationsWarnings: [...input.nominationsWarnings] }
 	};
 }
 /**
@@ -765,7 +766,8 @@ function renderCuratorReportMarkdown(report) {
 		`- **Archived**: ${report.archived.length}`,
 		`- **Failed**: ${report.failed.length}`,
 		...report.snapshotPath === void 0 ? [] : [`- **Snapshot**: ${report.snapshotPath}`],
-		...report.llmReviewEnabled === void 0 ? [] : [`- **llmReview**: ${report.llmReviewEnabled}`]
+		...report.llmReviewEnabled === void 0 ? [] : [`- **llmReview**: ${report.llmReviewEnabled}`],
+		...report.nominationsWarnings === void 0 || report.nominationsWarnings.length === 0 ? [] : [`- **Nomination warnings**: ${report.nominationsWarnings.join("; ")}`]
 	];
 	const section = (title, items) => items.length === 0 ? [] : [
 		"",
@@ -791,10 +793,16 @@ const NOMINATION_NAME_RE = /^[a-z0-9][a-z0-9-]*$/;
 function parseCuratorNominations(text) {
 	const prunings = [];
 	const consolidations = [];
+	const warnings = [];
 	let section = null;
 	let currentFrom = "";
 	let currentMode;
 	for (const line of text.split("\n")) {
+		const header = /^\s*(consolidations|prunings)\s*:\s*$/.exec(line);
+		if (header) {
+			section = header[1] === "consolidations" ? "consolidations" : "prunings";
+			continue;
+		}
 		const consolidated = /^\s*-\s*from:\s*([a-z0-9][a-z0-9-]*)\s*$/.exec(line);
 		if (consolidated) {
 			section = "consolidations";
@@ -805,6 +813,7 @@ function parseCuratorNominations(text) {
 		const mode = /^\s*mode:\s*(append|reference)\s*$/.exec(line);
 		if (mode) {
 			if (currentFrom !== "") currentMode = mode[1] === "reference" ? "reference" : "append";
+			else warnings.push(`mode: ${mode[1]} ignored — no preceding "- from:" entry (the consolidation falls back to append)`);
 			continue;
 		}
 		const into = /^\s*into:\s*([a-z0-9][a-z0-9-]*)\s*$/.exec(line);
@@ -821,6 +830,7 @@ function parseCuratorNominations(text) {
 		}
 		const pruned = /^\s*-\s*name:\s*([a-z0-9][a-z0-9-]*)\s*$/.exec(line);
 		if (pruned) {
+			if (section === "consolidations") warnings.push("\"- name:\" inside the consolidations section flips the parse to prunings");
 			section = "prunings";
 			const name = pruned[1];
 			if (name) prunings.push(name);
@@ -829,7 +839,8 @@ function parseCuratorNominations(text) {
 	const valid = (name) => NOMINATION_NAME_RE.test(name);
 	return {
 		prunings: prunings.filter(valid),
-		consolidations: consolidations.filter((item) => valid(item.from) && valid(item.into))
+		consolidations: consolidations.filter((item) => valid(item.from) && valid(item.into)),
+		warnings
 	};
 }
 /**
@@ -864,7 +875,8 @@ function computeScopeView(usage, config, protectedNames, gates) {
 		}
 		const bundled = config.bundledNames?.has(name) === true;
 		const suppressed = gateSet.suppressed.has(name);
-		if (record.pinned || bundled || suppressed || protectedNames?.has(name) === true) protectedSet.add(name);
+		const isBuiltin = PROTECTED_BUILTIN_SKILLS.has(name);
+		if (record.pinned || bundled || suppressed || protectedNames?.has(name) === true || isBuiltin) protectedSet.add(name);
 		if (lifecycleCandidate(name, record, config, bundled, gateSet)) {
 			managed.push(name);
 			if (record.state === "stale" || record.quality_warn === true) watched.push(name);
@@ -4502,6 +4514,15 @@ var SkillLibrary = class {
 		};
 		const target = join(dir, ...filePath.replace(/\\/g, "/").split("/").filter(Boolean));
 		return await this.runSingleWrite(target, (current) => {
+			if (current !== null && content.trimEnd() === current.trimEnd()) return {
+				result: {
+					ok: true,
+					message: `Support file "${filePath}" unchanged: the supplied content already matches the current file; nothing written.`,
+					noop: true,
+					path: target
+				},
+				write: null
+			};
 			return {
 				result: {
 					ok: true,

package/lib/types/curator.d.ts

@@ -63,6 +63,8 @@ export interface CuratorRunReport {
     snapshotPath?: string;
     /** Whether the LLM nomination pass was enabled for this run (decision visibility). */
     llmReviewEnabled?: boolean;
+    /** V6-35 (0.3.36): lenient-parse shape notes from the LLM nomination block. */
+    nominationsWarnings?: string[];
 }
 export interface CuratorReportInput {
     runId: string;
@@ -76,6 +78,7 @@ export interface CuratorReportInput {
     consolidated?: readonly CuratorConsolidation[];
     snapshotPath?: string;
     llmReviewEnabled?: boolean;
+    nominationsWarnings?: readonly string[];
 }
 export declare function buildCuratorRunReport(input: CuratorReportInput): CuratorRunReport;
 /**
@@ -97,6 +100,11 @@ export interface CuratorConsolidation {
 export interface CuratorNominations {
     prunings: string[];
     consolidations: CuratorConsolidation[];
+    /** V6-35 (0.3.36): lenient-parse shape notes (an entry the lenient logic
+     * silently dropped or re-routed). Parsing stays lenient — these are advisory
+     * and flow into the run report so the operator sees why a mode or a pruning
+     * went somewhere unexpected. */
+    warnings: string[];
 }
 /**
  * Parse the curator LLM's YAML nomination block (consolidations + prunings).

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-core",
   "description": "Shared stores, prompts, signals and lifecycle logic for the dsh-evolution plugin family (community build)",
-  "version": "0.3.35",
+  "version": "0.3.36",
   "publishConfig": {
     "access": "public"
   },