mixdog 0.9.63 → 0.9.64

package/README.md

@@ -219,6 +219,9 @@ src/
 vendor/
 ```
 
+Test, smoke, and bench scripts are excluded from the tarball; they live in the
+repository only.
+
 `docs/` is not included in the published package unless `package.json#files` is
 changed.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mixdog",
-  "version": "0.9.63",
+  "version": "0.9.64",
   "private": false,
   "type": "module",
   "description": "Standalone mixdog coding-agent CLI/TUI workspace.",
@@ -28,6 +28,13 @@
     "scripts/",
     "!scripts/bench/",
     "!scripts/recall-bench-*.txt",
+    "!scripts/*-test.mjs",
+    "!scripts/*-smoke.mjs",
+    "!scripts/smoke-loop*.mjs",
+    "!scripts/*-bench.mjs",
+    "!scripts/*bench-*.mjs",
+    "!scripts/*.ps1",
+    "!scripts/*.jsx",
     "src/",
     "!src/tui/dev",
     "vendor/"
@@ -72,7 +79,7 @@
     "test:tui-streaming-window": "node --test scripts/streaming-tail-window-test.mjs",
     "test:tui-ambiguous-width": "node --test scripts/tui-ambiguous-width-test.mjs",
     "test:release-assets": "node --check scripts/verify-release-assets.mjs && node --check scripts/verify-release-assets-test.mjs && node --test scripts/verify-release-assets-test.mjs",
-    "test:release-focused": "npm run test:release-assets && npm run test:patch-binary-cache && npm run test:providers && npm run test:deferred-tools && npm run smoke:compact && node --test scripts/code-graph-aggregate-cwd-test.mjs && npm run test:code-graph-dispatch && node --test scripts/code-graph-disk-hit-test.mjs && npm run test:shellhardening && node --test scripts/windows-hide-spawn-options-test.mjs && npm run test:session && node --test scripts/tui-transcript-perf-test.mjs",
+    "test:release-focused": "npm run test:release-assets && npm run test:tool-contracts && npm run test:placeholder && npm run smoke:patch && npm run test:patch-binary-cache && npm run test:providers && npm run test:deferred-tools && npm run smoke:compact && node --test scripts/code-graph-root-federation-test.mjs scripts/code-graph-aggregate-cwd-test.mjs && npm run test:code-graph-dispatch && node --test scripts/code-graph-disk-hit-test.mjs && npm run test:shellhardening && node --test scripts/windows-hide-spawn-options-test.mjs && npm run test:session && node --test scripts/tui-transcript-perf-test.mjs",
     "test:native-edit-wire": "node --test scripts/native-edit-wire-test.mjs",
     "test:patch-binary-cache": "node --test scripts/patch-binary-cache-test.mjs",
     "test:session": "node --test scripts/session-orphan-sweep-test.mjs scripts/interrupted-turn-history-test.mjs scripts/session-heartbeat-lifecycle-test.mjs",

package/scripts/code-graph-description-contract.mjs

@@ -25,7 +25,7 @@ const removeIdentifier = (text, identifier) => (
   text.replace(new RegExp(`(?<![a-z0-9_])${escapeRegExp(identifier)}(?![a-z0-9_])`, 'gi'), '')
 );
 
