@lmzhen/dsh-skill-usage 0.1.0-rc.49 → 0.1.0-rc.50

package/lib/index.js

@@ -1,6 +1,6 @@
 import { Service } from "@deepseek-ai/cordis";
 import z from "@deepseek-ai/schemastery";
-import { bumpPatch, bumpUse, bumpView, evolutionIoAdapter, getRecord, loadUsage, markAgentCreated, saveUsage, skillsRoot } from "@lmzhen/dsh-evolution-core";
+import { bumpPatch, bumpUse, bumpView, evolutionIoAdapter, getRecord, loadUsage, markAgentCreated, mutateUsage, skillsRoot } from "@lmzhen/dsh-evolution-core";
 //#region lib/types/index.js
 /**
 * Skill usage telemetry service for the evolution family.
@@ -11,41 +11,45 @@ var SkillUsageRegistry = class extends Service {
 	static Config = z.object({ root: z.string().default("") });
 	root;
 	io;
-	usage = null;
 	chain = Promise.resolve();
 	constructor(ctx, config = {}) {
 		super(ctx, "skillUsage");
 		this.root = config.root || skillsRoot();
 		this.io = evolutionIoAdapter(() => ctx.evolutionIo.provider());
 	}
-	async map() {
-		if (!this.usage) this.usage = await loadUsage(this.root, this.io);
-		return this.usage;
-	}
-	async flush() {
-		if (this.usage) await saveUsage(this.root, this.usage, this.io);
-	}
-	/** Serialize read-modify-write cycles so concurrent record calls never lose updates. */
+	/**
+	* Serialize read-modify-write cycles so concurrent record calls never lose
+	* updates, then run each cycle as ONE atomic transact (rc.50 P2-2): the map
+	* is read from disk inside the lock, so a second process that shares
+	* DSH_HOME cannot interleave its own RMW between our read and write.
+	*/
 	mutate(task) {
-		const run = this.chain.then(async () => task(await this.map()));
+		const run = this.chain.then(async () => {
+			let outcome = void 0;
+			await mutateUsage(this.root, this.io, async (map) => {
+				outcome = await task(map);
+			});
+			return outcome;
+		});
 		this.chain = run.then(() => void 0, () => void 0);
 		return run;
 	}
 	async record(name, kind, at = /* @__PURE__ */ new Date()) {
-		await this.mutate(async (map) => {
+		await this.mutate((map) => {
 			if (kind === "use") bumpUse(map, name, at);
 			else if (kind === "view") bumpView(map, name, at);
 			else bumpPatch(map, name, at);
-			await this.flush();
 		});
 	}
-	report() {
-		return this.mutate((map) => Promise.resolve(new Map(map)));
+	/** Read-only snapshot of the sidecar (no disk write — reading never mutates). */
+	async report() {
+		const run = this.chain.then(async () => new Map(await loadUsage(this.root, this.io)));
+		this.chain = run.then(() => void 0, () => void 0);
+		return run;
 	}
 	async markAgentCreated(name) {
-		await this.mutate(async (map) => {
+		await this.mutate((map) => {
 			markAgentCreated(map, name);
-			await this.flush();
 		});
 	}
 	/**
@@ -55,9 +59,8 @@ var SkillUsageRegistry = class extends Service {
 	* mutation maturity is not inflated by mere creation.
 	*/
 	async ensureRecord(name) {
-		await this.mutate(async (map) => {
+		await this.mutate((map) => {
 			getRecord(map, name);
-			await this.flush();
 		});
 	}
 	/**
@@ -66,32 +69,29 @@ var SkillUsageRegistry = class extends Service {
 	* a state transition, not a content mutation.
 	*/
 	async markArchived(name, at = /* @__PURE__ */ new Date()) {
-		await this.mutate(async (map) => {
+		await this.mutate((map) => {
 			const record = map.get(name);
 			if (record) {
 				record.state = "archived";
 				record.archived_at = at.toISOString();
 			}
-			await this.flush();
 		});
 	}
 	/**
-	* Drop the in-memory cache once queued work drains, so the next `map()` reads
-	* the file again. The curator writes the sidecar directly; without this, the
-	* next tool telemetry flush would re-cover its quality/state/pinned writes.
+	* Barrier for external writers: every mutate reads the sidecar fresh from
+	* disk (rc.50 P2-2 transact), so a curator direct-write is visible on the
+	* next call without a cache flush; this waits for queued work to drain.
 	*/
 	async invalidate() {
 		await this.chain;
-		this.usage = null;
 	}
 	/** Write feedback-derived quality onto the usage sidecar; curator reads it. */
 	async setQuality(name, score, warn) {
-		await this.mutate(async (map) => {
+		await this.mutate((map) => {
 			const record = map.get(name);
 			if (!record) return;
 			record.quality_score = score;
 			record.quality_warn = warn;
-			await this.flush();
 		});
 	}
 };

package/lib/types/index.d.ts

@@ -18,14 +18,17 @@ export declare class SkillUsageRegistry extends Service {
     static Config: Schema<Config>;
     readonly root: string;
     private readonly io;
-    private usage;
     private chain;
     constructor(ctx: Context, config?: Config);
-    private map;
-    flush(): Promise<void>;
-    /** Serialize read-modify-write cycles so concurrent record calls never lose updates. */
+    /**
+     * Serialize read-modify-write cycles so concurrent record calls never lose
+     * updates, then run each cycle as ONE atomic transact (rc.50 P2-2): the map
+     * is read from disk inside the lock, so a second process that shares
+     * DSH_HOME cannot interleave its own RMW between our read and write.
+     */
     private mutate;
     record(name: string, kind: 'use' | 'view' | 'patch', at?: Date): Promise<void>;
+    /** Read-only snapshot of the sidecar (no disk write — reading never mutates). */
     report(): Promise<UsageMap>;
     markAgentCreated(name: string): Promise<void>;
     /**
@@ -42,9 +45,9 @@ export declare class SkillUsageRegistry extends Service {
      */
     markArchived(name: string, at?: Date): Promise<void>;
     /**
-     * Drop the in-memory cache once queued work drains, so the next `map()` reads
-     * the file again. The curator writes the sidecar directly; without this, the
-     * next tool telemetry flush would re-cover its quality/state/pinned writes.
+     * Barrier for external writers: every mutate reads the sidecar fresh from
+     * disk (rc.50 P2-2 transact), so a curator direct-write is visible on the
+     * next call without a cache flush; this waits for queued work to drain.
      */
     invalidate(): Promise<void>;
     /** Write feedback-derived quality onto the usage sidecar; curator reads it. */

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-skill-usage",
   "description": "Skill usage telemetry service (community build)",
-  "version": "0.1.0-rc.49",
+  "version": "0.1.0-rc.50",
   "publishConfig": {
     "access": "public"
   },
@@ -33,16 +33,16 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.1",
-    "@lmzhen/dsh-evolution-core": "^0.1.0-rc.49"
+    "@lmzhen/dsh-evolution-core": "^0.1.0-rc.50"
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
     "@deepseek-ai/dsh-invariants": "^0.1.0-rc.6",
-    "@lmzhen/dsh-evolution-io": "^0.1.0-rc.49"
+    "@lmzhen/dsh-evolution-io": "^0.1.0-rc.50"
   },
   "devDependencies": {
     "@deepseek-ai/dsh-invariants": "^0.1.0-rc.6",
-    "@lmzhen/dsh-evolution-core": "^0.1.0-rc.49",
-    "@lmzhen/dsh-evolution-io": "^0.1.0-rc.49"
+    "@lmzhen/dsh-evolution-core": "^0.1.0-rc.50",
+    "@lmzhen/dsh-evolution-io": "^0.1.0-rc.50"
   }
 }