Skip to content

fix(pydantic-ai): Stop setting tokens on Invoke Agent spans#6320

Merged
alexander-alderman-webb merged 2 commits into
masterfrom
webb/pydantic-ai/remove-agent-tokens
May 21, 2026
Merged

fix(pydantic-ai): Stop setting tokens on Invoke Agent spans#6320
alexander-alderman-webb merged 2 commits into
masterfrom
webb/pydantic-ai/remove-agent-tokens

Conversation

@alexander-alderman-webb
Copy link
Copy Markdown
Contributor

@alexander-alderman-webb alexander-alderman-webb commented May 21, 2026

Description

Token attribute are not attached on agent spans per the RFC below, due to ambiguity about whether to include token counts consumed by subagents and tool calls.

https://github.com/getsentry/rfcs/blob/main/text/0153-decoupling-sentrys-generative-ai-conventions-from-open-telemetry.md

Issues

Reminders

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 21, 2026

Codecov Results 📊

282 passed | Total: 282 | Pass Rate: 100% | Execution Time: 43.76s

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 15017 uncovered lines.

Files with missing lines (1)
File Patch % Lines
invoke_agent.py 0.00% ⚠️ 75 Missing

Generated by Codecov Action

@alexander-alderman-webb alexander-alderman-webb marked this pull request as ready for review May 21, 2026 14:11
@alexander-alderman-webb alexander-alderman-webb requested a review from a team as a code owner May 21, 2026 14:11
Copy link
Copy Markdown
Member

@ericapisani ericapisani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔥

@alexander-alderman-webb alexander-alderman-webb merged commit b1e999d into master May 21, 2026
157 checks passed
@alexander-alderman-webb alexander-alderman-webb deleted the webb/pydantic-ai/remove-agent-tokens branch May 21, 2026 16:42
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.

3 participants