fix: restore logger compatibility and fix major TS errors in services

apps/api/src/logger.ts

@@ -1,35 +1,33 @@
 import pino from 'pino';
 
-// 2. Optional: Logtail transport (if token is provided)
-const targets = [
-    {
-        target: 'pino-pretty',
-        options: {
-            colorize: true,
-            ignore: 'pid,hostname',
-            translateTime: 'SYS:standard',
-        },
-        level: process.env.NODE_ENV === 'production' ? 'info' : 'debug'
-    }
-];
-
-if (process.env.LOGTAIL_TOKEN) {
-    targets.push({
-        target: '@logtail/pino',
-        options: { sourceToken: process.env.LOGTAIL_TOKEN },
-        level: 'info'
-    });
-}
-
-const transport = pino.transport({ targets });
+// Use a simple configuration to maintain compatibility with existing logger.info/error calls
+const isProduction = process.env.NODE_ENV === 'production';
 
 export const logger = pino({
-    level: process.env.LOG_LEVEL || (process.env.NODE_ENV === 'production' ? 'info' : 'debug'),
+    level: process.env.LOG_LEVEL || (isProduction ? 'info' : 'debug'),
     base: {
         env: process.env.NODE_ENV,
         service: 'api-gateway'
+    },
+    // Standard transport configuration
+    transport: {
+        targets: [
+            {
+                target: 'pino-pretty',
+                level: isProduction ? 'info' : 'debug',
+                options: {
+                    colorize: true,
+                    ignore: 'pid,hostname',
+                    translateTime: 'SYS:standard',
+                }
+            },
+            ...(process.env.LOGTAIL_TOKEN ? [{
+                target: '@logtail/pino',
+                level: 'info',
+                options: { sourceToken: process.env.LOGTAIL_TOKEN }
+            }] : [])
+        ]
     }
-}, transport);
+});
 
-// Compatibility with legacy logger usage if any
 export const log = logger;

apps/api/src/middleware/rateLimit.ts

@@ -17,6 +17,6 @@ export async function setupRateLimit(server: FastifyInstance) {
         });
         logger.info('[RATE-LIMIT] Configured successfully.');
     } catch (err) {
-        logger.error('[RATE-LIMIT] Initialization failed:', err);
+        logger.error({ err }, "[RATE-LIMIT] Initialization failed:");
     }
 }

apps/api/src/services/ai/index.ts

@@ -59,7 +59,7 @@ class AIService {
             const cached = await redis.get(cacheKey);
             if (cached) return JSON.parse(cached);
         } catch (err) {
-            logger.error('[AI_SERVICE] Redis error:', err);
+            logger.error({ err }, "[AI_SERVICE] Redis error:");
         }
 
         try {
@@ -71,7 +71,7 @@ class AIService {
             await redis.set(cacheKey, JSON.stringify(personality), 'EX', 3600);
             return personality;
         } catch (err) {
-            logger.error('[AI_SERVICE] Failed to fetch tenant personality:', err);
+            logger.error({ err }, "[AI_SERVICE] Failed to fetch tenant personality:");
             return {};
         }
     }

apps/api/src/services/stripe.ts

@@ -80,9 +80,9 @@ export class StripeService implements PaymentProvider {
             });
 
             return session.url || '';
-        } catch (error) {
-            logger.error('[StripeService] Failed to create checkout session:', error);
-            throw error;
+        } catch (err) {
+            logger.error({ err }, "[StripeService] Failed to create checkout session:");
+            throw err;
         }
     }
 
@@ -137,9 +137,9 @@ export class StripeService implements PaymentProvider {
                 return_url: `${this.clientUrl}/settings`,
             });
             return session.url;
-        } catch (error) {
-            logger.error('[StripeService] Failed to create portal session:', error);
-            throw error;
+        } catch (err) {
+            logger.error({ err }, '[StripeService] Failed to create portal session:');
+            throw err;
         }
     }
 }