Skip to content

refact: google colab - #16

Merged
echarles merged 3 commits into
mainfrom
var/google-colab
Aug 10, 2026
Merged

refact: google colab#16
echarles merged 3 commits into
mainfrom
var/google-colab

Conversation

@echarles

Copy link
Copy Markdown
Member

No description provided.

Copilot AI lite review requested due to automatic review settings August 10, 2026 06:00

Copilot AI 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.

Pull request overview

This PR refactors the Google Colab integration by introducing google_colab as the canonical variant name, adding dedicated google_colab client/sandbox modules, and updating the CLI/tests/examples/docs accordingly while aiming to preserve backwards compatibility with the previous colab naming.

Changes:

  • Add new code_sandboxes.google_colab / code_sandboxes.google_colab_sandbox modules and update public exports.
  • Treat google_colab as the canonical variant while mapping legacy CLI/API inputs (colab, google-colab) to google_colab.
  • Update documentation, examples, and tests to use the new names and variant strings.

Reviewed changes

Copilot reviewed 28 out of 28 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tests/test_models.py Adds assertion for the new GOOGLE_COLAB enum value.
tests/test_modal_google_colab_sandbox.py Updates sandbox import/instantiation to GoogleColabSandbox.
tests/test_google_colab.py Updates imports and symbol names to google_colab equivalents.
tests/test_factory.py Updates factory expectations and uses google_colab in forwarded-kwargs test.
tests/test_cli_repl.py Switches CLI variant usage to google-colab alias and expects normalized variant.
README.md Updates canonical variant list and Colab-related usage examples/names.
pyproject.toml Adds google-colab optional dependency extra placeholder.
examples/repl/Makefile Renames colab target to google-colab.
examples/repl/google_colab_sandbox_example.py Uses variant="google_colab".
examples/README.md Updates supported variants, commands, and notes to google-colab.
examples/exec/Makefile Renames colab target to google-colab.
examples/exec/google_colab_sandbox_example.py Uses variant="google_colab".
docs/docs/sandboxes/index.mdx Updates canonical variant list and docs table label to google_colab + alias note.
docs/docs/sandboxes/google-colab.mdx Renames API references to GoogleColabKernelClient and google_colab variant string.
docs/docs/index.mdx Updates variant table to google-colab.
docs/docs/examples/index.mdx Updates Colab example source link and make target.
docs/docs/comparison/index.mdx Updates supported variant list to google-colab.
docs/docs/cli/index.mdx Updates CLI variant list/description to google-colab.
docs/docs/api-reference/index.mdx Updates API reference variant strings and exported symbol names.
code_sandboxes/models.py Adds SandboxVariant.GOOGLE_COLAB = "google_colab".
code_sandboxes/google_colab.py New Colab kernel client module with new names + backward-compatible aliases.
code_sandboxes/google_colab_sandbox.py New Colab sandbox module for google_colab variant.
code_sandboxes/colab.py Converts old module into a backward-compatible re-export shim.
code_sandboxes/colab_sandbox.py Converts old module into a backward-compatible re-export shim.
code_sandboxes/cli.py Adds/normalizes google_colab and google-colab (and maps legacy inputs).
code_sandboxes/base.py Normalizes legacy colab variants to google_colab and creates the new sandbox class.
code_sandboxes/version.py Bumps package version to 1.0.3.
code_sandboxes/init.py Updates public exports to include GoogleColab* symbols.

馃挕 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread code_sandboxes/google_colab_sandbox.py Outdated

@echarles echarles left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

LGTM

@echarles
echarles merged commit 8fc8875 into main Aug 10, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants