@onlineapps/service-wrapper 2.1.113 → 2.1.114

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlineapps/service-wrapper",
-  "version": "2.1.113",
+  "version": "2.1.114",
   "description": "Thin orchestration layer for microservices - delegates all infrastructure concerns to specialized connectors",
   "main": "src/index.js",
   "scripts": {

package/src/ServiceWrapper.js

@@ -1078,7 +1078,7 @@ class ServiceWrapper {
   _setupHealthChecks() {
     const healthEndpoint = this.config.wrapper?.health?.endpoint || '/health';
 
-    this.app.get(healthEndpoint, async (req, res) => {
+    const healthHandler = async (req, res) => {
       const health = {
         status: 'healthy',
         service: this.config.service?.name || 'unnamed-service',
@@ -1113,7 +1113,27 @@ class ServiceWrapper {
       }
 
       res.json(health);
-    });
+    };
+
+    // Insert health route BEFORE catch-all 404/error handlers in Express stack.
+    // Services register routes in app.js (loaded before ServiceWrapper), including
+    // 404 middleware. A plain app.get() would add AFTER the 404 handler and never match.
+    this.app.get(healthEndpoint, healthHandler);
+
+    const stack = this.app._router?.stack;
+    if (stack && stack.length > 1) {
+      const healthLayer = stack.pop();
+      // Find first catch-all middleware (no route, not a built-in parser)
+      const builtins = new Set(['query', 'expressInit', 'jsonParser', 'urlencodedParser', 'tenantContext', 'router']);
+      let insertAt = stack.length;
+      for (let i = 0; i < stack.length; i++) {
+        if (!stack[i].route && !builtins.has(stack[i].name)) {
+          insertAt = i;
+          break;
+        }
+      }
+      stack.splice(insertAt, 0, healthLayer);
+    }
 
     this.logger?.info(`Health check endpoint registered at ${healthEndpoint}`);