fix: support tiles v2 nullable cells

Author: pregnantboy

packages/backend/src/apps/tiles/actions/find-multiple-rows/get-data-out-metadata.ts

@@ -1,5 +1,7 @@
 import { IDataOutMetadata, IExecutionStep } from '@plumber/types'
 
+import logger from '@/helpers/logger'
+
 import { dataOutSchema } from './schema'
 
 async function getDataOutMetadata(
@@ -10,12 +12,20 @@ async function getDataOutMetadata(
   if (!rawDataOut) {
     return null
   }
-  const dataOut = dataOutSchema.parse(rawDataOut)
+  const dataOut = dataOutSchema.safeParse(rawDataOut)
+  if (!dataOut.success) {
+    logger.error({
+      message: 'Invalid data out schema for tiles find multiple rows action',
+      executionId: executionStep.executionId,
+      action: 'find-multiple-rows',
+    })
+    return null
+  }
 
   const metadata: IDataOutMetadata = {
     data: {
       label: 'Row(s) found',
-      displayedValue: `Preview ${dataOut.rowsFound} row(s)`,
+      displayedValue: `Preview ${dataOut.data.rowsFound} row(s)`,
       type: 'table',
       order: 1,
     },

packages/backend/src/apps/tiles/actions/find-multiple-rows/schema.ts

@@ -2,7 +2,9 @@ import { z } from 'zod'
 
 const tableRowOutputSchema = z.object({
   rowId: z.string(),
-  data: z.record(z.string(), z.string().or(z.number())),
+  // for tiles v1, empty cells is either an empty string or wont even have their key returned
+  // for tiles v2, empty cells return null
+  data: z.record(z.string(), z.union([z.string(), z.number(), z.null()])),
 })
 
 export const dataOutSchema = z.object({

packages/frontend/src/components/VariablesList/schema.ts

@@ -3,7 +3,9 @@ import { z } from 'zod'
 const RowDataSchema = z.object({
   rows: z.array(
     z.object({
-      data: z.record(z.string(), z.union([z.string(), z.number()])),
+      // for tiles v1, empty cells is either an empty string or wont even have their key returned
+      // for tiles v2, empty cells return null
+      data: z.record(z.string(), z.union([z.string(), z.number(), z.null()])),
       rowId: z.string().optional(), // only Tiles will have this
     }),
   ),

packages/frontend/src/components/VariablesList/utils.tsx

@@ -9,7 +9,7 @@ export interface RawColumn {
 }
 
 export interface RawRow {
-  data: Record<string, string | number>
+  data: Record<string, string | number | null>
   rowId?: string // only Tiles will have this
 }
 

packages/frontend/src/helpers/variables.ts

@@ -163,7 +163,7 @@ const process = (
      */
     const { columns, rows } = data
     const columnVariables = columns.map((column: RawColumn) => {
-      const rowValues: (string | number)[] = []
+      const rowValues: (string | number | null)[] = []
       rows.forEach((row: RawRow) => {
         /**
          * NOTE: do not push empty values as we do not want to cause any errors