Skip to content

ref(openai-agents): Stop setting tokens on Invoke Agent spans - #7515

Merged
alexander-alderman-webb merged 1 commit into
major/3.0from
webb/openai-agents/stop-setting-tokens
Sep 15, 2026
Merged

alexander-alderman-webb merged 1 commit into
major/3.0from
webb/openai-agents/stop-setting-tokens

Conversation

@alexander-alderman-webb

Copy link
Copy Markdown
Collaborator

Description

Remove token attributes from Invoke Agent spans because it is ambiguous whether they include usage from tool calls.

Issues

Reminders

@alexander-alderman-webb
alexander-alderman-webb requested a review from a team as a code owner September 15, 2026 13:07
@alexander-alderman-webb alexander-alderman-webb changed the title ref(openai-agents): Stop settings tokens on Invoke Agent spans ref(openai-agents): Stop setting tokens on Invoke Agent spans Sep 15, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Codecov Results 📊

✅ 63917 passed | ❌ 1 failed | ⏭️ 2998 skipped | Total: 66916 | Pass Rate: 95.52% | Execution Time: 182m 24s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -70790
Passed Tests 📉 -66667
Failed Tests 📈 +1
Skipped Tests 📉 -4124

➕ New Tests (1)

View new tests
  • test_async_transport_rate_limiting_with_concurrency
    • File: tests.test_transport
    • Status: ❌ Failing

❌ Failed Tests

test_async_transport_rate_limiting_with_concurrency

File: tests.test_transport
Suite: py3.8-common
Error: AssertionError: assert False is True + where False = _check_disabled('error') + where _check_disabled = <sentry_sdk.transport.AsyncHttpTransport object at 0x7f817c7808e0>._check_disabled + where <sentry_sdk.transport.AsyncHttpTransport object at 0x7f817c7808e0> = <sentry_sdk.client._Client object at 0x7f818c45fc40>.transport

Stack Trace
tests/test_transport.py:1037: in test_async_transport_rate_limiting_with_concurrency
    assert client.transport._check_disabled("error") is True
E   AssertionError: assert False is True
E    +  where False = _check_disabled('error')
E    +    where _check_disabled = <sentry_sdk.transport.AsyncHttpTransport object at 0x7f817c7808e0>._check_disabled
E    +      where <sentry_sdk.transport.AsyncHttpTransport object at 0x7f817c7808e0> = <sentry_sdk.client._Client object at 0x7f818c45fc40>.transport

❌ Patch coverage is 62.50%. Project has 2080 uncovered lines.

Files with missing lines (1)
File Patch % Lines
sentry_sdk/integrations/openai_agents/patches/agent_run.py 57.14% ⚠️ 3 Missing

Generated by Codecov Action

@alexander-alderman-webb
alexander-alderman-webb merged commit 93ba5f1 into major/3.0 Sep 15, 2026
144 of 149 checks passed
@alexander-alderman-webb
alexander-alderman-webb deleted the webb/openai-agents/stop-setting-tokens branch September 15, 2026 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants