Rewrite the no-account copy, and point Claude at its integrations docs

The four sentences now put every sign-in route and the reload into one "or"
list, ranked as the buttons rank them, and give the docs their own trailing
sentence. Claude's docs URL moves to the third-party integrations page, which
is what "other ways to set up Claude" actually means: Console, Bedrock, Vertex,
Foundry, Teams and Enterprise. "Set up" is the verb, two words, as the rest of
the string already had it.

Both agents' URL constants still described the workbench as labelling a button.
It has been a link since docs stopped being an action.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Author: TylerLeonhardt

src/vs/platform/agentHost/node/claude/claudeAgent.ts

@@ -68,8 +68,8 @@ import { IAgentHostOTelService } from '../../common/otel/agentHostOTelService.js
 
 const USER_AGENT_PREFIX = 'vscode_claude_code';
 
-/** Where a user goes to establish Claude credentials; the workbench labels the button. */
-const CLAUDE_SETUP_DOCS_URL = 'https://docs.claude.com/en/docs/claude-code/setup';
+/** Where a user goes to establish Claude credentials; the workbench labels the link. */
+const CLAUDE_SETUP_DOCS_URL = 'https://code.claude.com/docs/en/third-party-integrations';
 
 /**
  * Returns true if `m` is a Claude-family model that should be advertised

src/vs/platform/agentHost/node/codex/codexAgent.ts

@@ -175,7 +175,7 @@ const CODEX_THINKING_LEVEL_KEY = 'thinkingLevel';
  */
 const USER_AGENT_PREFIX = 'vscode_codex';
 
