From e0d3855945660bd50173fe23e405f48cb5621362 Mon Sep 17 00:00:00 2001
From: Adria Navarro <adria@budibase.com>
Date: Thu, 18 Jul 2024 16:28:15 +0200
Subject: [PATCH 01/14] Display row action trigger info

---
 .../AutomationBuilder/FlowChart/FlowItem.svelte        | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/packages/builder/src/components/automation/AutomationBuilder/FlowChart/FlowItem.svelte b/packages/builder/src/components/automation/AutomationBuilder/FlowChart/FlowItem.svelte
index 811909845a..503cc39187 100644
--- a/packages/builder/src/components/automation/AutomationBuilder/FlowChart/FlowItem.svelte
+++ b/packages/builder/src/components/automation/AutomationBuilder/FlowChart/FlowItem.svelte
@@ -14,7 +14,9 @@
     notifications,
     Label,
     AbsTooltip,
+    InlineAlert,
   } from "@budibase/bbui"
+  import { AutomationTriggerStepId } from "@budibase/types"
   import AutomationBlockSetup from "../../SetupPanel/AutomationBlockSetup.svelte"
   import CreateWebhookModal from "components/automation/Shared/CreateWebhookModal.svelte"
   import ActionModal from "./ActionModal.svelte"
@@ -49,6 +51,8 @@
   $: isAppAction && setPermissions(role)
   $: isAppAction && getPermissions(automationId)
 
