@onlineapps/service-wrapper 2.1.60 → 2.1.61

package/package.json

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

package/src/index.js

@@ -67,6 +67,35 @@ async function bootstrap(serviceRoot, options = {}) {
   });
   app.use(accountContextMw);
 
+  // Express routes are processed in registration order.
+  // Services usually register routes inside src/app BEFORE bootstrap runs, so app.use() here would be too late.
+  // We must enforce account context BEFORE routes: move our middleware to the beginning of the router stack.
+  // Fail-fast if we cannot guarantee correct order.
+  if (!app || !app._router || !Array.isArray(app._router.stack) || app._router.stack.length === 0) {
+    throw new Error(
+      `[service-wrapper][AccountContext] Cannot enforce account context - Express router stack not available. ` +
+      `Fix: ensure the service exports an Express app instance with registered routes before bootstrap.`
+    );
+  }
+
+  const stack = app._router.stack;
+  const lastLayer = stack[stack.length - 1];
+  const isOurMiddleware =
+    lastLayer &&
+    lastLayer.handle &&
+    typeof lastLayer.handle === 'function' &&
+    lastLayer.handle.name === accountContextMw.name;
+
+  if (!isOurMiddleware) {
+    throw new Error(
+      `[service-wrapper][AccountContext] Cannot enforce account context - middleware placement is not deterministic. ` +
+      `Fix: ensure bootstrap registers account context middleware before routes, or update service to export app factory.`
+    );
+  }
+
+  stack.pop();
+  stack.unshift(lastLayer);
+
   // 1. Start HTTP server
   const PORT = config.service.port;
   const server = app.listen(PORT, () => {