Skip to content

refactor!: migrate code-interpreter paths from /opt/opensandbox to /opt/code-interpreter#1012

Merged
jwx0925 merged 1 commit into
mainfrom
refactor/code-interpreter-path-migration
Jun 11, 2026
Merged

refactor!: migrate code-interpreter paths from /opt/opensandbox to /opt/code-interpreter#1012
jwx0925 merged 1 commit into
mainfrom
refactor/code-interpreter-path-migration

Conversation

@Pangjiping

@Pangjiping Pangjiping commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

Migrate all code-interpreter paths from /opt/opensandbox/ to /opt/code-interpreter/, unifying the directory structure under /opt/code-interpreter for architectural consistency with the injection point /opt/opensandbox.

Breaking Change

This is a breaking change. The following paths are removed:

Removed path Replacement
/opt/opensandbox/code-interpreter.sh /opt/code-interpreter/code-interpreter.sh
/opt/opensandbox/code-interpreter-env.sh /opt/code-interpreter/code-interpreter-env.sh
/opt/opensandbox/jupyter.log /opt/code-interpreter/jupyter.log

Migration Guide

If you use opensandbox/code-interpreter:latest

Update your sandbox creation to use the new entrypoint:

# Before
sandbox = client.create_sandbox(
    image="opensandbox/code-interpreter:latest",
    entrypoint=["/opt/opensandbox/code-interpreter.sh"],
)

# After
sandbox = client.create_sandbox(
    image="opensandbox/code-interpreter:latest",
    entrypoint=["/opt/code-interpreter/code-interpreter.sh"],
)

If you pin a specific tag (e.g., v1.0.2)

No action needed — existing tagged images are immutable and continue to work with /opt/opensandbox/code-interpreter.sh. Only switch to the new path when upgrading to a tag that includes this change.

If you source the env script interactively

# Before
source /opt/opensandbox/code-interpreter-env.sh python 3.14

# After
source /opt/code-interpreter/code-interpreter-env.sh python 3.14

If you use the Go SDK (sdks/sandbox/go)

The CodeInterpreterEntrypoint constant has been updated. If you use CreateCodeInterpreter() without explicitly setting Entrypoint, no change needed. If you hardcoded the old path, update to the new constant.

Changed Files

25 files: Dockerfiles, entrypoint scripts, SDK constants, E2E tests, examples, and READMEs.

🤖 Generated with Claude Code

…-interpreter

All paths referencing /opt/opensandbox/code-interpreter{,-env}.sh moved to
/opt/code-interpreter/. Covers Dockerfiles, entrypoint scripts, SDK constants,
E2E tests, examples, and READMEs.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 52fc3fc31e

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread sandboxes/code-interpreter/Dockerfile
Comment thread sandboxes/code-interpreter/Dockerfile
Comment thread README.md
Comment thread sandboxes/code-interpreter/Dockerfile_base

@Pangjiping Pangjiping left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the review. All four comments point to the same architectural decision:

This is an intentional one-time breaking change. The goal is to separate concerns — /opt/opensandbox is reserved as the architecture injection point, while /opt/code-interpreter is the code-interpreter component's own namespace.

To address each point:

  1. Base image build order — correct, build.sh has base commented out. The release that ships this PR must build base first. That's a release-process concern, not a code defect.

  2. Old entrypoint path removed — by design. No symlink. Callers update to the new path per the migration guide in the PR description.

  3. Pinned tag examples — the v1.0.2 examples in READMEs are stable references to the old image, which still ships the old path. Examples targeting latest use the new path. This is correct.

  4. Env script path removed — same as (2). No compatibility symlink. Migration guide covers it.

PR description has the full migration matrix.

@Pangjiping Pangjiping changed the title refactor: migrate code-interpreter from /opt/opensandbox to /opt/code-interpreter refactor!: migrate code-interpreter paths from /opt/opensandbox to /opt/code-interpreter Jun 10, 2026

@jwx0925 jwx0925 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jwx0925 jwx0925 merged commit d2fba03 into main Jun 11, 2026
26 of 27 checks passed
@Pangjiping Pangjiping deleted the refactor/code-interpreter-path-migration branch June 11, 2026 02:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants