fix(agent-kit): rename sandbox to forks#86
Conversation
Greptile SummaryThis PR is a mechanical rename across Confidence Score: 5/5Safe 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
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>
d656b4f to
4646cf8
Compare
|
🎉 This PR is included in version 0.1.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 3.2.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Summary
@tigrisdata/agent-kittigris forks create), docs, and SDKWhat changed
createSandboxcreateForksteardownSandboxteardownForksSandboxForksSandboxForkForkCreateSandboxOptionsCreateForksOptionsTeardownSandboxOptionsTeardownForksOptionssandbox.tsforks.tsUpdated: source, types, tests, README, package.json
Test plan
npm run build— all packages buildnpm run lint— no lint errorsnpm run format:check— all formattednpm test— 11/11 agent-kit tests passing🤖 Generated with Claude Code
Note
Medium Risk
Primarily a terminology/API rename, but it is a breaking change for consumers importing
createSandbox/Sandboxtypes. 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/Forkand 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’smigrate.ts.Reviewed by Cursor Bugbot for commit 4646cf8. Bugbot is set up for automated code reviews on this repo. Configure here.