🐛 Fixed 404 when navigating between post stats and editor

[troyciesco]

Closes ONC-1421: 404 when navigating between post analytics & editor

Problem

• Opening editor from analytics set a bad back path, so < Analytics could point to an invalid post URL and 404.

Solution

• In lexical-editor.js, derive the previous analytics path from transition route info instead of guessing.
• Handle both analytics families (posts-x, stats-x), including subroutes and query params.
• Store the resolved path in controller.fromAnalytics for existing posts only.
• Reset fromAnalytics on route reset to prevent stale breadcrumbs.
• Add acceptance coverage in editor-test.js for:
  • post analytics root/subpage
  • stats root/subpage/query params
  • new post (no analytics breadcrumb)
  • default posts breadcrumb fallback

[coderabbitai]

Walkthrough

The pull request refactors analytics route handling in the Lexical Editor and extends the editor's acceptance tests. The route definition introduces helper utilities and constants to classify and reconstruct analytics routes from transitions, implementing a new buildAnalyticsSourcePath function for analytics path construction. The route service now includes router and ui service declarations. The acceptance tests are expanded to cover multiple analytics breadcrumb scenarios, including breadcrumbs to post analytics root, post analytics subpages, analytics root, and analytics subpages with query parameters.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately reflects the main purpose of the PR—fixing a 404 error when navigating between post analytics and the editor—which aligns with the changes across both the route handler and test files.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Description check	✅ Passed	The pull request description clearly explains the problem being fixed, the solution implemented, and the acceptance test coverage added.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch ONC-1421_fix-post-stats-breadcrumb

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cmraible]

LGTM!