Skip to content

fix(agent-kit): rename sandbox to forks#86

Merged
designcode merged 1 commit intomainfrom
refactor/rename-sandbox-to-forks
Apr 20, 2026
Merged

fix(agent-kit): rename sandbox to forks#86
designcode merged 1 commit intomainfrom
refactor/rename-sandbox-to-forks

Conversation

@designcode
Copy link
Copy Markdown
Collaborator

@designcode designcode commented Apr 19, 2026

Summary

  • Rename all "sandbox" references to "forks" across @tigrisdata/agent-kit
  • Aligns with Tigris terminology used in CLI (tigris forks create), docs, and SDK
  • Avoids confusion with compute sandboxes

What changed

Before After
createSandbox createForks
teardownSandbox teardownForks
Sandbox Forks
SandboxFork Fork
CreateSandboxOptions CreateForksOptions
TeardownSandboxOptions TeardownForksOptions
sandbox.ts forks.ts

Updated: source, types, tests, README, package.json

Test plan

  • npm run build — all packages build
  • npm run lint — no lint errors
  • npm run format:check — all formatted
  • npm test — 11/11 agent-kit tests passing
  • Zero instances of "sandbox" remaining in agent-kit package

🤖 Generated with Claude Code


Note

Medium Risk
Primarily a terminology/API rename, but it is a breaking change for consumers importing createSandbox/Sandbox types. Runtime logic is effectively unchanged aside from default fork name prefix and updated error messages.

Overview
Renames the Agent Kit copy-on-write “sandbox” workflow to Forks, updating the public API (createForks/teardownForks, Forks/Fork and option types), barrel exports, and all README/package metadata references.

Updates integration tests to use the new names and tweaks fork bucket default prefix and related error/teardown messaging; also includes a small formatting-only change in storage’s migrate.ts.

Reviewed by Cursor Bugbot for commit 4646cf8. Bugbot is set up for automated code reviews on this repo. Configure here.

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Apr 19, 2026

Greptile Summary

This PR is a mechanical rename across @tigrisdata/agent-kitcreateSandbox/teardownSandbox and their associated types (Sandbox, SandboxFork, etc.) are replaced with createForks/teardownForks (Forks, Fork, etc.), aligning with Tigris CLI and SDK terminology. The rename is thorough: source, types, exports, tests, README, and package.json are all updated with zero residual "sandbox" references in the package.

Confidence Score: 5/5

Safe to merge — purely a rename with no logic changes and complete coverage across all files.

All changes are mechanical renames with no behavioral differences. No sandbox references remain in the package, all exports are consistent, and the integration tests were updated to match. The unrelated migrate.ts change is a trivial line-wrap fix.

No files require special attention.

Important Files Changed

Filename Overview
packages/agent-kit/src/forks.ts Renamed from sandbox.ts — all types (CreateForksOptions, Fork, Forks, TeardownForksOptions) and functions (createForks, teardownForks) consistently updated; logic unchanged.
packages/agent-kit/src/index.ts Re-exports updated to use new forks.ts module; all four renamed types and both renamed functions are exported correctly.
packages/agent-kit/test/integration.test.ts All sandbox references updated to forks equivalents; describe block name, variable names, and function calls are consistently renamed.
packages/agent-kit/README.md Section headings, code examples, and API reference table updated; no sandbox references remain.
packages/agent-kit/package.json Description and keywords updated from 'sandbox' to 'forks'; version remains 0.0.1 (pre-release, so breaking rename is acceptable).
packages/storage/src/lib/object/migrate.ts Pure whitespace/line-wrapping fix for a long boolean expression; no logic changes.

Reviews (1): Last reviewed commit: "refactor(agent-kit): rename sandbox to f..." | Re-trigger Greptile

Rename all sandbox references to forks to align with Tigris
terminology (CLI, docs, SDK all use "forks") and avoid confusion
with compute sandboxes.

- createSandbox → createForks / teardownSandbox → teardownForks
- Sandbox → Forks / SandboxFork → Fork
- CreateSandboxOptions → CreateForksOptions
- TeardownSandboxOptions → TeardownForksOptions
- sandbox.ts → forks.ts
- Updated tests, README, package.json description and keywords

Assisted-by: Claude Opus 4.6 via Claude Code
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@designcode designcode force-pushed the refactor/rename-sandbox-to-forks branch from d656b4f to 4646cf8 Compare April 19, 2026 07:15
@designcode designcode changed the title refactor(agent-kit): rename sandbox to forks fix(agent-kit): rename sandbox to forks Apr 19, 2026
@designcode designcode merged commit f181782 into main Apr 20, 2026
2 checks passed
@designcode designcode deleted the refactor/rename-sandbox-to-forks branch April 20, 2026 04:25
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 0.1.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 3.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

3 participants