Skip to content

feat(android-nav3): [Android Nav3 2] Model navigation routes - #6107

Merged
0xadam-brown merged 1 commit into
feat/android-nav3-module-stackfrom
feat/android-nav3-module-2-routes
Sep 16, 2026
Merged

0xadam-brown merged 1 commit into
feat/android-nav3-module-stackfrom
feat/android-nav3-module-2-routes

Conversation

@0xadam-brown

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

Copy link
Copy Markdown
Member

PR Stack (Android Nav3)


📜 Description

Add the route modeling layer for Android Nav3.

This introduces the code that turns app-owned back stack entries into normalized route names and
sanitized argument payloads, along with focused tests for those translation rules.

💡 Motivation and Context

The rest of the Nav3 integration depends on having one place that defines what a route looks like
and which argument shapes are safe to propagate into Sentry state.

💚 How did you test it?

  • ./gradlew :sentry-android-navigation3:testReleaseUnitTest
  • ./gradlew spotlessApply
  • ./gradlew apiDump

📝 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

Build the observer layer that consumes these routes and updates Sentry state.

#skip-changelog

⚠️ Merge this PR using a merge commit (not squash). Only the collection branch is squash-merged into main.

@github-actions

github-actions Bot commented Sep 12, 2026

Copy link
Copy Markdown
Contributor
Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against c9f806c

@sentry

sentry Bot commented Sep 12, 2026

Copy link
Copy Markdown

📲 Install Builds

Android

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

⚙️ sentry-android Build Distribution Settings

@0xadam-brown 0xadam-brown added the deep-dive PR needs a thorough review of design, behavior, and edge cases label Sep 12, 2026
@0xadam-brown
0xadam-brown force-pushed the feat/android-nav3-module-1-module branch from 99c3a91 to 6c2b8f9 Compare September 16, 2026 03:29
Introduce the route translation layer for Navigation 3 so later stages can reason about route names and sanitized arguments in one place. At this stage the module still exposes no meaningful public API surface.
@0xadam-brown
0xadam-brown force-pushed the feat/android-nav3-module-2-routes branch from addea5d to c9f806c Compare September 16, 2026 03:31
Base automatically changed from feat/android-nav3-module-1-module to feat/android-nav3-module-stack September 16, 2026 03:31
@0xadam-brown
0xadam-brown merged commit c9f806c into feat/android-nav3-module-stack Sep 16, 2026
@0xadam-brown
0xadam-brown deleted the feat/android-nav3-module-2-routes branch September 16, 2026 03:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deep-dive PR needs a thorough review of design, behavior, and edge cases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant