test(python-sdk): parallelize async sandbox teardown

[mishushakov]

Summary

• Replace the sequential for sandbox in sandboxes: await sandbox.kill() teardown in async_sandbox_factory with asyncio.gather(..., return_exceptions=True) so a failure on one kill no longer blocks cleanup of the rest, and teardown runs concurrently.
• Hoist the _test_failed check above the loop so the failure log is printed once per fixture rather than per sandbox.

Test plan

• pnpm run format, pnpm run lint, pnpm run typecheck (python-sdk) all pass
• pnpm run test against affected python async sandbox tests

🤖 Generated with Claude Code

[changeset-bot]

⚠️ No Changeset found

Latest commit: 69cc8b5

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[cursor]

PR Summary

Low Risk
Low risk test-fixture change, but teardown errors are now swallowed and only printed, which could hide cleanup issues in CI output.

Overview
Async sandbox teardown now uses asyncio.gather(..., return_exceptions=True) so one kill() failure won’t block cleanup of other sandboxes, and it logs any teardown exceptions separately; test-failure logging is gated once per fixture instead of inside the kill loop.

^{Reviewed by Cursor Bugbot for commit 69cc8b5. Bugbot is set up for automated code reviews on this repo. Configure here.}

[github-actions]

Package Artifacts

Built from d68b2d7. Download artifacts from this workflow run.

JS SDK (e2b@2.20.1-mishushakov-async-teardown-fix.0):

npm install ./e2b-2.20.1-mishushakov-async-teardown-fix.0.tgz

CLI (@e2b/cli@2.10.2-mishushakov-async-teardown-fix.0):

npm install ./e2b-cli-2.10.2-mishushakov-async-teardown-fix.0.tgz

Python SDK (e2b==2.21.0+mishushakov-async-teardown-fix):

pip install ./e2b-2.21.0+mishushakov.async.teardown.fix-py3-none-any.whl

[claude]

LGTM — straightforward test fixture cleanup parallelization.

Extended reasoning...

Overview

This PR modifies a single test fixture file (packages/python-sdk/tests/conftest.py) in the async_sandbox_factory fixture. It replaces a sequential for sandbox in sandboxes: await sandbox.kill() teardown loop with asyncio.gather(..., return_exceptions=True) for concurrent cleanup, and hoists the _test_failed check above the loop so the failure log prints once per fixture rather than per sandbox.

Security risks

None. This is test-only code (tests/conftest.py), not shipped or used in production. There is no auth, crypto, or permissions logic involved.

Level of scrutiny

Low scrutiny is appropriate. The change is a small, mechanical optimization to test teardown logic. return_exceptions=True preserves the previous swallow-errors behavior of the original try/except, so semantics are preserved while gaining concurrency. The hoisted _test_failed check is a clear correctness improvement (avoids duplicated log lines).

Other factors

The change is self-contained, limited to ~10 lines, and follows an established pattern. The changeset-bot warning about no changeset is expected and acceptable for a test-only change. No bugs were flagged by the bug hunting system.