Linting.

packages/builder/src/builderStore/api.js

@@ -2,21 +2,23 @@ import { store } from "./index"
 import { get as svelteGet } from "svelte/store"
 import { removeCookie, Cookies } from "./cookies"
 
-const apiCall =
+const apiCall = method => async (
-  method =>
+  url,
-  async (url, body, headers = { "Content-Type": "application/json" }) => {
+  body,
-    headers["x-budibase-app-id"] = svelteGet(store).appId
+  headers = { "Content-Type": "application/json" }
-    const json = headers["Content-Type"] === "application/json"
+) => {
-    const resp = await fetch(url, {
+  headers["x-budibase-app-id"] = svelteGet(store).appId
-      method: method,
+  const json = headers["Content-Type"] === "application/json"
-      body: json ? JSON.stringify(body) : body,
+  const resp = await fetch(url, {
-      headers,
+    method: method,
-    })
+    body: json ? JSON.stringify(body) : body,
-    if (resp.status === 403) {
+    headers,
-      removeCookie(Cookies.Auth)
+  })
-    }
+  if (resp.status === 403) {
-    return resp
+    removeCookie(Cookies.Auth)
   }
+  return resp
+}
 
 export const post = apiCall("POST")
 export const get = apiCall("GET")

packages/builder/src/builderStore/store/automation/index.js

@@ -100,10 +100,9 @@ const automationActions = store => ({
   },
   deleteAutomationBlock: block => {
     store.update(state => {
-      const idx =
+      const idx = state.selectedAutomation.automation.definition.steps.findIndex(
-        state.selectedAutomation.automation.definition.steps.findIndex(
+        x => x.id === block.id
-          x => x.id === block.id
+      )
-        )
       state.selectedAutomation.deleteBlock(block.id)
 
       // Select next closest step

packages/builder/src/components/common/bindings/BindingPanel.svelte

@@ -59,7 +59,9 @@
         <section>
           <Heading size="XS">Columns</Heading>
           <ul>
-            {#each context.filter( context => context.readableBinding.match(searchRgx) ) as { readableBinding }}
+            {#each context.filter(context =>
+              context.readableBinding.match(searchRgx)
+            ) as { readableBinding }}
               <li
                 on:click={() => {
                   value = addToText(value, getCaretPosition(), readableBinding)
@@ -75,7 +77,9 @@
         <section>
           <Heading size="XS">Components</Heading>
           <ul>
-            {#each instance.filter( instance => instance.readableBinding.match(searchRgx) ) as { readableBinding }}
+            {#each instance.filter(instance =>
+              instance.readableBinding.match(searchRgx)
+            ) as { readableBinding }}
               <li on:click={() => addToText(readableBinding)}>
                 {readableBinding}
               </li>

packages/builder/src/components/common/bindings/ServerBindingPanel.svelte

@@ -49,7 +49,9 @@
       <div class="section">
         {#each categories as [categoryName, bindings]}
           <Heading size="XS">{categoryName}</Heading>
-          {#each bindings.filter( binding => binding.label.match(searchRgx) ) as binding}
+          {#each bindings.filter(binding =>
+            binding.label.match(searchRgx)
+          ) as binding}
             <div
               class="binding"
               on:click={() => {

packages/builder/src/components/design/PropertiesPanel/PropertyControls/DataSourceSelect.svelte

@@ -103,9 +103,8 @@
   }
 
   function fetchQueryDefinition(query) {
-    const source = $datasources.list.find(
+    const source = $datasources.list.find(ds => ds._id === query.datasourceId)
-      ds => ds._id === query.datasourceId
+      .source
-    ).source
     return $integrations[source].query[query.queryVerb]
   }
 </script>

packages/builder/src/components/design/PropertiesPanel/PropertyControls/EventsEditor/actions/ExecuteQuery.svelte

@@ -18,9 +18,8 @@
   )
 
   function fetchQueryDefinition(query) {
-    const source = $datasources.list.find(
+    const source = $datasources.list.find(ds => ds._id === query.datasourceId)
-      ds => ds._id === query.datasourceId
+      .source
-    ).source
     return $integrations[source].query[query.queryVerb]
   }
 </script>

packages/builder/src/stores/backend/tests/fixtures/queries.js

@@ -9,7 +9,8 @@ export const SOME_QUERY = {
   queryVerb: "read",
   schema: {},
   name: "Speakers",
-  _id: "query_datasource_04b003a7b4a8428eadd3bb2f7eae0255_bcb8ffc6fcbc484e8d63121fc0bf986f",
+  _id:
+    "query_datasource_04b003a7b4a8428eadd3bb2f7eae0255_bcb8ffc6fcbc484e8d63121fc0bf986f",
   _rev: "2-941f8699eb0adf995f8bd59c99203b26",
   readable: true,
 }
@@ -74,7 +75,8 @@ export const SAVE_QUERY_RESPONSE = {
     },
   },
   name: "Speakers",
-  _id: "query_datasource_04b003a7b4a8428eadd3bb2f7eae0255_bcb8ffc6fcbc484e8d63121fc0bf986f",
+  _id:
+    "query_datasource_04b003a7b4a8428eadd3bb2f7eae0255_bcb8ffc6fcbc484e8d63121fc0bf986f",
   _rev: "3-5a64adef494b1e9c793dc91b51ce73c6",
   readable: true,
 }

packages/server/src/api/routes/row.js

@@ -40,10 +40,12 @@ router
   )
   // DEPRECATED - this is an old API, but for backwards compat it needs to be
   // supported still
-  .post("/api/search/:tableId/rows",
+  .post(
+    "/api/search/:tableId/rows",
     paramResource("tableId"),
     authorized(PermissionTypes.TABLE, PermissionLevels.READ),
-    rowController.search)
+    rowController.search
+  )
   .post(
     "/api/:tableId/rows",
     paramResource("tableId"),

packages/server/src/integrations/elasticsearch.js

@@ -2,7 +2,8 @@ const { Client } = require("@elastic/elasticsearch")
 const { QUERY_TYPES, FIELD_TYPES } = require("./Integration")
 
 const SCHEMA = {
-  docs: "https://www.elastic.co/guide/en/elasticsearch/client/javascript-api/current/index.html",
+  docs:
+    "https://www.elastic.co/guide/en/elasticsearch/client/javascript-api/current/index.html",
   description:
     "Elasticsearch is a search engine based on the Lucene library. It provides a distributed, multitenant-capable full-text search engine with an HTTP web interface and schema-free JSON documents.",
   friendlyName: "ElasticSearch",

packages/server/src/middleware/authorized.js

@@ -14,52 +14,50 @@ const WEBHOOK_ENDPOINTS = new RegExp(
   ["webhooks/trigger", "webhooks/schema"].join("|")
 )
 
-module.exports =
+module.exports = (permType, permLevel = null) => async (ctx, next) => {
-  (permType, permLevel = null) =>
+  // webhooks don't need authentication, each webhook unique
-  async (ctx, next) => {
+  if (WEBHOOK_ENDPOINTS.test(ctx.request.url)) {
-    // webhooks don't need authentication, each webhook unique
-    if (WEBHOOK_ENDPOINTS.test(ctx.request.url)) {
-      return next()
-    }
-
-    if (!ctx.user) {
-      return ctx.throw(403, "No user info found")
-    }
-
-    // check general builder stuff, this middleware is a good way
-    // to find API endpoints which are builder focused
-    await builderMiddleware(ctx, permType)
-
-    const isAuthed = ctx.isAuthenticated
-    const { basePermissions, permissions } = await getUserPermissions(
-      ctx.appId,
-      ctx.roleId
-    )
-
-    // builders for now have permission to do anything
-    // TODO: in future should consider separating permissions with an require("@budibase/auth").isClient check
-    let isBuilder = ctx.user && ctx.user.builder && ctx.user.builder.global
-    const isBuilderApi = permType === PermissionTypes.BUILDER
-    if (isBuilder) {
-      return next()
-    } else if (isBuilderApi && !isBuilder) {
-      return ctx.throw(403, "Not Authorized")
-    }
-
-    if (
-      hasResource(ctx) &&
-      doesHaveResourcePermission(permissions, permLevel, ctx)
-    ) {
-      return next()
-    }
-
-    if (!isAuthed) {
-      ctx.throw(403, "Session not authenticated")
-    }
-
-    if (!doesHaveBasePermission(permType, permLevel, basePermissions)) {
-      ctx.throw(403, "User does not have permission")
-    }
-
     return next()
   }
+
+  if (!ctx.user) {
+    return ctx.throw(403, "No user info found")
+  }
+
+  // check general builder stuff, this middleware is a good way
+  // to find API endpoints which are builder focused
+  await builderMiddleware(ctx, permType)
+
+  const isAuthed = ctx.isAuthenticated
+  const { basePermissions, permissions } = await getUserPermissions(
+    ctx.appId,
+    ctx.roleId
+  )
+
+  // builders for now have permission to do anything
+  // TODO: in future should consider separating permissions with an require("@budibase/auth").isClient check
+  let isBuilder = ctx.user && ctx.user.builder && ctx.user.builder.global
+  const isBuilderApi = permType === PermissionTypes.BUILDER
+  if (isBuilder) {
+    return next()
+  } else if (isBuilderApi && !isBuilder) {
+    return ctx.throw(403, "Not Authorized")
+  }
+
+  if (
+    hasResource(ctx) &&
+    doesHaveResourcePermission(permissions, permLevel, ctx)
+  ) {
+    return next()
+  }
+
+  if (!isAuthed) {
+    ctx.throw(403, "Session not authenticated")
+  }
+
+  if (!doesHaveBasePermission(permType, permLevel, basePermissions)) {
+    ctx.throw(403, "User does not have permission")
+  }
+
+  return next()
+}

packages/server/src/middleware/currentapp.js

@@ -1,5 +1,9 @@
-const { getAppId, setCookie, getCookie, clearCookie } =
+const {
-  require("@budibase/auth").utils
+  getAppId,
+  setCookie,
+  getCookie,
+  clearCookie,
+} = require("@budibase/auth").utils
 const { Cookies } = require("@budibase/auth").constants
 const { getRole } = require("@budibase/auth/roles")
 const { getGlobalSelf } = require("../utilities/workerRequests")

packages/standard-components/src/forms/validation.js

@@ -90,17 +90,15 @@ const numericalConstraint = (constraint, error) => value => {
   return null
 }
 
-const inclusionConstraint =
+const inclusionConstraint = (options = []) => value => {
-  (options = []) =>
+  if (value == null || value === "") {
-  value => {
-    if (value == null || value === "") {
-      return null
-    }
-    if (!options.includes(value)) {
-      return "Invalid value"
-    }
     return null
   }
+  if (!options.includes(value)) {
+    return "Invalid value"
+  }
+  return null
+}
 
 const dateConstraint = (dateString, isEarliest) => {
   const dateLimit = Date.parse(dateString)

packages/worker/src/api/controllers/admin/email.js

@@ -5,8 +5,15 @@ const authPkg = require("@budibase/auth")
 const GLOBAL_DB = authPkg.StaticDatabases.GLOBAL.name
 
 exports.sendEmail = async ctx => {
-  const { groupId, email, userId, purpose, contents, from, subject } =
+  const {
-    ctx.request.body
+    groupId,
+    email,
+    userId,
+    purpose,
+    contents,
+    from,
+    subject,
+  } = ctx.request.body
   let user
   if (userId) {
     const db = new CouchDB(GLOBAL_DB)

packages/worker/src/api/controllers/admin/groups.js

@@ -1,6 +1,9 @@
 const CouchDB = require("../../../db")
-const { getGroupParams, generateGroupID, StaticDatabases } =
+const {
-  require("@budibase/auth").db
+  getGroupParams,
+  generateGroupID,
+  StaticDatabases,
+} = require("@budibase/auth").db
 
 const GLOBAL_DB = StaticDatabases.GLOBAL.name
 

packages/worker/src/api/controllers/admin/users.js

@@ -1,6 +1,9 @@
 const CouchDB = require("../../../db")
-const { generateGlobalUserID, getGlobalUserParams, StaticDatabases } =
+const {
-  require("@budibase/auth").db
+  generateGlobalUserID,
+  getGlobalUserParams,
+  StaticDatabases,
+} = require("@budibase/auth").db
 const { hash, getGlobalUserByEmail } = require("@budibase/auth").utils
 const { UserStatus, EmailTemplatePurpose } = require("../../../constants")
 const { checkInviteCode } = require("../../../utilities/redis")