Paul (pwang347) · Oct 13, 2026 · Oct 12, 2026 · Oct 12, 2026 · Oct 13, 2026
diff --git a/src/vs/workbench/contrib/chat/browser/aiCustomization/connectorPresentation.ts b/src/vs/workbench/contrib/chat/browser/aiCustomization/connectorPresentation.ts
@@ -101,6 +101,8 @@ export function getConnectorRowPresentation(connector: ICopilotConnector): IConn
 			return {
 				statusLabel: getConnectorStatusLabel(connector.connectionStatus),
 				statusIcon: 'pending',
+				action: 'retry',
+				actionLabel: localize('connectors.action.tryAgain', "Try Again"),
 			};
 		case 'error':
 			return {

diff --git a/src/vs/workbench/contrib/chat/test/browser/aiCustomization/connectorPresentation.test.ts b/src/vs/workbench/contrib/chat/test/browser/aiCustomization/connectorPresentation.test.ts
@@ -50,7 +50,7 @@ suite('Connector presentation', () => {
 			{ statusLabel: 'Connection status not checked', statusIcon: 'info', action: 'check', actionLabel: 'Check Connection' },
 			{ statusLabel: 'Connected', statusIcon: 'connected', action: 'more' },
 			{ statusLabel: 'Not connected', action: 'connect', actionLabel: 'Connect' },
-			{ statusLabel: 'Connection pending', statusIcon: 'pending' },
+			{ statusLabel: 'Connection pending', statusIcon: 'pending', action: 'retry', actionLabel: 'Try Again' },
 			{ statusLabel: 'Sign in required', statusIcon: 'attention', action: 'sign_in', actionLabel: 'Sign in' },
 			{ statusLabel: 'Review required', statusIcon: 'attention', action: 'review', actionLabel: 'Review' },
 			{ statusLabel: 'Connection failed', statusIcon: 'error', action: 'retry', actionLabel: 'Try Again' },
@@ -67,16 +67,23 @@ suite('Connector presentation', () => {
 		const detail = disposables.add(new EmbeddedConnectorDetail(
 			container, () => { }, service, new class extends mock<INotificationService>() { }(), new class extends mock<IOpenerService>() { }(),
 		));
-		const states = ['sign_in_required', 'review_required', 'retryable_error', 'unavailable'] as const;
+		const states = [
+			connector('pending'),
+			connector('error', 'sign_in_required'),
+			connector('error', 'review_required'),
+			connector('error', 'retryable_error'),
+			connector('error', 'unavailable'),
+		];
 		const presentations = states.map(state => {
-			detail.setInput(connector('error', state));
+			detail.setInput(state);
 			return {
 				status: container.querySelector('.connector-detail-facts .embedded-detail-fact-value')?.textContent,
 				action: container.querySelector('.embedded-detail-title-actions .monaco-button')?.textContent,
 			};
 		});
 
 		assert.deepStrictEqual(presentations, [
+			{ status: 'Connection pending', action: 'Try Again' },
 			{ status: 'Sign in required', action: 'Sign in' },
 			{ status: 'Review required', action: 'Review' },
 			{ status: 'Connection failed', action: 'Try Again' },
@@ -108,15 +115,13 @@ suite('Connector presentation', () => {
 		assert.deepStrictEqual({ started: actionToken !== undefined, cancelled: actionToken?.isCancellationRequested }, { started: true, cancelled: true });
 	});
 
-	test('an earlier refresh cannot replace a newly selected connector', async () => {
+	test('an earlier retry cannot replace a newly selected connector', async () => {
 		const container = DOM.append(document.body, DOM.$('.connector-detail-test'));
 		disposables.add(toDisposable(() => container.remove()));
 		const result = new DeferredPromise<readonly ICopilotConnector[]>();
 		const service = new class extends mock<ICopilotConnectorsService>() {
 			override readonly onDidChange = Event.None;
-			override async refresh(): Promise<readonly ICopilotConnector[]> {
-				return [];
-			}
+			override async connect(): Promise<void> { }
 			override getConnectors(): Promise<readonly ICopilotConnector[]> {
 				return result.p;
 			}

diff --git a/.../contrib/chat/test/browser/aiCustomization/customizationMarketplaceInstallService.test.ts b/.../contrib/chat/test/browser/aiCustomization/customizationMarketplaceInstallService.test.ts
@@ -2419,28 +2419,52 @@ suite('CustomizationMarketplaceInstallService', () => {
 			});
 		});
 
-		test('pending and unavailable connectors cannot start duplicate or unactionable connections', async () => {
+		test('a connector left pending after authorization can be retried', async () => {
 			const fixture = await createFixture();
 			const candidate = connectorResource();
-			const states: Array<ReturnType<typeof fixture.service.getInstallState>> = [];
-			for (const scenario of [
-				{ status: 'pending' as const, detail: undefined },
-				{ status: 'error' as const, detail: 'unavailable' as const },
-			]) {
-				fixture.connectorsService.statusOverride = scenario.status;
-				fixture.connectorsService.statusDetailOverride = scenario.detail;
-				const state = fixture.service.getInstallState(candidate);
-				states.push(state);
-				await assert.rejects(fixture.service.install(candidate), /cannot be connected/);
-			}
+			fixture.connectorsService.statusOverride = 'pending';
+			const before = fixture.service.getInstallState(candidate);
+			const retry = new DeferredPromise<void>();
+			fixture.connectorsService.onConnect = async name => {
+				await retry.p;
+				fixture.connectorsService.statusOverride = undefined;
+				fixture.connectedConnectors.add(name);
+				fixture.connectorChanges.fire();
+			};
+
+			const first = fixture.service.install(candidate);
+			const second = fixture.service.install(candidate);
+			const during = fixture.service.getInstallState(candidate);
+			retry.complete();
+			await Promise.all([first, second]);
+
 			assert.deepStrictEqual({
-				states,
+				before,
+				during,
+				connects: fixture.connectorsService.connectCalls,
+				installed: fixture.service.getInstallState(candidate).kind,
+			}, {
+				before: { kind: 'available' },
+				during: { kind: 'installing' },
+				connects: ['mail'],
+				installed: 'installed',
+			});
+		});
+
+		test('unavailable connectors cannot start an unactionable connection', async () => {
+			const fixture = await createFixture();
+			const candidate = connectorResource();
+			fixture.connectorsService.statusOverride = 'error';
+			fixture.connectorsService.statusDetailOverride = 'unavailable';
+			const state = fixture.service.getInstallState(candidate);
+
+			await assert.rejects(fixture.service.install(candidate), /cannot be connected/);
+
+			assert.deepStrictEqual({
+				state,
 				connects: fixture.connectorsService.connectCalls,
 			}, {
-				states: [
-					{ kind: 'unavailable', message: 'This connector cannot be connected while its status is \'Connection pending\'. Refresh and try again.' },
-					{ kind: 'unavailable', message: 'This connector cannot be connected while its status is \'Currently unavailable\'. Refresh and try again.' },
-				],
+				state: { kind: 'unavailable', message: 'This connector cannot be connected while its status is \'Currently unavailable\'. Refresh and try again.' },
 				connects: [],
 			});
 		});