@morojs/moro 1.5.3 → 1.5.4

package/src/core/middleware/built-in/request-logger.ts

@@ -1,12 +1,16 @@
 // Simple request logging middleware
+import { createFrameworkLogger } from '../../logger';
+
+const logger = createFrameworkLogger('RequestLogger');
+
 export const requestLogger = async (context: any): Promise<void> => {
   const startTime = Date.now();
 
-  console.log(`[${new Date().toISOString()}] ${context.request?.method} ${context.request?.path}`);
+  logger.info(`${context.request?.method} ${context.request?.path}`, 'RequestLogger');
 
   // Log completion after response
   context.onComplete = () => {
     const duration = Date.now() - startTime;
-    console.log(`Request completed in ${duration}ms`);
+    logger.info(`Request completed in ${duration}ms`, 'RequestLogger');
   };
 };

package/src/core/modules/auto-discovery.ts

@@ -3,10 +3,12 @@ import { readdirSync, statSync } from 'fs';
 import { join, extname } from 'path';
 import { ModuleConfig } from '../../types/module';
 import { DiscoveryOptions } from '../../types/discovery';
+import { createFrameworkLogger } from '../logger';
 
 export class ModuleDiscovery {
   private baseDir: string;
   private options: DiscoveryOptions;
+  private discoveryLogger = createFrameworkLogger('MODULE_DISCOVERY');
 
   constructor(baseDir: string = process.cwd(), options: DiscoveryOptions = {}) {
     this.baseDir = baseDir;
@@ -28,12 +30,14 @@ export class ModuleDiscovery {
         const module = await this.loadModule(modulePath);
         if (module) {
           modules.push(module);
-          console.log(
+          this.discoveryLogger.info(
             `Auto-discovered module: ${module.name}@${module.version} from ${modulePath}`
           );
         }
       } catch (error) {
-        console.warn(`Failed to load module from ${modulePath}:`, error);
+        this.discoveryLogger.warn(`Failed to load module from ${modulePath}`, 'MODULE_DISCOVERY', {
+          error: error instanceof Error ? error.message : String(error),
+        });
       }
     }
 
@@ -63,7 +67,9 @@ export class ModuleDiscovery {
               const module = await this.loadModule(indexPath);
               if (module) {
                 modules.push(module);
-                console.log(`Auto-discovered module directory: ${module.name} from ${item}/`);
+                this.discoveryLogger.info(
+                  `Auto-discovered module directory: ${module.name} from ${item}/`
+                );
               }
             }
           } catch {
@@ -77,7 +83,9 @@ export class ModuleDiscovery {
                   const module = await this.loadModule(altPath);
                   if (module) {
                     modules.push(module);
-                    console.log(`Auto-discovered module: ${module.name} from ${item}/${alt}`);
+                    this.discoveryLogger.info(
+                      `Auto-discovered module: ${module.name} from ${item}/${alt}`
+                    );
                     break;
                   }
                 }
@@ -177,7 +185,7 @@ export class ModuleDiscovery {
     modulePaths.forEach(path => {
       try {
         fs.watchFile(path, async () => {
-          console.log(`Module file changed: ${path}`);
+          this.discoveryLogger.info(`Module file changed: ${path}`);
           const modules = await this.discoverModules();
           callback(modules);
         });

package/src/core/modules/modules.ts

@@ -4,6 +4,7 @@ import path from 'path';
 import { Container } from '../utilities';
 import { ModuleConfig } from '../../types/module';
 import { ModuleDefinition, ModuleRoute, ModuleSocket } from '../../types/module';
+import { createFrameworkLogger } from '../logger';
 
 // Module Definition Function
 export function defineModule(definition: ModuleDefinition): ModuleConfig {
@@ -66,6 +67,8 @@ export function defineModule(definition: ModuleDefinition): ModuleConfig {
 
 // Module Loader Class
 export class ModuleLoader {
+  private moduleLogger = createFrameworkLogger('MODULE_LOADER');
+
   constructor(private container: Container) {}
 
   async discoverModules(directory: string): Promise<ModuleConfig[]> {
@@ -91,15 +94,16 @@ export class ModuleLoader {
               }
             }
           } catch (error) {
-            console.warn(
-              `⚠️  Could not load module from ${modulePath}:`,
-              error instanceof Error ? error.message : String(error)
-            );
+            this.moduleLogger.warn(`Could not load module from ${modulePath}`, 'MODULE_LOADER', {
+              error: error instanceof Error ? error.message : String(error),
+            });
           }
         }
       }
     } catch (error) {
-      console.error('Failed to discover modules:', error);
+      this.moduleLogger.error('Failed to discover modules', 'MODULE_LOADER', {
+        error: error instanceof Error ? error.message : String(error),
+      });
     }
 
     return modules;

package/src/core/networking/adapters/ws-adapter.ts

@@ -9,6 +9,7 @@ import {
   WebSocketEmitter,
   WebSocketMiddleware,
 } from '../websocket-adapter';
+import { createFrameworkLogger } from '../../logger';
 
 /**
  * Native WebSocket adapter using the 'ws' library
@@ -18,6 +19,7 @@ export class WSAdapter implements WebSocketAdapter {
   private wss: any; // WebSocket server instance
   private namespaces = new Map<string, WSNamespaceWrapper>();
   private connections = new Map<string, WSConnectionWrapper>();
+  private wsLogger = createFrameworkLogger('WEBSOCKET_ADAPTER');
   private customIdGenerator?: () => string;
   private connectionCounter = 0;
 
@@ -98,7 +100,7 @@ export class WSAdapter implements WebSocketAdapter {
     // ws library handles compression at the browser level
     // This is a no-op but kept for interface compatibility
     if (enabled) {
-      console.warn('Compression is handled automatically by the ws library and browsers');
+      this.wsLogger.warn('Compression is handled automatically by the ws library and browsers');
     }
   }
 

package/src/core/networking/service-discovery.ts

@@ -1,5 +1,6 @@
 // Service Discovery Client for Microservices
 // Supports Consul, Kubernetes, and in-memory registry
+import { createFrameworkLogger } from '../logger';
 
 export interface ServiceInfo {
   name: string;
@@ -23,6 +24,7 @@ export class ServiceRegistry {
   private services = new Map<string, ServiceInfo[]>();
   private options: ServiceDiscoveryOptions;
   private healthCheckInterval?: NodeJS.Timeout;
+  private serviceLogger = createFrameworkLogger('SERVICE_DISCOVERY');
 
   constructor(options: ServiceDiscoveryOptions) {
     this.options = options;
@@ -44,7 +46,7 @@ export class ServiceRegistry {
         break;
     }
 
-    console.log(`Service registered: ${name}@${service.host}:${service.port}`);
+    this.serviceLogger.info(`Service registered: ${name}@${service.host}:${service.port}`);
   }
 
   async discover(serviceName: string): Promise<ServiceInfo[]> {
@@ -73,7 +75,7 @@ export class ServiceRegistry {
         break;
     }
 
-    console.log(`Service deregistered: ${serviceName}`);
+    this.serviceLogger.info(`Service deregistered: ${serviceName}`);
   }
 
   // In-memory registry methods
@@ -121,7 +123,9 @@ export class ServiceRegistry {
         throw new Error(`Consul registration failed: ${response.statusText}`);
       }
     } catch (error) {
-      console.error('Failed to register with Consul:', error);
+      this.serviceLogger.error('Failed to register with Consul', 'SERVICE_DISCOVERY', {
+        error: error instanceof Error ? error.message : String(error),
+      });
       // Fallback to in-memory
       this.registerInMemory(service);
     }
@@ -150,7 +154,9 @@ export class ServiceRegistry {
         metadata: entry.Service.Meta,
       }));
     } catch (error) {
-      console.error('Failed to discover from Consul:', error);
+      this.serviceLogger.error('Failed to discover from Consul', 'SERVICE_DISCOVERY', {
+        error: error instanceof Error ? error.message : String(error),
+      });
       return this.discoverFromMemory(serviceName);
     }
   }
@@ -163,7 +169,9 @@ export class ServiceRegistry {
         method: 'PUT',
       });
     } catch (error) {
-      console.error('Failed to deregister from Consul:', error);
+      this.serviceLogger.error('Failed to deregister from Consul', 'SERVICE_DISCOVERY', {
+        error: error instanceof Error ? error.message : String(error),
+      });
     }
   }
 
@@ -171,7 +179,7 @@ export class ServiceRegistry {
   private async registerWithKubernetes(service: ServiceInfo): Promise<void> {
     // In Kubernetes, services are registered via Service/Endpoints resources
     // This would typically be handled by the K8s API, not application code
-    console.log(`K8s service registration: ${service.name} (handled by Kubernetes)`);
+    this.serviceLogger.info(`K8s service registration: ${service.name} (handled by Kubernetes)`);
 
     // Fallback to in-memory for local development
     this.registerInMemory(service);
@@ -196,7 +204,9 @@ export class ServiceRegistry {
         },
       ];
     } catch (error) {
-      console.error('Failed to discover from Kubernetes:', error);
+      this.serviceLogger.error('Failed to discover from Kubernetes', 'SERVICE_DISCOVERY', {
+        error: error instanceof Error ? error.message : String(error),
+      });
       return this.discoverFromMemory(serviceName);
     }
   }
@@ -224,12 +234,16 @@ export class ServiceRegistry {
             );
 
             if (!response.ok) {
-              console.warn(`Health check failed for ${serviceName}: ${response.statusText}`);
+              this.serviceLogger.warn(
+                `Health check failed for ${serviceName}: ${response.statusText}`
+              );
               // Remove unhealthy service
               this.removeUnhealthyService(serviceName, service);
             }
           } catch (error) {
-            console.warn(`Health check failed for ${serviceName}:`, error);
+            this.serviceLogger.warn(`Health check failed for ${serviceName}`, 'SERVICE_DISCOVERY', {
+              error: error instanceof Error ? error.message : String(error),
+            });
             this.removeUnhealthyService(serviceName, service);
           }
         }

package/src/types/logger.ts

@@ -30,6 +30,7 @@ export interface LoggerOptions {
   outputs?: LogOutput[];
   filters?: LogFilter[];
   maxEntries?: number;
+  maxBufferSize?: number;
 }
 
 export interface LogOutput {
@@ -78,6 +79,7 @@ export interface LogMetrics {
   averageLogRate: number;
   errorRate: number;
   memoryUsage: number;
+  outputErrors?: Record<string, number>;
 }
 
 export interface ColorScheme {
@@ -90,4 +92,5 @@ export interface ColorScheme {
   context: string;
   metadata: string;
   performance: string;
+  reset: string;
 }