@lmzhen/dsh-evolution-feedback 0.3.21 → 0.3.23

package/lib/index.js

@@ -1,6 +1,5 @@
 import z from "@deepseek-ai/schemastery";
-import { EVENT_LOG_VERSION, appendEvolutionEvent, eventsFile, evolutionIoAdapter, listEventArchives, parseEvolutionEvents, readEvolutionTimeline, transactIo } from "@lmzhen/dsh-evolution-core";
-import { homedir } from "node:os";
+import { EVENT_LOG_VERSION, appendEvolutionEvent, clampedNumber, eventsFile, evolutionIoAdapter, evolutionRoot, listEventArchives, parseEvolutionEvents, readEvolutionTimeline, transactIo } from "@lmzhen/dsh-evolution-core";
 import { join } from "node:path";
 //#region lib/types/index.js
 /**
@@ -34,7 +33,7 @@ var EvolutionFeedback = class {
 	eventsPath;
 	io;
 	warn;
-	constructor(io, home = process.env.DSH_HOME ?? join(homedir(), ".dsh"), pathOverride, warn = () => {}) {
+	constructor(io, home = evolutionRoot(), pathOverride, warn = () => {}) {
 		this.path = pathOverride ?? join(home, "evolution", "feedback.json");
 		this.eventsPath = eventsFile(home);
 		this.io = io;
@@ -363,11 +362,24 @@ function synthesizeFeedbackEvents(aggregate) {
 }
 const name = "evolution-feedback";
 const Config = z.object({
-	qualityWarnThreshold: z.number().default(-.25),
+	qualityWarnThreshold: z.number().min(-1).max(1).default(-.25),
 	path: z.string().default("")
 });
+/** Resolve the effective quality warn threshold, clamped to its [-1, 1] domain
+* (G3.1). 0 and negative values are legal (a lower threshold is stricter); NaN,
+* ±Infinity and out-of-range values fall back to the default -0.25. Exported so
+* the clamp is directly testable; `apply` warns when a value was corrected. */
+function resolveQualityWarnThreshold(config) {
+	return clampedNumber(config.qualityWarnThreshold ?? -.25, -.25, {
+		min: -1,
+		max: 1
+	});
+}
 function apply(ctx, rawConfig = {}) {
-	const feedback = new EvolutionFeedback(void 0, process.env.DSH_HOME ?? join(homedir(), ".dsh"), rawConfig.path || void 0, (message) => {
+	const configuredThreshold = rawConfig.qualityWarnThreshold ?? -.25;
+	const qualityWarnThreshold = resolveQualityWarnThreshold(rawConfig);
+	if (qualityWarnThreshold !== configuredThreshold) ctx.logger.warn(`evolution-feedback: qualityWarnThreshold=${String(configuredThreshold)} is invalid; falling back to the default -0.25`);
+	const feedback = new EvolutionFeedback(void 0, evolutionRoot(), rawConfig.path || void 0, (message) => {
 		ctx.logger.warn(message);
 	});
 	ctx.provide("evolutionFeedback", feedback);
@@ -389,7 +401,7 @@ function apply(ctx, rawConfig = {}) {
 			baseRecord(target, rating, note, kind ?? "session");
 			if (kind === "skill") {
 				const score = feedback.score(target, "skill");
-				const warn = score < (rawConfig.qualityWarnThreshold ?? -.25);
+				const warn = score < qualityWarnThreshold;
 				skillUsage.setQuality(target, score, warn).catch((error) => {
 					skillCtx.logger.warn(error);
 				});
@@ -401,4 +413,4 @@ function apply(ctx, rawConfig = {}) {
 	}, "evolution-feedback.records");
 }
 //#endregion
-export { Config, EvolutionFeedback, apply, migrateFeedbackEvents, name };
+export { Config, EvolutionFeedback, apply, migrateFeedbackEvents, name, resolveQualityWarnThreshold };

package/lib/types/index.d.ts

@@ -73,6 +73,11 @@ export interface Config {
     path?: string;
 }
 export declare const Config: z<Config>;
+/** Resolve the effective quality warn threshold, clamped to its [-1, 1] domain
+ * (G3.1). 0 and negative values are legal (a lower threshold is stricter); NaN,
+ * ±Infinity and out-of-range values fall back to the default -0.25. Exported so
+ * the clamp is directly testable; `apply` warns when a value was corrected. */
+export declare function resolveQualityWarnThreshold(config: Config): number;
 export declare function apply(ctx: Context, rawConfig?: Config): void;
 export {};
 //# sourceMappingURL=index.d.ts.map

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-feedback",
   "description": "Feedback-to-quality scoring for self-evolution (community build)",
-  "version": "0.3.21",
+  "version": "0.3.23",
   "publishConfig": {
     "access": "public"
   },
@@ -31,18 +31,18 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.1",
-    "@lmzhen/dsh-evolution-core": "^0.3.21"
+    "@lmzhen/dsh-evolution-core": "^0.3.23"
   },
   "peerDependencies": {
     "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2",
     "@deepseek-ai/cordis": "^4.0.1",
-    "@lmzhen/dsh-evolution-io": "^0.3.21",
-    "@lmzhen/dsh-skill-usage": "^0.3.21"
+    "@lmzhen/dsh-evolution-io": "^0.3.23",
+    "@lmzhen/dsh-skill-usage": "^0.3.23"
   },
   "devDependencies": {
     "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2",
-    "@lmzhen/dsh-evolution-io": "^0.3.21",
-    "@lmzhen/dsh-evolution-io-node": "^0.3.21",
-    "@lmzhen/dsh-skill-usage": "^0.3.21"
+    "@lmzhen/dsh-evolution-io": "^0.3.23",
+    "@lmzhen/dsh-evolution-io-node": "^0.3.23",
+    "@lmzhen/dsh-skill-usage": "^0.3.23"
   }
 }