diff --git a/site/src/pages/AgentsPage/components/ChatElements/tools/ExecuteTool.tsx b/site/src/pages/AgentsPage/components/ChatElements/tools/ExecuteTool.tsx index 11064f5630c86..8bc80e6e8422f 100644 --- a/site/src/pages/AgentsPage/components/ChatElements/tools/ExecuteTool.tsx +++ b/site/src/pages/AgentsPage/components/ChatElements/tools/ExecuteTool.tsx @@ -1,9 +1,7 @@ import { - CheckIcon, CircleAlertIcon, ExternalLinkIcon, LayersIcon, - LoaderIcon, OctagonXIcon, } from "lucide-react"; import type React from "react"; @@ -283,81 +281,3 @@ export const ExecuteAuthRequiredTool: React.FC<{ ); }; - -export const WaitForExternalAuthTool: React.FC<{ - providerLabel: string; - status: ToolStatus; - authenticated: boolean; - timedOut: boolean; - isError: boolean; - errorMessage?: string; -}> = ({ - providerLabel, - status, - authenticated, - timedOut, - isError, - errorMessage, -}) => { - const isRunning = status === "running"; - let label = `Waiting for ${providerLabel} authentication...`; - let statusIcon: React.ReactNode = isRunning ? ( - - ) : null; - if (isError) { - label = - errorMessage || - `Failed while waiting for ${providerLabel} authentication`; - statusIcon = ( - - ); - } else if (timedOut) { - label = `Timed out waiting for ${providerLabel} authentication`; - statusIcon = ( - - ); - } else if (authenticated && !isRunning) { - label = `Authenticated with ${providerLabel}`; - statusIcon = ( - - ); - } - - return ( - - - - {statusIcon} - - {label} - - - - - ); -}; diff --git a/site/src/pages/AgentsPage/components/ChatElements/tools/Tool.stories.tsx b/site/src/pages/AgentsPage/components/ChatElements/tools/Tool.stories.tsx index aa73dbbe2bb48..712b550f527e0 100644 --- a/site/src/pages/AgentsPage/components/ChatElements/tools/Tool.stories.tsx +++ b/site/src/pages/AgentsPage/components/ChatElements/tools/Tool.stories.tsx @@ -100,11 +100,6 @@ const allToolShowcaseItems: ToolShowcaseItem[] = [ args: { process_id: "storybook-process", signal: "terminate" }, result: { success: true }, }, - { - name: "wait_for_external_auth", - args: { provider: "github" }, - result: { provider_display_name: "GitHub", authenticated: true }, - }, { name: "read_file", args: { path: "site/src/pages/AgentsPage/AgentChatPage.tsx" }, @@ -601,68 +596,6 @@ export const ExecuteAuthRequired: Story = { }, }; -// --------------------------------------------------------------------------- -// WaitForExternalAuth stories -// --------------------------------------------------------------------------- - -export const WaitForExternalAuthRunning: Story = { - args: { - name: "wait_for_external_auth", - status: "running", - result: { - provider_display_name: "GitHub", - authenticated: false, - }, - }, - play: async ({ canvasElement }) => { - const canvas = within(canvasElement); - expect( - canvas.getByText("Waiting for GitHub authentication..."), - ).toBeInTheDocument(); - expect( - canvas.getByRole("img", { name: "Authentication in progress" }), - ).toBeVisible(); - }, -}; - -export const WaitForExternalAuthAuthenticated: Story = { - args: { - name: "wait_for_external_auth", - status: "completed", - result: { - provider_display_name: "GitHub", - authenticated: true, - }, - }, - play: async ({ canvasElement }) => { - const canvas = within(canvasElement); - expect(canvas.getByText("Authenticated with GitHub")).toBeInTheDocument(); - }, -}; - -export const WaitForExternalAuthTimedOut: Story = { - args: { - name: "wait_for_external_auth", - status: "completed", - result: { - provider_display_name: "GitHub", - timed_out: true, - }, - }, -}; - -export const WaitForExternalAuthError: Story = { - args: { - name: "wait_for_external_auth", - status: "error", - isError: true, - result: { - provider_display_name: "GitHub", - error: "Authentication failed: token exchange was rejected.", - }, - }, -}; - // --------------------------------------------------------------------------- // Subagent stories // --------------------------------------------------------------------------- diff --git a/site/src/pages/AgentsPage/components/ChatElements/tools/Tool.tsx b/site/src/pages/AgentsPage/components/ChatElements/tools/Tool.tsx index 494dbc70ad77e..0d8bf0663a062 100644 --- a/site/src/pages/AgentsPage/components/ChatElements/tools/Tool.tsx +++ b/site/src/pages/AgentsPage/components/ChatElements/tools/Tool.tsx @@ -17,7 +17,6 @@ import { EditFilesTool } from "./EditFilesTool"; import { ExecuteAuthRequiredTool, ExecuteTool as ExecuteToolComponent, - WaitForExternalAuthTool, } from "./ExecuteTool"; import { ListAgentsTool } from "./ListAgentsTool"; import { ListTemplatesTool } from "./ListTemplatesTool"; @@ -59,7 +58,6 @@ import { parseServerEditDiffText, parseServerEditResults, type ToolStatus, - toProviderLabel, } from "./utils"; import { WriteFileTool } from "./WriteFileTool"; @@ -285,33 +283,6 @@ const ProcessOutputRenderer: FC = ({ ); }; -const WaitForExternalAuthRenderer: FC = ({ - status, - result, - isError, -}) => { - const rec = asRecord(result); - const providerLabel = toProviderLabel( - rec ? asString(rec.provider_display_name).trim() : "", - rec ? asString(rec.provider_id).trim() : "", - rec ? asString(rec.provider_type).trim() : "", - ); - const authenticated = rec ? Boolean(rec.authenticated) : false; - const timedOut = rec ? Boolean(rec.timed_out) : false; - const errorMessage = rec ? asString(rec.error || rec.message) : ""; - - return ( - - ); -}; - const ReadFileRenderer: FC = ({ status, args, @@ -1066,7 +1037,6 @@ const toolRenderers: Record> = { execute: ExecuteRenderer, process_output: ProcessOutputRenderer, process_signal: ProcessSignalRenderer, - wait_for_external_auth: WaitForExternalAuthRenderer, read_file: ReadFileRenderer, write_file: WriteFileRenderer, edit_files: EditFilesRenderer, diff --git a/site/src/pages/AgentsPage/components/ChatElements/tools/ToolIcon.tsx b/site/src/pages/AgentsPage/components/ChatElements/tools/ToolIcon.tsx index 6db7eb5604e22..16e9bc148b3c7 100644 --- a/site/src/pages/AgentsPage/components/ChatElements/tools/ToolIcon.tsx +++ b/site/src/pages/AgentsPage/components/ChatElements/tools/ToolIcon.tsx @@ -5,7 +5,6 @@ import { FilePenLineIcon, FileTextIcon, LightbulbIcon, - LogInIcon, MonitorIcon, PowerIcon, RouteIcon, @@ -80,8 +79,6 @@ export const ToolIcon: React.FC<{ case "process_list": case "process_signal": return ; - case "wait_for_external_auth": - return ; case "read_file": case "read_skill": case "read_skill_file":