Skip to content

feat(app): add native side chat tabs - #41185

Closed
wr-web wants to merge 6 commits into
anomalyco:devfrom
wr-web:side-chat
Closed

wr-web wants to merge 6 commits into
anomalyco:devfrom
wr-web:side-chat

Conversation

@wr-web

@wr-web wr-web commented Aug 8, 2026 •

Copy link
Copy Markdown

Issue for this PR

Closes #33469

Related to #16992, #20713, #25709, #32298, and #37246.

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

Adds Side Chat to the existing desktop/web review panel instead of introducing a separate sidebar.

Side Chats are forked child sessions that inherit the main conversation at creation time. Their inherited transcript is hidden in the panel, new Side Chat turns stay isolated from the main session, and closing a tab removes the temporary child session.

The panel supports multiple Side Chat tabs, the native panel + menu, Ctrl+Shift+N, and create/switch actions in Ctrl+P. Selected timeline text can start a quoted Side Chat, and selected Side Chat text can be quoted back into the main composer.

This PR covers the desktop/web side-session workflow. It does not add TUI session graphs or automatic interrupt/resume behavior from the related issues.

How did you verify your code works?

  • bun turbo typecheck (30/30 tasks passed through the repository pre-push hook)
  • bun test src/desktop-menu.test.ts src/pages/session/side-chat.test.ts src/pages/session/helpers.test.ts (23 passed)
  • bun test for e2e/performance/unit/*.test.ts (43 passed)
  • bun run typecheck:e2e
  • bun run build
  • Regression coverage for reusing closed trailing and middle Side Chat numbers while keeping internal tab IDs unique
  • Manual Windows Desktop testing for multiple tabs, +, Ctrl+P, Ctrl+Shift+N, closing, and selection quotes
  • Real model checks with MiMo V2.5 Free: Side Chat could read the main context, while a Side Chat-only fact was not visible to the next main-chat turn

Screenshots / recordings

Native panel menu

Open File and New Side Chat in the native panel menu

Multiple Side Chat tabs

Multiple Side Chat tabs in the review panel

Ctrl+P integration

New Side Chat in the command palette with Ctrl Shift N shortcut

Main context is available to Side Chat

Side Chat answering from inherited main context

Side Chat context does not leak into Main Chat

The Side Chat remembers copper comet, but the next Main Chat turn only sees its own alpine cedar fact.

Side Chat-only fact excluded from the next Main Chat turn

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

@thdxr

thdxr commented Oct 10, 2026

Copy link
Copy Markdown
Member

This branch has been upgraded to V2. We're no longer taking PRs for V1. If you think this is still relevant, please port it over and open it for V2.

— from 𝕺𝖕𝖊𝖓𝕮𝖔𝖉𝖊

@thdxr thdxr closed this Oct 10, 2026
@wr-web wr-web mentioned this pull request Oct 10, 2026
3 of 6 tasks
@wr-web

wr-web commented Oct 10, 2026

Copy link
Copy Markdown
Author

Ported to V2 in #54356. It is a rewrite as a built-in GUI extension rather than a rebase, since the app was restructured: Ctrl+Shift+N or the command palette forks the session into a numbered side panel tab, the inherited history stays hidden, and closing the tab deletes the fork. This PR can stay closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE]: Desktop version side session support (like /btw in TUI)

2 participants