Async streaming response wrappers leak an unawaited coroutine when not entered

オープン 初心者向け
#175 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
82/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
静か
技術スタック
python
領域
api

調査の方向性

src/browserbase/_response.py:632-637 と 669-685 から始め、AsyncResponseContextManager と同期的な ResponseContextManager を比較します。Issue にある、manager を破棄したものの manager に入っていないケースを再現し、その後、await されていないコルーチンの警告が発生しないこと、および manager に入ると期待されるリクエストが引き続き実行されることを示すライフサイクルテストを追加します。

索引モデルが issue の本文から書いたものです。

説明

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.

主要言語
Python
スター
93
フォーク
16
平均マージ
11分
マージ済み PR(30日)
3

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

browserbase/sdk-python のほかの issue

browserbase/sdk-python の issue をすべて見る

似ている issue

Python の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。