feat(web): Add provider leads with first-class providers (#25753)

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>

Author: juliusmarminge

apps/web/src/components/settings/AcpRegistrySearchStep.browser.test.tsx

@@ -116,3 +116,12 @@ it("rechecks the grant before preparing an agent", async () => {
   expect(state.prepare).not.toHaveBeenCalled();
   expect(onPrepared).not.toHaveBeenCalled();
 });
+
+it("leaves out Registry agents that T3 Code supports as their own providers", async () => {
+  setup();
+  state.result = { agents: [gemini, { ...gemini, id: "codex-acp", name: "Codex" }] };
+  await using _screen = await render(<Step />);
+
+  expect(page.getByRole("button", { name: "Add Gemini CLI" }).query()).not.toBeNull();
+  expect(page.getByRole("button", { name: "Add Codex" }).query()).toBeNull();
+});

apps/web/src/components/settings/AcpRegistrySearchStep.tsx

@@ -25,6 +25,7 @@ import { Tooltip, TooltipPopup, TooltipTrigger } from "../ui/tooltip";
 import { isConfiguredAcpRegistryAgent } from "./AddProviderInstanceDialog.logic";
 import { ProviderDriverKind } from "@t3tools/contracts";
 import { ProviderInstanceIcon } from "../chat/ProviderInstanceIcon";
+import { providerClients } from "./providerDriverMeta";
 
 function errorMessage(error: unknown): string {
   return error instanceof Error && error.message.trim()
@@ -33,6 +34,9 @@ function errorMessage(error: unknown): string {
 }
 
 const ACP_REGISTRY_SEARCH_DEBOUNCE_MS = 300;
+const FIRST_CLASS_REGISTRY_AGENT_IDS = new Set(
+  providerClients.definitions.flatMap((definition) => definition.acpRegistryAgentIds ?? []),
+);
 
 function applyAcpRegistryPrepareResult(
   agent: AcpRegistrySearchAgent,
@@ -144,7 +148,8 @@ export function AcpRegistrySearchStep({
     void registry.prepare(agent, onPrepared);
   };
 
-  const results = search.data?.agents ?? null;
+  const results =
+    search.data?.agents.filter((agent) => !FIRST_CLASS_REGISTRY_AGENT_IDS.has(agent.id)) ?? null;
   const isInitialSearch = search.isPending && results === null;
   const isRefreshing = search.isPending && results !== null;
   const resultCount = results?.length ?? 0;

apps/web/src/components/settings/AddProviderInstanceDialog.environment.browser.test.tsx

@@ -114,6 +114,10 @@ function button(label: string) {
   return found!;
 }
 const click = (label: string) => act(async () => button(label).click());
+const radio = (label: string) =>
+  [...document.querySelectorAll<HTMLElement>('[role="radio"]')].find(
+    (element) => element.textContent?.trim() === label,
+  );
 const input = (selector: string) => {
   const found = document.querySelector<HTMLInputElement>(selector);
   expect(found, selector).not.toBeNull();
@@ -166,11 +170,32 @@ describe("AddProviderInstanceDialog environment routing", () => {
     });
   });
 
+  it("keeps fx and the ACP Registry under Show more", async () => {
+    setup();
+    await using _screen = await render(dialog());
+    expect(radio("Codex")).toBeDefined();
+    expect(radio("fx")).toBeUndefined();
+    expect(document.body.textContent).not.toContain("Use Kilo Code");
+
+    await click("Show more");
+    button("Use Kilo Code");
+    await act(async () => radio("fx")!.click());
+    await click("Next");
+    await click("Next");
+    await click("Add instance");
+    expect(settingsHooks.mutate).toHaveBeenCalledWith({
+      operation: "create",
+      instanceId: "fx",
+      instance: { driver: "fx", enabled: true, displayName: "fx" },
+    });
+  });
+
   it("awaits an atomic create before closing and retains prepared ACP metadata", async () => {
     setup();
     const mutation = deferred<{ readonly _tag: "Success"; readonly value: unknown }>();
     settingsHooks.mutate.mockReturnValueOnce(mutation.promise);
     await using _screen = await render(dialog());
+    await click("Show more");
     await click("Use Kilo Code");
 
     expect(document.body.textContent).toContain("v4.2.0 · binary");
@@ -202,6 +227,7 @@ describe("AddProviderInstanceDialog environment routing", () => {
   it("creates a local command in the selected environment without a sign-in step", async () => {
     setup();
     await using _screen = await render(dialog());
+    await click("Show more");
     await click("Use local command");
     await click("Next");
     expect(document.body.textContent).toContain("Executable is required.");

apps/web/src/components/settings/AddProviderInstanceDialog.tsx

@@ -17,11 +17,13 @@ export const providerClients = makeProviderClientRegistry([
   {
     driverKind: ProviderDriverKind.make("codex"),
     label: "Codex",
+    acpRegistryAgentIds: ["codex-acp"],
     settingsSchema: CodexSettings,
   },
   {
     driverKind: ProviderDriverKind.make("claudeAgent"),
     label: "Claude",
+    acpRegistryAgentIds: ["claude-acp"],
     settingsSchema: ClaudeSettings,
   },
   cursorClient,
@@ -31,6 +33,7 @@ export const providerClients = makeProviderClientRegistry([
   {
     driverKind: ProviderDriverKind.make("antigravity"),
     label: "Antigravity",
+    acpRegistryAgentIds: ["antigravity-acp"],
     settingsSchema: AntigravitySettings,
   },
   museClient,

apps/web/src/components/settings/providerDriverMeta.ts

@@ -5,13 +5,14 @@ T3 Code can run installed ACP commands or coding agents from the official
 own models, tools, and sign-in, while T3 Code provides projects, threads, checkpoints, and task
 delegation.
 
-Google Antigravity is available through its official `antigravity-acp` Registry entry. Add it like
-any other Registry agent; it does not need an Antigravity-specific T3 Code provider.
+Agents that T3 Code supports as their own providers, such as Codex, Claude, Cursor, Grok, OpenCode,
+Pi, and [Antigravity](./providers-antigravity.md), don't appear in the Registry list. Enable them in
+**Settings → Providers** instead.
 
 T3 Code prefers the current ACP v2 preview protocol and uses its richer messages, usage, plans,
 configuration, compaction, and agent-terminal updates when the agent supports them. It also
-negotiates ACP v1 for Registry agents that have not migrated yet, so agents such as Pi continue to
-work through the same generic integration.
+negotiates ACP v1 for Registry agents that have not migrated yet, so they continue to work through
+the same generic integration.
 
 Claude Agent uses the ACP v2 preview by default for Registry and local `claude-agent-acp`
 commands. If you encounter preview issues, set `CLAUDE_AGENT_ACP_EXPERIMENTAL_V2=0` in the
@@ -20,16 +21,16 @@ provider instance's environment overrides to use ACP v1.
 ## Add an agent
 
 1. Open **Settings → Providers**.
-2. Select **Add provider** and search the ACP Registry.
-3. Search for the agent and select **Add** on its result.
+2. Select **Add provider**, then **Show more**.
+3. Search the ACP Registry for the agent and select **Add** on its result.
 4. Confirm the name and instance ID, then complete the agent's sign-in step.
 
 Search only shows agents that can run on the connected server. Registry agents are third-party
 code; review an agent's source and license before adding it.
 
 ## Add a local command
 
-In **Settings → Providers → Add provider**, select **Local ACP command**. Enter the executable
+In **Settings → Providers → Add provider**, select **Show more**, then **Local ACP command**. Enter the executable
 name or path on the selected environment, add one literal argument per row, and choose a display
 name. For DeepSeek Harness, use `dsh` with arguments `--profile` and `acp`.
 

docs/user/providers-acp.md

@@ -1,6 +1,6 @@
 # Muse Code
 
-Muse Code is a beta integration and is disabled by default. Install
+Muse Code is disabled by default. Install
 [Muse Code](https://dev.meta.ai/docs/muse-code) on the machine hosting your
 environment, then run `muse login` as the account that runs T3 Code.
 

docs/user/providers-muse.md

@@ -1305,8 +1305,8 @@ export function createServerEnvironmentAtoms<R, E>(
     searchAcpRegistry: createEnvironmentRpcQueryAtomFamily(runtime, {
       label: "environment-data:server:acp-registry:search",
       tag: WS_METHODS.serverSearchAcpRegistry,
-      staleTimeMs: 0,
-      idleTtlMs: 0,
+      // Reopening Add provider reuses recent results; submitting the same search refreshes it.
+      staleTimeMs: 5 * 60_000,
     }),
     configProjection,
     welcome,

packages/client-runtime/src/state/server.ts

@@ -40,6 +40,10 @@ export interface ProviderClientDefinition {
   readonly environmentFields?: ReadonlyArray<ProviderEnvironmentField>;
   readonly hasDefaultInstance?: boolean;
   readonly badgeLabel?: string;
+  /** `false` lists the provider under Add provider's "Show more" instead of up front. */
+  readonly featured?: boolean;
+  /** ACP Registry entries for this same agent, left out of Add provider's Registry list. */
+  readonly acpRegistryAgentIds?: ReadonlyArray<string>;
 }
 
 export function defineProviderClient<const Definition extends ProviderClientDefinition>(

packages/provider-core/src/client.ts

@@ -6,6 +6,7 @@ import { CursorSettings } from "./settings.ts";
 export const cursorClient = defineProviderClient({
   driverKind: ProviderDriverKind.make("cursor"),
   label: "Cursor",
+  acpRegistryAgentIds: ["cursor"],
   settingsSchema: CursorSettings,
   environmentFields: [
     {