@usebruno/js 0.13.0 → 0.14.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@usebruno/js",
-  "version": "0.13.0",
+  "version": "0.14.0",
   "license": "MIT",
   "main": "src/index.js",
   "files": [
@@ -8,12 +8,13 @@
     "package.json"
   ],
   "peerDependencies": {
-    "@n8n/vm2": "^3.9.23"
+    "@usebruno/vm2": "^3.9.13"
   },
   "scripts": {
     "test": "jest --testPathIgnorePatterns test.js"
   },
   "dependencies": {
+    "@usebruno/common": "0.2.0",
     "@usebruno/query": "0.1.0",
     "ajv": "^8.12.0",
     "ajv-formats": "^2.1.1",
@@ -23,7 +24,6 @@
     "chai": "^4.3.7",
     "chai-string": "^1.5.0",
     "crypto-js": "^4.1.1",
-    "handlebars": "^4.7.8",
     "json-query": "^2.2.2",
     "lodash": "^4.17.21",
     "moment": "^2.29.4",

package/src/bru.js

@@ -1,6 +1,6 @@
-const Handlebars = require('handlebars');
 const { cloneDeep } = require('lodash');
 const { uuid } = require('./utils');
+const { interpolate } = require('@usebruno/common');
 
 const variableNameRegex = /^[\w-.]*$/;
 
@@ -12,31 +12,36 @@ class Bru {
     collectionPath,
     historyLogger,
     setHtmlVisualizationString,
-    secretVariables
+    secretVariables,
+    requestVariables
   ) {
-    this.envVariables = envVariables;
-    this.collectionVariables = collectionVariables;
+    this.envVariables = envVariables || {};
+    this.collectionVariables = collectionVariables || {};
     this.processEnvVars = cloneDeep(processEnvVars || {});
     this.secretVariables = cloneDeep(secretVariables || {});
+    this.requestVariables = requestVariables || {};
     this.collectionPath = collectionPath;
     this.historyLogger = historyLogger;
     this.setHtmlVisualizationString = setHtmlVisualizationString;
   }
 
-  _interpolateEnvVar = (str) => {
+  _interpolate = (str) => {
     if (!str || !str.length || typeof str !== 'string') {
       return str;
     }
 
-    const template = Handlebars.compile(str, { noEscape: true });
-
-    return template({
+    const combinedVars = {
+      ...this.envVariables,
+      ...this.requestVariables,
+      ...this.collectionVariables,
       process: {
         env: {
           ...this.processEnvVars
         }
       }
-    });
+    };
+
+    return interpolate(str, combinedVars);
   };
 
   cwd() {
@@ -51,8 +56,12 @@ class Bru {
     return this.processEnvVars[key];
   }
 
+  hasEnvVar(key) {
+    return Object.hasOwn(this.envVariables, key);
+  }
+
   getEnvVar(key) {
-    return this._interpolateEnvVar(this.envVariables[key]);
+    return this._interpolate(this.envVariables[key]);
   }
 
   setEnvVar(key, value) {
@@ -72,6 +81,10 @@ class Bru {
     this.envVariables[key] = value;
   }
 
+  hasVar(key) {
+    return Object.hasOwn(this.collectionVariables, key);
+  }
+
   setVar(key, value) {
     if (!key) {
       throw new Error('Creating a variable without specifying a name is not allowed.');
@@ -104,7 +117,15 @@ class Bru {
       );
     }
 
-    return this.collectionVariables[key];
+    return this._interpolate(this.collectionVariables[key]);
+  }
+
+  deleteVar(key) {
+    delete this.collectionVariables[key];
+  }
+
+  getRequestVar(key) {
+    return this._interpolate(this.requestVariables[key]);
   }
 
   setNextRequest(nextRequest) {

package/src/interpolate-string.js

@@ -1,44 +1,16 @@
-const Handlebars = require('handlebars');
-const { forOwn, cloneDeep } = require('lodash');
+const { interpolate } = require('@usebruno/common');
 
-const interpolateEnvVars = (str, processEnvVars) => {
+const interpolateString = (
+  str,
+  { envVariables = {}, collectionVariables = {}, processEnvVars = {}, requestVariables = {} }
+) => {
   if (!str || !str.length || typeof str !== 'string') {
     return str;
   }
 
-  const template = Handlebars.compile(str, { noEscape: true });
-
-  return template({
-    process: {
-      env: {
-        ...processEnvVars
-      }
-    }
-  });
-};
-
-const interpolateString = (str, { envVariables, collectionVariables, processEnvVars }) => {
-  if (!str || !str.length || typeof str !== 'string') {
-    return str;
-  }
-
-  processEnvVars = processEnvVars || {};
-  collectionVariables = collectionVariables || {};
-
-  // we clone envVariables because we don't want to modify the original object
-  envVariables = envVariables ? cloneDeep(envVariables) : {};
-
-  // envVariables can inturn have values as {{process.env.VAR_NAME}}
-  // so we need to interpolate envVariables first with processEnvVars
-  forOwn(envVariables, (value, key) => {
-    envVariables[key] = interpolateEnvVars(value, processEnvVars);
-  });
-
-  const template = Handlebars.compile(str, { noEscape: true });
-
-  // collectionVariables take precedence over envVariables
   const combinedVars = {
     ...envVariables,
+    ...requestVariables,
     ...collectionVariables,
     process: {
       env: {
@@ -47,7 +19,7 @@ const interpolateString = (str, { envVariables, collectionVariables, processEnvV
     }
   };
 
-  return template(combinedVars);
+  return interpolate(str, combinedVars);
 };
 
 module.exports = {

package/src/runtime/assert-runtime.js

@@ -66,6 +66,7 @@ chai.use(function (chai, utils) {
  * isNumber    : is number
  * isString    : is string
  * isBoolean   : is boolean
+ * isArray     : is array
  */
 const parseAssertionOperator = (str = '') => {
   if (!str || typeof str !== 'string' || !str.length) {
@@ -101,7 +102,8 @@ const parseAssertionOperator = (str = '') => {
     'isJson',
     'isNumber',
     'isString',
-    'isBoolean'
+    'isBoolean',
+    'isArray'
   ];
 
   const unaryOperators = [
@@ -114,7 +116,8 @@ const parseAssertionOperator = (str = '') => {
     'isJson',
     'isNumber',
     'isString',
-    'isBoolean'
+    'isBoolean',
+    'isArray'
   ];
 
   const [operator, ...rest] = str.trim().split(' ');
@@ -151,7 +154,8 @@ const isUnaryOperator = (operator) => {
     'isJson',
     'isNumber',
     'isString',
-    'isBoolean'
+    'isBoolean',
+    'isArray'
   ];
 
   return unaryOperators.includes(operator);
@@ -163,6 +167,7 @@ const evaluateRhsOperand = (rhsOperand, operator, context) => {
   }
 
   const interpolationContext = {
+    requestVariables: context.bru.requestVariables,
     collectionVariables: context.bru.collectionVariables,
     envVariables: context.bru.envVariables,
     processEnvVars: context.bru.processEnvVars
@@ -205,10 +210,11 @@ class AssertRuntime {
     response,
     envVariables,
     collectionVariables,
-    collectionPath,
+    processEnvVars,
     historyLogger,
     secretVariables
   ) {
+    const requestVariables = request?.requestVariables || {};
     const enabledAssertions = _.filter(assertions, (a) => a.enabled);
     if (!enabledAssertions.length) {
       return [];
@@ -217,11 +223,12 @@ class AssertRuntime {
     const bru = new Bru(
       envVariables,
       collectionVariables,
-      undefined, // processEnvVars,
+      processEnvVars,
       undefined, // collectionPath,
       undefined, // historyLogger,
       undefined, // setHtmlVisualizationString,
-      secretVariables
+      secretVariables,
+      requestVariables
     );
     const req = new BrunoRequest(request, historyLogger);
     const res = createResponseParser(response);
@@ -234,7 +241,9 @@ class AssertRuntime {
 
     const context = {
       ...envVariables,
+      ...requestVariables,
       ...collectionVariables,
+      ...processEnvVars,
       ...bruContext
     };
 
@@ -330,6 +339,9 @@ class AssertRuntime {
           case 'isBoolean':
             expect(lhs).to.be.a('boolean');
             break;
+          case 'isArray':
+            expect(lhs).to.be.a('array');
+            break;
           default:
             expect(lhs).to.equal(rhs);
             break;

package/src/runtime/script-runtime.js

@@ -1,4 +1,4 @@
-const { NodeVM } = require('vm2');
+const { NodeVM } = require('@usebruno/vm2');
 const path = require('path');
 const http = require('http');
 const https = require('https');
@@ -50,6 +50,7 @@ class ScriptRuntime {
     let setHtmlVisualizationString = (htmlString) => {
       htmlVisualizationString = htmlString;
     };
+    const requestVariables = request?.requestVariables || {};
     const bru = new Bru(
       envVariables,
       collectionVariables,
@@ -57,7 +58,8 @@ class ScriptRuntime {
       collectionPath,
       historyLogger,
       setHtmlVisualizationString,
-      secretVariables
+      secretVariables,
+      requestVariables
     );
     const req = new BrunoRequest(request, historyLogger);
     const allowScriptFilesystemAccess = get(scriptingConfig, 'filesystemAccess.allow', false);
@@ -162,6 +164,7 @@ class ScriptRuntime {
     let setHtmlVisualizationString = (htmlString) => {
       htmlVisualizationString = htmlString;
     };
+    const requestVariables = request?.requestVariables || {};
     const bru = new Bru(
       envVariables,
       collectionVariables,
@@ -169,7 +172,8 @@ class ScriptRuntime {
       collectionPath,
       historyLogger,
       setHtmlVisualizationString,
-      secretVariables
+      secretVariables,
+      requestVariables
     );
     const req = new BrunoRequest(request, historyLogger);
     const res = new BrunoResponse(response);

package/src/runtime/test-runtime.js

@@ -1,4 +1,4 @@
-const { NodeVM } = require('vm2');
+const { NodeVM } = require('@usebruno/vm2');
 const chai = require('chai');
 const path = require('path');
 const http = require('http');
@@ -47,6 +47,7 @@ class TestRuntime {
     historyLogger,
     secretVariables
   ) {
+    const requestVariables = request?.requestVariables || {};
     const bru = new Bru(
       envVariables,
       collectionVariables,
@@ -54,7 +55,8 @@ class TestRuntime {
       collectionPath,
       historyLogger,
       undefined, // setHtmlVisualizationString
-      secretVariables
+      secretVariables,
+      requestVariables
     );
     const req = new BrunoRequest(request, historyLogger);
     const res = new BrunoResponse(response);

package/src/runtime/vars-runtime.js

@@ -14,6 +14,9 @@ class VarsRuntime {
     historyLogger,
     secretVars = {}
   ) {
+    if (!request?.requestVariables) {
+      request.requestVariables = {};
+    }
     const enabledVars = _.filter(vars, (v) => v.enabled);
     if (!enabledVars.length) {
       return;
@@ -44,12 +47,8 @@ class VarsRuntime {
 
     _.each(enabledVars, (v) => {
       const value = evaluateJsTemplateLiteral(v.value, context);
-      bru.setVar(v.name, value);
+      request?.requestVariables && (request.requestVariables[v.name] = value);
     });
-
-    return {
-      collectionVariables
-    };
   }
 
   runPostResponseVars(
@@ -63,6 +62,7 @@ class VarsRuntime {
     historyLogger,
     secretVars = {}
   ) {
+    const requestVariables = request?.requestVariables || {};
     const enabledVars = _.filter(vars, (v) => v.enabled);
     if (!enabledVars.length) {
       return;
@@ -75,7 +75,8 @@ class VarsRuntime {
       null,
       historyLogger,
       undefined, // setHtmlVisualizationString
-      secretVars
+      secretVars,
+      requestVariables
     );
     const req = new BrunoRequest(request, historyLogger);
     const res = createResponseParser(response);
@@ -93,14 +94,27 @@ class VarsRuntime {
       ...bruContext
     };
 
+    const errors = new Map();
     _.each(enabledVars, (v) => {
-      const value = evaluateJsExpression(v.value, context);
-      bru.setVar(v.name, value);
+      try {
+        const value = evaluateJsExpression(v.value, context);
+        bru.setVar(v.name, value);
+      } catch (error) {
+        errors.set(v.name, error);
+      }
     });
 
+    let error = null;
+    if (errors.size > 0) {
+      // Format all errors as a single string to be displayed in a toast
+      const errorMessage = [...errors.entries()].map(([name, err]) => `${name}: ${err.message ?? err}`).join('\n');
+      error = `${errors.size} error${errors.size === 1 ? '' : 's'} in post response variables: \n${errorMessage}`;
+    }
+
     return {
       envVariables,
-      collectionVariables
+      collectionVariables,
+      error
     };
   }
 }