fix(resources): sort unknown owners last and align the knowledge sort menu

An owner id that resolves to no workspace member renders an empty cell,
but its sort key was `''`, so those rows floated to the TOP of an
ascending Owner sort while every other valueless cell sorted last. They
now key `null` and follow the same nulls-last rule on all three pages.

The Knowledge sort menu also listed Owner after Last Updated while its
column order — and Tables' menu — put Owner before it.

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/components/folders/resource-sort.test.ts

@@ -94,6 +94,15 @@ describe('sortResources', () => {
     expect(names(sorted)).toEqual(['folder-z', 'item-a'])
   })
 
+  it('sorts a row whose cell renders empty last, not first', () => {
+    // An owner id that resolves to no workspace member renders an empty cell, so its key is
+    // `null` — passing `''` instead would float those rows to the top of an ascending sort.
+    const rows = [entry('unknown-owner', 'item', null), entry('ada', 'item', 'Ada')]
+
+    expect(names(sortResources([...rows], 'asc'))).toEqual(['ada', 'unknown-owner'])
+    expect(names(sortResources([...rows], 'desc'))).toEqual(['ada', 'unknown-owner'])
+  })
+
   it('breaks ties by name ascending regardless of direction', () => {
     const rows = [
       entry('charlie', 'item', 1),

apps/sim/app/workspace/[workspaceId]/files/files.tsx

@@ -545,7 +545,7 @@ export function Files() {
                 : sortColumn === 'updated'
                   ? new Date(folder.updatedAt).getTime()
                   : sortColumn === 'owner'
-                    ? (membersById.get(folder.userId)?.name ?? '')
+                    ? (membersById.get(folder.userId)?.name ?? null)
                     : folder.name,
       })
     }
@@ -565,7 +565,7 @@ export function Files() {
                 : sortColumn === 'updated'
                   ? new Date(file.updatedAt).getTime()
                   : sortColumn === 'owner'
-                    ? (membersById.get(file.uploadedBy)?.name ?? '')
+                    ? (membersById.get(file.uploadedBy)?.name ?? null)
                     : file.name,
       })
     }

apps/sim/app/workspace/[workspaceId]/knowledge/knowledge.tsx

@@ -490,7 +490,7 @@ export function Knowledge() {
               : sortColumn === 'updated'
                 ? new Date(folder.updatedAt).getTime()
                 : sortColumn === 'owner'
-                  ? (membersById.get(folder.userId)?.name ?? '')
+                  ? (membersById.get(folder.userId)?.name ?? null)
                   : folder.name,
       })
     }
@@ -512,7 +512,7 @@ export function Knowledge() {
                   : sortColumn === 'updated'
                     ? new Date(kb.updatedAt).getTime()
                     : sortColumn === 'owner'
-                      ? (membersById.get(kb.userId)?.name ?? '')
+                      ? (membersById.get(kb.userId)?.name ?? null)
                       : kb.name,
       })
     }
@@ -966,8 +966,8 @@ export function Knowledge() {
         { id: 'tokens', label: 'Tokens' },
         { id: 'connectors', label: 'Connectors' },
         { id: 'created', label: 'Created' },
-        { id: 'updated', label: 'Last Updated' },
         { id: 'owner', label: 'Owner' },
+        { id: 'updated', label: 'Last Updated' },
       ],
       active: activeSort,
       onSort: onSortColumn,

apps/sim/app/workspace/[workspaceId]/tables/tables.tsx

@@ -342,7 +342,7 @@ export function Tables() {
               : sortColumn === 'updated'
                 ? new Date(folder.updatedAt).getTime()
                 : sortColumn === 'owner'
-                  ? (membersById.get(folder.userId)?.name ?? '')
+                  ? (membersById.get(folder.userId)?.name ?? null)
                   : folder.name,
       })
     }
@@ -362,7 +362,7 @@ export function Tables() {
                 : sortColumn === 'updated'
                   ? new Date(table.updatedAt).getTime()
                   : sortColumn === 'owner'
-                    ? (membersById.get(table.createdBy)?.name ?? '')
+                    ? (membersById.get(table.createdBy)?.name ?? null)
                     : table.name,
       })
     }