feat: Navigate to Reader post detail view deep links

[dcalhoun]

Description

Ref CMM-1005. Improve the deep link experience by navigating to an individual post detail view rather than the top-level Reader view.

Testing instructions

See Automattic/wp-calypso#107407.

reader-deep-links.mp4

[dangermattic]

	1 Warning
⚠️	PR is not assigned to a milestone.

Generated by 🚫 Danger

[wpmobilebot]

📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.

	App Name	Jetpack
	Flavor	Jalapeno
	Build Type	Debug
	Version	pr22380-08f5081
	Commit	08f5081
	Direct Download	jetpack-prototype-build-pr22380-08f5081.apk

Note: Google Login is not supported on these builds.

[wpmobilebot]

📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.

	App Name	WordPress
	Flavor	Jalapeno
	Build Type	Debug
	Version	pr22380-08f5081
	Commit	08f5081
	Direct Download	wordpress-prototype-build-pr22380-08f5081.apk

Note: Google Login is not supported on these builds.

[codecov]

Codecov Report

❌ Patch coverage is 89.65517% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 39.03%. Comparing base (2be71e4) to head (08f5081).

Files with missing lines	Patch %	Lines
...android/ui/deeplinks/handlers/ReaderLinkHandler.kt	91.30%	0 Missing and 2 partials ⚠️
...ordpress/android/ui/deeplinks/DeepLinkNavigator.kt	83.33%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##            trunk   #22380      +/-   ##
==========================================
+ Coverage   39.02%   39.03%   +0.01%     
==========================================
  Files        2203     2203              
  Lines      106351   106376      +25     
  Branches    15061    15067       +6     
==========================================
+ Hits        41501    41523      +22     
- Misses      61359    61360       +1     
- Partials     3491     3493       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[dcalhoun]

From what I understand of DEEP_LINK_HOST_READ and DEEP_LINK_HOST_VIEWPOST, these changes are sound and do not introduce regressions. However, I am less familiar with this code and welcome additional review and testing.

[dcalhoun]

@adalpari I just discovered #22374. This PR undoubtedly conflicts with efforts there. Let me know if you think these changes are worth keeping in some form or if they should be discarded.

[adalpari]

@adalpari I just discovered #22374. This PR undoubtedly conflicts with efforts there. Let me know if you think these changes are worth keeping in some form or if they should be discarded.

Definitely, there seems to be some collisions. Also, the other PR is targeting the release branch and not trunk. So, merging both could be tricky.
I would say to wait on this one and see what extra changes are necessary after merging the general deelinks PR to support your changes in Automattic/wp-calypso#107407

[adalpari]

I've included the necessary changes in 3817aa1, so I think it's safe to close this PR

Thank you for taking care of it! :)