Skip to content

Conversation

VleemingM
Copy link
Contributor

@VleemingM VleemingM commented Sep 26, 2025

📜 Preserve modifier in SentryTraced

When enableUserInteractionTracing is true use the provided modifier.

💡 Motivation and Context

When using the enableUserInteractionTracing I noticed my Modifier was ignored. Looking at the git history for this line this looks like an unintended bug.

📝 Checklist

  • I added GH Issue ID & Linear ID
  • I added tests to verify the changes.
  • [ x] No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • [ x] I updated the docs if needed.
  • [ x] I updated the wizard if needed.
  • Review from the native team if needed.
  • [ x] No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.

🔮 Next steps


Note

Preserves the passed modifier in SentryTraced by chaining .sentryTag(tag) onto it when interaction tracing is enabled, and documents the fix in the changelog.

  • Compose (Jetpack):
    • Preserve caller-provided modifier in SentryTraced by using modifier.sentryTag(tag) instead of replacing it with Modifier.sentryTag(tag).
  • Changelog:
    • Add fix entry for preserving modifiers in SentryTraced under Unreleased > Fixes.

Written by Cursor Bugbot for commit ea42e11. This will update automatically on new commits. Configure here.

@markushi
Copy link
Member

@VleemingM Thanks for your contribution, much appreciated!

@markushi markushi merged commit b66ccf3 into getsentry:main Sep 29, 2025
61 checks passed
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