From 2da0cbe365fd69dab300f7648c2dc55e67051ce3 Mon Sep 17 00:00:00 2001 From: Siddharth Ganesan Date: Mon, 6 Apr 2026 17:29:01 -0700 Subject: [PATCH] Remove dead tools --- .../home/components/message-content/utils.ts | 4 - .../resource-content/resource-content.tsx | 2 +- .../app/workspace/[workspaceId]/home/types.ts | 10 -- .../lib/copilot/generated/request-trace-v1.ts | 4 - .../lib/copilot/generated/tool-catalog-v1.ts | 168 +----------------- .../tools/client/tool-display-registry.ts | 64 ------- 6 files changed, 7 insertions(+), 245 deletions(-) diff --git a/apps/sim/app/workspace/[workspaceId]/home/components/message-content/utils.ts b/apps/sim/app/workspace/[workspaceId]/home/components/message-content/utils.ts index a71a8dc30f..a60860b964 100644 --- a/apps/sim/app/workspace/[workspaceId]/home/components/message-content/utils.ts +++ b/apps/sim/app/workspace/[workspaceId]/home/components/message-content/utils.ts @@ -4,7 +4,6 @@ import { Blimp, Bug, Calendar, - ClipboardList, Database, Eye, File, @@ -55,10 +54,7 @@ const TOOL_ICONS: Record = { agent: AgentIcon, custom_tool: Wrench, research: Search, - plan: ClipboardList, debug: Bug, - edit: Pencil, - fast_edit: Pencil, context_compaction: Asterisk, open_resource: Eye, file_write: File, diff --git a/apps/sim/app/workspace/[workspaceId]/home/components/mothership-view/components/resource-content/resource-content.tsx b/apps/sim/app/workspace/[workspaceId]/home/components/mothership-view/components/resource-content/resource-content.tsx index fac1fbac64..ee29209dbf 100644 --- a/apps/sim/app/workspace/[workspaceId]/home/components/mothership-view/components/resource-content/resource-content.tsx +++ b/apps/sim/app/workspace/[workspaceId]/home/components/mothership-view/components/resource-content/resource-content.tsx @@ -537,7 +537,7 @@ function extractFileContent(raw: string): string { const marker = '"content":' const idx = raw.indexOf(marker) if (idx === -1) return '' - let rest = raw.slice(idx + marker.length).trimStart() + const rest = raw.slice(idx + marker.length).trimStart() if (!rest.startsWith('"')) return rest // Walk the JSON string value to find the unescaped closing quote. diff --git a/apps/sim/app/workspace/[workspaceId]/home/types.ts b/apps/sim/app/workspace/[workspaceId]/home/types.ts index 1d43c825d2..e410ee7c9c 100644 --- a/apps/sim/app/workspace/[workspaceId]/home/types.ts +++ b/apps/sim/app/workspace/[workspaceId]/home/types.ts @@ -186,10 +186,7 @@ export const SUBAGENT_LABELS: Record = { table: 'Table agent', custom_tool: 'Custom Tool agent', superagent: 'Superagent', - plan: 'Plan agent', debug: 'Debug agent', - edit: 'Edit agent', - fast_edit: 'Build agent', run: 'Run agent', agent: 'Agent manager', job: 'Job agent', @@ -311,14 +308,7 @@ export const TOOL_UI_METADATA: Record = { phase: 'subagent', }, [Research.id]: { title: 'Researching', phaseLabel: 'Research', phase: 'subagent' }, - plan: { title: 'Planning', phaseLabel: 'Plan', phase: 'subagent' }, [Debug.id]: { title: 'Debugging', phaseLabel: 'Debug', phase: 'subagent' }, - edit: { title: 'Editing workflow', phaseLabel: 'Edit', phase: 'subagent' }, - fast_edit: { - title: 'Editing workflow', - phaseLabel: 'Edit', - phase: 'subagent', - }, [OpenResource.id]: { title: 'Opening resource', phaseLabel: 'Resource', diff --git a/apps/sim/lib/copilot/generated/request-trace-v1.ts b/apps/sim/lib/copilot/generated/request-trace-v1.ts index 8213b11e22..309b49ad06 100644 --- a/apps/sim/lib/copilot/generated/request-trace-v1.ts +++ b/apps/sim/lib/copilot/generated/request-trace-v1.ts @@ -84,12 +84,8 @@ export interface RequestTraceV1MergedTrace { cost?: RequestTraceV1CostSummary durationMs: number endMs: number - executionId?: string goTraceId: string - model?: string outcome: RequestTraceV1Outcome - provider?: string - runId?: string serviceCharges?: MothershipStreamV1AdditionalPropertiesMap simRequestId?: string spans: RequestTraceV1Span[] diff --git a/apps/sim/lib/copilot/generated/tool-catalog-v1.ts b/apps/sim/lib/copilot/generated/tool-catalog-v1.ts index 42bcd87a4d..867a5032c7 100644 --- a/apps/sim/lib/copilot/generated/tool-catalog-v1.ts +++ b/apps/sim/lib/copilot/generated/tool-catalog-v1.ts @@ -8,11 +8,8 @@ export interface ToolCatalogEntry { hidden?: boolean id: | 'agent' - | 'agent_respond' | 'auth' - | 'auth_respond' | 'build' - | 'build_respond' | 'check_deployment_status' | 'complete_job' | 'context_write' @@ -22,7 +19,6 @@ export interface ToolCatalogEntry { | 'create_workflow' | 'create_workspace_mcp_server' | 'debug' - | 'debug_respond' | 'delete_folder' | 'delete_workflow' | 'delete_workspace_mcp_server' @@ -30,11 +26,8 @@ export interface ToolCatalogEntry { | 'deploy_api' | 'deploy_chat' | 'deploy_mcp' - | 'deploy_respond' | 'download_to_workspace_file' - | 'edit_respond' | 'edit_workflow' - | 'fast_edit_respond' | 'file_write' | 'function_execute' | 'generate_api_key' @@ -53,10 +46,8 @@ export interface ToolCatalogEntry { | 'glob' | 'grep' | 'job' - | 'job_respond' | 'knowledge' | 'knowledge_base' - | 'knowledge_respond' | 'list_folders' | 'list_user_workspaces' | 'list_workspace_mcp_servers' @@ -69,16 +60,14 @@ export interface ToolCatalogEntry { | 'oauth_get_auth_link' | 'oauth_request_access' | 'open_resource' - | 'plan_respond' | 'read' | 'redeploy' | 'research' - | 'research_respond' + | 'respond' | 'revert_to_version' | 'run' | 'run_block' | 'run_from_block' - | 'run_respond' | 'run_workflow' | 'run_workflow_until_block' | 'scrape_page' @@ -89,9 +78,7 @@ export interface ToolCatalogEntry { | 'set_environment_variables' | 'set_global_workflow_variables' | 'superagent' - | 'superagent_respond' | 'table' - | 'table_respond' | 'tool_search_tool_regex' | 'update_job_history' | 'update_workspace_mcp_server' @@ -102,11 +89,8 @@ export interface ToolCatalogEntry { mode: 'async' | 'sync' name: | 'agent' - | 'agent_respond' | 'auth' - | 'auth_respond' | 'build' - | 'build_respond' | 'check_deployment_status' | 'complete_job' | 'context_write' @@ -116,7 +100,6 @@ export interface ToolCatalogEntry { | 'create_workflow' | 'create_workspace_mcp_server' | 'debug' - | 'debug_respond' | 'delete_folder' | 'delete_workflow' | 'delete_workspace_mcp_server' @@ -124,11 +107,8 @@ export interface ToolCatalogEntry { | 'deploy_api' | 'deploy_chat' | 'deploy_mcp' - | 'deploy_respond' | 'download_to_workspace_file' - | 'edit_respond' | 'edit_workflow' - | 'fast_edit_respond' | 'file_write' | 'function_execute' | 'generate_api_key' @@ -147,10 +127,8 @@ export interface ToolCatalogEntry { | 'glob' | 'grep' | 'job' - | 'job_respond' | 'knowledge' | 'knowledge_base' - | 'knowledge_respond' | 'list_folders' | 'list_user_workspaces' | 'list_workspace_mcp_servers' @@ -163,16 +141,14 @@ export interface ToolCatalogEntry { | 'oauth_get_auth_link' | 'oauth_request_access' | 'open_resource' - | 'plan_respond' | 'read' | 'redeploy' | 'research' - | 'research_respond' + | 'respond' | 'revert_to_version' | 'run' | 'run_block' | 'run_from_block' - | 'run_respond' | 'run_workflow' | 'run_workflow_until_block' | 'scrape_page' @@ -183,9 +159,7 @@ export interface ToolCatalogEntry { | 'set_environment_variables' | 'set_global_workflow_variables' | 'superagent' - | 'superagent_respond' | 'table' - | 'table_respond' | 'tool_search_tool_regex' | 'update_job_history' | 'update_workspace_mcp_server' @@ -219,15 +193,6 @@ export const Agent: ToolCatalogEntry = { requiredPermission: 'write', } -export const AgentRespond: ToolCatalogEntry = { - id: 'agent_respond', - name: 'agent_respond', - executor: 'sim', - mode: 'async', - internal: true, - hidden: true, -} - export const Auth: ToolCatalogEntry = { id: 'auth', name: 'auth', @@ -237,15 +202,6 @@ export const Auth: ToolCatalogEntry = { internal: true, } -export const AuthRespond: ToolCatalogEntry = { - id: 'auth_respond', - name: 'auth_respond', - executor: 'sim', - mode: 'async', - internal: true, - hidden: true, -} - export const Build: ToolCatalogEntry = { id: 'build', name: 'build', @@ -255,15 +211,6 @@ export const Build: ToolCatalogEntry = { internal: true, } -export const BuildRespond: ToolCatalogEntry = { - id: 'build_respond', - name: 'build_respond', - executor: 'sim', - mode: 'async', - internal: true, - hidden: true, -} - export const CheckDeploymentStatus: ToolCatalogEntry = { id: 'check_deployment_status', name: 'check_deployment_status', @@ -333,15 +280,6 @@ export const Debug: ToolCatalogEntry = { internal: true, } -export const DebugRespond: ToolCatalogEntry = { - id: 'debug_respond', - name: 'debug_respond', - executor: 'sim', - mode: 'async', - internal: true, - hidden: true, -} - export const DeleteFolder: ToolCatalogEntry = { id: 'delete_folder', name: 'delete_folder', @@ -405,15 +343,6 @@ export const DeployMcp: ToolCatalogEntry = { requiredPermission: 'admin', } -export const DeployRespond: ToolCatalogEntry = { - id: 'deploy_respond', - name: 'deploy_respond', - executor: 'sim', - mode: 'async', - internal: true, - hidden: true, -} - export const DownloadToWorkspaceFile: ToolCatalogEntry = { id: 'download_to_workspace_file', name: 'download_to_workspace_file', @@ -422,15 +351,6 @@ export const DownloadToWorkspaceFile: ToolCatalogEntry = { requiredPermission: 'write', } -export const EditRespond: ToolCatalogEntry = { - id: 'edit_respond', - name: 'edit_respond', - executor: 'sim', - mode: 'async', - internal: true, - hidden: true, -} - export const EditWorkflow: ToolCatalogEntry = { id: 'edit_workflow', name: 'edit_workflow', @@ -439,15 +359,6 @@ export const EditWorkflow: ToolCatalogEntry = { requiredPermission: 'write', } -export const FastEditRespond: ToolCatalogEntry = { - id: 'fast_edit_respond', - name: 'fast_edit_respond', - executor: 'sim', - mode: 'async', - internal: true, - hidden: true, -} - export const FileWrite: ToolCatalogEntry = { id: 'file_write', name: 'file_write', @@ -583,15 +494,6 @@ export const Job: ToolCatalogEntry = { internal: true, } -export const JobRespond: ToolCatalogEntry = { - id: 'job_respond', - name: 'job_respond', - executor: 'sim', - mode: 'async', - internal: true, - hidden: true, -} - export const Knowledge: ToolCatalogEntry = { id: 'knowledge', name: 'knowledge', @@ -609,15 +511,6 @@ export const KnowledgeBase: ToolCatalogEntry = { requiresConfirmation: true, } -export const KnowledgeRespond: ToolCatalogEntry = { - id: 'knowledge_respond', - name: 'knowledge_respond', - executor: 'sim', - mode: 'async', - internal: true, - hidden: true, -} - export const ListFolders: ToolCatalogEntry = { id: 'list_folders', name: 'list_folders', @@ -711,15 +604,6 @@ export const OpenResource: ToolCatalogEntry = { mode: 'async', } -export const PlanRespond: ToolCatalogEntry = { - id: 'plan_respond', - name: 'plan_respond', - executor: 'sim', - mode: 'async', - internal: true, - hidden: true, -} - export const Read: ToolCatalogEntry = { id: 'read', name: 'read', @@ -745,9 +629,9 @@ export const Research: ToolCatalogEntry = { internal: true, } -export const ResearchRespond: ToolCatalogEntry = { - id: 'research_respond', - name: 'research_respond', +export const Respond: ToolCatalogEntry = { + id: 'respond', + name: 'respond', executor: 'sim', mode: 'async', internal: true, @@ -790,15 +674,6 @@ export const RunFromBlock: ToolCatalogEntry = { requiresConfirmation: true, } -export const RunRespond: ToolCatalogEntry = { - id: 'run_respond', - name: 'run_respond', - executor: 'sim', - mode: 'async', - internal: true, - hidden: true, -} - export const RunWorkflow: ToolCatalogEntry = { id: 'run_workflow', name: 'run_workflow', @@ -879,15 +754,6 @@ export const Superagent: ToolCatalogEntry = { internal: true, } -export const SuperagentRespond: ToolCatalogEntry = { - id: 'superagent_respond', - name: 'superagent_respond', - executor: 'sim', - mode: 'async', - internal: true, - hidden: true, -} - export const Table: ToolCatalogEntry = { id: 'table', name: 'table', @@ -897,15 +763,6 @@ export const Table: ToolCatalogEntry = { internal: true, } -export const TableRespond: ToolCatalogEntry = { - id: 'table_respond', - name: 'table_respond', - executor: 'sim', - mode: 'async', - internal: true, - hidden: true, -} - export const ToolSearchToolRegex: ToolCatalogEntry = { id: 'tool_search_tool_regex', name: 'tool_search_tool_regex', @@ -954,11 +811,8 @@ export const WorkspaceFile: ToolCatalogEntry = { export const TOOL_CATALOG: Record = { [Agent.id]: Agent, - [AgentRespond.id]: AgentRespond, [Auth.id]: Auth, - [AuthRespond.id]: AuthRespond, [Build.id]: Build, - [BuildRespond.id]: BuildRespond, [CheckDeploymentStatus.id]: CheckDeploymentStatus, [CompleteJob.id]: CompleteJob, [ContextWrite.id]: ContextWrite, @@ -968,7 +822,6 @@ export const TOOL_CATALOG: Record = { [CreateWorkflow.id]: CreateWorkflow, [CreateWorkspaceMcpServer.id]: CreateWorkspaceMcpServer, [Debug.id]: Debug, - [DebugRespond.id]: DebugRespond, [DeleteFolder.id]: DeleteFolder, [DeleteWorkflow.id]: DeleteWorkflow, [DeleteWorkspaceMcpServer.id]: DeleteWorkspaceMcpServer, @@ -976,11 +829,8 @@ export const TOOL_CATALOG: Record = { [DeployApi.id]: DeployApi, [DeployChat.id]: DeployChat, [DeployMcp.id]: DeployMcp, - [DeployRespond.id]: DeployRespond, [DownloadToWorkspaceFile.id]: DownloadToWorkspaceFile, - [EditRespond.id]: EditRespond, [EditWorkflow.id]: EditWorkflow, - [FastEditRespond.id]: FastEditRespond, [FileWrite.id]: FileWrite, [FunctionExecute.id]: FunctionExecute, [GenerateApiKey.id]: GenerateApiKey, @@ -999,10 +849,8 @@ export const TOOL_CATALOG: Record = { [Glob.id]: Glob, [Grep.id]: Grep, [Job.id]: Job, - [JobRespond.id]: JobRespond, [Knowledge.id]: Knowledge, [KnowledgeBase.id]: KnowledgeBase, - [KnowledgeRespond.id]: KnowledgeRespond, [ListFolders.id]: ListFolders, [ListUserWorkspaces.id]: ListUserWorkspaces, [ListWorkspaceMcpServers.id]: ListWorkspaceMcpServers, @@ -1015,16 +863,14 @@ export const TOOL_CATALOG: Record = { [OauthGetAuthLink.id]: OauthGetAuthLink, [OauthRequestAccess.id]: OauthRequestAccess, [OpenResource.id]: OpenResource, - [PlanRespond.id]: PlanRespond, [Read.id]: Read, [Redeploy.id]: Redeploy, [Research.id]: Research, - [ResearchRespond.id]: ResearchRespond, + [Respond.id]: Respond, [RevertToVersion.id]: RevertToVersion, [Run.id]: Run, [RunBlock.id]: RunBlock, [RunFromBlock.id]: RunFromBlock, - [RunRespond.id]: RunRespond, [RunWorkflow.id]: RunWorkflow, [RunWorkflowUntilBlock.id]: RunWorkflowUntilBlock, [ScrapePage.id]: ScrapePage, @@ -1035,9 +881,7 @@ export const TOOL_CATALOG: Record = { [SetEnvironmentVariables.id]: SetEnvironmentVariables, [SetGlobalWorkflowVariables.id]: SetGlobalWorkflowVariables, [Superagent.id]: Superagent, - [SuperagentRespond.id]: SuperagentRespond, [Table.id]: Table, - [TableRespond.id]: TableRespond, [ToolSearchToolRegex.id]: ToolSearchToolRegex, [UpdateJobHistory.id]: UpdateJobHistory, [UpdateWorkspaceMcpServer.id]: UpdateWorkspaceMcpServer, diff --git a/apps/sim/lib/copilot/tools/client/tool-display-registry.ts b/apps/sim/lib/copilot/tools/client/tool-display-registry.ts index cad83d1ec4..6d0b04ae28 100644 --- a/apps/sim/lib/copilot/tools/client/tool-display-registry.ts +++ b/apps/sim/lib/copilot/tools/client/tool-display-registry.ts @@ -4,7 +4,6 @@ import { Bug, Check, CheckCircle, - ClipboardList, Database, Eye, FileSearch, @@ -2087,66 +2086,6 @@ const META_superagent: ToolMetadata = { }, } -const META_plan: ToolMetadata = { - displayNames: { - [ClientToolCallState.generating]: { text: 'Planning', icon: Loader2 }, - [ClientToolCallState.pending]: { text: 'Planning', icon: Loader2 }, - [ClientToolCallState.executing]: { text: 'Planning', icon: Loader2 }, - [ClientToolCallState.success]: { text: 'Planned', icon: ClipboardList }, - [ClientToolCallState.error]: { text: 'Failed to plan', icon: XCircle }, - [ClientToolCallState.rejected]: { text: 'Skipped planning', icon: XCircle }, - [ClientToolCallState.aborted]: { text: 'Aborted planning', icon: XCircle }, - }, - uiConfig: { - subagent: { - streamingLabel: 'Planning', - completedLabel: 'Planned', - shouldCollapse: true, - outputArtifacts: [], - }, - }, -} - -const META_edit: ToolMetadata = { - displayNames: { - [ClientToolCallState.generating]: { text: 'Editing', icon: Loader2 }, - [ClientToolCallState.pending]: { text: 'Editing', icon: Loader2 }, - [ClientToolCallState.executing]: { text: 'Editing', icon: Loader2 }, - [ClientToolCallState.success]: { text: 'Edited', icon: PencilLine }, - [ClientToolCallState.error]: { text: 'Failed to edit', icon: XCircle }, - [ClientToolCallState.rejected]: { text: 'Skipped editing', icon: XCircle }, - [ClientToolCallState.aborted]: { text: 'Aborted editing', icon: XCircle }, - }, - uiConfig: { - subagent: { - streamingLabel: 'Editing', - completedLabel: 'Edited', - shouldCollapse: true, - outputArtifacts: [], - }, - }, -} - -const META_fast_edit: ToolMetadata = { - displayNames: { - [ClientToolCallState.generating]: { text: 'Building', icon: Loader2 }, - [ClientToolCallState.pending]: { text: 'Building', icon: Loader2 }, - [ClientToolCallState.executing]: { text: 'Building', icon: Loader2 }, - [ClientToolCallState.success]: { text: 'Built', icon: Wrench }, - [ClientToolCallState.error]: { text: 'Failed to build', icon: XCircle }, - [ClientToolCallState.rejected]: { text: 'Skipped build', icon: XCircle }, - [ClientToolCallState.aborted]: { text: 'Aborted build', icon: XCircle }, - }, - uiConfig: { - subagent: { - streamingLabel: 'Building', - completedLabel: 'Built', - shouldCollapse: true, - outputArtifacts: [], - }, - }, -} - const META_debug: ToolMetadata = { displayNames: { [ClientToolCallState.generating]: { text: 'Debugging', icon: Loader2 }, @@ -2369,8 +2308,6 @@ const TOOL_METADATA_BY_ID: Record = { deploy_api: META_deploy_api, deploy_chat: META_deploy_chat, deploy_mcp: META_deploy_mcp, - edit: META_edit, - fast_edit: META_fast_edit, edit_workflow: META_edit_workflow, get_block_outputs: META_get_block_outputs, get_block_upstream_references: META_get_block_upstream_references, @@ -2401,7 +2338,6 @@ const TOOL_METADATA_BY_ID: Record = { oauth_get_auth_link: META_oauth_get_auth_link, oauth_request_access: META_oauth_request_access, open_resource: META_open_resource, - plan: META_plan, read: META_read, redeploy: META_redeploy, rename_workflow: META_rename_workflow,