@caupulican/pi-adaptative 0.81.37 → 0.81.39

package/dist/core/settings-manager.js

@@ -1,16 +1,19 @@
 import { createHash } from "crypto";
 import { existsSync, mkdirSync, readFileSync, realpathSync, rmSync, statSync, writeFileSync } from "fs";
-import { minimatch } from "minimatch";
 import { homedir } from "os";
 import { basename, dirname, join, relative, resolve, sep } from "path";
 import lockfile from "proper-lockfile";
 import { CONFIG_DIR_NAME, getAgentDir, getProfilesDir } from "../config.js";
 import { normalizePath, resolvePath } from "../utils/paths.js";
+import { configFile } from "./agent-paths.js";
+import { DEFAULT_CONTEXT_GC_SETTINGS } from "./context-gc.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 { ProfileRegistry } from "./profile-registry.js";
 import { mergeResourceProfileMap, mergeResourceProfileSettings } from "./resource-profile-blocks.js";
+import { isWorkerSession } from "./session-role.js";
 import { validateSkillName } from "./skills.js";
+import { matchesCompiledPattern } from "./util/minimatch-cache.js";
 export const MEMORY_RETRIEVAL_MAX_RESULTS_MIN = 1;
 export const MEMORY_RETRIEVAL_MAX_RESULTS_MAX = 20;
 const MEMORY_RETRIEVAL_MAX_RESULTS_DEFAULT = 5;
@@ -40,7 +43,13 @@ export const DEFAULT_WORKER_DELEGATION_MAX_USD = 0.5;
 export const DEFAULT_WORKER_DELEGATION_MAX_WALL_CLOCK_MS = 120_000;
 export const MAX_WORKER_DELEGATION_MAX_USD = 5;
 export const MAX_WORKER_DELEGATION_MAX_WALL_CLOCK_MS = 3_600_000;
