@lmzhen/dsh-evolution-core 0.3.17 → 0.3.18

package/lib/index.js

@@ -45,6 +45,10 @@ function evolutionIoAdapter(provider) {
 		isSymlink: (path) => {
 			const io = provider();
 			return io.isSymlink ? io.isSymlink(path) : Promise.resolve(null);
+		},
+		mtime: (path) => {
+			const io = provider();
+			return io.mtime ? io.mtime(path) : Promise.resolve(null);
 		}
 	};
 }
@@ -226,6 +230,14 @@ function nodeEvolutionIo() {
 			} catch {
 				return null;
 			}
+		},
+		async mtime(path) {
+			try {
+				return (await stat(path)).mtimeMs;
+			} catch (error) {
+				if (isMissing(error)) return null;
+				throw error;
+			}
 		}
 	};
 }
@@ -2469,7 +2481,7 @@ function assessStructureHealth(snapshot, thresholds = DEFAULT_HEALTH_THRESHOLDS)
 	const needs = snapshot.bodyChars >= thresholds.softBodyChars * 2;
 	if (needs) reasons.push(`body ${snapshot.bodyChars} chars is >= 2x the soft limit (${thresholds.softBodyChars}) — consider splitting or offloading`);
 	else if (snapshot.bodyChars >= thresholds.softBodyChars) reasons.push(`body ${snapshot.bodyChars} chars above the soft limit (${thresholds.softBodyChars})`);
