fix(copilot): store tool file outputs from workspace chats under the Copilot user

Author: waleedlatif1

apps/sim/lib/internal/tool-operations/file-result.server.test.ts

@@ -53,6 +53,7 @@ const runContext: InternalToolOperationContext = {
 const copilotContext: InternalToolOperationContext = {
   workspaceId: 'workspace-1',
   workflowId: '',
+  executionId: 'copilot-run-1',
   userId: 'user-1',
   copilotToolExecution: true,
 }
@@ -158,6 +159,20 @@ describe('presentInternalToolOperationResult', () => {
     expect(mockUploadExecutionFile).not.toHaveBeenCalled()
   })
 
+  it("keeps a Mothership block's files in its parent workflow run", async () => {
+    await presentInternalToolOperationResult(
+      createInternalToolFileResult(file(), (stored) => ({ file: stored })),
+      { ...runContext, copilotToolExecution: true }
+    )
+
+    expect(mockUploadExecutionFile.mock.calls[0]?.[0]).toEqual({
+      workspaceId: 'workspace-1',
+      workflowId: 'workflow-1',
+      executionId: 'execution-1',
+    })
+    expect(mockUploadCopilotFile).not.toHaveBeenCalled()
+  })
+
   it('replaces binary representation headers before the JSON transport size check', async () => {
     const input = file(Buffer.alloc(12 * 1024 * 1024))
     const headers = new Headers({

apps/sim/lib/internal/tool-operations/file-result.server.ts

@@ -52,7 +52,9 @@ function resolveCopilotUserId(context: InternalToolOperationContext): string {
 }
 
 function resolveFileStorageScope(context: InternalToolOperationContext): FileStorageScope {
-  if (context.executionId) {
+  // A workspace Copilot turn's executionId names the turn, not a workflow run.
+  const isWorkflowlessCopilotTurn = context.copilotToolExecution && !context.workflowId.trim()
+  if (context.executionId && !isWorkflowlessCopilotTurn) {
     if (!context.workspaceId?.trim() || !context.workflowId.trim() || !context.executionId.trim()) {
       throw new Error('Execution file output requires a complete trusted execution scope')
     }