@lmzhen/dsh-evolution-core 0.3.16 → 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);
 		}
 	};
 }
@@ -64,45 +68,81 @@ function nodeEvolutionIo() {
 	};
 	/**
 	* Cross-process write lock (claw `withFileLock` parity): an O_EXCL lock file
-	* guards the atomic write. A >5s-old lock is taken over ONLY after probing
+	* guards the atomic write. A >1s-old lock is taken over ONLY after probing
 	* the holder pid it carries (rc.66): a LIVE holder is never stolen, so a
-	* slow writer no longer loses its lock to a peer at the 5s mark (the
+	* slow writer no longer loses its lock to a peer at the threshold (the
 	* takeover is the only best-effort surface; the retry budget fails loud —
 	* rc.65 — instead of ever proceeding unlocked). Budget = 40 * 50ms (~2s,
 	* rc.69): 8-writer contention bursts on a loaded CI runner exceed 10
 	* attempts (500ms), and a fail-loud throw was observed instead of a clean
 	* serialization.
+	* 0.3.17 (E-8): acquisition and the task are now SEPARATE try blocks — a
+	* task error (win32 rename/EBUSY surfaces as EPERM) used to be mistaken for
+	* lock contention, retried up to 40x and finally reported as
+	* "could not acquire write lock" while the real cause was hidden.
+	* 0.3.17 (E-8a): the takeover threshold (1000ms) now fits INSIDE the ~2s
+	* retry budget (budget >= 2 x threshold), so a dead holder's lock is
+	* actually recoverable within one budget instead of being arithmetically
+	* unreachable.
 	*/
 	const withWriteLock = async (path, task) => {
 		const lock = `${path}.lock`;
-		for (let attempt = 0; attempt < 40; attempt += 1) try {
-			await writeFile(lock, String(process.pid), { flag: "wx" });
+		for (let attempt = 0; attempt < 40; attempt += 1) {
 			try {
-				return await task();
-			} finally {
-				await rm(lock, { force: true }).catch(() => {});
-			}
-		} catch (error) {
-			const code = error?.code;
-			if (code !== "EEXIST" && code !== "EPERM") throw error;
-			try {
-				const st = await stat(lock);
-				if (Date.now() - st.mtimeMs > 5e3) {
-					const holder = Number(await readFile(lock, "utf8").catch(() => ""));
-					if (!(Number.isInteger(holder) && holder > 0 && isAlive(holder))) {
-						try {
-							await rm(lock, { force: true });
-						} catch {}
-						continue;
+				await writeFile(lock, String(process.pid), { flag: "wx" });
+			} catch (error) {
+				const code = error?.code;
+				if (code !== "EEXIST" && code !== "EPERM") throw error;
+				try {
+					const st = await stat(lock);
+					if (Date.now() - st.mtimeMs > 1e3) {
+						const holder = Number(await readFile(lock, "utf8").catch(() => ""));
+						if (!(Number.isInteger(holder) && holder > 0 && isAlive(holder))) {
+							try {
+								await rm(lock, { force: true });
+							} catch {}
+							continue;
+						}
 					}
+				} catch {
+					continue;
 				}
-			} catch {
+				await new Promise((resolve) => setTimeout(resolve, 50));
 				continue;
 			}
-			await new Promise((resolve) => setTimeout(resolve, 50));
+			try {
+				return await task();
+			} finally {
+				await rm(lock, { force: true }).catch(() => {});
+			}
 		}
 		throw new Error(`could not acquire write lock for ${path} after 40 attempts`);
 	};
+	/** 0.3.17 (E-8b): sweep tmp files a crashed writer left behind — same
+	* `<target>.<pid>.<rand>.tmp` shape, older than 1h AND held by a dead pid.
+	* Lazy: only the directory a write is about to touch gets swept, once per
+	* write, inside the write lock. */
+	const sweepStaleTmps = async (path) => {
+		const dir = dirname(path);
+		const base = basename(path);
+		let entries;
+		try {
+			entries = await readdir(dir);
+		} catch {
+			return;
+		}
+		const prefix = `${base}.`;
+		for (const name of entries) {
+			if (!name.startsWith(prefix) || !name.endsWith(".tmp")) continue;
+			const tmpPath = join(dir, name);
+			const holder = Number(name.slice(prefix.length, name.length - 4).split(".")[0] ?? "");
+			try {
+				const st = await stat(tmpPath);
+				const deadHolder = !Number.isInteger(holder) || holder <= 0 || !isAlive(holder);
+				if (Date.now() - st.mtimeMs > 36e5 && deadHolder) await rm(tmpPath, { force: true });
+			} catch {}
+		}
+	};
 	return {
 		async readText(path) {
 			try {
@@ -115,6 +155,7 @@ function nodeEvolutionIo() {
 		async writeText(path, content) {
 			await mkdir(dirname(path), { recursive: true });
 			await withWriteLock(path, async () => {
+				await sweepStaleTmps(path);
 				const tmp = `${path}.${process.pid}.${randomBytes(6).toString("hex")}.tmp`;
 				await writeFile(tmp, content, "utf8");
 				await rename(tmp, path);
@@ -123,6 +164,7 @@ function nodeEvolutionIo() {
 		async transact(path, task) {
 			await mkdir(dirname(path), { recursive: true });
 			await withWriteLock(path, async () => {
+				await sweepStaleTmps(path);
 				let current;
 				try {
 					current = await readFile(path, "utf8");
@@ -188,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;
+			}
 		}
 	};
 }
@@ -494,6 +544,19 @@ const DEFAULT_USER_CHAR_LIMIT = 1375;
 /** Consolidation-failure backoff cap, shared by MemoryStore and memory-files' Config default. */
 const DEFAULT_CONSOLIDATION_FAILURES = 3;
 const DEFAULT_SKILL_CONTENT_CHARS = 1e5;
+/** 0.3.17 (S3.10, T-1): control-plane fields a model-facing write call may
+* never carry — single source for plan-validator, evolution-policy and the
+* threat scanner (they used to each hardcode the list). */
+const FORBIDDEN_CONTROL_KEYS = [
+	"policy",
+	"threshold",
+	"prompt_hash",
+	"model_route",
+	"evolution_config"
+];
+/** 0.3.17 (S3.10): the model-facing write tools the policy guard and threat
+* scanner cover. */
+const EVOLUTION_WRITE_TOOLS = ["memory", "skill_manage"];
 /** Hermes authoring quality bar for descriptions (the 60-char Rule). The
 * platform's own index limit stays in validateFrontmatter; this bar is the
 * target the authoring standard names, enforced as ADVISORY feedback.
@@ -2362,6 +2425,24 @@ function redactSecrets(text) {
 	return out;
 }
 //#endregion
+//#region lib/types/serial.js
+/**
+* A process-local serial task queue: each task starts only after the previous
+* one settles (success or failure), so read-modify-write sequences that share
+* one file never interleave inside this process. The durable cross-process
+* serialization layer is the IO backend's transact lock; this chain is the
+* second layer (0.3.17 S2.8, T-1: the shape was duplicated in state-json and
+* memory-files — one factory now).
+*/
+function makeSerialQueue() {
+	let chain = Promise.resolve();
+	return (task) => {
+		const run = chain.then(task, task);
+		chain = run.then(() => void 0, () => void 0);
+		return run;
+	};
+}
+//#endregion
 //#region lib/types/skill-health.js
 /**
 * Skill structure-health domain (rc.73 A1, 008 design): a SECOND assessment
@@ -2400,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)`);
@@ -2683,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
@@ -3511,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({
@@ -3544,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.`
@@ -4301,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, EvolutionGateSet, 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, 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/constants.d.ts

@@ -50,6 +50,13 @@ export declare const DEFAULT_USER_CHAR_LIMIT = 1375;
 /** Consolidation-failure backoff cap, shared by MemoryStore and memory-files' Config default. */
 export declare const DEFAULT_CONSOLIDATION_FAILURES = 3;
 export declare const DEFAULT_SKILL_CONTENT_CHARS = 100000;
+/** 0.3.17 (S3.10, T-1): control-plane fields a model-facing write call may
+ * never carry — single source for plan-validator, evolution-policy and the
+ * threat scanner (they used to each hardcode the list). */
+export declare const FORBIDDEN_CONTROL_KEYS: readonly ["policy", "threshold", "prompt_hash", "model_route", "evolution_config"];
+/** 0.3.17 (S3.10): the model-facing write tools the policy guard and threat
+ * scanner cover. */
+export declare const EVOLUTION_WRITE_TOOLS: readonly ["memory", "skill_manage"];
 /** Hermes authoring quality bar for descriptions (the 60-char Rule). The
  * platform's own index limit stays in validateFrontmatter; this bar is the
  * target the authoring standard names, enforced as ADVISORY feedback.

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/index.d.ts

@@ -19,6 +19,7 @@ export * from './preset-composition.ts';
 export * from './prompts.ts';
 export * from './quality.ts';
 export * from './redact.ts';
+export * from './serial.ts';
 export * from './skill-health.ts';
 export * from './signals.ts';
 export * from './drift-signals.ts';

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/serial.d.ts

@@ -0,0 +1,10 @@
+/**
+ * A process-local serial task queue: each task starts only after the previous
+ * one settles (success or failure), so read-modify-write sequences that share
+ * one file never interleave inside this process. The durable cross-process
+ * serialization layer is the IO backend's transact lock; this chain is the
+ * second layer (0.3.17 S2.8, T-1: the shape was duplicated in state-json and
+ * memory-files — one factory now).
+ */
+export declare function makeSerialQueue(): <T>(task: () => Promise<T>) => Promise<T>;
+//# sourceMappingURL=serial.d.ts.map

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