@lmzhen/dsh-memory-files 0.4.1 → 0.6.0

package/README.md

@@ -12,6 +12,11 @@ Local layered memory provider: registers the `files` provider into `ctx.memory`
 
 - `providerName` (default `files`): the registered name.
 - `memoryCharLimit` / `userCharLimit` (`2200` / `1375`) the budget the STORE enforces per target.
+  Deprecated names (G0/S0.3): the canonical ids are the policy row's `memoryChars` /
+  `userChars` (what the review planner reads). The two sides are not
+  auto-synchronised — `/evolution doctor` reports the divergence (`budgetIssues`).
+  Reading the legacy names still works; writing them is refused, and both are
+  removed in 0.7.0.
 - `root`: directory override; `''` means `$DSH_HOME/memories`.
 - `addDatePrefix` / `maxConsolidationFailures` (`false` / `3`) date prefixes on entries; consolidation failures one turn tolerates.
 - `threatExemptLabels` (default `[]`): benign threat labels for the MEMORY store (per-site rule in `evolution-threat`).

package/lib/index.js

@@ -1,12 +1,18 @@
 import { createHash } from "node:crypto";
 import { resolve } from "node:path";
 import z from "@deepseek-ai/schemastery";
-import { DEFAULT_CONSOLIDATION_FAILURES, DEFAULT_MEMORY_CHAR_LIMIT, DEFAULT_USER_CHAR_LIMIT, MemoryStore, clampedNumber, evolutionIoAdapter, makeSerialQueue } from "@lmzhen/dsh-evolution-core";
+import { DEFAULT_CONSOLIDATION_FAILURES, DEFAULT_MEMORY_CHAR_LIMIT, DEFAULT_USER_CHAR_LIMIT, MemoryStore, PARAM_NAMESPACES, clampedNumber, evolutionIoAdapter, installParamSection, makeSerialQueue } from "@lmzhen/dsh-evolution-core";
 //#region lib/types/index.js
 /**
 * Local layered memory provider for `ctx.memory`.
 * @module @lmzhen/dsh-memory-files
 */
+const MEMORY_SETTINGS_SCHEMA = z.object({
+	memoryChars: z.number().min(1).default(DEFAULT_MEMORY_CHAR_LIMIT),
+	userChars: z.number().min(1).default(DEFAULT_USER_CHAR_LIMIT),
+	addDatePrefix: z.boolean().default(false),
+	maxConsolidationFailures: z.number().min(1).default(DEFAULT_CONSOLIDATION_FAILURES)
+});
 const name = "memory-files";
 const inject = ["memory", "evolutionIo"];
 const Config = z.object({
@@ -48,15 +54,30 @@ function apply(ctx, rawConfig = {}) {
 	const serializedWrite = makeSerialQueue();
 	const trimmedRoot = (config.root || "").trim();
 	const resolvedRoot = trimmedRoot === "" ? "" : resolve(trimmedRoot);
-	const store = new MemoryStore({
-		memoryCharLimit: config.memoryCharLimit,
-		userCharLimit: config.userCharLimit,
-		addDatePrefix: config.addDatePrefix,
-		maxConsolidationFailures: config.maxConsolidationFailures,
+	const section = {};
+	const buildStore = () => new MemoryStore({
+		memoryCharLimit: section.overrides?.get("memoryChars") ?? config.memoryCharLimit,
+		userCharLimit: section.overrides?.get("userChars") ?? config.userCharLimit,
+		addDatePrefix: section.overrides?.get("addDatePrefix") ?? config.addDatePrefix,
+		maxConsolidationFailures: section.overrides?.get("maxConsolidationFailures") ?? config.maxConsolidationFailures,
 		threatExemptLabels: config.threatExemptLabels,
 		...resolvedRoot !== "" ? { root: resolvedRoot } : {},
 		io
 	});
+	let store = buildStore();
+	section.overrides = installParamSection(ctx, PARAM_NAMESPACES["memory-files"] ?? "evolution-memory", MEMORY_SETTINGS_SCHEMA, {
+		memoryChars: config.memoryCharLimit,
+		userChars: config.userCharLimit,
+		addDatePrefix: config.addDatePrefix,
+		maxConsolidationFailures: config.maxConsolidationFailures
+	}, {
+		warn: (message) => {
+			ctx.logger.warn("memory-files: " + message);
+		},
+		onChange: () => {
+			store = buildStore();
+		}
+	});
 	const provider = {
 		name: config.providerName,
 		read: (target) => store.read(target),
@@ -80,15 +101,28 @@ function apply(ctx, rawConfig = {}) {
 		},
 		renderContext: () => serializedWrite(() => store.renderContext())
 	};
+	const limitSource = (name, explicit, policyValue) => {
+		if (section.overrides?.get(name) !== void 0) return "user";
+		if (explicit) return "config";
+		return policyValue === void 0 ? "default" : "policy";
+	};
 	ctx.provide("evolutionMemoryBudget", {
-		memoryCharLimit: config.memoryCharLimit,
-		userCharLimit: config.userCharLimit,
-		memorySource: explicitLimit("memoryCharLimit") ? "config" : budget.memory === void 0 ? "default" : "policy",
-		userSource: explicitLimit("userCharLimit") ? "config" : budget.user === void 0 ? "default" : "policy",
+		get memoryCharLimit() {
+			return section.overrides?.get("memoryChars") ?? config.memoryCharLimit;
+		},
+		get userCharLimit() {
+			return section.overrides?.get("userChars") ?? config.userCharLimit;
+		},
+		get memorySource() {
+			return limitSource("memoryChars", explicitLimit("memoryCharLimit"), budget.memory);
+		},
+		get userSource() {
+			return limitSource("userChars", explicitLimit("userCharLimit"), budget.user);
+		},
 		policyMemoryChars: budget.memory,
 		policyUserChars: budget.user
 	});
 	ctx.effect(() => ctx.memory.registerProvider(provider), "memory-files.provider");
 }
 //#endregion
-export { Config, apply, inject, name };
+export { Config, MEMORY_SETTINGS_SCHEMA, apply, inject, name };

package/lib/types/index.d.ts

@@ -4,11 +4,34 @@
  */
 import type { Context } from '@deepseek-ai/cordis';
 import z from '@deepseek-ai/schemastery';
+/** G3/S3.2: the memory section's schema. Field names are the CANONICAL ids from
+ * the parameter registry (`memoryChars`/`userChars` are the policy-side names the
+ * review planner reads too), so the settings document, the params output, the
+ * doctor report and the cards spell one name. */
+export interface MemorySettings {
+    /** Character budget the store enforces for MEMORY.md. */
+    memoryChars: number;
+    /** Character budget the store enforces for USER.md. */
+    userChars: number;
+    /** Prefix stored entries with their date heading. */
+    addDatePrefix: boolean;
+    /** Consolidation failures one turn tolerates before the tool stops retrying. */
+    maxConsolidationFailures: number;
+}
+export declare const MEMORY_SETTINGS_SCHEMA: z<MemorySettings>;
 export declare const name = "memory-files";
 export declare const inject: string[];
 export interface Config {
     providerName?: string;
+    /** Store-enforced character budget for MEMORY.md. The policy row's
+     * `memoryChars` is the same semantic on the review-planner side and is the
+     * canonical id (G0/S0.2); the two are NOT auto-synchronised —
+     * `/evolution doctor` reports the divergence (budgetIssues). Deprecated alias
+     * (G0/S0.3): still readable, refused by writes; removed in 0.7.0. */
     memoryCharLimit?: number;
+    /** Store-enforced character budget for USER.md; the canonical id is the policy
+     * row's `userChars`. Divergence is reported by `/evolution doctor`. Deprecated
+     * alias (G0/S0.3): still readable, refused by writes; removed in 0.7.0. */
     userCharLimit?: number;
     addDatePrefix?: boolean;
     root?: string;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-memory-files",
   "description": "Local layered memory provider (community build)",
-  "version": "0.4.1",
+  "version": "0.6.0",
   "publishConfig": {
     "access": "public"
   },
@@ -27,16 +27,16 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.1",
-    "@lmzhen/dsh-evolution-core": "^0.4.1"
+    "@lmzhen/dsh-evolution-core": "^0.6.0"
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
-    "@lmzhen/dsh-evolution-io": "^0.4.1",
-    "@lmzhen/dsh-memory": "^0.4.1"
+    "@lmzhen/dsh-evolution-io": "^0.6.0",
+    "@lmzhen/dsh-memory": "^0.6.0"
   },
   "devDependencies": {
-    "@lmzhen/dsh-evolution-core": "^0.4.1",
-    "@lmzhen/dsh-evolution-io": "^0.4.1",
-    "@lmzhen/dsh-memory": "^0.4.1"
+    "@lmzhen/dsh-evolution-core": "^0.6.0",
+    "@lmzhen/dsh-evolution-io": "^0.6.0",
+    "@lmzhen/dsh-memory": "^0.6.0"
   }
 }