-	if (snapshot.bodyText && snapshot.bodyChars >= MIN_STAMP_BODY_CHARS) {
+	if (snapshot.bodyText && snapshot.bodyChars >= 2e3) {
 		const kb = Math.max(1, snapshot.bodyChars / 1024);
 		dims.stampDensityPerKb = (snapshot.bodyText.match(HEALTH_STAMP_RE) ?? []).length / kb;
 		if (dims.stampDensityPerKb >= thresholds.stampDensityPerKb) reasons.push(`stamp density ${dims.stampDensityPerKb.toFixed(1)}/KB (rc/sha/date lines — log-like content in the body)`);
@@ -2752,6 +2764,14 @@ const SNAPSHOT_EXTRA_NAME_RE = /^[a-z0-9][a-z0-9._-]*$/;
 function skillsRoot(env = process.env) {
 	return join(env.DSH_HOME || join(homedir(), ".dsh"), "skills");
 }
+/** 0.3.18 (S4.1, E-30): the ONE root resolution for every member that reads
+* the skills tree — tool-skill-manage / evolution-skill-catalog / skill-usage
+* / evolution-learning-graph used to each resolve `config.root || skillsRoot()`
+* (and the graph ignored config entirely). Empty/whitespace config falls
+* through to the default; callers pass their raw Config. */
+function resolveSkillsRoot(config = {}) {
+	return (config.root ?? "").trim() || skillsRoot();
+}
 /**
 * Map a requesting session onto the two origin surfaces (rc.44 plan M2-2.3):
 * the APPROVAL surface treats every delegated subagent as the autonomous
@@ -3580,6 +3600,12 @@ var SkillLibrary = class {
 			ok: false,
 			message: threat
 		};
+		if (writeContent.trimEnd() + "\n" === md) return {
+			ok: true,
+			message: `Skill "${name}" unchanged: old_string already equals the replacement (${patchLabel}); nothing written.`,
+			noop: true,
+			path: dir
+		};
 		await this.io.writeText(target, writeContent.trimEnd() + "\n");
 		await this.audit(name, "patch", md, writeContent, `patched ${patchLabel}`);
 		this.notifyMutation({
@@ -3613,6 +3639,10 @@ var SkillLibrary = class {
 			message: `Skill "${name}" is protected (${protection}).`
 		};
 		if (options.absorbedInto) {
+			if (options.absorbedInto.trim() === name) return {
+				ok: false,
+				message: "absorbed_into cannot be the skill being archived (cannot absorb into itself)."
+			};
 			if (!await this.io.readText(join(this.dirOf(options.absorbedInto), "SKILL.md"))) return {
 				ok: false,
 				message: `absorbed_into="${options.absorbedInto}" does not exist.`
@@ -4370,4 +4400,4 @@ function evolutionHome(env = process.env) {
 	return join(env.DSH_HOME || join(homedir(), ".dsh"), "evolution");
 }
 //#endregion
-export { AUTHORING_DESCRIPTION_BAR, COMBINED_REVIEW_PLAN_PROMPT, COMBINED_REVIEW_PROMPT, COMPLETION_SKILL_REVIEW_PROMPT, CURATOR_DRY_RUN_BANNER, CURATOR_PROMPT, DEFAULT_ARCHIVE_AFTER_DAYS, DEFAULT_CONSOLIDATION_FAILURES, DEFAULT_CURATOR_INTERVAL_HOURS, DEFAULT_HEALTH_THRESHOLDS, DEFAULT_MAX_OPS_PER_PLAN, DEFAULT_MEMORY_CHAR_LIMIT, DEFAULT_MIN_IDLE_HOURS, DEFAULT_MUTATION_CAP, DEFAULT_REVIEW_MEMORY_INTERVAL, DEFAULT_REVIEW_SKILL_INTERVAL, DEFAULT_SKILL_CONTENT_CHARS, DEFAULT_SKILL_LIMITS, DEFAULT_SKILL_REVIEW_COMPLETION_MIN_TOOL_CALLS, DEFAULT_SKILL_REVIEW_TRIGGER, DEFAULT_STALE_AFTER_DAYS, DEFAULT_SUBSTANTIVE_MIN_AGENT_CHARS, DEFAULT_SUBSTANTIVE_MIN_TOOL_CALLS, DEFAULT_SUBSTANTIVE_MIN_USER_CHARS, DEFAULT_USER_CHAR_LIMIT, DRIFT_MAX_LINE_CHARS, DRIFT_SIGNALS_VERSION, DRIFT_SIGNAL_NOUNS, DSH_AUTHORING_STANDARDS, ENTRY_DELIMITER, EVENT_ARCHIVE_PREFIX, EVENT_LOG_RETAIN_ARCHIVES, EVENT_LOG_ROTATE_AT, EVENT_LOG_VERSION, EVOLUTION_WRITE_TOOLS, EvolutionGateSet, FORBIDDEN_CONTROL_KEYS, HEALTH_STAMP_RE, LOW_QUALITY_THRESHOLD, MAINTAIN_PROMPT, MAX_DESCRIPTION_LENGTH, MAX_RESTRUCTURE_MOVES, MAX_SKILL_CONTENT_CHARS, MAX_SKILL_FILE_BYTES, MAX_SKILL_NAME_LENGTH, MEMORY_REVIEW_PROMPT, MUTATIONS_FILE_VERSION, MemoryStore, PROMPT_BUNDLE, PROMPT_BUNDLE_ID, PROMPT_BUNDLE_VERSION, PROTECTED_BUILTIN_SKILLS, QUALITY_WEIGHTS, RESTRUCTURE_TARGET_RE, SKILLS_GUIDANCE, SKILL_NAME_RE, SKILL_REVIEW_PLAN_PROMPT, SKILL_REVIEW_PROMPT, SNAPSHOT_EXTRA_NAME_RE, SUPPORT_DIRS, SUPPRESSED_FILE_VERSION, SkillLibrary, advanceReview, appendEvolutionEvent, applyCuratorFields, applyCuratorLifecycleFields, applyCuratorMetaFields, assessStructureHealth, authoringFeedback, buildCuratorRunReport, buildLearnPrompt, bumpPatch, bumpUse, bumpView, composePresetComposition, computeDedupGroups, computeDriftSignals, computeLifecycleTransitions, computePrefixClusters, computeQualityScores, computeScopeView, contentHash, createGateSet, duplicateHeadings, emptyRecord, evaluateThreat, eventsFile, evolutionHome, evolutionIoAdapter, findDriftSignal, foldCuratorFields, foldTurn, frontmatterBlock, frontmatterYamlUnsafeValues, getRecord, latestActivityAt, lifecycleCandidate, listEventArchives, loadMutations, loadSuppressedNames, loadUsage, makeSerialQueue, markAgentCreated, memoryRoot, missingSupportPointers, mutateUsage, mutationsFile, narrowNameMatches, nodeEvolutionIo, normalizeFrontmatter, normalizeUsageRecord, observeEvent, overlongLines, parseCuratorNominations, parseEvolutionEvents, parseFrontmatter, readEvolutionEvents, readEvolutionTimeline, recordMutation, redactSecrets, relatedSkillNames, renderCuratorReportMarkdown, resolveOrigins, retainEventArchives, reviewPrompt, saveSuppressedNames, saveUsage, scanContentThreats, scanMemoryThreats, scanThreats, skillsRoot, suppressedFile, transactIo, updateSuppressedNames, usageFile, usageObserved, validateFrontmatter, verifyPromptBundle, yamlPlainScalarNeedsQuotes };
+export { AUTHORING_DESCRIPTION_BAR, COMBINED_REVIEW_PLAN_PROMPT, COMBINED_REVIEW_PROMPT, COMPLETION_SKILL_REVIEW_PROMPT, CURATOR_DRY_RUN_BANNER, CURATOR_PROMPT, DEFAULT_ARCHIVE_AFTER_DAYS, DEFAULT_CONSOLIDATION_FAILURES, DEFAULT_CURATOR_INTERVAL_HOURS, DEFAULT_HEALTH_THRESHOLDS, DEFAULT_MAX_OPS_PER_PLAN, DEFAULT_MEMORY_CHAR_LIMIT, DEFAULT_MIN_IDLE_HOURS, DEFAULT_MUTATION_CAP, DEFAULT_REVIEW_MEMORY_INTERVAL, DEFAULT_REVIEW_SKILL_INTERVAL, DEFAULT_SKILL_CONTENT_CHARS, DEFAULT_SKILL_LIMITS, DEFAULT_SKILL_REVIEW_COMPLETION_MIN_TOOL_CALLS, DEFAULT_SKILL_REVIEW_TRIGGER, DEFAULT_STALE_AFTER_DAYS, DEFAULT_SUBSTANTIVE_MIN_AGENT_CHARS, DEFAULT_SUBSTANTIVE_MIN_TOOL_CALLS, DEFAULT_SUBSTANTIVE_MIN_USER_CHARS, DEFAULT_USER_CHAR_LIMIT, DRIFT_MAX_LINE_CHARS, DRIFT_SIGNALS_VERSION, DRIFT_SIGNAL_NOUNS, DSH_AUTHORING_STANDARDS, ENTRY_DELIMITER, EVENT_ARCHIVE_PREFIX, EVENT_LOG_RETAIN_ARCHIVES, EVENT_LOG_ROTATE_AT, EVENT_LOG_VERSION, EVOLUTION_WRITE_TOOLS, EvolutionGateSet, FORBIDDEN_CONTROL_KEYS, HEALTH_STAMP_RE, LOW_QUALITY_THRESHOLD, MAINTAIN_PROMPT, MAX_DESCRIPTION_LENGTH, MAX_RESTRUCTURE_MOVES, MAX_SKILL_CONTENT_CHARS, MAX_SKILL_FILE_BYTES, MAX_SKILL_NAME_LENGTH, MEMORY_REVIEW_PROMPT, MIN_STAMP_BODY_CHARS, MUTATIONS_FILE_VERSION, MemoryStore, PROMPT_BUNDLE, PROMPT_BUNDLE_ID, PROMPT_BUNDLE_VERSION, PROTECTED_BUILTIN_SKILLS, QUALITY_WEIGHTS, RESTRUCTURE_TARGET_RE, SKILLS_GUIDANCE, SKILL_NAME_RE, SKILL_REVIEW_PLAN_PROMPT, SKILL_REVIEW_PROMPT, SNAPSHOT_EXTRA_NAME_RE, SUPPORT_DIRS, SUPPRESSED_FILE_VERSION, SkillLibrary, advanceReview, appendEvolutionEvent, applyCuratorFields, applyCuratorLifecycleFields, applyCuratorMetaFields, assessStructureHealth, authoringFeedback, buildCuratorRunReport, buildLearnPrompt, bumpPatch, bumpUse, bumpView, composePresetComposition, computeDedupGroups, computeDriftSignals, computeLifecycleTransitions, computePrefixClusters, computeQualityScores, computeScopeView, contentHash, createGateSet, duplicateHeadings, emptyRecord, evaluateThreat, eventsFile, evolutionHome, evolutionIoAdapter, findDriftSignal, foldCuratorFields, foldTurn, frontmatterBlock, frontmatterYamlUnsafeValues, getRecord, latestActivityAt, lifecycleCandidate, listEventArchives, loadMutations, loadSuppressedNames, loadUsage, makeSerialQueue, markAgentCreated, memoryRoot, missingSupportPointers, mutateUsage, mutationsFile, narrowNameMatches, nodeEvolutionIo, normalizeFrontmatter, normalizeUsageRecord, observeEvent, overlongLines, parseCuratorNominations, parseEvolutionEvents, parseFrontmatter, readEvolutionEvents, readEvolutionTimeline, recordMutation, redactSecrets, relatedSkillNames, renderCuratorReportMarkdown, resolveOrigins, resolveSkillsRoot, retainEventArchives, reviewPrompt, saveSuppressedNames, saveUsage, scanContentThreats, scanMemoryThreats, scanThreats, skillsRoot, suppressedFile, transactIo, updateSuppressedNames, usageFile, usageObserved, validateFrontmatter, verifyPromptBundle, yamlPlainScalarNeedsQuotes };

package/lib/types/events.d.ts

@@ -42,11 +42,23 @@ export interface EvolutionSkillMutatedEvent {
     file?: string;
     archivedPath?: string;
 }
+/** 0.3.18 (E-6): a turn-end review pipeline failure was caught (never an
+ * unhandled rejection); this event lets operators/observability see it. The
+ * reason is already logged by the emitter — the event is a timestamped signal. */
+export interface EvolutionReviewErrorEvent {
+    sessionId: string;
+}
 declare module '@deepseek-ai/cordis' {
     interface Events {
         'evolution/review-scheduled'(event: EvolutionReviewScheduledEvent): void;
         'evolution/plan-applied'(event: EvolutionPlanAppliedEvent): void;
         'evolution/skill-mutated'(event: EvolutionSkillMutatedEvent): void;
+        /** 0.3.18 (E-71): explicit catalog refresh request (`/evolution skills
+         * refresh`). Out-of-band tree edits (manual, git) may bypass the mutation
+         * event; listeners drop caches and invalidate downstream catalogs. No
+         * payload — it is a bare "re-read" signal, never a mutation record. */
+        'evolution/skills-refresh'(): void;
+        'evolution/review-error'(event: EvolutionReviewErrorEvent): void;
     }
 }
 //# sourceMappingURL=events.d.ts.map

package/lib/types/io.d.ts

@@ -36,6 +36,14 @@ export interface EvolutionIoLike {
      * the path does not exist). Consumers treat `null` as "let it through".
      */
     isSymlink?(this: void, path: string): Promise<boolean | null>;
+    /**
+     * Optional mtime-generation probe (0.3.18, E-71): the path's mtime in
+     * milliseconds since epoch, or `null` when unknown (unsupported backend,
+     * missing path, stat failure). Consumers use it as a cheap invalidation
+     * stamp for a cached directory listing; a backend without it keeps
+     * event-driven invalidation only.
+     */
+    mtime?(this: void, path: string): Promise<number | null>;
 }
 /**
  * Run `task` inside `io.transact` when the backend provides it; otherwise fall

package/lib/types/skill-health.d.ts

@@ -24,6 +24,13 @@ export interface SkillHealthThresholds {
 export declare const DEFAULT_HEALTH_THRESHOLDS: SkillHealthThresholds;
 /** Stamp regex shared by health assessment and the maintenance probe (single source, 011). */
 export declare const HEALTH_STAMP_RE: RegExp;
+/**
+ * Bodies below this size skip stamp-density assessment: a few dates or shas
+ * in a short body are ordinary documentation, not log-like content. With the
+ * 1KB density floor a 3-date sentence in a small skill measured 3.0/KB and
+ * warned on a perfectly healthy body (audit 2026-08-31 X1).
+ */
+export declare const MIN_STAMP_BODY_CHARS = 2000;
 export type SkillHealthVerdict = 'healthy' | 'warn' | 'needs-restructure';
 /** Facts a caller already has; assessors never do IO. */
 export interface SkillHealthSnapshot {

package/lib/types/skill-store.d.ts

@@ -32,6 +32,9 @@ export interface SkillActionResult {
     /** Frontmatter keys auto-quoted for catalog-loadable YAML (0.3.11) — set
      * only when the write path modified the block. */
     normalizedFrontmatterFields?: string[];
+    /** 0.3.18 (E-68): patch produced byte-identical content (old===new) — no
+     * write, no audit, no mutation event; callers must not count a patch. */
+    noop?: boolean;
 }
 /**
  * One section move of a restructure proposal (008 batch B): a body section
@@ -81,6 +84,14 @@ export interface ArchiveOptions {
     allowBundled?: boolean;
 }
 export declare function skillsRoot(env?: NodeJS.ProcessEnv): string;
+/** 0.3.18 (S4.1, E-30): the ONE root resolution for every member that reads
+ * the skills tree — tool-skill-manage / evolution-skill-catalog / skill-usage
+ * / evolution-learning-graph used to each resolve `config.root || skillsRoot()`
+ * (and the graph ignored config entirely). Empty/whitespace config falls
+ * through to the default; callers pass their raw Config. */
+export declare function resolveSkillsRoot(config?: {
+    root?: string;
+}): string;
 /**
  * Map a requesting session onto the two origin surfaces (rc.44 plan M2-2.3):
  * the APPROVAL surface treats every delegated subagent as the autonomous

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.17",
+  "version": "0.3.18",
   "publishConfig": {
     "access": "public"
   },