@onlineapps/service-wrapper 2.3.4 → 2.3.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlineapps/service-wrapper",
-  "version": "2.3.4",
+  "version": "2.3.5",
   "description": "Thin orchestration layer for microservices - delegates all infrastructure concerns to specialized connectors",
   "main": "src/index.js",
   "scripts": {
@@ -29,9 +29,9 @@
     "@onlineapps/conn-base-monitoring": "1.0.10",
     "@onlineapps/conn-infra-error-handler": "1.0.9",
     "@onlineapps/conn-infra-mq": "1.1.69",
-    "@onlineapps/conn-orch-api-mapper": "1.0.31",
+    "@onlineapps/conn-orch-api-mapper": "1.0.32",
     "@onlineapps/conn-orch-cookbook": "2.0.38",
-    "@onlineapps/conn-orch-orchestrator": "1.0.107",
+    "@onlineapps/conn-orch-orchestrator": "1.0.108",
     "@onlineapps/conn-orch-registry": "1.1.55",
     "@onlineapps/conn-orch-validator": "2.0.33",
     "@onlineapps/monitoring-core": "1.0.21",

package/src/ServiceWrapper.js

@@ -1396,15 +1396,22 @@ class ServiceWrapper {
     // Store error handler for use in message processing
     this.errorHandler = errorHandler;
 
-    // Create API mapper for HTTP calls (never direct calls)
+    if (!this.logger) {
+      throw new Error('[ServiceWrapper] Logger not initialized — cannot create orchestrator');
+    }
+
     const apiMapper = ApiMapperConnector.create({
-      openApiSpec: this.operations, // Using operations.json instead of OpenAPI
+      openApiSpec: this.operations,
       serviceUrl: serviceUrl,
-      directCall: false, // ALWAYS use HTTP pattern
-      logger: this.logger || console
+      directCall: false,
+      logger: this.logger
     });
 
-    // Create orchestrator
+    const timeout = this.config.wrapper?.timeout;
+    if (!timeout || typeof timeout !== 'number') {
+      throw new Error('[ServiceWrapper] config.wrapper.timeout is required — Expected number (ms)');
+    }
+
     this.orchestrator = OrchestratorConnector.create({
       mqClient: this.mqClient,
       registryClient: this.registryClient,
@@ -1412,8 +1419,8 @@ class ServiceWrapper {
       cookbook: CookbookConnector,
       cache: this.cacheConnector,
       errorHandler: errorHandler,
-      logger: this.logger || console,
-      defaultTimeout: this.config.wrapper?.timeout || 30000
+      logger: this.logger,
+      defaultTimeout: timeout
     });
 
     this.logger?.info('Orchestrator initialized');