Skip to content

docs: call the capability Biorouter Copilot everywhere a person reads it - #419

Merged
Broccolito merged 1 commit into
mainfrom
claude/biorouter-copilot-naming
Oct 20, 2026
Merged

Broccolito merged 1 commit into
mainfrom
claude/biorouter-copilot-naming

Conversation

@Broccolito

Copy link
Copy Markdown
Collaborator

What this fixes

The capability is Biorouter Copilot. "Computer Use" was its previous name, and "Computer Controller" was a different, removed capability. scripts/check-computer-use-naming.sh enforces this, but only in Rust, the desktop's TS/TSX, prompts and built-in skills.

Everywhere else the old names were still live:

  • living docs
  • workflow display names
  • packaging and build comments
  • script messages
  • code comments

A few lines also described the removed Computer Controller's web, office and cache tools as if they still lived there. Those tools moved to Web & Documents (webdocuments).

How it was found

  1. Sweep. Five agents each swept one disjoint slice of the tree:
    • docs/extensions, docs/design and docs/desktop-ui
    • the rest of docs
    • root files, .github and packaging
    • crates
    • ui/desktop and scripts
  2. Verify. A second agent per slice tried to reject every proposal against the carve-outs below.
  3. Apply. Only the surviving edits were applied.
  4. Critic. A final pass re-swept the whole tree and reviewed the diff. Its few missed items are included here.

Changes

  • Workflow display names:
    • computer-use-native.yml → "Biorouter Copilot native payloads"
    • computer-use-package-acceptance.yml → "Biorouter Copilot installed packages"
    • Nothing matches the old names. Branch protection requires job names (test (*), Unit tests (vitest), no-ai-coauthor), and no script filters runs by workflow name. The old names appear only in docs/history.
  • Stale facts, now correct:
    • security/global_memory.rs and security/session_store.rs, docs/extensions/built-in/memory.md and an artifactUtils.ts comment now name Web & Documents for the file tools.
    • biorouter-sandbox's crate doc names the servers that really use the sandbox: Developer and Compute.
    • The develop-biorouter built-in skill lists Web & Documents among biorouter-mcp's servers.
  • API schema prose: "The most recent actual Biorouter Copilot request…". The text changed together in the Rust doc, openapi.json and types.gen.ts, so check-openapi-schema.sh still matches.
  • Upstream by its own name: the vendored tree's comments in .gitattributes and .gitignore say Open Computer Use.
  • Prose and comments: bare "Copilot", "computer-use" and "Computer-Controller" used for the capability now say Biorouter Copilot (or legacy Computer Controller, where the text is about the removed one). This covers README, CONTRIBUTING, docs, biorouter-self-test.yaml, packaging, release.sh, Python test scripts, and desktop comments.

Unchanged on purpose

  • Identifiers: computercontroller and its tool prefix, computer_use, computer-use paths, X-Computer-Use-Key, --computer-use-approval, the computer-use: digest scope, and route and schema names.
  • File names: scripts/*computer-use* is a workflow path filter, and docs links point at existing files.
  • The macOS helper BioRouter Computer Use.app: permission grants are keyed on its bundle, and the setup copy names what System Settings shows.
  • Upstream's name and NOTICE.
  • Records and migration copy: docs/history, release notes, and copy that names an old name as old.
  • Test inputs: fixtures of data written by older versions, and the adversarial test inputs.

The site copies of two of these lines (the vaulting docs page, landing/docs.html and two news lines in landing/) went straight to main, per the rule for website edits.

Checks run locally

  • scripts/check-computer-use-naming.sh, check-no-goose-block.sh, check-brand-consistency.sh: pass.
  • Landing deploy checks (docs privacy 21/21, look rules 23/23, consistency, assemble-docs, build-registry): pass.
  • vitest on the touched areas: artifactUtils, CopilotControl, CopilotSetup, CapabilitiesSection, toolGlyph. 406/406 passed.
  • prettier --check on the touched src/ files: pass.
    • forge.config.ts reports drift at line 61, which is already on main and outside this change.
  • rustfmt --check on the touched Rust files and py_compile on the scripts: pass.

🤖 Generated with Claude Code

The naming guard (scripts/check-computer-use-naming.sh) covers Rust, the
desktop's TS/TSX, prompts and built-in skills. Outside that set, living
docs, workflow names, packaging comments, script messages and code comments
still said "Computer Use", "Computer-Controller", "computer-use" or a bare
"Copilot" for the capability, and a few described the removed Computer
Controller's web, office and cache tools as if they still lived there.

- Workflow display names: "Biorouter Copilot native payloads" and
  "Biorouter Copilot installed packages". Nothing matches the old names:
  branch protection requires job names, and no script filters runs by
  workflow name.
- Stale facts: the global memory barrier, the session store notes, the
  memory extension doc and an artifact comment now name Web & Documents
  (`webdocuments`), which is where those file tools moved. The sandbox
  crate names its real consumers (Developer and Compute).
- The API schema description of the last request says Biorouter Copilot in
  the Rust doc, openapi.json and the generated types together.
- The vendored upstream tree is called by its own name, Open Computer Use.

Unchanged on purpose: every identifier (`computercontroller`,
`computer_use`, `computer-use` paths, `X-Computer-Use-Key`,
`--computer-use-approval`), file names, the macOS helper
"BioRouter Computer Use.app" that permission grants are keyed on,
upstream's name and notice, docs/history and release notes, migration copy
that names an old name as old, and test fixtures of old data.
Broccolito added a commit that referenced this pull request Oct 20, 2026
…the v1.91.2 news

The vaulting docs page (and its assembled copy in landing/docs.html) said
"every Copilot tool", and the v1.91.2 news item on the about page and in
the site content notes said "Copilot" alone. The capability is Biorouter
Copilot; the app-side sweep of the same wording is PR #419.
@Broccolito
Broccolito merged commit 06ca6bd into main Oct 20, 2026
24 checks passed
@Broccolito
Broccolito deleted the claude/biorouter-copilot-naming branch October 20, 2026 07:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant