fix: resolve all TS compiling errs

Author: jddunn

src/compatibility/loggers/BaseCompatibleLogger.ts

@@ -257,7 +257,7 @@ export abstract class BaseCompatibleLogger {
    */
   private getCircularReplacer(): (key: string, value: any) => any {
     const seen = new WeakSet();
-    return (key: string, value: any) => {
+    return (_key: string, value: any) => {
       if (typeof value === 'object' && value !== null) {
         if (seen.has(value)) {
           return '[Circular]';
@@ -288,18 +288,27 @@ export abstract class BaseCompatibleLogger {
             // Fall through to plain format on error
           }
         }
-      // Fall through to plain if no formatter or error
+        // If no formatter or formatter failed, use plain format
+        return this.formatAsPlain(entry);
 
       case 'plain':
       default:
-        if (typeof entry === 'string') {
-          return entry;
-        }
-        if (typeof entry === 'object' && entry.message) {
-          return entry.message;
-        }
-        return this.safeSerialize(entry);
+        return this.formatAsPlain(entry);
+    }
+  }
+
+  /**
+   * Format entry as plain text.
+   * @private
+   */
+  private formatAsPlain(entry: unknown): string {
+    if (typeof entry === 'string') {
+      return entry;
+    }
+    if (typeof entry === 'object' && entry && 'message' in entry) {
+      return String((entry as { message: unknown }).message);
     }
+    return this.safeSerialize(entry);
   }
 
   /**

src/compatibility/loggers/BunyanCompatibleLogger.ts

@@ -410,11 +410,14 @@ export class BunyanCompatibleLogger extends BaseCompatibleLogger {
 
       const match = line.match(/at\s+(?:(.+?)\s+)?\((.+?):(\d+):(\d+)\)/);
       if (match) {
-        return {
-          file: match[2] || 'unknown',
-          line: parseInt(match[3], 10),
-          func: match[1] || undefined,
+        const result: BunyanRecord['src'] = {
+          file: match[2] ? match[2] : 'unknown',
+          line: parseInt(match[3] || '0', 10),
         };
+        if (match[1]) {
+          result.func = match[1].trim();
+        }
+        return result;
       }
     }
 

src/compatibility/loggers/PinoCompatibleLogger.ts

@@ -423,6 +423,9 @@ export class PinoCompatibleLogger extends BaseCompatibleLogger {
 
     const [current, ...rest] = pathParts;
 
+    // Check if current path part exists
+    if (!current) return;
+
     // Handle wildcards
     if (current === '*') {
       if (Array.isArray(obj)) {
@@ -444,7 +447,7 @@ export class PinoCompatibleLogger extends BaseCompatibleLogger {
 
     // Handle array notation
     const arrayMatch = current.match(/^(.+)\[\*\]$/);
-    if (arrayMatch) {
+    if (arrayMatch && arrayMatch[1]) {
       const arrayKey = arrayMatch[1];
       if (arrayKey in obj && Array.isArray(obj[arrayKey])) {
         const arr = obj[arrayKey] as unknown[];

src/transports/base/TransportManager.ts

@@ -133,12 +133,6 @@ export class TransportManager extends EventEmitter {
    */
   private readonly stopOnSuccess: boolean;
 
-  /**
-   * Default timeout for transport operations.
-   * @private
-   */
-  private readonly defaultTimeout?: number;
-
   /**
    * Global error handler for transport errors.
    * @private
@@ -197,7 +191,6 @@ export class TransportManager extends EventEmitter {
     this.maxPauseQueueSize = options.maxPauseQueueSize || 10000;
     this.healthCheckIntervalMs = options.healthCheckIntervalMs || 60000;
     this.stopOnSuccess = options.stopOnSuccess || false;
-    this.defaultTimeout = options.defaultTimeout;
     this.errorHandler = options.errorHandler;
 
     // Set max listeners for better event handling
@@ -641,6 +634,11 @@ export class TransportManager extends EventEmitter {
 
     results.forEach((result, index) => {
       const transport = availableTransports[index];
+      if (!transport) {
+        // Skip if transport is undefined (shouldn't happen but handle gracefully)
+        return;
+      }
+      
       if (result.status === 'fulfilled') {
         successfulTransports.push(transport.name);
       } else {