@lmzhen/dsh-skill-usage 0.3.62 → 0.3.64

package/lib/index.js

@@ -143,9 +143,18 @@ var SkillUsageRegistry = class extends Service {
 	async record(name, kind, at = /* @__PURE__ */ new Date()) {
 		const normalized = name.trim();
 		await this.mutate((map) => {
-			if (kind === "use") bumpUse(map, normalized, at);
-			else if (kind === "view") bumpView(map, normalized, at);
-			else bumpPatch(map, normalized, at);
+			switch (kind) {
+				case "use":
+					bumpUse(map, normalized, at);
+					break;
+				case "view":
+					bumpView(map, normalized, at);
+					break;
+				case "patch":
+					bumpPatch(map, normalized, at);
+					break;
+				default: throw new Error(`skill-usage: unknown record kind ${String(kind)}`);
+			}
 		});
 	}
 	/** Read-only snapshot of the sidecar (no disk write — reading never mutates).
@@ -197,18 +206,27 @@ var SkillUsageRegistry = class extends Service {
 	* next call without a cache flush; this waits for queued work to drain.
 	* V6-44 (0.3.37): test-support API — no production consumer uses it
 	* (tests use it as a drain barrier); kept by declaration.
+	* @internal Exported for this package's own tests only.
 	*/
 	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

@@ -88,10 +88,19 @@ export declare class SkillUsageRegistry extends Service {
      * next call without a cache flush; this waits for queued work to drain.
      * V6-44 (0.3.37): test-support API — no production consumer uses it
      * (tests use it as a drain barrier); kept by declaration.
+     * @internal Exported for this package's own tests only.
      */
     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.62",
+  "version": "0.3.64",
   "publishConfig": {
     "access": "public"
   },
@@ -31,18 +31,18 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.1",
-    "@lmzhen/dsh-evolution-core": "^0.3.62"
+    "@lmzhen/dsh-evolution-core": "^0.3.64"
   },
   "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.62"
+    "@lmzhen/dsh-evolution-io": "^0.3.64"
   },
   "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.62",
-    "@lmzhen/dsh-evolution-io": "^0.3.62"
+    "@lmzhen/dsh-evolution-core": "^0.3.64",
+    "@lmzhen/dsh-evolution-io": "^0.3.64"
   }
 }