Skip to content

fix(ai): Gate response tool calls on the gen_ai inputs setting - #7174

Open
ericapisani wants to merge 1 commit into
masterfrom
py-2588-tool-calls-inputs
Open

fix(ai): Gate response tool calls on the gen_ai inputs setting#7174
ericapisani wants to merge 1 commit into
masterfrom
py-2588-tool-calls-inputs

Conversation

@ericapisani

Copy link
Copy Markdown
Member

gen_ai.response.tool_calls was gated on data_collection.gen_ai.outputs in the OpenAI, Anthropic, OpenAI Agents, and Google GenAI integrations. Tool calls belong to the inputs category, so they are now gated on data_collection.gen_ai.inputs while response text stays on outputs.

Where both attributes previously shared a single gate, the block is split so each is evaluated independently. The legacy send_default_pii/include_prompts path is unchanged.

Refs PY-2588
Refs #6748

`gen_ai.response.tool_calls` was gated on `data_collection.gen_ai.outputs` in the
OpenAI, Anthropic, OpenAI Agents, and Google GenAI integrations. Tool calls belong
to the inputs category, so they are now gated on `data_collection.gen_ai.inputs`
while response text stays on `outputs`.

Where both attributes previously shared a single gate, the block is split so each
is evaluated independently. The legacy `send_default_pii`/`include_prompts` path is
unchanged.

Refs PY-2588
Refs #6748
@ericapisani
ericapisani requested a review from a team as a code owner August 11, 2026 18:56
@linear-code

linear-code Bot commented Aug 11, 2026

Copy link
Copy Markdown

PY-2588

@github-actions

Copy link
Copy Markdown
Contributor

Codecov Results 📊

104810 passed | ⏭️ 6749 skipped | Total: 111559 | Pass Rate: 93.95% | Execution Time: 361m 17s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +480
Passed Tests 📈 +408
Failed Tests
Skipped Tests 📈 +72

All tests are passing successfully.

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

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    90.14%    90.14%        —%
==========================================
  Files          193       193         —
  Lines        25183     25192        +9
  Branches      9176      9174        -2
==========================================
+ Hits         22700     22709        +9
- Misses        2483      2483         —
- Partials      1429      1428        -1

Generated by Codecov Action

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.

1 participant