mcp: fix unhelpful error case and picker opens and closes (microsoft#259914)

Fixes microsoft#259495

Author: connor4312

src/vs/workbench/contrib/mcp/browser/mcpCommands.ts

@@ -494,7 +494,7 @@ export class MCPServerActionRendering extends Disposable implements IWorkbenchCo
 					} else if (state === DisplayedState.Error) {
 						const server = findLast(servers, isServer);
 						if (server) {
-							server.showOutput();
+							await server.showOutput(true);
 							commandService.executeCommand(McpCommandIds.ServerOptions, server.definition.id);
 						}
 					} else {

src/vs/workbench/contrib/mcp/common/mcpServer.ts

@@ -450,9 +450,9 @@ export class McpServer extends Disposable implements IMcpServer {
 		return this._fullDefinitions;
 	}
 
-	public showOutput(): void {
+	public showOutput(preserveFocus?: boolean) {
 		this._loggerService.setVisibility(this._loggerId, true);
-		this._outputService.showChannel(this._loggerId);
+		return this._outputService.showChannel(this._loggerId, preserveFocus);
 	}
 
 	public resources(token?: CancellationToken): AsyncIterable<IMcpResource[]> {

src/vs/workbench/contrib/mcp/common/mcpTypes.ts

@@ -288,7 +288,7 @@ export interface IMcpServer extends IDisposable {
 	 */
 	readDefinitions(): IObservable<{ server: McpServerDefinition | undefined; collection: McpCollectionDefinition | undefined }>;
 
-	showOutput(): void;
+	showOutput(preserveFocus?: boolean): Promise<void>;
 	/**
 	 * Starts the server and returns its resulting state. One of:
 	 * - Running, if all good