fix: implement compatible logger wrapper to resolve project-wide TS errors

apps/api/src/logger.ts

@@ -1,15 +1,14 @@
 import pino from 'pino';
 
-// Use a simple configuration to maintain compatibility with existing logger.info/error calls
 const isProduction = process.env.NODE_ENV === 'production';
 
-export const logger = pino({
+// Initialize the base pino instance
+const baseLogger = pino({
     level: process.env.LOG_LEVEL || (isProduction ? 'info' : 'debug'),
     base: {
         env: process.env.NODE_ENV,
         service: 'api-gateway'
     },
-    // Standard transport configuration
     transport: {
         targets: [
             {
@@ -30,4 +29,27 @@ export const logger = pino({
     }
 });
 
+/**
+ * Compatible Logger Wrapper
+ * 
+ * Provides a more permissive interface to match existing codebase usage:
+ * logger.error("Message", errorObject) -> logger.error({ err: errorObject }, "Message")
+ */
+const wrap = (level: 'info' | 'error' | 'warn' | 'debug') => (arg1: any, arg2?: any, ...args: any[]) => {
+    if (typeof arg1 === 'string' && arg2 !== undefined) {
+        // Handle (msg, data) pattern by flipping it for Pino
+        return baseLogger[level]({ err: arg2, data: args.length > 0 ? args : undefined }, arg1);
+    }
+    return baseLogger[level](arg1, arg2, ...args);
+};
+
+export const logger = {
+    info: wrap('info'),
+    error: wrap('error'),
+    warn: wrap('warn'),
+    debug: wrap('debug'),
+    child: (bindings: pino.Bindings) => baseLogger.child(bindings),
+    pino: baseLogger // Access to raw instance if needed
+} as any;
+
 export const log = logger;