vibe-gx 4.1.4 → 4.2.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vibe-gx",
-  "version": "4.1.4",
+  "version": "4.2.0",
   "description": "A lightweight, high-performance Node.js web framework.",
   "type": "module",
   "main": "vibe.js",

package/utils/core/logger.js

@@ -9,6 +9,7 @@ const LOG_LEVELS = {
   warn: 40,
   error: 50,
   fatal: 60,
+  silent: 100, // Higher than all levels — suppresses all output (logger: false)
 };
 
 const LEVEL_NAMES = {
@@ -136,45 +137,59 @@ export class Logger {
   _printPretty(log) {
     const time = new Date(log.time).toLocaleTimeString();
     const lvlName = LEVEL_NAMES[log.level] || "INFO";
-    let prefixC = color.cyan;
-    if (log.level >= 50) prefixC = color.red;
-    else if (log.level === 40) prefixC = color.yellow;
-    else if (log.level <= 20) prefixC = color.dim;
-
-    const prefix = prefixC(`[VIBE ${lvlName} ${time}]`);
-    let context = "";
-    if (log.reqId) {
-      context = `\x1b[90m[${log.reqId}]\x1b[0m `;
-    }
 
-    let content = log.msg || "";
-    if (log.color && color[log.color]) {
-      content = color[log.color](content);
-    }
+    const isError = log.level >= 50;
+    const isWarn  = log.level === 40;
+    const isDebug = log.level <= 20;
+
+    // Build context tag (reqId)
+    const context = log.reqId ? `[${log.reqId}] ` : "";
 
+    // Build message content
+    let content = log.msg || "";
     if (log.err && log.err.stack) {
-      content += "\n" + prefixC(log.err.stack);
+      content += "\n" + log.err.stack;
     }
 
-    // Attempt to print remaining metadata if it's not standard
+    // Build metadata string (skip standard keys)
     const skipKeys = [
-      "level",
-      "time",
-      "pid",
-      "hostname",
-      "reqId",
-      "msg",
-      "err",
-      "color",
+      "level", "time", "pid", "hostname", "reqId", "msg", "err", "color",
     ];
     let metaStr = "";
     for (const key of Object.keys(log)) {
       if (!skipKeys.includes(key)) {
-        metaStr += ` \x1b[90m${key}=${JSON.stringify(log[key])}\x1b[0m`;
+        metaStr += ` ${key}=${JSON.stringify(log[key])}`;
       }
     }
 
-    this.stream.write(`${prefix} ${context}${content}${metaStr}\n`);
+    const rawPrefix = `[VIBE ${lvlName} ${time}]`;
+
+    if (isError) {
+      // Entire line is red — prefix, context, message, stack, metadata
+      const fullLine = `${rawPrefix} ${context}${content}${metaStr}`;
+      this.stream.write(color.red(fullLine) + "\n");
+    } else if (isWarn) {
+      // Yellow prefix, bright content
+      const coloredContent = log.color && color[log.color]
+        ? color[log.color](content)
+        : color.bright(content);
+      this.stream.write(
+        color.yellow(rawPrefix) + " " + context + coloredContent +
+        (metaStr ? color.dim(metaStr) : "") + "\n",
+      );
+    } else if (isDebug) {
+      // Dim entire line for trace/debug
+      this.stream.write(color.dim(`${rawPrefix} ${context}${content}${metaStr}`) + "\n");
+    } else {
+      // Info — green prefix + bright content (matches [VIBE LOG] style)
+      const coloredContent = log.color && color[log.color]
+        ? color[log.color](content)
+        : color.bright(content);
+      this.stream.write(
+        color.green(rawPrefix) + " " + context + coloredContent +
+        (metaStr ? color.dim(metaStr) : "") + "\n",
+      );
+    }
   }
 }
 

package/utils/core/parser.js

@@ -103,7 +103,7 @@ function parseMultipart(req, res, media, options, resolve, reject) {
       },
     });
   } catch (err) {
-    console.error("Busboy init failed:", err);
+    options.logger?.error(err, "[VIBE] Busboy init failed");
     return resolve();
   }
 
@@ -152,7 +152,10 @@ function parseMultipart(req, res, media, options, resolve, reject) {
       media.public &&
       !dest.startsWith(path.resolve(options.publicFolder || ""))
     ) {
-      console.warn("Attempted upload outside public folder, skipping");
+      options.logger?.warn(
+        { dest, publicFolder: options.publicFolder },
+        "[VIBE] Attempted upload outside public folder, skipping",
+      );
       pendingWrites--;
       checkComplete();
       return file.resume();
@@ -161,7 +164,7 @@ function parseMultipart(req, res, media, options, resolve, reject) {
     try {
       if (!fs.existsSync(dest)) fs.mkdirSync(dest, { recursive: true });
     } catch (err) {
-      console.error("Failed to create upload folder:", err);
+      options.logger?.error(err, "[VIBE] Failed to create upload folder");
       pendingWrites--;
       checkComplete();
       return file.resume();
@@ -200,14 +203,14 @@ function parseMultipart(req, res, media, options, resolve, reject) {
     });
 
     file.on("error", (err) => {
-      console.error("File stream error:", err);
+      options.logger?.error(err, "[VIBE] File stream error");
       writeStream.end();
       pendingWrites--;
       checkComplete();
     });
 
     writeStream.on("error", (err) => {
-      console.error("Write stream error:", err);
+      options.logger?.error(err, "[VIBE] Write stream error");
       file.resume();
       pendingWrites--;
       checkComplete();
@@ -231,7 +234,7 @@ function parseMultipart(req, res, media, options, resolve, reject) {
   });
 
   bb.on("error", (err) => {
-    console.error("Busboy error:", err);
+    options.logger?.error(err, "[VIBE] Busboy error");
     req.unpipe(bb);
     reject(err);
   });
@@ -266,7 +269,10 @@ function parseJson(req, res, media, options, resolve, reject) {
   req.on("data", (chunk) => {
     body += chunk;
     if (body.length > limit) {
-      console.warn("JSON payload too large, destroying connection");
+      options.logger?.warn(
+        { limit, received: body.length },
+        "[VIBE] JSON payload too large, destroying connection",
+      );
       req.destroy();
     }
   });

package/utils/core/response.js

@@ -243,7 +243,12 @@ const vibeResponseMethods = {
    * @param {Error} error
    */
   serverError(error) {
-    console.error(error);
+    const logger = this._vibeOptions?.logger;
+    if (logger) {
+      logger.error(error, "[VIBE] Internal server error");
+    } else {
+      console.error(error);
+    }
     this.writeHead(500, JSON_CT);
     this.end(RESPONSES.serverError);
   },

package/utils/core/server.js

@@ -319,8 +319,14 @@ async function server(options, port, host, callback) {
     getNetworkIP(mainHost, port);
 
     const strategy = useTrieMatching ? "Trie (O(log n))" : "Linear (O(n))";
-    console.log(
-      `[VIBE] Route matching: ${strategy} (${options.routeCount} routes, ${staticRoutes.size} static, threshold: ${options.trieThreshold})`,
+    options.logger.info(
+      {
+        strategy,
+        routeCount: options.routeCount,
+        staticRoutes: staticRoutes.size,
+        trieThreshold: options.trieThreshold,
+      },
+      "[VIBE] Route matching strategy initialized",
     );
 
     if (callback) callback();

package/vibe.d.ts

@@ -237,10 +237,8 @@ export interface VibeRequest extends IncomingMessage {
   body: Record<string, any>;
   /** Uploaded files array (if multipart/form-data) */
   files?: UploadedFile[];
-  /** Client IP address */
+  /** Real client IP — first entry from x-forwarded-for, x-real-ip, or socket address */
   ip?: string;
-  /** Detailed client IP info */
-  fullIp?: string;
   /** Automatically generated UUID for the request lifecycle */
   id: string;
   /** Context-bound logger automatically stamped with the req.id constraint */
@@ -302,9 +300,58 @@ export type Interceptor = (
   res: VibeResponse,
 ) => boolean | void | Promise<boolean | void>;
 
+/**
+ * Scoped app interface passed to register() plugin callbacks.
+ * A subset of VibeApp — excludes server-level methods that make no sense
+ * inside an encapsulated plugin (listen, logRoutes, setPublicFolder, include).
+ */
+export interface ScopedVibeApp {
+  get: RouteRegistrar;
+  post: RouteRegistrar;
+  put: RouteRegistrar;
+  del: RouteRegistrar;
+  patch: RouteRegistrar;
+  head: RouteRegistrar;
+
+  /** Register a global interceptor within this plugin scope */
+  plugin: (interceptor: Interceptor) => void;
+
+  /** Register a nested plugin */
+  register: (fn: PluginCallback, opts?: RegisterOptions) => Promise<void>;
+
+  /** Decorate the app with a custom property */
+  decorate: (name: string, value: any) => void;
+
+  /** Decorate request objects with a custom property */
+  decorateRequest: (name: string, value: any) => void;
+
+  /** Decorate response objects with a custom property */
+  decorateReply: (name: string, value: any) => void;
+
+  /** Override the error handler for this plugin scope */
+  setErrorHandler: (
+    fn: (error: Error, req: VibeRequest, res: VibeResponse) => void,
+  ) => void;
+
+  /** Structured logger — app.log.info(), .warn(), .error() etc. */
+  log: LoggerAPI;
+
+  /** Alias for log */
+  logger: LoggerAPI;
+
+  /** Legacy colorized string logger */
+  logLegacy: (
+    value: any,
+    typeOrColor?: ColorName | "info" | "error" | "warn" | "req",
+  ) => void;
+
+  /** Any decorators registered via decorate() are available as direct properties */
+  [key: string]: any;
+}
+
 /** Plugin callback function (Fastify-style) */
 export type PluginCallback = (
-  app: VibeApp,
+  app: ScopedVibeApp,
   opts: RegisterOptions,
 ) => void | Promise<void>;
 
@@ -391,11 +438,23 @@ export interface RouterAPI {
   head: RouteRegistrar;
 
   /**
-   * Log helper supporting native colors and Vibe-stylized log levels
-   * @param value The message or object to log
-   * @param typeOrColor Optional color name (e.g. 'green') or level ('info', 'warn', 'error', 'req')
+   * Pino/Fastify-compatible structured logger.
+   * Available inside both `app.register()` plugins and `app.include()` sub-routers.
+   * @example
+   * api.log.info("Route registered");
+   * api.log.warn({ userId: 1 }, "Slow query");
+   */
+  log: LoggerAPI;
+
+  /** Alias for `log` — consistent with `app.logger` */
+  logger: LoggerAPI;
+
+  /**
+   * Legacy colorized string logger (plain terminal output).
+   * @param value The message to log
+   * @param typeOrColor Optional color name (e.g. 'green', 'red')
    */
-  log: (
+  logLegacy: (
     value: any,
     typeOrColor?: ColorName | "info" | "error" | "warn" | "req",
   ) => void;
@@ -487,6 +546,17 @@ export interface VibeApp extends RouterAPI {
 
   /** Alias for `app.log` */
   logger: LoggerAPI;
+
+  /**
+   * Legacy colorized string logger (plain terminal output).
+   * Bypasses the Pino JSON interface — for simple dev-time messages.
+   * @param value The message to log
+   * @param typeOrColor Optional color name (e.g. 'green', 'red')
+   */
+  logLegacy: (
+    value: any,
+    typeOrColor?: ColorName | "info" | "error" | "warn" | "req",
+  ) => void;
 }
 
 /**
@@ -759,11 +829,3 @@ export function adapt(
   mw: (req: any, res: any, next: (err?: any) => void) => void,
 ): Interceptor;
 
-/**
- * Adapt multiple Express middlewares at once.
- * @param middlewares - Express middleware functions
- * @returns Array of Vibe-compatible interceptors
- */
-export function adaptAll(
-  ...middlewares: Array<(req: any, res: any, next: (err?: any) => void) => void>
-): Interceptor[];

package/vibe.js

@@ -37,7 +37,6 @@ function pathToRegex(path) {
  *     size: number
  *   }>,
  *   ip?: string,
- *   fullIp?: string
  * }} VibeRequest
  */
 
@@ -430,7 +429,10 @@ const vibe = (config = {}) => {
       decorateRequest,
       decorateReply,
       register,
-      log,
+      log: appLogger,        // Structured logger (api.log.info / warn / error etc.)
+      logger: appLogger,     // Alias — consistent with root app.logger
+      logLegacy: log,        // Legacy colorized string logger (api.logLegacy(msg, color))
+      setErrorHandler: (fn) => { options.errorHandler = fn; },
       // Expose decorators
       ...options.decorators,
     };
@@ -486,7 +488,9 @@ const vibe = (config = {}) => {
       del: wrap("DELETE"),
       patch: wrap("PATCH"),
       head: wrap("HEAD"),
-      log,
+      log: appLogger,    // Structured logger — consistent with app.log
+      logger: appLogger, // Alias
+      logLegacy: log,    // Legacy colorized string logger
       plugin,
     };
   }