@onlineapps/conn-orch-api-mapper 1.0.9 → 1.0.11

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlineapps/conn-orch-api-mapper",
-  "version": "1.0.9",
+  "version": "1.0.11",
   "description": "API mapping connector for OA Drive - maps cookbook operations to HTTP endpoints",
   "main": "src/index.js",
   "scripts": {

package/src/ApiMapper.js

@@ -260,11 +260,50 @@ class ApiMapper {
   /**
    * Convert handler output to Descriptor
    * @private
+   * 
+   * Supported formats from handler/HTTP response:
+   * 1. { data: base64_string, encoding: 'base64', filename, content_type } - HTTP transport format
+   * 2. { temp_path, filename, content_type } - Local file (only for same-process calls)
+   * 3. Buffer - Raw binary data
+   * 4. string - Text content
    */
   async _convertToDescriptor(output, schema, getContentResolver, logPrefix, context, fieldName = null) {
     const fieldLabel = fieldName ? `field '${fieldName}'` : 'output';
     
-    // Handler should return object with temp_path for file operations
+    // Format 1: Base64-encoded data from HTTP response (primary format for distributed services)
+    if (output && typeof output === 'object' && output.data && output.encoding === 'base64') {
+      console.log(`${logPrefix}:PROCESS] ${JSON.stringify({
+        action: 'convert_base64',
+        field: fieldName,
+        filename: output.filename,
+        size: output.size
+      })}`);
+      
+      // Decode base64 to Buffer
+      const buffer = Buffer.from(output.data, 'base64');
+      
+      const resolver = getContentResolver();
+      const descriptor = await resolver.createDescriptor(buffer, {
+        filename: output.filename || schema.filename || 'output.bin',
+        content_type: output.content_type || schema.content_type,
+        context: {
+          workflow_id: context.workflow_id || 'standalone',
+          step_id: context.step_id || 'api-mapper'
+        },
+        forceFile: true  // Binary data → always store as file
+      });
+      
+      console.log(`${logPrefix}:PROCESS] ${JSON.stringify({
+        action: 'descriptor_created',
+        field: fieldName,
+        storage_ref: descriptor.storage_ref,
+        size: descriptor.size
+      })}`);
+      
+      return descriptor;
+    }
+    
+    // Format 2: Temp file path (for same-process calls, e.g. tests)
     if (output && typeof output === 'object' && output.temp_path) {
       console.log(`${logPrefix}:PROCESS] ${JSON.stringify({
         action: 'convert_temp_file',
@@ -294,7 +333,7 @@ class ApiMapper {
       return descriptor;
     }
     
-    // Handler returned Buffer - store it
+    // Format 3: Raw Buffer
     if (Buffer.isBuffer(output)) {
       console.log(`${logPrefix}:PROCESS] ${JSON.stringify({
         action: 'convert_buffer',
@@ -314,7 +353,7 @@ class ApiMapper {
       });
     }
     
-    // Handler returned string (for content type) - may be inline or stored
+    // Format 4: Plain string (text content)
     if (typeof output === 'string') {
       console.log(`${logPrefix}:PROCESS] ${JSON.stringify({
         action: 'convert_string',
@@ -338,11 +377,11 @@ class ApiMapper {
     console.error(`${logPrefix}:FAIL] ${JSON.stringify({
       error: 'UNEXPECTED_OUTPUT_FORMAT',
       field: fieldName,
-      expected: 'object with temp_path, Buffer, or string',
+      expected: '{ data, encoding: "base64" } or { temp_path } or Buffer or string',
       received: typeof output,
       receivedKeys: output && typeof output === 'object' ? Object.keys(output) : null
     })}`);
-    throw new Error(`[ApiMapper] FAIL-FAST: ${errorMsg}. Expected temp_path/Buffer/string, got ${typeof output}`);
+    throw new Error(`[ApiMapper] FAIL-FAST: ${errorMsg}. Expected base64/temp_path/Buffer/string, got ${typeof output}`);
   }
 
   /**