@budibase/server 0.9.169-alpha.9 → 0.9.172

package/dist/api/controllers/backup.js

@@ -8,13 +8,13 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
         step((generator = generator.apply(thisArg, _arguments || [])).next());
     });
 };
-const { streamBackup } = require("../../utilities/fileSystem");
+const { performBackup } = require("../../utilities/fileSystem");
 exports.exportAppDump = function (ctx) {
     return __awaiter(this, void 0, void 0, function* () {
         const { appId } = ctx.query;
-        const appName = decodeURI(ctx.query.appname);
-        const backupIdentifier = `${appName}-export-${new Date().getTime()}.txt`;
+        const appname = decodeURI(ctx.query.appname);
+        const backupIdentifier = `${appname}Backup${new Date().getTime()}.txt`;
         ctx.attachment(backupIdentifier);
-        ctx.body = yield streamBackup(appId);
+        ctx.body = yield performBackup(appId, backupIdentifier);
     });
 };

package/dist/api/controllers/dev.js

@@ -16,11 +16,11 @@ const { request } = require("../../utilities/workerRequests");
 const { clearLock } = require("../../utilities/redis");
 const { Replication } = require("@budibase/auth").db;
 const { DocumentTypes } = require("../../db/utils");
-function redirect(ctx, method, path = "global") {
+function redirect(ctx, method) {
     return __awaiter(this, void 0, void 0, function* () {
         const { devPath } = ctx.params;
         const queryString = ctx.originalUrl.split("?")[1] || "";
-        const response = yield fetch(checkSlashesInUrl(`${env.WORKER_URL}/api/${path}/${devPath}?${queryString}`), request(ctx, {
+        const response = yield fetch(checkSlashesInUrl(`${env.WORKER_URL}/api/global/${devPath}?${queryString}`), request(ctx, {
             method,
             body: ctx.request.body,
         }, true));
@@ -45,21 +45,15 @@ function redirect(ctx, method, path = "global") {
         ctx.cookies;
     });
 }
-exports.buildRedirectGet = path => {
-    return (ctx) => __awaiter(void 0, void 0, void 0, function* () {
-        yield redirect(ctx, "GET", path);
-    });
-};
-exports.buildRedirectPost = path => {
-    return (ctx) => __awaiter(void 0, void 0, void 0, function* () {
-        yield redirect(ctx, "POST", path);
-    });
-};
-exports.buildRedirectDelete = path => {
-    return (ctx) => __awaiter(void 0, void 0, void 0, function* () {
-        yield redirect(ctx, "DELETE", path);
-    });
-};
+exports.redirectGet = (ctx) => __awaiter(void 0, void 0, void 0, function* () {
+    yield redirect(ctx, "GET");
+});
+exports.redirectPost = (ctx) => __awaiter(void 0, void 0, void 0, function* () {
+    yield redirect(ctx, "POST");
+});
+exports.redirectDelete = (ctx) => __awaiter(void 0, void 0, void 0, function* () {
+    yield redirect(ctx, "DELETE");
+});
 exports.clearLock = (ctx) => __awaiter(void 0, void 0, void 0, function* () {
     const { appId } = ctx.params;
     try {

package/dist/api/controllers/permission.js

@@ -132,7 +132,6 @@ exports.getResourcePerms = function (ctx) {
             for (let role of roles) {
                 const rolePerms = role.permissions;
                 if (rolePerms &&
-                    rolePerms[resourceId] &&
                     (rolePerms[resourceId] === level ||
                         rolePerms[resourceId].indexOf(level) !== -1)) {
                     permissions[level] = getExternalRoleID(role._id);

package/dist/api/controllers/query.js

@@ -14,7 +14,6 @@ const { generateQueryID, getQueryParams } = require("../../db/utils");
 const { integrations } = require("../../integrations");
 const { BaseQueryVerbs } = require("../../constants");
 const env = require("../../environment");
-const ScriptRunner = require("../../utilities/scriptRunner");
 // simple function to append "readable" to all read queries
 function enrichQueries(input) {
     const wasArray = Array.isArray(input);
@@ -36,29 +35,11 @@ function formatResponse(resp) {
             resp = { response: resp };
         }
     }
+    if (!Array.isArray(resp)) {
+        resp = [resp];
+    }
     return resp;
 }
-function runAndTransform(integration, queryVerb, enrichedQuery, transformer) {
-    return __awaiter(this, void 0, void 0, function* () {
-        let rows = formatResponse(yield integration[queryVerb](enrichedQuery));
-        // transform as required
-        if (transformer) {
-            const runner = new ScriptRunner(transformer, { data: rows });
-            rows = runner.execute();
-        }
-        // needs to an array for next step
-        if (!Array.isArray(rows)) {
-            rows = [rows];
-        }
-        // map into JSON if just raw primitive here
-        if (rows.find(row => typeof row !== "object")) {
-            rows = rows.map(value => ({ value }));
-        }
-        // get all the potential fields in the schema
-        let keys = rows.flatMap(Object.keys);
-        return { rows, keys };
-    });
-}
 exports.fetch = function (ctx) {
     return __awaiter(this, void 0, void 0, function* () {
         const db = new CouchDB(ctx.appId);
@@ -138,10 +119,12 @@ exports.preview = function (ctx) {
         if (!Integration) {
             ctx.throw(400, "Integration type does not exist.");
         }
-        const { fields, parameters, queryVerb, transformer } = ctx.request.body;
+        const { fields, parameters, queryVerb } = ctx.request.body;
         const enrichedQuery = yield enrichQueryFields(fields, parameters);
         const integration = new Integration(datasource.config);
-        const { rows, keys } = yield runAndTransform(integration, queryVerb, enrichedQuery, transformer);
+        const rows = formatResponse(yield integration[queryVerb](enrichedQuery));
+        // get all the potential fields in the schema
+        const keys = rows.flatMap(Object.keys);
         ctx.body = {
             rows,
             schemaFields: [...new Set(keys)],
@@ -164,8 +147,7 @@ exports.execute = function (ctx) {
         const enrichedQuery = yield enrichQueryFields(query.fields, ctx.request.body.parameters);
         const integration = new Integration(datasource.config);
         // call the relevant CRUD method on the integration class
-        const { rows } = yield runAndTransform(integration, query.queryVerb, enrichedQuery, query.transformer);
-        ctx.body = rows;
+        ctx.body = formatResponse(yield integration[query.queryVerb](enrichedQuery));
         // cleanup
         if (integration.end) {
             integration.end();

package/dist/api/controllers/row/ExternalRequest.js

@@ -22,10 +22,11 @@ var __rest = (this && this.__rest) || function (s, e) {
 Object.defineProperty(exports, "__esModule", { value: true });
 const datasource_1 = require("../../../definitions/datasource");
 const utils_1 = require("../../../integrations/utils");
+const constants_1 = require("../../../constants");
 var External;
 (function (External) {
     const { makeExternalQuery } = require("./utils");
-    const { DataSourceOperation, FieldTypes, RelationshipTypes } = require("../../../constants");
+    const { DataSourceOperation, FieldTypes } = require("../../../constants");
     const { breakExternalTableId, isSQL } = require("../../../integrations/utils");
     const { processObjectSync } = require("@budibase/string-templates");
     const { cloneDeep } = require("lodash/fp");
@@ -66,46 +67,6 @@ var External;
             equal,
         };
     }
-    /**
-     * This function checks the incoming parameters to make sure all the inputs are
-     * valid based on on the table schema. The main thing this is looking for is when a
-     * user has made use of the _id field of a row for a foreign key or a search parameter.
-     * In these cases the key will be sent up as [1], rather than 1. In these cases we will
-     * simplify it down to the requirements. This function is quite complex as we try to be
-     * relatively restrictive over what types of columns we will perform this action for.
-     */
-    function cleanupConfig(config, table) {
-        const primaryOptions = [
-            FieldTypes.STRING,
-            FieldTypes.LONGFORM,
-            FieldTypes.OPTIONS,
-            FieldTypes.NUMBER,
-        ];
-        // filter out fields which cannot be keys
-        const fieldNames = Object.entries(table.schema)
-            .filter(schema => primaryOptions.find(val => val === schema[1].type))
-            .map(([fieldName]) => fieldName);
-        const iterateObject = (obj) => {
-            for (let [field, value] of Object.entries(obj)) {
-                if (fieldNames.find(name => name === field) && utils_1.isRowId(value)) {
-                    obj[field] = utils_1.convertRowId(value);
-                }
-            }
-        };
-        // check the row and filters to make sure they aren't a key of some sort
-        if (config.filters) {
-            for (let filter of Object.values(config.filters)) {
-                if (typeof filter !== "object" || Object.keys(filter).length === 0) {
-                    continue;
-                }
-                iterateObject(filter);
-            }
-        }
-        if (config.row) {
-            iterateObject(config.row);
-        }
-        return config;
-    }
     function generateIdForRow(row, table) {
         const primary = table.primary;
         if (!row || !primary) {
@@ -355,7 +316,7 @@ var External;
                     if (field.type !== FieldTypes.LINK || !field.fieldName) {
                         continue;
                     }
-                    const isMany = field.relationshipType === RelationshipTypes.MANY_TO_MANY;
+                    const isMany = field.relationshipType === constants_1.RelationshipTypes.MANY_TO_MANY;
                     const tableId = isMany ? field.through : field.tableId;
                     const manyKey = field.throughFrom || primaryKey;
                     const fieldName = isMany ? manyKey : field.fieldName;
@@ -469,7 +430,7 @@ var External;
             }
             return fields;
         }
-        run(config) {
+        run({ id, row, filters, sort, paginate }) {
             return __awaiter(this, void 0, void 0, function* () {
                 const { appId, operation, tableId } = this;
                 let { datasourceId, tableName } = breakExternalTableId(tableId);
@@ -486,11 +447,9 @@ var External;
                 if (!table) {
                     throw `Unable to process query, table "${tableName}" not defined.`;
                 }
-                // look for specific components of config which may not be considered acceptable
-                let { id, row, filters, sort, paginate } = cleanupConfig(config, table);
+                // clean up row on ingress using schema
                 filters = buildFilters(id, filters || {}, table);
                 const relationships = this.buildRelationships(table);
-                // clean up row on ingress using schema
                 const processed = this.inputProcessing(row, table);
                 row = processed.row;
                 if (operation === DataSourceOperation.DELETE &&

package/dist/api/controllers/script.js

@@ -8,12 +8,24 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
         step((generator = generator.apply(thisArg, _arguments || [])).next());
     });
 };
-const ScriptRunner = require("../../utilities/scriptRunner");
+const fetch = require("node-fetch");
+const vm = require("vm");
+class ScriptExecutor {
+    constructor(body) {
+        const code = `let fn = () => {\n${body.script}\n}; out = fn();`;
+        this.script = new vm.Script(code);
+        this.context = vm.createContext(body.context);
+        this.context.fetch = fetch;
+    }
+    execute() {
+        this.script.runInContext(this.context);
+        return this.context.out;
+    }
+}
 exports.execute = function (ctx) {
     return __awaiter(this, void 0, void 0, function* () {
-        const { script, context } = ctx.request.body;
-        const runner = new ScriptRunner(script, context);
-        ctx.body = runner.execute();
+        const executor = new ScriptExecutor(ctx.request.body);
+        ctx.body = executor.execute();
     });
 };
 exports.save = function (ctx) {

package/dist/api/controllers/table/index.js

@@ -24,7 +24,8 @@ const linkRows = require("../../../db/linkedRows");
 const csvParser = require("../../../utilities/csvParser");
 const { getRowParams, getTableParams, generateTableID, getDatasourceParams, BudibaseInternalDB, } = require("../../../db/utils");
 const { FieldTypes } = require("../../../constants");
-const { TableSaveFunctions, getTable } = require("./utils");
+const { TableSaveFunctions, getExternalTable } = require("./utils");
+const { isExternalTable, breakExternalTableId, } = require("../../../integrations/utils");
 exports.fetch = function (ctx) {
     return __awaiter(this, void 0, void 0, function* () {
         const db = new CouchDB(ctx.appId);
@@ -43,8 +44,15 @@ exports.fetch = function (ctx) {
 };
 exports.find = function (ctx) {
     return __awaiter(this, void 0, void 0, function* () {
+        const db = new CouchDB(ctx.appId);
         const tableId = ctx.params.id;
-        ctx.body = yield getTable(ctx.appId, tableId);
+        if (isExternalTable(tableId)) {
+            let { datasourceId, tableName } = breakExternalTableId(tableId);
+            ctx.body = yield getExternalTable(ctx.appId, datasourceId, tableName);
+        }
+        else {
+            ctx.body = yield db.get(ctx.params.id);
+        }
     });
 };
 exports.save = function (ctx) {
@@ -73,9 +81,6 @@ exports.save = function (ctx) {
             for (let propKey of Object.keys(tableToSave.schema)) {
                 let column = tableToSave.schema[propKey];
                 let oldColumn = oldTable.schema[propKey];
-                if (oldColumn && oldColumn.type === "internal") {
-                    oldColumn.type = "auto";
-                }
                 if (oldColumn && oldColumn.type !== column.type) {
                     ctx.throw(400, "Cannot change the type of a column");
                 }

package/dist/api/controllers/table/utils.js

@@ -9,15 +9,13 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
     });
 };
 const CouchDB = require("../../../db");
+const linkRows = require("../../../db/linkedRows");
 const csvParser = require("../../../utilities/csvParser");
 const { getRowParams, generateRowID, InternalTables, } = require("../../../db/utils");
 const { isEqual } = require("lodash/fp");
 const { AutoFieldSubTypes, FieldTypes } = require("../../../constants");
 const { inputProcessing } = require("../../../utilities/rowProcessor");
 const { USERS_TABLE_SCHEMA } = require("../../../constants");
-const { isExternalTable, breakExternalTableId, } = require("../../../integrations/utils");
-const { getViews, saveView } = require("../view/utils");
-const viewTemplate = require("../view/viewBuilder");
 exports.checkForColumnUpdates = (db, oldTable, updatedTable) => __awaiter(void 0, void 0, void 0, function* () {
     let updatedRows = [];
     const rename = updatedTable._rename;
@@ -27,7 +25,6 @@ exports.checkForColumnUpdates = (db, oldTable, updatedTable) => __awaiter(void 0
     }
     // check for renaming of columns or deleted columns
     if (rename || deletedColumns.length !== 0) {
-        // Update all rows
         const rows = yield db.allDocs(getRowParams(updatedTable._id, null, {
             include_docs: true,
         }));
@@ -41,8 +38,6 @@ exports.checkForColumnUpdates = (db, oldTable, updatedTable) => __awaiter(void 0
             }
             return doc;
         });
-        // Update views
-        yield exports.checkForViewUpdates(db, updatedTable, rename, deletedColumns);
         delete updatedTable._rename;
     }
     return { rows: updatedRows, table: updatedTable };
@@ -91,9 +86,16 @@ exports.handleDataImport = (appId, user, table, dataImport) => __awaiter(void 0,
                     ];
                 }
             }
-            finalData.push(row);
+            // make sure link rows are up to date
+            finalData.push(linkRows.updateLinks({
+                appId,
+                eventType: linkRows.EventType.ROW_SAVE,
+                row,
+                tableId: row.tableId,
+                table,
+            }));
         }
-        yield db.bulkDocs(finalData);
+        yield db.bulkDocs(yield Promise.all(finalData));
         let response = yield db.put(table);
         table._rev = response._rev;
     }
@@ -202,79 +204,4 @@ exports.getExternalTable = (appId, datasourceId, tableName) => __awaiter(void 0,
     const entities = yield exports.getAllExternalTables(appId, datasourceId);
     return entities[tableName];
 });
-exports.getTable = (appId, tableId) => __awaiter(void 0, void 0, void 0, function* () {
-    const db = new CouchDB(appId);
-    if (isExternalTable(tableId)) {
-        let { datasourceId, tableName } = breakExternalTableId(tableId);
-        return exports.getExternalTable(appId, datasourceId, tableName);
-    }
-    else {
-        return db.get(tableId);
-    }
-});
-exports.checkForViewUpdates = (db, table, rename, deletedColumns) => __awaiter(void 0, void 0, void 0, function* () {
-    const views = yield getViews(db);
-    const tableViews = views.filter(view => view.meta.tableId === table._id);
-    // Check each table view to see if impacted by this table action
-    for (let view of tableViews) {
-        let needsUpdated = false;
-        // First check for renames, otherwise check for deletions
-        if (rename) {
-            // Update calculation field if required
-            if (view.meta.field === rename.old) {
-                view.meta.field = rename.updated;
-                needsUpdated = true;
-            }
-            // Update group by field if required
-            if (view.meta.groupBy === rename.old) {
-                view.meta.groupBy = rename.updated;
-                needsUpdated = true;
-            }
-            // Update filters if required
-            if (view.meta.filters) {
-                view.meta.filters.forEach(filter => {
-                    if (filter.key === rename.old) {
-                        filter.key = rename.updated;
-                        needsUpdated = true;
-                    }
-                });
-            }
-        }
-        else if (deletedColumns) {
-            deletedColumns.forEach(column => {
-                // Remove calculation statement if required
-                if (view.meta.field === column) {
-                    delete view.meta.field;
-                    delete view.meta.calculation;
-                    delete view.meta.groupBy;
-                    needsUpdated = true;
-                }
-                // Remove group by field if required
-                if (view.meta.groupBy === column) {
-                    delete view.meta.groupBy;
-                    needsUpdated = true;
-                }
-                // Remove filters referencing deleted field if required
-                if (view.meta.filters && view.meta.filters.length) {
-                    const initialLength = view.meta.filters.length;
-                    view.meta.filters = view.meta.filters.filter(filter => {
-                        return filter.key !== column;
-                    });
-                    if (initialLength !== view.meta.filters.length) {
-                        needsUpdated = true;
-                    }
-                }
-            });
-        }
-        // Update view if required
-        if (needsUpdated) {
-            const newViewTemplate = viewTemplate(view.meta);
-            yield saveView(db, null, view.name, newViewTemplate);
-            if (!newViewTemplate.meta.schema) {
-                newViewTemplate.meta.schema = table.schema;
-            }
-            table.views[view.name] = newViewTemplate.meta;
-        }
-    }
-});
 exports.TableSaveFunctions = TableSaveFunctions;

package/dist/api/controllers/templates.js

@@ -15,24 +15,9 @@ const DEFAULT_TEMPLATES_BUCKET = "prod-budi-templates.s3-eu-west-1.amazonaws.com
 exports.fetch = function (ctx) {
     return __awaiter(this, void 0, void 0, function* () {
         const { type = "app" } = ctx.query;
-        let response, error = false;
-        try {
-            response = yield fetch(`https://${DEFAULT_TEMPLATES_BUCKET}/manifest.json`);
-            if (response.status !== 200) {
-                error = true;
-            }
-        }
-        catch (err) {
-            error = true;
-        }
-        // if there is an error, simply return no templates
-        if (!error && response) {
-            const json = yield response.json();
-            ctx.body = Object.values(json.templates[type]);
-        }
-        else {
-            ctx.body = [];
-        }
+        const response = yield fetch(`https://${DEFAULT_TEMPLATES_BUCKET}/manifest.json`);
+        const json = yield response.json();
+        ctx.body = Object.values(json.templates[type]);
     });
 };
 // can't currently test this, have to ignore from coverage

package/dist/api/routes/dev.js

@@ -5,15 +5,11 @@ const env = require("../../environment");
 const authorized = require("../../middleware/authorized");
 const { BUILDER } = require("@budibase/auth/permissions");
 const router = Router();
-function redirectPath(path) {
-    router
-        .get(`/api/${path}/:devPath(.*)`, controller.buildRedirectGet(path))
-        .post(`/api/${path}/:devPath(.*)`, controller.buildRedirectPost(path))
-        .delete(`/api/${path}/:devPath(.*)`, controller.buildRedirectDelete(path));
-}
 if (env.isDev() || env.isTest()) {
-    redirectPath("global");
-    redirectPath("system");
+    router
+        .get("/api/global/:devPath(.*)", controller.redirectGet)
+        .post("/api/global/:devPath(.*)", controller.redirectPost)
+        .delete("/api/global/:devPath(.*)", controller.redirectDelete);
 }
 router
     .get("/api/dev/version", authorized(BUILDER), controller.getBudibaseVersion)

package/dist/api/routes/query.js

@@ -22,8 +22,7 @@ function generateQueryValidation() {
         })),
         queryVerb: Joi.string().allow().required(),
         extra: Joi.object().optional(),
-        schema: Joi.object({}).required().unknown(true),
-        transformer: Joi.string().optional(),
+        schema: Joi.object({}).required().unknown(true)
     }));
 }
 function generateQueryPreviewValidation() {
@@ -33,7 +32,6 @@ function generateQueryPreviewValidation() {
         queryVerb: Joi.string().allow().required(),
         extra: Joi.object().optional(),
         datasourceId: Joi.string().required(),
-        transformer: Joi.string().optional(),
         parameters: Joi.object({}).required().unknown(true)
     }));
 }

package/dist/automations/actions.js

@@ -13,6 +13,7 @@ const createRow = require("./steps/createRow");
 const updateRow = require("./steps/updateRow");
 const deleteRow = require("./steps/deleteRow");
 const executeScript = require("./steps/executeScript");
+const bash = require("./steps/bash");
 const executeQuery = require("./steps/executeQuery");
 const outgoingWebhook = require("./steps/outgoingWebhook");
 const serverLog = require("./steps/serverLog");
@@ -23,7 +24,6 @@ const integromat = require("./steps/integromat");
 let filter = require("./steps/filter");
 let delay = require("./steps/delay");
 let queryRow = require("./steps/queryRows");
-const env = require("../environment");
 const ACTION_IMPLS = {
     SEND_EMAIL_SMTP: sendSmtpEmail.run,
     CREATE_ROW: createRow.run,
@@ -31,6 +31,7 @@ const ACTION_IMPLS = {
     DELETE_ROW: deleteRow.run,
     OUTGOING_WEBHOOK: outgoingWebhook.run,
     EXECUTE_SCRIPT: executeScript.run,
+    EXECUTE_BASH: bash.run,
     EXECUTE_QUERY: executeQuery.run,
     SERVER_LOG: serverLog.run,
     DELAY: delay.run,
@@ -50,6 +51,7 @@ const ACTION_DEFINITIONS = {
     OUTGOING_WEBHOOK: outgoingWebhook.definition,
     EXECUTE_SCRIPT: executeScript.definition,
     EXECUTE_QUERY: executeQuery.definition,
+    EXECUTE_BASH: bash.definition,
     SERVER_LOG: serverLog.definition,
     DELAY: delay.definition,
     FILTER: filter.definition,
@@ -60,14 +62,6 @@ const ACTION_DEFINITIONS = {
     zapier: zapier.definition,
     integromat: integromat.definition,
 };
-// don't add the bash script/definitions unless in self host
-// the fact this isn't included in any definitions means it cannot be
-// ran at all
-if (env.SELF_HOSTED) {
-    const bash = require("./steps/bash");
-    ACTION_IMPLS["EXECUTE_BASH"] = bash.run;
-    ACTION_DEFINITIONS["EXECUTE_BASH"] = bash.definition;
-}
 /* istanbul ignore next */
 exports.getAction = function (actionName) {
     return __awaiter(this, void 0, void 0, function* () {

package/dist/automations/automationUtils.js

@@ -8,7 +8,9 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
         step((generator = generator.apply(thisArg, _arguments || [])).next());
     });
 };
-const { getTable } = require("../api/controllers/table/utils");
+const CouchDB = require("../db");
+const { isExternalTable, breakExternalTableId, } = require("../integrations/utils");
+const { getExternalTable } = require("../api/controllers/table/utils");
 /**
  * When values are input to the system generally they will be of type string as this is required for template strings.
  * This can generate some odd scenarios as the Schema of the automation requires a number but the builder might supply
@@ -25,7 +27,7 @@ const { getTable } = require("../api/controllers/table/utils");
  * @returns {object} The inputs object which has had all the various types supported by this function converted to their
  * primitive types.
  */
-exports.cleanInputValues = (inputs, schema) => {
+module.exports.cleanInputValues = (inputs, schema) => {
     if (schema == null) {
         return inputs;
     }
@@ -66,10 +68,29 @@ exports.cleanInputValues = (inputs, schema) => {
  * @param {object} row The input row structure which requires clean-up after having been through template statements.
  * @returns {Promise<Object>} The cleaned up rows object, will should now have all the required primitive types.
  */
-exports.cleanUpRow = (appId, tableId, row) => __awaiter(void 0, void 0, void 0, function* () {
-    let table = yield getTable(appId, tableId);
-    return exports.cleanInputValues(row, { properties: table.schema });
+module.exports.cleanUpRow = (appId, tableId, row) => __awaiter(void 0, void 0, void 0, function* () {
+    const db = new CouchDB(appId);
+    let table;
+    if (isExternalTable(tableId)) {
+        const { datasourceId, tableName } = breakExternalTableId(tableId);
+        table = yield getExternalTable(appId, datasourceId, tableName);
+    }
+    else {
+        table = yield db.get(tableId);
+    }
+    return module.exports.cleanInputValues(row, { properties: table.schema });
+});
+/**
+ * A utility function for the cleanUpRow, which can be used if only the row ID is known (not the table ID) to clean
+ * up a row after template statements have been replaced. This is specifically useful for the update row action.
+ *
+ * @param {string} appId The instance which the Table/Table is contained under.
+ * @param {string} rowId The ID of the row from which the tableId will be extracted, to get the Table/Table schema.
+ * @param {object} row The input row structure which requires clean-up after having been through template statements.
+ * @returns {Promise<Object>} The cleaned up rows object, which will now have all the required primitive types.
+ */
+module.exports.cleanUpRowById = (appId, rowId, row) => __awaiter(void 0, void 0, void 0, function* () {
+    const db = new CouchDB(appId);
+    const foundRow = yield db.get(rowId);
+    return module.exports.cleanUpRow(appId, foundRow.tableId, row);
 });
-exports.getError = err => {
-    return typeof err !== "string" ? err.toString() : err;
-};

package/dist/automations/steps/bash.js

@@ -10,7 +10,6 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
 };
 const { execSync } = require("child_process");
 const { processStringSync } = require("@budibase/string-templates");
-const automationUtils = require("../automationUtils");
 exports.definition = {
     name: "Bash Scripting",
     tagline: "Execute a bash command",
@@ -71,7 +70,7 @@ exports.run = function ({ inputs, context }) {
         catch (err) {
             return {
                 success: false,
-                response: automationUtils.getError(err),
+                response: err,
             };
         }
     });

package/dist/automations/steps/createRow.js

@@ -102,7 +102,7 @@ exports.run = function ({ inputs, appId, emitter }) {
         catch (err) {
             return {
                 success: false,
-                response: automationUtils.getError(err),
+                response: err,
             };
         }
     });

package/dist/automations/steps/deleteRow.js

@@ -12,7 +12,6 @@ const rowController = require("../../api/controllers/row");
 const env = require("../../environment");
 const usage = require("../../utilities/usageQuota");
 const { buildCtx } = require("./utils");
-const automationUtils = require("../automationUtils");
 exports.definition = {
     description: "Delete a row from your database",
     icon: "TableRowRemoveCenter",
@@ -94,7 +93,7 @@ exports.run = function ({ inputs, appId, emitter }) {
         catch (err) {
             return {
                 success: false,
-                response: automationUtils.getError(err),
+                response: err,
             };
         }
     });

package/dist/automations/steps/executeQuery.js

@@ -21,7 +21,6 @@ var __rest = (this && this.__rest) || function (s, e) {
 };
 const queryController = require("../../api/controllers/query");
 const { buildCtx } = require("./utils");
-const automationUtils = require("../automationUtils");
 exports.definition = {
     name: "External Data Connector",
     tagline: "Execute Data Connector",
@@ -93,7 +92,7 @@ exports.run = function ({ inputs, appId, emitter }) {
         catch (err) {
             return {
                 success: false,
-                response: automationUtils.getError(err),
+                response: err,
             };
         }
     });