Sep 26, 2025 · Sep 24, 2025 · Sep 24, 2025 · Sep 25, 2025 · Sep 25, 2025 · Sep 25, 2025
diff --git a/site/src/modules/tasks/tasks.ts b/site/src/modules/tasks/tasks.ts
 })),
 );
 }

 export function getTaskWebApps(task: Task): WorkspaceAppWithAgent[] {
 // The Chat UI app will be displayed in the sidebar, so we don't want to
 // show it as a web app.
 return getTaskApps(task).filter(
 (app) => app.id !== task.workspace.latest_build.ai_task_sidebar_app_id,
 );
 }
diff --git a/site/src/pages/TaskPage/TaskApps.tsx b/site/src/pages/TaskPage/TaskApps.tsx
 import { ChevronDownIcon, LayoutGridIcon } from "lucide-react";
 import { useAppLink } from "modules/apps/useAppLink";
 import {
 getTaskWebApps,
 getTaskApps,
 type Task,
 type WorkspaceAppWithAgent,
 } from "modules/tasks/tasks";
 };

 export const TaskApps: FC<TaskAppsProps> = ({ task }) => {
 const apps = getTaskWebApps(task);
 const apps = getTaskApps(task).filter(
 // The Chat UI app will be displayed in the sidebar, so we don't want to
 // show it as a web app.
 (app) => app.id !== task.workspace.latest_build.ai_task_sidebar_app_id,
 );
 const [embeddedApps, externalApps] = splitEmbeddedAndExternalApps(apps);
 const [activeAppId, setActiveAppId] = useState(embeddedApps.at(0)?.id);

diff --git a/site/src/pages/TaskPage/TaskPage.tsx b/site/src/pages/TaskPage/TaskPage.tsx
 } else if (agent && ["created", "starting"].includes(agent.lifecycle_state)) {
 content = <TaskStartingAgent agent={agent} />;
 } else {
 const chatApp = selectChatApp(task);
 const chatApp = getTaskApps(task).find(
 (app) => app.id === task.workspace.latest_build.ai_task_sidebar_app_id,
 );
 content = (
 <PanelGroup autoSaveId="task" direction="horizontal">
 <Panel defaultSize={25} minSize={20}>

 return agents.at(0);
 }

 function selectChatApp(task: Task) {
 return getTaskApps(task).find(
 (app) => app.id === task.workspace.latest_build.ai_task_sidebar_app_id,
 );
 }
Original file line number	Diff line number	Diff line change
Expand Up		@@ -25,11 +25,3 @@ export function getTaskApps(task: Task): WorkspaceAppWithAgent[] {
		})),
		);
		}

		export function getTaskWebApps(task: Task): WorkspaceAppWithAgent[] {
		// The Chat UI app will be displayed in the sidebar, so we don't want to
		// show it as a web app.
		return getTaskApps(task).filter(
		(app) => app.id !== task.workspace.latest_build.ai_task_sidebar_app_id,
		);
		}
Original file line number	Diff line number	Diff line change
Expand Up		@@ -12,7 +12,7 @@ import { ScrollArea, ScrollBar } from "components/ScrollArea/ScrollArea";
		import { ChevronDownIcon, LayoutGridIcon } from "lucide-react";
		import { useAppLink } from "modules/apps/useAppLink";
		import {
		getTaskWebApps,
		getTaskApps,
		type Task,
		type WorkspaceAppWithAgent,
		} from "modules/tasks/tasks";
Expand All		@@ -28,7 +28,11 @@ type TaskAppsProps = {
		};

		export const TaskApps: FC<TaskAppsProps> = ({ task }) => {
		const apps = getTaskWebApps(task);
		const apps = getTaskApps(task).filter(
		// The Chat UI app will be displayed in the sidebar, so we don't want to
		// show it as a web app.
		(app) => app.id !== task.workspace.latest_build.ai_task_sidebar_app_id,
		);
Comment on lines 31 to 37 Copy link Member aslilacSep 25, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. I think a`getTaskApp` helper could still be nice here Copy link CollaboratorAuthor BrunoQuaresmaSep 26, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. Not sure if I understood—`getTaskApps` is already being used 🤔 Copy link Member aslilacSep 26, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. sorry, I meant`getChatApp` Copy link Member aslilacSep 26, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. and you're right, it's doing the opposite here 🤦‍♀️
		const [embeddedApps, externalApps] = splitEmbeddedAndExternalApps(apps);
		const [activeAppId, setActiveAppId] = useState(embeddedApps.at(0)?.id);

Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -130,7 +130,9 @@ const TaskPage = () => {
		} else if (agent && ["created", "starting"].includes(agent.lifecycle_state)) {
		content = <TaskStartingAgent agent={agent} />;
		} else {
		const chatApp = selectChatApp(task);
		const chatApp = getTaskApps(task).find(
		(app) => app.id === task.workspace.latest_build.ai_task_sidebar_app_id,
		);
Comment on lines +149 to +151 Copy link Member aslilacSep 25, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. ...and could be used here as well Copy link CollaboratorAuthor BrunoQuaresmaSep 26, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. They’re quite different: The first removes the chat app from the apps list. The second fetches the chat app. The only common part is accessing`ai_task_sidebar_app_id`, so a helper could be: functiongetChatAppId(task:Task){returntask.workspace.latest_build.ai_task_sidebar_app_id;} That said, I don’t see much value in adding this helper right now—what do you think?
		content = (
		<PanelGroup autoSaveId="task" direction="horizontal">
		<Panel defaultSize={25} minSize={20}>
Expand DownExpand Up		@@ -405,9 +407,3 @@ function selectAgent(task: Task) {

		return agents.at(0);
		}

		function selectChatApp(task: Task) {
		return getTaskApps(task).find(
		(app) => app.id === task.workspace.latest_build.ai_task_sidebar_app_id,
		);
		}