refact: google colab - #16
Merged
Merged
Conversation
There was a problem hiding this comment.
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_sandboxmodules and update public exports. - Treat
google_colabas the canonical variant while mapping legacy CLI/API inputs (colab,google-colab) togoogle_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.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.