salmon-loop 0.3.1 → 0.4.1

package/dist/core/mcp/bridge/prompt-command-provider.js

@@ -1,4 +1,5 @@
 import { z } from 'zod';
+import { isRecord } from '../../utils/serialize.js';
 import { jsonSchemaToZod } from '../schema/json-schema-to-zod.js';
 const SAFE_TOKEN_PATTERN = /^[a-z0-9][a-z0-9_-]*$/;
 function normalizeToken(value) {
@@ -16,11 +17,11 @@ function safeToken(value, fallback) {
     return fallback;
 }
 function isPromptOptions(value) {
-    return Boolean(value &&
-        typeof value === 'object' &&
+    return Boolean(isRecord(value) &&
         'serverName' in value &&
         'client' in value &&
-        typeof value.client?.listPrompts === 'function');
+        isRecord(value.client) &&
+        typeof value.client.listPrompts === 'function');
 }
 function buildFallbackSchemaFromArguments(args = []) {
     const shape = {};

package/dist/core/mcp/bridge/tool-bridge.js

@@ -1,6 +1,7 @@
 import { z } from 'zod';
 import { LIMITS } from '../../config/limits.js';
 import { Phase } from '../../types/runtime.js';
+import { safeStringify } from '../../utils/serialize.js';
 import { classifyMcpTool } from '../policy/classifier.js';
 import { jsonSchemaToZod } from '../schema/json-schema-to-zod.js';
 const MCP_NAME_PATTERN = /^[a-zA-Z][a-zA-Z0-9_.-]*$/;
@@ -111,10 +112,9 @@ export function wrapMcpToolResult(result) {
 }
 export function mcpToolToToolSpec(input) {
     const override = findOverride(input.server.capabilities, input.tool.name);
-    const classification = classifyMcpTool({
-        tool: input.tool,
+    const classification = classifyMcpTool(input.tool, {
         trust: input.server.trust,
-        override,
+        override: override ? { sideEffects: override } : undefined,
     });
     const phase = input.server.capabilities.tools.phases[0] ?? Phase.VERIFY;
     const grantDecision = input.policy.decideTool({
@@ -169,7 +169,7 @@ export async function registerMcpV2Tools(input) {
         if (!catalog)
             continue;
         for (const tool of catalog.tools) {
-            const classification = classifyMcpTool({ tool: tool, trust: server.trust });
+            const classification = classifyMcpTool(tool, { trust: server.trust });
             const decision = input.policy.decideTool({
                 server: server.name,
                 toolName: tool.name,
@@ -267,16 +267,7 @@ function summarizeMcpAuthorization(input, args, riskLevel, sideEffects) {
             : { kind: 'classified', reason: input.classification.reason },
         args,
     };
-    return safeStringify(payload);
-}
-function safeStringify(value, maxLength = 1200) {
-    try {
-        const raw = JSON.stringify(value);
-        return raw.length <= maxLength ? raw : `${raw.slice(0, maxLength)}...`;
-    }
-    catch {
-        return '[Unserializable]';
-    }
+    return safeStringify(payload, { maxLength: 1200 });
 }
 function coerceRecord(input) {
     if (!input || typeof input !== 'object' || Array.isArray(input)) {

package/dist/core/mcp/client/connection-manager.js

@@ -2,6 +2,7 @@ import { Client } from '@modelcontextprotocol/sdk/client/index.js';
 import { CallToolResultSchema, PromptListChangedNotificationSchema, ReadResourceResultSchema, ResourceListChangedNotificationSchema, ResourceUpdatedNotificationSchema, ToolListChangedNotificationSchema, GetPromptResultSchema, } from '@modelcontextprotocol/sdk/types.js';
 import { LIMITS } from '../../config/limits.js';
 import { getLogger } from '../../observability/logger.js';
+import { errorMessage } from '../../utils/error.js';
 import { PACKAGE_VERSION } from '../../version.js';
 import { discoverMcpCatalog } from '../catalog/discovery.js';
 import { McpNotificationRouter } from '../catalog/notification-router.js';
@@ -93,7 +94,7 @@ export class McpConnectionManager {
         }
         catch (error) {
             entry.status = 'degraded';
-            entry.error = error instanceof Error ? error.message : String(error);
+            entry.error = errorMessage(error);
             getLogger().warn(`Failed to connect MCP server ${server.name}: ${entry.error}`);
         }
         return this.view(entry);
@@ -183,7 +184,7 @@ export class McpConnectionManager {
             entry.subscribedResources.add(uri);
         }
         catch (error) {
-            const message = error instanceof Error ? error.message : String(error);
+            const message = errorMessage(error);
             getLogger().warn(`MCP server ${entry.server.name} resource subscription failed for ${uri}: ${message}`);
         }
     }

package/dist/core/mcp/host/sampling-provider.js

@@ -120,7 +120,7 @@ export class McpSamplingProvider {
             throw new Error('MCP_SAMPLING_TOKEN_LIMIT_EXCEEDED');
         }
         const result = await this.llm.chat([{ role: 'user', content: input.prompt }]);
-        return typeof result === 'string' ? result : String(result?.content ?? '');
+        return typeof result === 'string' ? result : String(result.content ?? '');
     }
     async createMessage(params, options = {}) {
         const sanitizedParams = sanitizeForAudit(params, this.maxDepth);

package/dist/core/mcp/schema/json-schema-to-zod.js

@@ -1,5 +1,6 @@
 import { isDeepStrictEqual } from 'node:util';
 import { z } from 'zod';
+import { isRecord } from '../../utils/serialize.js';
 export function jsonSchemaToZod(jsonSchema) {
     return jsonSchemaToZodWithContext(jsonSchema, jsonSchema);
 }
@@ -499,7 +500,7 @@ function propertyNamesToZod(jsonSchema, rootSchema) {
     return jsonSchemaToZodWithContext(jsonSchema, rootSchema);
 }
 function isPlainObject(value) {
-    return typeof value === 'object' && value !== null && !Array.isArray(value);
+    return isRecord(value);
 }
 function isMultipleOf(value, divisor) {
     const quotient = value / divisor;

package/dist/core/memory/relevant-retrieval.js

@@ -77,11 +77,12 @@ export function buildRelevantMemoryCandidates(context) {
             tags: ['rules', 'project'],
         });
     }
-    if (trimToUndefined(knowledge?.user_preferences)) {
+    const userPrefs = trimToUndefined(knowledge?.user_preferences);
+    if (userPrefs) {
         candidates.push({
             path: '.salmonloop/knowledge/user_preferences',
             title: 'User preferences',
-            summary: knowledge.user_preferences,
+            summary: userPrefs,
             tags: ['preferences', 'user'],
         });
     }
@@ -96,11 +97,12 @@ export function buildRelevantMemoryCandidates(context) {
             tags: ['architecture', ...(decision.related_files ?? []).map((file) => file.toLowerCase())],
         });
     }
-    if (trimToUndefined(metadata?.aiInstructions)) {
+    const aiInstructions = trimToUndefined(metadata?.aiInstructions);
+    if (aiInstructions) {
         candidates.push({
             path: '.salmonloop/project/ai-instructions',
             title: 'Project AI instructions',
-            summary: metadata.aiInstructions,
+            summary: aiInstructions,
             tags: ['instructions', 'project'],
         });
     }

package/dist/core/observability/authorization-decisions.js

@@ -1,3 +1,4 @@
+import { asRecord } from '../utils/serialize.js';
 import { getAuditTrail } from './audit-trail.js';
 function safeString(value) {
     if (typeof value !== 'string')
@@ -23,13 +24,13 @@ export function extractAuthorizationDecisionsFromAuditTrail(auditTrail) {
             continue;
         if (event.action !== 'authorization.decision')
             continue;
-        const details = event.details;
-        if (!details || typeof details !== 'object')
+        if (!event.details || typeof event.details !== 'object')
             continue;
-        const callId = safeString(details.callId);
-        const toolName = safeString(details.toolName);
-        const phase = safeString(details.phase) ?? safeString(event.phase);
-        const outcome = safeString(details.outcome);
+        const d = asRecord(event.details);
+        const callId = safeString(d.callId);
+        const toolName = safeString(d.toolName);
+        const phase = safeString(d.phase) ?? safeString(event.phase);
+        const outcome = safeString(d.outcome);
         if (!callId || !toolName || !phase || !outcome)
             continue;
         decisions.push({
@@ -37,12 +38,12 @@ export function extractAuthorizationDecisionsFromAuditTrail(auditTrail) {
             toolName,
             phase: phase,
             outcome: outcome,
-            source: (safeString(details.source) ?? 'unknown'),
-            reason: safeString(details.reason),
-            ttlMs: safeNumber(details.ttlMs),
-            persist: safeString(details.persist),
-            riskLevel: safeString(details.riskLevel),
-            sideEffects: safeStringArray(details.sideEffects),
+            source: (safeString(d.source) ?? 'unknown'),
+            reason: safeString(d.reason),
+            ttlMs: safeNumber(d.ttlMs),
+            persist: safeString(d.persist),
+            riskLevel: safeString(d.riskLevel),
+            sideEffects: safeStringArray(d.sideEffects),
             timestamp: safeString(event.timestamp) ?? new Date().toISOString(),
         });
     }

package/dist/core/observability/error-mapping.js

@@ -1,4 +1,5 @@
 import { text } from '../../locales/index.js';
+import { isRecord } from '../utils/serialize.js';
 import { getAuditTrail } from './audit-trail.js';
 import { REDACTED_ERROR_TOKEN } from './error-envelope.js';
 function mapLlmCodeToMessage(code) {
@@ -193,7 +194,7 @@ export function mapErrorForAudit(input) {
     };
 }
 function buildLangfuseHttpFailed(details) {
-    const status = typeof details?.status === 'number' ? details.status : undefined;
+    const status = isRecord(details) && typeof details.status === 'number' ? details.status : undefined;
     if (!status)
         return undefined;
     if (status === 401 || status === 403) {

package/dist/core/observability/token-usage.js

@@ -1,3 +1,4 @@
+import { asRecord } from '../utils/serialize.js';
 import { getAuditTrail } from './audit-trail.js';
 function safeFiniteNumber(value) {
     if (typeof value !== 'number' || !Number.isFinite(value))
@@ -12,11 +13,11 @@ export function extractTokenUsageFromAuditTrail(auditTrail) {
             continue;
         if (event.action !== 'llm.usage')
             continue;
-        const details = event.details;
-        if (!details || typeof details !== 'object')
+        if (!event.details || typeof event.details !== 'object')
             continue;
-        const promptTokens = safeFiniteNumber(details.promptTokens);
-        const completionTokens = safeFiniteNumber(details.completionTokens);
+        const d = asRecord(event.details);
+        const promptTokens = safeFiniteNumber(d.promptTokens);
+        const completionTokens = safeFiniteNumber(d.completionTokens);
         if (typeof promptTokens === 'number')
             inputTokens += promptTokens;
         if (typeof completionTokens === 'number')

package/dist/core/plugin/loader.js

@@ -2,6 +2,7 @@ import { join } from 'path';
 import { typescriptPlugin, tsxPlugin, javascriptPlugin } from '../../languages/typescript/index.js';
 import { readdir } from '../adapters/fs/node-fs.js';
 import { getLogger } from '../observability/logger.js';
+import { errorMessage } from '../utils/error.js';
 import { validateQueryPack } from './validator.js';
 // Import built-in plugins (Phase 1: explicit import)
 export class PluginLoader {
@@ -31,11 +32,11 @@ export class PluginLoader {
         catch (error) {
             // In test environment, we want to know why it failed
             if (process.env.NODE_ENV === 'test') {
-                const errorMsg = error instanceof Error ? error.message : String(error);
+                const errorMsg = errorMessage(error);
                 getLogger().error(`CRITICAL: Failed to load plugins: ${errorMsg}`);
                 throw error;
             }
-            getLogger().error(`Failed to load plugins: ${error instanceof Error ? error.message : String(error)}`);
+            getLogger().error(`Failed to load plugins: ${errorMessage(error)}`);
         }
     }
     /**
@@ -70,7 +71,7 @@ export class PluginLoader {
                 }
                 catch (err) {
                     if (err && typeof err === 'object' && 'code' in err && err.code !== 'ENOENT') {
-                        getLogger().warn(`Failed to load user plugin from ${dirName}: ${err instanceof Error ? (err instanceof Error ? err.message : String(err)) : String(err)}`);
+                        getLogger().warn(`Failed to load user plugin from ${dirName}: ${errorMessage(err)}`);
                     }
                 }
             }
@@ -78,7 +79,7 @@ export class PluginLoader {
         catch (err) {
             // Ignore if directory doesn't exist
             if (err && typeof err === 'object' && 'code' in err && err.code !== 'ENOENT') {
-                getLogger().debug(`Error scanning for user plugins: ${err instanceof Error ? err.message : String(err)}`);
+                getLogger().debug(`Error scanning for user plugins: ${errorMessage(err)}`);
             }
         }
     }

package/dist/core/prompts/registry.js

@@ -2,6 +2,7 @@ import { fileURLToPath } from 'node:url';
 import Handlebars from 'handlebars';
 import { z } from 'zod';
 import { readFile } from '../adapters/fs/node-fs.js';
+import { unwrapZodSchema } from '../utils/zod.js';
 const TEMPLATE_URLS = {
     'system/_tool_defs.hbs': new URL('./templates/system/_tool_defs.hbs', import.meta.url),
     'system/main_system.hbs': new URL('./templates/system/main_system.hbs', import.meta.url),
@@ -60,8 +61,8 @@ export class PromptRegistry {
         if (!url) {
             throw new Error(`Unknown prompt template path: ${relativePath}`);
         }
-        const bunAny = globalThis;
-        const bun = bunAny.Bun;
+        const bunGlobal = globalThis;
+        const bun = bunGlobal.Bun;
         if (bun?.file) {
             return bun.file(url).text();
         }
@@ -113,34 +114,15 @@ export class PromptRegistry {
         if (!zodSchema) {
             return { type: 'object', description: 'Schema details unavailable' };
         }
-        const unwrapForJsonSchema = (schema) => {
-            let current = schema;
-            for (let depth = 0; depth < 20; depth++) {
-                const ZodEffects = z.ZodEffects;
-                if (ZodEffects && current instanceof ZodEffects) {
-                    current = current._def.schema;
-                    continue;
-                }
-                if (current instanceof z.ZodPipe) {
-                    current = current._def.out;
-                    continue;
-                }
-                if (current instanceof z.ZodOptional ||
-                    current instanceof z.ZodNullable ||
-                    current instanceof z.ZodDefault) {
-                    current = current._def.innerType;
-                    continue;
-                }
-                break;
-            }
-            return current;
-        };
         try {
-            const unwrapped = unwrapForJsonSchema(zodSchema);
-            const schema = z.toJSONSchema(unwrapped);
-            if (schema && typeof schema === 'object') {
-                const { $schema: _ignored, ...rest } = schema;
-                return rest;
+            const unwrapped = unwrapZodSchema(zodSchema);
+            const zWithJson = z;
+            if (zWithJson.toJSONSchema) {
+                const schema = zWithJson.toJSONSchema(unwrapped);
+                if (schema && typeof schema === 'object') {
+                    const { $schema: _ignored, ...rest } = schema;
+                    return rest;
+                }
             }
         }
         catch (_e) {

package/dist/core/protocols/a2a/sdk/server.js

@@ -406,9 +406,8 @@ const normalizeA2AExtensionHeadersByVersion = (req, res, next) => {
     });
     next();
 };
-function isObjectRecord(value) {
-    return typeof value === 'object' && value !== null && !Array.isArray(value);
-}
+import { isRecord } from '../../../utils/serialize.js';
+const isObjectRecord = isRecord;
 function looksLikeTask(result) {
     return typeof result.id === 'string' && 'status' in result;
 }

package/dist/core/protocols/acp/formal-agent.js

@@ -7,6 +7,7 @@ import { toAcpPublicModes } from '../../public-capabilities/projections.js';
 import { buildPublicCapabilityRegistry } from '../../public-capabilities/registry.js';
 import { parseSlashInput } from '../../slash/parser.js';
 import { Phase } from '../../types/runtime.js';
+import { isRecord } from '../../utils/serialize.js';
 import { buildCanonicalExecutionRequest } from '../shared/execution-request.js';
 import { parseAcpFlowMode } from '../shared/flow-mode-mapping.js';
 import { probeCheckpoint, probeCheckpointForNewSession, } from './acp-checkpoint-probe.js';
@@ -18,7 +19,7 @@ import { createAcpSessionStore, isTerminalTaskEvent } from './handlers.js';
 import { createAcpToolAuthorizationProvider } from './permission-provider.js';
 import { mapToolKind } from './tool-kind-mapping.js';
 function formatInputRequiredMessage(inputRequired) {
-    if (!inputRequired || !Array.isArray(inputRequired.questions))
+    if (!inputRequired || !isRecord(inputRequired) || !Array.isArray(inputRequired.questions))
         return null;
     const questions = inputRequired.questions;
     if (questions.length === 0)
@@ -525,6 +526,7 @@ function acpMcpServersToExtensions(mcpServers) {
         mcpServers: resolvedServers,
         toolPlugins: [],
         skillDiscovery: { paths: [], scope: 'repo' },
+        agentProfiles: [],
     };
 }
 function validateAcpMcpServers(mcpServers) {
@@ -569,12 +571,13 @@ function isPersistableSession(session) {
 async function awaitTerminalEvent(params) {
     if (!params.eventBus)
         return null;
-    const history = params.eventBus.list(params.taskId);
+    const { eventBus } = params;
+    const history = eventBus.list(params.taskId);
     const terminal = history.find(isTerminalTaskEvent);
     if (terminal)
         return terminal;
     return await new Promise((resolve) => {
-        const unsubscribe = params.eventBus.subscribe((event) => {
+        const unsubscribe = eventBus.subscribe((event) => {
             if (event.taskId !== params.taskId)
                 return;
             if (!isTerminalTaskEvent(event))
@@ -1092,7 +1095,10 @@ export function createAcpFormalAgent(deps) {
                         ...current,
                         history: [
                             ...current.history,
-                            { role: 'assistant', content: [buildTextContentBlock(responseText)] },
+                            {
+                                role: 'assistant',
+                                content: [buildTextContentBlock(responseText)],
+                            },
                         ],
                     }));
                     await sessionPersistence.persist();

package/dist/core/protocols/acp/stdio-server.js

@@ -1,6 +1,8 @@
 import { Readable, Writable } from 'node:stream';
 import { AgentSideConnection } from '@agentclientprotocol/sdk';
 import { tryGetLogger } from '../../observability/logger.js';
+import { errorMessage } from '../../utils/error.js';
+import { isRecord } from '../../utils/serialize.js';
 const INVALID_REQUEST = {
     jsonrpc: '2.0',
     id: null,
@@ -11,9 +13,7 @@ const PARSE_ERROR = {
     id: null,
     error: { code: -32700, message: 'Parse error' },
 };
-function isJsonObject(value) {
-    return typeof value === 'object' && value !== null && !Array.isArray(value);
-}
+const isJsonObject = isRecord;
 function hasOwn(value, key) {
     return Object.prototype.hasOwnProperty.call(value, key);
 }
@@ -50,9 +50,9 @@ function createNdjsonWriter(output) {
                 .catch(() => undefined)
                 .then(() => writer.write(data))
                 .catch((error) => {
-                const detail = error instanceof Error ? error.message : String(error);
+                const detail = errorMessage(error);
                 safeWarn(`ACP stdio failed to write NDJSON line. reason="${detail}"`);
-                lastError = error instanceof Error ? new Error(detail) : new Error(detail);
+                lastError = new Error(detail);
             });
             await tail;
         },
@@ -83,7 +83,7 @@ async function processStdioLine(line, ndjson, controller) {
         controller.enqueue(parsed);
     }
     catch (error) {
-        const detail = error instanceof Error ? error.message : String(error);
+        const detail = errorMessage(error);
         safeWarn(`ACP stdio failed to parse JSON line. reason="${detail}"`);
         await ndjson.write(PARSE_ERROR);
     }

package/dist/core/runtime/agent-server-runtime.js

@@ -65,12 +65,13 @@ export function createAgentServerRuntime(deps) {
         }
     }
     async function close() {
-        if (!a2aServerInstance) {
+        const instance = a2aServerInstance;
+        if (!instance) {
             started = false;
             return;
         }
         await new Promise((resolve, reject) => {
-            a2aServerInstance.close((error) => {
+            instance.close((error) => {
                 if (error) {
                     reject(error);
                     return;

package/dist/core/runtime/initialize.js

@@ -1,19 +1,24 @@
+import { readFileSync } from '../adapters/fs/node-fs.js';
 import { initializeDefaultCalculator } from '../context/policies/pack-until-full.js';
+import { getRepoAgentsConfigPath, getUserAgentsConfigPath } from '../extensions/paths.js';
+import { AgentsConfigSchema } from '../extensions/schemas.js';
 import { createLogger, getLogger, setLogger, tryGetLogger } from '../observability/logger.js';
 import { createMonitor, setMonitor, tryGetMonitor } from '../observability/monitor.js';
 import { registerDefaultSubAgentProfiles } from '../sub-agent/registry-defaults.js';
 import { createSubAgentRegistry, setSubAgentRegistry, tryGetSubAgentRegistry, } from '../sub-agent/registry.js';
+import { isRecord } from '../utils/serialize.js';
 /**
  * Initializes the Core safety runtime.
  * Mounts global error handlers and ensures environment safety.
  */
+const GLOBAL_FLAG = '__SALMON_RUNTIME_INITIALIZED__';
 export function initializeRuntime() {
     // Prevent duplicate initialization
-    if (globalThis.__SALMON_RUNTIME_INITIALIZED__)
+    if (globalThis[GLOBAL_FLAG])
         return;
     // Bypass interception in debug mode to allow raw console/stream output
     if (process.env.SALMONLOOP_DEBUG === 'true') {
-        globalThis.__SALMON_RUNTIME_INITIALIZED__ = true;
+        globalThis[GLOBAL_FLAG] = true;
         return;
     }
     // Preload token calculator in background (non-blocking)
@@ -32,6 +37,7 @@ export function initializeRuntime() {
     if (!tryGetSubAgentRegistry()) {
         const registry = createSubAgentRegistry();
         registerDefaultSubAgentProfiles(registry);
+        loadUserAgentProfiles(registry);
         setSubAgentRegistry(registry);
     }
     const isGui = process.argv.includes('--gui');
@@ -40,10 +46,12 @@ export function initializeRuntime() {
     const originalConsoleError = console.error;
     console.error = (...args) => {
         const sanitizedArgs = args.map((arg) => {
-            if (typeof arg === 'object' && arg !== null) {
+            if (isRecord(arg)) {
                 // Drop the object structure entirely for console output to prevent UI pollution
-                const code = arg.code || arg.llmCode || 'TECHNICAL_ERROR';
-                const msg = arg.message || 'No detail provided';
+                const code = (typeof arg.code === 'string' ? arg.code : undefined) ||
+                    (typeof arg.llmCode === 'string' ? arg.llmCode : undefined) ||
+                    'TECHNICAL_ERROR';
+                const msg = (typeof arg.message === 'string' ? arg.message : undefined) || 'No detail provided';
                 return `[${code}] ${msg}`;
             }
             return arg;
@@ -127,6 +135,62 @@ export function initializeRuntime() {
     process.on('uncaughtException', (error) => {
         getLogger().error('Uncaught Exception detected in Core runtime', error, true);
     });
-    globalThis.__SALMON_RUNTIME_INITIALIZED__ = true;
+    globalThis[GLOBAL_FLAG] = true;
+}
+function loadUserAgentProfiles(registry) {
+    const tryLoadSync = (filePath) => {
+        try {
+            const content = readFileSync(filePath, 'utf-8');
+            return AgentsConfigSchema.parse(JSON.parse(content));
+        }
+        catch {
+            return null;
+        }
+    };
+    // Load from repo and user scopes; repo takes priority
+    const repoRoot = process.cwd();
+    const userConfig = tryLoadSync(getUserAgentsConfigPath());
+    const repoConfig = tryLoadSync(getRepoAgentsConfigPath(repoRoot));
+    const toProfile = (raw) => ({
+        id: raw.id,
+        name: raw.name,
+        role: raw.role,
+        description: raw.description,
+        allowedTools: raw.allowedTools ?? ['code.search', 'fs.read'],
+        readOnly: raw.readOnly ?? false,
+        stratagem: raw.stratagem ?? 'investigator',
+        toolInheritance: raw.toolInheritance,
+        permissionMode: raw.permissionMode,
+        systemPrompt: raw.systemPrompt,
+        maxTokens: raw.maxTokens,
+        maxAttempts: raw.maxAttempts,
+        timeoutMs: raw.timeoutMs,
+    });
+    // User profiles first (lower priority)
+    if (userConfig) {
+        for (const agent of userConfig.agents) {
+            if (agent.enabled === false)
+                continue;
+            // Don't override built-in profiles
+            if (registry.has(agent.id)) {
+                tryGetLogger()?.debug(`[initializeRuntime] Skipping user agent '${agent.id}': conflicts with built-in profile`);
+                continue;
+            }
+            registry.register(toProfile(agent));
+        }
+    }
+    // Repo profiles override user (higher priority)
+    if (repoConfig) {
+        for (const agent of repoConfig.agents) {
+            if (agent.enabled === false)
+                continue;
+            // Don't override built-in profiles
+            if (registry.has(agent.id)) {
+                tryGetLogger()?.debug(`[initializeRuntime] Skipping repo agent '${agent.id}': conflicts with built-in profile`);
+                continue;
+            }
+            registry.register(toProfile(agent));
+        }
+    }
 }
 //# sourceMappingURL=initialize.js.map

package/dist/core/session/compaction/index.js

@@ -1,6 +1,7 @@
 import { getModelRecommendedBudget } from '../../context/token/adaptive-budget.js';
 import { LlmError } from '../../llm/errors.js';
 import { getLogger } from '../../observability/logger.js';
+import { isRecord } from '../../utils/serialize.js';
 import { refreshSessionSummary } from '../summary-sync.js';
 import { TokenTracker } from '../token-tracker.js';
 import { isCircuitBreakerTripped, onCompactionFailure, onCompactionSuccess } from './tracking.js';
@@ -11,8 +12,8 @@ function isContextOverflowLike(error) {
     }
     const message = error instanceof Error
         ? error.message
-        : error && typeof error === 'object' && typeof error.message === 'string'
-            ? String(error.message)
+        : isRecord(error) && typeof error.message === 'string'
+            ? error.message
             : '';
     if (!message)
         return false;
@@ -136,7 +137,7 @@ export async function autocompact(params) {
             performed: true,
             tracking: onCompactionSuccess(tracking),
             preTokens: totalTokens,
-            trigger: trigger,
+            trigger,
         };
     }
     catch (error) {