Skip to content

fix(navigation): show Assistant sessions in the sidebar and repo switcher - #368

Merged
chriswritescode-dev merged 1 commit into
mainfrom
fix/sidebar-assistant-sessions
Sep 29, 2026
Merged

fix(navigation): show Assistant sessions in the sidebar and repo switcher#368
chriswritescode-dev merged 1 commit into
mainfrom
fix/sidebar-assistant-sessions

Conversation

@chriswritescode-dev

@chriswritescode-dev chriswritescode-dev commented Sep 29, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Owner

Problem

The desktop sidebar session tree and the repo switcher filtered out the Assistant, so its sessions were only reachable from the Assistant page.

Changes

  • Pin the Assistant as the first repo group in the sidebar and repo switcher; other repos keep last-accessed ordering.
  • Display the Assistant repo as "Assistant" via the shared repo display-name helper.
  • getSessionPath appends ?assistant=1 for Assistant sessions; new getRepoPath routes the Assistant to /assistant.
  • Expand and highlight the Assistant group on Assistant routes.
  • Reuse the shared helpers in AssistantRedirect and GlobalSchedules instead of building paths by hand.

Testing

  • Frontend: navigation, sidebar, and page tests pass (258 tests).
  • Backend: full suite passes (2407 tests).
  • pnpm lint -> 0 errors (pre-existing no-explicit-any warnings only).

Summary by CodeRabbit

  • Navigation
    • The Assistant now appears first in the repository quick switcher and session navigation lists.
    • Assistant sessions and repository links now open the correct Assistant destinations, including from schedules.
    • Repository navigation consistently directs you to the appropriate destination, while other repositories retain their existing paths.
  • Bug Fixes
    • Assistant routes are now recognized correctly when identifying the active repository and matching the current session.

@coderabbitai

coderabbitai Bot commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Advanced

Run ID: b6426050-45e5-4df9-82b6-6302ff7693ce

📥 Commits

Reviewing files that changed from the base of the PR and between 68b13c8 and 3e5a20f.

📒 Files selected for processing (11)
  • frontend/src/components/navigation/DesktopSessionTree.tsx
  • frontend/src/components/navigation/RepoQuickSwitchSheet.test.tsx
  • frontend/src/components/navigation/RepoQuickSwitchSheet.tsx
  • frontend/src/components/navigation/RepoSessionNav.tsx
  • frontend/src/components/navigation/sidebar-session-tree.test.ts
  • frontend/src/components/navigation/sidebar-session-tree.ts
  • frontend/src/lib/navigation.test.ts
  • frontend/src/lib/navigation.ts
  • frontend/src/pages/AssistantRedirect.tsx
  • frontend/src/pages/GlobalSchedules.tsx
  • shared/src/utils/repo.ts
 _____________________________________________
< I'm not sure if this is a bug or a feature. >
 ---------------------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       / 　 づ
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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

Comment @coderabbitai help to get the list of available commands.

@chriswritescode-dev
chriswritescode-dev merged commit e80016c into main Sep 29, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@chriswritescode-dev