fix(cleanup): skip chat attachments whose key is not a string

`collectChatFiles` accepted any truthy `fileAttachments[].key` and then called
string methods on it, so one message row carrying a numeric or object key
threw and failed the whole chat purge, leaving every other key of the batch
behind. It now uses the same guard as the reference check in
`chat-cleanup.ts` (`isRecordLike` plus a non-empty string key) and skips the
entry.

Autopsy: the purge tests only fed well-formed attachments; message content is
untyped jsonb written by several clients, and the cast to string hid the gap
from type-check. The new case fails on the old code with
`key.startsWith is not a function`.

Author: waleedlatif1

apps/sim/lib/cleanup/chat-cleanup.test.ts

@@ -80,6 +80,17 @@ describe('chat purge storage', () => {
     expect(deleted).toEqual({ mothership: [own] })
   })
 
+  it('skips an attachment whose key is not a string instead of failing the purge', async () => {
+    const deleted = await purge([
+      {
+        role: 'user',
+        content: 'look',
+        fileAttachments: [{ ...attachment('x'), key: 42 }, attachment('copilot/1234/kept.png')],
+      },
+    ])
+    expect(deleted).toEqual({ copilot: ['copilot/1234/kept.png'] })
+  })
+
   it('deletes the inline images an assistant message published', async () => {
     const deleted = await purge([
       {

apps/sim/lib/cleanup/chat-files.ts

@@ -150,12 +150,8 @@ export async function collectChatFiles(
       const attachments = (msg as Record<string, unknown>).fileAttachments
       if (!Array.isArray(attachments)) continue
       for (const attachment of attachments) {
-        if (
-          attachment &&
-          typeof attachment === 'object' &&
-          (attachment as Record<string, unknown>).key
-        ) {
-          const key = (attachment as Record<string, unknown>).key as string
+        if (isRecordLike(attachment) && typeof attachment.key === 'string' && attachment.key) {
+          const key = attachment.key
           if (seen.has(key)) continue
           const organizationId = organizationAttachmentOwner(key)
           if (organizationId) {