Revert "fix(release): bump Code Interpreter Python past PyPI collision" - #1787
Revert "fix(release): bump Code Interpreter Python past PyPI collision"#1787mishushakov wants to merge 1 commit into
Conversation
|
PR SummaryLow Risk Overview Reviewed by Cursor Bugbot for commit d3a6347. Bugbot is set up for automated code reviews on this repo. Configure here. |
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
There was a problem hiding this comment.
Reviewed against TASTE.md (e2b-dev/sdk-harness). This PR's only change is removing the changeset file .changeset/code-interpreter-python-pypi-clash.md; it touches no SDK code in packages/js-sdk, packages/python-sdk, or packages/cli, so no TASTE.md principles (API shape, parity, errors, timeouts, config, docs) apply. 0 violations, no inline comments.
Package ArtifactsBuilt from 4e9ab2d. Download artifacts from this workflow run. JS SDK ( npm install ./e2b-2.46.2-revert-1785-devin-1787865524-code-interpreter-pypi-clash.0.tgzCLI ( npm install ./e2b-cli-2.18.1-revert-1785-devin-1787865524-code-interpreter-pypi-clash.0.tgzCode Interpreter JS SDK ( npm install ./e2b-code-interpreter-2.7.3-revert-1785-devin-1787865524-code-interpreter-pypi-clash.0.tgzDesktop JS SDK ( npm install ./e2b-desktop-2.3.4-revert-1785-devin-1787865524-code-interpreter-pypi-clash.0.tgzPython SDK ( pip install ./e2b-2.46.1+revert.1785.devin.1787865524.code.interpreter.pypi.clash-py3-none-any.whlCode Interpreter Python SDK ( pip install ./e2b_code_interpreter-2.9.2+revert.1785.devin.1787865524.code.interpreter.pypi.clash-py3-none-any.whlDesktop Python SDK ( pip install ./e2b_desktop-2.4.5+revert.1785.devin.1787865524.code.interpreter.pypi.clash-py3-none-any.whl |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d3a63470c1
ℹ️ 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".
| @@ -1,5 +0,0 @@ | |||
| --- | |||
| '@e2b/code-interpreter-python': patch | |||
There was a problem hiding this comment.
Deleting this changeset leaves Code Interpreter Python at 2.9.2, whose PyPI artifacts have different contents from this repository's build. In the checked .github/workflows/publish_packages.yml, pnpm run publish invokes the root publish script, which unconditionally runs every workspace's postPublish; this package therefore rebuilds 2.9.2 and calls uv publish --check-url during any subsequent release. uv publish --help describes --check-url as “Check an index URL for existing files to skip duplicate uploads”; since these files are not duplicates, the upload cannot be skipped and the release fails again. Keep a patch changeset so the artifact is published under 2.9.3.
Useful? React with 👍 / 👎.
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
Additional findings (outside the current diff — GitHub can't attach inline comments there):
-
🔴
.changeset/code-interpreter-python-pypi-clash.md— Revert removes the only pending fix for the code-interpreter-python PyPI version collision, without any replacement changeset or manual version bump. packages/code-interpreter-python/pyproject.toml and package.json still pin version 2.9.2, which was already published to PyPI under a different checksum.Extended reasoning...
Next release run executes postPublish (
uv publish --token ${PYPI_TOKEN} --check-url https://pypi.org/simple/) for @ e2b/code-interpreter-python still at version 2.9.2;uv publish --check-urlagain fails with the local/remote sha256 mismatch that PR #1785 was created to fix, breaking the release pipeline for this package exactly as before the fix — a regression versus the base branch where the queued patch changeset would have bumped it past the collision.Verification: normal. The revert deletes the only pending fix and reintroduces the release-pipeline failure that the base branch (5a56c87) had healed. Root package.json:6
"publish": "pnpm changeset publish && pnpm run -r postPublish"runs postPublish recursively for every workspace package, and packages/code-interpreter-python/package.json:8 `"postPublish": "uv build && uv publish --token… | Severity:…
1 participant
Reverts #1785