@caupulican/pi-adaptative 0.94.0 → 0.95.0

package/dist/core/settings-manager.js

@@ -10,7 +10,7 @@ import { DEFAULT_CONTEXT_GC_SETTINGS } from "./context-gc.js";
 import { DEFAULT_COST_GUARD_SETTINGS } from "./cost-guard.js";
 import { DEFAULT_GOAL_AUTO_CONTINUE, DEFAULT_GOAL_AUTO_CONTINUE_DELAY_MS, DEFAULT_GOAL_CONTINUE_MAX_STALL_TURNS, DEFAULT_GOAL_CONTINUE_MAX_TURNS, DEFAULT_GOAL_CONTINUE_MAX_WALL_CLOCK_MINUTES, MAX_GOAL_AUTO_CONTINUE_DELAY_MS, MAX_GOAL_CONTINUE_MAX_STALL_TURNS, MAX_GOAL_CONTINUE_MAX_TURNS, MAX_GOAL_CONTINUE_MAX_WALL_CLOCK_MINUTES, } from "./goals/goal-continuation-defaults.js";
 import { DEFAULT_HTTP_IDLE_TIMEOUT_MS, parseHttpIdleTimeoutMs } from "./http-dispatcher.js";
-import { MAX_ORCHESTRATION_IDENTIFIER_LENGTH, MAX_WORKER_AUTHORITY_PATH_LENGTH, MAX_WORKER_AUTHORITY_PATHS, } from "./orchestration/contracts.js";
+import { MAX_ORCHESTRATION_IDENTIFIER_LENGTH } from "./orchestration/contracts.js";
 import { compileWorkerModelPinPolicy, } from "./orchestration/worker-model-pins.js";
 import { ProfileRegistry } from "./profile-registry.js";
 import { mergeResourceProfileMap, mergeResourceProfileSettings, normalizeResourceProfileFilter, } from "./resource-profile-blocks.js";
@@ -48,7 +48,6 @@ export const DEFAULT_WORKER_DELEGATION_MAX_USD = 0;
 export const DEFAULT_WORKER_DELEGATION_MAX_WALL_CLOCK_MS = 0;
 export const DEFAULT_WORKER_DELEGATION_MAX_CONCURRENT = 20;
 export const DEFAULT_WORKER_DELEGATION_WRITE_ENABLED = true;
-export const DEFAULT_WORKER_DELEGATION_WRITE_PATHS = ["."];
 export const MAX_WORKER_DELEGATION_MAX_USD = Number.MAX_SAFE_INTEGER;
 export const MAX_WORKER_DELEGATION_MAX_WALL_CLOCK_MS = Number.MAX_SAFE_INTEGER;
 export const MAX_WORKER_DELEGATION_MAX_CONCURRENT = Number.MAX_SAFE_INTEGER;
@@ -59,10 +58,10 @@ export const DEFAULT_PROCESS_MATRIX_HEARTBEAT_MS = 30_000;
 export const DEFAULT_PROCESS_MATRIX_ADOPTION_GRACE_MS = 300_000;
 export const DEFAULT_PROCESS_MATRIX_WATCHER_POLL_MS = 25_000;
 export const DEFAULT_LEARNING_POLICY_ENABLED = true;
-export const DEFAULT_LEARNING_POLICY_AUTO_APPLY_ENABLED = false;
-export const DEFAULT_LEARNING_POLICY_CONFIDENCE_THRESHOLD = 90;
-export const DEFAULT_LEARNING_POLICY_MIN_OBSERVATIONS = 2;
-export const DEFAULT_LEARNING_POLICY_ALLOWED_AUTO_APPLY_LAYERS = ["memory"];
+export const DEFAULT_LEARNING_POLICY_AUTO_APPLY_ENABLED = true;
+export const DEFAULT_LEARNING_POLICY_CONFIDENCE_THRESHOLD = 50;
+export const DEFAULT_LEARNING_POLICY_MIN_OBSERVATIONS = 1;
+export const DEFAULT_LEARNING_POLICY_ALLOWED_AUTO_APPLY_LAYERS = ["memory", "skill"];
 export const DEFAULT_LEARNING_POLICY_REFLECTION_SOURCE_CONFIDENCE = 50;
 export const DEFAULT_LEARNING_POLICY_AUTO_APPLY_SUPERSESSIONS = false;
 export const DEFAULT_MODEL_CAPABILITY_MODE = "auto";