-export const DEFAULT_LEARNING_POLICY_ENABLED = false;
+export const DEFAULT_WORKTREE_SYNC_POLICY = "on_land_mandatory";
+export const DEFAULT_WORKTREE_SYNC_GATE_TIMEOUT_MS = 900_000;
+export const DEFAULT_WORKTREE_SYNC_MAX_LANES = 5;
+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;
@@ -95,10 +104,14 @@ function findDirectoryProfileRoot(cwd) {
 export function getDirectoryResourceProfileInfo(cwd, agentDir = getAgentDir()) {
     const root = findDirectoryProfileRoot(cwd);
     const hash = createHash("sha256").update(root).digest("hex").slice(0, 16);
+    // resource-profiles/ stays at the agentDir root (not state/): moving it needs the two user-visible
+    // scope descriptions in profile-menu-controller.ts updated in the same change, which has not
+    // happened yet. Still routed through the SSOT root accessor so the eventual move is a one-line
+    // change here.
     return {
         root,
         hash,
-        path: join(resolvePath(agentDir), "resource-profiles", hash, "settings.json"),
+        path: join(configFile(resolvePath(agentDir), "resource-profiles"), hash, "settings.json"),
     };
 }
 export function matchesResourceProfilePattern(resourcePath, patterns, baseDir = "") {
@@ -114,12 +127,12 @@ export function matchesResourceProfilePattern(resourcePath, patterns, baseDir =
     const parentDirPosix = toPosixPath(parentDir);
     return patterns.some((pattern) => {
         const normalizedPattern = toPosixPath(pattern);
-        return (minimatch(rel, normalizedPattern) ||
-            minimatch(name, normalizedPattern) ||
-            minimatch(filePathPosix, normalizedPattern) ||
-            minimatch(parentRel, normalizedPattern) ||
-            minimatch(parentName, normalizedPattern) ||
-            minimatch(parentDirPosix, normalizedPattern));
+        return (matchesCompiledPattern(rel, normalizedPattern) ||
+            matchesCompiledPattern(name, normalizedPattern) ||
+            matchesCompiledPattern(filePathPosix, normalizedPattern) ||
+            matchesCompiledPattern(parentRel, normalizedPattern) ||
+            matchesCompiledPattern(parentName, normalizedPattern) ||
+            matchesCompiledPattern(parentDirPosix, normalizedPattern));
     });
 }
 function normalizeResourceProfileNames(value) {
@@ -784,12 +797,12 @@ export class SettingsManager {
         return matchesResourceProfilePattern(resourcePath, filter.block, baseDir);
     }
     /**
-     * Situational soul(s) of the currently active profile(s) (R6): a system-prompt identity prefix
+     * Situational soul(s) of the currently active profile(s): a system-prompt identity prefix
      * injected while the profile is active. Multiple active profiles' souls are concatenated.
      */
     getActiveProfileSoul() {
         // First-wins precedence (like profile model/thinking): the most-specific active profile's soul
-        // is the identity — concatenating multiple souls would inject contradictory identities (Bug #16).
+        // is the identity — concatenating multiple souls would inject contradictory identities.
         const registry = this.getProfileRegistry();
         const seen = new Set();
         for (const profileName of this.getActiveResourceProfileNames()) {
@@ -1502,6 +1515,12 @@ export class SettingsManager {
         this.modifiedProjectNestedFields.clear();
     }
     enqueueWrite(scope, task) {
+        // Zero-footprint (worker session): this is the universal disk-write choke for every settings
+        // scope (save() -> "global", saveProjectSettings() -> "project", directoryProfile writes) --
+        // a worker session never writes settings.json (or any scoped variant) to disk. In-memory
+        // settings are already updated by the caller before reaching here, so reads are unaffected.
+        if (isWorkerSession())
+            return;
         this.writeQueue = this.writeQueue
             .then(() => {
             if (scope === "project") {
@@ -1867,63 +1886,40 @@ export class SettingsManager {
         this.save();
     }
     getContextGcSettings() {
+        // Per-field fallback throughout (never a whole-object `??`) so an explicit partial override --
+        // e.g. only `contextGc.minToolResultChars`, or only `semanticMemory.enabled` -- still gets the
+        // canonical default for every field it didn't set. Every fallback DERIVES from context-gc.ts's
+        // exported DEFAULT_CONTEXT_GC_SETTINGS (the same object applyContextGc's own normalizer falls
+        // back to) instead of a hand-kept copy, so this method and context-gc.ts's defaults can never
+        // drift again -- hand-copied fallbacks here previously (a) missed the "<task_steps_context"
+        // marker context-gc.ts's own default carries, silently disabling GC packing
+        // for that page under default settings, and (b) diverged on the `tools` list (missing
+        // "run_toolkit_script", present only here) -- both classes of drift are now structurally
+        // impossible since there is exactly one place these defaults are written down.
         return {
-            enabled: this.settings.contextGc?.enabled ?? true,
-            preserveRecentMessages: this.settings.contextGc?.preserveRecentMessages ?? 8,
-            minToolResultChars: this.settings.contextGc?.minToolResultChars ?? 1200,
-            tools: this.settings.contextGc?.tools ?? [
-                "read",
-                "bash",
-                "python",
-                "powershell",
-                "rg",
-                "grep",
-                "find",
-                "ls",
-                "skill_open",
-                "automata_graph_status",
-                "automata_graph_search",
-                "automata_graph_query",
-                "automata_graph_neighbors",
-                "automata_graph_path",
-                "automata_graph_pointer_pack",
-                "learning_query_memory",
-                "subagent",
-                "delegate",
-                "task_steps",
-                "task_background",
-                "task_goal",
-                "run_ledger",
-                "context_headroom_retrieve",
-                "headroom_retrieve",
-            ],
+            enabled: this.settings.contextGc?.enabled ?? DEFAULT_CONTEXT_GC_SETTINGS.enabled,
+            preserveRecentMessages: this.settings.contextGc?.preserveRecentMessages ?? DEFAULT_CONTEXT_GC_SETTINGS.preserveRecentMessages,
+            minToolResultChars: this.settings.contextGc?.minToolResultChars ?? DEFAULT_CONTEXT_GC_SETTINGS.minToolResultChars,
+            tools: this.settings.contextGc?.tools ?? DEFAULT_CONTEXT_GC_SETTINGS.tools,
             semanticMemory: {
-                enabled: this.settings.contextGc?.semanticMemory?.enabled ?? true,
-                preserveRecentPages: this.settings.contextGc?.semanticMemory?.preserveRecentPages ?? 1,
-                minChars: this.settings.contextGc?.semanticMemory?.minChars ?? 900,
-                markers: this.settings.contextGc?.semanticMemory?.markers ?? [
-                    // Generic recall-page marker the bundled default provider (transcript-recall)
-                    // emits; must be present here because a non-empty settings list fully replaces
-                    // the context-gc code default that already includes it.
-                    "<memory_context",
-                    "<automata_context",
-                    "<automata_response",
-                    "<automata_query",
-                    "<automata_fetch",
-                    "<memory_lifecycle_audit",
-                    "<memory_lifecycle_purge",
-                    "<automata_doctor",
-                    "<automata_optimizer",
-                    "<automata_mesh",
-                ],
+                enabled: this.settings.contextGc?.semanticMemory?.enabled ?? DEFAULT_CONTEXT_GC_SETTINGS.semanticMemory.enabled,
+                preserveRecentPages: this.settings.contextGc?.semanticMemory?.preserveRecentPages ??
+                    DEFAULT_CONTEXT_GC_SETTINGS.semanticMemory.preserveRecentPages,
+                minChars: this.settings.contextGc?.semanticMemory?.minChars ?? DEFAULT_CONTEXT_GC_SETTINGS.semanticMemory.minChars,
+                markers: this.settings.contextGc?.semanticMemory?.markers ?? DEFAULT_CONTEXT_GC_SETTINGS.semanticMemory.markers,
             },
         };
     }
     getContextPromptEnforcementSettings() {
+        // Enforcement's own default MIRRORS the live context-gc settings (not a second hardcoded
+        // literal) so tuning contextGc.preserveRecentMessages/minToolResultChars moves this default
+        // with it instead of silently drifting. An explicitly configured enforcement value always
+        // wins over the mirror -- see the ContextPromptEnforcementSettings doc comment.
+        const gcSettings = this.getContextGcSettings();
         return {
             enabled: this.settings.contextPolicy?.enforcement?.enabled ?? false,
-            preserveRecentMessages: this.settings.contextPolicy?.enforcement?.preserveRecentMessages ?? 8,
-            minChars: this.settings.contextPolicy?.enforcement?.minChars ?? 1200,
+            preserveRecentMessages: this.settings.contextPolicy?.enforcement?.preserveRecentMessages ?? gcSettings.preserveRecentMessages,
+            minChars: this.settings.contextPolicy?.enforcement?.minChars ?? gcSettings.minToolResultChars,
         };
     }
     getContextCurationSettings() {
@@ -2408,6 +2404,44 @@ export class SettingsManager {
         this.markModified("researchLane");
         this.save();
     }
+    getWorktreeSyncSettings() {
+        const configured = this.settings.worktreeSync ?? {};
+        const resolved = {
+            enabled: configured.enabled !== false,
+            syncPolicy: configured.syncPolicy === "overlap_mandatory" || configured.syncPolicy === "land_time_only"
+                ? configured.syncPolicy
+                : DEFAULT_WORKTREE_SYNC_POLICY,
+            gate: configured.gate === "off" ? "off" : "on",
+            gateTimeoutMs: sanitizeIntegerSetting(configured.gateTimeoutMs, DEFAULT_WORKTREE_SYNC_GATE_TIMEOUT_MS, 1000, 3_600_000),
+            maxLanes: sanitizeIntegerSetting(configured.maxLanes, DEFAULT_WORKTREE_SYNC_MAX_LANES, 1, 32),
+            workerLand: configured.workerLand === "allow" ? "allow" : "deny",
+        };
+        if (typeof configured.mainBranch === "string" && configured.mainBranch.trim().length > 0) {
+            resolved.mainBranch = configured.mainBranch.trim();
+        }
+        if (typeof configured.gateCommand === "string" && configured.gateCommand.trim().length > 0) {
+            resolved.gateCommand = configured.gateCommand.trim();
+        }
+        if (typeof configured.worktreesRoot === "string" && configured.worktreesRoot.trim().length > 0) {
+            resolved.worktreesRoot = configured.worktreesRoot.trim();
+        }
+        return resolved;
+    }
+    getProcessMatrixSettings() {
+        const configured = this.settings.processMatrix ?? {};
+        return {
+            enabled: configured.enabled !== false,
+            heartbeatMs: sanitizeIntegerSetting(configured.heartbeatMs, DEFAULT_PROCESS_MATRIX_HEARTBEAT_MS, 1000, 600_000),
+            adoptionGraceMs: sanitizeIntegerSetting(configured.adoptionGraceMs, DEFAULT_PROCESS_MATRIX_ADOPTION_GRACE_MS, 5000, 3_600_000),
+            watcherPollMs: sanitizeIntegerSetting(configured.watcherPollMs, DEFAULT_PROCESS_MATRIX_WATCHER_POLL_MS, 1000, 600_000),
+        };
+    }
+    getWindowsShellSettings() {
+        const configured = this.settings.windowsShell ?? {};
+        return {
+            pythonEngine: configured.pythonEngine !== false,
+        };
+    }
     getWorkerDelegationSettings() {
         const configured = this.settings.workerDelegation ?? {};
         const resolved = {