Skip to content

chore(compose): Rename file for SentryTraced - #6059

Open
0xadam-brown wants to merge 1 commit into
fix/sentry-traced-origin-filtering-bugfrom
chore/new-file-for-sentry-traced
Open

chore(compose): Rename file for SentryTraced#6059
0xadam-brown wants to merge 1 commit into
fix/sentry-traced-origin-filtering-bugfrom
chore/new-file-for-sentry-traced

Conversation

@0xadam-brown

@0xadam-brown 0xadam-brown commented Sep 5, 2026

Copy link
Copy Markdown
Member

📜 Description

Renamed SentryComposeTracing.kt -> SentryTraced.kt to match our usual conventions + make it easier to find SentryTraced in the IDE.

Rename keeps binary compatibility with previous file name for Java clients.

💡 Motivation and Context

Having file names match public composable APIs is especially helpful b/c composables are functions, not classes, so most IDEs don't have a type-based global search for them (but do let you search files by name).

💚 How did you test it?

📝 Checklist

  • I added GH Issue ID & Linear ID
  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.
  • Public API changes reviewed by another Mobile SDK team member or implemented according to the develop docs spec.

🔮 Next steps

#skip-changelog

Note: Rename keeps binary compatibility with previous file name for Java clients.
@@ -1,3 +1,5 @@
@file:JvmName("SentryComposeTracingKt")

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Note: SDK Size Analysis is failing because this rename preserves the old top-level JVM class name via @file:JvmName("SentryComposeTracingKt"), and the size-test app’s R8 config strips Kotlin runtime deps so the shrinker can’t resolve kotlin.jvm.JvmName.

This is a size-test classpath quirk, not a production/runtime breakage for Java callers.

@0xadam-brown 0xadam-brown added the sanity-check PR needs a lightweight review for obvious issues label Sep 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sanity-check PR needs a lightweight review for obvious issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants