@basaltkit/express 1.4.1 → 1.5.0

package/dist/index.d.ts

@@ -1,9 +1,9 @@
 import { Container } from '@basaltkit/core';
-import { type BasaltRoute, type RequestEnricher, type RouteGuard } from '@basaltkit/http';
+import { type HttpErrorReporter, type BasaltRoute, type RequestEnricher, type RouteGuard } from '@basaltkit/http';
 import { type Express } from 'express';
 export declare const EXPRESS: import("@basaltkit/core").Token<Express>;
 /** Mounts Basalt routes on an Express app (usable without the plugin). */
-export declare function registerRoutes(app: Express, routes: BasaltRoute[], container?: Container, enrichers?: RequestEnricher[], guards?: RouteGuard[]): void;
+export declare function registerRoutes(app: Express, routes: BasaltRoute[], container?: Container, enrichers?: RequestEnricher[], guards?: RouteGuard[], onError?: HttpErrorReporter): void;
 export interface ExpressPluginOptions {
     routes?: BasaltRoute[];
     /**
@@ -21,6 +21,12 @@ export interface ExpressPluginOptions {
      * HTML default. Default: true. Pass false to keep Express's own handling
      * (e.g. when the app mounts its own catch-all after boot).
      */
+    /**
+     * Where failed requests are reported. Default: 5xx via `console.error` (with
+     * the stack) and 4xx via `console.warn`, prefixed `[basalt:http]`. Pass your
+     * own to route them into a real logger, or `() => {}` to silence them.
+     */
+    onError?: HttpErrorReporter;
     notFound?: boolean;
 }
 /**

package/dist/index.js

@@ -1,5 +1,5 @@
 import { Container, createToken, definePlugin, ensureMetadata } from '@basaltkit/core';
-import { NOT_FOUND_RESPONSE, HttpServerCollector, HTTP_SERVER, runRoute, toErrorResponse, isSseResponse, sseProducerOf, driveSse, SSE_HEADERS, GUARDED_META_BUCKET, assertRoutesGuarded, } from '@basaltkit/http';
+import { NOT_FOUND_RESPONSE, HttpServerCollector, HTTP_SERVER, runRoute, toErrorResponse, reportHttpError, isSseResponse, sseProducerOf, driveSse, SSE_HEADERS, GUARDED_META_BUCKET, assertRoutesGuarded, } from '@basaltkit/http';
 import express, {} from 'express';
 export const EXPRESS = createToken('express');
 function toNeutralRequest(req) {
@@ -52,7 +52,7 @@ class ExpressReply {
         return this;
     }
 }
-function basaltHandler(definition, container, enrichers, guards) {
+function basaltHandler(definition, container, enrichers, guards, onError) {
     return async (req, res) => {
         const reply = new ExpressReply(res);
         try {
@@ -75,16 +75,23 @@ function basaltHandler(definition, container, enrichers, guards) {
         }
         catch (error) {
             const { status, body } = toErrorResponse(error);
+            // This adapter previously reported nothing at all — a 500 reached the
+            // client and left no trace whatsoever on the server.
+            const entry = { error, status, code: body.error.code, method: req.method, url: req.originalUrl };
+            if (onError)
+                onError(entry);
+            else
+                reportHttpError(entry);
             if (!res.headersSent)
                 res.status(status).json(body);
         }
     };
 }
 /** Mounts Basalt routes on an Express app (usable without the plugin). */
-export function registerRoutes(app, routes, container, enrichers = [], guards = []) {
+export function registerRoutes(app, routes, container, enrichers = [], guards = [], onError) {
     const router = app;
     for (const definition of routes) {
-        router[definition.method.toLowerCase()](definition.url, basaltHandler(definition, container, enrichers, guards));
+        router[definition.method.toLowerCase()](definition.url, basaltHandler(definition, container, enrichers, guards, onError));
     }
 }
 /**
@@ -134,7 +141,7 @@ export function expressPlugin(options = {}) {
                         return;
                     next();
                 });
-                registerRoutes(app, routes, container, enrichers, guards);
+                registerRoutes(app, routes, container, enrichers, guards, options.onError);
                 for (const { method, url, handler } of collector.extraRoutes) {
                     router[method.toLowerCase()](url, async (req, res) => {
                         const reply = new ExpressReply(res);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@basaltkit/express",
-  "version": "1.4.1",
+  "version": "1.5.0",
   "engines": {
     "node": ">=22.5.0"
   },
@@ -19,7 +19,7 @@
   ],
   "dependencies": {
     "@basaltkit/core": "^1.3.1",
-    "@basaltkit/http": "^1.14.0"
+    "@basaltkit/http": "^1.15.0"
   },
   "peerDependencies": {
     "express": "^4.19.0 || ^5.0.0"