fix(files): resolve CI audits and test types after restacking on staging

- Keep prepareProjectsForAccountDeletion returning the Project IDs to delete.
  The object return added an unused storageCleanupEventIds field; the
  durable cleanup outbox already processes those events, so restore the
  staging contract and its stronger integration assertions.
- Exempt lib/mothership/generated from the relative-import convention.
  contracts:sync copies those wire contracts verbatim and byte-checks them,
  so their sibling imports must stay relative; drop their baseline rows.
- Re-record the files page module-count baseline for the Project browser.
- Type the test fixtures that regressed, including a queue-recovery
  fixture that persisted the invalid completion status 'success'.

Author: mzxchandra

.claude/rules/sim-imports.md

@@ -10,6 +10,7 @@ paths:
 ## Absolute Imports
 
 **Always use absolute imports.** Never use relative imports, except that a barrel `index.ts` re-exports its own siblings relatively (`export { X } from './x'`). `check:source-conventions` ratchets this per file.
+The wire contracts in `lib/mothership/generated/` are also exempt: the worker's `contracts:sync` copies them verbatim and byte-checks the copies, so never hand-edit their imports.
 
 ```typescript
 // ✓ Good

.cursor/rules/sim-imports.mdc

@@ -10,6 +10,7 @@ globs: ["apps/sim/**/*.ts","apps/sim/**/*.tsx"]
 ## Absolute Imports
 
 **Always use absolute imports.** Never use relative imports, except that a barrel `index.ts` re-exports its own siblings relatively (`export { X } from './x'`). `check:source-conventions` ratchets this per file.
+The wire contracts in `lib/mothership/generated/` are also exempt: the worker's `contracts:sync` copies them verbatim and byte-checks the copies, so never hand-edit their imports.
 
 ```typescript
 // ✓ Good

apps/sim/app/workspace/[workspaceId]/files/components/file-viewer/rich-markdown-editor/editor-lifecycle.test.tsx

