@realtimex/email-automator 2.11.0 → 2.11.1

package/api/src/routes/sdk.ts

@@ -1,6 +1,7 @@
 import { Router, Request, Response } from 'express';
 import { SDKService } from '../services/SDKService.js';
 import { ProvidersResponse } from '@realtimex/sdk';
+import { apiRateLimit } from '../middleware/rateLimit.js';
 
 const router = Router();
 
@@ -8,7 +9,7 @@ const router = Router();
  * GET /api/sdk/providers/chat
  * Returns available chat providers and their models
  */
-router.get('/providers/chat', async (req: Request, res: Response) => {
+router.get('/providers/chat', apiRateLimit, async (req: Request, res: Response) => {
     try {
         const sdk = SDKService.getSDK();
         if (!sdk) {
@@ -23,7 +24,6 @@ router.get('/providers/chat', async (req: Request, res: Response) => {
 
         res.json({ success: true, providers: providers || [] });
     } catch (error: any) {
-        console.warn('[SDK API] Failed to fetch chat providers:', error.message);
         res.json({ success: false, providers: [], message: error.message });
     }
 });
@@ -32,7 +32,7 @@ router.get('/providers/chat', async (req: Request, res: Response) => {
  * GET /api/sdk/providers/embed
  * Returns available embedding providers and their models
  */
-router.get('/providers/embed', async (req: Request, res: Response) => {
+router.get('/providers/embed', apiRateLimit, async (req: Request, res: Response) => {
     try {
         const sdk = SDKService.getSDK();
         if (!sdk) {
@@ -47,7 +47,6 @@ router.get('/providers/embed', async (req: Request, res: Response) => {
 
         res.json({ success: true, providers: providers || [] });
     } catch (error: any) {
-        console.warn('[SDK API] Failed to fetch embed providers:', error.message);
         res.json({ success: false, providers: [], message: error.message });
     }
 });

package/api/src/routes/settings.ts

@@ -165,10 +165,14 @@ router.post('/test-llm',
     apiRateLimit,
     authMiddleware,
     asyncHandler(async (req, res) => {
+        const { llm_provider, llm_model } = req.body;
         const { getIntelligenceService } = await import('../services/intelligence.js');
         const intelligence = getIntelligenceService();
 
-        const result = await intelligence.testConnection();
+        const result = await intelligence.testConnection({
+            provider: llm_provider,
+            model: llm_model
+        });
         res.json(result);
     })
 );

package/api/src/services/SDKService.ts

@@ -1,6 +1,15 @@
 import { RealtimeXSDK, ProvidersResponse } from '@realtimex/sdk';
 import os from 'os';
 import path from 'path';
+import { createLogger } from '../utils/logger.js';
+
+const logger = createLogger('SDKService');
+
+export interface ProviderResult {
+    provider: string;
+    model: string;
+    isDefaultFallback?: boolean;
+}
 
 /**
  * Centralized SDK Service
@@ -10,6 +19,13 @@ export class SDKService {
     private static instance: RealtimeXSDK | null = null;
     private static initAttempted = false;
 
+    // Default provider/model configuration
+    // realtimexai routes through RealTimeX Desktop to user's configured providers
+    static readonly DEFAULT_LLM_PROVIDER = 'realtimexai';
+    static readonly DEFAULT_LLM_MODEL = 'gpt-4o-mini';
+    static readonly DEFAULT_EMBED_PROVIDER = 'realtimexai';
+    static readonly DEFAULT_EMBED_MODEL = 'text-embedding-3-small';
+
     /**
      * Initialize SDK with required permissions
      * Safe to call multiple times - returns existing instance
@@ -39,16 +55,16 @@ export class SDKService {
                     ],
                 });
 
-                console.log('[SDKService] RealTimeX SDK initialized successfully');
+                logger.info('RealTimeX SDK initialized successfully');
 
                 // Verify connection with Desktop App
                 // @ts-ignore - Dev Mode feature
                 this.instance.ping()
-                    .then((status: any) => console.log('[SDKService] Desktop App Connection:', status))
-                    .catch((err: any) => console.warn('[SDKService] Desktop App Connection failed:', err.message));
+                    .then((status: any) => logger.debug('Desktop App Connection:', { status }))
+                    .catch((err: any) => logger.warn('Desktop App Connection failed', { error: err.message }));
 
             } catch (error: any) {
-                console.warn('[SDKService] Failed to initialize SDK:', error.message);
+                logger.error('Failed to initialize SDK', error);
                 this.instance = null;
             }
         }
@@ -84,8 +100,8 @@ export class SDKService {
                 await sdk.llm.chatProviders();
                 return true;
             }
-        } catch (error) {
-            console.warn('[SDKService] SDK not available:', error);
+        } catch (error: any) {
+            logger.warn('SDK not available', { error: error.message });
             return false;
         }
     }
@@ -108,14 +124,14 @@ export class SDKService {
     }
 
     // Cache for default providers (avoid repeated SDK calls)
-    private static defaultChatProvider: { provider: string; model: string } | null = null;
-    private static defaultEmbedProvider: { provider: string; model: string } | null = null;
+    private static defaultChatProvider: ProviderResult | null = null;
+    private static defaultEmbedProvider: ProviderResult | null = null;
 
     /**
      * Get default chat provider/model from SDK dynamically
      * Caches result to avoid repeated SDK calls
      */
-    static async getDefaultChatProvider(): Promise<{ provider: string; model: string }> {
+    static async getDefaultChatProvider(): Promise<ProviderResult> {
         // Return cached if available
         if (this.defaultChatProvider) {
             return this.defaultChatProvider;
@@ -137,22 +153,40 @@ export class SDKService {
                 throw new Error('No LLM providers available. Please configure a provider in RealTimeX Desktop.');
             }
 
-            // Find first provider with available models
+            // 1. Try to find the preferred default provider (realtimexai)
+            const preferredProvider = providers.find(p => p.provider === this.DEFAULT_LLM_PROVIDER);
+            if (preferredProvider && preferredProvider.models && preferredProvider.models.length > 0) {
+                // Try to find the preferred default model (gpt-4o-mini) within it
+                const preferredModel = preferredProvider.models.find(m => m.id === this.DEFAULT_LLM_MODEL) || preferredProvider.models[0];
+
+                this.defaultChatProvider = {
+                    provider: preferredProvider.provider,
+                    model: preferredModel.id
+                };
+                logger.info(`Using preferred default chat provider: ${this.defaultChatProvider.provider}/${this.defaultChatProvider.model}`);
+                return this.defaultChatProvider;
+            }
+
+            // 2. Fallback to the first provider with available models
             for (const p of providers) {
                 if (p.models && p.models.length > 0) {
                     this.defaultChatProvider = {
                         provider: p.provider,
                         model: p.models[0].id
                     };
-                    console.log(`[SDKService] Default chat provider: ${this.defaultChatProvider.provider}/${this.defaultChatProvider.model}`);
+                    logger.info(`Defaulting to first available chat provider: ${this.defaultChatProvider.provider}/${this.defaultChatProvider.model}`);
                     return this.defaultChatProvider;
                 }
             }
 
             throw new Error('No LLM models available. Please configure a model in RealTimeX Desktop.');
         } catch (error: any) {
-            console.error('[SDKService] Failed to get default chat provider:', error.message);
-            throw error;
+            logger.warn('Failed to get default chat provider from SDK. Using hardcoded fallback.', error);
+            return {
+                provider: this.DEFAULT_LLM_PROVIDER,
+                model: this.DEFAULT_LLM_MODEL,
+                isDefaultFallback: true
+            };
         }
     }
 
@@ -160,7 +194,7 @@ export class SDKService {
      * Get default embedding provider/model from SDK dynamically
      * Caches result to avoid repeated SDK calls
      */
-    static async getDefaultEmbedProvider(): Promise<{ provider: string; model: string }> {
+    static async getDefaultEmbedProvider(): Promise<ProviderResult> {
         // Return cached if available
         if (this.defaultEmbedProvider) {
             return this.defaultEmbedProvider;
@@ -189,61 +223,46 @@ export class SDKService {
                         provider: p.provider,
                         model: p.models[0].id
                     };
-                    console.log(`[SDKService] Default embed provider: ${this.defaultEmbedProvider.provider}/${this.defaultEmbedProvider.model}`);
+                    logger.info(`Default embed provider: ${this.defaultEmbedProvider.provider}/${this.defaultEmbedProvider.model}`);
                     return this.defaultEmbedProvider;
                 }
             }
 
             throw new Error('No embedding models available. Please configure a model in RealTimeX Desktop.');
         } catch (error: any) {
-            console.error('[SDKService] Failed to get default embed provider:', error.message);
-            throw error;
+            logger.warn('Failed to get default embed provider from SDK. Using hardcoded fallback.', error);
+            return {
+                provider: this.DEFAULT_EMBED_PROVIDER,
+                model: this.DEFAULT_EMBED_MODEL,
+                isDefaultFallback: true
+            };
         }
     }
 
-    // Default provider/model configuration
-    // realtimexai routes through RealTimeX Desktop to user's configured providers
-    static readonly DEFAULT_LLM_PROVIDER = 'realtimexai';
-    static readonly DEFAULT_LLM_MODEL = 'gpt-4o-mini';
-    static readonly DEFAULT_EMBED_PROVIDER = 'realtimexai';
-    static readonly DEFAULT_EMBED_MODEL = 'text-embedding-3-small';
-
     /**
      * Resolve LLM provider/model - use settings if available, otherwise use defaults
      */
-    static async resolveChatProvider(settings: { llm_provider?: string; llm_model?: string }): Promise<{ provider: string; model: string }> {
+    static async resolveChatProvider(settings: { llm_provider?: string; llm_model?: string }): Promise<ProviderResult> {
         // If both provider and model are set in settings, use them
         if (settings.llm_provider && settings.llm_model) {
             return { provider: settings.llm_provider, model: settings.llm_model };
         }
 
         // Try to get from SDK discovery first
-        try {
-            return await this.getDefaultChatProvider();
-        } catch (error) {
-            // Fallback to hardcoded defaults if SDK discovery fails
-            console.log(`[SDKService] Using default LLM: ${this.DEFAULT_LLM_PROVIDER}/${this.DEFAULT_LLM_MODEL}`);
-            return { provider: this.DEFAULT_LLM_PROVIDER, model: this.DEFAULT_LLM_MODEL };
-        }
+        return await this.getDefaultChatProvider();
     }
 
     /**
      * Resolve embedding provider/model - use settings if available, otherwise use defaults
      */
-    static async resolveEmbedProvider(settings: { embedding_provider?: string; embedding_model?: string }): Promise<{ provider: string; model: string }> {
+    static async resolveEmbedProvider(settings: { embedding_provider?: string; embedding_model?: string }): Promise<ProviderResult> {
         // If both provider and model are set in settings, use them
         if (settings.embedding_provider && settings.embedding_model) {
             return { provider: settings.embedding_provider, model: settings.embedding_model };
         }
 
         // Try to get from SDK discovery first
-        try {
-            return await this.getDefaultEmbedProvider();
-        } catch (error) {
-            // Fallback to hardcoded defaults if SDK discovery fails
-            console.log(`[SDKService] Using default embedding: ${this.DEFAULT_EMBED_PROVIDER}/${this.DEFAULT_EMBED_MODEL}`);
-            return { provider: this.DEFAULT_EMBED_PROVIDER, model: this.DEFAULT_EMBED_MODEL };
-        }
+        return await this.getDefaultEmbedProvider();
     }
 
     /**

package/api/src/services/intelligence.ts

@@ -91,9 +91,7 @@ export class IntelligenceService {
         return this.isConfigured && !!SDKService.getSDK();
     }
 
-    async analyzeEmail(content: string, context: EmailContext, eventLogger?: EventLogger, emailId?: string): Promise<EmailAnalysis | null> {
-        console.log('[Intelligence] analyzeEmail called for:', context.subject);
-
+    async analyzeEmail(content: string, context: EmailContext, eventLogger?: EventLogger, emailId?: string): Promise<(EmailAnalysis & { _metadata?: any }) | null> {
         const sdk = SDKService.getSDK();
         if (!sdk) {
             logger.warn('Intelligence service not ready, skipping analysis');
@@ -103,7 +101,7 @@ export class IntelligenceService {
             return null;
         }
 
-        const { provider, model } = await SDKService.resolveChatProvider({});
+        const { provider, model, isDefaultFallback } = await SDKService.resolveChatProvider({});
 
         const cleanedContent = ContentCleaner.cleanEmailBody(content).substring(0, 2500);
 
@@ -138,6 +136,7 @@ REQUIRED JSON STRUCTURE:
             await eventLogger.info('Thinking', `Analyzing email: ${context.subject}`, {
                 model,
                 provider,
+                is_fallback: isDefaultFallback,
                 content_preview: cleanedContent
             }, emailId);
         }
@@ -151,14 +150,24 @@ REQUIRED JSON STRUCTURE:
             const rawResponse = response.response?.content || '';
             const validated = this.parseRobustJSON<EmailAnalysis>(rawResponse, EmailAnalysisSchema);
 
-            if (eventLogger && emailId && validated) {
+            const result = validated ? {
+                ...validated,
+                _metadata: {
+                    provider,
+                    model,
+                    is_fallback: isDefaultFallback,
+                    timestamp: new Date().toISOString()
+                }
+            } : null;
+
+            if (eventLogger && emailId && result) {
                 await eventLogger.analysis('Decided', emailId, {
-                    ...validated,
+                    ...result,
                     _raw_response: rawResponse
                 });
             }
 
-            return validated;
+            return result;
         } catch (error: any) {
             logger.error('Analysis failed', error);
             if (eventLogger) await eventLogger.error('Error', error.message, emailId);
@@ -200,11 +209,11 @@ REQUIRED JSON STRUCTURE:
         compiledRulesContext: string | RuleContext[],
         eventLogger?: EventLogger,
         emailId?: string
-    ): Promise<ContextAwareAnalysis | null> {
+    ): Promise<(ContextAwareAnalysis & { _metadata?: any }) | null> {
         const sdk = SDKService.getSDK();
         if (!sdk) return null;
 
-        const { provider, model } = await SDKService.resolveChatProvider({});
+        const { provider, model, isDefaultFallback } = await SDKService.resolveChatProvider({});
         const cleanedContent = ContentCleaner.cleanEmailBody(content).substring(0, 2500);
 
         let rulesContext: string;
@@ -217,7 +226,11 @@ REQUIRED JSON STRUCTURE:
         const systemPrompt = `You are an AI Automation Agent. Match email against these rules:\n${rulesContext}\n\nReturn JSON with matched_rule, actions_to_execute, and draft_content.`;
 
         if (eventLogger) {
-            await eventLogger.info('Thinking', `Context-aware analysis: ${context.subject}`, { model, provider }, emailId);
+            await eventLogger.info('Thinking', `Context-aware analysis: ${context.subject}`, {
+                model,
+                provider,
+                is_fallback: isDefaultFallback
+            }, emailId);
         }
 
         try {
@@ -229,14 +242,24 @@ REQUIRED JSON STRUCTURE:
             const rawResponse = response.response?.content || '';
             const validated = this.parseRobustJSON<ContextAwareAnalysis>(rawResponse, ContextAwareAnalysisSchema);
 
-            if (eventLogger && emailId && validated) {
+            const result = validated ? {
+                ...validated,
+                _metadata: {
+                    provider,
+                    model,
+                    is_fallback: isDefaultFallback,
+                    timestamp: new Date().toISOString()
+                }
+            } : null;
+
+            if (eventLogger && emailId && result) {
                 await eventLogger.analysis('Decided', emailId, {
-                    ...validated,
+                    ...result,
                     _raw_response: rawResponse
                 });
             }
 
-            return validated;
+            return result;
         } catch (error: any) {
             logger.error('Rule analysis failed', error);
             if (eventLogger) await eventLogger.error('Error', error.message, emailId);
@@ -244,12 +267,15 @@ REQUIRED JSON STRUCTURE:
         }
     }
 
-    async testConnection(): Promise<{ success: boolean; message: string }> {
+    async testConnection(overrides?: { provider?: string; model?: string }): Promise<{ success: boolean; message: string }> {
         const sdk = SDKService.getSDK();
         if (!sdk) return { success: false, message: 'SDK not linked' };
 
         try {
-            const { provider, model } = await SDKService.resolveChatProvider({});
+            const { provider, model } = await SDKService.resolveChatProvider({
+                llm_provider: overrides?.provider,
+                llm_model: overrides?.model
+            });
             await sdk.llm.chat([{ role: 'user', content: 'Say OK' }], { provider, model });
             return { success: true, message: `Connected to ${provider}/${model}` };
         } catch (error: any) {
@@ -272,7 +298,7 @@ REQUIRED JSON STRUCTURE:
 
 let defaultInstance: IntelligenceService | null = null;
 
-export function getIntelligenceService(overrides?: any): IntelligenceService {
+export function getIntelligenceService(): IntelligenceService {
     if (!defaultInstance) {
         defaultInstance = new IntelligenceService();
     }

package/dist/api/src/routes/sdk.js

@@ -1,11 +1,12 @@
 import { Router } from 'express';
 import { SDKService } from '../services/SDKService.js';
+import { apiRateLimit } from '../middleware/rateLimit.js';
 const router = Router();
 /**
  * GET /api/sdk/providers/chat
  * Returns available chat providers and their models
  */
-router.get('/providers/chat', async (req, res) => {
+router.get('/providers/chat', apiRateLimit, async (req, res) => {
     try {
         const sdk = SDKService.getSDK();
         if (!sdk) {
@@ -15,7 +16,6 @@ router.get('/providers/chat', async (req, res) => {
         res.json({ success: true, providers: providers || [] });
     }
     catch (error) {
-        console.warn('[SDK API] Failed to fetch chat providers:', error.message);
         res.json({ success: false, providers: [], message: error.message });
     }
 });
@@ -23,7 +23,7 @@ router.get('/providers/chat', async (req, res) => {
  * GET /api/sdk/providers/embed
  * Returns available embedding providers and their models
  */
-router.get('/providers/embed', async (req, res) => {
+router.get('/providers/embed', apiRateLimit, async (req, res) => {
     try {
         const sdk = SDKService.getSDK();
         if (!sdk) {
@@ -33,7 +33,6 @@ router.get('/providers/embed', async (req, res) => {
         res.json({ success: true, providers: providers || [] });
     }
     catch (error) {
-        console.warn('[SDK API] Failed to fetch embed providers:', error.message);
         res.json({ success: false, providers: [], message: error.message });
     }
 });

package/dist/api/src/routes/settings.js

@@ -129,9 +129,13 @@ router.patch('/', apiRateLimit, authMiddleware, validateBody(schemas.updateSetti
 }));
 // Test LLM Connection
 router.post('/test-llm', apiRateLimit, authMiddleware, asyncHandler(async (req, res) => {
+    const { llm_provider, llm_model } = req.body;
     const { getIntelligenceService } = await import('../services/intelligence.js');
     const intelligence = getIntelligenceService();
-    const result = await intelligence.testConnection();
+    const result = await intelligence.testConnection({
+        provider: llm_provider,
+        model: llm_model
+    });
     res.json(result);
 }));
 // Get analytics/stats

package/dist/api/src/services/SDKService.js

@@ -1,4 +1,6 @@
 import { RealtimeXSDK } from '@realtimex/sdk';
+import { createLogger } from '../utils/logger.js';
+const logger = createLogger('SDKService');
 /**
  * Centralized SDK Service
  * Manages RealTimeX SDK initialization and provides singleton access
@@ -6,6 +8,12 @@ import { RealtimeXSDK } from '@realtimex/sdk';
 export class SDKService {
     static instance = null;
     static initAttempted = false;
+    // Default provider/model configuration
+    // realtimexai routes through RealTimeX Desktop to user's configured providers
+    static DEFAULT_LLM_PROVIDER = 'realtimexai';
+    static DEFAULT_LLM_MODEL = 'gpt-4o-mini';
+    static DEFAULT_EMBED_PROVIDER = 'realtimexai';
+    static DEFAULT_EMBED_MODEL = 'text-embedding-3-small';
     /**
      * Initialize SDK with required permissions
      * Safe to call multiple times - returns existing instance
@@ -33,15 +41,15 @@ export class SDKService {
                         'vectors.write', // Store vectors
                     ],
                 });
-                console.log('[SDKService] RealTimeX SDK initialized successfully');
+                logger.info('RealTimeX SDK initialized successfully');
                 // Verify connection with Desktop App
                 // @ts-ignore - Dev Mode feature
                 this.instance.ping()
-                    .then((status) => console.log('[SDKService] Desktop App Connection:', status))
-                    .catch((err) => console.warn('[SDKService] Desktop App Connection failed:', err.message));
+                    .then((status) => logger.debug('Desktop App Connection:', { status }))
+                    .catch((err) => logger.warn('Desktop App Connection failed', { error: err.message }));
             }
             catch (error) {
-                console.warn('[SDKService] Failed to initialize SDK:', error.message);
+                logger.error('Failed to initialize SDK', error);
                 this.instance = null;
             }
         }
@@ -77,7 +85,7 @@ export class SDKService {
             }
         }
         catch (error) {
-            console.warn('[SDKService] SDK not available:', error);
+            logger.warn('SDK not available', { error: error.message });
             return false;
         }
     }
@@ -118,22 +126,38 @@ export class SDKService {
             if (!providers || providers.length === 0) {
                 throw new Error('No LLM providers available. Please configure a provider in RealTimeX Desktop.');
             }
-            // Find first provider with available models
+            // 1. Try to find the preferred default provider (realtimexai)
+            const preferredProvider = providers.find(p => p.provider === this.DEFAULT_LLM_PROVIDER);
+            if (preferredProvider && preferredProvider.models && preferredProvider.models.length > 0) {
+                // Try to find the preferred default model (gpt-4o-mini) within it
+                const preferredModel = preferredProvider.models.find(m => m.id === this.DEFAULT_LLM_MODEL) || preferredProvider.models[0];
+                this.defaultChatProvider = {
+                    provider: preferredProvider.provider,
+                    model: preferredModel.id
+                };
+                logger.info(`Using preferred default chat provider: ${this.defaultChatProvider.provider}/${this.defaultChatProvider.model}`);
+                return this.defaultChatProvider;
+            }
+            // 2. Fallback to the first provider with available models
             for (const p of providers) {
                 if (p.models && p.models.length > 0) {
                     this.defaultChatProvider = {
                         provider: p.provider,
                         model: p.models[0].id
                     };
-                    console.log(`[SDKService] Default chat provider: ${this.defaultChatProvider.provider}/${this.defaultChatProvider.model}`);
+                    logger.info(`Defaulting to first available chat provider: ${this.defaultChatProvider.provider}/${this.defaultChatProvider.model}`);
                     return this.defaultChatProvider;
                 }
             }
             throw new Error('No LLM models available. Please configure a model in RealTimeX Desktop.');
         }
         catch (error) {
-            console.error('[SDKService] Failed to get default chat provider:', error.message);
-            throw error;
+            logger.warn('Failed to get default chat provider from SDK. Using hardcoded fallback.', error);
+            return {
+                provider: this.DEFAULT_LLM_PROVIDER,
+                model: this.DEFAULT_LLM_MODEL,
+                isDefaultFallback: true
+            };
         }
     }
     /**
@@ -161,23 +185,21 @@ export class SDKService {
                         provider: p.provider,
                         model: p.models[0].id
                     };
-                    console.log(`[SDKService] Default embed provider: ${this.defaultEmbedProvider.provider}/${this.defaultEmbedProvider.model}`);
+                    logger.info(`Default embed provider: ${this.defaultEmbedProvider.provider}/${this.defaultEmbedProvider.model}`);
                     return this.defaultEmbedProvider;
                 }
             }
             throw new Error('No embedding models available. Please configure a model in RealTimeX Desktop.');
         }
         catch (error) {
-            console.error('[SDKService] Failed to get default embed provider:', error.message);
-            throw error;
+            logger.warn('Failed to get default embed provider from SDK. Using hardcoded fallback.', error);
+            return {
+                provider: this.DEFAULT_EMBED_PROVIDER,
+                model: this.DEFAULT_EMBED_MODEL,
+                isDefaultFallback: true
+            };
         }
     }
-    // Default provider/model configuration
-    // realtimexai routes through RealTimeX Desktop to user's configured providers
-    static DEFAULT_LLM_PROVIDER = 'realtimexai';
-    static DEFAULT_LLM_MODEL = 'gpt-4o-mini';
-    static DEFAULT_EMBED_PROVIDER = 'realtimexai';
-    static DEFAULT_EMBED_MODEL = 'text-embedding-3-small';
     /**
      * Resolve LLM provider/model - use settings if available, otherwise use defaults
      */
@@ -187,14 +209,7 @@ export class SDKService {
             return { provider: settings.llm_provider, model: settings.llm_model };
         }
         // Try to get from SDK discovery first
-        try {
-            return await this.getDefaultChatProvider();
-        }
-        catch (error) {
-            // Fallback to hardcoded defaults if SDK discovery fails
-            console.log(`[SDKService] Using default LLM: ${this.DEFAULT_LLM_PROVIDER}/${this.DEFAULT_LLM_MODEL}`);
-            return { provider: this.DEFAULT_LLM_PROVIDER, model: this.DEFAULT_LLM_MODEL };
-        }
+        return await this.getDefaultChatProvider();
     }
     /**
      * Resolve embedding provider/model - use settings if available, otherwise use defaults
@@ -205,14 +220,7 @@ export class SDKService {
             return { provider: settings.embedding_provider, model: settings.embedding_model };
         }
         // Try to get from SDK discovery first
-        try {
-            return await this.getDefaultEmbedProvider();
-        }
-        catch (error) {
-            // Fallback to hardcoded defaults if SDK discovery fails
-            console.log(`[SDKService] Using default embedding: ${this.DEFAULT_EMBED_PROVIDER}/${this.DEFAULT_EMBED_MODEL}`);
-            return { provider: this.DEFAULT_EMBED_PROVIDER, model: this.DEFAULT_EMBED_MODEL };
-        }
+        return await this.getDefaultEmbedProvider();
     }
     /**
      * Clear provider cache (useful when providers change)