fix: watch property in model does not support maps

Author: pregnantboy

packages/backend/src/graphql/admin/mutations/patch-gsi-rows.ts

@@ -91,7 +91,7 @@ const patchGsiRows: AdminMutationResolvers['patchGsiRows'] = async (
    * We update with a concurrency of 10 to prevent overloading the partition
    * and server
    */
-  const updateLimit = pLimit(100)
+  const updateLimit = pLimit(10)
 
   await Promise.all(
     /**

packages/backend/src/models/dynamodb/__tests__/table-row/functions.itest.ts

@@ -543,7 +543,7 @@ describe('dynamodb table row functions', () => {
       const updatedRow = await patchTableRow({
         tableId: dummyTable.id,
         rowId: row.rowId,
-        patchData: { [dummyColumnIds[0]]: '123' },
+        patchData: { set: { [dummyColumnIds[0]]: '123' } },
       })
       expect(updatedRow.updatedAt).toBeGreaterThan(row.updatedAt)
     })

packages/backend/src/models/dynamodb/table-row/model.ts

@@ -36,10 +36,19 @@ export const TableRow = new Entity(
       },
       updatedAt: {
         type: 'number',
-        watch: ['data'],
+        watch: '*',
         required: true,
         default: () => Date.now(),
-        set: () => Date.now(),
+        set: (_, changeObject) => {
+          /**
+           * We only update the updatedAt if the data is being updated
+           * This is to prevent updating the updatedAt when patching GSI
+           */
+          const keys = Object.keys(changeObject)
+          if (keys.some((key) => key === 'data' || key.startsWith('data.'))) {
+            return Date.now()
+          }
+        },
       },
       skString1: {
         type: 'string',