refactor: remove /openChat route and chatPanelProvider from URI handler

The /open route with an optional agentId param is the only deep link
path for opening the chat panel. The standalone /openChat route and
the chatPanelProvider dependency in the URI handler are no longer
needed — the chat panel is opened from extension.ts after the
remote-authority reload picks up the persisted agentId.

Author: ThomasK33

src/extension.ts

@@ -235,12 +235,7 @@ export async function activate(ctx: vscode.ExtensionContext): Promise<void> {
 	);
 
 	ctx.subscriptions.push(
-		registerUriHandler(
-			serviceContainer,
-			deploymentManager,
-			commands,
-			chatPanelProvider,
-		),
+		registerUriHandler(serviceContainer, deploymentManager, commands),
 		vscode.commands.registerCommand(
 			"coder.login",
 			commands.login.bind(commands),

src/uri/uriHandler.ts

@@ -8,22 +8,19 @@ import { CALLBACK_PATH } from "../oauth/utils";
 import { maybeAskUrl } from "../promptUtils";
 import { toSafeHost } from "../util";
 import { vscodeProposed } from "../vscodeProposed";
-import { type ChatPanelProvider } from "../webviews/chat/chatPanelProvider";
 
 interface UriRouteContext {
 	params: URLSearchParams;
 	serviceContainer: ServiceContainer;
 	deploymentManager: DeploymentManager;
 	commands: Commands;
-	chatPanelProvider: ChatPanelProvider;
 }
 
 type UriRouteHandler = (ctx: UriRouteContext) => Promise<void>;
 
 const routes: Readonly<Record<string, UriRouteHandler>> = {
 	"/open": handleOpen,
 	"/openDevContainer": handleOpenDevContainer,
-	"/openChat": handleOpenChat,
 	[CALLBACK_PATH]: handleOAuthCallback,
 };
 
@@ -34,20 +31,13 @@ export function registerUriHandler(
 	serviceContainer: ServiceContainer,
 	deploymentManager: DeploymentManager,
 	commands: Commands,
-	chatPanelProvider: ChatPanelProvider,
 ): vscode.Disposable {
 	const output = serviceContainer.getLogger();
 
 	return vscode.window.registerUriHandler({
 		handleUri: async (uri) => {
 			try {
-				await routeUri(
-					uri,
-					serviceContainer,
-					deploymentManager,
-					commands,
-					chatPanelProvider,
-				);
+				await routeUri(uri, serviceContainer, deploymentManager, commands);
 			} catch (error) {
 				const message = errToStr(error, "No error message was provided");
 				output.warn(`Failed to handle URI ${uri.toString()}: ${message}`);
@@ -66,7 +56,6 @@ async function routeUri(
 	serviceContainer: ServiceContainer,
 	deploymentManager: DeploymentManager,
 	commands: Commands,
-	chatPanelProvider: ChatPanelProvider,
 ): Promise<void> {
 	const handler = routes[uri.path];
 	if (!handler) {
@@ -78,7 +67,6 @@ async function routeUri(
 		serviceContainer,
 		deploymentManager,
 		commands,
-		chatPanelProvider,
 	});
 }
 
@@ -201,17 +189,6 @@ async function setupDeployment(
 	});
 }
 
-async function handleOpenChat(ctx: UriRouteContext): Promise<void> {
-	const { params, serviceContainer, deploymentManager, chatPanelProvider } =
-		ctx;
-
-	const agentId = getRequiredParam(params, "agentId");
-
-	await setupDeployment(params, serviceContainer, deploymentManager);
-
-	chatPanelProvider.openChat(agentId);
-}
-
 async function handleOAuthCallback(ctx: UriRouteContext): Promise<void> {
 	const { params, serviceContainer } = ctx;
 	const logger = serviceContainer.getLogger();

test/unit/uri/uriHandler.test.ts

@@ -6,7 +6,6 @@ import { SecretsManager } from "@/core/secretsManager";
 import { CALLBACK_PATH } from "@/oauth/utils";
 import { maybeAskUrl } from "@/promptUtils";
 import { registerUriHandler } from "@/uri/uriHandler";
-import { type ChatPanelProvider } from "@/webviews/chat/chatPanelProvider";
 
 import {
 	createMockLogger,
@@ -99,13 +98,10 @@ function createTestContext() {
 		.mocked(vscode.window.showErrorMessage)
 		.mockResolvedValue(undefined);
 
-	const chatPanelProvider = {} as unknown as ChatPanelProvider;
-
 	registerUriHandler(
 		container,
 		deploymentManager as unknown as DeploymentManager,
 		commands as unknown as Commands,
-		chatPanelProvider,
 	);
 
 	return {