refactor(desktop): shorten bundled-adapter Doctor copy to "Bundled with Buzz."

The Doctor runtime row's status line for a bundled adapter read
"ACP bridge bundled with Buzz." — the protocol jargon adds nothing in
a row that already names the runtime, and the sentence only needs to
say where the tool comes from. Shorten it to "Bundled with Buzz."
(keeping the trailing period to match the sibling "Available via ..."
line).

Updated in step: the DoctorSettingsPanel comment quoting the line, the
09-bundled-adapter e2e assertion and its doc comment, and the
AcpRuntimeCatalogEntry::adapter_bundled doc comment that cites the UI
copy. No other test or fixture asserts the old string; the node-runtime
check's backend-sourced messages keep their ACP wording per the
row-detail/page-framing split from the de-jargon pass.

Verification:
- tsc --noEmit: clean; biome check on both touched TS files: clean
- cargo fmt --check (desktop/src-tauri): clean (doc-comment-only change)
- playwright doctor-states.spec.ts: 9 passed, including
  09-bundled-adapter against the new copy

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Signed-off-by: Matt Toohey <contact@matttoohey.com>
This commit is contained in:
Matt Toohey
2026-07-15 13:48:47 +10:00
co-authored by Claude Fable 5
parent a4e592ed3a
commit b3aed32a88
3 changed files with 6 additions and 6 deletions
@@ -571,8 +571,8 @@ pub struct AcpRuntimeCatalogEntry {
pub command: Option<String>,
pub binary_path: Option<String>,
/// true when `binary_path` is the pinned bridge bundled with the app
/// rather than a user install. The Doctor UI says "ACP bridge bundled
/// with Buzz" instead of rendering the resource-dir path.
/// rather than a user install. The Doctor UI says "Bundled with Buzz"
/// instead of rendering the resource-dir path.
pub adapter_bundled: bool,
pub default_args: Vec<String>,
pub mcp_command: Option<String>,
@@ -202,7 +202,7 @@ function RuntimeRow({
<>
<p className="mt-1 text-sm font-normal text-muted-foreground">
{runtime.adapterBundled
? "ACP bridge bundled with Buzz."
? "Bundled with Buzz."
: `Available via ${describeResolvedCommand(runtime.command, runtime.binaryPath)}.`}
</p>
{runtime.defaultArgs.length > 0 ? (
@@ -214,7 +214,7 @@ function RuntimeRow({
</p>
) : null}
{/* The bundled bridge's resource-dir path is noise — the
"ACP bridge bundled with Buzz." line above covers it. The
"Bundled with Buzz." line above covers it. The
user-CLI path row is retired with the cli_missing gate: the
bundled bridges vendor their own CLI, so no runtime reports a
separate CLI path anymore. */}
+2 -2
View File
@@ -451,7 +451,7 @@ test.describe("Doctor panel state screenshots", () => {
/**
* 09 — available runtime whose adapter is the bridge bundled with the app:
* the row says "ACP bridge bundled with Buzz" instead of rendering the
* the row says "Bundled with Buzz" instead of rendering the
* resource-dir path, and no CLI path renders — the bundled bridge vendors
* its own CLI, so the user-CLI row retired with the cli_missing gate.
*/
@@ -476,7 +476,7 @@ test.describe("Doctor panel state screenshots", () => {
const row = page.getByTestId("doctor-runtime-claude");
await expect(row).toBeVisible({ timeout: 10_000 });
await expect(row).toContainText("ACP bridge bundled with Buzz.");
await expect(row).toContainText("Bundled with Buzz.");
await expect(row).not.toContainText("CLI:");
await expect(row).not.toContainText(bundledPath);
await expect(row).not.toContainText("installed on PATH");