@lmzhen/dsh-evolution-core 0.1.0-rc.55 → 0.1.0-rc.57

package/lib/index.js

@@ -197,7 +197,11 @@ function emptyRecord() {
 		archived_at: null
 	};
 }
-const isTimestamp = (value) => value === null || typeof value === "string";
+/** A timestamp passes only when `Date.parse` yields a finite epoch (N-3): a bare
+* string check let garbage like "not-a-date" propagate as Invalid Date → NaN
+* into quality math and lifecycle comparisons. */
+const validTimestamp = (value) => typeof value === "string" && Number.isFinite(Date.parse(value));
+const nullableTimestamp = (value) => value === null || validTimestamp(value);
 /**
 * Field-level normalization for one sidecar record (rc.42 audit P2-3): the
 * spread used to copy any junk through verbatim, so a corrupted file could
@@ -205,6 +209,8 @@ const isTimestamp = (value) => value === null || typeof value === "string";
 * NaN. Every field falls back to its `emptyRecord()` baseline unless it has
 * exactly the declared type; an invalid `created_at` anchors the age clock at
 * now (first-sight defer semantics for a record whose age is unknowable).
+* Timestamps additionally require a parseable date (N-3): `"not-a-date"`
+* would otherwise survive the type check as Invalid Date.
 * Pure — exported for unit tests; `loadUsage` is the production caller.
 */
 function normalizeUsageRecord(record) {
@@ -218,13 +224,13 @@ function normalizeUsageRecord(record) {
 		use_count: num(raw.use_count, base.use_count),
 		view_count: num(raw.view_count, base.view_count),
 		patch_count: num(raw.patch_count, base.patch_count),
-		last_used_at: isTimestamp(raw.last_used_at) ? raw.last_used_at : base.last_used_at,
-		last_viewed_at: isTimestamp(raw.last_viewed_at) ? raw.last_viewed_at : base.last_viewed_at,
-		last_patched_at: isTimestamp(raw.last_patched_at) ? raw.last_patched_at : base.last_patched_at,
-		created_at: typeof raw.created_at === "string" ? raw.created_at : base.created_at,
+		last_used_at: nullableTimestamp(raw.last_used_at) ? raw.last_used_at : base.last_used_at,
+		last_viewed_at: nullableTimestamp(raw.last_viewed_at) ? raw.last_viewed_at : base.last_viewed_at,
+		last_patched_at: nullableTimestamp(raw.last_patched_at) ? raw.last_patched_at : base.last_patched_at,
+		created_at: validTimestamp(raw.created_at) ? raw.created_at : base.created_at,
 		state: raw.state === "stale" || raw.state === "archived" ? raw.state : "active",
 		pinned: bool(raw.pinned, base.pinned),
-		archived_at: isTimestamp(raw.archived_at) ? raw.archived_at : base.archived_at,
+		archived_at: nullableTimestamp(raw.archived_at) ? raw.archived_at : base.archived_at,
 		quality_score: typeof raw.quality_score === "number" && Number.isFinite(raw.quality_score) ? raw.quality_score : void 0,
 		quality_warn: typeof raw.quality_warn === "boolean" ? raw.quality_warn : void 0
 	};
@@ -2308,6 +2314,7 @@ var SkillLibrary = class {
 		if (await this.io.exists(dest)) {
 			const stamp = (/* @__PURE__ */ new Date()).toISOString().replace(/[-:T]/g, "").slice(0, 14);
 			dest = join(archiveRoot, `${name.trim()}-${stamp}`);
+			while (await this.io.exists(dest)) dest = join(archiveRoot, `${name.trim()}-${stamp}-${Math.random().toString(36).slice(2, 8)}`);
 		}
 		if (this.io.isSymlink) {
 			if (await this.io.isSymlink(dir) === true) return {
@@ -2623,7 +2630,7 @@ var SkillLibrary = class {
 			return null;
 		}
 	}
-	/** Keep only the newest N snapshots (Hermes keep=5 parity); oldest folded into .backups history. */
+	/** Keep only the newest N snapshots (Hermes keep=5 parity); older ones are removed outright. */
 	async retainSnapshots(keep) {
 		const snapshots = await this.listSnapshots();
 		for (const snapshot of snapshots.slice(keep)) try {

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

@@ -167,7 +167,7 @@ export declare class SkillLibrary {
     snapshotAll(reason?: string, extras?: SnapshotExtra[]): Promise<string>;
     /** Read and normalize a snapshot manifest; null when the file is missing or unparsable. */
     readSnapshotManifest(path: string): Promise<SnapshotManifest | null>;
-    /** Keep only the newest N snapshots (Hermes keep=5 parity); oldest folded into .backups history. */
+    /** Keep only the newest N snapshots (Hermes keep=5 parity); older ones are removed outright. */
     private retainSnapshots;
     listSnapshots(): Promise<Array<{
         path: string;

package/lib/types/usage.d.ts

@@ -29,6 +29,8 @@ export declare function emptyRecord(): UsageRecord;
  * NaN. Every field falls back to its `emptyRecord()` baseline unless it has
  * exactly the declared type; an invalid `created_at` anchors the age clock at
  * now (first-sight defer semantics for a record whose age is unknowable).
+ * Timestamps additionally require a parseable date (N-3): `"not-a-date"`
+ * would otherwise survive the type check as Invalid Date.
  * Pure — exported for unit tests; `loadUsage` is the production caller.
  */
 export declare function normalizeUsageRecord(record: unknown): UsageRecord;

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.55",
+  "version": "0.1.0-rc.57",
   "publishConfig": {
     "access": "public"
   },
@@ -32,13 +32,13 @@
   ],
   "license": "MIT",
   "peerDependencies": {
-    "@deepseek-ai/dsh-invariants": "^0.1.0-rc.6",
+    "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2",
     "@deepseek-ai/cordis": "^4.0.1",
-    "@deepseek-ai/dsh-session": "^0.1.0-rc.6"
+    "@deepseek-ai/dsh-session": "^0.1.1-rc.2"
   },
   "devDependencies": {
-    "@deepseek-ai/dsh-invariants": "^0.1.0-rc.6",
+    "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2",
     "@deepseek-ai/cordis": "^4.0.1",
-    "@deepseek-ai/dsh-session": "^0.1.0-rc.6"
+    "@deepseek-ai/dsh-session": "^0.1.1-rc.2"
   }
 }