improvement(mothership): drop get_job operation — poll background jobs via query_rows

Per review, get_job is unnecessary: the model observes background
import/delete progress with query_rows (rows appearing as an import lands;
the delete mask already makes query_rows reflect the post-delete view).
Removes the operation and repoints the dispatch messages at query_rows.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: TheodoreSpeaks

apps/sim/lib/copilot/tools/server/table/user-table.test.ts

@@ -833,49 +833,3 @@ describe('userTableServerTool.delete_rows_by_filter', () => {
     expect(mockDeleteRowsByFilter).toHaveBeenCalledTimes(1)
   })
 })
-
-describe('userTableServerTool.get_job', () => {
-  beforeEach(() => {
-    vi.clearAllMocks()
-  })
-
-  it('returns the active job derived on the table', async () => {
-    mockGetTableById.mockResolvedValue(
-      buildTable({
-        rowCount: 1234,
-        jobId: 'job-1',
-        jobType: 'import',
-        jobStatus: 'running',
-        jobError: null,
-        jobRowsProcessed: 500,
-      })
-    )
-
-    const result = await userTableServerTool.execute(
-      { operation: 'get_job', args: { tableId: 'tbl_1' } },
-      { userId: 'user-1', workspaceId: 'workspace-1' }
-    )
-
-    expect(result.success).toBe(true)
-    expect(result.data?.job).toEqual({
-      id: 'job-1',
-      type: 'import',
-      status: 'running',
-      error: null,
-      rowsProcessed: 500,
-    })
-    expect(result.data?.rowCount).toBe(1234)
-  })
-
-  it('returns job: null when the table has no job', async () => {
-    mockGetTableById.mockResolvedValue(buildTable())
-
-    const result = await userTableServerTool.execute(
-      { operation: 'get_job', args: { tableId: 'tbl_1' } },
-      { userId: 'user-1', workspaceId: 'workspace-1' }
-    )
-
-    expect(result.success).toBe(true)
-    expect(result.data?.job).toBeNull()
-  })
-})

apps/sim/lib/copilot/tools/server/table/user-table.ts

@@ -374,43 +374,6 @@ export const userTableServerTool: BaseServerTool<UserTableArgs, UserTableResult>
           }
         }
 
-        case 'get_job': {
-          if (!args.tableId) {
-            return { success: false, message: 'Table ID is required' }
-          }
-          if (!workspaceId) {
-            return { success: false, message: 'Workspace ID is required' }
-          }
-
-          const table = await getTableById(args.tableId)
-          if (!table || table.workspaceId !== workspaceId) {
-            return { success: false, message: `Table not found: ${args.tableId}` }
-          }
-
-          if (!table.jobId) {
-            return {
-              success: true,
-              message: `No active or recent job for table "${table.name}"`,
-              data: { job: null, rowCount: table.rowCount },
-            }
-          }
-
-          return {
-            success: true,
-            message: `Job ${table.jobId} (${table.jobType}) is ${table.jobStatus}`,
-            data: {
-              job: {
-                id: table.jobId,
-                type: table.jobType,
-                status: table.jobStatus,
-                error: table.jobError ?? null,
-                rowsProcessed: table.jobRowsProcessed ?? 0,
-              },
-              rowCount: table.rowCount,
-            },
-          }
-        }
-
         case 'delete': {
           const tableIds: string[] = args.tableIds ?? (args.tableId ? [args.tableId] : [])
           if (tableIds.length === 0) {
@@ -812,7 +775,7 @@ export const userTableServerTool: BaseServerTool<UserTableArgs, UserTableResult>
               })
               return {
                 success: true,
-                message: `Started background delete of ${doomedCount} matching rows (job ${jobId}). The rows are hidden from reads immediately; call get_job to track progress.`,
+                message: `Started background delete of ${doomedCount} matching rows (job ${jobId}). The rows are hidden from reads immediately — query_rows already reflects the post-delete view.`,
                 data: { jobId, doomedCount },
               }
             }
@@ -1005,7 +968,7 @@ export const userTableServerTool: BaseServerTool<UserTableArgs, UserTableResult>
             }
             return {
               success: true,
-              message: `Created table "${table.name}" (${table.id}); importing rows from "${record.name}" in the background (job ${importId}). Columns and rows appear as the import progresses — call get_job with this tableId to track it.`,
+              message: `Created table "${table.name}" (${table.id}); importing rows from "${record.name}" in the background (job ${importId}). Columns and rows appear as the import progresses — query_rows to check what has landed.`,
               data: {
                 tableId: table.id,
                 tableName: table.name,
@@ -1170,7 +1133,7 @@ export const userTableServerTool: BaseServerTool<UserTableArgs, UserTableResult>
             })
             return {
               success: true,
-              message: `Started background ${mode} import of "${record.name}" into "${table.name}" (job ${importId}). Rows appear as the import progresses — call get_job to track it.`,
+              message: `Started background ${mode} import of "${record.name}" into "${table.name}" (job ${importId}). Rows appear as the import progresses — query_rows to check what has landed.`,
               data: { tableId: table.id, jobId: importId, mode },
             }
           }