+  $: isRowAction = block?.stepId === AutomationTriggerStepId.ROW_ACTION
+
   async function setPermissions(role) {
     if (!role || !automationId) {
       return
@@ -183,6 +187,12 @@
           {block}
           {webhookModal}
         />
+        {#if isRowAction && isTrigger}
+          <InlineAlert
+            header="Automation trigger"
+            message="This trigger is tied to the row action TODO on your TODO"
+          />
+        {/if}
         {#if lastStep}
           <Button on:click={() => testDataModal.show()} cta>
             Finish and test automation

From 1b2182a6904c4bf0c1504116782e2505a5162f31 Mon Sep 17 00:00:00 2001
From: Adria Navarro <adria@budibase.com>
Date: Fri, 19 Jul 2024 11:35:25 +0200
Subject: [PATCH 02/14] Run without required

---
 .../automation/AutomationBuilder/FlowChart/TestDataModal.svelte | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/builder/src/components/automation/AutomationBuilder/FlowChart/TestDataModal.svelte b/packages/builder/src/components/automation/AutomationBuilder/FlowChart/TestDataModal.svelte
index 2a553f8b48..f656a1ecb9 100644
--- a/packages/builder/src/components/automation/AutomationBuilder/FlowChart/TestDataModal.svelte
+++ b/packages/builder/src/components/automation/AutomationBuilder/FlowChart/TestDataModal.svelte
@@ -81,7 +81,7 @@
   // Check the schema to see if required fields have been entered
   $: isError =
     !isTriggerValid(trigger) ||
-    !trigger.schema.outputs.required?.every(
+    !(trigger.schema.outputs.required || []).every(
       required => $memoTestData?.[required] || required !== "row"
     )
 

From 6221b9320e21a44222979765e6e26102577c39d0 Mon Sep 17 00:00:00 2001
From: Adria Navarro <adria@budibase.com>
Date: Fri, 19 Jul 2024 12:24:45 +0200
Subject: [PATCH 03/14] Enrich

---
 .../builder/src/stores/builder/automations.js |  2 +-
 packages/frontend-core/src/api/automations.js |  8 +++++--
 .../server/src/api/controllers/automation.ts  |  9 +++++++-
 .../server/src/sdk/app/automations/crud.ts    | 22 +++++++++++++++++--
 4 files changed, 35 insertions(+), 6 deletions(-)

diff --git a/packages/builder/src/stores/builder/automations.js b/packages/builder/src/stores/builder/automations.js
index a2ee4b3ccc..b6af177442 100644
--- a/packages/builder/src/stores/builder/automations.js
+++ b/packages/builder/src/stores/builder/automations.js
@@ -59,7 +59,7 @@ const automationActions = store => ({
   },
   fetch: async () => {
     const responses = await Promise.all([
-      API.getAutomations(),
+      API.getAutomations({ enrich: true }),
       API.getAutomationDefinitions(),
     ])
     store.update(state => {
diff --git a/packages/frontend-core/src/api/automations.js b/packages/frontend-core/src/api/automations.js
index 37a834cf04..863f98f5b9 100644
--- a/packages/frontend-core/src/api/automations.js
+++ b/packages/frontend-core/src/api/automations.js
@@ -26,9 +26,13 @@ export const buildAutomationEndpoints = API => ({
   /**
    * Gets a list of all automations.
    */
-  getAutomations: async () => {
+  getAutomations: async ({ enrich }) => {
+    let url = "/api/automations?"
+    if (enrich) {
+      url += "enrich=true"
+    }
     return await API.get({
-      url: "/api/automations",
+      url,
     })
   },
 
diff --git a/packages/server/src/api/controllers/automation.ts b/packages/server/src/api/controllers/automation.ts
index 7eca17e0e8..b9b1793425 100644
--- a/packages/server/src/api/controllers/automation.ts
+++ b/packages/server/src/api/controllers/automation.ts
@@ -74,7 +74,14 @@ export async function update(ctx: UserCtx) {
 }
 
 export async function fetch(ctx: UserCtx) {
-  ctx.body = await sdk.automations.fetch()
+  const enrich = ctx.request.query["enrich"] === "true"
+
+  const automations = await sdk.automations.fetch()
+  if (enrich) {
+    ctx.body = await sdk.automations.enrichDisplayData(automations)
+  } else {
+    ctx.body = automations
+  }
 }
 
 export async function find(ctx: UserCtx) {
diff --git a/packages/server/src/sdk/app/automations/crud.ts b/packages/server/src/sdk/app/automations/crud.ts
index 59b6b5dc2a..95b9d5895f 100644
--- a/packages/server/src/sdk/app/automations/crud.ts
+++ b/packages/server/src/sdk/app/automations/crud.ts
@@ -1,4 +1,9 @@
-import { Automation, Webhook, WebhookActionType } from "@budibase/types"
+import {
+  Automation,
+  AutomationTriggerStepId,
+  Webhook,
+  WebhookActionType,
+} from "@budibase/types"
 import { generateAutomationID, getAutomationParams } from "../../../db/utils"
 import { deleteEntityMetadata } from "../../../utilities"
 import { MetadataTypes } from "../../../constants"
@@ -81,7 +86,7 @@ export async function fetch() {
       include_docs: true,
     })
   )
-  return response.rows.map(row => row.doc)
+  return response.rows.map(row => row.doc).filter(doc => !!doc)
 }
 
 export async function get(automationId: string) {
@@ -254,6 +259,7 @@ async function checkForWebhooks({ oldAuto, newAuto }: any) {
   }
   return newAuto
 }
+
 function guardInvalidUpdatesAndThrow(
   automation: Automation,
   oldAutomation: Automation
@@ -281,3 +287,15 @@ function guardInvalidUpdatesAndThrow(
     })
   }
 }
+
+export async function enrichDisplayData(automations: Automation[]) {
+  const rowActionAutomations = automations.filter(
+    ({ definition }) =>
+      definition.trigger.stepId === AutomationTriggerStepId.ROW_ACTION
+  )
+
+  for (const automation of rowActionAutomations) {
+    automation.name = `TODO: ${automation.name}`
+  }
+  return automations
+}

From b7a969280f442729493b0d21662a389ec118d575 Mon Sep 17 00:00:00 2001
From: Adria Navarro <adria@budibase.com>
Date: Fri, 19 Jul 2024 13:35:50 +0200
Subject: [PATCH 04/14] Display data

---
 .../FlowChart/FlowItem.svelte                 | 10 +++----
 .../AutomationPanel/AutomationPanel.svelte    |  6 ++++
 .../builder/src/stores/builder/automations.js | 20 ++++++++++---
 packages/builder/src/stores/builder/index.js  |  2 ++
 .../server/src/api/controllers/automation.ts  |  8 ++---
 .../server/src/sdk/app/automations/crud.ts    | 29 ++++++++++++++-----
 packages/types/src/api/web/automation.ts      | 14 +++++++++
 7 files changed, 67 insertions(+), 22 deletions(-)

diff --git a/packages/builder/src/components/automation/AutomationBuilder/FlowChart/FlowItem.svelte b/packages/builder/src/components/automation/AutomationBuilder/FlowChart/FlowItem.svelte
index 503cc39187..99cf382585 100644
--- a/packages/builder/src/components/automation/AutomationBuilder/FlowChart/FlowItem.svelte
+++ b/packages/builder/src/components/automation/AutomationBuilder/FlowChart/FlowItem.svelte
@@ -3,6 +3,7 @@
     automationStore,
     selectedAutomation,
     permissions,
+    selectedAutomationDisplayData,
   } from "stores/builder"
   import {
     Icon,
@@ -16,7 +17,6 @@
     AbsTooltip,
     InlineAlert,
   } from "@budibase/bbui"
-  import { AutomationTriggerStepId } from "@budibase/types"
   import AutomationBlockSetup from "../../SetupPanel/AutomationBlockSetup.svelte"
   import CreateWebhookModal from "components/automation/Shared/CreateWebhookModal.svelte"
   import ActionModal from "./ActionModal.svelte"
@@ -51,8 +51,6 @@
   $: isAppAction && setPermissions(role)
   $: isAppAction && getPermissions(automationId)
 
-  $: isRowAction = block?.stepId === AutomationTriggerStepId.ROW_ACTION
-
   async function setPermissions(role) {
     if (!role || !automationId) {
       return
@@ -187,10 +185,10 @@
           {block}
           {webhookModal}
         />
-        {#if isRowAction && isTrigger}
+        {#if isTrigger && $selectedAutomationDisplayData?.triggerInfo}
           <InlineAlert
-            header="Automation trigger"
-            message="This trigger is tied to the row action TODO on your TODO"
+            header={$selectedAutomationDisplayData.triggerInfo.title}
+            message={$selectedAutomationDisplayData.triggerInfo.description}
           />
         {/if}
         {#if lastStep}
diff --git a/packages/builder/src/components/automation/AutomationPanel/AutomationPanel.svelte b/packages/builder/src/components/automation/AutomationPanel/AutomationPanel.svelte
index c3ef3157bd..718a0dbcb5 100644
--- a/packages/builder/src/components/automation/AutomationPanel/AutomationPanel.svelte
+++ b/packages/builder/src/components/automation/AutomationPanel/AutomationPanel.svelte
@@ -17,6 +17,12 @@
         automation.name.toLowerCase().includes(searchString.toLowerCase())
       )
     })
+    .map(automation => ({
+      ...automation,
+      name:
+        $automationStore.automationDisplayData[automation._id].displayName ||
+        automation.name,
+    }))
     .sort((a, b) => {
       const lowerA = a.name.toLowerCase()
       const lowerB = b.name.toLowerCase()
diff --git a/packages/builder/src/stores/builder/automations.js b/packages/builder/src/stores/builder/automations.js
index b6af177442..c1cb963ac4 100644
--- a/packages/builder/src/stores/builder/automations.js
+++ b/packages/builder/src/stores/builder/automations.js
@@ -15,6 +15,7 @@ const initialAutomationState = {
     ACTION: [],
   },
   selectedAutomationId: null,
+  automationDisplayData: {},
 }
 
 // If this functions, remove the actions elements
@@ -58,18 +59,19 @@ const automationActions = store => ({
     return response
   },
   fetch: async () => {
-    const responses = await Promise.all([
+    const [automationResponse, definitions] = await Promise.all([
       API.getAutomations({ enrich: true }),
       API.getAutomationDefinitions(),
     ])
     store.update(state => {
-      state.automations = responses[0]
+      state.automations = automationResponse.automations
       state.automations.sort((a, b) => {
         return a.name < b.name ? -1 : 1
       })
+      state.automationDisplayData = automationResponse.builderData
       state.blockDefinitions = {
-        TRIGGER: responses[1].trigger,
-        ACTION: responses[1].action,
+        TRIGGER: definitions.trigger,
+        ACTION: definitions.action,
       }
       return state
     })
@@ -386,3 +388,13 @@ export const selectedAutomation = derived(automationStore, $automationStore => {
     x => x._id === $automationStore.selectedAutomationId
   )
 })
+
+export const selectedAutomationDisplayData = derived(
+  [automationStore, selectedAutomation],
+  ([$automationStore, $selectedAutomation]) => {
+    if (!$selectedAutomation._id) {
+      return null
+    }
+    return $automationStore.automationDisplayData[$selectedAutomation._id]
+  }
+)
diff --git a/packages/builder/src/stores/builder/index.js b/packages/builder/src/stores/builder/index.js
index c47782c0f5..aa0062dd7d 100644
--- a/packages/builder/src/stores/builder/index.js
+++ b/packages/builder/src/stores/builder/index.js
@@ -11,6 +11,7 @@ import {
   automationStore,
   selectedAutomation,
   automationHistoryStore,
+  selectedAutomationDisplayData,
 } from "./automations.js"
 import { userStore, userSelectedResourceMap, isOnlyUser } from "./users.js"
 import { deploymentStore } from "./deployments.js"
@@ -44,6 +45,7 @@ export {
   previewStore,
   automationStore,
   selectedAutomation,
+  selectedAutomationDisplayData,
   automationHistoryStore,
   sortedScreens,
   userStore,
diff --git a/packages/server/src/api/controllers/automation.ts b/packages/server/src/api/controllers/automation.ts
index b9b1793425..4495125e7f 100644
--- a/packages/server/src/api/controllers/automation.ts
+++ b/packages/server/src/api/controllers/automation.ts
@@ -11,6 +11,7 @@ import {
   AutomationResults,
   UserCtx,
   DeleteAutomationResponse,
+  FetchAutomationResponse,
 } from "@budibase/types"
 import { getActionDefinitions as actionDefs } from "../../automations/actions"
 import sdk from "../../sdk"
@@ -73,14 +74,13 @@ export async function update(ctx: UserCtx) {
   builderSocket?.emitAutomationUpdate(ctx, automation)
 }
 
-export async function fetch(ctx: UserCtx) {
+export async function fetch(ctx: UserCtx<void, FetchAutomationResponse>) {
   const enrich = ctx.request.query["enrich"] === "true"
 
   const automations = await sdk.automations.fetch()
+  ctx.body = { automations }
   if (enrich) {
-    ctx.body = await sdk.automations.enrichDisplayData(automations)
-  } else {
-    ctx.body = automations
+    ctx.body.builderData = await sdk.automations.getBuilderData(automations)
   }
 }
 
diff --git a/packages/server/src/sdk/app/automations/crud.ts b/packages/server/src/sdk/app/automations/crud.ts
index 95b9d5895f..8836677039 100644
--- a/packages/server/src/sdk/app/automations/crud.ts
+++ b/packages/server/src/sdk/app/automations/crud.ts
@@ -1,5 +1,6 @@
 import {
   Automation,
+  AutomationBuilderData,
   AutomationTriggerStepId,
   Webhook,
   WebhookActionType,
@@ -288,14 +289,26 @@ function guardInvalidUpdatesAndThrow(
   }
 }
 
-export async function enrichDisplayData(automations: Automation[]) {
-  const rowActionAutomations = automations.filter(
-    ({ definition }) =>
-      definition.trigger.stepId === AutomationTriggerStepId.ROW_ACTION
-  )
+export async function getBuilderData(
+  automations: Automation[]
+): Promise<Record<string, AutomationBuilderData>> {
+  const result: Record<string, AutomationBuilderData> = {}
+  for (const automation of automations) {
+    const isRowAction =
+      automation.definition.trigger.stepId ===
+      AutomationTriggerStepId.ROW_ACTION
+    if (!isRowAction) {
+      result[automation._id!] = { displayName: automation.name }
+      continue
+    }
 
-  for (const automation of rowActionAutomations) {
-    automation.name = `TODO: ${automation.name}`
+    result[automation._id!] = {
+      displayName: `TODO: ${automation.name}`,
+      triggerInfo: {
+        title: "Automation trigger",
+        description: "This trigger is tied to the row action TODO on your TODO",
+      },
+    }
   }
-  return automations
+  return result
 }
diff --git a/packages/types/src/api/web/automation.ts b/packages/types/src/api/web/automation.ts
index c1f3d01b2f..58faf55a41 100644
--- a/packages/types/src/api/web/automation.ts
+++ b/packages/types/src/api/web/automation.ts
@@ -1,3 +1,17 @@
 import { DocumentDestroyResponse } from "@budibase/nano"
+import { Automation } from "../../documents"
 
 export interface DeleteAutomationResponse extends DocumentDestroyResponse {}
+
+export interface AutomationBuilderData {
+  displayName: string
+  triggerInfo?: {
+    title: string
+    description: string
+  }
+}
+
+export interface FetchAutomationResponse {
+  automations: Automation[]
+  builderData?: Record<string, AutomationBuilderData> // The key will be the automationId
+}

From 3c9c988d8c4e7442a9a071c7bd40855b2311521e Mon Sep 17 00:00:00 2001
From: Adria Navarro <adria@budibase.com>
Date: Mon, 22 Jul 2024 10:51:18 +0200
Subject: [PATCH 05/14] Populate table name

---
 .../server/src/api/controllers/automation.ts  |  4 +-
 .../server/src/sdk/app/automations/utils.ts   | 43 ++++++++++++++++++-
 2 files changed, 45 insertions(+), 2 deletions(-)

diff --git a/packages/server/src/api/controllers/automation.ts b/packages/server/src/api/controllers/automation.ts
index 4495125e7f..bceebf39b7 100644
--- a/packages/server/src/api/controllers/automation.ts
+++ b/packages/server/src/api/controllers/automation.ts
@@ -80,7 +80,9 @@ export async function fetch(ctx: UserCtx<void, FetchAutomationResponse>) {
   const automations = await sdk.automations.fetch()
   ctx.body = { automations }
   if (enrich) {
-    ctx.body.builderData = await sdk.automations.getBuilderData(automations)
+    ctx.body.builderData = await sdk.automations.utils.getBuilderData(
+      automations
+    )
   }
 }
 
diff --git a/packages/server/src/sdk/app/automations/utils.ts b/packages/server/src/sdk/app/automations/utils.ts
index 79e70923f1..63a7d37bc1 100644
--- a/packages/server/src/sdk/app/automations/utils.ts
+++ b/packages/server/src/sdk/app/automations/utils.ts
@@ -1,7 +1,48 @@
-import { Automation, AutomationActionStepId } from "@budibase/types"
+import {
+  Automation,
+  AutomationActionStepId,
+  AutomationBuilderData,
+  AutomationTriggerStepId,
+} from "@budibase/types"
+import sdk from "src/sdk"
 
 export function checkForCollectStep(automation: Automation) {
   return automation.definition.steps.some(
     (step: any) => step.stepId === AutomationActionStepId.COLLECT
   )
 }
+
+export async function getBuilderData(
+  automations: Automation[]
+): Promise<Record<string, AutomationBuilderData>> {
+  const tableNameCache: Record<string, string> = {}
+  async function getTableName(tableId: string) {
+    if (!tableNameCache[tableId]) {
+      const table = await sdk.tables.getTable(tableId)
+      tableNameCache[tableId] = table.name
+    }
+
+    return tableNameCache[tableId]
+  }
+
+  const result: Record<string, AutomationBuilderData> = {}
+  for (const automation of automations) {
+    const { trigger } = automation.definition
+    const isRowAction = trigger.stepId === AutomationTriggerStepId.ROW_ACTION
+    if (!isRowAction) {
+      result[automation._id!] = { displayName: automation.name }
+      continue
+    }
+
+    const tableName = await getTableName(trigger.inputs.tableId)
+
+    result[automation._id!] = {
+      displayName: `${tableName}: ${automation.name}`,
+      triggerInfo: {
+        title: "Automation trigger",
+        description: `This trigger is tied to the row action TODO on your ${tableName} table`,
+      },
+    }
+  }
+  return result
+}

From a6f51caa46c1a028677d6af284bdb4062f649d62 Mon Sep 17 00:00:00 2001
From: Adria Navarro <adria@budibase.com>
Date: Mon, 22 Jul 2024 11:05:46 +0200
Subject: [PATCH 06/14] Display rowaction

---
 .../server/src/sdk/app/automations/crud.ts    |  2 +-
 .../server/src/sdk/app/automations/utils.ts   | 19 +++++++++++++++++--
 packages/server/src/sdk/app/rowActions.ts     | 12 ++++++++----
 3 files changed, 26 insertions(+), 7 deletions(-)

diff --git a/packages/server/src/sdk/app/automations/crud.ts b/packages/server/src/sdk/app/automations/crud.ts
index 8836677039..9765ecd2f2 100644
--- a/packages/server/src/sdk/app/automations/crud.ts
+++ b/packages/server/src/sdk/app/automations/crud.ts
@@ -17,7 +17,7 @@ import {
 import { definitions } from "../../../automations/triggerInfo"
 import automations from "."
 
-interface PersistedAutomation extends Automation {
+export interface PersistedAutomation extends Automation {
   _id: string
   _rev: string
 }
diff --git a/packages/server/src/sdk/app/automations/utils.ts b/packages/server/src/sdk/app/automations/utils.ts
index 63a7d37bc1..fbd0b5700c 100644
--- a/packages/server/src/sdk/app/automations/utils.ts
+++ b/packages/server/src/sdk/app/automations/utils.ts
@@ -3,6 +3,7 @@ import {
   AutomationActionStepId,
   AutomationBuilderData,
   AutomationTriggerStepId,
+  TableRowActions,
 } from "@budibase/types"
 import sdk from "src/sdk"
 
@@ -25,6 +26,16 @@ export async function getBuilderData(
     return tableNameCache[tableId]
   }
 
+  const rowActionNameCache: Record<string, TableRowActions> = {}
+  async function getRowActionName(tableId: string, rowActionId: string) {
+    if (!rowActionNameCache[tableId]) {
+      const rowActions = await sdk.rowActions.get(tableId)
+      rowActionNameCache[tableId] = rowActions
+    }
+
+    return rowActionNameCache[tableId].actions[rowActionId]?.name
+  }
+
   const result: Record<string, AutomationBuilderData> = {}
   for (const automation of automations) {
     const { trigger } = automation.definition
@@ -34,13 +45,17 @@ export async function getBuilderData(
       continue
     }
 
-    const tableName = await getTableName(trigger.inputs.tableId)
+    const { tableId, rowActionId } = trigger.inputs
+
+    const tableName = await getTableName(tableId)
+
+    const rowActionName = await getRowActionName(tableId, rowActionId)
 
     result[automation._id!] = {
       displayName: `${tableName}: ${automation.name}`,
       triggerInfo: {
         title: "Automation trigger",
-        description: `This trigger is tied to the row action TODO on your ${tableName} table`,
+        description: `This trigger is tied to the row action ${rowActionName} on your ${tableName} table`,
       },
     }
   }
diff --git a/packages/server/src/sdk/app/rowActions.ts b/packages/server/src/sdk/app/rowActions.ts
index df5625cce3..abfc0f1478 100644
--- a/packages/server/src/sdk/app/rowActions.ts
+++ b/packages/server/src/sdk/app/rowActions.ts
@@ -54,6 +54,10 @@ export async function create(tableId: string, rowAction: { name: string }) {
     throw new Error("Could not get the current appId")
   }
 
+  const newRowActionId = `${
+    VirtualDocumentType.ROW_ACTION
+  }${SEPARATOR}${utils.newid()}`
+
   const automation = await automations.create({
     name: `${tableName}: ${action.name}`,
     appId,
@@ -68,6 +72,7 @@ export async function create(tableId: string, rowAction: { name: string }) {
         stepId: AutomationTriggerStepId.ROW_ACTION,
         inputs: {
           tableId,
+          rowActionId: newRowActionId,
         },
         schema: {
           inputs: {
@@ -88,16 +93,15 @@ export async function create(tableId: string, rowAction: { name: string }) {
     },
   })
 
-  const newId = `${VirtualDocumentType.ROW_ACTION}${SEPARATOR}${utils.newid()}`
-  doc.actions[newId] = {
+  doc.actions[newRowActionId] = {
     name: action.name,
     automationId: automation._id!,
   }
   await db.put(doc)
 
   return {
-    id: newId,
-    ...doc.actions[newId],
+    id: newRowActionId,
+    ...doc.actions[newRowActionId],
   }
 }
 

From 3bb3376a826120f83d600a3bef05372a64645741 Mon Sep 17 00:00:00 2001
From: Adria Navarro <adria@budibase.com>
Date: Mon, 22 Jul 2024 11:11:42 +0200
Subject: [PATCH 07/14] Format on frontend

---
 .../AutomationBuilder/FlowChart/FlowItem.svelte       |  8 +++++---
 packages/server/src/sdk/app/automations/utils.ts      |  8 ++++++--
 packages/server/src/sdk/app/rowActions.ts             |  5 +----
 packages/types/src/api/web/automation.ts              | 11 +++++++++--
 4 files changed, 21 insertions(+), 11 deletions(-)

diff --git a/packages/builder/src/components/automation/AutomationBuilder/FlowChart/FlowItem.svelte b/packages/builder/src/components/automation/AutomationBuilder/FlowChart/FlowItem.svelte
index 99cf382585..c88317c79f 100644
--- a/packages/builder/src/components/automation/AutomationBuilder/FlowChart/FlowItem.svelte
+++ b/packages/builder/src/components/automation/AutomationBuilder/FlowChart/FlowItem.svelte
@@ -51,6 +51,8 @@
   $: isAppAction && setPermissions(role)
   $: isAppAction && getPermissions(automationId)
 
+  $: triggerInfo = $selectedAutomationDisplayData?.triggerInfo
+
   async function setPermissions(role) {
     if (!role || !automationId) {
       return
@@ -185,10 +187,10 @@
           {block}
           {webhookModal}
         />
-        {#if isTrigger && $selectedAutomationDisplayData?.triggerInfo}
+        {#if isTrigger && triggerInfo}
           <InlineAlert
-            header={$selectedAutomationDisplayData.triggerInfo.title}
-            message={$selectedAutomationDisplayData.triggerInfo.description}
+            header={triggerInfo.type}
+            message={`This trigger is tied to the row action ${triggerInfo.rowAction.name} on your ${triggerInfo.table.name} table`}
           />
         {/if}
         {#if lastStep}
diff --git a/packages/server/src/sdk/app/automations/utils.ts b/packages/server/src/sdk/app/automations/utils.ts
index fbd0b5700c..bd16c5aca6 100644
--- a/packages/server/src/sdk/app/automations/utils.ts
+++ b/packages/server/src/sdk/app/automations/utils.ts
@@ -54,8 +54,12 @@ export async function getBuilderData(
     result[automation._id!] = {
       displayName: `${tableName}: ${automation.name}`,
       triggerInfo: {
-        title: "Automation trigger",
-        description: `This trigger is tied to the row action ${rowActionName} on your ${tableName} table`,
+        type: "Automation trigger",
+        table: { id: tableId, name: tableName },
+        rowAction: {
+          id: rowActionId,
+          name: rowActionName,
+        },
       },
     }
   }
diff --git a/packages/server/src/sdk/app/rowActions.ts b/packages/server/src/sdk/app/rowActions.ts
index abfc0f1478..01716a55b4 100644
--- a/packages/server/src/sdk/app/rowActions.ts
+++ b/packages/server/src/sdk/app/rowActions.ts
@@ -11,7 +11,6 @@ import {
   VirtualDocumentType,
 } from "@budibase/types"
 import automations from "./automations"
-import tables from "./tables"
 
 function ensureUniqueAndThrow(
   doc: TableRowActions,
@@ -45,8 +44,6 @@ export async function create(tableId: string, rowAction: { name: string }) {
     doc = { _id: rowActionsId, actions: {} }
   }
 
-  const { name: tableName } = await tables.getTable(tableId)
-
   ensureUniqueAndThrow(doc, action.name)
 
   const appId = context.getAppId()
@@ -59,7 +56,7 @@ export async function create(tableId: string, rowAction: { name: string }) {
   }${SEPARATOR}${utils.newid()}`
 
   const automation = await automations.create({
-    name: `${tableName}: ${action.name}`,
+    name: action.name,
     appId,
     definition: {
       trigger: {
diff --git a/packages/types/src/api/web/automation.ts b/packages/types/src/api/web/automation.ts
index 58faf55a41..6c810d5e78 100644
--- a/packages/types/src/api/web/automation.ts
+++ b/packages/types/src/api/web/automation.ts
@@ -6,8 +6,15 @@ export interface DeleteAutomationResponse extends DocumentDestroyResponse {}
 export interface AutomationBuilderData {
   displayName: string
   triggerInfo?: {
-    title: string
-    description: string
+    type: string
+    table: {
+      id: string
+      name: string
+    }
+    rowAction: {
+      id: string
+      name: string
+    }
   }
 }
 

From 7574b27b4bf411c15bf51be9927f8f683f5c75b3 Mon Sep 17 00:00:00 2001
From: Adria Navarro <adria@budibase.com>
Date: Mon, 22 Jul 2024 12:14:02 +0200
Subject: [PATCH 08/14] Fix tests

---
 packages/server/src/api/routes/tests/utilities/TestFunctions.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/server/src/api/routes/tests/utilities/TestFunctions.ts b/packages/server/src/api/routes/tests/utilities/TestFunctions.ts
index 15a3ede39b..a9a8e7051b 100644
--- a/packages/server/src/api/routes/tests/utilities/TestFunctions.ts
+++ b/packages/server/src/api/routes/tests/utilities/TestFunctions.ts
@@ -54,7 +54,7 @@ export const clearAllApps = async (
 }
 
 export const clearAllAutomations = async (config: TestConfiguration) => {
-  const automations = await config.getAllAutomations()
+  const { automations } = await config.getAllAutomations()
   for (let auto of automations) {
     await context.doInAppContext(config.getAppId(), async () => {
       await config.deleteAutomation(auto)

From 3fa78587121d98cfd7c81f45f4abda19b9897ef8 Mon Sep 17 00:00:00 2001
From: Adria Navarro <adria@budibase.com>
Date: Mon, 22 Jul 2024 12:20:26 +0200
Subject: [PATCH 09/14] Fix references

---
 packages/server/src/sdk/app/automations/utils.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/server/src/sdk/app/automations/utils.ts b/packages/server/src/sdk/app/automations/utils.ts
index bd16c5aca6..5ce0292ffd 100644
--- a/packages/server/src/sdk/app/automations/utils.ts
+++ b/packages/server/src/sdk/app/automations/utils.ts
@@ -5,7 +5,7 @@ import {
   AutomationTriggerStepId,
   TableRowActions,
 } from "@budibase/types"
-import sdk from "src/sdk"
+import sdk from "../../../sdk"
 
 export function checkForCollectStep(automation: Automation) {
   return automation.definition.steps.some(

From d6e268d4755feba14e55012e0e41f697ef6a17b4 Mon Sep 17 00:00:00 2001
From: Adria Navarro <adria@budibase.com>
Date: Mon, 22 Jul 2024 13:16:42 +0200
Subject: [PATCH 10/14] Fix tests

---
 packages/server/src/api/controllers/automation.ts       | 3 ++-
 packages/server/src/api/routes/tests/automation.spec.ts | 4 +++-
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/packages/server/src/api/controllers/automation.ts b/packages/server/src/api/controllers/automation.ts
index bceebf39b7..937b052d9c 100644
--- a/packages/server/src/api/controllers/automation.ts
+++ b/packages/server/src/api/controllers/automation.ts
@@ -75,7 +75,8 @@ export async function update(ctx: UserCtx) {
 }
 
 export async function fetch(ctx: UserCtx<void, FetchAutomationResponse>) {
-  const enrich = ctx.request.query["enrich"] === "true"
+  const query = ctx.request.query || {}
+  const enrich = query["enrich"] === "true"
 
   const automations = await sdk.automations.fetch()
   ctx.body = { automations }
diff --git a/packages/server/src/api/routes/tests/automation.spec.ts b/packages/server/src/api/routes/tests/automation.spec.ts
index 98d38f4973..f0c3d6e1c5 100644
--- a/packages/server/src/api/routes/tests/automation.spec.ts
+++ b/packages/server/src/api/routes/tests/automation.spec.ts
@@ -398,7 +398,9 @@ describe("/automations", () => {
         .expect("Content-Type", /json/)
         .expect(200)
 
-      expect(res.body[0]).toEqual(expect.objectContaining(autoConfig))
+      expect(res.body.automations[0]).toEqual(
+        expect.objectContaining(autoConfig)
+      )
     })
 
     it("should apply authorization to endpoint", async () => {

From cec24fb40dc8f01aa8595f28910d979694e0e542 Mon Sep 17 00:00:00 2001
From: Adria Navarro <adria@budibase.com>
Date: Mon, 22 Jul 2024 14:34:58 +0200
Subject: [PATCH 11/14] Use URLSearchParams

---
 packages/frontend-core/src/api/automations.js | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/packages/frontend-core/src/api/automations.js b/packages/frontend-core/src/api/automations.js
index 863f98f5b9..ce5f7d3aba 100644
--- a/packages/frontend-core/src/api/automations.js
+++ b/packages/frontend-core/src/api/automations.js
@@ -27,12 +27,13 @@ export const buildAutomationEndpoints = API => ({
    * Gets a list of all automations.
    */
   getAutomations: async ({ enrich }) => {
-    let url = "/api/automations?"
+    const params = new URLSearchParams()
     if (enrich) {
-      url += "enrich=true"
+      params.set("enrich", true)
     }
+
     return await API.get({
-      url,
+      url: `/api/automations?${params.toString()}`,
     })
   },
 

From 286883da140c22eee2f9c605c983cc992ecac3b1 Mon Sep 17 00:00:00 2001
From: Adria Navarro <adria@budibase.com>
Date: Mon, 22 Jul 2024 14:39:26 +0200
Subject: [PATCH 12/14] Types

---
 packages/server/src/api/controllers/automation.ts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/packages/server/src/api/controllers/automation.ts b/packages/server/src/api/controllers/automation.ts
index 937b052d9c..6d1f3125fd 100644
--- a/packages/server/src/api/controllers/automation.ts
+++ b/packages/server/src/api/controllers/automation.ts
@@ -75,8 +75,8 @@ export async function update(ctx: UserCtx) {
 }
 
 export async function fetch(ctx: UserCtx<void, FetchAutomationResponse>) {
-  const query = ctx.request.query || {}
-  const enrich = query["enrich"] === "true"
+  const query: { enrich?: string } = ctx.request.query || {}
+  const enrich = query.enrich === "true"
 
   const automations = await sdk.automations.fetch()
   ctx.body = { automations }

From 9bd22bbcdf59d280f656264c3a6e7614f5d22945 Mon Sep 17 00:00:00 2001
From: Adria Navarro <adria@budibase.com>
Date: Mon, 22 Jul 2024 14:45:58 +0200
Subject: [PATCH 13/14] Fix after merge

---
 .../server/src/sdk/app/automations/crud.ts    | 32 +------------------
 1 file changed, 1 insertion(+), 31 deletions(-)

diff --git a/packages/server/src/sdk/app/automations/crud.ts b/packages/server/src/sdk/app/automations/crud.ts
index 9765ecd2f2..f3dd3e7a3c 100644
--- a/packages/server/src/sdk/app/automations/crud.ts
+++ b/packages/server/src/sdk/app/automations/crud.ts
@@ -1,10 +1,4 @@
-import {
-  Automation,
-  AutomationBuilderData,
-  AutomationTriggerStepId,
-  Webhook,
-  WebhookActionType,
-} from "@budibase/types"
+import { Automation, Webhook, WebhookActionType } from "@budibase/types"
 import { generateAutomationID, getAutomationParams } from "../../../db/utils"
 import { deleteEntityMetadata } from "../../../utilities"
 import { MetadataTypes } from "../../../constants"
@@ -288,27 +282,3 @@ function guardInvalidUpdatesAndThrow(
     })
   }
 }
-
-export async function getBuilderData(
-  automations: Automation[]
-): Promise<Record<string, AutomationBuilderData>> {
-  const result: Record<string, AutomationBuilderData> = {}
-  for (const automation of automations) {
-    const isRowAction =
-      automation.definition.trigger.stepId ===
-      AutomationTriggerStepId.ROW_ACTION
-    if (!isRowAction) {
-      result[automation._id!] = { displayName: automation.name }
-      continue
-    }
-
-    result[automation._id!] = {
-      displayName: `TODO: ${automation.name}`,
-      triggerInfo: {
-        title: "Automation trigger",
-        description: "This trigger is tied to the row action TODO on your TODO",
-      },
-    }
-  }
-  return result
-}

From 6b58549fd9f896a0b7e4f0083e54ac4c8cd1de35 Mon Sep 17 00:00:00 2001
From: Adria Navarro <adria@budibase.com>
Date: Mon, 22 Jul 2024 16:34:14 +0200
Subject: [PATCH 14/14] Fix issues with circular references and barrel files

---
 packages/server/src/sdk/app/automations/utils.ts | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/packages/server/src/sdk/app/automations/utils.ts b/packages/server/src/sdk/app/automations/utils.ts
index 5ce0292ffd..52ed935f3b 100644
--- a/packages/server/src/sdk/app/automations/utils.ts
+++ b/packages/server/src/sdk/app/automations/utils.ts
@@ -5,7 +5,6 @@ import {
   AutomationTriggerStepId,
   TableRowActions,
 } from "@budibase/types"
-import sdk from "../../../sdk"
 
 export function checkForCollectStep(automation: Automation) {
   return automation.definition.steps.some(
@@ -16,6 +15,8 @@ export function checkForCollectStep(automation: Automation) {
 export async function getBuilderData(
   automations: Automation[]
 ): Promise<Record<string, AutomationBuilderData>> {
+  const sdk = (await import("../../../sdk")).default
+
   const tableNameCache: Record<string, string> = {}
   async function getTableName(tableId: string) {
     if (!tableNameCache[tableId]) {