fix(drizzle): sort by join field on versioned collection with custom text id

When the list view is sorted by a `join` field on a collection that has both
`versions` enabled and a custom text id (e.g. nanoid), the query crashes with:

  operator does not exist: integer = character varying

`getTableColumnFromPath` builds the join condition using the versions table's
`id` column. On a versions table, `id` is the versions row PK (always integer),
not the parent document id. The joined `_rels` / target table column references
the parent document id, which is `varchar` for custom text ids, so the types
do not match.

Use the versions table's `parent` column (mapped from the `parent_id` db
column, which is the actual reference to the parent document id) instead of
`id` whenever the source table is a versions table. Fixes both branches of
`case 'join'` (single-segment `on` with `hasMany`, and the non-`hasMany` path).

Made-with: Cursor

Author: nathanbowang

packages/drizzle/src/queries/getTableColumnFromPath.ts

@@ -393,10 +393,14 @@ export const getTableColumnFromPath = ({
           }
 
           if (!existingTable) {
+            const rootIDColumn =
+              adapter.versionsSuffix && rootTableName.endsWith(adapter.versionsSuffix)
+                ? adapter.tables[rootTableName].parent
+                : adapter.tables[rootTableName].id
             addJoinTable({
               condition: and(
                 eq(
-                  adapter.tables[rootTableName].id,
+                  rootIDColumn,
                   aliasRelationshipTable[
                     `${(relationshipField.field as RelationshipField).relationTo as string}ID`
                   ],
@@ -490,18 +494,20 @@ export const getTableColumnFromPath = ({
               ? adapter.tableNameMap.get(`${newTableName}_${toSnakeCase(onSegments[0])}`)
               : undefined
 
+          const sourceIDColumn = aliasTable
+            ? aliasTable.id
+            : adapter.versionsSuffix && tableName.endsWith(adapter.versionsSuffix)
+              ? adapter.tables[tableName].parent
+              : adapter.tables[tableName].id
+
           if (arrayTableName) {
-            // join from main table to array table
             const { newAliasTable: arrayAliasTable } = getTableAlias({
               adapter,
               tableName: arrayTableName,
             })
 
             joins.push({
-              condition: eq(
-                arrayAliasTable[onSegments.slice(1).join('_')],
-                aliasTable ? aliasTable.id : adapter.tables[tableName].id,
-              ),
+              condition: eq(arrayAliasTable[onSegments.slice(1).join('_')], sourceIDColumn),
               queryPath: `${constraintPath}${field.name}._array`,
               table: arrayAliasTable,
             })
@@ -516,10 +522,7 @@ export const getTableColumnFromPath = ({
             })
           } else {
             joins.push({
-              condition: eq(
-                newAliasTable[field.on.replaceAll('.', '_')],
-                aliasTable ? aliasTable.id : adapter.tables[tableName].id,
-              ),
+              condition: eq(newAliasTable[field.on.replaceAll('.', '_')], sourceIDColumn),
               queryPath: `${constraintPath}${field.name}`,
               table: newAliasTable,
             })