@finatic/client 0.0.140 → 0.0.142

package/src/utils/brokerUtils.ts

@@ -2,6 +2,18 @@
  * Broker filtering utility functions
  */
 
+import { setupLogger, buildLoggerExtra, LoggerExtra } from '../lib/logger';
+
+const brokerLogger = setupLogger('FinaticClientSDK.BrokerUtils', undefined, {
+  codebase: 'FinaticClientSDK',
+});
+
+const buildBrokerExtra = (functionName: string, metadata?: Record<string, unknown>): LoggerExtra => ({
+  module: 'BrokerUtils',
+  function: functionName,
+  ...(metadata ? buildLoggerExtra(metadata) : {}),
+});
+
 // Supported broker names and their corresponding IDs (including aliases)
 const SUPPORTED_BROKERS: Record<string, string> = {
   'alpaca': 'alpaca',
@@ -52,7 +64,11 @@ export function appendBrokerFilterToURL(baseUrl: string, brokerNames?: string[])
     const { brokerIds, warnings } = convertBrokerNamesToIds(brokerNames);
 
     // Log warnings for unsupported broker names
-    warnings.forEach(warning => console.warn(`[FinaticConnect] ${warning}`));
+    warnings.forEach(warning =>
+      brokerLogger.warn('Unsupported broker name provided', buildBrokerExtra('appendBrokerFilterToURL', {
+        warning,
+      })),
+    );
 
     // Only add broker filter if we have valid broker IDs
     if (brokerIds.length > 0) {
@@ -62,7 +78,10 @@ export function appendBrokerFilterToURL(baseUrl: string, brokerNames?: string[])
 
     return url.toString();
   } catch (error) {
-    console.error('Failed to append broker filter to URL:', error);
+    brokerLogger.exception('Failed to append broker filter to URL', error, buildBrokerExtra('appendBrokerFilterToURL', {
+      base_url: baseUrl,
+      brokers_count: brokerNames?.length ?? 0,
+    }));
     return baseUrl;
   }
 }

package/src/utils/events.ts

@@ -1,5 +1,11 @@
+import { setupLogger, buildLoggerExtra } from '../lib/logger';
+
 type EventCallback = (...args: any[]) => void;
 
+const eventsLogger = setupLogger('FinaticClientSDK.Events', undefined, {
+  codebase: 'FinaticClientSDK',
+});
+
 export class EventEmitter {
   private events: Map<string, Set<EventCallback>> = new Map();
 
@@ -30,7 +36,13 @@ export class EventEmitter {
         try {
           callback(...args);
         } catch (error) {
-          console.error(`Error in event handler for ${event}:`, error);
+          eventsLogger.exception('Error in event handler', error, {
+            module: 'EventEmitter',
+            function: 'emit',
+            ...buildLoggerExtra({
+              event_name: event,
+            }),
+          });
         }
       });
     }

package/src/utils/themeUtils.ts

@@ -1,5 +1,16 @@
 import { PortalTheme, PortalThemeConfig } from '../types/portal';
 import { portalThemePresets } from '../themes/portalPresets';
+import { setupLogger, buildLoggerExtra, LoggerExtra } from '../lib/logger';
+
+const themeLogger = setupLogger('FinaticClientSDK.ThemeUtils', undefined, {
+  codebase: 'FinaticClientSDK',
+});
+
+const buildThemeExtra = (functionName: string, metadata?: Record<string, unknown>): LoggerExtra => ({
+  module: 'ThemeUtils',
+  function: functionName,
+  ...(metadata ? buildLoggerExtra(metadata) : {}),
+});
 
 /**
  * Generate a portal URL with theme parameters
@@ -27,7 +38,10 @@ export function generatePortalThemeURL(baseUrl: string, theme?: PortalTheme): st
 
     return url.toString();
   } catch (error) {
-    console.error('Failed to generate theme URL:', error);
+    themeLogger.exception('Failed to generate theme URL', error, buildThemeExtra('generatePortalThemeURL', {
+      base_url: baseUrl,
+      has_theme: Boolean(theme),
+    }));
     return baseUrl;
   }
 }
@@ -58,7 +72,10 @@ export function appendThemeToURL(baseUrl: string, theme?: PortalTheme): string {
 
     return url.toString();
   } catch (error) {
-    console.error('Failed to append theme to URL:', error);
+    themeLogger.exception('Failed to append theme to URL', error, buildThemeExtra('appendThemeToURL', {
+      base_url: baseUrl,
+      has_theme: Boolean(theme),
+    }));
     return baseUrl;
   }
 }
@@ -118,7 +135,7 @@ export function validateCustomTheme(theme: PortalThemeConfig): boolean {
 
     return true;
   } catch (error) {
-    console.error('Theme validation error:', error);
+    themeLogger.exception('Theme validation error', error, buildThemeExtra('validateCustomTheme'));
     return false;
   }
 }
@@ -135,7 +152,9 @@ export function createCustomThemeFromPreset(
 ): PortalThemeConfig | null {
   const baseTheme = getThemePreset(preset);
   if (!baseTheme) {
-    console.error(`Preset theme '${preset}' not found`);
+    themeLogger.warn('Preset theme not found', buildThemeExtra('createCustomThemeFromPreset', {
+      preset,
+    }));
     return null;
   }