FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix(openai): Gate `gen_ai.response.tool_calls` on outputs, not inputs by ericapisani · Pull Request #7205 · getsentry/sentry-python · GitHub

fix(openai): Gate gen_ai.response.tool_calls on outputs, not inputs - #7205

Merged
ericapisani merged 1 commit into
py-2734-update-tool-definition-attr-langchainfrom
py-2734-update-tool-attr-openai
Aug 24, 2026
Merged

fix(openai): Gate gen_ai.response.tool_calls on outputs, not inputs#7205
ericapisani merged 1 commit into
py-2734-update-tool-definition-attr-langchainfrom
py-2734-update-tool-attr-openai

Conversation

ericapisani commented Aug 14, 2026
edited
Loading

Copy link
Copy Markdown
Member

Tool call data is part of the model's response, so it should be recorded when data_collection.gen_ai.outputs is enabled, not gated on inputs.

Refs PY-2734
Refs #7200

Tool call data is part of the model's response, so it should be
recorded when `data_collection.gen_ai.outputs` is enabled, not gated
on `inputs`. Previously, disabling outputs while leaving inputs
enabled still collected tool calls, and disabling inputs alone
dropped them even though outputs remained enabled.

Refs PY-2734
Refs #7200
ericapisani requested a review from a team as a code owner August 14, 2026 13:43

linear-code Bot commented Aug 14, 2026

Copy link
Copy Markdown

PY-2734

Copy link
Copy Markdown
Contributor

Codecov Results 📊

113171 passed | ⏭️ 6721 skipped | Total: 119892 | Pass Rate: 94.39% | Execution Time: 413m 2s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -31
Passed Tests 📈 +14
Failed Tests 📉 -1
Skipped Tests 📉 -44

➖ Removed Tests (1)

View removed tests
  • test_span_streaming_quart_auth_user_id[42-True]
    • File: tests.integrations.quart.test_quart

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2483 uncovered lines.
✅ Project coverage is 90.23%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    90.20%    90.23%    +0.03%
==========================================
  Files          193       193         —
  Lines        25423     25421        -2
  Branches      9336      9336         —
==========================================
+ Hits         22933     22938        +5
- Misses        2490      2483        -7
- Partials      1436      1437        +1

Generated by Codecov Action

ericapisani merged commit 14d2cd0 into master Aug 24, 2026
137 checks passed
ericapisani deleted the py-2734-update-tool-attr-openai branch August 24, 2026 11:41
sentrivana pushed a commit that referenced this pull request Aug 25, 2026
…#7205)

Tool call data is part of the model's response, so it should be recorded
when `data_collection.gen_ai.outputs` is enabled, not gated on `inputs`.

Refs PY-2734
Refs #7200
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL