Skip to content

chore: remove archived TypeScript claw server - #2486

Merged
Nikhil (shadowfax92) merged 1 commit into
mainfrom
chore/remove-archive
Aug 31, 2026
Merged

chore: remove archived TypeScript claw server#2486
Nikhil (shadowfax92) merged 1 commit into
mainfrom
chore/remove-archive

Conversation

@shadowfax92

Copy link
Copy Markdown
Contributor

Summary

  • remove the retired packages/archive/claw-server-ts implementation
  • delete its obsolete source, migrations, configuration, and tests
  • keep the current Rust and BrowserOS server implementations as the maintained paths

Design

This is a pure deletion of an unreferenced archive. Repository-wide reference scans found no build, runtime, or documentation consumers.

Test plan

  • bun run check
  • bun run --filter @browseros/app clean:cache
  • bun run test:all
  • verify packages/archive is absent
  • verify no repository references to the removed archive remain

@shadowfax92

Copy link
Copy Markdown
Contributor Author

Greptile (@greptileai) review

@shadowfax92

Copy link
Copy Markdown
Contributor Author

Claude (@claude) review

@shadowfax92
Nikhil (shadowfax92) merged commit e92416b into main Aug 31, 2026
4 of 5 checks passed
@github-actions github-actions Bot added the chore label Aug 31, 2026
@claude

claude Bot commented Aug 31, 2026

Copy link
Copy Markdown

Claude finished Nikhil (@shadowfax92)'s task in 1m 46s —— View job


Reviewing PR #2486

  • Gather context (PR diff, workspace config)
  • Verify no remaining references to packages/archive/claw-server-ts
  • Check workspace/build config for removed package references
  • Verify test plan claims
  • Post review feedback

Review

This is a clean, low-risk deletion. Verified against the pre-removal tree:

  • Not part of the active workspace: the real Bun/Turbo workspace root is packages/browseros-agent/package.json, with workspaces: ["apps/*", "packages/*", "crates/*"] resolved relative to that directory. packages/archive/claw-server-ts lives at the repo-root packages/archive/, a sibling of packages/browseros-agent/, so it was never actually included in the workspace glob — its workspace:* deps (@browseros/agent-mcp-manager, @browseros/claw-api, etc.) in package.json would not even have resolved. This confirms it was already orphaned/dead before this PR.
  • No live references to the removed package name: grepped the whole repo for claw-server-ts and for the package name @browseros/claw-server (excluding the -rust variant, which is the actively maintained implementation) — zero hits outside the deleted directory itself. bun.lock in packages/browseros-agent has no entry for @browseros/claw-server.
  • CI/docs/root config: no references in .github/workflows/*, README.md, docs/, or turbo.json. The few "archive" matches elsewhere in the repo are unrelated (zip archiving in build scripts, Expand-Archive in PowerShell, the CLI's own update-archive logic).
  • Test plan is valid: bun run check, bun run test:all, and bun run --filter @browseros/app clean:cache all correspond to real scripts in packages/browseros-agent/package.json and apps/app/package.json.

No concerns. This is a straightforward, well-verified removal of dead code with the Rust (claw-server-rust) and BrowserOS server implementations remaining as the maintained paths.

@greptile-apps

greptile-apps Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Removes the retired TypeScript claw server archive while leaving the maintained Rust and BrowserOS server implementations unchanged.

  • Deletes the archived package manifest, configuration, runtime source, and documentation.
  • Deletes the archived Drizzle migrations, schema snapshots, and tests.
  • Removes only files under packages/archive/claw-server-ts, which was outside the active workspace.

Confidence Score: 5/5

The PR appears safe to merge because the deleted archive has no remaining build, runtime, test, CI, release, or documentation consumers.

The changes exclusively delete a reference-only package outside the active workspace, and repository-wide reference checks found no remaining dependency on its paths, package identity, entry points, or scripts.

Important Files Changed

Filename Overview
packages/archive/claw-server-ts/package.json Deletes the retired package manifest; no active workspace, build, test, CI, or release configuration consumes it.
packages/archive/claw-server-ts/src/main.ts Deletes the archived TypeScript server entry point without affecting maintained server entry points.
packages/archive/claw-server-ts/README.md Deletes documentation that explicitly identified this implementation as a reference-only frozen snapshot outside active tooling.
packages/archive/claw-server-ts/drizzle/meta/_journal.json Deletes migration metadata belonging exclusively to the retired server archive.

Reviews (1): Last reviewed commit: "chore: remove archived TypeScript claw s..." | Re-trigger 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