refactor: show dashboard sidebar for inbox list view

[2witstudios]

Only display InboxSidebar when a conversation (DM or channel) is
opened. The inbox list view now uses the regular dashboard sidebar,
providing a consistent navigation experience until the user enters
a specific conversation.

https://claude.ai/code/session_01LJmnwrwXiXkP72zguowP3g

Summary by CodeRabbit

• Bug Fixes
  • Sidebar now correctly shows the inbox conversation view only for direct conversation routes while keeping the default sidebar for other pages.
• New Features
  • Replaced individual dashboard/inbox links with a consolidated primary navigation for main app sections.
  • Added a dashboard footer to the bottom of the left sidebar and adjusted layout so navigation and footer are positioned cleanly.

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

[coderabbitai]

📝 Walkthrough

Walkthrough

Narrows inbox-specific rendering and restructures left-sidebar navigation: InboxSidebar is now rendered only for direct conversation routes, and the InboxSidebar itself delegates main links to a new PrimaryNavigation and renders a DashboardFooter at the bottom.

Changes

Cohort / File(s)	Summary
Route detection apps/web/src/components/layout/left-sidebar/MemoizedSidebar.tsx	Replaced broad inbox-route check with a narrower condition that renders InboxSidebar only for direct conversation paths (/dashboard/inbox/dm/..., /dashboard/inbox/channel/...).
Sidebar composition apps/web/src/components/layout/left-sidebar/InboxSidebar.tsx	Removed explicit Dashboard/Inbox link items; added PrimaryNavigation to provide main links and DashboardFooter rendered at the bottom; adjusted layout to place primary navigation above the scrollable inbox list and footer below it.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• Refactor inbox to unified messaging interface with channels and DMs #345: Modifies the same left-sidebar files (MemoizedSidebar.tsx, InboxSidebar.tsx) and inbox route detection logic.
• feat(inbox): add unified inbox view with DMs and channels #327: Changes left-sidebar components and navigation composition overlapping PrimaryNavigation/footer integration.
• feat(mobile): add Dashboard link to sidebar for easy navigation #296: Also alters how the Dashboard link is surfaced in the left sidebar (mobile-focused), related to navigation rendering changes.

Poem

🐰 I hopped the codepath, light and merry,

Only conversations get my wary,
DMs and channels, clear and bright,
Navigation stacked just right,
Footer rests as day turns cheery.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main refactoring change: modifying when InboxSidebar is displayed versus when the dashboard sidebar is shown, specifically targeting the inbox list view scenario.

_{✏️ 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 claude/inbox-sidebar-refactor-dGiVc

---

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.}