@lmzhen/dsh-skill-usage 0.3.61 → 0.3.63

package/lib/index.js

@@ -59,7 +59,7 @@ var SkillUsageRegistry = class extends Service {
 	constructor(ctx, config = {}) {
 		super(ctx, "skillUsage");
 		this.root = resolveSkillsRoot(config);
-		this.eventsHome = config.eventsHome || evolutionRoot();
+		this.eventsHome = config.eventsHome?.trim() || evolutionRoot();
 		this.io = evolutionIoAdapter(() => ctx.evolutionIo.provider());
 		ctx.effect(() => {
 			return ctx.on("session/event", (_session, event) => {
@@ -148,7 +148,11 @@ var SkillUsageRegistry = class extends Service {
 			else bumpPatch(map, normalized, at);
 		});
 	}
-	/** Read-only snapshot of the sidecar (no disk write — reading never mutates). */
+	/** Read-only snapshot of the sidecar (no disk write — reading never mutates).
+	* P3-18 (v14): unlike `mutate()` this takes no cross-process transact lock.
+	* That is safe by construction: every writer commits through tmp+rename, so a
+	* reader observes a complete generation (old or new), never a torn file, and
+	* the lock exists for the read-modify-write cycle this method does not do. */
 	async report() {
 		const run = this.chain.then(async () => new Map(await loadUsage(this.root, this.io)));
 		this.chain = run.then(() => void 0, () => void 0);
@@ -197,14 +201,22 @@ var SkillUsageRegistry = class extends Service {
 	async invalidate() {
 		await this.chain;
 	}
-	/** Write feedback-derived quality onto the usage sidecar; curator reads it. */
-	async setQuality(name, score, warn) {
+	/** P1-1 (v15): write the FEEDBACK-owned quality signal onto the usage
+	* sidecar. Renamed from `setQuality` (it no longer writes the fields the
+	* name claims): the curator's six-factor `scoreTree` overwrites
+	* `quality_score`/`quality_warn` on every run BEFORE the lifecycle engine
+	* reads them, so feedback written to those fields was a dead channel — the
+	* value never survived to a decision. The lifecycle engine and the scope
+	* view now read the union `quality_warn || feedback_warn`, so a negative
+	* feedback is decision-relevant again. Field-ownership contract: see
+	* `UsageRecord` in evolution-core/usage.ts. */
+	async setFeedbackQuality(name, score, warn) {
 		const normalized = name.trim();
 		await this.mutate((map) => {
 			const record = map.get(normalized);
 			if (!record) return;
-			record.quality_score = score;
-			record.quality_warn = warn;
+			record.feedback_score = score;
+			record.feedback_warn = warn;
 		});
 	}
 };

package/lib/types/index.d.ts

@@ -60,7 +60,11 @@ export declare class SkillUsageRegistry extends Service {
      */
     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). */
+    /** Read-only snapshot of the sidecar (no disk write — reading never mutates).
+     * P3-18 (v14): unlike `mutate()` this takes no cross-process transact lock.
+     * That is safe by construction: every writer commits through tmp+rename, so a
+     * reader observes a complete generation (old or new), never a torn file, and
+     * the lock exists for the read-modify-write cycle this method does not do. */
     report(): Promise<UsageMap>;
     markAgentCreated(name: string): Promise<void>;
     /**
@@ -86,8 +90,16 @@ export declare class SkillUsageRegistry extends Service {
      * (tests use it as a drain barrier); kept by declaration.
      */
     invalidate(): Promise<void>;
-    /** Write feedback-derived quality onto the usage sidecar; curator reads it. */
-    setQuality(name: string, score: number, warn: boolean): Promise<void>;
+    /** P1-1 (v15): write the FEEDBACK-owned quality signal onto the usage
+     * sidecar. Renamed from `setQuality` (it no longer writes the fields the
+     * name claims): the curator's six-factor `scoreTree` overwrites
+     * `quality_score`/`quality_warn` on every run BEFORE the lifecycle engine
+     * reads them, so feedback written to those fields was a dead channel — the
+     * value never survived to a decision. The lifecycle engine and the scope
+     * view now read the union `quality_warn || feedback_warn`, so a negative
+     * feedback is decision-relevant again. Field-ownership contract: see
+     * `UsageRecord` in evolution-core/usage.ts. */
+    setFeedbackQuality(name: string, score: number, warn: boolean): Promise<void>;
 }
 export default SkillUsageRegistry;
 //# sourceMappingURL=index.d.ts.map

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-skill-usage",
   "description": "Skill usage telemetry service (community build)",
-  "version": "0.3.61",
+  "version": "0.3.63",
   "publishConfig": {
     "access": "public"
   },
@@ -31,18 +31,18 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.1",
-    "@lmzhen/dsh-evolution-core": "^0.3.61"
+    "@lmzhen/dsh-evolution-core": "^0.3.63"
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
     "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2",
     "@deepseek-ai/dsh-session": "^0.1.1-rc.2",
-    "@lmzhen/dsh-evolution-io": "^0.3.61"
+    "@lmzhen/dsh-evolution-io": "^0.3.63"
   },
   "devDependencies": {
     "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2",
     "@deepseek-ai/dsh-session": "^0.1.1-rc.2",
-    "@lmzhen/dsh-evolution-core": "^0.3.61",
-    "@lmzhen/dsh-evolution-io": "^0.3.61"
+    "@lmzhen/dsh-evolution-core": "^0.3.63",
+    "@lmzhen/dsh-evolution-io": "^0.3.63"
   }
 }