@@ -271,39 +270,6 @@ function sanitizeNumberSetting(value, fallback, min, max) {
 function reportInvalidWorkerDelegationField(reportDiagnostic, field, expectation) {
     reportDiagnostic?.(`workerDelegation.${field} must be ${expectation}; the configured value was ignored`);
 }
-function normalizeWorkerDelegationWritePaths(value, reportDiagnostic) {
-    if (!Array.isArray(value)) {
-        reportInvalidWorkerDelegationField(reportDiagnostic, "writePaths", "an array of path strings");
-        return undefined;
-    }
-    const normalized = [];
-    const seen = new Set();
-    let canonicalized = false;
-    for (let index = 0; index < value.length; index++) {
-        const entry = value[index];
-        if (typeof entry !== "string") {
-            canonicalized = true;
-            continue;
-        }
-        const trimmed = entry.trim();
-        if (!trimmed || trimmed.length > MAX_WORKER_AUTHORITY_PATH_LENGTH || seen.has(trimmed)) {
-            canonicalized = true;
-            continue;
-        }
-        if (trimmed !== entry)
-            canonicalized = true;
-        seen.add(trimmed);
-        normalized.push(trimmed);
-        if (normalized.length === MAX_WORKER_AUTHORITY_PATHS && index < value.length - 1) {
-            canonicalized = true;
-            break;
-        }
-    }
-    if (canonicalized) {
-        reportDiagnostic?.(`workerDelegation.writePaths was normalized to at most ${MAX_WORKER_AUTHORITY_PATHS} unique, nonempty paths of at most ${MAX_WORKER_AUTHORITY_PATH_LENGTH} characters`);
-    }
-    return normalized;
-}
 /** Normalize one precedence layer before merging so malformed overrides cannot widen lower-scope authority. */
 function normalizeWorkerDelegationLayer(value, reportDiagnostic) {
     if (!isPlainRecord(value)) {
@@ -360,11 +326,6 @@ function normalizeWorkerDelegationLayer(value, reportDiagnostic) {
         else
             reportInvalidWorkerDelegationField(reportDiagnostic, "writeEnabled", "a boolean");
     }
-    if (Object.hasOwn(value, "writePaths")) {
-        const writePaths = normalizeWorkerDelegationWritePaths(value.writePaths, reportDiagnostic);
-        if (writePaths !== undefined)
-            normalized.writePaths = writePaths;
-    }
     if (Object.hasOwn(value, "maxConcurrent")) {
         if (typeof value.maxConcurrent === "number" &&
             Number.isSafeInteger(value.maxConcurrent) &&
@@ -598,6 +559,7 @@ export class SettingsManager {
         this.projectSettingsLoadError = null; // Track if project settings file had parse errors
         this.directoryProfileInfo = null;
         this.writeQueue = Promise.resolve();
+        this.changeListeners = new Set();
         this.profileDiagnosticKeys = new Set();
         this.storage = storage;
         this.globalSettings = initialGlobal;
@@ -720,6 +682,22 @@ export class SettingsManager {
     recomputeSettings() {
         this.settings = this.mergeEffectiveSettings();
         this.refreshProfileRegistry();
+        this.notifyChanges();
+    }
+    /** Subscribe to synchronous effective-settings transitions. Listener failures never break persistence. */
+    subscribeChanges(listener) {
+        this.changeListeners.add(listener);
+        return () => this.changeListeners.delete(listener);
+    }
+    notifyChanges() {
+        for (const listener of this.changeListeners) {
+            try {
+                listener();
+            }
+            catch {
+                // Settings persistence is authoritative; a runtime projection can recover on its next refresh.
+            }
+        }
     }
     /** Create a SettingsManager that loads from files */
     static create(cwd, agentDir = getAgentDir(), options = {}) {
@@ -1098,6 +1076,7 @@ export class SettingsManager {
         this.directoryProfileInfo = snapshot.directoryProfileInfo ? { ...snapshot.directoryProfileInfo } : null;
         this.errors = [...snapshot.errors];
         this.refreshProfileRegistry();
+        this.notifyChanges();
     }
     /** Apply additional overrides on top of current settings */
     applyOverrides(overrides) {
@@ -1110,6 +1089,7 @@ export class SettingsManager {
                 delete normalizedOverrides.workerDelegation;
         }
         this.settings = deepMergeSettings(this.settings, normalizedOverrides);
+        this.notifyChanges();
     }
     /** Select runtime-only resource profiles, e.g. from CLI/subagent launch options. */
     setRuntimeResourceProfiles(profileNames) {
@@ -1881,15 +1861,39 @@ export class SettingsManager {
         };
     }
     /**
-     * Optional per-turn cost guard (#34). Disabled by default so productive work has no implicit spend
-     * limiter. Set a positive `maxTurnUsd` to warn at that projected ceiling, or pair it with
-     * `action: "downgrade"` to also auto-reduce reasoning effort.
+     * Optional per-turn cost guard (#34). Explicit `enabled: true` is mandatory; this keeps positive
+     * thresholds written by older releases dormant instead of silently restoring a spend guard.
      */
     getCostGuardSettings() {
+        const configuredMaxTurnUsd = this.settings.costGuard?.maxTurnUsd;
         return {
-            maxTurnUsd: this.settings.costGuard?.maxTurnUsd ?? DEFAULT_COST_GUARD_SETTINGS.maxTurnUsd,
-            action: this.settings.costGuard?.action ?? DEFAULT_COST_GUARD_SETTINGS.action,
+            enabled: this.settings.costGuard?.enabled === true,
+            maxTurnUsd: typeof configuredMaxTurnUsd === "number" &&
+                Number.isFinite(configuredMaxTurnUsd) &&
+                configuredMaxTurnUsd >= 0
+                ? configuredMaxTurnUsd
+                : DEFAULT_COST_GUARD_SETTINGS.maxTurnUsd,
+            action: this.settings.costGuard?.action === "downgrade" ? "downgrade" : DEFAULT_COST_GUARD_SETTINGS.action,
+        };
+    }
+    setCostGuardSettings(settings, scope = "global") {
+        const normalized = {
+            enabled: settings.enabled === true,
+            maxTurnUsd: Number.isFinite(settings.maxTurnUsd) && settings.maxTurnUsd >= 0
+                ? settings.maxTurnUsd
+                : DEFAULT_COST_GUARD_SETTINGS.maxTurnUsd,
+            action: settings.action === "downgrade" ? "downgrade" : "warn",
         };
+        if (scope === "project") {
+            const projectSettings = structuredClone(this.projectSettings);
+            projectSettings.costGuard = normalized;
+            this.markProjectModified("costGuard");
+            this.saveProjectSettings(projectSettings);
+            return;
+        }
+        this.globalSettings.costGuard = normalized;
+        this.markModified("costGuard");
+        this.save();
     }
     getFailoverSettings() {
         return { subscriptionHop: this.settings.failover?.subscriptionHop ?? true };
@@ -2652,7 +2656,6 @@ export class SettingsManager {
             writeEnabled: typeof configured.writeEnabled === "boolean"
                 ? configured.writeEnabled
                 : DEFAULT_WORKER_DELEGATION_WRITE_ENABLED,
-            writePaths: configured.writePaths ? [...configured.writePaths] : [...DEFAULT_WORKER_DELEGATION_WRITE_PATHS],
             maxConcurrent: sanitizeIntegerSetting(configured.maxConcurrent, DEFAULT_WORKER_DELEGATION_MAX_CONCURRENT, 1, MAX_WORKER_DELEGATION_MAX_CONCURRENT),
         };
         if (typeof configured.orchestrationProfile === "string" && configured.orchestrationProfile.trim().length > 0) {