From 069343e0bee7180f3419e42d801c985721af9e3e Mon Sep 17 00:00:00 2001 From: morgmart <98432065+morgmart@users.noreply.github.com> Date: Thu, 23 Jul 2026 17:08:48 -0700 Subject: [PATCH 1/3] Restore Goose and Buzz Agent to onboarding harness selection MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Reverts the launch-only restriction from #2233 by adding goose and buzz-agent back to ONBOARDING_RUNTIME_ORDER — the single centralized onboarding visibility allowlist. Setup cards, readiness handoff, and the defaults harness picker all follow from that one list. Also generalizes the empty-state copy and updates the unit + E2E specs that pinned the hidden behavior. --- .../src/features/onboarding/ui/SetupStep.tsx | 4 ++-- .../ui/onboardingRuntimeSelection.test.mjs | 13 +++++++------ .../ui/onboardingRuntimeSelection.ts | 7 ++++++- .../e2e/onboarding-agent-defaults.spec.ts | 18 +++++++----------- 4 files changed, 22 insertions(+), 20 deletions(-) diff --git a/desktop/src/features/onboarding/ui/SetupStep.tsx b/desktop/src/features/onboarding/ui/SetupStep.tsx index 96e7322d14..9be69d1e10 100644 --- a/desktop/src/features/onboarding/ui/SetupStep.tsx +++ b/desktop/src/features/onboarding/ui/SetupStep.tsx @@ -624,8 +624,8 @@ function RuntimeProvidersSection({ className="max-w-[560px] rounded-2xl bg-white/70 px-6 py-6 text-sm text-muted-foreground" data-testid="onboarding-acp-empty" > - No supported agent harnesses were detected yet. Install Claude Code - or Codex, then check again. + No supported agent harnesses were detected yet. Install a supported + harness, then check again.

)} diff --git a/desktop/src/features/onboarding/ui/onboardingRuntimeSelection.test.mjs b/desktop/src/features/onboarding/ui/onboardingRuntimeSelection.test.mjs index b10aa19154..221702ebb2 100644 --- a/desktop/src/features/onboarding/ui/onboardingRuntimeSelection.test.mjs +++ b/desktop/src/features/onboarding/ui/onboardingRuntimeSelection.test.mjs @@ -12,11 +12,11 @@ function runtime(id, availability, status) { return { id, availability, authStatus: { status } }; } -test("only Claude Code and Codex are visible in onboarding", () => { +test("all bundled harnesses are visible in onboarding", () => { assert.equal(runtimeIsVisibleInOnboarding("claude"), true); assert.equal(runtimeIsVisibleInOnboarding("codex"), true); - assert.equal(runtimeIsVisibleInOnboarding("goose"), false); - assert.equal(runtimeIsVisibleInOnboarding("buzz-agent"), false); + assert.equal(runtimeIsVisibleInOnboarding("goose"), true); + assert.equal(runtimeIsVisibleInOnboarding("buzz-agent"), true); assert.equal(runtimeIsVisibleInOnboarding("custom"), false); }); @@ -30,7 +30,7 @@ test("visible onboarding runtimes use the product order", () => { assert.deepEqual( getVisibleOnboardingRuntimes(runtimes).map(({ id }) => id), - ["claude", "codex"], + ["claude", "codex", "goose", "buzz-agent"], ); }); @@ -55,16 +55,17 @@ test("readiness requires an available and authenticated runtime", () => { ); }); -test("ready onboarding runtimes exclude hidden ready harnesses", () => { +test("ready onboarding runtimes exclude unknown and non-ready harnesses", () => { const runtimes = [ runtime("goose", "available", "not_applicable"), runtime("codex", "available", "logged_out"), runtime("buzz-agent", "available", "not_applicable"), runtime("claude", "available", "logged_in"), + runtime("custom", "available", "not_applicable"), ]; assert.deepEqual( getReadyOnboardingRuntimes(runtimes).map(({ id }) => id), - ["claude"], + ["claude", "goose", "buzz-agent"], ); }); diff --git a/desktop/src/features/onboarding/ui/onboardingRuntimeSelection.ts b/desktop/src/features/onboarding/ui/onboardingRuntimeSelection.ts index 51339e2afe..cd491dfcc5 100644 --- a/desktop/src/features/onboarding/ui/onboardingRuntimeSelection.ts +++ b/desktop/src/features/onboarding/ui/onboardingRuntimeSelection.ts @@ -1,6 +1,11 @@ import type { AcpRuntimeCatalogEntry } from "@/shared/api/types"; -export const ONBOARDING_RUNTIME_ORDER = ["claude", "codex"]; +export const ONBOARDING_RUNTIME_ORDER = [ + "claude", + "codex", + "goose", + "buzz-agent", +]; const VISIBLE_ONBOARDING_RUNTIME_IDS = new Set( ONBOARDING_RUNTIME_ORDER, diff --git a/desktop/tests/e2e/onboarding-agent-defaults.spec.ts b/desktop/tests/e2e/onboarding-agent-defaults.spec.ts index f8e6322ed3..d43fa29eb7 100644 --- a/desktop/tests/e2e/onboarding-agent-defaults.spec.ts +++ b/desktop/tests/e2e/onboarding-agent-defaults.spec.ts @@ -57,9 +57,7 @@ async function readSavedRuntime(page: Parameters[0]) { }); } -test("setup shows only Claude Code and Codex as detected harnesses", async ({ - page, -}) => { +test("setup shows all bundled harnesses as detected", async ({ page }) => { await installMockBridge( page, { @@ -77,10 +75,8 @@ test("setup shows only Claude Code and Codex as detected harnesses", async ({ await expect(page.getByTestId("onboarding-runtime-claude")).toBeVisible(); await expect(page.getByTestId("onboarding-runtime-codex")).toBeVisible(); - await expect(page.getByTestId("onboarding-runtime-goose")).toHaveCount(0); - await expect(page.getByTestId("onboarding-runtime-buzz-agent")).toHaveCount( - 0, - ); + await expect(page.getByTestId("onboarding-runtime-goose")).toBeVisible(); + await expect(page.getByTestId("onboarding-runtime-buzz-agent")).toBeVisible(); await expect(page.getByRole("checkbox")).toHaveCount(0); }); @@ -525,8 +521,8 @@ test("defaults auto-selects the only ready visible harness", async ({ page, { acpRuntimesCatalog: [ - runtime("buzz-agent", "available", { status: "not_applicable" }), - runtime("goose", "available", { status: "not_applicable" }), + runtime("buzz-agent", "not_installed", { status: "not_applicable" }), + runtime("goose", "not_installed", { status: "not_applicable" }), runtime("claude", "available", { status: "logged_in" }), runtime("codex", "available", { status: "logged_out" }), ], @@ -626,10 +622,10 @@ test("defaults requires a choice when multiple visible harnesses are ready", asy ).toBeVisible(); await expect( page.getByTestId("global-agent-default-harness-option-goose"), - ).toHaveCount(0); + ).toBeVisible(); await expect( page.getByTestId("global-agent-default-harness-option-buzz-agent"), - ).toHaveCount(0); + ).toBeVisible(); await page.getByTestId("global-agent-default-harness-option-codex").click(); await expect(harness).toHaveText("Codex"); await expect(page.getByTestId("onboarding-finish")).toBeEnabled(); From a1c687dba78bd8aa6569f08b6aeabfaf366ff68c Mon Sep 17 00:00:00 2001 From: morgmart <98432065+morgmart@users.noreply.github.com> Date: Fri, 24 Jul 2026 09:49:41 -0700 Subject: [PATCH 2/3] Lay onboarding harness cards out as a single row of four With Goose and Buzz Agent restored, the 2x2 grid read as two tiers. A single lg:grid-cols-4 row (falling back to 2-up below lg and 1-up on narrow viewports) presents the four harnesses as peers. --- desktop/src/features/onboarding/ui/SetupStep.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/desktop/src/features/onboarding/ui/SetupStep.tsx b/desktop/src/features/onboarding/ui/SetupStep.tsx index 9be69d1e10..00825ca2f5 100644 --- a/desktop/src/features/onboarding/ui/SetupStep.tsx +++ b/desktop/src/features/onboarding/ui/SetupStep.tsx @@ -603,7 +603,7 @@ function RuntimeProvidersSection({
{orderedItems.length > 0 ? ( -
+
{orderedItems.map((runtime) => ( Date: Sun, 26 Jul 2026 13:21:07 -0700 Subject: [PATCH 3/3] Gate onboarding Finish on actual config validity MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The defaults page rendered provider/model/credential fields for provider-required harnesses (buzz-agent, goose) but Finish only checked that a harness was selected — a fresh user could persist a default that fails at first spawn. Wire AgentConfigFields' existing onValidityChange signal into the completion gate. Baked build env and runtime-file config satisfy the gate, so internal builds and existing Goose users are never blocked. Pinned by two new E2E cases; AGENTS.md rule 7 updated to document the gate. Signed-off-by: morgmart <98432065+morgmart@users.noreply.github.com> --- desktop/src/features/agents/AGENTS.md | 6 +- .../onboarding/ui/DefaultConfigStep.tsx | 15 +++- .../e2e/onboarding-agent-defaults.spec.ts | 86 +++++++++++++++++++ 3 files changed, 104 insertions(+), 3 deletions(-) diff --git a/desktop/src/features/agents/AGENTS.md b/desktop/src/features/agents/AGENTS.md index 2af9ddb98c..06e6c02acb 100644 --- a/desktop/src/features/agents/AGENTS.md +++ b/desktop/src/features/agents/AGENTS.md @@ -67,7 +67,11 @@ with a TypeScript lookup table or an id comparison in a component. 7. **Onboarding setup detects readiness; it does not select defaults.** The setup page derives visible and ready harnesses from the runtime catalog and only offers install or sign-in actions. The following defaults page is the - sole onboarding surface that chooses and persists `preferred_runtime`. + sole onboarding surface that chooses and persists `preferred_runtime`, and + its Finish gate consumes the shared renderer's `onValidityChange` signal — + a harness selection alone does not complete onboarding when the harness + requires provider/model/credential config (e.g. buzz-agent with no + provider). Baked build env and runtime-file config satisfy the gate. `onboarding-agent-defaults.spec.ts` is the acceptance gate for anything touching this flow or the shared renderer. 8. **Omit the Model control only after a confirmed successful empty diff --git a/desktop/src/features/onboarding/ui/DefaultConfigStep.tsx b/desktop/src/features/onboarding/ui/DefaultConfigStep.tsx index 90f8b22d62..50887f08aa 100644 --- a/desktop/src/features/onboarding/ui/DefaultConfigStep.tsx +++ b/desktop/src/features/onboarding/ui/DefaultConfigStep.tsx @@ -68,6 +68,7 @@ function AgentDefaultsSection({ cancel: () => void; } | null>(null); const [isSaving, setIsSaving] = React.useState(false); + const [configIsValid, setConfigIsValid] = React.useState(false); React.useEffect(() => { let unmounted = false; @@ -187,10 +188,19 @@ function AgentDefaultsSection({ ); React.useEffect(() => { onPersistenceStateChange({ - canComplete: selectedRuntimeId.length > 0 && !isSaving, + // configIsValid comes from AgentConfigFields' onValidityChange and + // covers model + provider credentials — a harness selection alone is + // not a working default (e.g. buzz-agent with no provider configured). + canComplete: selectedRuntimeId.length > 0 && configIsValid && !isSaving, flush: flushPersistence, }); - }, [flushPersistence, isSaving, onPersistenceStateChange, selectedRuntimeId]); + }, [ + configIsValid, + flushPersistence, + isSaving, + onPersistenceStateChange, + selectedRuntimeId, + ]); return (
@@ -239,6 +249,7 @@ function AgentDefaultsSection({ }} onCustomModelEditingChange={setIsCustomModelEditing} onIsCustomProviderChange={setIsCustomProvider} + onValidityChange={setConfigIsValid} placeholderClassName="text-foreground/70" runtimeFileConfig={runtimeFileConfig} selectClassName="h-12 rounded-2xl border-foreground/15 bg-white px-4 py-2 text-sm shadow-none hover:bg-white/95" diff --git a/desktop/tests/e2e/onboarding-agent-defaults.spec.ts b/desktop/tests/e2e/onboarding-agent-defaults.spec.ts index d029150fd2..1c3e86f133 100644 --- a/desktop/tests/e2e/onboarding-agent-defaults.spec.ts +++ b/desktop/tests/e2e/onboarding-agent-defaults.spec.ts @@ -832,3 +832,89 @@ test("concurrent installs each keep their own state — one fails, one succeeds" }); expect(hasHorizontalOverflow).toBe(false); }); + +test("Finish stays disabled until a provider-required harness is fully configured", async ({ + page, +}) => { + await installMockBridge( + page, + { + acpRuntimesCatalog: [ + runtime("buzz-agent", "available", { status: "not_applicable" }), + ], + discoverAgentModels: { + models: [{ id: "claude-sonnet-4", name: "Claude Sonnet 4" }], + supportsSwitching: true, + }, + globalAgentConfig: { + env_vars: {}, + provider: null, + model: null, + preferred_runtime: null, + }, + }, + { skipCommunitySeed: true, skipOnboardingSeed: true }, + ); + await page.goto("/"); + await navigateToSetupPage(page); + await page.getByTestId("onboarding-setup-next").click(); + await expect(page.getByTestId("onboarding-page-config")).toBeVisible(); + + // buzz-agent auto-selects as the only ready harness, but with no provider + // configured the default is not launchable — Finish must be gated. + await expect(page.getByTestId("global-agent-default-harness")).toHaveText( + "Buzz", + ); + const finish = page.getByTestId("onboarding-finish"); + await expect(finish).toBeDisabled(); + + // Configure provider + credential; model resolves via discovery/fallback. + await page.getByTestId("global-agent-provider").click(); + await page.getByTestId("global-agent-provider-option-anthropic").click(); + await page.getByTestId("persona-provider-api-key").fill("sk-test-key"); + + await expect(finish).toBeEnabled(); + await finish.click(); + await expect(page.getByText("Join or create a community")).toBeVisible(); + expect(await readSavedRuntime(page)).toBe("buzz-agent"); +}); + +test("baked build config keeps Finish enabled without manual provider setup", async ({ + page, +}) => { + await installMockBridge( + page, + { + acpRuntimesCatalog: [ + runtime("buzz-agent", "available", { status: "not_applicable" }), + ], + bakedBuildEnv: [ + { key: "BUZZ_AGENT_PROVIDER", masked: false, value: "databricks_v2" }, + { + key: "DATABRICKS_HOST", + masked: false, + value: "https://example.cloud.databricks.com", + }, + { key: "DATABRICKS_MODEL", masked: false, value: "baked-model" }, + ], + globalAgentConfig: { + env_vars: {}, + provider: null, + model: null, + preferred_runtime: null, + }, + }, + { skipCommunitySeed: true, skipOnboardingSeed: true }, + ); + await page.goto("/"); + await navigateToSetupPage(page); + await page.getByTestId("onboarding-setup-next").click(); + await expect(page.getByTestId("onboarding-page-config")).toBeVisible(); + + // Internal builds bake provider/model/credentials — the gate must treat + // baked config as complete and never block Finish. + await expect(page.getByTestId("global-agent-default-harness")).toHaveText( + "Buzz", + ); + await expect(page.getByTestId("onboarding-finish")).toBeEnabled(); +});