@hassangameryt/freeflow 0.3.0 → 0.4.0

package/router/dist/config/config.js

@@ -1,453 +1,632 @@
 import { isValidPostToolRoutingMode, validatePositiveIntegerThreshold } from "./router-contract.js";
-import { OBSERVED_ROUTING_FAILURE_MODES, OBSERVED_ROUTING_PERSISTENCE_MODES, OUTPUT_ROUTER_PROFILES, RESERVED_OBSERVED_ROUTING_PERSISTENCE_MODES, STORAGE_POLICY_MODES, } from "./types.js";
+import {
+  OBSERVED_ROUTING_FAILURE_MODES,
+  OBSERVED_ROUTING_PERSISTENCE_MODES,
+  OUTPUT_ROUTER_PROFILES,
+  RESERVED_OBSERVED_ROUTING_PERSISTENCE_MODES,
+  STORAGE_POLICY_MODES,
+} from "./types.js";
 export const OUTPUT_ROUTER_SKILL_PATH = "skills/output-router/SKILL.md";
 export const DEFAULT_VAULT_ROOT = "~/.cache/freeflow-router/vault";
 export const DEFAULT_POST_TOOL_ROUTING = "off";
-export const DEFAULT_OUTPUT_ROUTER_ENABLED = true;
+export const DEFAULT_OUTPUT_ROUTER_ENABLED = false;
 export const DEFAULT_OUTPUT_ROUTER_PROFILE = "standard";
 export const DEFAULT_STORAGE_POLICY = "hybrid-dedupe";
 export const DEFAULT_VAULT_RETENTION = {
-    strategy: "ttl",
-    ttlDays: 7,
+  strategy: "ttl",
+  ttlDays: 7,
 };
 export const DEFAULT_ROUTER_THRESHOLDS = {
-    largeOutputBytes: 64_000,
-    largeOutputLines: 1_000,
+  largeOutputBytes: 64_000,
+  largeOutputLines: 1_000,
 };
 export const DEFAULT_OBSERVED_ROUTING_PERSISTENCE = "none";
 export const SAFE_OBSERVED_ROUTING_PERSISTENCE_FALLBACK = "metadata-only";
 export const DEFAULT_OBSERVED_ROUTING_CONFIG = {
-    enabled: false,
-    onRoutingFailure: "fail-open",
-    mcp: { servers: {} },
-    web: { enabled: false, persistence: DEFAULT_OBSERVED_ROUTING_PERSISTENCE },
-    fetch: { enabled: false, persistence: DEFAULT_OBSERVED_ROUTING_PERSISTENCE },
-    codeSearch: { enabled: false, persistence: DEFAULT_OBSERVED_ROUTING_PERSISTENCE },
+  enabled: false,
+  onRoutingFailure: "fail-open",
+  mcp: { servers: {} },
+  web: { enabled: false, persistence: DEFAULT_OBSERVED_ROUTING_PERSISTENCE },
+  fetch: { enabled: false, persistence: DEFAULT_OBSERVED_ROUTING_PERSISTENCE },
+  codeSearch: { enabled: false, persistence: DEFAULT_OBSERVED_ROUTING_PERSISTENCE },
 };
 export const SCRIPT_TRANSFORM_LANGUAGES = ["javascript", "python", "jq"];
 export const DEFAULT_SCRIPT_TRANSFORM_LIMITS = {
-    timeoutMs: 5_000,
-    maxInputBytes: 1_048_576,
-    maxOutputBytes: 65_536,
+  timeoutMs: 5_000,
+  maxInputBytes: 1_048_576,
+  maxOutputBytes: 65_536,
 };
 export const MAX_SCRIPT_TRANSFORM_LIMITS = {
-    timeoutMs: 30_000,
-    maxInputBytes: 10_485_760,
-    maxOutputBytes: 1_048_576,
+  timeoutMs: 30_000,
+  maxInputBytes: 10_485_760,
+  maxOutputBytes: 1_048_576,
 };
 export const DEFAULT_SCRIPT_TRANSFORM_CONFIG = {
-    enabled: false,
-    sandbox: "auto",
-    languages: [...SCRIPT_TRANSFORM_LANGUAGES],
-    network: "off",
-    limits: { ...DEFAULT_SCRIPT_TRANSFORM_LIMITS },
-    rawScriptPersistence: "disabled",
+  enabled: false,
+  sandbox: "auto",
+  languages: [...SCRIPT_TRANSFORM_LANGUAGES],
+  network: "off",
+  limits: { ...DEFAULT_SCRIPT_TRANSFORM_LIMITS },
+  rawScriptPersistence: "disabled",
 };
 export const DEFAULT_LOCAL_FREEFLOW_CONFIG = {
-    processing: {
-        unsafeUnsandboxed: {
-            enabled: false,
-        },
+  processing: {
+    unsafeUnsandboxed: {
+      enabled: false,
     },
+  },
 };
 export function createDefaultRouterConfig(options = {}) {
-    return {
-        enabled: DEFAULT_OUTPUT_ROUTER_ENABLED,
-        profile: DEFAULT_OUTPUT_ROUTER_PROFILE,
-        postToolRouting: DEFAULT_POST_TOOL_ROUTING,
-        storagePolicy: DEFAULT_STORAGE_POLICY,
-        thresholds: { ...DEFAULT_ROUTER_THRESHOLDS },
-        vault: {
-            root: options.vaultRoot ?? DEFAULT_VAULT_ROOT,
-            retention: options.vaultRetention ?? DEFAULT_VAULT_RETENTION,
-        },
-    };
+  return {
+    enabled: DEFAULT_OUTPUT_ROUTER_ENABLED,
+    profile: DEFAULT_OUTPUT_ROUTER_PROFILE,
+    postToolRouting: DEFAULT_POST_TOOL_ROUTING,
+    storagePolicy: DEFAULT_STORAGE_POLICY,
+    thresholds: { ...DEFAULT_ROUTER_THRESHOLDS },
+    vault: {
+      root: options.vaultRoot ?? DEFAULT_VAULT_ROOT,
+      retention: options.vaultRetention ?? DEFAULT_VAULT_RETENTION,
+    },
+  };
 }
 export function normalizeRouterConfig(input) {
-    const config = createDefaultRouterConfig();
-    const warnings = [];
-    if (input === undefined || input === null) {
-        return { config, warnings };
-    }
-    if (!isRecord(input)) {
-        return {
-            config,
-            warnings: ["outputRouter config must be an object; using built-in output router defaults."],
-        };
-    }
-    applyRouterEnabled(config, warnings, input.enabled);
-    applyRouterProfile(config, warnings, input.profile);
-    applyPostToolRouting(config, warnings, input.postToolRouting);
-    applyStringEnum(config, warnings, input.storagePolicy, "storagePolicy", "outputRouter.storagePolicy", STORAGE_POLICY_MODES, DEFAULT_STORAGE_POLICY);
-    applyPositiveInteger(config.thresholds, warnings, input.largeOutputBytes, "largeOutputBytes");
-    applyPositiveInteger(config.thresholds, warnings, input.largeOutputLines, "largeOutputLines");
-    applyVaultRoot(config, warnings, input.vaultRoot);
-    applyVaultRetention(config, warnings, input.vaultRetentionDays);
-    applyHints(config, warnings, input.generatedPaths, input.noisyCommandHints);
+  const config = createDefaultRouterConfig();
+  const warnings = [];
+  if (input === undefined || input === null) {
     return { config, warnings };
+  }
+  if (!isRecord(input)) {
+    return {
+      config,
+      warnings: ["outputRouter config must be an object; using built-in output router defaults."],
+    };
+  }
+  applyRouterEnabled(config, warnings, input.enabled);
+  applyRouterProfile(config, warnings, input.profile);
+  applyPostToolRouting(config, warnings, input.postToolRouting);
+  applyStringEnum(
+    config,
+    warnings,
+    input.storagePolicy,
+    "storagePolicy",
+    "outputRouter.storagePolicy",
+    STORAGE_POLICY_MODES,
+    DEFAULT_STORAGE_POLICY,
+  );
+  applyRouterThresholds(config, warnings, input);
+  applyRouterVault(config, warnings, input);
+  applyRouterHints(config, warnings, input);
+  return { config, warnings };
 }
 export function normalizeObservedRoutingConfig(input) {
-    const config = createDefaultObservedRoutingConfig();
-    const warnings = [];
-    if (input === undefined || input === null) {
-        return { config, warnings };
-    }
-    if (!isRecord(input)) {
-        return {
-            config,
-            warnings: ["observedRouting config must be an object; observed routing is off by default."],
-        };
-    }
-    applyObservedRoutingEnabled(config, warnings, input.enabled);
-    applyStringEnum(config, warnings, input.onRoutingFailure, "onRoutingFailure", "observedRouting.onRoutingFailure", OBSERVED_ROUTING_FAILURE_MODES, DEFAULT_OBSERVED_ROUTING_CONFIG.onRoutingFailure);
-    applyObservedMcpConfig(config, warnings, input.mcp);
-    config.web = parseObservedProducerConfig(input.web, "observedRouting.web", warnings);
-    config.fetch = parseObservedProducerConfig(input.fetch, "observedRouting.fetch", warnings);
-    config.codeSearch = parseObservedProducerConfig(input.codeSearch, "observedRouting.codeSearch", warnings);
+  const config = createDefaultObservedRoutingConfig();
+  const warnings = [];
+  if (input === undefined || input === null) {
     return { config, warnings };
+  }
+  if (!isRecord(input)) {
+    return {
+      config,
+      warnings: ["observedRouting config must be an object; observed routing is off by default."],
+    };
+  }
+  applyObservedRoutingEnabled(config, warnings, input.enabled);
+  applyStringEnum(
+    config,
+    warnings,
+    input.onRoutingFailure,
+    "onRoutingFailure",
+    "observedRouting.onRoutingFailure",
+    OBSERVED_ROUTING_FAILURE_MODES,
+    DEFAULT_OBSERVED_ROUTING_CONFIG.onRoutingFailure,
+  );
+  applyObservedMcpConfig(config, warnings, input.mcp);
+  config.web = parseObservedProducerConfig(input.web, "observedRouting.web", warnings);
+  config.fetch = parseObservedProducerConfig(input.fetch, "observedRouting.fetch", warnings);
+  config.codeSearch = parseObservedProducerConfig(input.codeSearch, "observedRouting.codeSearch", warnings);
+  return { config, warnings };
 }
 export function normalizeScriptTransformConfig(input) {
-    const config = {
-        ...DEFAULT_SCRIPT_TRANSFORM_CONFIG,
-        languages: [...DEFAULT_SCRIPT_TRANSFORM_CONFIG.languages],
-        limits: { ...DEFAULT_SCRIPT_TRANSFORM_CONFIG.limits },
-    };
-    const warnings = [];
-    if (input === undefined || input === null) {
-        return { config, warnings };
-    }
-    if (!isRecord(input)) {
-        return {
-            config,
-            warnings: ["scriptTransform config must be an object; script transform is disabled by default."],
-        };
-    }
-    if (input.enabled !== undefined) {
-        if (typeof input.enabled === "boolean") {
-            config.enabled = input.enabled;
-        }
-        else {
-            warnings.push(`Invalid scriptTransform.enabled=${JSON.stringify(input.enabled)}; using false.`);
-        }
-    }
-    applyStringEnum(config, warnings, input.sandbox, "sandbox", "scriptTransform.sandbox", ["auto"], DEFAULT_SCRIPT_TRANSFORM_CONFIG.sandbox);
-    applyStringEnum(config, warnings, input.network, "network", "scriptTransform.network", ["off"], DEFAULT_SCRIPT_TRANSFORM_CONFIG.network);
-    applyStringEnum(config, warnings, input.rawScriptPersistence, "rawScriptPersistence", "scriptTransform.rawScriptPersistence", ["disabled"], DEFAULT_SCRIPT_TRANSFORM_CONFIG.rawScriptPersistence);
-    applyScriptTransformLanguages(config, warnings, input.languages);
-    applyScriptTransformLimits(config, warnings, input.limits);
+  const config = {
+    ...DEFAULT_SCRIPT_TRANSFORM_CONFIG,
+    languages: [...DEFAULT_SCRIPT_TRANSFORM_CONFIG.languages],
+    limits: { ...DEFAULT_SCRIPT_TRANSFORM_CONFIG.limits },
+  };
+  const warnings = [];
+  if (input === undefined || input === null) {
     return { config, warnings };
+  }
+  if (!isRecord(input)) {
+    return {
+      config,
+      warnings: ["scriptTransform config must be an object; script transform is disabled by default."],
+    };
+  }
+  if (input.enabled !== undefined) {
+    if (typeof input.enabled === "boolean") {
+      config.enabled = input.enabled;
+    } else {
+      warnings.push(`Invalid scriptTransform.enabled=${JSON.stringify(input.enabled)}; using false.`);
+    }
+  }
+  applyStringEnum(
+    config,
+    warnings,
+    input.sandbox,
+    "sandbox",
+    "scriptTransform.sandbox",
+    ["auto"],
+    DEFAULT_SCRIPT_TRANSFORM_CONFIG.sandbox,
+  );
+  applyStringEnum(
+    config,
+    warnings,
+    input.network,
+    "network",
+    "scriptTransform.network",
+    ["off"],
+    DEFAULT_SCRIPT_TRANSFORM_CONFIG.network,
+  );
+  applyStringEnum(
+    config,
+    warnings,
+    input.rawScriptPersistence,
+    "rawScriptPersistence",
+    "scriptTransform.rawScriptPersistence",
+    ["disabled"],
+    DEFAULT_SCRIPT_TRANSFORM_CONFIG.rawScriptPersistence,
+  );
+  applyScriptTransformLanguages(config, warnings, input.languages);
+  applyScriptTransformLimits(config, warnings, input.limits);
+  return { config, warnings };
 }
 export function normalizeLocalFreeflowConfig(input) {
-    const config = {
-        processing: {
-            unsafeUnsandboxed: { ...DEFAULT_LOCAL_FREEFLOW_CONFIG.processing.unsafeUnsandboxed },
-        },
+  const config = {
+    processing: {
+      unsafeUnsandboxed: { ...DEFAULT_LOCAL_FREEFLOW_CONFIG.processing.unsafeUnsandboxed },
+    },
+  };
+  const warnings = [];
+  if (input === undefined || input === null) {
+    return { config, warnings };
+  }
+  if (!isRecord(input)) {
+    return {
+      config,
+      warnings: [".freeflow/local.json must be an object; local unsafe processing opt-ins are disabled."],
     };
-    const warnings = [];
-    if (input === undefined || input === null) {
-        return { config, warnings };
-    }
-    if (!isRecord(input)) {
-        return {
-            config,
-            warnings: [".freeflow/local.json must be an object; local unsafe processing opt-ins are disabled."],
-        };
-    }
-    if (input.processing === undefined) {
-        return { config, warnings };
-    }
-    if (!isRecord(input.processing)) {
-        warnings.push("Invalid local processing config; expected an object. unsafeUnsandboxed remains disabled.");
-        return { config, warnings };
-    }
-    const unsafe = input.processing.unsafeUnsandboxed;
-    if (unsafe === undefined) {
-        return { config, warnings };
-    }
-    if (!isRecord(unsafe)) {
-        warnings.push("Invalid local processing.unsafeUnsandboxed config; expected an object with enabled boolean. unsafeUnsandboxed remains disabled.");
-        return { config, warnings };
-    }
-    if (typeof unsafe.enabled === "boolean") {
-        config.processing.unsafeUnsandboxed.enabled = unsafe.enabled;
-    }
-    else if (unsafe.enabled !== undefined) {
-        warnings.push(`Invalid local processing.unsafeUnsandboxed.enabled=${JSON.stringify(unsafe.enabled)}; using false.`);
-    }
+  }
+  if (input.processing === undefined) {
+    return { config, warnings };
+  }
+  if (!isRecord(input.processing)) {
+    warnings.push("Invalid local processing config; expected an object. unsafeUnsandboxed remains disabled.");
     return { config, warnings };
+  }
+  const unsafe = input.processing.unsafeUnsandboxed;
+  if (unsafe === undefined) {
+    return { config, warnings };
+  }
+  if (!isRecord(unsafe)) {
+    warnings.push(
+      "Invalid local processing.unsafeUnsandboxed config; expected an object with enabled boolean. unsafeUnsandboxed remains disabled.",
+    );
+    return { config, warnings };
+  }
+  if (typeof unsafe.enabled === "boolean") {
+    config.processing.unsafeUnsandboxed.enabled = unsafe.enabled;
+  } else if (unsafe.enabled !== undefined) {
+    warnings.push(`Invalid local processing.unsafeUnsandboxed.enabled=${JSON.stringify(unsafe.enabled)}; using false.`);
+  }
+  return { config, warnings };
 }
 export function normalizeFreeflowConfig(input) {
-    const warnings = [];
-    if (input !== undefined && input !== null && !isRecord(input)) {
-        const router = normalizeRouterConfig(undefined);
-        const observedRouting = normalizeObservedRoutingConfig(undefined);
-        const scriptTransform = normalizeScriptTransformConfig(undefined);
-        return {
-            config: {
-                outputRouter: router.config,
-                observedRouting: observedRouting.config,
-                scriptTransform: scriptTransform.config,
-            },
-            warnings: ["Freeflow config must be an object; using built-in defaults."],
-        };
-    }
-    const source = isRecord(input) ? input : {};
-    if (source.processing !== undefined) {
-        warnings.push(".freeflow/config.json processing config is ignored; unsafe unsandboxed processing must be enabled in local-only .freeflow/local.json.");
-    }
-    const router = normalizeRouterConfig(source.outputRouter);
-    const observedRouting = normalizeObservedRoutingConfig(source.observedRouting);
-    const scriptTransform = normalizeScriptTransformConfig(source.scriptTransform);
-    warnings.push(...router.warnings, ...observedRouting.warnings, ...scriptTransform.warnings);
+  const warnings = [];
+  if (input !== undefined && input !== null && !isRecord(input)) {
+    const router = normalizeRouterConfig(undefined);
+    const observedRouting = normalizeObservedRoutingConfig(undefined);
+    const scriptTransform = normalizeScriptTransformConfig(undefined);
     return {
-        config: {
-            outputRouter: router.config,
-            observedRouting: observedRouting.config,
-            scriptTransform: scriptTransform.config,
-        },
-        warnings,
+      config: {
+        outputRouter: router.config,
+        observedRouting: observedRouting.config,
+        scriptTransform: scriptTransform.config,
+      },
+      warnings: ["Freeflow config must be an object; using built-in defaults."],
     };
+  }
+  const source = isRecord(input) ? input : {};
+  if (source.processing !== undefined) {
+    warnings.push(
+      ".freeflow/config.json processing config is ignored; unsafe unsandboxed processing must be enabled in local-only .freeflow/local.json.",
+    );
+  }
+  const routerSource = source.outputRouter;
+  const routerSourceRecord = isRecord(routerSource) ? routerSource : {};
+  const router = normalizeRouterConfig(routerSource);
+  const observedRoutingSource =
+    routerSourceRecord.observedRouting !== undefined ? routerSourceRecord.observedRouting : source.observedRouting;
+  const observedRouting = normalizeObservedRoutingConfig(observedRoutingSource);
+  const observedRoutingWarnings =
+    routerSourceRecord.observedRouting !== undefined
+      ? prefixConfigWarnings(observedRouting.warnings, "observedRouting", "outputRouter.observedRouting")
+      : observedRouting.warnings;
+  const scriptTransformSource =
+    routerSourceRecord.scriptTransform !== undefined ? routerSourceRecord.scriptTransform : source.scriptTransform;
+  const scriptTransform = normalizeScriptTransformConfig(scriptTransformSource);
+  const scriptTransformWarnings =
+    routerSourceRecord.scriptTransform !== undefined
+      ? prefixConfigWarnings(scriptTransform.warnings, "scriptTransform", "outputRouter.scriptTransform")
+      : scriptTransform.warnings;
+  const effectiveObservedRouting = router.config.enabled
+    ? observedRouting.config
+    : { ...observedRouting.config, enabled: false };
+  const effectiveScriptTransform = router.config.enabled
+    ? scriptTransform.config
+    : { ...scriptTransform.config, enabled: false };
+  warnings.push(...router.warnings, ...observedRoutingWarnings, ...scriptTransformWarnings);
+  return {
+    config: {
+      outputRouter: router.config,
+      observedRouting: effectiveObservedRouting,
+      scriptTransform: effectiveScriptTransform,
+    },
+    warnings,
+  };
 }
 function isRecord(value) {
-    return typeof value === "object" && value !== null && !Array.isArray(value);
+  return typeof value === "object" && value !== null && !Array.isArray(value);
+}
+function prefixConfigWarnings(warnings, from, to) {
+  return warnings.map((warning) => warning.replaceAll(from, to));
 }
 function applyRouterEnabled(config, warnings, value) {
-    if (value === undefined) {
-        return;
-    }
-    if (typeof value === "boolean") {
-        config.enabled = value;
-        return;
-    }
-    warnings.push(`Invalid outputRouter.enabled=${JSON.stringify(value)}; using ${DEFAULT_OUTPUT_ROUTER_ENABLED}.`);
+  if (value === undefined) {
+    return;
+  }
+  if (typeof value === "boolean") {
+    config.enabled = value;
+    return;
+  }
+  warnings.push(`Invalid outputRouter.enabled=${JSON.stringify(value)}; using ${DEFAULT_OUTPUT_ROUTER_ENABLED}.`);
 }
 function applyRouterProfile(config, warnings, value) {
-    if (value === undefined) {
-        return;
-    }
-    if (isStringIn(value, OUTPUT_ROUTER_PROFILES)) {
-        config.profile = value;
-        return;
-    }
-    warnings.push(`Invalid outputRouter.profile=${JSON.stringify(value)}; using ${DEFAULT_OUTPUT_ROUTER_PROFILE}.`);
+  if (value === undefined) {
+    return;
+  }
+  if (isStringIn(value, OUTPUT_ROUTER_PROFILES)) {
+    config.profile = value;
+    return;
+  }
+  warnings.push(`Invalid outputRouter.profile=${JSON.stringify(value)}; using ${DEFAULT_OUTPUT_ROUTER_PROFILE}.`);
 }
 function applyPostToolRouting(config, warnings, value) {
-    if (value === undefined) {
-        return;
-    }
-    if (isValidPostToolRoutingMode(value)) {
-        config.postToolRouting = value;
-        return;
-    }
-    warnings.push(`Invalid outputRouter.postToolRouting=${JSON.stringify(value)}; using ${DEFAULT_POST_TOOL_ROUTING}.`);
+  if (value === undefined) {
+    return;
+  }
+  if (isValidPostToolRoutingMode(value)) {
+    config.postToolRouting = value;
+    return;
+  }
+  warnings.push(`Invalid outputRouter.postToolRouting=${JSON.stringify(value)}; using ${DEFAULT_POST_TOOL_ROUTING}.`);
 }
-function applyPositiveInteger(thresholds, warnings, value, key) {
-    if (value === undefined) {
-        return;
-    }
-    if (validatePositiveIntegerThreshold(value, `outputRouter.${key}`).length === 0) {
-        thresholds[key] = value;
-        return;
-    }
-    warnings.push(`Invalid outputRouter.${key}=${JSON.stringify(value)}; using ${DEFAULT_ROUTER_THRESHOLDS[key]}.`);
+function applyRouterThresholds(config, warnings, input) {
+  const thresholds = input.thresholds;
+  const thresholdRecord = isRecord(thresholds) ? thresholds : {};
+  if (thresholds !== undefined && !isRecord(thresholds)) {
+    warnings.push("Invalid outputRouter.thresholds; expected an object with positive integer thresholds.");
+  }
+  applyPositiveInteger(
+    config.thresholds,
+    warnings,
+    thresholdRecord.largeOutputBytes ?? input.largeOutputBytes,
+    "largeOutputBytes",
+    thresholdRecord.largeOutputBytes !== undefined
+      ? "outputRouter.thresholds.largeOutputBytes"
+      : "outputRouter.largeOutputBytes",
+  );
+  applyPositiveInteger(
+    config.thresholds,
+    warnings,
+    thresholdRecord.largeOutputLines ?? input.largeOutputLines,
+    "largeOutputLines",
+    thresholdRecord.largeOutputLines !== undefined
+      ? "outputRouter.thresholds.largeOutputLines"
+      : "outputRouter.largeOutputLines",
+  );
 }
-function applyVaultRoot(config, warnings, value) {
-    if (value === undefined) {
-        return;
-    }
-    if (typeof value === "string" && value.trim().length > 0) {
-        config.vault.root = value;
-        return;
-    }
-    warnings.push(`Invalid outputRouter.vaultRoot=${JSON.stringify(value)}; using ${DEFAULT_VAULT_ROOT}.`);
+function applyPositiveInteger(thresholds, warnings, value, key, path) {
+  if (value === undefined) {
+    return;
+  }
+  if (validatePositiveIntegerThreshold(value, path).length === 0) {
+    thresholds[key] = value;
+    return;
+  }
+  warnings.push(`Invalid ${path}=${JSON.stringify(value)}; using ${DEFAULT_ROUTER_THRESHOLDS[key]}.`);
 }
-function applyVaultRetention(config, warnings, value) {
-    if (value === undefined) {
-        return;
-    }
-    if (validatePositiveIntegerThreshold(value, "outputRouter.vaultRetentionDays").length === 0) {
-        config.vault.retention = { strategy: "ttl", ttlDays: value };
-        return;
-    }
-    warnings.push(`Invalid outputRouter.vaultRetentionDays=${JSON.stringify(value)}; using ${DEFAULT_VAULT_RETENTION.ttlDays}.`);
+function applyRouterVault(config, warnings, input) {
+  const vault = input.vault;
+  const vaultRecord = isRecord(vault) ? vault : {};
+  if (vault !== undefined && !isRecord(vault)) {
+    warnings.push("Invalid outputRouter.vault; expected an object with root and retention.");
+  }
+  applyVaultRoot(
+    config,
+    warnings,
+    vaultRecord.root ?? input.vaultRoot,
+    vaultRecord.root !== undefined ? "outputRouter.vault.root" : "outputRouter.vaultRoot",
+  );
+  const retention = vaultRecord.retention;
+  if (retention !== undefined) {
+    applyVaultRetentionPolicy(config, warnings, retention);
+  } else {
+    applyVaultRetentionDays(
+      config,
+      warnings,
+      vaultRecord.retentionDays ?? input.vaultRetentionDays,
+      vaultRecord.retentionDays !== undefined ? "outputRouter.vault.retentionDays" : "outputRouter.vaultRetentionDays",
+    );
+  }
 }
-function applyHints(config, warnings, generatedPaths, noisyCommandHints) {
-    const hints = {};
-    if (generatedPaths !== undefined) {
-        const parsed = parseStringArray(generatedPaths, "generatedPaths", warnings);
-        if (parsed) {
-            hints.generatedPathGlobs = parsed;
-        }
-    }
-    if (noisyCommandHints !== undefined) {
-        const parsed = parseStringArray(noisyCommandHints, "noisyCommandHints", warnings);
-        if (parsed) {
-            hints.noisyCommandPatterns = parsed;
-        }
-    }
-    if (hints.generatedPathGlobs || hints.noisyCommandPatterns) {
-        config.hints = hints;
-    }
+function applyVaultRoot(config, warnings, value, path) {
+  if (value === undefined) {
+    return;
+  }
+  if (typeof value === "string" && value.trim().length > 0) {
+    config.vault.root = value;
+    return;
+  }
+  warnings.push(`Invalid ${path}=${JSON.stringify(value)}; using ${DEFAULT_VAULT_ROOT}.`);
+}
+function applyVaultRetentionPolicy(config, warnings, value) {
+  if (!isRecord(value)) {
+    warnings.push(
+      "Invalid outputRouter.vault.retention; expected { strategy: 'ttl', ttlDays } or { strategy: 'manual' }.",
+    );
+    return;
+  }
+  if (value.strategy === "manual") {
+    config.vault.retention = { strategy: "manual" };
+    return;
+  }
+  if (value.strategy === undefined || value.strategy === "ttl") {
+    applyVaultRetentionDays(config, warnings, value.ttlDays, "outputRouter.vault.retention.ttlDays");
+    return;
+  }
+  warnings.push(
+    `Invalid outputRouter.vault.retention.strategy=${JSON.stringify(value.strategy)}; using ${DEFAULT_VAULT_RETENTION.strategy}.`,
+  );
+}
+function applyVaultRetentionDays(config, warnings, value, path) {
+  if (value === undefined) {
+    return;
+  }
+  if (validatePositiveIntegerThreshold(value, path).length === 0) {
+    config.vault.retention = { strategy: "ttl", ttlDays: value };
+    return;
+  }
+  warnings.push(`Invalid ${path}=${JSON.stringify(value)}; using ${DEFAULT_VAULT_RETENTION.ttlDays}.`);
+}
+function applyRouterHints(config, warnings, input) {
+  const hintsInput = input.hints;
+  const hintsRecord = isRecord(hintsInput) ? hintsInput : {};
+  if (hintsInput !== undefined && !isRecord(hintsInput)) {
+    warnings.push(
+      "Invalid outputRouter.hints; expected an object with generatedPathGlobs/noisyCommandPatterns arrays.",
+    );
+  }
+  const generatedPaths = hintsRecord.generatedPathGlobs ?? input.generatedPaths;
+  const noisyCommandHints = hintsRecord.noisyCommandPatterns ?? input.noisyCommandHints;
+  const hints = {};
+  if (generatedPaths !== undefined) {
+    const parsed = parseStringArray(
+      generatedPaths,
+      hintsRecord.generatedPathGlobs !== undefined
+        ? "outputRouter.hints.generatedPathGlobs"
+        : "outputRouter.generatedPaths",
+      warnings,
+    );
+    if (parsed) {
+      hints.generatedPathGlobs = parsed;
+    }
+  }
+  if (noisyCommandHints !== undefined) {
+    const parsed = parseStringArray(
+      noisyCommandHints,
+      hintsRecord.noisyCommandPatterns !== undefined
+        ? "outputRouter.hints.noisyCommandPatterns"
+        : "outputRouter.noisyCommandHints",
+      warnings,
+    );
+    if (parsed) {
+      hints.noisyCommandPatterns = parsed;
+    }
+  }
+  if (hints.generatedPathGlobs || hints.noisyCommandPatterns) {
+    config.hints = hints;
+  }
 }
 function applyScriptTransformLanguages(config, warnings, value) {
-    if (value === undefined) {
-        return;
-    }
-    if (!Array.isArray(value)) {
-        warnings.push("Invalid scriptTransform.languages; expected an array of supported language ids.");
-        return;
-    }
-    const languages = [];
-    for (const item of value) {
-        if (isStringIn(item, SCRIPT_TRANSFORM_LANGUAGES)) {
-            if (!languages.includes(item)) {
-                languages.push(item);
-            }
-        }
-        else {
-            warnings.push(`Invalid scriptTransform.languages entry=${JSON.stringify(item)}; supported languages are ${SCRIPT_TRANSFORM_LANGUAGES.join(", ")}.`);
-        }
-    }
-    if (languages.length > 0) {
-        config.languages = languages;
-    }
+  if (value === undefined) {
+    return;
+  }
+  if (!Array.isArray(value)) {
+    warnings.push("Invalid scriptTransform.languages; expected an array of supported language ids.");
+    return;
+  }
+  const languages = [];
+  for (const item of value) {
+    if (isStringIn(item, SCRIPT_TRANSFORM_LANGUAGES)) {
+      if (!languages.includes(item)) {
+        languages.push(item);
+      }
+    } else {
+      warnings.push(
+        `Invalid scriptTransform.languages entry=${JSON.stringify(item)}; supported languages are ${SCRIPT_TRANSFORM_LANGUAGES.join(", ")}.`,
+      );
+    }
+  }
+  if (languages.length > 0) {
+    config.languages = languages;
+  }
 }
 function applyScriptTransformLimits(config, warnings, value) {
-    if (value === undefined) {
-        return;
-    }
-    if (!isRecord(value)) {
-        warnings.push("Invalid scriptTransform.limits; expected an object with positive integer limits.");
-        return;
-    }
-    applyScriptLimit(config, warnings, value.timeoutMs, "timeoutMs");
-    applyScriptLimit(config, warnings, value.maxInputBytes, "maxInputBytes");
-    applyScriptLimit(config, warnings, value.maxOutputBytes, "maxOutputBytes");
+  if (value === undefined) {
+    return;
+  }
+  if (!isRecord(value)) {
+    warnings.push("Invalid scriptTransform.limits; expected an object with positive integer limits.");
+    return;
+  }
+  applyScriptLimit(config, warnings, value.timeoutMs, "timeoutMs");
+  applyScriptLimit(config, warnings, value.maxInputBytes, "maxInputBytes");
+  applyScriptLimit(config, warnings, value.maxOutputBytes, "maxOutputBytes");
 }
 function applyScriptLimit(config, warnings, value, key) {
-    if (value === undefined) {
-        return;
-    }
-    const max = MAX_SCRIPT_TRANSFORM_LIMITS[key];
-    if (Number.isInteger(value) && Number(value) > 0 && Number(value) <= max) {
-        config.limits[key] = Number(value);
-        return;
-    }
-    warnings.push(`Invalid scriptTransform.limits.${key}=${JSON.stringify(value)}; using ${DEFAULT_SCRIPT_TRANSFORM_LIMITS[key]}.`);
+  if (value === undefined) {
+    return;
+  }
+  const max = MAX_SCRIPT_TRANSFORM_LIMITS[key];
+  if (Number.isInteger(value) && Number(value) > 0 && Number(value) <= max) {
+    config.limits[key] = Number(value);
+    return;
+  }
+  warnings.push(
+    `Invalid scriptTransform.limits.${key}=${JSON.stringify(value)}; using ${DEFAULT_SCRIPT_TRANSFORM_LIMITS[key]}.`,
+  );
 }
 function applyStringEnum(config, warnings, value, key, path, allowed, fallback) {
-    if (value === undefined) {
-        return;
-    }
-    if (isStringIn(value, allowed)) {
-        config[String(key)] = value;
-        return;
-    }
-    warnings.push(`Invalid ${path}=${JSON.stringify(value)}; using ${fallback}.`);
+  if (value === undefined) {
+    return;
+  }
+  if (isStringIn(value, allowed)) {
+    config[String(key)] = value;
+    return;
+  }
+  warnings.push(`Invalid ${path}=${JSON.stringify(value)}; using ${fallback}.`);
 }
 function createDefaultObservedRoutingConfig() {
-    return {
-        enabled: DEFAULT_OBSERVED_ROUTING_CONFIG.enabled,
-        onRoutingFailure: DEFAULT_OBSERVED_ROUTING_CONFIG.onRoutingFailure,
-        mcp: { servers: {} },
-        web: { ...DEFAULT_OBSERVED_ROUTING_CONFIG.web },
-        fetch: { ...DEFAULT_OBSERVED_ROUTING_CONFIG.fetch },
-        codeSearch: { ...DEFAULT_OBSERVED_ROUTING_CONFIG.codeSearch },
-    };
+  return {
+    enabled: DEFAULT_OBSERVED_ROUTING_CONFIG.enabled,
+    onRoutingFailure: DEFAULT_OBSERVED_ROUTING_CONFIG.onRoutingFailure,
+    mcp: { servers: {} },
+    web: { ...DEFAULT_OBSERVED_ROUTING_CONFIG.web },
+    fetch: { ...DEFAULT_OBSERVED_ROUTING_CONFIG.fetch },
+    codeSearch: { ...DEFAULT_OBSERVED_ROUTING_CONFIG.codeSearch },
+  };
 }
 function applyObservedRoutingEnabled(config, warnings, value) {
-    if (value === undefined) {
-        return;
-    }
-    if (typeof value === "boolean") {
-        config.enabled = value;
-        return;
-    }
-    warnings.push(`Invalid observedRouting.enabled=${JSON.stringify(value)}; using ${DEFAULT_OBSERVED_ROUTING_CONFIG.enabled}.`);
+  if (value === undefined) {
+    return;
+  }
+  if (typeof value === "boolean") {
+    config.enabled = value;
+    return;
+  }
+  warnings.push(
+    `Invalid observedRouting.enabled=${JSON.stringify(value)}; using ${DEFAULT_OBSERVED_ROUTING_CONFIG.enabled}.`,
+  );
 }
 function applyObservedMcpConfig(config, warnings, value) {
-    if (value === undefined) {
-        return;
-    }
-    if (!isRecord(value)) {
-        warnings.push("Invalid observedRouting.mcp; expected an object with explicit servers. MCP observed routing remains disabled.");
-        return;
-    }
-    if (value.servers === undefined) {
-        return;
-    }
-    if (!isRecord(value.servers)) {
-        warnings.push("Invalid observedRouting.mcp.servers; expected an object keyed by MCP server id.");
-        return;
-    }
-    for (const [serverId, serverConfig] of Object.entries(value.servers)) {
-        const normalizedServerId = parseConfigString(serverId, "observedRouting.mcp.servers server id", warnings);
-        if (!normalizedServerId) {
-            continue;
-        }
-        config.mcp.servers[normalizedServerId] = parseObservedProducerConfig(serverConfig, `observedRouting.mcp.servers.${normalizedServerId}`, warnings);
-    }
+  if (value === undefined) {
+    return;
+  }
+  if (!isRecord(value)) {
+    warnings.push(
+      "Invalid observedRouting.mcp; expected an object with explicit servers. MCP observed routing remains disabled.",
+    );
+    return;
+  }
+  if (value.servers === undefined) {
+    return;
+  }
+  if (!isRecord(value.servers)) {
+    warnings.push("Invalid observedRouting.mcp.servers; expected an object keyed by MCP server id.");
+    return;
+  }
+  for (const [serverId, serverConfig] of Object.entries(value.servers)) {
+    const normalizedServerId = parseConfigString(serverId, "observedRouting.mcp.servers server id", warnings);
+    if (!normalizedServerId) {
+      continue;
+    }
+    config.mcp.servers[normalizedServerId] = parseObservedProducerConfig(
+      serverConfig,
+      `observedRouting.mcp.servers.${normalizedServerId}`,
+      warnings,
+    );
+  }
 }
 function parseObservedProducerConfig(value, path, warnings) {
-    const fallback = { enabled: false, persistence: DEFAULT_OBSERVED_ROUTING_PERSISTENCE };
-    if (value === undefined || value === null) {
-        return fallback;
-    }
-    if (!isRecord(value)) {
-        warnings.push(`Invalid ${path}; expected an object with enabled and persistence. Observed routing remains disabled for this producer.`);
-        return fallback;
-    }
-    const enabled = parseObservedProducerEnabled(value.enabled, path, warnings);
-    const persistence = parseObservedPersistence(value.persistence, `${path}.persistence`, enabled, warnings);
-    return { enabled, persistence };
+  const fallback = { enabled: false, persistence: DEFAULT_OBSERVED_ROUTING_PERSISTENCE };
+  if (value === undefined || value === null) {
+    return fallback;
+  }
+  if (!isRecord(value)) {
+    warnings.push(
+      `Invalid ${path}; expected an object with enabled and persistence. Observed routing remains disabled for this producer.`,
+    );
+    return fallback;
+  }
+  const enabled = parseObservedProducerEnabled(value.enabled, path, warnings);
+  const persistence = parseObservedPersistence(value.persistence, `${path}.persistence`, enabled, warnings);
+  return { enabled, persistence };
 }
 function parseObservedProducerEnabled(value, path, warnings) {
-    if (value === undefined) {
-        return false;
-    }
-    if (typeof value === "boolean") {
-        return value;
-    }
-    warnings.push(`Invalid ${path}.enabled=${JSON.stringify(value)}; using false.`);
+  if (value === undefined) {
     return false;
+  }
+  if (typeof value === "boolean") {
+    return value;
+  }
+  warnings.push(`Invalid ${path}.enabled=${JSON.stringify(value)}; using false.`);
+  return false;
 }
 function parseObservedPersistence(value, path, enabled, warnings) {
-    if (value === undefined) {
-        if (enabled) {
-            warnings.push(`Missing ${path}; setup must choose persistence explicitly. Using ${SAFE_OBSERVED_ROUTING_PERSISTENCE_FALLBACK}.`);
-            return SAFE_OBSERVED_ROUTING_PERSISTENCE_FALLBACK;
-        }
-        return DEFAULT_OBSERVED_ROUTING_PERSISTENCE;
-    }
-    if (isStringIn(value, OBSERVED_ROUTING_PERSISTENCE_MODES)) {
-        return value;
-    }
-    if (isStringIn(value, RESERVED_OBSERVED_ROUTING_PERSISTENCE_MODES)) {
-        warnings.push(`Unsupported ${path}=${JSON.stringify(value)}; redacted persistence is reserved for future work. Using ${SAFE_OBSERVED_ROUTING_PERSISTENCE_FALLBACK}.`);
-        return SAFE_OBSERVED_ROUTING_PERSISTENCE_FALLBACK;
-    }
-    const fallback = enabled ? SAFE_OBSERVED_ROUTING_PERSISTENCE_FALLBACK : DEFAULT_OBSERVED_ROUTING_PERSISTENCE;
-    warnings.push(`Invalid ${path}=${JSON.stringify(value)}; expected exact, metadata-only, or none. Using ${fallback}.`);
-    return fallback;
+  if (value === undefined) {
+    if (enabled) {
+      warnings.push(
+        `Missing ${path}; setup must choose persistence explicitly. Using ${SAFE_OBSERVED_ROUTING_PERSISTENCE_FALLBACK}.`,
+      );
+      return SAFE_OBSERVED_ROUTING_PERSISTENCE_FALLBACK;
+    }
+    return DEFAULT_OBSERVED_ROUTING_PERSISTENCE;
+  }
+  if (isStringIn(value, OBSERVED_ROUTING_PERSISTENCE_MODES)) {
+    return value;
+  }
+  if (isStringIn(value, RESERVED_OBSERVED_ROUTING_PERSISTENCE_MODES)) {
+    warnings.push(
+      `Unsupported ${path}=${JSON.stringify(value)}; redacted persistence is reserved for future work. Using ${SAFE_OBSERVED_ROUTING_PERSISTENCE_FALLBACK}.`,
+    );
+    return SAFE_OBSERVED_ROUTING_PERSISTENCE_FALLBACK;
+  }
+  const fallback = enabled ? SAFE_OBSERVED_ROUTING_PERSISTENCE_FALLBACK : DEFAULT_OBSERVED_ROUTING_PERSISTENCE;
+  warnings.push(`Invalid ${path}=${JSON.stringify(value)}; expected exact, metadata-only, or none. Using ${fallback}.`);
+  return fallback;
 }
 function parseConfigString(value, path, warnings) {
-    if (typeof value !== "string" || value.trim().length === 0 || /[\u0000-\u001F\u007F-\u009F\u2028\u2029]/.test(value)) {
-        warnings.push(`Invalid ${path}; expected a non-empty single-line string.`);
-        return undefined;
-    }
-    return value.trim();
+  if (
+    typeof value !== "string" ||
+    value.trim().length === 0 ||
+    /[\u0000-\u001F\u007F-\u009F\u2028\u2029]/.test(value)
+  ) {
+    warnings.push(`Invalid ${path}; expected a non-empty single-line string.`);
+    return undefined;
+  }
+  return value.trim();
 }
 function isStringIn(value, allowed) {
-    return typeof value === "string" && allowed.includes(value);
+  return typeof value === "string" && allowed.includes(value);
 }
-function parseStringArray(value, key, warnings) {
-    if (!Array.isArray(value)) {
-        warnings.push(`Invalid outputRouter.${key}; expected an array of strings.`);
-        return undefined;
-    }
-    if (!value.every((item) => typeof item === "string")) {
-        warnings.push(`Invalid outputRouter.${key}; expected an array of strings.`);
-        return undefined;
-    }
-    return value;
+function parseStringArray(value, path, warnings) {
+  if (!Array.isArray(value)) {
+    warnings.push(`Invalid ${path}; expected an array of strings.`);
+    return undefined;
+  }
+  if (!value.every((item) => typeof item === "string")) {
+    warnings.push(`Invalid ${path}; expected an array of strings.`);
+    return undefined;
+  }
+  return value;
 }