-/** Where a user finishes setting Codex up outside the app; the workbench labels the button. */
+/** Where a user finishes setting Codex up outside the app; the workbench labels the link. */
 const CODEX_SETUP_DOCS_URL = 'https://learn.chatgpt.com/codex/auth';
 
 /**

src/vs/platform/agentHost/test/node/claudeAgent.test.ts

@@ -6092,7 +6092,7 @@ suite('ClaudeAgent — agent SDK setup channel', () => {
 		assert.deepStrictEqual(readSetup(ctx), {
 			agent: 'claude',
 			download: 'ready',
-			setupDocsUrl: 'https://docs.claude.com/en/docs/claude-code/setup',
+			setupDocsUrl: 'https://code.claude.com/docs/en/third-party-integrations',
 			// No in-app sign-in: every Claude credential is established outside the
 			// app, so the banner can only point at the docs.
 			signInProviderName: undefined,

src/vs/workbench/contrib/chat/browser/agentSessions/agentHost/agentHostSdkSetupNotification.ts

@@ -93,10 +93,9 @@ function setupMarkdown(value: string): MarkdownString {
 /**
  * The "no account" second line: one whole sentence per combination of routes,
  * never assembled from localized fragments, because clause order is not stable
- * across languages. The GitHub clause is unconditional and leads: it is the
- * primary button, and reaching models through our Copilot proxy is workbench
- * knowledge rather than something an agent could declare. Reload and docs trail
- * the sign-in routes, ranked as the copy's only links rather than buttons.
+ * across languages. The routes share one "or" list, ranked as the buttons rank
+ * them and led by the unconditional GitHub clause: reaching models through our
+ * Copilot proxy is workbench knowledge, not something an agent declares.
  */
 function noAccountDescription(setup: IAgentSdkSetupInfo, displayName: string): IMarkdownString {
 	// Both nouns are the host's, and this string is trusted for two commands, so
@@ -110,15 +109,15 @@ function noAccountDescription(setup: IAgentSdkSetupInfo, displayName: string): I
 	const reload = createCommandUri(AGENT_SDK_SETUP_RELOAD_COMMAND_ID, setup.agent).toString();
 	const docs = setup.setupDocsUrl ? createCommandUri(AGENT_SDK_SETUP_OPEN_DOCS_COMMAND_ID, setup.agent).toString() : undefined;
 	if (provider && docs) {
-		return setupMarkdown(localize('agentHost.sdkSetup.noAccountDescription.all', "Sign in to GitHub to use GitHub Copilot models, or sign in to {2} to use your {2} subscription. If you already set up {0} elsewhere, [reload {0} configuration]({1}). [Learn more]({3}) about other ways to set up {0}.", name, reload, provider, docs));
+		return setupMarkdown(localize('agentHost.sdkSetup.noAccountDescription.all', "Sign in to GitHub to use GitHub Copilot models, sign in to {2} to use your {2} subscription, or [reload the configuration]({1}) if you have set up {0} elsewhere. For other ways to set up {0}, [learn more]({3}) on their docs.", name, reload, provider, docs));
 	}
 	if (provider) {
-		return setupMarkdown(localize('agentHost.sdkSetup.noAccountDescription.signIn', "Sign in to GitHub to use GitHub Copilot models, or sign in to {2} to use your {2} subscription. If you already set up {0} elsewhere, [reload {0} configuration]({1}).", name, reload, provider));
+		return setupMarkdown(localize('agentHost.sdkSetup.noAccountDescription.signIn', "Sign in to GitHub to use GitHub Copilot models, sign in to {2} to use your {2} subscription, or [reload the configuration]({1}) if you have set up {0} elsewhere.", name, reload, provider));
 	}
 	if (docs) {
-		return setupMarkdown(localize('agentHost.sdkSetup.noAccountDescription.docs', "Sign in to GitHub to use GitHub Copilot models. If you already set up {0} elsewhere, [reload {0} configuration]({1}). [Learn more]({2}) about other ways to set up {0}.", name, reload, docs));
+		return setupMarkdown(localize('agentHost.sdkSetup.noAccountDescription.docs', "Sign in to GitHub to use GitHub Copilot models or [reload the configuration]({1}) if you have set up {0} elsewhere. For other ways to set up {0}, [learn more]({2}) on their docs.", name, reload, docs));
 	}
-	return setupMarkdown(localize('agentHost.sdkSetup.noAccountDescription', "Sign in to GitHub to use GitHub Copilot models. If you already set up {0} elsewhere, [reload {0} configuration]({1}).", name, reload));
+	return setupMarkdown(localize('agentHost.sdkSetup.noAccountDescription', "Sign in to GitHub to use GitHub Copilot models or [reload the configuration]({1}) if you have set up {0} elsewhere.", name, reload));
 }
 
 /**

src/vs/workbench/contrib/chat/test/browser/agentSessions/agentHostSdkSetupNotification.test.ts

@@ -128,21 +128,21 @@ suite('Agent SDK setup banner', () => {
 			const gitHub = 'Sign in to GitHub to use GitHub Copilot models';
 			// Unconditional: setup finished in a terminal has no completion signal, so
 			// every agent needs the "look again" route whatever else it declares.
-			const reload = `If you already set up Claude elsewhere, [reload Claude configuration](command:${AGENT_SDK_SETUP_RELOAD_COMMAND_ID}?%255B%2522claude%2522%255D).`;
+			const reload = `[reload the configuration](command:${AGENT_SDK_SETUP_RELOAD_COMMAND_ID}?%255B%2522claude%2522%255D) if you have set up Claude elsewhere.`;
 			// The agent id, like every button carries — the command resolves the URL
 			// from the agent's own declaration rather than trusting the banner's copy.
-			const docs = `[Learn more](command:${AGENT_SDK_SETUP_OPEN_DOCS_COMMAND_ID}?%255B%2522claude%2522%255D) about other ways to set up Claude.`;
+			const docs = `For other ways to set up Claude, [learn more](command:${AGENT_SDK_SETUP_OPEN_DOCS_COMMAND_ID}?%255B%2522claude%2522%255D) on their docs.`;
 
 			assert.deepStrictEqual({
 				gitHubOnly: noAccount({}),
 				docs: noAccount({ setupDocsUrl: 'https://example.test/claude' }),
 				signIn: noAccount({ signInProviderName: 'ChatGPT' }),
 				both: noAccount({ setupDocsUrl: 'https://example.test/claude', signInProviderName: 'ChatGPT' }),
 			}, {
-				gitHubOnly: `${gitHub}. ${reload}`,
-				docs: `${gitHub}. ${reload} ${docs}`,
-				signIn: `${gitHub}, or sign in to ChatGPT to use your ChatGPT subscription. ${reload}`,
-				both: `${gitHub}, or sign in to ChatGPT to use your ChatGPT subscription. ${reload} ${docs}`,
+				gitHubOnly: `${gitHub} or ${reload}`,
+				docs: `${gitHub} or ${reload} ${docs}`,
+				signIn: `${gitHub}, sign in to ChatGPT to use your ChatGPT subscription, or ${reload}`,
+				both: `${gitHub}, sign in to ChatGPT to use your ChatGPT subscription, or ${reload} ${docs}`,
 			});
 		});
 
@@ -158,7 +158,7 @@ suite('Agent SDK setup banner', () => {
 			const name = 'Claude \\[x\\]\\(command:evil\\)';
 
 			assert.strictEqual(typeof description === 'string' ? description : description?.value,
-				`Sign in to GitHub to use GitHub Copilot models, or sign in to Chat\\[G\\]PT to use your Chat\\[G\\]PT subscription. If you already set up ${name} elsewhere, [reload ${name} configuration](command:${AGENT_SDK_SETUP_RELOAD_COMMAND_ID}?%255B%2522claude%2522%255D). [Learn more](command:${AGENT_SDK_SETUP_OPEN_DOCS_COMMAND_ID}?%255B%2522claude%2522%255D) about other ways to set up ${name}.`);
+				`Sign in to GitHub to use GitHub Copilot models, sign in to Chat\\[G\\]PT to use your Chat\\[G\\]PT subscription, or [reload the configuration](command:${AGENT_SDK_SETUP_RELOAD_COMMAND_ID}?%255B%2522claude%2522%255D) if you have set up ${name} elsewhere. For other ways to set up ${name}, [learn more](command:${AGENT_SDK_SETUP_OPEN_DOCS_COMMAND_ID}?%255B%2522claude%2522%255D) on their docs.`);
 		});
 
 		test('the copy is trusted for its own two commands alone, so its links render and reach nothing else', () => {