@morojs/moro 1.5.2 → 1.5.4

package/src/core/auth/morojs-adapter.ts

@@ -8,6 +8,10 @@
  * @see https://github.com/nextauthjs/next-auth/tree/main/packages
  */
 
+import { createFrameworkLogger } from '../logger';
+
+const logger = createFrameworkLogger('AuthAdapter');
+
 // Mock Auth.js types until we have the actual package
 // These would come from @auth/core in a real implementation
 export interface AuthConfig {
@@ -247,7 +251,7 @@ export async function MoroJSAuth(config: MoroJSAuthConfig): Promise<{
           basePath,
           raw: (code: any, ...message: any[]) => {
             if (config.morojs?.debug) {
-              console.log(`[MoroJS Auth] ${code}:`, ...message);
+              logger.debug(`[MoroJS Auth] ${code}:`, 'AuthAdapter', { message });
             }
           },
         });
@@ -261,7 +265,9 @@ export async function MoroJSAuth(config: MoroJSAuthConfig): Promise<{
         // Convert Web API response to MoroJS response
         await fromWebResponse(finalResponse, res);
       } catch (error) {
-        console.error('[MoroJS Auth] Error:', error);
+        logger.error('[MoroJS Auth] Error', 'AuthAdapter', {
+          error: error instanceof Error ? error.message : String(error),
+        });
         // Robust error handling - check if response methods exist
         if (typeof (res as any).status === 'function' && typeof (res as any).json === 'function') {
           (res as any).status(500).json({
@@ -308,7 +314,9 @@ export async function MoroJSAuth(config: MoroJSAuthConfig): Promise<{
         return null;
       } catch (error) {
         if (config.morojs?.debug) {
-          console.error('[MoroJS Auth] Session error:', error);
+          logger.error('[MoroJS Auth] Session error', 'AuthAdapter', {
+            error: error instanceof Error ? error.message : String(error),
+          });
         }
         return null;
       }
@@ -322,18 +330,21 @@ export async function MoroJSAuth(config: MoroJSAuthConfig): Promise<{
  * This creates a MoroJS-compatible middleware for authentication
  */
 export function createAuthMiddleware(config: MoroJSAuthConfig) {
-  console.log('🏭 createAuthMiddleware called - creating middleware function');
+  logger.info('createAuthMiddleware called - creating middleware function', 'AuthAdapter');
   // Return a function that MoroJS can call directly
   return async (app: any) => {
-    console.log('🔧 Installing Auth.js middleware...');
-    console.log('📦 App object received:', typeof app, app.constructor.name);
+    logger.info('Installing Auth.js middleware...', 'AuthAdapter');
+    logger.debug('App object received', 'AuthAdapter', {
+      appType: typeof app,
+      appConstructor: app.constructor.name,
+    });
 
     // Get the hooks from the app's middleware system
     const hooks =
       (app as any).coreFramework?.middlewareManager?.hooks || (app as any).middlewareManager?.hooks;
 
     if (!hooks) {
-      console.error('❌ Could not access MoroJS hooks system');
+      logger.error('Could not access MoroJS hooks system', 'AuthAdapter');
       return;
     }
 
@@ -344,9 +355,9 @@ export function createAuthMiddleware(config: MoroJSAuthConfig) {
 
     // Register request hook
     hooks.before('request', async (context: any) => {
-      console.log('🔒 Native adapter hook starting...');
+      logger.debug('Native adapter hook starting', 'AuthAdapter');
       const req = context.request;
-      console.log('📝 Request path:', req.path || req.url);
+      logger.debug('Request path', 'AuthAdapter', { path: req.path || req.url });
 
       try {
         // Just add auth object to request - don't touch response
@@ -383,14 +394,16 @@ export function createAuthMiddleware(config: MoroJSAuthConfig) {
             return { url: signOutUrl };
           },
         };
-        console.log('✅ Native adapter hook completed successfully');
+        logger.debug('Native adapter hook completed successfully', 'AuthAdapter');
       } catch (error) {
-        console.error('❌ Error in native adapter hook:', error);
+        logger.error('Error in native adapter hook', 'AuthAdapter', {
+          error: error instanceof Error ? error.message : String(error),
+        });
         throw error;
       }
     });
 
-    console.log('✅ Auth.js middleware installed successfully!');
+    logger.info('Auth.js middleware installed successfully!', 'AuthAdapter');
   };
 }
 

package/src/core/database/README.md

@@ -7,16 +7,19 @@ The MoroJS database module provides a pluggable adapter system for different dat
 ### SQL Databases
 
 #### MySQL Adapter
+
 - **Package Required**: `mysql2`
 - **Usage**: Production-ready with connection pooling
 - **Type**: `mysql`
 
-#### PostgreSQL Adapter  
+#### PostgreSQL Adapter
+
 - **Package Required**: `pg` and `@types/pg`
 - **Usage**: Full PostgreSQL feature support
 - **Type**: `postgresql`, `postgres`, or `pg`
 
 #### SQLite Adapter
+
 - **Package Required**: `better-sqlite3`
 - **Usage**: Lightweight, file-based database
 - **Type**: `sqlite` or `sqlite3`
@@ -24,11 +27,13 @@ The MoroJS database module provides a pluggable adapter system for different dat
 ### NoSQL Databases
 
 #### MongoDB Adapter
+
 - **Package Required**: `mongodb`
 - **Usage**: Document database with aggregation support
 - **Type**: `mongodb` or `mongo`
 
 #### Redis Adapter
+
 - **Package Required**: `ioredis`
 - **Usage**: In-memory key-value store with pub/sub
 - **Type**: `redis`
@@ -36,6 +41,7 @@ The MoroJS database module provides a pluggable adapter system for different dat
 ### ORM
 
 #### Drizzle Adapter
+
 - **Package Required**: `drizzle-orm` + database driver
 - **Usage**: Type-safe ORM with schema validation
 - **Type**: `drizzle` or `orm`
@@ -43,7 +49,7 @@ The MoroJS database module provides a pluggable adapter system for different dat
 ## Factory Pattern (Recommended)
 
 ```typescript
-import { createDatabaseAdapter } from 'moro';
+import { createDatabaseAdapter } from '@morojs/moro';
 
 // SQL Databases
 const mysql = createDatabaseAdapter('mysql', {
@@ -52,7 +58,7 @@ const mysql = createDatabaseAdapter('mysql', {
   user: 'root',
   password: 'password',
   database: 'my_app',
-  connectionLimit: 10
+  connectionLimit: 10,
 });
 
 const postgres = createDatabaseAdapter('postgresql', {
@@ -62,12 +68,12 @@ const postgres = createDatabaseAdapter('postgresql', {
   password: 'password',
   database: 'my_app',
   connectionLimit: 10,
-  ssl: false
+  ssl: false,
 });
 
 const sqlite = createDatabaseAdapter('sqlite', {
   filename: 'app.db',
-  memory: false
+  memory: false,
 });
 
 // NoSQL Databases
@@ -76,34 +82,34 @@ const mongodb = createDatabaseAdapter('mongodb', {
   port: 27017,
   database: 'my_app',
   username: 'user',
-  password: 'password'
+  password: 'password',
 });
 
 const redis = createDatabaseAdapter('redis', {
   host: 'localhost',
   port: 6379,
   password: 'password',
-  keyPrefix: 'myapp:'
+  keyPrefix: 'myapp:',
 });
 
 // ORM
 const drizzle = createDatabaseAdapter('drizzle', {
   database: drizzleInstance,
-  schema: schemaObject
+  schema: schemaObject,
 });
 ```
 
 ## Direct Instantiation
 
 ```typescript
-import { 
-  MySQLAdapter, 
-  PostgreSQLAdapter, 
+import {
+  MySQLAdapter,
+  PostgreSQLAdapter,
   SQLiteAdapter,
   MongoDBAdapter,
   RedisAdapter,
   DrizzleAdapter
-} from 'moro';
+} from '@morojs/moro';
 
 // SQL
 const mysql = new MySQLAdapter({ host: 'localhost', ... });
@@ -151,7 +157,7 @@ const updated = await db.update('users', { name: 'Jane' }, { id: 1 });
 const deleted = await db.delete('users', { id: 1 });
 
 // Transactions
-const result = await db.transaction(async (tx) => {
+const result = await db.transaction(async tx => {
   const user = await tx.insert('users', userData);
   await tx.insert('profiles', { user_id: user.id, ...profileData });
   return user;
@@ -161,6 +167,7 @@ const result = await db.transaction(async (tx) => {
 ## Usage Examples
 
 ### SQL Operations
+
 ```typescript
 // Standard CRUD operations work across all SQL adapters
 const users = await db.query('SELECT * FROM users WHERE age > ?', [18]);
@@ -171,6 +178,7 @@ const deleted = await db.delete('users', { id: 1 });
 ```
 
 ### MongoDB Operations
+
 ```typescript
 // MongoDB uses collections instead of tables
 const users = await mongoDb.query('users'); // Get all
@@ -179,12 +187,13 @@ const user = await mongoDb.queryOne('users', { email: 'john@example.com' });
 
 // MongoDB-specific methods
 const stats = await mongoDb.aggregate('users', [
-  { $group: { _id: null, avgAge: { $avg: '$age' } } }
+  { $group: { _id: null, avgAge: { $avg: '$age' } } },
 ]);
 await mongoDb.createIndex('users', { email: 1 }, { unique: true });
 ```
 
 ### Redis Operations
+
 ```typescript
 // Key-value operations
 await redisDb.set('user:123', userData, 3600); // with TTL
@@ -197,6 +206,7 @@ await redisDb.publish('notifications', message);
 ```
 
 ### Drizzle ORM Operations
+
 ```typescript
 // Type-safe queries (requires schema setup)
 const users = await drizzleDb.select('users').where(eq(schema.users.age, 25));
@@ -213,7 +223,7 @@ Choose and install the appropriate database package:
 ```bash
 # SQL Databases
 npm install mysql2                    # MySQL
-npm install pg @types/pg             # PostgreSQL  
+npm install pg @types/pg             # PostgreSQL
 npm install better-sqlite3           # SQLite
 
 # NoSQL Databases
@@ -225,4 +235,4 @@ npm install drizzle-orm              # Drizzle ORM
 # Plus the appropriate driver (mysql2, pg, better-sqlite3, etc.)
 ```
 
-The adapters will gracefully handle missing packages with helpful error messages. 
+The adapters will gracefully handle missing packages with helpful error messages.

package/src/core/http/http-server.ts

@@ -281,12 +281,13 @@ export class MoroHttpServer {
       await route.handler(httpReq, httpRes);
     } catch (error) {
       // Debug: Log the actual error and where it came from
-      console.log('🚨 MoroJS Request Error Details:');
-      console.log('📍 Error type:', typeof error);
-      console.log('📍 Error message:', error instanceof Error ? error.message : String(error));
-      console.log('📍 Error stack:', error instanceof Error ? error.stack : 'No stack trace');
-      console.log('📍 Request path:', req.url);
-      console.log('📍 Request method:', req.method);
+      this.logger.debug('Request error details', 'RequestHandler', {
+        errorType: typeof error,
+        errorMessage: error instanceof Error ? error.message : String(error),
+        errorStack: error instanceof Error ? error.stack : 'No stack trace',
+        requestPath: req.url,
+        requestMethod: req.method,
+      });
 
       this.logger.error('Request error', 'RequestHandler', {
         error: error instanceof Error ? error.message : String(error),
@@ -310,11 +311,10 @@ export class MoroHttpServer {
             httpRes.setHeader('Content-Type', 'application/json');
           } else {
             // Even setHeader doesn't exist - object is completely wrong
-            console.error(
-              '❌ Response object is not a proper ServerResponse:',
-              typeof httpRes,
-              Object.keys(httpRes)
-            );
+            this.logger.error('Response object is not a proper ServerResponse', 'RequestHandler', {
+              responseType: typeof httpRes,
+              responseKeys: Object.keys(httpRes),
+            });
           }
 
           if (typeof httpRes.end === 'function') {
@@ -326,7 +326,10 @@ export class MoroHttpServer {
               })
             );
           } else {
-            console.error('❌ Cannot send error response - end() method missing');
+            this.logger.error(
+              'Cannot send error response - end() method missing',
+              'RequestHandler'
+            );
           }
         }
       }

package/src/core/logger/filters.ts

@@ -22,16 +22,24 @@ export const contextFilter = (allowedContexts: string[]): LogFilter => ({
 // Rate limiting filter
 export const rateLimitFilter = (maxPerSecond: number): LogFilter => {
   const timestamps: number[] = [];
+  let lastCleanup = 0;
 
   return {
     name: 'rate-limit',
     filter: (entry: LogEntry) => {
       const now = Date.now();
-      const oneSecondAgo = now - 1000;
 
-      // Remove old timestamps
-      while (timestamps.length > 0 && timestamps[0] < oneSecondAgo) {
-        timestamps.shift();
+      // Batch cleanup for better performance and thread safety
+      if (now - lastCleanup > 1000) {
+        const cutoff = now - 1000;
+        let keepIndex = 0;
+        for (let i = 0; i < timestamps.length; i++) {
+          if (timestamps[i] >= cutoff) {
+            timestamps[keepIndex++] = timestamps[i];
+          }
+        }
+        timestamps.length = keepIndex;
+        lastCleanup = now;
       }
 
       // Check rate limit