@caupulican/pi-adaptative 0.81.38 → 0.81.40

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 { existsSync, mkdirSync, readFileSync, realpathSync, rmdirSync, rmSync, statSync, writeFileSync } from "fs";
 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, directoryProfilesDir } 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;
@@ -98,7 +107,7 @@ export function getDirectoryResourceProfileInfo(cwd, agentDir = getAgentDir()) {
     return {
         root,
         hash,
-        path: join(resolvePath(agentDir), "resource-profiles", hash, "settings.json"),
+        path: join(directoryProfilesDir(resolvePath(agentDir)), hash, "settings.json"),
     };
 }
 export function matchesResourceProfilePattern(resourcePath, patterns, baseDir = "") {
@@ -114,12 +123,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) {
@@ -308,14 +317,16 @@ export class FileSettingsStorage {
     globalSettingsPath;
     projectSettingsPath;
     directoryProfileInfo;
+    legacyDirectoryProfilePath;
     profilesDir;
     constructor(cwd, agentDir) {
         const resolvedCwd = resolvePath(cwd);
         const resolvedAgentDir = resolvePath(agentDir);
         this.profileResolutionBaseDir = resolvedCwd;
-        this.globalSettingsPath = join(resolvedAgentDir, "settings.json");
+        this.globalSettingsPath = configFile(resolvedAgentDir, "settings.json");
         this.projectSettingsPath = join(resolvedCwd, CONFIG_DIR_NAME, "settings.json");
         this.directoryProfileInfo = getDirectoryResourceProfileInfo(resolvedCwd, resolvedAgentDir);
+        this.legacyDirectoryProfilePath = join(resolvedAgentDir, "resource-profiles", this.directoryProfileInfo.hash, "settings.json");
         this.profilesDir = getProfilesDir(resolvedAgentDir);
     }
     getDirectoryResourceProfileInfo() {
@@ -329,7 +340,30 @@ export class FileSettingsStorage {
     }
     readDirectoryResourceProfile() {
         const path = this.directoryProfileInfo.path;
-        return existsSync(path) ? readFileSync(path, "utf-8") : undefined;
+        if (existsSync(path))
+            return readFileSync(path, "utf-8");
+        return existsSync(this.legacyDirectoryProfilePath)
+            ? readFileSync(this.legacyDirectoryProfilePath, "utf-8")
+            : undefined;
+    }
+    removeMigratedDirectoryProfile() {
+        if (!existsSync(this.legacyDirectoryProfilePath))
+            return;
+        try {
+            rmSync(this.legacyDirectoryProfilePath, { force: true });
+        }
+        catch {
+            return;
+        }
+        for (const directory of [
+            dirname(this.legacyDirectoryProfilePath),
+            dirname(dirname(this.legacyDirectoryProfilePath)),
+        ]) {
+            try {
+                rmdirSync(directory);
+            }
+            catch { }
+        }
     }
     acquireLockSyncWithRetry(path) {
         const maxAttempts = 10;
@@ -362,30 +396,47 @@ export class FileSettingsStorage {
                 ? this.projectSettingsPath
                 : this.directoryProfileInfo.path;
         const dir = dirname(path);
-        let release;
+        const legacyPath = scope === "directoryProfile" ? this.legacyDirectoryProfilePath : undefined;
+        const releases = [];
+        let migratedLegacy = false;
         try {
-            // Only create directory and lock if file exists or we need to write
-            const fileExists = existsSync(path);
-            if (fileExists) {
-                release = this.acquireLockSyncWithRetry(path);
+            const legacyExists = legacyPath !== undefined && existsSync(legacyPath);
+            const canonicalExists = existsSync(path);
+            let currentPath = canonicalExists ? path : legacyExists ? (legacyPath ?? path) : path;
+            const fileExists = canonicalExists || legacyExists;
+            // Directory-profile locks always follow legacy -> canonical order, including a partial migration
+            // where both files exist. This keeps old and new SDK clients from deadlocking each other.
+            if (legacyPath && legacyExists)
+                releases.push(this.acquireLockSyncWithRetry(legacyPath));
+            if (canonicalExists)
+                releases.push(this.acquireLockSyncWithRetry(path));
+            // A legacy directory overlay can race another process that already knows the canonical path.
+            // Hold the legacy lock, then bind to the canonical lock before deriving the update.
+            if (legacyExists && !canonicalExists) {
+                if (!existsSync(dir))
+                    mkdirSync(dir, { recursive: true });
+                releases.push(this.acquireLockSyncWithRetry(path));
+                if (existsSync(path))
+                    currentPath = path;
             }
-            const current = fileExists ? readFileSync(path, "utf-8") : undefined;
+            const current = fileExists ? readFileSync(currentPath, "utf-8") : undefined;
             const next = fn(current);
             if (next !== undefined) {
                 // Only create directory when we actually need to write
                 if (!existsSync(dir)) {
                     mkdirSync(dir, { recursive: true });
                 }
-                if (!release) {
-                    release = this.acquireLockSyncWithRetry(path);
-                }
+                if (releases.length === 0)
+                    releases.push(this.acquireLockSyncWithRetry(path));
                 writeFileSync(path, next, "utf-8");
+                migratedLegacy = legacyExists;
             }
         }
         finally {
-            if (release) {
+            for (const release of releases.reverse())
                 release();
-            }
+            if (migratedLegacy)
+                this.removeMigratedDirectoryProfile();
         }
     }
 }
@@ -784,12 +835,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 +1553,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 +1924,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() {
@@ -2250,6 +2284,22 @@ export class SettingsManager {
         this.markModified("images", "blockImages");
         this.save();
     }
+    getClipboardImageDirectory() {
+        return this.settings.images?.clipboardDirectory?.trim() || undefined;
+    }
+    setClipboardImageDirectory(directory) {
+        if (!this.globalSettings.images) {
+            this.globalSettings.images = {};
+        }
+        if (directory?.trim()) {
+            this.globalSettings.images.clipboardDirectory = directory.trim();
+        }
+        else {
+            delete this.globalSettings.images.clipboardDirectory;
+        }
+        this.markModified("images", "clipboardDirectory");
+        this.save();
+    }
     getEnabledModels() {
         return this.settings.enabledModels;
     }
@@ -2408,6 +2458,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 = {
@@ -2420,17 +2508,15 @@ export class SettingsManager {
                 : [],
             maxConcurrent: sanitizeIntegerSetting(configured.maxConcurrent, 1, 1, 3),
         };
-        if (typeof configured.model === "string" && configured.model.trim().length > 0) {
-            resolved.model = configured.model;
-        }
-        if (typeof configured.profile === "string" && configured.profile.trim().length > 0) {
-            resolved.profile = configured.profile;
-        }
-        if (typeof configured.systemPrompt === "string" && configured.systemPrompt.trim().length > 0) {
-            resolved.systemPrompt = configured.systemPrompt;
+        if (typeof configured.orchestrationProfile === "string" && configured.orchestrationProfile.trim().length > 0) {
+            resolved.orchestrationProfile = configured.orchestrationProfile.trim();
         }
         return resolved;
     }
+    getActiveOrchestrationProfile() {
+        const profileId = this.settings.activeOrchestrationProfile;
+        return typeof profileId === "string" && profileId.trim().length > 0 ? profileId.trim() : undefined;
+    }
     setWorkerDelegationSettings(settings, scope = "global") {
         if (scope === "project") {
             const projectSettings = structuredClone(this.projectSettings);