Skip to content

chore(android): Add Nav2 sample app - #6037

Merged
0xadam-brown merged 4 commits into
mainfrom
chore/nav2-sample-app
Sep 1, 2026
Merged

chore(android): Add Nav2 sample app#6037
0xadam-brown merged 4 commits into
mainfrom
chore/nav2-sample-app

Conversation

@0xadam-brown

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

Copy link
Copy Markdown
Member

This is a big PR, but it's also purely sample code. Best way to review is to check out the branch, build the sample app, and see what the new functionality can do (cf. the video below).

📜 Description

Adds a Nav2 sample app for validating Sentry's fragment and Compose existing NavigationController integrations under different tracing setups.

Developers can configure active transactions (nav vs ui.load vs user interaction), navigation data (breadcrumbs and screen names), and type of spans to be generated at each nav destination (HTTP requests, manual spans, or none). They can also view transaction/span generation in real time via the Transaction History Bottom Sheet. A performance tab lets folks stress test our nav integration in the presence of large back stacks, frequent recompositions, etc.

💡 Motivation and Context

Milestone 1 of our Nav3 support aims to match what we currently provide under Nav2. The sample app helped me figure out what precisely needs to be included and how it should show up in the Sentry backend. It also helped me uncover some existing bugs (🎉), track behavioral differences against the Nav3 sample app (forthcoming), and compare performance characteristics.

It's sure to be useful going forward too.

supports: JAVA-274

In action

nav2-pr-demo-v2.mp4

Real-time transactions and their spans + URLs to the actual Sentry UI:

real-time-transactions

Transactions (and other features) you can toggle on and off:

setup-2

💚 How did you test it?

Manually and with my clanker.

📝 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

@sentry

sentry Bot commented Sep 1, 2026

Copy link
Copy Markdown

📲 Install Builds

Android

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

⚙️ sentry-android Build Distribution Settings

Adds a Nav2 sample app for validating Sentry's fragment and Compose navigation integrations under different tracing setups.

Developers can configure active transactions (nav vs ui.load vs user interaction), navigation data (breadcrumbs and screen names), and work performed at each nav destination (HTTP requets, manual spans, or nothing). They can also view transaction/span generation in real time via the Transaction History Bottom Sheet.

Co-Authored-By: OpenAI GPT-5.4 <noreply@openai.com>
@0xadam-brown
0xadam-brown marked this pull request as ready for review September 1, 2026 11:53

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 3 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 37dac13. Configure here.

@0xadam-brown 0xadam-brown added the ship-it PR is ready to merge from a reviewer perspective label Sep 1, 2026

@runningcode runningcode left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Sorry I misunderstood that this was a new sample app. Since it is in the existing sample app i'm approving it.
Two optional comments.

@0xadam-brown
0xadam-brown enabled auto-merge (squash) September 1, 2026 18:38
@0xadam-brown
0xadam-brown merged commit 5183e3a into main Sep 1, 2026
69 checks passed
@0xadam-brown
0xadam-brown deleted the chore/nav2-sample-app branch September 1, 2026 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ship-it PR is ready to merge from a reviewer perspective

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants