@lmzhen/dsh-skill-usage 0.3.62 → 0.3.63

package/lib/index.js

@@ -201,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

@@ -90,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.62",
+  "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.62"
+    "@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.62"
+    "@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.62",
-    "@lmzhen/dsh-evolution-io": "^0.3.62"
+    "@lmzhen/dsh-evolution-core": "^0.3.63",
+    "@lmzhen/dsh-evolution-io": "^0.3.63"
   }
 }