Skip to content

chore(deps): Bump navigation and compose UI deps - #6080

Merged
0xadam-brown merged 3 commits into
mainfrom
chore/bump-compose-deps
Sep 9, 2026
Merged

chore(deps): Bump navigation and compose UI deps#6080
0xadam-brown merged 3 commits into
mainfrom
chore/bump-compose-deps

Conversation

@0xadam-brown

Copy link
Copy Markdown
Member

📜 Description

PR bumps the (stale) Compose / Navigation baseline in sentry-compose enough to remove the old Robolectric test workaround discussed with @runningcode here.

The final versions are limited to a coherent set that fixes the issue and still behaves well:

  • androidxNavigation -> 2.9.8
  • androidxCompose -> 1.9.5
  • androidx.activity:activity-compose -> 1.9.3

I initially tried pushing Navigation Compose to 2.10.0, but that raised the sample app floor through navigation-runtime-android and broke the sample app install path (minSdk 23 vs 24).

💡 Motivation and Context

Let's us avoid introducing compose test deps all over the place just to keep Robolectric happy, and gets us off compose libraries from 2022.

⚠️ Callouts

  • PR doesn't change our published API or ABI surface.
  • Our compose deps are compileOnly, so consumers' dependency graphs won't be affected

💚 How did you test it?

Existing test suite still succeeds.

Compared runtime behavior pre- and post-bump via the sample app + golden flows. No observed differences.

📝 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-changlog

Align sentry-compose and related test surfaces on a newer AndroidX Compose and Navigation baseline. This removes the stale navigation-compose classpath that required explicit test-only foundation overrides and keeps the Compose test/runtime mix coherent again.

Co-Authored-By: OpenAI GPT-5.4 <noreply@openai.com>
Comment thread gradle/libs.versions.toml Outdated
@sentry

sentry Bot commented Sep 9, 2026

Copy link
Copy Markdown

📲 Install Builds

Android

🔗 App Name App ID Version Configuration
SDK Size io.sentry.tests.size 8.55.0 (1) release

⚙️ sentry-android Build Distribution Settings

Comment thread gradle/libs.versions.toml
…se 1.9.3

The androidx.activity bump to 1.9.3 changed RequestPermission's input
generic to non-null String, so registerForActivityResult now returns
ActivityResultLauncher<String>. Update the field type to match and fix
the compileReleaseKotlin break in sentry-uitest-android-critical.
@0xadam-brown 0xadam-brown added the sanity-check PR needs a lightweight review for obvious issues label Sep 9, 2026
@runningcode

Copy link
Copy Markdown
Contributor

YESSS!

@0xadam-brown
0xadam-brown merged commit 96297e6 into main Sep 9, 2026
70 checks passed
@0xadam-brown
0xadam-brown deleted the chore/bump-compose-deps branch September 9, 2026 09:16
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