Skip to content

Commit 8e503e5

Browse files
authored
mcp: fix user interaction notification for variable resolution (microsoft#269624)
The UserInteractionRequiredError should instead be thrown to be handled by the autostart logic.
1 parent bd50258 commit 8e503e5

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -508,6 +508,10 @@ export class McpRegistry extends Disposable implements IMcpRegistry {
508508
launch = await this._instantiationService.invokeFunction(accessor => accessor.get(IMcpDevModeDebugging).transform(definition, launch!));
509509
}
510510
} catch (e) {
511+
if (e instanceof UserInteractionRequiredError) {
512+
throw e;
513+
}
514+
511515
this._notificationService.notify({
512516
severity: Severity.Error,
513517
message: localize('mcp.launchError', 'Error starting {0}: {1}', definition.label, String(e)),

0 commit comments

Comments
 (0)