Skip to content

fix(desktop): restore sidebar sessions with projects - #58069

Closed
yahya3867 wants to merge 1 commit into
NousResearch:mainfrom
yahya3867:codex/fix-57968-sidebar-sessions-list
Closed

fix(desktop): restore sidebar sessions with projects#58069
yahya3867 wants to merge 1 commit into
NousResearch:mainfrom
yahya3867:codex/fix-57968-sidebar-sessions-list

Conversation

@yahya3867

Copy link
Copy Markdown

Summary

  • Restore the flat Sessions roster as its own section when the project-grouped sidebar is active.
  • Keep the Projects browser scoped to project overview/project content instead of replacing recents.
  • Add a regression test for the visibility rule so project overview keeps flat sessions available.

Fixes #57968.

Tests

  • npm run test:ui --workspace apps/desktop -- sidebar/visibility.test.ts
  • npm run test:ui --workspace apps/desktop -- sidebar
  • npm run build --workspace apps/desktop

Notes

@yahya3867
yahya3867 marked this pull request as ready for review July 4, 2026 04:25
@alt-glitch alt-glitch added type/bug Something isn't working comp/desktop Electron desktop app (apps/desktop/*) P3 Low — cosmetic, nice to have labels Jul 4, 2026
@teknium1

Copy link
Copy Markdown
Contributor

Thanks for the focused desktop sidebar fix. The premise remains valid on current main: ChatSidebar still makes its only section a Projects/Sessions switcher (apps/desktop/src/app/chat/sidebar/index.tsx:752-756,1318-1329), while SidebarSessionsSection renders projectOverview before its flat sessions input (apps/desktop/src/app/chat/sidebar/sessions-section.tsx:238-334).

The PR addresses that mechanism directly by retaining the grouped project browser with an empty flat input and rendering displayAgentSessions in a separate Sessions section (apps/desktop/src/app/chat/sidebar/index.tsx:1334-1401 at f32921681ab3). The visibility helper tests the three relevant grouped/project-scope states. GitHub currently reports the PR as mergeable.

Automated hermes-sweeper review.

@teknium1 teknium1 added sweeper:blast-moderate Sweeper blast radius: moderate — a subsystem or single platform area/sessions Session lifecycle, resume, persistence, history labels Jul 15, 2026
@yahya3867 yahya3867 closed this by deleting the head repository Aug 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/sessions Session lifecycle, resume, persistence, history comp/desktop Electron desktop app (apps/desktop/*) P3 Low — cosmetic, nice to have sweeper:blast-moderate Sweeper blast radius: moderate — a subsystem or single platform type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(desktop): flat "Sessions" list missing from sidebar after update (only Pinned/Projects show, search still works)

3 participants