Skip to content

chore: remove unsupported Go CLI - #2915

Merged
Nikhil (shadowfax92) merged 1 commit into
mainfrom
feat/fix-remove-go-cli
Oct 13, 2026
Merged

Nikhil (shadowfax92) merged 1 commit into
mainfrom
feat/fix-remove-go-cli

Conversation

@shadowfax92

Copy link
Copy Markdown
Contributor

Summary

Remove the unsupported Go browseros-cli application and its release pipeline. The repository no longer advertises or builds that client; the existing MCP servers and unrelated developer tools remain available.

- packages/browseros-agent/apps/cli/          # Go app, analytics, npm wrapper, updater, installers
- packages/browseros-agent/scripts/build/cli/ # release policy, npm access, CDN uploader
- packages/browseros-agent/scripts/build/cli.ts
- packages/browseros-agent/apps/server/tests/cli/ # builds and exercises the Go binary
- .github/workflows/release-cli.yml

Remove the CLI package script, Fallow exclusion, env-migration special cases, analytics test examples, README/contributor references, and contributor-guidance links. Keep the shared analytics implementation and server entry points unchanged.

Evidence

  • Before: the Go app, publishing workflow, build/upload tooling, and runtime tests were present, and the README and contributing guide offered it as a supported path.

  • After: the requested recursive reference grep returns no matches in this checkout. An additional scan of README, contributor, and docs text confirms remaining CLI mentions refer to other tools.

  • bun install passed; the lockfile is unchanged and matches the surviving workspace manifests.

  • bun run check passed (lint, workspace typecheck, Fallow; existing non-fatal warnings remain).

  • bun run test passed: 2,108 passing, 0 failing, 3 skipped.

  • bun test scripts/env/migrate-local.test.ts passed: 6 passing.

  • bun run build:agent passed.

  • bun scripts/build/server.ts --target=darwin-arm64 --ci --no-upload passed.

  • rustfmt --check --edition 2024 packages/browseros-agent/apps/claw-server/src/analytics/events.rs and git diff --check passed.

  • Independent ft-code-review round completed; its one minor documentation finding was accepted and fixed. No unresolved findings.

Docs and intentional historical references

The starting checkout already has no Go CLI pages, nav entries, images, or snippets under docs/; repository docs history contains no such published pages either. No page URLs are removed by this PR, so there are no new redirects to add. docs/docs.json parses as JSON and all 34 navigation pages resolve.

The uninitialized .internal-docs submodule was inspected through its pinned Git objects. Its sole intentional historical mention is architecture/rust-port/README.md:71-72, at pinned commit f04863b14e338267a5d7b3f6b2d8e6aed45dda41: apps/cli (Go) is cited as precedent for non-JS apps not needing a package.json. That design history remains true; the submodule and gitlink are unchanged.

Follow-ups for the owner

  • Decide how to retire the published browseros-cli npm package. This PR does not deprecate or unpublish it. Its existing postinstall wrapper downloads versioned binaries and checksums from GitHub releases such as cli/v0.4.0; preserve those releases/tags/assets until an explicit retirement decision.
  • Decide whether to preserve/freeze or retire https://cdn.browseros.com/cli/install.sh, /cli/install.ps1, /cli/latest/version.txt, /cli/latest/manifest.json, and versioned /cli/v0.4.0/* binaries/checksums. Old installs self-update from the latest manifest, and the hosted installers resolve the latest version and fetch those archives. Read-only GET checks confirm both installers and both latest metadata endpoints return HTTP 200, currently advertising 0.4.0 across six platform assets. Removing the release workflow stops future refreshes; this PR changes no hosted objects.

Merge Danger

Door: two-way; the source deletion can be reverted in Git.
Blast Radius: building and publishing the unsupported Go client stops. Existing npm, GitHub, and CDN artifacts are unchanged; their eventual retirement is an owner follow-up.

@github-actions github-actions Bot added the chore label Oct 13, 2026
@github-actions

Copy link
Copy Markdown
Contributor

✅ Tests passed: 2555/2558

Ran 13 of 16 suites (3 not affected by this change).

Suite Passed Failed Skipped
✅ server-agent 216/216 0 0
✅ server-api 289/289 0 0
✅ server-tools 248/248 0 0
✅ server-browser 10/10 0 0
✅ server-integration 10/10 0 0
✅ server-lib 199/199 0 0
✅ server-root 38/41 0 3
✅ agent 372/372 0 0
⏩ claw-app n/a n/a not affected
⏩ claw-onboard n/a n/a not affected
⏩ app-onboard n/a n/a not affected
✅ build 26/26 0 0
✅ release 66/66 0 0
✅ claw-server 929/929 0 0
✅ claw-server-quality passed 0 0
✅ claw-mcp 152/152 0 0

passed = ran successfully but emits no JUnit counts (a lint/format gate).

View workflow run

@shadowfax92
Nikhil (shadowfax92) merged commit fe5b823 into main Oct 13, 2026
26 checks passed
@greptile-apps

greptile-apps Bot commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[Low impact] The PR appears safe to merge and cleanly removes the unsupported Go client.

Summary

Removes the unsupported Go browseros-cli application and its build, install, update, and release tooling.

  • The repository no longer includes the Go browser-control CLI.
  • CLI release builds and uploads no longer run from this repository.
  • Project guides and maintenance scripts no longer point to the CLI.

Acknowledged scope:

  • shadowfax92 explicitly deferred retirement of published npm, GitHub, and CDN artifacts to the owner. This PR changes no hosted objects.
  • shadowfax92 intentionally retained the pinned historical apps/cli reference in the unchanged internal-docs submodule because it remains valid design history.

Reviews (1) · Last reviewed commit: "chore: remove unsupported Go CLI" · Reviewed by Greptile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant