fix(deps): bump smol-toml override to 1.7.1 - #1869
Conversation
|
PR SummaryLow Risk Overview Reviewed by Cursor Bugbot for commit a254470. Bugbot is set up for automated code reviews on this repo. Configure here. |
Package ArtifactsBuilt from 757119e. Download artifacts from this workflow run. JS SDK ( npm install ./e2b-2.49.1-fix-smol-toml-override.0.tgzCLI ( npm install ./e2b-cli-2.19.1-fix-smol-toml-override.0.tgzCode Interpreter JS SDK ( npm install ./e2b-code-interpreter-2.8.1-fix-smol-toml-override.0.tgzDesktop JS SDK ( npm install ./e2b-desktop-2.4.1-fix-smol-toml-override.0.tgzPython SDK ( pip install ./e2b-2.49.0+fix.smol.toml.override-py3-none-any.whlCode Interpreter Python SDK ( pip install ./e2b_code_interpreter-2.10.0+fix.smol.toml.override-py3-none-any.whlDesktop Python SDK ( pip install ./e2b_desktop-2.5.0+fix.smol.toml.override-py3-none-any.whl |
There was a problem hiding this comment.
Looks good, straightforward dependency override bump. Reviewed: pnpm-workspace.yaml's override bump for smol-toml from <1.6.1: ^1.6.1 to <1.7.1: ^1.7.1, and the corresponding pnpm-lock.yaml regeneration (overrides section, integrity hash, and packages/snapshots entries for smol-toml@ 1.8.0, consumed by knip). No source code, package public surfaces, or specs are affected.
Extended reasoning...
Overview
The diff touches exactly two files: pnpm-workspace.yaml and pnpm-lock.yaml. It raises a pnpm dependency override for the transitive dev dependency smol-toml (pulled in via knip) from <1.6.1: ^1.6.1 to <1.7.1: ^1.7.1, which causes the lockfile to resolve smol-toml@ 1.8.0 instead of 1.6.1. The lockfile change is confined to the overrides: block, the package resolution/integrity entry, and the snapshots: entry for the package that depends on it (knip). This is consistent end-to-end and matches the PR's stated intent of bumping a dependency-override range.
Security risks
None identified. This is a version bump of a dev-only transitive dependency (used by knip, a lint/dead-code tool), not runtime/production code. The change does not touch auth, crypto, permissions, or any application logic. If anything, this is a security-motivated fix per the PR description (addressing a dependency advisory), and moving to a newer, presumably patched version reduces risk rather than introducing it.
Level of scrutiny
Low. This is a mechanical, self-contained lockfile/override change with no ambiguity — exactly the class of change (version bumps, config tweaks) that does not need deep human review. I verified the diff directly rather than relying solely on the PR description, and the actual file contents match the stated summary precisely.
Other factors
No SDK package (packages/cli, packages/js-sdk, packages/python-sdk) public surface is touched, so per CLAUDE.md a changeset is not required. No spec/ files are touched, so the Copybara-sync restriction is not implicated. No tests are needed for a pure dependency bump. The bug hunter reported no findings, and I found nothing beyond that on my own read of the diff.
Resolves Dependabot alert #382 (CVE-2026-85730, high).
smol-tomlis a transitive dev dependency ofknip@5.43.6. This PR moves the existing pnpm override from<1.6.1 -> ^1.6.1to<1.7.1 -> ^1.7.1. The lockfile now resolvessmol-toml@1.8.0.Only
pnpm-workspace.yamlandpnpm-lock.yamlchange.