Skip to content

feat(trace-item-stats): Hide internal attributes#116380

Open
nsdeschenes wants to merge 2 commits into
masterfrom
nd/EXP-975/feat-trace-item-stats-hide-internal-attributes
Open

feat(trace-item-stats): Hide internal attributes#116380
nsdeschenes wants to merge 2 commits into
masterfrom
nd/EXP-975/feat-trace-item-stats-hide-internal-attributes

Conversation

@nsdeschenes
Copy link
Copy Markdown
Contributor

The goal of this PR is to remove "internal" attributes as defined in the sentry conventions package.

Closes EXP-975

nsdeschenes and others added 2 commits May 28, 2026 11:08
Filter generated trace item stats distribution keys through the API attribute visibility helper before querying final distributions.

Refs EXP-975

Co-Authored-By: Codex <noreply@openai.com>
Add endpoint coverage for filtering hidden attributes out of generated trace item attribute distributions.

Refs EXP-975

Co-Authored-By: Codex <noreply@openai.com>
@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 28, 2026

EXP-975

@nsdeschenes
Copy link
Copy Markdown
Contributor Author

@cursor review

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label May 28, 2026
Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 47f7a7f. Configure here.

@nsdeschenes nsdeschenes marked this pull request as ready for review May 28, 2026 14:27
@nsdeschenes nsdeschenes requested review from a team as code owners May 28, 2026 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant