Skip to content

fix(desktop): open chats from the skills view - #68326

Closed
WillieTheWhale wants to merge 1 commit into
NousResearch:mainfrom
WillieTheWhale:agent/fix-session-click-from-tools
Closed

fix(desktop): open chats from the skills view#68326
WillieTheWhale wants to merge 1 commit into
NousResearch:mainfrom
WillieTheWhale:agent/fix-session-click-from-tools

Conversation

@WillieTheWhale

Copy link
Copy Markdown

What changed

Clicking a sidebar session now returns to its chat when Skills & Tools is open. Sessions displayed in a separate tile still focus that tile instead.

Why

The click handler treated the main chat and separate tiles as the same surface, so it swallowed the navigation back to chat.

Tests

  • Desktop test suite: 2,208 passed, 2 skipped
  • Typecheck and lint passed

Fixes #68302

@alt-glitch alt-glitch added type/bug Something isn't working P3 Low — cosmetic, nice to have comp/desktop Electron desktop app (apps/desktop/*) sweeper:risk-session-state Sweeper risk: may lose/corrupt/mis-associate session or context state duplicate This issue or pull request already exists labels Jul 21, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

This was generated by AI during triage.

Duplicate of #66880: both repair the Desktop full-page session-click routing defect. #66880 already covers /skills and the open-tile case; this patch should not replace that broader regression coverage.

@teknium1

Copy link
Copy Markdown
Contributor

Thanks for the focused Desktop regression fix. This is now redundant on current main.

Closing as implemented on main.

@teknium1 teknium1 closed this Jul 30, 2026
@teknium1 teknium1 added the sweeper:implemented-on-main Sweeper: behavior already present on current main label Jul 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/desktop Electron desktop app (apps/desktop/*) duplicate This issue or pull request already exists P3 Low — cosmetic, nice to have sweeper:implemented-on-main Sweeper: behavior already present on current main sweeper:risk-session-state Sweeper risk: may lose/corrupt/mis-associate session or context state type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Clicking a sidebar session has no effect while Skills & Tools is the active view

3 participants