fix(integrations): openai-agents fix multi-patching of get_model function

[constantinius]

Description

When passing in an explicit model instance instead of a string the model was patched multiple times event if it was already patched before. This PR prohibits that.

Issues

Contributes to: https://linear.app/getsentry/issue/TET-1511/openai-agents-double-span-reporting

[linear]

TET-1511 OpenAI Agents: double span reporting

[cursor]

Bug: Stale agent reference captured in closure when model reused

When a model instance is shared across multiple agents, the early return on line 37-38 prevents re-patching but causes wrapped_fetch_response and wrapped_get_response to use a stale agent reference from their closures. The closures capture the agent from the first call to wrapped_get_model, so subsequent calls with different agents will incorrectly write/read _sentry_raw_response_model on the wrong agent and create spans associated with the wrong agent. The wrapper functions need access to the current agent being used, not the one captured at patch time.

Additional Locations (2)

• sentry_sdk/integrations/openai_agents/patches/models.py#L43-L52
• sentry_sdk/integrations/openai_agents/patches/models.py#L55-L76

 

[alexander-alderman-webb]

this looks like a valid concern @constantinius

[codecov]

❌ 1 Tests Failed:

Tests completed	Failed	Passed	Skipped
29714	1	29713	2319

View the top 1 failed test(s) by shortest run time

tests.profiler.test_continuous_profiler::test_continuous_profiler_auto_start_and_stop_sampled[experiment-thread]

Stack Traces | 1.07s run time

tests/profiler/test_continuous_profiler.py:506: in test_continuous_profiler_auto_start_and_stop_sampled
    assert get_profiler_id() is None, "profiler should not be running"
E   AssertionError: profiler should not be running
E   assert '92f18ef36e9443faae5a4956573cc131' is None
E    +  where '92f18ef36e9443faae5a4956573cc131' = get_profiler_id()

To view more test analytics, go to the Test Analytics Dashboard
_{📋 Got 3 mins? Take this short survey to help us improve Test Analytics.}