Fix Dependabot security alerts (12 vulnerabilities)#264
Conversation
Bump vulnerable npm and Python dev/transitive dependencies: - npm: postcss, vite, brace-expansion, smol-toml - python/: pytest (^9.0.3), pytest-asyncio (^1.3.0), pillow, pygments, python-dotenv - chart_data_extractor/: pytest (^9.0.3), pillow, pygments - template/requirements.txt: pytest 9.0.3 Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
PR SummaryMedium Risk Overview Adjusts Python test configuration for the Reviewed by Cursor Bugbot for commit 9cefae1. Bugbot is set up for automated code reviews on this repo. Configure here. |
The session-scoped event_loop fixture override is no longer honored in pytest-asyncio 1.x. Replace it with `asyncio_default_fixture_loop_scope` and `asyncio_default_test_loop_scope` set to session in pytest.ini, and convert async_sandbox_factory to an async fixture that yields and awaits sandbox cleanup directly instead of calling run_until_complete from a sync finalizer. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
CI installs Poetry from .tool-versions (currently 1.8.5). Regenerating the locks under Poetry 2.1.1 produced lock-version 2.1 with Poetry-2.x- only metadata, which makes Poetry 1.8.5 fall back to a re-resolve and defeats the deterministic install this PR depends on. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Use asyncio.gather(..., return_exceptions=True) so a failure killing one sandbox doesn't skip cleanup of the others, matching the independence the previous per-finalizer approach provided. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit b529018. Configure here.
Restore chart_data_extractor/poetry.lock to its Poetry 2.1.1 form (matches main; pydantic stays at 2.9.2). Rebuild python/poetry.lock from main with poetry lock --no-update + targeted poetry update for the vulnerable packages, so runtime deps (attrs, e2b, httpx) stay on their existing pins instead of pulling latest. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Summary
Resolves all 12 open Dependabot alerts by bumping vulnerable dev/transitive dependencies across the npm and Python projects.
All updates affect dev or transitive-dev dependencies of the published packages, so no changeset is needed (matching prior security-bump precedent).
Test plan
pnpm buildinjs/(verified locally)🤖 Generated with Claude Code