perf(drizzle): further optimize postgres row updates (#13184)

This is a follow-up to #13060.

There are a bunch of other db adapter methods that use `upsertRow` for
updates: `updateGlobal`, `updateGlobalVersion`, `updateJobs`,
`updateMany`, `updateVersion`.

The previous PR had the logic for using the optimized row updating logic
inside the `updateOne` adapter. This PR moves that logic to the original
`upsertRow` function. Benefits:
- all the other db methods will benefit from this massive optimization
as well. This will be especially relevant for optimizing postgres job
queue initial updates - we should be able to close
#11865 after another follow-up
PR
- easier to read db adapter methods due to less code.

---
- To see the specific tasks where the Asana app for GitHub is being
used, see below:
  - https://app.asana.com/0/0/1210803039809810

Author: AlessioGr

packages/drizzle/src/updateOne.ts

@@ -1,67 +1,15 @@
 import type { LibSQLDatabase } from 'drizzle-orm/libsql'
-import type { FlattenedField, UpdateOne } from 'payload'
+import type { UpdateOne } from 'payload'
 
-import { eq } from 'drizzle-orm'
 import toSnakeCase from 'to-snake-case'
 
 import type { DrizzleAdapter } from './types.js'
 
-import { buildFindManyArgs } from './find/buildFindManyArgs.js'
 import { buildQuery } from './queries/buildQuery.js'
 import { selectDistinct } from './queries/selectDistinct.js'
-import { transform } from './transform/read/index.js'
-import { transformForWrite } from './transform/write/index.js'
 import { upsertRow } from './upsertRow/index.js'
 import { getTransaction } from './utilities/getTransaction.js'
 
-/**
- * Checks whether we should use the upsertRow function for the passed data and otherwise use a simple SQL SET call.
- * We need to use upsertRow only when the data has arrays, blocks, hasMany select/text/number, localized fields, complex relationships.
- */
-const shouldUseUpsertRow = ({
-  data,
-  fields,
-}: {
-  data: Record<string, unknown>
-  fields: FlattenedField[]
-}) => {
-  for (const key in data) {
-    const value = data[key]
-    const field = fields.find((each) => each.name === key)
-
-    if (!field) {
-      continue
-    }
-
-    if (
-      field.type === 'array' ||
-      field.type === 'blocks' ||
-      ((field.type === 'text' ||
-        field.type === 'relationship' ||
-        field.type === 'upload' ||
-        field.type === 'select' ||
-        field.type === 'number') &&
-        field.hasMany) ||
-      ((field.type === 'relationship' || field.type === 'upload') &&
-        Array.isArray(field.relationTo)) ||
-      field.localized
-    ) {
-      return true
-    }
-
-    if (
-      (field.type === 'group' || field.type === 'tab') &&
-      value &&
-      typeof value === 'object' &&
-      shouldUseUpsertRow({ data: value as Record<string, unknown>, fields: field.flattenedFields })
-    ) {
-      return true
-    }
-  }
-
-  return false
-}
-
 export const updateOne: UpdateOne = async function updateOne(
   this: DrizzleAdapter,
   {
@@ -126,72 +74,23 @@ export const updateOne: UpdateOne = async function updateOne(
     return null
   }
 
-  if (!idToUpdate || shouldUseUpsertRow({ data, fields: collection.flattenedFields })) {
-    const result = await upsertRow({
-      id: idToUpdate,
-      adapter: this,
-      data,
-      db,
-      fields: collection.flattenedFields,
-      ignoreResult: returning === false,
-      joinQuery,
-      operation: 'update',
-      req,
-      select,
-      tableName,
-    })
-
-    if (returning === false) {
-      return null
-    }
-
-    return result
-  }
-
-  const { row } = transformForWrite({
+  const result = await upsertRow({
+    id: idToUpdate,
     adapter: this,
     data,
-    enableAtomicWrites: true,
+    db,
     fields: collection.flattenedFields,
+    ignoreResult: returning === false,
+    joinQuery,
+    operation: 'update',
+    req,
+    select,
     tableName,
   })
 
-  const drizzle = db as LibSQLDatabase
-  await drizzle
-    .update(this.tables[tableName])
-    .set(row)
-    // TODO: we can skip fetching idToUpdate here with using the incoming where
-    .where(eq(this.tables[tableName].id, idToUpdate))
-
   if (returning === false) {
     return null
   }
 
-  const findManyArgs = buildFindManyArgs({
-    adapter: this,
-    depth: 0,
-    fields: collection.flattenedFields,
-    joinQuery: false,
-    select,
-    tableName,
-  })
-
-  findManyArgs.where = eq(this.tables[tableName].id, idToUpdate)
-
-  const doc = await db.query[tableName].findFirst(findManyArgs)
-
-  // //////////////////////////////////
-  // TRANSFORM DATA
-  // //////////////////////////////////
-
-  const result = transform({
-    adapter: this,
-    config: this.payload.config,
-    data: doc,
-    fields: collection.flattenedFields,
-    joinQuery: false,
-    tableName,
-  })
-
   return result
 }