@@ -832,7 +832,7 @@ it('inserts an uploaded inline image using its Project content address', async (
   fileSource.owner = { entityType: 'project', entityId: 'project-a' }
   uploadProjectFile.mockResolvedValue({ file: { id: 'image-a' } })
   await render('Project document')
-  await act(async () => getEditor().storage.slashCommand.insertImage(1))
+  await act(async () => getEditor().storage.slashCommand.insertImage?.(1))
   const input = container.querySelector<HTMLInputElement>('input[type="file"]')
   if (!input) throw new Error('Editor image picker is unavailable')
   Object.defineProperty(input, 'files', {

apps/sim/app/workspace/[workspaceId]/home/components/mothership-view/mothership-view.test.tsx

@@ -5,6 +5,7 @@ import { authClientMock } from '@sim/testing/mocks/auth-client.mock'
 import { QueryClient, QueryClientProvider } from '@tanstack/react-query'
 import { createRoot } from 'react-dom/client'
 import { afterEach, expect, it, vi } from 'vitest'
+import type { ProjectFileRecord } from '@/lib/api/contracts/project-files'
 import { getChatResourceSelectionId } from '@/lib/mothership/resources/types'
 import { MothershipResourcesProvider } from '@/app/workspace/[workspaceId]/home/components/mothership-resources-context'
 import { MothershipView } from '@/app/workspace/[workspaceId]/home/components/mothership-view/mothership-view'
@@ -64,8 +65,21 @@ it.each([false, true])(
       owner: { entityType: 'project' as const, entityId: 'project' },
     }
     const options = getProjectFileQueryOptions('project', 'file')
+    const file: ProjectFileRecord = {
+      id: 'file',
+      owner: resource.owner,
+      name: 'notes.txt',
+      key: 'project/project/notes.txt',
+      path: '/notes.txt',
+      size: 5,
+      type: 'text/plain',
+      uploadedBy: 'user',
+      folderId: null,
+      uploadedAt: new Date('2026-10-01T00:00:00.000Z'),
+      updatedAt: new Date('2026-10-01T00:00:00.000Z'),
+    }
     client.setQueryData(options.queryKey, {
-      file: { id: 'file', name: 'notes.txt', type: 'text/plain' },
+      file,
       capabilities: { canRead: true, canWrite: false },
     })
     const container = document.createElement('div')

apps/sim/app/workspace/[workspaceId]/home/hooks/use-chat.dom.test.tsx

@@ -90,7 +90,10 @@ import {
 } from '@/lib/core/config/deployment-shape'
 import { MothershipHandoffStorage } from '@/lib/core/utils/browser-storage'
 import { MOTHERSHIP_STREAM_REPLAY_HEADER } from '@/lib/mothership/constants'
-import type { MothershipStreamV1EventEnvelope } from '@/lib/mothership/generated/mothership-stream-v1'
+import {
+  MothershipStreamV1CompletionStatus,
+  type MothershipStreamV1EventEnvelope,
+} from '@/lib/mothership/generated/mothership-stream-v1'
 import type { PersistedStreamEventEnvelope } from '@/lib/mothership/request/session/contract'
 import { getChatResourceSelectionId } from '@/lib/mothership/resources/types'
 import { ChatSurfaceProvider } from '@/app/workspace/[workspaceId]/home/components/chat-surface-context'
@@ -984,7 +987,14 @@ describe('useChat remount send recovery', () => {
       id: 'chat-strict-reconnect',
       title: 'Project architecture',
       activeStreamId: 'accepted-architecture',
-      messages: [{ id: 'accepted-architecture', role: 'user', content: 'Read the architecture' }],
+      messages: [
+        {
+          id: 'accepted-architecture',
+          role: 'user',
+          content: 'Read the architecture',
+          timestamp: '',
+        },
+      ],
       resources: [],
     }
     const event: MothershipStreamV1EventEnvelope = {
@@ -1024,7 +1034,9 @@ describe('useChat remount send recovery', () => {
       id: 'chat-reconnecting-search',
       title: 'Search',
       activeStreamId: 'accepted-search',
-      messages: [{ id: 'accepted-search', role: 'user', content: 'Find the policy' }],
+      messages: [
+        { id: 'accepted-search', role: 'user', content: 'Find the policy', timestamp: '' },
+      ],
       resources: [
         {
           type: 'search',
@@ -4942,19 +4954,27 @@ describe('useChat remount send recovery', () => {
         title: 'Trace inspection',
         activeStreamId: status === 'active' ? 'accepted-request' : null,
         messages: [
-          { id: 'accepted-request', role: 'user', content: 'check the trace for this req' },
+          {
+            id: 'accepted-request',
+            role: 'user',
+            content: 'check the trace for this req',
+            timestamp: '',
+          },
           ...(status === 'active'
             ? []
             : [
                 {
                   id: 'accepted-answer',
                   role: 'assistant' as const,
                   content: '',
+                  timestamp: '',
                   contentBlocks: [
                     {
                       type: 'complete' as const,
                       status:
-                        status === 'cancelled' ? ('cancelled' as const) : ('success' as const),
+                        status === 'cancelled'
+                          ? MothershipStreamV1CompletionStatus.cancelled
+                          : MothershipStreamV1CompletionStatus.complete,
                     },
                   ],
                 },
@@ -4980,7 +5000,12 @@ describe('useChat remount send recovery', () => {
       title: 'Trace inspection',
       activeStreamId: null,
       messages: [
-        { id: 'different-request', role: 'user', content: 'check the trace for this req' },
+        {
+          id: 'different-request',
+          role: 'user',
+          content: 'check the trace for this req',
+          timestamp: '',
+        },
       ],
       resources: [],
     })

apps/sim/hooks/queries/workspace-files.test.tsx

@@ -179,7 +179,7 @@ describe('useReloadWorkspaceFileContent', () => {
     async (cached) => {
       const owner = { entityType: 'project', entityId: 'project-1' } as const
       const oldRecord = {
-        file: { ...file, owner, key: 'project/project-1/old.md' },
+        file: { ...file, folderId: null, owner, key: 'project/project-1/old.md' },
         capabilities: { canRead: true, canWrite: true },
       }
       const currentRecord = {

apps/sim/lib/mothership/application/resource-adapter.test.ts

@@ -141,7 +141,7 @@ describe('Copilot resource authoring admission', () => {
   it('refuses a registered operation that does not admit Copilot resource delegation', () => {
     const refusedOperation = Object.freeze({
       ...operation,
-      delegatedServices: Object.freeze(['realtime']),
+      delegatedServices: Object.freeze(['realtime'] as const),
     } as const)
     const execute = createCopilotApplicationAdapter({
       mode: 'resource',

apps/sim/lib/projects/__integration__/foundation.integration.ts

@@ -1253,11 +1253,10 @@ describe('Project foundation at the database and application boundary', () => {
       })
       const blocker = await held.promise
       const deletion = db.transaction(async (tx) => {
-        const { projectIdsToDelete } = await prepareProjectsForAccountDeletion(tx, f.ownerId, [
-          f.ids[1],
-        ])
+        const projectIds = await prepareProjectsForAccountDeletion(tx, f.ownerId, [f.ids[1]])
+        expect(projectIds).toHaveLength(1)
         await tx.delete(workspace).where(eq(workspace.id, f.ids[1]))
-        await tx.delete(project).where(inArray(project.id, projectIdsToDelete))
+        await tx.delete(project).where(inArray(project.id, projectIds))
       })
       try {
         await waitUntilBlockedBy(blocker, "lock='project'")
@@ -1384,13 +1383,14 @@ describe('Project foundation at the database and application boundary', () => {
           .where(eq(workspace.projectId, privateProject.projectId))
       ).toEqual([{ workspaceId: privateProject.ids[0] }])
       await db.transaction(async (tx) => {
-        const { projectIdsToDelete } = await prepareProjectsForAccountDeletion(
+        const projectIds = await prepareProjectsForAccountDeletion(
           tx,
           privateProject.ownerId,
           privateProject.ids
         )
+        expect(projectIds).toEqual([privateProject.projectId])
         await tx.delete(workspace).where(inArray(workspace.id, privateProject.ids))
-        await tx.delete(project).where(inArray(project.id, projectIdsToDelete))
+        await tx.delete(project).where(inArray(project.id, projectIds))
       })
       expect(
         await db.select().from(project).where(eq(project.id, privateProject.projectId))

apps/sim/lib/projects/account-deletion.ts

@@ -148,13 +148,13 @@ export async function getProjectAccountDeletionBlockers(
   return blockers
 }
 
-/** Prepares private Project cleanup and returns IDs to delete after their workspaces in this transaction. */
+/** Transfers surviving Projects and returns private Projects to delete after their workspaces. */
 export async function prepareProjectsForAccountDeletion(
   tx: DbTransaction,
   userId: string,
   doomedWorkspaceIds: string[],
   additionalWorkspaceIds: string[] = []
-): Promise<{ projectIdsToDelete: string[]; storageCleanupEventIds: string[] }> {
+): Promise<string[]> {
   const ownedEnvironments = await tx
     .select({ id: workspace.id })
     .from(workspace)
@@ -264,12 +264,8 @@ export async function prepareProjectsForAccountDeletion(
       (environments.get(record.id) ?? []).filter((row) => !doomed.has(row.id)).map((row) => row.id)
     )
   }
-  const storageCleanupEventIds: string[] = []
   for (const { projectId: id, billableBytes } of removedProjects) {
-    storageCleanupEventIds.push(...(await purgeProjectFilesInTx(tx, id, billableBytes)))
-  }
-  return {
-    projectIdsToDelete: removedProjects.map(({ projectId }) => projectId),
-    storageCleanupEventIds,
+    await purgeProjectFilesInTx(tx, id, billableBytes)
   }
+  return projectsToDelete
 }

apps/sim/lib/projects/files/__integration__/private-purge.integration.ts

@@ -235,8 +235,7 @@ async function erasePrivateProject(f: Awaited<ReturnType<typeof fixture>>) {
   return db.transaction(async (tx) => {
     const result = await prepareProjectsForAccountDeletion(tx, f.userId, [f.workspaceId])
     await tx.delete(workspace).where(eq(workspace.id, f.workspaceId))
-    if (result.projectIdsToDelete.length)
-      await tx.delete(project).where(inArray(project.id, result.projectIdsToDelete))
+    await tx.delete(project).where(inArray(project.id, result))
     return result
   })
 }
@@ -291,10 +290,9 @@ describe('Private Project teardown and durable object cleanup', () => {
       const f = await fixture()
       await expect(
         db.transaction(async (tx) => {
-          const result = await prepareProjectsForAccountDeletion(tx, f.userId, [f.workspaceId])
+          const projectIds = await prepareProjectsForAccountDeletion(tx, f.userId, [f.workspaceId])
           await tx.delete(workspace).where(eq(workspace.id, f.workspaceId))
-          if (result.projectIdsToDelete.length)
-            await tx.delete(project).where(inArray(project.id, result.projectIdsToDelete))
+          await tx.delete(project).where(inArray(project.id, projectIds))
           throw new Error('Later account teardown failed')
         })
       ).rejects.toThrow('Later account teardown failed')
@@ -339,10 +337,9 @@ describe('Private Project teardown and durable object cleanup', () => {
     const deletion = db.transaction(async (tx) => {
       const [connection] = await tx.execute<{ pid: number }>(sql`SELECT pg_backend_pid() AS pid`)
       pid.resolve(connection.pid)
-      const result = await prepareProjectsForAccountDeletion(tx, f.userId, [f.workspaceId])
+      const projectIds = await prepareProjectsForAccountDeletion(tx, f.userId, [f.workspaceId])
       await tx.delete(workspace).where(eq(workspace.id, f.workspaceId))
-      if (result.projectIdsToDelete.length)
-        await tx.delete(project).where(inArray(project.id, result.projectIdsToDelete))
+      await tx.delete(project).where(inArray(project.id, projectIds))
     })
     const observedDeletion = deletion.then(
       () => null,