Skip to content

feat(openai): Gate Chat Completions inputs behind data collection - #6965

Merged
ericapisani merged 5 commits into
py-2588-openai-responses-input-datafrom
py-2588-openai-completions-input-data
Jul 31, 2026
Merged

feat(openai): Gate Chat Completions inputs behind data collection#6965
ericapisani merged 5 commits into
py-2588-openai-responses-input-datafrom
py-2588-openai-completions-input-data

Conversation

@ericapisani

Copy link
Copy Markdown
Member

Apply data_collection.gen_ai.inputs to Chat Completions prompt messages,
system instructions, and tool definitions, matching the Responses API.
Preserve legacy send_default_pii behavior when data collection is not configured.

Refs PY-2588

Apply data_collection.gen_ai.inputs to Chat Completions prompt messages,
system instructions, and tool definitions, matching the Responses API.
Preserve legacy send_default_pii behavior when data collection is not configured.

Refs PY-2588
@linear-code

linear-code Bot commented Jul 30, 2026

Copy link
Copy Markdown

PY-2588

@ericapisani
ericapisani changed the base branch from master to py-2588-openai-responses-input-data July 30, 2026 18:14
@github-actions

github-actions Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

100250 passed | ⏭️ 7021 skipped | Total: 107271 | Pass Rate: 93.45% | Execution Time: 355m 29s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +10024
Passed Tests 📈 +9305
Failed Tests
Skipped Tests 📈 +719

All tests are passing successfully.

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

Files with missing lines (1)
File Patch % Lines
sentry_sdk/integrations/openai.py 100.00% ⚠️ 2 partials
Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    89.88%    89.92%    +0.04%
==========================================
  Files          193       193         —
  Lines        24747     24763       +16
  Branches      8858      8884       +26
==========================================
+ Hits         22242     22267       +25
- Misses        2505      2496        -9
- Partials      1416      1414        -2

Generated by Codecov Action

@ericapisani
ericapisani marked this pull request as ready for review July 30, 2026 18:34
@ericapisani
ericapisani requested a review from a team as a code owner July 30, 2026 18:34
Comment thread tests/integrations/openai/test_openai.py Outdated

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit c8c3d09. Configure here.

Comment thread tests/integrations/openai/test_openai.py Outdated
Comment thread tests/integrations/openai/test_openai.py Outdated
@ericapisani
ericapisani merged commit 75f8a27 into master Jul 31, 2026
145 checks passed
@ericapisani
ericapisani deleted the py-2588-openai-completions-input-data branch July 31, 2026 12:44
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