arazzo-runner 0.0.12 → 0.0.13

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "arazzo-runner",
-  "version": "0.0.12",
+  "version": "0.0.13",
   "description": "A runner to run through Arazzo Document workflows",
   "main": "index.js",
   "scripts": {
@@ -40,6 +40,7 @@
     "@swaggerexpert/arazzo-runtime-expression": "^1.0.1",
     "@swaggerexpert/json-pointer": "^2.10.2",
     "ajv": "^8.17.1",
+    "fast-xml-parser": "^5.3.3",
     "jsonpath": "^1.1.1",
     "openapi-params": "^0.0.5",
     "openapi-server-url-templating": "^1.3.0",

package/src/Arazzo.js

@@ -348,9 +348,9 @@ class Arazzo extends Document {
     } else {
       this.logger.verbose("Using fetch call");
 
-      this.logger.verbose(`url: ${url}`);
-      this.logger.verbose(`method: ${options.method}`);
-      this.logger.verbose("headers:");
+      this.logger.verbose(`request url: ${url}`);
+      this.logger.verbose(`request method: ${options.method}`);
+      this.logger.verbose("request headers:");
       for (const [key, value] of options.headers.entries()) {
         this.logger.verbose(`${key}: ${value}`);
       }
@@ -360,6 +360,12 @@ class Arazzo extends Document {
       response = await fetch(url, options);
     }
 
+    this.logger.verbose(`received StatusCode: ${response.status}`);
+    this.logger.verbose(`received headers:`);
+    for (const [key, value] of response.headers.entries()) {
+      this.logger.verbose(`${key}: ${value}`);
+    }
+
     if (response.headers.has("retry-after")) {
       const retryAfter = parseRetryAfter(response.headers.get("retry-after"));
       if (retryAfter !== null) {
@@ -504,32 +510,48 @@ class Arazzo extends Document {
    * @param {*} response
    */
   async dealWithResponse(response) {
+    if (
+      response.headers.has("Content-Type") &&
+      response.headers.get("Content-Type") === "application/json"
+    ) {
+      const json = await response?.json().catch((err) => {
+        this.logger.error(
+          `Error trying to resolve ${this.step.stepId} outputs`,
+        );
+        throw new Error(err);
+      });
+
+      this.expression.addToContext("response.body", json);
+    } else {
+      const body = await response.body;
+
+      this.expression.addToContext("response.body", body);
+    }
+
     this.doNotProcessStep = false;
     this.alreadyProcessingOnFailure = false;
 
     if (this.step.successCriteria) {
-      if (this.step.successCriteria) {
-        const passedSuccessCriteria = this.hasPassedSuccessCriteria();
-
-        if (passedSuccessCriteria) {
-          this.logger.success("All criteria checks passed");
-          if (this.currentRetryRule) {
-            if (this.retryContext.doNotDeleteRetryLimits) {
-              this.retryLimits[this.currentRetryRule] = 0;
-              this.logger.notice("Retries stopped");
-            }
+      const passedSuccessCriteria = this.hasPassedSuccessCriteria();
+
+      if (passedSuccessCriteria) {
+        this.logger.success("All criteria checks passed");
+        if (this.currentRetryRule) {
+          if (this.retryContext.doNotDeleteRetryLimits) {
+            this.retryLimits[this.currentRetryRule] = 0;
+            this.logger.notice("Retries stopped");
           }
+        }
 
-          await this.dealWithPassedRule(response);
+        await this.dealWithPassedRule(response);
+      } else {
+        this.logger.error("Not all criteria checks passed");
+        if (this.step.onFailure) {
+          await this.dealWithFailedRule();
         } else {
-          this.logger.error("Not all criteria checks passed");
-          if (this.step.onFailure) {
-            await this.dealWithFailedRule();
-          } else {
-            throw new Error(
-              `${this.step.stepId} step of the ${this.workflow.workflowId} workflow failed the successCriteria`,
-            );
-          }
+          throw new Error(
+            `${this.step.stepId} step of the ${this.workflow.workflowId} workflow failed the successCriteria`,
+          );
         }
       }
     } else {
@@ -843,14 +865,7 @@ class Arazzo extends Document {
    * @private
    * @param {*} response
    */
-  async dealWithStepOutputs(response) {
-    const json = await response?.json().catch((err) => {
-      this.logger.error(`Error trying to resolve ${this.step.stepId} outputs`);
-      throw new Error(err);
-    });
-
-    this.expression.addToContext("response.body", json);
-
+  async dealWithStepOutputs() {
     const outputs = {};
     for (const key in this.step.outputs) {
       const value = this.expression.resolveExpression(this.step.outputs[key]);

package/src/Expression.js

@@ -242,6 +242,18 @@ class Expression {
   //   return expression;
   // }
 
+  /**
+   * @private
+   * @param {*} value
+   * @returns {boolean}
+   */
+  isXML(value) {
+    return (
+      typeof value === "string" &&
+      (value.trim().startsWith("<?xml") || value.trim().startsWith("<"))
+    );
+  }
+
   /**
    * Resolves a runtime expression to its value in the context
    * @public
@@ -470,6 +482,18 @@ class Expression {
         throw new Error(`Context path '${normalised}' not found`);
       }
 
+      // NEW: Check if contextData is XML
+      if (this.isXML(contextData)) {
+        const { XMLParser } = require("fast-xml-parser");
+        const parser = new XMLParser({
+          ignoreAttributes: false,
+          attributeNamePrefix: "@_",
+        });
+        const jsonData = parser.parse(contextData);
+        return evaluate(jsonData, pointer);
+      }
+
+      // Regular JSON pointer evaluation
       try {
         return evaluate(contextData, pointer);
       } catch (err) {