chore: rename the infra repository references to e2b-dev/runtime - #1864
Conversation
The e2b-dev/infra repository was renamed to e2b-dev/runtime. Git clone URLs follow the redirect, but the GitHub App installation-token endpoint does not, so the generated-files check failed to mint a token for 'belt,infra'. Point the token mint, the Copybara origin, the spec pin (spec/runtime-ref, E2B_RUNTIME_REF) and the docs at the new name. Co-Authored-By: mish@e2b.dev <mish@e2b.dev>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
|
There was a problem hiding this comment.
TASTE.md review: no violations. The PR only renames e2b-dev/infra → e2b-dev/runtime in comments, docstrings, CI config, Copybara config, and the spec-fetch script; it changes no public SDK surface. Checked the touched SDK docstrings (sandboxApi.ts, sandbox_api.py, sandbox_sync/main.py, sandbox_async/main.py) against T-1/T-2 parity, T-69–T-74 docstring rules, and T-12 acronym casing — all still comply, and the sync/async Python edits mirror each other.
Package ArtifactsBuilt from e1afd5d. Download artifacts from this workflow run. JS SDK ( npm install ./e2b-2.49.1-devin-1789050962-rename-infra-to-runtime.0.tgzCLI ( npm install ./e2b-cli-2.19.1-devin-1789050962-rename-infra-to-runtime.0.tgzCode Interpreter JS SDK ( npm install ./e2b-code-interpreter-2.8.1-devin-1789050962-rename-infra-to-runtime.0.tgzDesktop JS SDK ( npm install ./e2b-desktop-2.4.1-devin-1789050962-rename-infra-to-runtime.0.tgzPython SDK ( pip install ./e2b-2.49.0+devin.1789050962.rename.infra.to.runtime-py3-none-any.whlCode Interpreter Python SDK ( pip install ./e2b_code_interpreter-2.10.0+devin.1789050962.rename.infra.to.runtime-py3-none-any.whlDesktop Python SDK ( pip install ./e2b_desktop-2.5.0+devin.1789050962.rename.infra.to.runtime-py3-none-any.whl |
There was a problem hiding this comment.
Looks good, straightforward rename/config fix. Reviewed: the actual CI fix (repositories: belt,infra → belt,runtime in generated_files.yml), the copy.bara.sky/fetch-spec.sh/Makefile env-var and pin renames (E2B_INFRA_REF→E2B_RUNTIME_REF, infra-ref→runtime-ref), the spec/infra-ref→spec/runtime-ref rename (commit hash unchanged), and the doc-comment updates in the JS/Python SDK and CLI. All are comment/config-only changes with no behavior impact.
Extended reasoning...
Overview
This PR renames every internal reference from e2b-dev/infra to e2b-dev/runtime following the upstream repo rename, fixing a broken "Generated files" CI check (the GitHub App installation-token action doesn't follow the rename redirect for its repositories: list). Changes span the workflow file, copy.bara.sky, scripts/fetch-spec.sh, Makefile, spec/README.md, CLAUDE.md, and doc comments/docstrings in the JS SDK, Python SDK, and CLI. The spec/infra-ref file was renamed to spec/runtime-ref with identical content (same pinned commit hash), confirmed via git show.
Security risks
None. No functional code paths change — only comments, docstrings, an env var name, a Copybara config variable name, and a GitHub Actions repositories: list value used for scoping a short-lived installation token to the (renamed) repo it already needed access to.
Level of scrutiny
Low scrutiny is appropriate: this is a mechanical, find-and-replace-style rename with no changeset required (correctly omitted per CLAUDE.md, since only comments/docstrings in published packages changed) and no behavioral changes to test. I verified the actual functional fix (the repositories: field) is correct and that the pin file rename preserved the exact commit reference. I also grepped for any remaining e2b-dev/infra / infra-ref / E2B_INFRA_REF references across the changed files and confirmed all were updated consistently, with no other workflow files left referencing the old name.
Other factors
One minor leftover: README.md:106 still links to github.com/e2b-dev/infra and says "E2B infrastructure" — untouched by this PR and outside its stated scope (GitHub's redirect keeps the link functional, so this is cosmetic, not a bug). No third-party reviewer objections are outstanding, and the change is small, self-contained, and low-risk enough that a human need not review it further.
Summary
e2b-dev/infrawas renamed toe2b-dev/runtimetoday. Git clone URLs and the REST API follow the redirect, but the GitHub App installation-token endpoint does not, so the "Generated files" check has failed on every PR since ~13:34 UTC:This renames every reference to the old repo name so nothing depends on the redirect:
.github/workflows/generated_files.yml:repositories: belt,infra→belt,runtime(the actual fix)copy.bara.sky:INFRA_REPO→RUNTIME_REPO = "https://github.com/e2b-dev/runtime.git"spec/infra-ref→spec/runtime-ref(same commit), override envE2B_INFRA_REF→E2B_RUNTIME_REFinscripts/fetch-spec.shMakefile,spec/README.md,CLAUDE.md, and the SDK/CLI docstrings that point at the server-side sourceNo changeset: only docstrings/comments change in the published packages. CHANGELOG entries and
infra#NNNNissue shorthands are left as historical.Link to Devin session: https://app.devin.ai/sessions/4b14cd0990b24663970c42a6dcd8152a
Open in Devin Desktop: https://app.devin.ai/desktop/session/4b14cd0990b24663970c42a6dcd8152a?variant=devin
Requested by: @mishushakov