fix: forward envs through _cls_connect_sandbox to prevent TypeError

Sandbox.connect(sandbox_id, envs=...) routed through _cls_connect_sandbox,
which lacked an explicit envs param. envs fell into **opts and was later
passed to ConnectionConfig(**opts), raising TypeError on the class-call path.

Added envs as an explicit param to both sync and async _cls_connect_sandbox
so it is forwarded to _cls_connect and kept out of the opts spread into
ConnectionConfig. Added regression tests covering the class-call path.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: FisherXZ

packages/python-sdk/e2b/sandbox_async/main.py

@@ -862,11 +862,13 @@ async def _cls_connect_sandbox(
         cls,
         sandbox_id: str,
         timeout: Optional[int] = None,
+        envs: Optional[Dict[str, str]] = None,
         **opts: Unpack[ApiParams],
     ) -> Self:
         sandbox = await SandboxApi._cls_connect(
             sandbox_id=sandbox_id,
             timeout=timeout,
+            envs=envs,
             **opts,
         )
 

packages/python-sdk/e2b/sandbox_sync/main.py

@@ -857,11 +857,13 @@ def _cls_connect_sandbox(
         cls,
         sandbox_id: str,
         timeout: Optional[int] = None,
+        envs: Optional[Dict[str, str]] = None,
         **opts: Unpack[ApiParams],
     ) -> Self:
         sandbox = SandboxApi._cls_connect(
             sandbox_id=sandbox_id,
             timeout=timeout,
+            envs=envs,
             **opts,
         )
 

packages/python-sdk/tests/async/sandbox_async/test_config_propagation.py

@@ -91,6 +91,34 @@ async def test_connect_envs_is_none_when_not_provided(monkeypatch):
     assert mock_connect.call_args.kwargs.get("envs") is None
 
 
+@pytest.mark.skip_debug()
+async def test_classmethod_connect_forwards_envs_without_polluting_opts(monkeypatch):
+    # Regression: AsyncSandbox.connect(id, envs=...) previously passed envs into
+    # **opts, which then reached ConnectionConfig(**opts) and raised TypeError.
+    mock_cls_connect = AsyncMock(return_value=SimpleNamespace(
+        sandbox_id="sbx-test",
+        domain="sandbox.e2b.dev",
+        envd_version="0.2.4",
+        envd_access_token="tok",
+        traffic_access_token="tok",
+    ))
+    monkeypatch.setattr(sandbox_async_main.SandboxApi, "_cls_connect", mock_cls_connect)
+    monkeypatch.setattr(
+        sandbox_async_main, "get_transport", lambda *_args, **_kwargs: SimpleNamespace(pool=object())
+    )
+    monkeypatch.setattr(sandbox_async_main.httpx, "AsyncClient", lambda *args, **kwargs: object())
+    monkeypatch.setattr(sandbox_async_main, "Filesystem", lambda *args, **kwargs: object())
+    monkeypatch.setattr(sandbox_async_main, "Commands", lambda *args, **kwargs: object())
+    monkeypatch.setattr(sandbox_async_main, "Pty", lambda *args, **kwargs: object())
+    monkeypatch.setattr(sandbox_async_main, "Git", lambda *args, **kwargs: object())
+
+    # This must not raise TypeError: ConnectionConfig() got unexpected keyword 'envs'
+    await sandbox_async_main.AsyncSandbox.connect("sbx-test", envs={"MY_KEY": "my_value"}, api_key="test-key")
+
+    mock_cls_connect.assert_awaited_once()
+    assert mock_cls_connect.call_args.kwargs["envs"] == {"MY_KEY": "my_value"}
+
+
 @pytest.mark.skip_debug()
 async def test_pause_applies_overrides(monkeypatch):
     mock_pause = AsyncMock(return_value="sbx-test")

packages/python-sdk/tests/sync/sandbox_sync/test_config_propagation.py

@@ -89,6 +89,34 @@ def test_connect_envs_is_none_when_not_provided(monkeypatch):
     assert mock_connect.call_args.kwargs.get("envs") is None
 
 
+@pytest.mark.skip_debug()
+def test_classmethod_connect_forwards_envs_without_polluting_opts(monkeypatch):
+    # Regression: Sandbox.connect(id, envs=...) previously passed envs into
+    # **opts, which then reached ConnectionConfig(**opts) and raised TypeError.
+    mock_cls_connect = Mock(return_value=SimpleNamespace(
+        sandbox_id="sbx-test",
+        domain="sandbox.e2b.dev",
+        envd_version="0.2.4",
+        envd_access_token="tok",
+        traffic_access_token="tok",
+    ))
+    monkeypatch.setattr(sandbox_sync_main.SandboxApi, "_cls_connect", mock_cls_connect)
+    monkeypatch.setattr(
+        sandbox_sync_main, "get_transport", lambda *_args, **_kwargs: SimpleNamespace(pool=object())
+    )
+    monkeypatch.setattr(sandbox_sync_main.httpx, "Client", lambda *args, **kwargs: object())
+    monkeypatch.setattr(sandbox_sync_main, "Filesystem", lambda *args, **kwargs: object())
+    monkeypatch.setattr(sandbox_sync_main, "Commands", lambda *args, **kwargs: object())
+    monkeypatch.setattr(sandbox_sync_main, "Pty", lambda *args, **kwargs: object())
+    monkeypatch.setattr(sandbox_sync_main, "Git", lambda *args, **kwargs: object())
+
+    # This must not raise TypeError: ConnectionConfig() got unexpected keyword 'envs'
+    sandbox_sync_main.Sandbox.connect("sbx-test", envs={"MY_KEY": "my_value"}, api_key="test-key")
+
+    mock_cls_connect.assert_called_once()
+    assert mock_cls_connect.call_args.kwargs["envs"] == {"MY_KEY": "my_value"}
+
+
 @pytest.mark.skip_debug()
 def test_pause_applies_overrides(monkeypatch):
     mock_pause = Mock(return_value="sbx-test")