@lmzhen/dsh-evolution-core 0.1.0-rc.28 → 0.1.0-rc.29

package/lib/index.js

@@ -28,12 +28,17 @@ function evolutionIoAdapter(provider) {
 	};
 }
 function nodeEvolutionIo() {
+	const isMissing = (error) => {
+		const code = error?.code;
+		return code === "ENOENT" || code === "ENOTDIR";
+	};
 	return {
 		async readText(path) {
 			try {
 				return await readFile(path, "utf8");
-			} catch {
-				return null;
+			} catch (error) {
+				if (isMissing(error)) return null;
+				throw error;
 			}
 		},
 		async writeText(path, content) {
@@ -59,8 +64,9 @@ function nodeEvolutionIo() {
 			try {
 				await stat(path);
 				return true;
-			} catch {
-				return false;
+			} catch (error) {
+				if (isMissing(error)) return false;
+				throw error;
 			}
 		},
 		async rename(path, destination) {
@@ -77,8 +83,9 @@ function nodeEvolutionIo() {
 		async size(path) {
 			try {
 				return (await stat(path)).size;
-			} catch {
-				return null;
+			} catch (error) {
+				if (isMissing(error)) return null;
+				throw error;
 			}
 		}
 	};
@@ -326,20 +333,22 @@ function lifecycleCandidate(name, record, config, bundled) {
 /**
 * Read-only scope classification, derived from the SAME gate the transition
 * engine uses (`lifecycleCandidate`), so the view always predicts what a
-* curator pass may touch.
+* curator pass may touch. `protectedNames` carries the marker info the usage
+* records lack (bundled / hub-installed / pinned from `SkillLibrary.list()`).
 */
-function computeScopeView(usage, config) {
+function computeScopeView(usage, config, protectedNames) {
 	const managed = [];
 	const watched = [];
 	const exempted = [];
-	const protectedNames = [];
+	const protectedSet = /* @__PURE__ */ new Set();
 	for (const [name, record] of usage) {
 		if (config.excludeSkillNames?.has(name) || config.referencedSkillNames?.has(name)) {
 			exempted.push(name);
 			continue;
 		}
 		const bundled = config.bundledNames?.has(name) === true;
-		if (record.pinned || bundled || config.suppressedNames?.has(name) === true) protectedNames.push(name);
+		const suppressed = config.suppressedNames?.has(name) === true;
+		if (record.pinned || bundled || suppressed || protectedNames?.has(name) === true) protectedSet.add(name);
 		if (lifecycleCandidate(name, record, config, bundled)) {
 			managed.push(name);
 			if (record.state === "stale" || record.quality_warn === true) watched.push(name);
@@ -349,7 +358,7 @@ function computeScopeView(usage, config) {
 		managed: managed.sort(),
 		watched: watched.sort(),
 		exempted: exempted.sort(),
-		protected: protectedNames.sort()
+		protected: [...protectedSet].sort()
 	};
 }
 function daysSince(iso, created, now) {
@@ -554,9 +563,10 @@ const PROMPT_BUNDLE = createPromptBundle({
 	completion: COMPLETION_SKILL_REVIEW_PROMPT
 });
 function verifyPromptBundle(bundle = PROMPT_BUNDLE) {
+	if (bundle.id !== "dsh-evolution@2" || bundle.version !== 2) return false;
 	const canonical = JSON.stringify({
-		id: bundle.id,
-		version: bundle.version,
+		id: PROMPT_BUNDLE_ID,
+		version: 2,
 		prompts: Object.fromEntries(Object.entries(bundle.prompts).sort())
 	});
 	return bundle.sha256 === sha256(canonical);
@@ -950,11 +960,15 @@ var MemoryStore = class {
 			limit: this.limitFor(target)
 		};
 	}
-	/** Percent-based storage hint appended to success message once the target is ≥80% full. */
+	/**
+	* StorageHint percentage must clamp at 100 like the render header: a drifted
+	* entry can push chars past the limit, and "Storage at 125%" contradicts the
+	* clamped usage indicator.
+	*/
 	storageHint(target, chars) {
 		const limit = this.limitFor(target);
 		if (limit <= 0) return "";
-		const percent = Math.floor(chars * 100 / limit);
+		const percent = Math.min(100, Math.floor(chars * 100 / limit));
 		return percent >= 80 ? ` ⚠️ Storage at ${percent}% (${chars}/${limit} chars).` : "";
 	}
 	/**
@@ -966,10 +980,10 @@ var MemoryStore = class {
 	*/
 	async backupFile(target) {
 		const path = fileFor(this.root, target);
-		const stamp = (/* @__PURE__ */ new Date()).toISOString().replace(/[-:T]/g, "").slice(0, 14);
+		const unique = `${(/* @__PURE__ */ new Date()).toISOString().replace(/[-:T]/g, "").slice(0, 14)}-${Math.random().toString(36).slice(2, 8)}`;
 		try {
-			await this.io.copy(path, `${path}.bak.${stamp}`);
-			return `${path}.bak.${stamp}`;
+			await this.io.copy(path, `${path}.bak.${unique}`);
+			return `${path}.bak.${unique}`;
 		} catch {
 			return null;
 		}
@@ -997,6 +1011,16 @@ var MemoryStore = class {
 	async add(target, facts) {
 		const refusal = await this.oversizedRefusal(target);
 		if (refusal) return refusal;
+		if (await this.detectDrift(target)) {
+			const backup = await this.backupFile(target);
+			return {
+				ok: false,
+				message: `External drift detected in memory file.${backup ? ` A backup was saved to ${basename(backup)}.` : ""} Resolve the drift before retrying.`,
+				entries: [],
+				chars: 0,
+				limit: this.limitFor(target)
+			};
+		}
 		const content = facts.trim();
 		if (!content) return {
 			ok: false,
@@ -1026,7 +1050,8 @@ var MemoryStore = class {
 		}
 		const next = [...entries, this.addDatePrefix ? `## ${(/* @__PURE__ */ new Date()).toISOString().slice(0, 10)}\n${content}` : content];
 		const total = next.join(ENTRY_DELIMITER).length;
-		if (total > this.limitFor(target)) return this.failure(target, `Adding this entry would exceed the ${this.limitFor(target)} char limit. Consolidate or remove stale entries, then retry.`, entries);
+		const addLimit = this.limitFor(target);
+		if (addLimit > 0 && total > addLimit) return this.failure(target, `Adding this entry would exceed the ${addLimit} char limit. Consolidate or remove stale entries, then retry.`, entries);
 		await this.write(target, next);
 		this.resetFailures();
 		return {
@@ -1100,7 +1125,8 @@ var MemoryStore = class {
 		if (action === "remove") next.splice(index, 1);
 		else next[index] = content;
 		const total = next.join(ENTRY_DELIMITER).length;
-		if (total > this.limitFor(target)) return this.failure(target, `Resulting memory would exceed the ${this.limitFor(target)} char limit.`, entries);
+		const mutateLimit = this.limitFor(target);
+		if (mutateLimit > 0 && total > mutateLimit) return this.failure(target, `Resulting memory would exceed the ${mutateLimit} char limit.`, entries);
 		await this.write(target, next);
 		this.resetFailures();
 		return {
@@ -1198,7 +1224,8 @@ var MemoryStore = class {
 			}
 		}
 		const total = working.join(ENTRY_DELIMITER).length;
-		if (total > this.limitFor(target)) return this.failure(target, `Batch result (${total} chars) exceeds the ${this.limitFor(target)} limit. Remove or shorten more entries in the same batch.`, entries);
+		const batchLimit = this.limitFor(target);
+		if (batchLimit > 0 && total > batchLimit) return this.failure(target, `Batch result (${total} chars) exceeds the ${batchLimit} limit. Remove or shorten more entries in the same batch.`, entries);
 		await this.write(target, working);
 		this.resetFailures();
 		return {
@@ -1262,7 +1289,8 @@ var MemoryStore = class {
 		const raw = await this.io.readText(fileFor(this.root, target));
 		if (raw === null) return false;
 		const entries = normalizeEntries(raw);
-		if (entries.some((entry) => entry.length > this.limitFor(target))) return true;
+		const limit = this.limitFor(target);
+		if (limit > 0 && entries.some((entry) => entry.length > limit)) return true;
 		return render(entries) !== raw;
 	}
 };

package/lib/types/curator.d.ts

@@ -106,8 +106,9 @@ export interface ScopeView {
 /**
  * Read-only scope classification, derived from the SAME gate the transition
  * engine uses (`lifecycleCandidate`), so the view always predicts what a
- * curator pass may touch.
+ * curator pass may touch. `protectedNames` carries the marker info the usage
+ * records lack (bundled / hub-installed / pinned from `SkillLibrary.list()`).
  */
-export declare function computeScopeView(usage: UsageMap, config: CuratorConfig): ScopeView;
+export declare function computeScopeView(usage: UsageMap, config: CuratorConfig, protectedNames?: ReadonlyMap<string, string>): ScopeView;
 export declare function computeLifecycleTransitions(usage: UsageMap, config: CuratorConfig, now?: Date): CuratorResult;
 //# sourceMappingURL=curator.d.ts.map

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

@@ -48,7 +48,11 @@ export declare class MemoryStore {
     write(target: MemoryTarget, entries: string[]): Promise<void>;
     resetFailures(): void;
     private failure;
-    /** Percent-based storage hint appended to success message once the target is ≥80% full. */
+    /**
+     * StorageHint percentage must clamp at 100 like the render header: a drifted
+     * entry can push chars past the limit, and "Storage at 125%" contradicts the
+     * clamped usage indicator.
+     */
     private storageHint;
     /**
      * Best-effort raw-copy backup of the on-disk file to `<file>.bak.<stamp>`

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.1.0-rc.28",
+  "version": "0.1.0-rc.29",
   "publishConfig": {
     "access": "public"
   },