@pellux/goodvibes-sdk 0.30.3 → 0.33.1

package/dist/platform/config/helper-model.js

@@ -7,44 +7,60 @@
  * Resolution order:
  *   1. Per-provider helper (helper.providers.{currentProvider}.provider + model)
  *   2. Global helper (helper.globalProvider + helper.globalModel)
- *   3. Tool LLM (tools.llmProvider + tools.llmModel) — if configured
- *   4. Main model (fallback — not a true helper, but ensures the task runs)
+ *   3. Tool LLM (tools.llmProvider + tools.llmModel) — when enabled/configured
  *
  * Design constraints:
- *   - Never throws from HelperModel.chat() — returns empty string on any error
- *   - Logs failures via logger.debug (non-fatal)
- *   - Singleton pattern — import `helperModel` for use
+ *   - Optional helper absence is explicit: helperOnly calls return null.
+ *   - Configured routes are provider-qualified through the model registry.
+ *   - HelperModel.chat() throws request and configuration failures.
  *   - Tracks token usage separately from the main model
  */
+import { splitModelRegistryKey } from '../providers/registry-helpers.js';
 import { logger } from '../utils/logger.js';
 import { summarizeError } from '../utils/error-display.js';
+export class HelperModelUnavailableError extends Error {
+    constructor(message) {
+        super(message);
+        this.name = 'HelperModelUnavailableError';
+    }
+}
+function readOptionalString(value) {
+    return typeof value === 'string' ? value.trim() : '';
+}
 /**
  * HelperRouter — resolves which model to use for a given helper task.
  *
  * Resolution order:
  *   1. Per-provider helper (helper.providers.{currentProvider}.provider + model)
  *   2. Global helper (helper.globalProvider + helper.globalModel)
- *   3. Tool LLM (tools.llmProvider + tools.llmModel) — if configured
- *   4. Main model (fallback — not a true helper, but ensures the task runs)
+ *   3. Tool LLM (tools.llmProvider + tools.llmModel) — when enabled/configured
  */
 export class HelperRouter {
     deps;
     constructor(deps) {
         this.deps = deps;
     }
-    requireProvider(providerId) {
-        if (this.deps.providerRegistry.require) {
-            return this.deps.providerRegistry.require(providerId);
+    resolveConfiguredProviderModel(providerId, modelId) {
+        const provider = providerId.trim();
+        const model = modelId.trim();
+        if (!provider || !model) {
+            throw new Error('Helper model route requires provider and model.');
+        }
+        const registryKey = model.includes(':') ? model : `${provider}:${model}`;
+        const parsed = splitModelRegistryKey(registryKey);
+        if (parsed.providerId !== provider) {
+            throw new Error(`Helper model '${model}' conflicts with provider '${provider}'.`);
         }
-        const provider = this.deps.providerRegistry.get?.(providerId);
-        if (!provider)
-            throw new Error(`Unknown provider: ${providerId}`);
-        return provider;
+        return {
+            provider: this.deps.providerRegistry.getForModel(registryKey, provider),
+            modelId: parsed.resolvedModelId,
+            isHelper: true,
+        };
     }
     /**
      * Resolve the best helper for the given task.
      *
-     * Returns null only if no provider can be resolved at all.
+     * Returns null only when no dedicated helper route is configured.
      */
     resolve(_task) {
         try {
@@ -58,55 +74,48 @@ export class HelperRouter {
                 const providers = helperConfig['providers'];
                 if (providers && currentProviderName && providers[currentProviderName]) {
                     const perProvider = providers[currentProviderName];
-                    if (perProvider.provider && perProvider.model) {
-                        try {
-                            const provider = this.requireProvider(perProvider.provider);
-                            return { provider, modelId: perProvider.model, isHelper: true };
-                        }
-                        catch {
-                            logger.debug(`HelperRouter: per-provider helper ${perProvider.provider} not found, falling through`);
+                    const provider = readOptionalString(perProvider.provider);
+                    const model = readOptionalString(perProvider.model);
+                    if (provider || model) {
+                        if (!provider || !model) {
+                            throw new Error(`Per-provider helper route for '${currentProviderName}' requires provider and model.`);
                         }
+                        return this.resolveConfiguredProviderModel(provider, model);
                     }
                 }
             }
             // 2. Global helper
-            const globalProvider = this.deps.configManager.get('helper.globalProvider');
-            const globalModel = this.deps.configManager.get('helper.globalModel');
-            if (globalProvider && globalModel) {
-                try {
-                    const provider = this.requireProvider(globalProvider);
-                    return { provider, modelId: globalModel, isHelper: true };
-                }
-                catch {
-                    logger.debug(`HelperRouter: global helper ${globalProvider} not found, falling through`);
+            const globalProvider = readOptionalString(this.deps.configManager.get('helper.globalProvider'));
+            const globalModel = readOptionalString(this.deps.configManager.get('helper.globalModel'));
+            if (globalProvider || globalModel) {
+                if (!globalProvider || !globalModel) {
+                    throw new Error('Global helper routing requires both helper.globalProvider and helper.globalModel.');
                 }
+                return this.resolveConfiguredProviderModel(globalProvider, globalModel);
             }
             // 3. Tool LLM
-            const toolProvider = this.deps.configManager.get('tools.llmProvider');
-            const toolModel = this.deps.configManager.get('tools.llmModel');
-            if (toolProvider && toolModel) {
-                try {
-                    const provider = this.requireProvider(toolProvider);
-                    return { provider, modelId: toolModel, isHelper: true };
-                }
-                catch {
-                    logger.debug(`HelperRouter: tool LLM ${toolProvider} not found, falling through`);
+            const toolEnabled = this.deps.configManager.get('tools.llmEnabled');
+            const toolProvider = readOptionalString(this.deps.configManager.get('tools.llmProvider'));
+            const toolModel = readOptionalString(this.deps.configManager.get('tools.llmModel'));
+            if (toolEnabled && (toolProvider || toolModel)) {
+                if (!toolProvider || !toolModel) {
+                    throw new Error('Helper tool-LLM routing requires both tools.llmProvider and tools.llmModel.');
                 }
+                return this.resolveConfiguredProviderModel(toolProvider, toolModel);
             }
-            // 4. Fallback to main model
-            const mainProvider = this.deps.providerRegistry.getForModel(currentModel.id, currentModel.provider);
-            return { provider: mainProvider, modelId: currentModel.id, isHelper: false };
+            return null;
         }
         catch (err) {
-            logger.debug('HelperRouter.resolve: failed', { task: _task, error: summarizeError(err) });
-            return null;
+            logger.error('HelperRouter.resolve: failed', { task: _task, error: summarizeError(err) });
+            throw err;
         }
     }
 }
 /**
  * HelperModel — lightweight LLM interface for helper tasks.
  *
- * Callers own HelperModel lifetimes explicitly. Never throws — returns empty string on failure.
+ * Callers own HelperModel lifetimes explicitly. Provider and configuration
+ * failures throw. Explicit optional-helper calls return null when unavailable.
  * Tracks token usage separately from the main model.
  */
 export class HelperModel {
@@ -123,43 +132,25 @@ export class HelperModel {
      * @param task     The helper task type (for routing).
      * @param prompt   The user prompt to send.
      * @param options  Optional maxTokens, systemPrompt, helperOnly.
-     * @returns        The assistant's text response, or empty string on failure.
+     * @returns        The assistant's text response, or null for explicit optional helper absence.
      */
     async chat(task, prompt, options = {}) {
-        // When helper is disabled, skip the resolution chain entirely
         const enabled = this.deps.configManager.get('helper.enabled');
         if (!enabled) {
-            if (options.helperOnly)
-                return '';
-            // Disabled: bypass helper resolution — use main model directly
-            try {
-                const currentModel = this.deps.providerRegistry.getCurrentModel();
-                const mainProvider = this.deps.providerRegistry.getForModel(currentModel.id, currentModel.provider);
-                const response = await mainProvider.chat({
-                    model: currentModel.id,
-                    messages: [{ role: 'user', content: prompt }],
-                    maxTokens: options.maxTokens ?? 2048,
-                    systemPrompt: options.systemPrompt,
-                });
-                this._usage.inputTokens += response.usage?.inputTokens ?? 0;
-                this._usage.outputTokens += response.usage?.outputTokens ?? 0;
-                this._usage.calls += 1;
-                return response.content ?? '';
-            }
-            catch (err) {
-                logger.debug('HelperModel.chat: main model fallback failed (non-fatal)', { task, error: summarizeError(err) });
-                return '';
+            if (options.helperOnly) {
+                logger.info('HelperModel.chat: helper disabled for optional helper request', { task });
+                return null;
             }
+            throw new HelperModelUnavailableError('Helper model routing is disabled.');
         }
         try {
             const resolved = this.router.resolve(task);
             if (!resolved) {
-                logger.debug('HelperModel.chat: no provider resolved', { task });
-                return '';
-            }
-            // If helperOnly is set and we fell back to the main model, return empty
-            if (options.helperOnly && !resolved.isHelper) {
-                return '';
+                if (options.helperOnly) {
+                    logger.info('HelperModel.chat: no helper route for optional helper request', { task });
+                    return null;
+                }
+                throw new HelperModelUnavailableError('No helper model route is configured.');
             }
             const response = await resolved.provider.chat({
                 model: resolved.modelId,
@@ -178,11 +169,14 @@ export class HelperModel {
                 inputTokens: response.usage?.inputTokens,
                 outputTokens: response.usage?.outputTokens,
             });
-            return response.content ?? '';
+            if (typeof response.content !== 'string') {
+                throw new Error('Helper model response did not include string content.');
+            }
+            return response.content;
         }
         catch (err) {
-            logger.debug('HelperModel.chat: request failed (non-fatal)', { task, error: summarizeError(err) });
-            return '';
+            logger.error('HelperModel.chat: request failed', { task, error: summarizeError(err) });
+            throw err;
         }
     }
     /** Get cumulative helper usage since last reset. */

package/dist/platform/config/index.d.ts

@@ -15,7 +15,6 @@ import { ConfigManager } from './manager.js';
 import type { GoodVibesConfig } from './schema.js';
 export declare function getConfigSnapshot(configManager: Pick<ConfigManager, 'getRaw'>): Readonly<GoodVibesConfig>;
 export declare function getConfiguredModelId(configManager: Pick<ConfigManager, 'get'>): string;
-export declare function getConfiguredProviderId(configManager: Pick<ConfigManager, 'get'>): string;
 export declare function getConfiguredEmbeddingProviderId(configManager: Pick<ConfigManager, 'get'>): string;
 export declare function isAutoApproveEnabled(configManager: Pick<ConfigManager, 'get'>): boolean;
 export declare function getWorkingDirectory(configManager: Pick<ConfigManager, 'getWorkingDirectory'>): string | null;
@@ -23,7 +22,7 @@ export declare function getConfiguredSystemPrompt(configManager: Pick<ConfigMana
 export { getConfiguredApiKeys, resolveApiKeys } from './api-keys.js';
 export { createOAuthLocalListener } from './oauth-local-listener.js';
 export type { OAuthLocalListener, OAuthLocalListenerConfig } from './oauth-local-listener.js';
-export { HelperModel, HelperRouter } from './helper-model.js';
+export { HelperModel, HelperModelUnavailableError, HelperRouter } from './helper-model.js';
 export type { HelperChatOptions, HelperModelDeps, HelperTask, HelperUsage, ResolvedHelper, } from './helper-model.js';
 export { beginOpenAICodexLogin, exchangeOpenAICodexCode, refreshOpenAICodexToken, OPENAI_CODEX_AUTHORIZE_URL, OPENAI_CODEX_CLIENT_ID, OPENAI_CODEX_REDIRECT_URI, OPENAI_CODEX_TOKEN_URL, } from './openai-codex-auth.js';
 export type { OpenAICodexLoginStart, OpenAICodexTokenResult } from './openai-codex-auth.js';
@@ -36,6 +35,6 @@ export { SubscriptionManager } from './subscriptions.js';
 export type { OAuthProviderConfig, PendingSubscriptionLogin, ProviderSubscription, } from './subscriptions.js';
 export * from './subscription-auth.js';
 export * from './subscription-providers.js';
-export { resolveToolLLM, ToolLLM } from './tool-llm.js';
+export { resolveToolLLM, ToolLLM, ToolLLMUnavailableError } from './tool-llm.js';
 export type { ResolvedToolLLM, ToolLLMChatOptions, ToolLLMDeps, } from './tool-llm.js';
 //# sourceMappingURL=index.d.ts.map

package/dist/platform/config/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/platform/config/index.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,EAAE,aAAa,EAAE,MAAM,cAAc,CAAC;AAC7C,YAAY,EAAE,YAAY,EAAE,MAAM,cAAc,CAAC;AACjD,YAAY,EAAE,eAAe,EAAE,SAAS,EAAE,WAAW,EAAE,aAAa,EAAE,cAAc,EAAE,gBAAgB,EAAE,qBAAqB,EAAE,mBAAmB,EAAE,MAAM,aAAa,CAAC;AACxK,OAAO,EAAE,cAAc,EAAE,WAAW,EAAE,aAAa,EAAE,gBAAgB,EAAE,MAAM,aAAa,CAAC;AAC3F,YAAY,EAAE,kBAAkB,EAAE,MAAM,mBAAmB,CAAC;AAC5D,OAAO,EAAE,WAAW,EAAE,MAAM,oBAAoB,CAAC;AAGjD,OAAO,EAAE,aAAa,EAAE,MAAM,cAAc,CAAC;AAC7C,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,aAAa,CAAC;AAInD,wBAAgB,iBAAiB,CAAC,aAAa,EAAE,IAAI,CAAC,aAAa,EAAE,QAAQ,CAAC,GAAG,QAAQ,CAAC,eAAe,CAAC,CAEzG;AAED,wBAAgB,oBAAoB,CAAC,aAAa,EAAE,IAAI,CAAC,aAAa,EAAE,KAAK,CAAC,GAAG,MAAM,CAEtF;AAED,wBAAgB,uBAAuB,CAAC,aAAa,EAAE,IAAI,CAAC,aAAa,EAAE,KAAK,CAAC,GAAG,MAAM,CAEzF;AAED,wBAAgB,gCAAgC,CAAC,aAAa,EAAE,IAAI,CAAC,aAAa,EAAE,KAAK,CAAC,GAAG,MAAM,CAElG;AAED,wBAAgB,oBAAoB,CAAC,aAAa,EAAE,IAAI,CAAC,aAAa,EAAE,KAAK,CAAC,GAAG,OAAO,CAEvF;AAED,wBAAgB,mBAAmB,CAAC,aAAa,EAAE,IAAI,CAAC,aAAa,EAAE,qBAAqB,CAAC,GAAG,MAAM,GAAG,IAAI,CAE5G;AAED,wBAAgB,yBAAyB,CAAC,aAAa,EAAE,IAAI,CAAC,aAAa,EAAE,KAAK,CAAC,GAAG,MAAM,GAAG,SAAS,CASvG;AAED,OAAO,EAAE,oBAAoB,EAAE,cAAc,EAAE,MAAM,eAAe,CAAC;AACrE,OAAO,EAAE,wBAAwB,EAAE,MAAM,2BAA2B,CAAC;AACrE,YAAY,EAAE,kBAAkB,EAAE,wBAAwB,EAAE,MAAM,2BAA2B,CAAC;AAC9F,OAAO,EAAE,WAAW,EAAE,YAAY,EAAE,MAAM,mBAAmB,CAAC;AAC9D,YAAY,EACV,iBAAiB,EACjB,eAAe,EACf,UAAU,EACV,WAAW,EACX,cAAc,GACf,MAAM,mBAAmB,CAAC;AAC3B,OAAO,EACL,qBAAqB,EACrB,uBAAuB,EACvB,uBAAuB,EACvB,0BAA0B,EAC1B,sBAAsB,EACtB,yBAAyB,EACzB,sBAAsB,GACvB,MAAM,wBAAwB,CAAC;AAChC,YAAY,EAAE,qBAAqB,EAAE,sBAAsB,EAAE,MAAM,wBAAwB,CAAC;AAC5F,cAAc,kBAAkB,CAAC;AACjC,OAAO,EAAE,cAAc,EAAE,MAAM,cAAc,CAAC;AAC9C,YAAY,EACV,mBAAmB,EACnB,YAAY,EACZ,WAAW,EACX,YAAY,EACZ,mBAAmB,EACnB,iBAAiB,EACjB,mBAAmB,EACnB,kBAAkB,EAClB,qBAAqB,GACtB,MAAM,cAAc,CAAC;AACtB,OAAO,EAAE,eAAe,EAAE,MAAM,uBAAuB,CAAC;AACxD,mBAAmB,uBAAuB,CAAC;AAC3C,OAAO,EAAE,mBAAmB,EAAE,MAAM,oBAAoB,CAAC;AACzD,YAAY,EACV,mBAAmB,EACnB,wBAAwB,EACxB,oBAAoB,GACrB,MAAM,oBAAoB,CAAC;AAC5B,cAAc,wBAAwB,CAAC;AACvC,cAAc,6BAA6B,CAAC;AAC5C,OAAO,EAAE,cAAc,EAAE,OAAO,EAAE,MAAM,eAAe,CAAC;AACxD,YAAY,EACV,eAAe,EACf,kBAAkB,EAClB,WAAW,GACZ,MAAM,eAAe,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/platform/config/index.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,EAAE,aAAa,EAAE,MAAM,cAAc,CAAC;AAC7C,YAAY,EAAE,YAAY,EAAE,MAAM,cAAc,CAAC;AACjD,YAAY,EAAE,eAAe,EAAE,SAAS,EAAE,WAAW,EAAE,aAAa,EAAE,cAAc,EAAE,gBAAgB,EAAE,qBAAqB,EAAE,mBAAmB,EAAE,MAAM,aAAa,CAAC;AACxK,OAAO,EAAE,cAAc,EAAE,WAAW,EAAE,aAAa,EAAE,gBAAgB,EAAE,MAAM,aAAa,CAAC;AAC3F,YAAY,EAAE,kBAAkB,EAAE,MAAM,mBAAmB,CAAC;AAC5D,OAAO,EAAE,WAAW,EAAE,MAAM,oBAAoB,CAAC;AAGjD,OAAO,EAAE,aAAa,EAAE,MAAM,cAAc,CAAC;AAC7C,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,aAAa,CAAC;AAEnD,wBAAgB,iBAAiB,CAAC,aAAa,EAAE,IAAI,CAAC,aAAa,EAAE,QAAQ,CAAC,GAAG,QAAQ,CAAC,eAAe,CAAC,CAEzG;AAED,wBAAgB,oBAAoB,CAAC,aAAa,EAAE,IAAI,CAAC,aAAa,EAAE,KAAK,CAAC,GAAG,MAAM,CAEtF;AAED,wBAAgB,gCAAgC,CAAC,aAAa,EAAE,IAAI,CAAC,aAAa,EAAE,KAAK,CAAC,GAAG,MAAM,CAElG;AAED,wBAAgB,oBAAoB,CAAC,aAAa,EAAE,IAAI,CAAC,aAAa,EAAE,KAAK,CAAC,GAAG,OAAO,CAEvF;AAED,wBAAgB,mBAAmB,CAAC,aAAa,EAAE,IAAI,CAAC,aAAa,EAAE,qBAAqB,CAAC,GAAG,MAAM,GAAG,IAAI,CAE5G;AAED,wBAAgB,yBAAyB,CAAC,aAAa,EAAE,IAAI,CAAC,aAAa,EAAE,KAAK,CAAC,GAAG,MAAM,GAAG,SAAS,CAIvG;AAED,OAAO,EAAE,oBAAoB,EAAE,cAAc,EAAE,MAAM,eAAe,CAAC;AACrE,OAAO,EAAE,wBAAwB,EAAE,MAAM,2BAA2B,CAAC;AACrE,YAAY,EAAE,kBAAkB,EAAE,wBAAwB,EAAE,MAAM,2BAA2B,CAAC;AAC9F,OAAO,EAAE,WAAW,EAAE,2BAA2B,EAAE,YAAY,EAAE,MAAM,mBAAmB,CAAC;AAC3F,YAAY,EACV,iBAAiB,EACjB,eAAe,EACf,UAAU,EACV,WAAW,EACX,cAAc,GACf,MAAM,mBAAmB,CAAC;AAC3B,OAAO,EACL,qBAAqB,EACrB,uBAAuB,EACvB,uBAAuB,EACvB,0BAA0B,EAC1B,sBAAsB,EACtB,yBAAyB,EACzB,sBAAsB,GACvB,MAAM,wBAAwB,CAAC;AAChC,YAAY,EAAE,qBAAqB,EAAE,sBAAsB,EAAE,MAAM,wBAAwB,CAAC;AAC5F,cAAc,kBAAkB,CAAC;AACjC,OAAO,EAAE,cAAc,EAAE,MAAM,cAAc,CAAC;AAC9C,YAAY,EACV,mBAAmB,EACnB,YAAY,EACZ,WAAW,EACX,YAAY,EACZ,mBAAmB,EACnB,iBAAiB,EACjB,mBAAmB,EACnB,kBAAkB,EAClB,qBAAqB,GACtB,MAAM,cAAc,CAAC;AACtB,OAAO,EAAE,eAAe,EAAE,MAAM,uBAAuB,CAAC;AACxD,mBAAmB,uBAAuB,CAAC;AAC3C,OAAO,EAAE,mBAAmB,EAAE,MAAM,oBAAoB,CAAC;AACzD,YAAY,EACV,mBAAmB,EACnB,wBAAwB,EACxB,oBAAoB,GACrB,MAAM,oBAAoB,CAAC;AAC5B,cAAc,wBAAwB,CAAC;AACvC,cAAc,6BAA6B,CAAC;AAC5C,OAAO,EAAE,cAAc,EAAE,OAAO,EAAE,uBAAuB,EAAE,MAAM,eAAe,CAAC;AACjF,YAAY,EACV,eAAe,EACf,kBAAkB,EAClB,WAAW,GACZ,MAAM,eAAe,CAAC"}

package/dist/platform/config/index.js

@@ -10,17 +10,12 @@ export { DEFAULT_CONFIG, CONFIG_KEYS, CONFIG_SCHEMA, isValidConfigKey } from './
 export { ConfigError } from '../types/errors.js';
 import { readFileSync } from 'fs';
 import { ConfigManager } from './manager.js';
-import { logger } from '../utils/logger.js';
-import { summarizeError } from '../utils/error-display.js';
 export function getConfigSnapshot(configManager) {
     return configManager.getRaw();
 }
 export function getConfiguredModelId(configManager) {
     return configManager.get('provider.model');
 }
-export function getConfiguredProviderId(configManager) {
-    return configManager.get('provider.provider');
-}
 export function getConfiguredEmbeddingProviderId(configManager) {
     return configManager.get('provider.embeddingProvider');
 }
@@ -34,17 +29,11 @@ export function getConfiguredSystemPrompt(configManager) {
     const file = configManager.get('provider.systemPromptFile');
     if (!file)
         return undefined;
-    try {
-        return readFileSync(file, 'utf-8');
-    }
-    catch (err) {
-        logger.debug('systemPrompt file read failed (non-fatal)', { file, error: summarizeError(err) });
-        return undefined;
-    }
+    return readFileSync(file, 'utf-8');
 }
 export { getConfiguredApiKeys, resolveApiKeys } from './api-keys.js';
 export { createOAuthLocalListener } from './oauth-local-listener.js';
-export { HelperModel, HelperRouter } from './helper-model.js';
+export { HelperModel, HelperModelUnavailableError, HelperRouter } from './helper-model.js';
 export { beginOpenAICodexLogin, exchangeOpenAICodexCode, refreshOpenAICodexToken, OPENAI_CODEX_AUTHORIZE_URL, OPENAI_CODEX_CLIENT_ID, OPENAI_CODEX_REDIRECT_URI, OPENAI_CODEX_TOKEN_URL, } from './openai-codex-auth.js';
 export * from './secret-refs.js';
 export { SecretsManager } from './secrets.js';
@@ -52,4 +41,4 @@ export { ServiceRegistry } from './service-registry.js';
 export { SubscriptionManager } from './subscriptions.js';
 export * from './subscription-auth.js';
 export * from './subscription-providers.js';
-export { resolveToolLLM, ToolLLM } from './tool-llm.js';
+export { resolveToolLLM, ToolLLM, ToolLLMUnavailableError } from './tool-llm.js';

package/dist/platform/config/manager.d.ts

@@ -6,24 +6,23 @@ export type DeepReadonly<T> = {
 };
 /** Constructor overrides for CLI args and programmatic instantiation. */
 interface ConfigCliOverrides {
-    model?: string;
-    provider?: string;
-    autoApprove?: boolean;
-    systemPromptFile?: string;
-    workingDir?: string;
-    surfaceRoot?: string;
+    model?: string | undefined;
+    autoApprove?: boolean | undefined;
+    systemPromptFile?: string | undefined;
+    workingDir?: string | undefined;
+    surfaceRoot?: string | undefined;
 }
 export type ConfigOverrides = ConfigCliOverrides & ({
     configDir: string;
-    homeDir?: string;
-    sharedConfigPath?: string;
+    homeDir?: string | undefined;
+    sharedConfigPath?: string | undefined;
 } | {
     homeDir: string;
-    configDir?: string;
-    sharedConfigPath?: string;
+    configDir?: string | undefined;
+    sharedConfigPath?: string | undefined;
 });
 export interface ConfigSetOptions {
-    bypassManagedLock?: boolean;
+    bypassManagedLock?: boolean | undefined;
 }
 /** Callback invoked when a watched config key changes. */
 export type ConfigChangeCallback<K extends ConfigKey> = (newValue: ConfigValue<K>, oldValue: ConfigValue<K>) => void;

package/dist/platform/config/manager.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"manager.d.ts","sourceRoot":"","sources":["../../../src/platform/config/manager.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,eAAe,EAAE,SAAS,EAAE,WAAW,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AAI1F,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,mBAAmB,CAAC;AAOxD,wFAAwF;AACxF,MAAM,MAAM,YAAY,CAAC,CAAC,IAAI;IAC5B,QAAQ,EAAE,CAAC,IAAI,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,SAAS,MAAM,GAAG,YAAY,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC;CACzE,CAAC;AAEF,yEAAyE;AACzE,UAAU,kBAAkB;IAC1B,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,WAAW,CAAC,EAAE,MAAM,CAAC;CACtB;AAED,MAAM,MAAM,eAAe,GAAG,kBAAkB,GAAG,CAC/C;IACA,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,gBAAgB,CAAC,EAAE,MAAM,CAAC;CAC3B,GACC;IACA,OAAO,EAAE,MAAM,CAAC;IAChB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,gBAAgB,CAAC,EAAE,MAAM,CAAC;CAC3B,CACF,CAAC;AASF,MAAM,WAAW,gBAAgB;IAC/B,iBAAiB,CAAC,EAAE,OAAO,CAAC;CAC7B;AAED,0DAA0D;AAC1D,MAAM,MAAM,oBAAoB,CAAC,CAAC,SAAS,SAAS,IAAI,CAAC,QAAQ,EAAE,WAAW,CAAC,CAAC,CAAC,EAAE,QAAQ,EAAE,WAAW,CAAC,CAAC,CAAC,KAAK,IAAI,CAAC;AAErH,gEAAgE;AAChE,MAAM,MAAM,iBAAiB,GAAG,MAAM,IAAI,CAAC;AAkD3C;;;;;GAKG;AACH,qBAAa,aAAa;IACxB,OAAO,CAAC,MAAM,CAAkB;IAChC,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAS;IACnC,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAS;IACpC,OAAO,CAAC,QAAQ,CAAC,iBAAiB,CAAgB;IAClD,OAAO,CAAC,QAAQ,CAAC,gBAAgB,CAAgB;IACjD,OAAO,CAAC,QAAQ,CAAC,aAAa,CAAgB;IAC9C,OAAO,CAAC,cAAc,CAA6C;IACnE,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAsE;gBAErF,SAAS,EAAE,eAAe;IA4CtC,wBAAwB,IAAI,MAAM;IAIlC,mBAAmB,IAAI,MAAM,GAAG,IAAI;IAIpC,gBAAgB,IAAI,MAAM,GAAG,IAAI;IAIjC,oBAAoB,CAAC,cAAc,EAAE,IAAI,CAAC,cAAc,EAAE,MAAM,CAAC,GAAG,IAAI,GAAG,IAAI;IAI/E,OAAO,CAAC,WAAW;IAwBnB,0CAA0C;IAC1C,GAAG,CAAC,CAAC,SAAS,SAAS,EAAE,GAAG,EAAE,CAAC,GAAG,WAAW,CAAC,CAAC,CAAC;IAKhD,gEAAgE;IAChE,GAAG,CAAC,CAAC,SAAS,SAAS,EAAE,GAAG,EAAE,CAAC,EAAE,KAAK,EAAE,WAAW,CAAC,CAAC,CAAC,EAAE,OAAO,GAAE,gBAAqB,GAAG,IAAI;IAuB7F;;;OAGG;IACH,SAAS,CAAC,CAAC,SAAS,SAAS,EAAE,GAAG,EAAE,CAAC,EAAE,EAAE,EAAE,oBAAoB,CAAC,CAAC,CAAC,GAAG,iBAAiB;IAatF,sDAAsD;IACtD,OAAO,CAAC,eAAe;IAevB;;OAEG;IACH,OAAO,CAAC,cAAc;IA0BtB;;;;OAIG;IACH,UAAU,CAAC,GAAG,EAAE,SAAS,EAAE,KAAK,EAAE,OAAO,EAAE,OAAO,GAAE,gBAAqB,GAAG,IAAI;IAIhF,wFAAwF;IACxF,MAAM,IAAI,YAAY,CAAC,eAAe,CAAC;IAIvC,0DAA0D;IAC1D,WAAW,CAAC,CAAC,SAAS,MAAM,eAAe,EAAE,QAAQ,EAAE,CAAC,GAAG,QAAQ,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC;IAIvF,+FAA+F;IAC/F,MAAM,IAAI,QAAQ,CAAC,eAAe,CAAC;IAInC,8BAA8B;IAC9B,SAAS,IAAI,aAAa,EAAE;IAI5B,0DAA0D;IAC1D,IAAI,IAAI,IAAI;IASZ,yEAAyE;IACzE,WAAW,IAAI,IAAI;IAYnB,4FAA4F;IAC5F,IAAI,IAAI,IAAI;IAyBZ;;;;;;OAMG;IACH,aAAa,CAAC,CAAC,SAAS,MAAM,eAAe,EAAE,QAAQ,EAAE,CAAC,EAAE,KAAK,EAAE,OAAO,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC,GAAG,IAAI;IAWrG;;;OAGG;IACH,KAAK,CAAC,GAAG,CAAC,EAAE,SAAS,GAAG,IAAI;CAY7B"}
+{"version":3,"file":"manager.d.ts","sourceRoot":"","sources":["../../../src/platform/config/manager.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,eAAe,EAAE,SAAS,EAAE,WAAW,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AAI1F,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,mBAAmB,CAAC;AAOxD,wFAAwF;AACxF,MAAM,MAAM,YAAY,CAAC,CAAC,IAAI;IAC5B,QAAQ,EAAE,CAAC,IAAI,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,SAAS,MAAM,GAAG,YAAY,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC;CACzE,CAAC;AAEF,yEAAyE;AACzE,UAAU,kBAAkB;IAC1B,KAAK,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAC3B,WAAW,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;IAClC,gBAAgB,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACtC,UAAU,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAChC,WAAW,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;CAClC;AAED,MAAM,MAAM,eAAe,GAAG,kBAAkB,GAAG,CAC/C;IACA,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAC7B,gBAAgB,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;CACvC,GACC;IACA,OAAO,EAAE,MAAM,CAAC;IAChB,SAAS,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAC/B,gBAAgB,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;CACvC,CACF,CAAC;AASF,MAAM,WAAW,gBAAgB;IAC/B,iBAAiB,CAAC,EAAE,OAAO,GAAG,SAAS,CAAC;CACzC;AAED,0DAA0D;AAC1D,MAAM,MAAM,oBAAoB,CAAC,CAAC,SAAS,SAAS,IAAI,CAAC,QAAQ,EAAE,WAAW,CAAC,CAAC,CAAC,EAAE,QAAQ,EAAE,WAAW,CAAC,CAAC,CAAC,KAAK,IAAI,CAAC;AAErH,gEAAgE;AAChE,MAAM,MAAM,iBAAiB,GAAG,MAAM,IAAI,CAAC;AA0C3C;;;;;GAKG;AACH,qBAAa,aAAa;IACxB,OAAO,CAAC,MAAM,CAAkB;IAChC,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAS;IACnC,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAS;IACpC,OAAO,CAAC,QAAQ,CAAC,iBAAiB,CAAgB;IAClD,OAAO,CAAC,QAAQ,CAAC,gBAAgB,CAAgB;IACjD,OAAO,CAAC,QAAQ,CAAC,aAAa,CAAgB;IAC9C,OAAO,CAAC,cAAc,CAA6C;IACnE,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAsE;gBAErF,SAAS,EAAE,eAAe;IAyCtC,wBAAwB,IAAI,MAAM;IAIlC,mBAAmB,IAAI,MAAM,GAAG,IAAI;IAIpC,gBAAgB,IAAI,MAAM,GAAG,IAAI;IAIjC,oBAAoB,CAAC,cAAc,EAAE,IAAI,CAAC,cAAc,EAAE,MAAM,CAAC,GAAG,IAAI,GAAG,IAAI;IAI/E,OAAO,CAAC,WAAW;IAwBnB,0CAA0C;IAC1C,GAAG,CAAC,CAAC,SAAS,SAAS,EAAE,GAAG,EAAE,CAAC,GAAG,WAAW,CAAC,CAAC,CAAC;IAKhD,gEAAgE;IAChE,GAAG,CAAC,CAAC,SAAS,SAAS,EAAE,GAAG,EAAE,CAAC,EAAE,KAAK,EAAE,WAAW,CAAC,CAAC,CAAC,EAAE,OAAO,GAAE,gBAAqB,GAAG,IAAI;IA4B7F;;;OAGG;IACH,SAAS,CAAC,CAAC,SAAS,SAAS,EAAE,GAAG,EAAE,CAAC,EAAE,EAAE,EAAE,oBAAoB,CAAC,CAAC,CAAC,GAAG,iBAAiB;IAatF,sDAAsD;IACtD,OAAO,CAAC,eAAe;IAevB;;OAEG;IACH,OAAO,CAAC,cAAc;IA0BtB;;;;OAIG;IACH,UAAU,CAAC,GAAG,EAAE,SAAS,EAAE,KAAK,EAAE,OAAO,EAAE,OAAO,GAAE,gBAAqB,GAAG,IAAI;IAIhF,wFAAwF;IACxF,MAAM,IAAI,YAAY,CAAC,eAAe,CAAC;IAIvC,0DAA0D;IAC1D,WAAW,CAAC,CAAC,SAAS,MAAM,eAAe,EAAE,QAAQ,EAAE,CAAC,GAAG,QAAQ,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC;IAIvF,+FAA+F;IAC/F,MAAM,IAAI,QAAQ,CAAC,eAAe,CAAC;IAInC,8BAA8B;IAC9B,SAAS,IAAI,aAAa,EAAE;IAI5B,0DAA0D;IAC1D,IAAI,IAAI,IAAI;IAKZ,yEAAyE;IACzE,WAAW,IAAI,IAAI;IAQnB,4FAA4F;IAC5F,IAAI,IAAI,IAAI;IAyBZ;;;;;;OAMG;IACH,aAAa,CAAC,CAAC,SAAS,MAAM,eAAe,EAAE,QAAQ,EAAE,CAAC,EAAE,KAAK,EAAE,OAAO,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC,GAAG,IAAI;IAWrG;;;OAGG;IACH,KAAK,CAAC,GAAG,CAAC,EAAE,SAAS,GAAG,IAAI;CAY7B"}

package/dist/platform/config/manager.js

@@ -14,10 +14,6 @@ function cloneDefaultConfig() {
 }
 function sanitizeConfigShape(config) {
     const sanitized = structuredClone(config);
-    const lineNumbers = sanitized.display.lineNumbers;
-    if (typeof lineNumbers === 'boolean') {
-        sanitized.display.lineNumbers = lineNumbers ? 'all' : 'off';
-    }
     for (const key of Object.keys(sanitized.permissions.tools)) {
         if (!PERMISSION_TOOL_KEYS.has(key)) {
             delete sanitized.permissions.tools[key];
@@ -44,12 +40,7 @@ function requireAbsoluteOwnedPath(path, name) {
 function ensureSharedConfig(sharedPath) {
     if (!existsSync(sharedPath)) {
         mkdirSync(dirname(sharedPath), { recursive: true });
-        try {
-            writeFileSync(sharedPath, '{}\n', 'utf-8');
-        }
-        catch (err) {
-            logger.debug('Could not create shared surface config (non-fatal)', { error: summarizeError(err) });
-        }
+        writeFileSync(sharedPath, '{}\n', 'utf-8');
     }
 }
 /**
@@ -95,9 +86,6 @@ export class ConfigManager {
         if (overrides.model !== undefined) {
             this.config.provider.model = overrides.model;
         }
-        if (overrides.provider !== undefined) {
-            this.config.provider.provider = overrides.provider;
-        }
         if (overrides.autoApprove !== undefined) {
             this.config.behavior.autoApprove = overrides.autoApprove;
         }
@@ -158,7 +146,13 @@ export class ConfigManager {
         const { parent, field } = this.resolvePath(key);
         const previousValue = parent[field];
         parent[field] = value;
-        this.save();
+        try {
+            this.save();
+        }
+        catch (error) {
+            parent[field] = previousValue;
+            throw error;
+        }
         this.notifyListeners(key, previousValue, value);
         this.emitConfigHook(key, previousValue, value);
     }
@@ -173,7 +167,7 @@ export class ConfigManager {
         // Cast via unknown -> (n: unknown, o: unknown) => void to avoid deeply-recursive ConfigValue<K> comparison
         // that exceeds TypeScript's stack depth limit on the 100-entry conditional type.
         const wrapped = (newVal, oldVal) => cb(newVal, oldVal);
-        this._listeners.get(key).add(wrapped);
+        this._listeners.get(key)?.add(wrapped);
         return () => {
             this._listeners.get(key)?.delete(wrapped);
         };
@@ -251,26 +245,16 @@ export class ConfigManager {
     }
     /** Persist current config to global TUI settings file. */
     save() {
-        try {
-            mkdirSync(dirname(this.configPath), { recursive: true });
-            writeFileSync(this.configPath, JSON.stringify(this.config, null, 2) + '\n', 'utf-8');
-        }
-        catch (err) {
-            logger.debug('Config save failed (non-fatal)', { error: summarizeError(err) });
-        }
+        mkdirSync(dirname(this.configPath), { recursive: true });
+        writeFileSync(this.configPath, JSON.stringify(this.config, null, 2) + '\n', 'utf-8');
     }
     /** Persist current config to the project-level surface settings file. */
     saveProject() {
         if (!this.projectConfigPath) {
             throw new Error('ConfigManager.saveProject requires an explicit workingDir.');
         }
-        try {
-            mkdirSync(dirname(this.projectConfigPath), { recursive: true });
-            writeFileSync(this.projectConfigPath, JSON.stringify(this.config, null, 2) + '\n', 'utf-8');
-        }
-        catch (err) {
-            logger.debug('Project config save failed (non-fatal)', { error: summarizeError(err) });
-        }
+        mkdirSync(dirname(this.projectConfigPath), { recursive: true });
+        writeFileSync(this.projectConfigPath, JSON.stringify(this.config, null, 2) + '\n', 'utf-8');
     }
     /** Load config from disk: global then project (project wins). Deep-merges with defaults. */
     load() {
@@ -282,7 +266,7 @@ export class ConfigManager {
                 this.config = sanitizeConfigShape(deepMerge(cloneDefaultConfig(), parsed));
             }
             catch (err) {
-                logger.debug('Global config load failed (non-fatal, using defaults)', { error: summarizeError(err) });
+                throw new ConfigError(`Global config load failed for ${this.configPath}: ${summarizeError(err)}`);
             }
         }
         // Load project settings and deep-merge on top (project wins)
@@ -293,7 +277,7 @@ export class ConfigManager {
                 this.config = sanitizeConfigShape(deepMerge(this.config, parsed));
             }
             catch (err) {
-                logger.debug('Project config load failed (non-fatal)', { error: summarizeError(err) });
+                throw new ConfigError(`Project config load failed for ${this.projectConfigPath}: ${summarizeError(err)}`);
             }
         }
     }

package/dist/platform/config/oauth-local-listener.d.ts

@@ -1,9 +1,9 @@
 export interface OAuthLocalListenerConfig {
     readonly expectedState: string;
-    readonly host?: string;
-    readonly port?: number;
-    readonly path?: string;
-    readonly timeoutMs?: number;
+    readonly host?: string | undefined;
+    readonly port?: number | undefined;
+    readonly path?: string | undefined;
+    readonly timeoutMs?: number | undefined;
 }
 export interface OAuthLocalListenerResult {
     readonly code: string;

package/dist/platform/config/oauth-local-listener.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"oauth-local-listener.d.ts","sourceRoot":"","sources":["../../../src/platform/config/oauth-local-listener.ts"],"names":[],"mappings":"AAIA,MAAM,WAAW,wBAAwB;IACvC,QAAQ,CAAC,aAAa,EAAE,MAAM,CAAC;IAC/B,QAAQ,CAAC,IAAI,CAAC,EAAE,MAAM,CAAC;IACvB,QAAQ,CAAC,IAAI,CAAC,EAAE,MAAM,CAAC;IACvB,QAAQ,CAAC,IAAI,CAAC,EAAE,MAAM,CAAC;IACvB,QAAQ,CAAC,SAAS,CAAC,EAAE,MAAM,CAAC;CAC7B;AAED,MAAM,WAAW,wBAAwB;IACvC,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;CACvB;AAED,MAAM,WAAW,kBAAkB;IACjC,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;IAC7B,gBAAgB,CAAC,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;IACtC,WAAW,IAAI,OAAO,CAAC,wBAAwB,CAAC,CAAC;IACjD,KAAK,IAAI,IAAI,CAAC;CACf;AAUD,wBAAsB,wBAAwB,CAAC,MAAM,EAAE,wBAAwB,GAAG,OAAO,CAAC,kBAAkB,CAAC,CA+H5G"}
+{"version":3,"file":"oauth-local-listener.d.ts","sourceRoot":"","sources":["../../../src/platform/config/oauth-local-listener.ts"],"names":[],"mappings":"AAIA,MAAM,WAAW,wBAAwB;IACvC,QAAQ,CAAC,aAAa,EAAE,MAAM,CAAC;IAC/B,QAAQ,CAAC,IAAI,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACnC,QAAQ,CAAC,IAAI,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACnC,QAAQ,CAAC,IAAI,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACnC,QAAQ,CAAC,SAAS,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;CACzC;AAED,MAAM,WAAW,wBAAwB;IACvC,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;CACvB;AAED,MAAM,WAAW,kBAAkB;IACjC,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;IAC7B,gBAAgB,CAAC,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;IACtC,WAAW,IAAI,OAAO,CAAC,wBAAwB,CAAC,CAAC;IACjD,KAAK,IAAI,IAAI,CAAC;CACf;AAUD,wBAAsB,wBAAwB,CAAC,MAAM,EAAE,wBAAwB,GAAG,OAAO,CAAC,kBAAkB,CAAC,CA+H5G"}

package/dist/platform/config/openai-codex-auth.d.ts

@@ -13,7 +13,7 @@ export interface OpenAICodexTokenResult {
     readonly refreshToken: string;
     readonly tokenType: string;
     readonly expiresAt: number;
-    readonly scopes?: readonly string[];
+    readonly scopes?: readonly string[] | undefined;
 }
 export declare function beginOpenAICodexLogin(): Promise<OpenAICodexLoginStart>;
 export declare function exchangeOpenAICodexCode(code: string, verifier: string): Promise<OpenAICodexTokenResult>;

package/dist/platform/config/openai-codex-auth.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"openai-codex-auth.d.ts","sourceRoot":"","sources":["../../../src/platform/config/openai-codex-auth.ts"],"names":[],"mappings":"AAMA,eAAO,MAAM,sBAAsB,iCAAiC,CAAC;AACrE,eAAO,MAAM,0BAA0B,4CAA4C,CAAC;AACpF,eAAO,MAAM,sBAAsB,wCAAwC,CAAC;AAC5E,eAAO,MAAM,yBAAyB,wCAAwC,CAAC;AAG/E,MAAM,WAAW,qBAAqB;IACpC,QAAQ,CAAC,gBAAgB,EAAE,MAAM,CAAC;IAClC,QAAQ,CAAC,KAAK,EAAE,MAAM,CAAC;IACvB,QAAQ,CAAC,QAAQ,EAAE,MAAM,CAAC;IAC1B,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;CAC9B;AAED,MAAM,WAAW,sBAAsB;IACrC,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;IAC7B,QAAQ,CAAC,YAAY,EAAE,MAAM,CAAC;IAC9B,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAC;IAC3B,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAC;IAC3B,QAAQ,CAAC,MAAM,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;CACrC;AAED,wBAAsB,qBAAqB,IAAI,OAAO,CAAC,qBAAqB,CAAC,CAsB5E;AAED,wBAAsB,uBAAuB,CAAC,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,sBAAsB,CAAC,CAyB7G;AAED,wBAAsB,uBAAuB,CAAC,YAAY,EAAE,MAAM,GAAG,OAAO,CAAC,sBAAsB,CAAC,CAqBnG"}
+{"version":3,"file":"openai-codex-auth.d.ts","sourceRoot":"","sources":["../../../src/platform/config/openai-codex-auth.ts"],"names":[],"mappings":"AAMA,eAAO,MAAM,sBAAsB,iCAAiC,CAAC;AACrE,eAAO,MAAM,0BAA0B,4CAA4C,CAAC;AACpF,eAAO,MAAM,sBAAsB,wCAAwC,CAAC;AAC5E,eAAO,MAAM,yBAAyB,wCAAwC,CAAC;AAG/E,MAAM,WAAW,qBAAqB;IACpC,QAAQ,CAAC,gBAAgB,EAAE,MAAM,CAAC;IAClC,QAAQ,CAAC,KAAK,EAAE,MAAM,CAAC;IACvB,QAAQ,CAAC,QAAQ,EAAE,MAAM,CAAC;IAC1B,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;CAC9B;AAED,MAAM,WAAW,sBAAsB;IACrC,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;IAC7B,QAAQ,CAAC,YAAY,EAAE,MAAM,CAAC;IAC9B,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAC;IAC3B,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAC;IAC3B,QAAQ,CAAC,MAAM,CAAC,EAAE,SAAS,MAAM,EAAE,GAAG,SAAS,CAAC;CACjD;AAED,wBAAsB,qBAAqB,IAAI,OAAO,CAAC,qBAAqB,CAAC,CAsB5E;AAED,wBAAsB,uBAAuB,CAAC,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,sBAAsB,CAAC,CAyB7G;AAED,wBAAsB,uBAAuB,CAAC,YAAY,EAAE,MAAM,GAAG,OAAO,CAAC,sBAAsB,CAAC,CAqBnG"}

package/dist/platform/config/schema-domain-core.d.ts

@@ -13,7 +13,6 @@ export declare const coreConfigDefaults: {
     provider: {
         reasoningEffort: string;
         model: string;
-        provider: string;
         embeddingProvider: string;
         systemPromptFile: string;
     };

package/dist/platform/config/schema-domain-core.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"schema-domain-core.d.ts","sourceRoot":"","sources":["../../../src/platform/config/schema-domain-core.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,uBAAuB,EAAE,MAAM,oBAAoB,CAAC;AAElE,eAAO,MAAM,kBAAkB;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CA6H9B,CAAC;AAEF,eAAO,MAAM,sBAAsB,EAAE,uBAAuB,EAkZ3D,CAAC;AAEF,eAAO,MAAM,sBAAsB,EAAE,uBAAuB,EA+H3D,CAAC"}
+{"version":3,"file":"schema-domain-core.d.ts","sourceRoot":"","sources":["../../../src/platform/config/schema-domain-core.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,uBAAuB,EAAE,MAAM,oBAAoB,CAAC;AAElE,eAAO,MAAM,kBAAkB;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;CA4H9B,CAAC;AAEF,eAAO,MAAM,sBAAsB,EAAE,uBAAuB,EAiZ3D,CAAC;AAEF,eAAO,MAAM,sBAAsB,EAAE,uBAAuB,EA+H3D,CAAC"}

package/dist/platform/config/schema-domain-core.js

@@ -11,8 +11,7 @@ export const coreConfigDefaults = {
     },
     provider: {
         reasoningEffort: 'medium',
-        model: 'openrouter/free',
-        provider: 'openrouter',
+        model: 'openrouter:openrouter/free',
         embeddingProvider: 'hashed-local',
         systemPromptFile: '',
     },
@@ -185,14 +184,14 @@ export const coreHeadConfigSettings = [
     {
         key: 'provider.model',
         type: 'string',
-        default: 'openrouter/free',
-        description: 'Default LLM model ID',
-    },
-    {
-        key: 'provider.provider',
-        type: 'string',
-        default: 'openrouter',
-        description: 'Default provider name',
+        default: 'openrouter:openrouter/free',
+        description: 'Default provider-qualified LLM model registry key',
+        validate: (v) => {
+            if (typeof v !== 'string')
+                return false;
+            const separator = v.trim().indexOf(':');
+            return separator > 0 && separator < v.trim().length - 1;
+        },
     },
     {
         key: 'provider.embeddingProvider',

package/dist/platform/config/schema-shared.d.ts

@@ -4,7 +4,7 @@ export interface ConfigSettingDefinition {
     type: 'boolean' | 'number' | 'string' | 'enum';
     default: unknown;
     description: string;
-    enumValues?: string[];
-    validate?: (value: unknown) => boolean;
+    enumValues?: string[] | undefined;
+    validate?: ((value: unknown) => boolean) | undefined;
 }
 //# sourceMappingURL=schema-shared.d.ts.map

package/dist/platform/config/schema-shared.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"schema-shared.d.ts","sourceRoot":"","sources":["../../../src/platform/config/schema-shared.ts"],"names":[],"mappings":"AAAA,qFAAqF;AAErF,MAAM,WAAW,uBAAuB;IACtC,GAAG,EAAE,MAAM,CAAC;IACZ,IAAI,EAAE,SAAS,GAAG,QAAQ,GAAG,QAAQ,GAAG,MAAM,CAAC;IAC/C,OAAO,EAAE,OAAO,CAAC;IACjB,WAAW,EAAE,MAAM,CAAC;IACpB,UAAU,CAAC,EAAE,MAAM,EAAE,CAAC;IACtB,QAAQ,CAAC,EAAE,CAAC,KAAK,EAAE,OAAO,KAAK,OAAO,CAAC;CACxC"}
+{"version":3,"file":"schema-shared.d.ts","sourceRoot":"","sources":["../../../src/platform/config/schema-shared.ts"],"names":[],"mappings":"AAAA,qFAAqF;AAErF,MAAM,WAAW,uBAAuB;IACtC,GAAG,EAAE,MAAM,CAAC;IACZ,IAAI,EAAE,SAAS,GAAG,QAAQ,GAAG,QAAQ,GAAG,MAAM,CAAC;IAC/C,OAAO,EAAE,OAAO,CAAC;IACjB,WAAW,EAAE,MAAM,CAAC;IACpB,UAAU,CAAC,EAAE,MAAM,EAAE,GAAG,SAAS,CAAC;IAClC,QAAQ,CAAC,EAAE,CAAC,CAAC,KAAK,EAAE,OAAO,KAAK,OAAO,CAAC,GAAG,SAAS,CAAC;CACtD"}