Async streaming response wrappers leak an unawaited coroutine when not entered

Aperta Adatta ai principianti
#175 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
82/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Tranquilla
Stack tecnologico
python
Ambito
api

Direzione di ricerca

Inizia in src/browserbase/_response.py:632-637 e 669-685, confrontando AsyncResponseContextManager con il ResponseContextManager sincrono. Riproduci il caso dell’issue in cui il manager viene scartato senza entrarvi, quindi aggiungi un test del ciclo di vita che mostri che non viene emesso alcun avviso relativo a una coroutine non attesa e che l’ingresso nel manager esegue comunque la richiesta prevista.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Description

The async .with_streaming_response wrappers create the endpoint coroutine immediately, before the returned context manager is entered. If the context manager is conditionally discarded or never entered, Python emits RuntimeWarning: coroutine ... was never awaited.

This differs from the synchronous wrapper, which stores a callable and does not invoke the endpoint until __enter__.

Code reference

  • src/browserbase/_response.py:669-685
  • AsyncResponseContextManager stores an already-created Awaitable at src/browserbase/_response.py:632-637

Reproduction

import asyncio
import gc
import warnings

from browserbase import AsyncBrowserbase

async def main():
    client = AsyncBrowserbase(api_key="test")
    with warnings.catch_warnings(record=True) as caught:
        warnings.simplefilter("always")
        cm = client.sessions.with_streaming_response.create(project_id="p")
        del cm
        gc.collect()
        await asyncio.sleep(0)
        print([str(w.message) for w in caught])
    await client.close()

asyncio.run(main())

Actual output includes:

coroutine 'AsyncSessionsResource.create' was never awaited

No HTTP request is needed to reproduce this.

Expected behavior

Creating and discarding an unentered response context manager should not allocate an unawaited coroutine or emit a resource warning. The async manager could retain a callable/partial and invoke it lazily from __aenter__, matching ResponseContextManager.

Why it matters

Warnings are promoted to errors in many test suites, and conditional streaming code can produce noisy, nondeterministic resource warnings. This path currently has no direct lifecycle test.

Lingua principale
Python
Stelle
93
Fork
16
Merge medio
11m
PR unite (30g)
3

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di browserbase/sdk-python

Tutte le issue di browserbase/sdk-python

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.