@onlineapps/conn-orch-api-mapper 1.0.26 → 1.0.28

package/API.md

@@ -67,7 +67,7 @@ Create API mapper instance
 ```js
 const apiMapper = create({
   openApiSpec: require('./openapi.json'),
-  serviceUrl: 'http://localhost:3000'
+  serviceUrl: 'http://127.0.0.1:3000'
 });
 ```
 <a name="ApiMapper"></a>
@@ -98,7 +98,7 @@ Create a new ApiMapper instance
 | --- | --- | --- | --- |
 | config | <code>Object</code> |  | Configuration object |
 | config.openApiSpec | <code>Object</code> \| <code>string</code> |  | OpenAPI specification object or path |
-| [config.serviceUrl] | <code>string</code> | <code>&quot;&#x27;http://localhost:3000&#x27;&quot;</code> | Base URL of the service |
+| config.serviceUrl | <code>string</code> |  | Base URL of the service (required; no defaults) |
 | [config.service] | <code>Object</code> |  | Express app instance for direct calls |
 | [config.directCall] | <code>boolean</code> | <code>false</code> | Use direct Express calls instead of HTTP |
 | [config.logger] | <code>Object</code> |  | Logger instance |
@@ -139,7 +139,7 @@ Create a new ApiMapper instance
 | --- | --- | --- | --- |
 | config | <code>Object</code> |  | Configuration object |
 | config.openApiSpec | <code>Object</code> \| <code>string</code> |  | OpenAPI specification object or path |
-| [config.serviceUrl] | <code>string</code> | <code>&quot;&#x27;http://localhost:3000&#x27;&quot;</code> | Base URL of the service |
+| config.serviceUrl | <code>string</code> |  | Base URL of the service (required; no defaults) |
 | [config.service] | <code>Object</code> |  | Express app instance for direct calls |
 | [config.directCall] | <code>boolean</code> | <code>false</code> | Use direct Express calls instead of HTTP |
 | [config.logger] | <code>Object</code> |  | Logger instance |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlineapps/conn-orch-api-mapper",
-  "version": "1.0.26",
+  "version": "1.0.28",
   "description": "API mapping connector for OA Drive - maps cookbook operations to HTTP endpoints",
   "main": "src/index.js",
   "scripts": {
@@ -21,7 +21,7 @@
   "license": "MIT",
   "dependencies": {
     "axios": "^1.4.0",
-    "@onlineapps/content-resolver": "1.1.14"
+    "@onlineapps/content-resolver": "1.1.15"
   },
   "devDependencies": {
     "jest": "^29.5.0",

package/src/ApiMapper.js

@@ -17,11 +17,10 @@ class ApiMapper {
    * @constructor
    * @param {Object} config - Configuration object
    * @param {Object|string} config.openApiSpec - OpenAPI specification object or path
-   * @param {string} [config.serviceUrl='http://localhost:3000'] - Base URL of the service
+   * @param {string} config.serviceUrl - Base URL of the service (required; no topology defaults)
    * @param {Object} [config.service] - Express app instance for direct calls
    * @param {boolean} [config.directCall=false] - Use direct Express calls instead of HTTP
    * @param {Object} [config.logger] - Logger instance
-   * @param {number} [config.port=3000] - Service port for direct calls
    *
    * @example
    * const apiMapper = new ApiMapper({
@@ -34,8 +33,12 @@ class ApiMapper {
       throw new Error('OpenAPI specification is required');
     }
 
+    if (typeof config.serviceUrl !== 'string' || config.serviceUrl.trim().length === 0) {
+      throw new Error('[ApiMapper] Missing configuration - serviceUrl is required (no defaults)');
+    }
+
     this.openApiSpec = config.openApiSpec;
-    this.serviceUrl = config.serviceUrl || `http://localhost:${config.port || 3000}`;
+    this.serviceUrl = config.serviceUrl;
     this.service = config.service; // Express app for direct calls
     this.directCall = config.directCall === true;
     this.logger = config.logger || console;
@@ -557,6 +560,9 @@ class ApiMapper {
         operations[operationId] = {
           method: (operation.method || 'POST').toUpperCase(),
           path: operation.endpoint || `/${operationId}`,
+          // Static headers from operations.json (e.g. x-validation-request).
+          // NOTE: account-id is controlled by workflow context (_system.account_id) and must not be overridden here.
+          headers: operation.headers || null,
           // Store original operations.json input schema for type-driven descriptor handling
           input: operation.input || null,
           parameters: operation.input ? this._convertOperationsJsonInputToOpenApi(operation.input) : [],
@@ -693,6 +699,27 @@ class ApiMapper {
       data: null
     };
 
+    // operations.json static headers (if present)
+    if (operation && operation.headers && typeof operation.headers === 'object') {
+      Object.entries(operation.headers).forEach(([k, v]) => {
+        if (!k) return;
+        // Never allow operation-level config to override tenant header
+        if (String(k).toLowerCase() === 'account-id') return;
+
+        // Optional: allow ${context.path} variable resolution (no env side-effects here).
+        if (typeof v === 'string' && v.startsWith('${') && v.endsWith('}')) {
+          const path = v.slice(2, -1);
+          const resolved = this._getValueFromPath(context, path);
+          if (resolved !== undefined) {
+            request.headers[k] = resolved;
+          }
+          return;
+        }
+
+        request.headers[k] = v;
+      });
+    }
+
     // Process path parameters
     operation.parameters.forEach(param => {
       const value = input[param.name];

package/src/index.js

@@ -27,7 +27,7 @@ const ApiMapper = require('./ApiMapper');
  * @example
  * const apiMapper = create({
  *   openApiSpec: require('./openapi.json'),
- *   serviceUrl: 'http://localhost:3000'
+ *   serviceUrl: 'http://127.0.0.1:3000'
  * });
  */
 function create(config) {