@lmzhen/dsh-evolution-feedback 0.3.30 → 0.3.31

package/lib/index.js

@@ -39,6 +39,14 @@ var EvolutionFeedback = class {
 	* unpersisted note there, and reverting to it resurrects a value the log
 	* never held). Seeded from the fold truth, updated on successful appends. */
 	durableNote = /* @__PURE__ */ new Map();
+	/** V5-32 (0.3.31): fire-and-forget append failures are warn-once per unique
+	* message — a persistent refusal (e.g. a future-version log) must not spam
+	* the log on every user feedback entry (family posture: process-level once). */
+	warnedMessages = /* @__PURE__ */ new Set();
+	/** V5-29 (0.3.31): invoked after a FAILED append rolled back, so a caller
+	* holding derived state (skillUsage quality score) can re-push it instead of
+	* keeping an optimistic value that never landed. */
+	onRollback;
 	constructor(io, home = evolutionRoot(), pathOverride, warn = () => {}) {
 		this.path = pathOverride ?? join(home, "evolution", "feedback.json");
 		this.eventsPath = eventsFile(home);
@@ -134,7 +142,13 @@ var EvolutionFeedback = class {
 						}
 					}
 				}
-				this.warn(`evolution-feedback: failed to append feedback event for ${kind} "${target}": ${error instanceof Error ? error.message : String(error)}`);
+				const message = `evolution-feedback: failed to append feedback event for ${kind} "${target}": ${error instanceof Error ? error.message : String(error)}`;
+				const cause = error instanceof Error ? error.message : String(error);
+				if (!this.warnedMessages.has(cause)) {
+					this.warnedMessages.add(cause);
+					this.warn(message);
+				}
+				if (rollback) this.onRollback?.(target, kind);
 			}
 		});
 	}
@@ -416,15 +430,20 @@ function apply(ctx, rawConfig = {}) {
 		if (qualityWired) return;
 		qualityWired = true;
 		const skillUsage = skillCtx.skillUsage;
+		const pushQuality = (target, kind) => {
+			if (kind !== "skill") return;
+			const score = feedback.score(target, "skill");
+			const warn = score < qualityWarnThreshold;
+			skillUsage.setQuality(target, score, warn).catch((error) => {
+				skillCtx.logger.warn(error);
+			});
+		};
+		feedback.onRollback = (target, kind) => {
+			pushQuality(target, kind);
+		};
 		feedback.record = (target, rating, note, kind) => {
 			baseRecord(target, rating, note, kind ?? "session");
-			if (kind === "skill") {
-				const score = feedback.score(target, "skill");
-				const warn = score < qualityWarnThreshold;
-				skillUsage.setQuality(target, score, warn).catch((error) => {
-					skillCtx.logger.warn(error);
-				});
-			}
+			pushQuality(target, kind ?? "session");
 		};
 	});
 	ctx.effect(() => () => {

package/lib/types/index.d.ts

@@ -43,6 +43,14 @@ export declare class EvolutionFeedback {
      * unpersisted note there, and reverting to it resurrects a value the log
      * never held). Seeded from the fold truth, updated on successful appends. */
     private readonly durableNote;
+    /** V5-32 (0.3.31): fire-and-forget append failures are warn-once per unique
+     * message — a persistent refusal (e.g. a future-version log) must not spam
+     * the log on every user feedback entry (family posture: process-level once). */
+    private readonly warnedMessages;
+    /** V5-29 (0.3.31): invoked after a FAILED append rolled back, so a caller
+     * holding derived state (skillUsage quality score) can re-push it instead of
+     * keeping an optimistic value that never landed. */
+    onRollback?: (target: string, kind: 'skill' | 'session') => void;
     constructor(io?: IoLike, home?: string, pathOverride?: string, warn?: (message: string) => void);
     /** Bind the evolution IO backend after construction (S6.4 deferred binding). */
     attachIo(io: IoLike): void;

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.30",
+  "version": "0.3.31",
   "publishConfig": {
     "access": "public"
   },
@@ -31,18 +31,18 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.1",
-    "@lmzhen/dsh-evolution-core": "^0.3.30"
+    "@lmzhen/dsh-evolution-core": "^0.3.31"
   },
   "peerDependencies": {
     "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2",
     "@deepseek-ai/cordis": "^4.0.1",
-    "@lmzhen/dsh-evolution-io": "^0.3.30",
-    "@lmzhen/dsh-skill-usage": "^0.3.30"
+    "@lmzhen/dsh-evolution-io": "^0.3.31",
+    "@lmzhen/dsh-skill-usage": "^0.3.31"
   },
   "devDependencies": {
     "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2",
-    "@lmzhen/dsh-evolution-io": "^0.3.30",
-    "@lmzhen/dsh-evolution-io-node": "^0.3.30",
-    "@lmzhen/dsh-skill-usage": "^0.3.30"
+    "@lmzhen/dsh-evolution-io": "^0.3.31",
+    "@lmzhen/dsh-evolution-io-node": "^0.3.31",
+    "@lmzhen/dsh-skill-usage": "^0.3.31"
   }
 }