chore: bump smol-toml override to ^1.7.1 for security advisory - #1867
Conversation
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
|
Package ArtifactsBuilt from 0c8927f. Download artifacts from this workflow run. JS SDK ( npm install ./e2b-2.49.1-devin-1789052742-smol-toml-override.0.tgzCLI ( npm install ./e2b-cli-2.19.1-devin-1789052742-smol-toml-override.0.tgzCode Interpreter JS SDK ( npm install ./e2b-code-interpreter-2.8.1-devin-1789052742-smol-toml-override.0.tgzDesktop JS SDK ( npm install ./e2b-desktop-2.4.1-devin-1789052742-smol-toml-override.0.tgzPython SDK ( pip install ./e2b-2.49.0+devin.1789052742.smol.toml.override-py3-none-any.whlCode Interpreter Python SDK ( pip install ./e2b_code_interpreter-2.10.0+devin.1789052742.smol.toml.override-py3-none-any.whlDesktop Python SDK ( pip install ./e2b_desktop-2.5.0+devin.1789052742.smol.toml.override-py3-none-any.whl |
There was a problem hiding this comment.
Looks good, straightforward dependency-override bump. Checked that the version change in pnpm-workspace.yaml (smol-toml override ^1.6.1 to ^1.7.1) matches the lockfile regeneration exactly, and confirmed the diff touches only these two files with no other unrelated changes.
Extended reasoning...
Overview
The diff is confined to pnpm-workspace.yaml and pnpm-lock.yaml. It raises the pnpm dependency override for the transitive package smol-toml from <1.6.1: ^1.6.1 to <1.7.1: ^1.7.1, following an existing override pattern already present in the file. The lockfile was regenerated accordingly: the resolved smol-toml version moved from 1.6.1 to 1.8.0 with a matching new integrity hash, in both the overrides block and the snapshots section for the package that depends on it (knip).
Security risks
None identified. This change itself is a security fix — it bumps a transitive dependency override to remediate an advisory affecting smol-toml <= 1.7.0. There is no application code, auth, crypto, or permission logic touched.
Level of scrutiny
Low. This is exactly the kind of mechanical, self-contained version bump that CLAUDE.md's approval guidelines call out as approvable without requiring a human look: a config/lockfile change with a single-line diff in the source config and a mirrored lockfile update, no ambiguity, and a clear, verifiable intent (advisory remediation).
Other factors
No changeset is required per CLAUDE.md guidance, since this doesn't change the public surface of packages/cli, js-sdk, or python-sdk — it's a workspace-level override only affecting knip, a dev tool. No spec/ files were touched, so the Copybara-sync rule is not implicated. No SDK code exists in this diff so the JS/Python parity rule doesn't apply. The PR description's claims (advisory fix version numbers, that knip is the sole consumer, that pnpm install --frozen-lockfile succeeds) are consistent with what's visible in the diff itself.
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Summary
Fixes the failing Dependabot security update run on
main.Dependabot could not apply the new
smol-tomladvisory (affects<= 1.7.0) becausesmol-tomlis only a transitive dep (viaknip@5.43.6, which declares^1.3.1) and the repo's pnpm override still pinned it to the previous advisory's fix version:Change, following the existing override pattern in
pnpm-workspace.yaml:Lockfile regenerated (
pnpm install --lockfile-only);smol-tomlresolves to1.8.0.pnpm install --frozen-lockfilesucceeds andknip(the only consumer) produces identical output before/after.No changeset: no published package source changed.
Link to Devin session: https://app.devin.ai/sessions/7eb29dfcecb0487ab1cceb9b6033d2ab
Open in Devin Desktop: https://app.devin.ai/desktop/session/7eb29dfcecb0487ab1cceb9b6033d2ab?variant=devin