@onlineapps/service-wrapper 2.3.3 → 2.3.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlineapps/service-wrapper",
-  "version": "2.3.3",
+  "version": "2.3.5",
   "description": "Thin orchestration layer for microservices - delegates all infrastructure concerns to specialized connectors",
   "main": "src/index.js",
   "scripts": {
@@ -29,11 +29,11 @@
     "@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.106",
+    "@onlineapps/conn-orch-orchestrator": "1.0.108",
     "@onlineapps/conn-orch-registry": "1.1.55",
-    "@onlineapps/conn-orch-validator": "2.0.32",
+    "@onlineapps/conn-orch-validator": "2.0.33",
     "@onlineapps/monitoring-core": "1.0.21",
     "@onlineapps/service-common": "1.0.18",
     "@onlineapps/runtime-config": "1.0.2"

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');
@@ -1426,8 +1433,7 @@ class ServiceWrapper {
    */
   async _startWorkflowInitListener() {
     if (!this.mqClient) {
-      this.logger?.warn('Cannot start workflow.init listener: MQ client not initialized');
-      return;
+      throw new Error('[ServiceWrapper] Cannot start workflow.init listener — MQ client not initialized. Queue infrastructure is required.');
     }
 
     const queueName = 'workflow.init';
@@ -1459,8 +1465,7 @@ class ServiceWrapper {
    */
   async _startWorkflowControlListener() {
     if (!this.mqClient) {
-      this.logger?.warn('Cannot start workflow.control listener: MQ client not initialized');
-      return;
+      throw new Error('[ServiceWrapper] Cannot start workflow.control listener — MQ client not initialized. Queue infrastructure is required.');
     }
 
     const queueName = 'workflow.control';
@@ -1491,8 +1496,7 @@ class ServiceWrapper {
    */
   async _startServiceWorkflowListener() {
     if (!this.mqClient) {
-      this.logger?.warn('Cannot start service workflow listener: MQ client not initialized');
-      return;
+      throw new Error('[ServiceWrapper] Cannot start service workflow listener — MQ client not initialized. Queue infrastructure is required.');
     }
 
     const serviceName = this.config.service?.name || 'unnamed-service';