-export const CODE_GRAPH_EQUIVALENT_DESCRIPTION_PROBES = [
+const CODE_GRAPH_EQUIVALENT_DESCRIPTION_PROBES = [
   {
     description: [
       'Keywords use search/symbol_search while callers/callees/references/find_symbol use exact identifiers.',
@@ -64,7 +64,7 @@ export const CODE_GRAPH_EQUIVALENT_DESCRIPTION_PROBES = [
   },
 ];
 
-export const CODE_GRAPH_DESCRIPTION_MUTATION_CORPUS = [
+const CODE_GRAPH_DESCRIPTION_MUTATION_CORPUS = [
   {
     name: 'contracted negated file assignment',
     mutate: (parts) => ({
@@ -140,7 +140,7 @@ export const CODE_GRAPH_DESCRIPTION_MUTATION_CORPUS = [
   },
 ];
 
-export function hasCodeGraphDescriptionContract({ description, modeDescription, symbolsDescription }) {
+function hasCodeGraphDescriptionContract({ description, modeDescription, symbolsDescription }) {
   return (
     hasPositiveClause(description, ['file modes', 'files[]'])
     && hasModeClause(description, 'symbol modes', SYMBOL_MODES, 'symbols[]')

package/scripts/verify-release-assets.mjs

@@ -27,8 +27,8 @@ const RUNTIME_PLATFORMS = [
 ];
 const STRICT_VERSION = /^(0|[1-9][0-9]*)\.(0|[1-9][0-9]*)\.(0|[1-9][0-9]*)$/;
 const SHA256 = /^[a-f0-9]{64}$/i;
-export const MAX_ASSET_BYTES = 256 * 1024 * 1024;
-export const MAX_DOWNLOAD_TIMEOUT_MS = 300_000;
+const MAX_ASSET_BYTES = 256 * 1024 * 1024;
+const MAX_DOWNLOAD_TIMEOUT_MS = 300_000;
 const PATCH_MANIFEST_PATH = 'src/runtime/agent/orchestrator/tools/patch-manifest.json';
 const PATCH_CARGO_PATH = 'native/mixdog-patch/Cargo.toml';
 const RUNTIME_MANIFEST_PATH = 'src/runtime/memory/data/runtime-manifest.json';

package/src/runtime/agent/orchestrator/agent-runtime/agent-dispatch.mjs

@@ -142,19 +142,19 @@ export function resolveHiddenRoleSchemaAllowedTools(hidden) {
 }
 
 /**
- * Resolve the maintenance ROUTE (or legacy preset name) for a dispatch.
+ * Resolve the maintenance ROUTE (or preset name) for a dispatch.
  *
  * Returns one of:
  *   - a route object `{ provider, model, effort?, fast? }` — the preferred
  *     shape: a maintenance slot now stores its model directly (parity with
  *     `agents.<role>`), so no preset-array name lookup is needed.
- *   - a string — a legacy preset NAME still stored in a maintenance slot, or an
- *     explicit `preset`/`opts.preset` override. The caller resolves the name
- *     against config.presets for backward compatibility.
+ *   - a string — a preset NAME (Main inheritance via config.default, or an
+ *     explicit `preset`/`opts.preset` override). The caller resolves the name
+ *     against config.presets.
  *   - null — unresolved.
  *
  * Explore and memory hidden roles mirror public spawning precedence:
- * `agents.<role>` (including legacy `agents.maintenance`) → workflow route →
+ * `agents.<role>` (including the `agents.maintenance` alias) → workflow route →
  * maintenance route → Main. The cycle1/2/3 agents share the memory knob via
  * their `maintKey: 'memory'` override. Scheduler and webhook are unchanged.
  */
@@ -207,9 +207,6 @@ export function resolveMaintenanceRoute({ preset, optsPreset, agent, config: cfg
     return null;
 }
 
-// Back-compat alias: older callers/tests import resolvePresetName. It now
-// returns whatever resolveMaintenanceRoute does (route object OR name string).
-export const resolvePresetName = resolveMaintenanceRoute;
 
 // A maintenance slot value is a direct route when it carries provider+model.
 function maintenanceRouteToPreset(routeOrName, agent) {
@@ -324,22 +321,22 @@ export function makeAgentDispatch(opts = {}) {
             );
         }
         // Preferred path: a maintenance slot that stores its model directly
-        // (route object). Legacy path: a slot still holding a preset NAME —
-        // resolve it against config.presets for backward compatibility.
+        // (route object). Name path: Main inheritance (config.default) or an
+        // explicit preset override still identify a preset by NAME.
         let preset = maintenanceRouteToPreset(routeOrName, agent);
         if (!preset) {
-            const legacyName = String(routeOrName || '').trim();
-            preset = config.presets?.find((p) => p.id === legacyName || p.name === legacyName) || null;
+            const routeName = String(routeOrName || '').trim();
+            preset = config.presets?.find((p) => p.id === routeName || p.name === routeName) || null;
             if (!preset) {
                 throw new Error(
                     `[agent-dispatch] maintenance route for agent "${agent}" is neither a `
-                    + `{provider,model} route nor a known preset name ("${legacyName}")`,
+                    + `{provider,model} route nor a known preset name ("${routeName}")`,
                 );
             }
         }
         // Stable label for traces / session metadata, derived from the resolved
         // preset object regardless of whether it came from a direct route or a
-        // legacy preset name.
+        // preset name.
         const presetName = preset.id || preset.name || `maint-${agent}`;
 
         const runtimeSpec = resolveRuntimeSpec(preset, {

package/src/runtime/agent/orchestrator/agent-runtime/agent-progress-watchdog.mjs

@@ -28,7 +28,7 @@ export class AgentStallAbortError extends Error {
     }
 }
 
-export function isAgentProgressWatchdogAbortError(err) {
+function isAgentProgressWatchdogAbortError(err) {
     const msg = err?.message;
     return typeof msg === 'string' && WATCHDOG_ABORT_RE.test(msg);
 }
@@ -104,7 +104,7 @@ export function resolveHandoffMessageStartIndex(session) {
     return messages.length;
 }
 
-export function collectSessionAssistantHandoffText(session, messageStartIndex = 0) {
+function collectSessionAssistantHandoffText(session, messageStartIndex = 0) {
     const messages = Array.isArray(session?.messages) ? session.messages : [];
     const start = Math.max(0, Math.floor(Number(messageStartIndex) || 0));
     const parts = [];
@@ -141,7 +141,7 @@ function resolveWatchdogAbortElapsedMs({ error, snapshot, policy, now, anchorTs,
     return null;
 }
 
-export function recordAgentWatchdogAbort({
+function recordAgentWatchdogAbort({
     sessionId,
     agent = null,
     error,
@@ -190,15 +190,15 @@ function envTimeoutMs(name, fallback) {
     return Number.isFinite(n) && n >= 0 ? Math.floor(n) : fallback;
 }
 
-export const DEFAULT_FIRST_RESPONSE_TIMEOUT_MS = envTimeoutMs(
+const DEFAULT_FIRST_RESPONSE_TIMEOUT_MS = envTimeoutMs(
     'MIXDOG_AGENT_FIRST_RESPONSE_TIMEOUT_MS',
     120_000,
 );
-export const DEFAULT_FIRST_VISIBLE_CEILING_MS = envTimeoutMs(
+const DEFAULT_FIRST_VISIBLE_CEILING_MS = envTimeoutMs(
     'MIXDOG_AGENT_FIRST_VISIBLE_TIMEOUT_MS',
     600_000,
 );
-export const DEFAULT_STALE_TIMEOUT_MS = envTimeoutMs(
+const DEFAULT_STALE_TIMEOUT_MS = envTimeoutMs(
     'MIXDOG_AGENT_STALE_TIMEOUT_MS',
     30 * 60_000,
 );

package/src/runtime/agent/orchestrator/agent-runtime/cache-strategy.mjs

@@ -463,7 +463,7 @@ export function buildProviderCacheOpts(provider, sessionId, agent, options = {})
  * order (BP1 / BP2 / ...), serialized deterministically as a JSON array.
  * Invariant: callers must pass an array.
  */
-export function computePrefixContent(systemPrompt, tools) {
+function computePrefixContent(systemPrompt, tools) {
     const systemMessages = Array.isArray(systemPrompt)
         ? systemPrompt.map(s => s == null ? '' : String(s))
         : [systemPrompt == null ? '' : String(systemPrompt)];
@@ -480,7 +480,7 @@ export function computePrefixContent(systemPrompt, tools) {
 /**
  * Longest-lived layer TTL (seconds) for registry expiry tracking.
  */
-export function ttlSecondsForCache(agent) {
+function ttlSecondsForCache(agent) {
     const ttls = resolveCacheStrategy(agent);
     if (
         ttls.tools === 'none'

package/src/runtime/agent/orchestrator/agent-trace-format.mjs

@@ -281,6 +281,7 @@ function classifyToolFailure(resultText, toolName) {
         if (/\[signal:\s*[^\]]+/i.test(leading)) return 'process/signal';
         return 'command-exit';
     }
+    if (/\[tool-input-validation\]|compacted-history placeholder/.test(text)) return 'schema/args';
     if (/requires either|invalid arguments|unknown parameter|must be|schema|expected|required|old_string is .*>=/.test(text)) return 'schema/args';
     if (/not in allow-list|not allowed/.test(text)) return 'permission';
     if (String(toolName || '') === 'shell' || /^\s*\[exit code:\s*\d+\]/i.test(raw)) return 'command-exit';

package/src/runtime/agent/orchestrator/config.mjs

@@ -46,7 +46,7 @@ export function getPluginData() {
 // SUBSET of DEFAULT_MAINTENANCE: scheduler/webhook carry a per-entry model and
 // are not shown as shared rows, but still inherit the haiku default above when
 // an entry omits its own model.
-export const MAINTENANCE_SLOTS = Object.freeze(['explore', 'memory']);
+const MAINTENANCE_SLOTS = Object.freeze(['explore', 'memory']);
 
 // --- User profile (statusline /profile) -------------------------------------
 // Supported response languages for the /profile picker. `system` is the default
@@ -129,10 +129,10 @@ function resolveAnthropicFamilyModel(family) {
 // Each maintenance slot stores its model route DIRECTLY ({provider, model}) —
 // parity with `agents.<role>`. The old shape stored a preset NAME string (e.g.
 // "haiku") that had to be looked up in the config.presets array; that
-// indirection is the legacy path. agent-dispatch.resolveMaintenanceRoute still
-// accepts a legacy name string for backward compatibility, but new configs and
-// these defaults use the direct route. loadConfig() migrates any stored string
-// slot to a route on read (see migrateMaintenanceRoutes). The cycle1/2/3 memory
+// indirection resolves at dispatch. agent-dispatch.resolveMaintenanceRoute
+// accepts a preset NAME (Main inheritance via config.default), and
+// these defaults use the direct route. loadConfig() normalizes stored
+// slots on read (see normalizeMaintenanceRoutes). The cycle1/2/3 memory
 // agents share ONE `memory` route via the `maintKey: 'memory'` override on
 // their hidden-role entries.
 const _HAIKU_ROUTE = Object.freeze({
@@ -227,15 +227,11 @@ function normalizeSearchRoute(route) {
     return out;
 }
 
-// Migrate stored maintenance slots from the legacy preset-NAME string shape to
-// the direct {provider, model} route shape. A slot value that is already a
-// route object is normalized (provider/model/effort/fast only); a string value
-// is resolved against the config.presets array (the legacy lookup) and rewritten
-// to a route. Unresolvable strings are dropped so the DEFAULT_MAINTENANCE route
-// fills the slot. `presets` is the normalized preset array for legacy lookup.
-function migrateMaintenanceRoutes(rawMaint, presets, defaultProvider) {
+// Normalize stored maintenance slots to the direct {provider, model} route
+// shape (provider/model/effort/fast only). Non-route values are dropped so the
+// DEFAULT_MAINTENANCE route fills the slot.
+function normalizeMaintenanceRoutes(rawMaint, defaultProvider) {
     const out = {};
-    const list = Array.isArray(presets) ? presets : [];
     const configuredProvider = String(defaultProvider || '').trim();
     const fallbackProvider = isConfiguredProviderId(configuredProvider)
         ? configuredProvider
@@ -255,16 +251,6 @@ function migrateMaintenanceRoutes(rawMaint, presets, defaultProvider) {
             }
             continue;
         }
-        const name = String(value || '').trim();
-        if (!name) continue;
-        const preset = list.find(p => p && (p.id === name || p.name === name));
-        if (preset && preset.provider && preset.model) {
-            const route = { provider: preset.provider, model: preset.model };
-            if (preset.effort) route.effort = preset.effort;
-            if (preset.fast === true) route.fast = true;
-            out[slot] = route;
-        }
-        // Unresolvable legacy name → dropped; DEFAULT_MAINTENANCE route fills it.
     }
     return out;
 }
@@ -288,25 +274,10 @@ async function persistAgentConfigAsync(build) {
 }
 
 // Recap toggle (recap.enabled, default true) gates ONLY the background memory
-// cycles. The memory module itself is always-on. On load we fold a legacy
-// `modules.memory === false` flag into recap.enabled=false one time; the caller
-// (loadConfig) persists the migration so the legacy flag is dropped from disk.
-let _migratedRecapLegacy = false;
-function normalizeRecapConfig(rawRecap, rawModules) {
+// cycles. The memory module itself is always-on.
+function normalizeRecapConfig(rawRecap) {
     const recap = rawRecap && typeof rawRecap === 'object' ? { ...rawRecap } : {};
-    if (!('enabled' in recap)) {
-        const legacyMemory = rawModules && typeof rawModules === 'object' ? rawModules.memory : undefined;
-        const legacyDisabled = legacyMemory === false
-            || (legacyMemory && typeof legacyMemory === 'object' && legacyMemory.enabled === false);
-        if (legacyDisabled) {
-            recap.enabled = false;
-            _migratedRecapLegacy = true;
-        } else {
-            recap.enabled = true;
-        }
-    } else {
-        recap.enabled = recap.enabled !== false;
-    }
+    recap.enabled = recap.enabled !== false;
     return recap;
 }
 
@@ -355,19 +326,7 @@ export function loadConfig(options = {}) {
             for (const [k, v] of Object.entries(raw.maintenance || {})) {
                 if (allowedMaintKeys.has(k)) rawMaint[k] = v;
             }
-            // One-time schema migration: the three memory-cycle MODEL presets
-            // (cycle1/cycle2/cycle3) collapsed into a single `memory` key. If the
-            // stored config still carries any old cycle key and no `memory`, fold
-            // the first present value into `memory` (preserving the user's
-            // choice), then the old keys drop via the allow-list above. This is a
-            // schema migration, NOT a runtime fallback — the persisted config is
-            // cleaned once so runtime never has to re-migrate.
-            const legacyCycleKeys = ['cycle1', 'cycle2', 'cycle3'];
-            let migratedMaintenance = false;
-            if (!('memory' in rawMaint) && legacyCycleKeys.some(k => k in (raw.maintenance || {}))) {
-                rawMaint.memory = raw.maintenance.cycle1 ?? raw.maintenance.cycle2 ?? raw.maintenance.cycle3 ?? DEFAULT_MAINTENANCE.memory;
-                migratedMaintenance = true;
-            }
+
             // Self-ref guard: mcpServers.mixdog / mcpServers["trib-plugin"]
             // would self-spawn through the in-process tool adapter. Strip on
             // ingress so user-edited configs cannot brick the agent boot.
@@ -400,72 +359,8 @@ export function loadConfig(options = {}) {
                     process.stderr.write(`[config] persist sanitized config failed: ${err?.message}\n`);
                 }
             }
-            // Persist the memory-cycle schema migration once. rawMaint already
-            // carries the folded `memory` key and excludes the dropped cycle1/2/3
-            // keys (not in the allow-list); rebase onto the in-lock current so a
-            // concurrent writer's unrelated edits survive, mirroring the
-            // mcpServers self-ref strip above.
-            if (migratedMaintenance) {
-                try {
-                    persistAgentConfig((current) => {
-                        const cur = { ...current };
-                        const target = (cur.agent && cur.agent.providers)
-                            ? (cur.agent = { ...cur.agent })
-                            : cur;
-                        const curMaint = (target.maintenance && typeof target.maintenance === 'object') ? { ...target.maintenance } : {};
-                        // Derive `memory` from the IN-LOCK current, not the
-                        // pre-lock rawMaint snapshot — a concurrent writer may
-                        // have set maintenance.memory or changed a legacy cycle
-                        // value between this loadConfig()'s read and the lock.
-                        // If `memory` is already present in-lock, preserve it
-                        // (lost-update guard); otherwise fold the in-lock legacy
-                        // cycle value first, with the pre-lock snapshot as the
-                        // last-resort seed.
-                        if (!('memory' in curMaint)) {
-                            curMaint.memory = curMaint.cycle1 ?? curMaint.cycle2 ?? curMaint.cycle3 ?? rawMaint.memory;
-                        }
-                        for (const k of legacyCycleKeys) delete curMaint[k];
-                        target.maintenance = curMaint;
-                        return cur;
-                    });
-                } catch (err) {
-                    process.stderr.write(`[config] persist maintenance migration failed: ${err?.message}\n`);
-                }
-            }
-            // One-time recap migration: fold a legacy `modules.memory === false`
-            // flag into `recap.enabled=false` and drop the legacy flag on disk.
-            // Compute the recap shape here (before the return) so the
-            // _migratedRecapLegacy latch is set, then persist under the lock.
-            const recapConfig = normalizeRecapConfig(raw.recap, raw.modules);
-            if (_migratedRecapLegacy) {
-                _migratedRecapLegacy = false;
-                try {
-                    persistAgentConfig((current) => {
-                        const cur = { ...current };
-                        const target = (cur.agent && cur.agent.providers)
-                            ? (cur.agent = { ...cur.agent })
-                            : cur;
-                        const recap = (target.recap && typeof target.recap === 'object') ? { ...target.recap } : {};
-                        if (!('enabled' in recap)) recap.enabled = false;
-                        target.recap = recap;
-                        if (target.modules && typeof target.modules === 'object' && target.modules.memory === false) {
-                            const modules = { ...target.modules };
-                            delete modules.memory;
-                            target.modules = modules;
-                        } else if (target.modules && typeof target.modules === 'object' && target.modules.memory) {
-                            const modules = { ...target.modules };
-                            const memoryMod = { ...modules.memory };
-                            delete memoryMod.enabled;
-                            if (Object.keys(memoryMod).length === 0) delete modules.memory;
-                            else modules.memory = memoryMod;
-                            target.modules = modules;
-                        }
-                        return cur;
-                    });
-                } catch (err) {
-                    process.stderr.write(`[config] persist recap migration failed: ${err?.message}\n`);
-                }
-            }
+            const recapConfig = normalizeRecapConfig(raw.recap);
+
             const rawPresets = Array.isArray(raw.presets) ? raw.presets : [];
             const normalizedPresets = rawPresets
                 .map(p => normalizePreset(p))
@@ -473,9 +368,9 @@ export function loadConfig(options = {}) {
                 .filter(p => p.id !== 'workflow-search');
             const workflowRoutes = raw.workflowRoutes && typeof raw.workflowRoutes === 'object' ? { ...raw.workflowRoutes } : {};
             delete workflowRoutes.search;
-            // Migrate legacy preset-name maintenance slots to direct routes,
-            // then overlay onto the route-shaped defaults.
-            const migratedMaint = migrateMaintenanceRoutes(rawMaint, normalizedPresets, raw.defaultProvider);
+            // Normalize maintenance slots to routes, then overlay onto the
+            // route-shaped defaults.
+            const normalizedMaint = normalizeMaintenanceRoutes(rawMaint, raw.defaultProvider);
             return {
                 providers: mergedProviders,
                 mcpServers,
@@ -484,7 +379,7 @@ export function loadConfig(options = {}) {
                     : {},
                 presets: normalizedPresets,
                 default: raw.default || null,
-                maintenance: { ...DEFAULT_MAINTENANCE, ...migratedMaint },
+                maintenance: { ...DEFAULT_MAINTENANCE, ...normalizedMaint },
                 workflowRoutes,
                 searchRoute: normalizeSearchRoute(raw.searchRoute),
                 fastModels: raw.fastModels && typeof raw.fastModels === 'object' ? raw.fastModels : {},

package/src/runtime/agent/orchestrator/context/collect.mjs

@@ -134,7 +134,7 @@ function normalizeSkillNameKey(name) {
     return String(name || '').trim().toLowerCase();
 }
 
-export function getDisabledSkillNameSet(config = null) {
+function getDisabledSkillNameSet(config = null) {
     const cfg = config || loadConfig({ secrets: false });
     const keys = normalizeSkillsConfig(cfg.skills).disabled
         .map((n) => normalizeSkillNameKey(n))
@@ -203,7 +203,7 @@ export function collectSkillsCached(cwd) {
 /**
  * Load full skill content by name.
  */
-export function loadSkillContent(name, cwd) {
+function loadSkillContent(name, cwd) {
     const skills = collectSkillsCached(cwd);
     const skill = skills.find(s => s.name === name);
     if (!skill)
@@ -250,7 +250,7 @@ export function buildSkillResultEnvelope(name, content, skillDir) {
  * body is delivered separately as ONE injected user message (newMessages),
  * never in the tool_result — so the body appears exactly once.
  */
-export function buildSkillStub(name) {
+function buildSkillStub(name) {
     return `Loaded skill: ${String(name || '').trim()}`;
 }
 
@@ -320,7 +320,7 @@ function sanitizeDeferredToolManifestName(name) {
     return text;
 }
 
-export function bp1HasDeferredToolManifestBlock(text) {
+function bp1HasDeferredToolManifestBlock(text) {
     const raw = String(text || '');
     return /<available-deferred-tools>[\s\S]*?<\/available-deferred-tools>/i.test(raw)
         || /<mcp-instructions>[\s\S]*?<\/mcp-instructions>/i.test(raw);
@@ -375,7 +375,7 @@ function sanitizeMcpInstructionText(text, max = MCP_INSTRUCTION_MAX_CHARS) {
  * Per-server MCP initialize instructions for deferred-pool tools only.
  * Empty when no instructions or no matching deferred MCP tools → omit block.
  */
-export function buildMcpInstructionsManifest(mcpServerInstructions, poolNames) {
+function buildMcpInstructionsManifest(mcpServerInstructions, poolNames) {
     const map = mcpServerInstructions && typeof mcpServerInstructions === 'object'
         ? mcpServerInstructions
         : {};
@@ -421,7 +421,7 @@ export function stripDeferredToolManifestBlock(text) {
 // manifest, agent rules, …) is reordered or dropped. The fresh manifest already
 // carries the mcp-instructions companion, so any pre-existing standalone one is
 // removed first to avoid duplication.
-export function rebuildDeferredToolManifestBlock(text, manifest) {
+function rebuildDeferredToolManifestBlock(text, manifest) {
     let out = String(text || '').replace(MCP_INSTRUCTIONS_BLOCK_RE, '');
     let replaced = false;
     out = out.replace(DEFERRED_TOOLS_BLOCK_RE, (match, sep) => {

package/src/runtime/agent/orchestrator/dispatch-persist.mjs

@@ -61,7 +61,7 @@ function setTail(dataDir, p) {
 // the drain window.  The window is bounded (process is exiting) and eliminating
 // it requires a fundamentally different design (e.g. a dedicated lock-owner
 // process).  Best-effort is the correct trade-off here.
-export function drainDispatchPersist() {
+function drainDispatchPersist() {
   // Prefer _pendingPayload (desired state captured at writeAll entry) over
   // _lastPayload (last successfully written state).  Pending is strictly
   // newer when a writeAll is still in-flight or queued at process exit.
@@ -296,7 +296,7 @@ export function hasPending(dataDir) {
   }
 }
 
-export function removePending(dataDir, handle) {
+function removePending(dataDir, handle) {
   if (!dataDir || !handle) return;
   const tail = getTail(dataDir).then(async () => {
     try {

package/src/runtime/agent/orchestrator/internal-agents.mjs

@@ -5,7 +5,7 @@
  * invoked only by internal handlers (explore / recall / search) and carry
  * their own system prompt + tool-set policy.
  *
- * Lookup order (agent-dispatch.resolvePresetName):
+ * Lookup order (agent-dispatch.resolveMaintenanceRoute):
  *   1. explicit preset arg
  *   2. opts.preset
  *   3. hidden-agent registry (defaults/agents.json + systemFile frontmatter)
@@ -15,7 +15,7 @@
  *
  * The preset names refer to entries seeded in mixdog-config.json (agent.presets)
  * via DEFAULT_PRESETS (see config.mjs). If the user deletes the referenced preset
- * from their config the hidden agents degrade gracefully — `resolvePresetName`
+ * from their config the hidden agents degrade gracefully — `resolveMaintenanceRoute`
  * returns a name, but session creation will fail with a clear "preset not
  * found" error rather than silently mis-dispatching.
  *

package/src/runtime/agent/orchestrator/mcp/child-tree.mjs

@@ -82,7 +82,7 @@ function descendantsOf(childrenOf, rootPid) {
 }
 
 // Exposed for tests: descendant pid list captured while the tree is intact.
-export async function collectDescendants(rootPid) {
+async function collectDescendants(rootPid) {
     const { childrenOf } = await enumerate();
     return descendantsOf(childrenOf, rootPid);
 }

package/src/runtime/agent/orchestrator/mcp/client.mjs

@@ -20,7 +20,7 @@ const AUTO_DETECT_PORTS = {
 };
 const DEFAULT_MCP_CALL_TIMEOUT_MS = 120000;
 // Per-server STARTUP handshake budget (connect + listTools). Codex parity: 10s.
-export const DEFAULT_MCP_STARTUP_TIMEOUT_MS = 10000;
+const DEFAULT_MCP_STARTUP_TIMEOUT_MS = 10000;
 // --- State ---
 const servers = new Map();
 let mcpSdkPromise = null;
@@ -60,7 +60,7 @@ async function loadMcpSdk() {
  * provided env map (defaults to process.env). Recurses into arrays/objects.
  * Unknown vars expand to an empty string. No shell execution.
  */
-export function expandEnvVars(value, env = process.env) {
+function expandEnvVars(value, env = process.env) {
     if (typeof value === 'string') {
         return value.replace(/\$\{(?:env:)?([A-Za-z_][A-Za-z0-9_]*)\}/g, (_m, name) => {
             const v = env?.[name];
@@ -249,7 +249,7 @@ export function normalizeMcpToolResult(result) {
 // per-server timeoutMs/callTimeoutMs config value (0/off/none/false disables).
 // On expiry we close the transport so the next dispatch reconnects fresh, but
 // we do not retry the timed-out call automatically (avoids side-effect dupes).
-export function resolveMcpCallTimeoutMs(cfg = {}, env = process.env) {
+function resolveMcpCallTimeoutMs(cfg = {}, env = process.env) {
     const raw = cfg?.timeoutMs ?? cfg?.timeout_ms ?? cfg?.callTimeoutMs ?? cfg?.call_timeout_ms
         ?? env?.MIXDOG_MCP_CALL_TIMEOUT_MS;
     if (raw == null || raw === '' || raw === false) return DEFAULT_MCP_CALL_TIMEOUT_MS;
@@ -259,7 +259,7 @@ export function resolveMcpCallTimeoutMs(cfg = {}, env = process.env) {
     return Math.round(parsed);
 }
 
-export function isMcpToolCallTimeoutError(err) {
+function isMcpToolCallTimeoutError(err) {
     return err?.code === 'EMCPTOOLTIMEOUT';
 }
 

package/src/runtime/agent/orchestrator/providers/admission-scheduler.mjs

@@ -62,7 +62,7 @@ function retryAfterMs(error, now) {
     return Number.isFinite(date) ? Math.max(0, date - now) : 0;
 }
 
-export class ProviderAdmissionQueueOverflowError extends Error {
+class ProviderAdmissionQueueOverflowError extends Error {
     constructor(key, maxQueue) {
         super(`provider admission queue full for ${key} (maximum ${maxQueue})`);
         this.name = 'ProviderAdmissionQueueOverflowError';
@@ -78,7 +78,7 @@ export class ProviderAdmissionQueueOverflowError extends Error {
  * silently parking the request until the quota window resets. httpStatus 429
  * routes it through the existing quota/rate-limit error presentation.
  */
-export class ProviderCooldownError extends Error {
+class ProviderCooldownError extends Error {
     constructor(key, cooldownUntil, now) {
         const remaining = Math.max(0, Number(cooldownUntil) - Number(now));
         super(

package/src/runtime/agent/orchestrator/providers/anthropic-betas.mjs

@@ -1,9 +1,9 @@
 import { EFFORT_BETA_HEADER } from './anthropic-effort.mjs';
 
-export const EXTENDED_CACHE_TTL_BETA_HEADER = 'extended-cache-ttl-2025-04-11';
-export const INTERLEAVED_THINKING_BETA_HEADER = 'interleaved-thinking-2025-05-14';
-export const FAST_MODE_BETA_HEADER = 'fast-mode-2026-02-01';
-export const TOOL_SEARCH_BETA_HEADER = 'advanced-tool-use-2025-11-20';
+const EXTENDED_CACHE_TTL_BETA_HEADER = 'extended-cache-ttl-2025-04-11';
+const INTERLEAVED_THINKING_BETA_HEADER = 'interleaved-thinking-2025-05-14';
+const FAST_MODE_BETA_HEADER = 'fast-mode-2026-02-01';
+const TOOL_SEARCH_BETA_HEADER = 'advanced-tool-use-2025-11-20';
 export { EFFORT_BETA_HEADER };
 
 export function supportsAnthropicFastMode(model) {

package/src/runtime/agent/orchestrator/providers/anthropic-effort.mjs

@@ -2,7 +2,7 @@
  * Anthropic "effort" (extended-thinking budget) request parameter handling.
  */
 
-export const EFFORT_LEVELS = Object.freeze(['low', 'medium', 'high', 'xhigh', 'max']);
+const EFFORT_LEVELS = Object.freeze(['low', 'medium', 'high', 'xhigh', 'max']);
 
 // Canonical display/transport ordering for effort levels. Used only to sort
 // whatever set the catalog actually advertises — NOT as an allowlist. The
@@ -157,7 +157,7 @@ export function normalizeAnthropicEffortInput(raw, model, logTag = 'anthropic')
     return undefined;
 }
 
-export function anthropicEffortUsesOutputConfig(model, opts = {}) {
+function anthropicEffortUsesOutputConfig(model, opts = {}) {
     const thinkingBudgetTokens = Number(opts.thinkingBudgetTokens);
     if (Number.isFinite(thinkingBudgetTokens) && thinkingBudgetTokens > 0) return false;
     return modelSupportsEffort(model);