Skip to content

CL-5779: Channels in the canvas; retire /chat nav - #30

Merged
TheGreatAxios merged 4 commits into
mainfrom
cl-5779-channels-everywhere-right-side-chat-surface-retire-the-chat
Aug 9, 2026
Merged

CL-5779: Channels in the canvas; retire /chat nav#30
TheGreatAxios merged 4 commits into
mainfrom
cl-5779-channels-everywhere-right-side-chat-surface-retire-the-chat

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

Summary

  • Host channel chat in the right-side canvas column on expanded layouts (ChatWorkspace), with a main-pane fallback when the canvas is unavailable.
  • Deep links at /c and /c/:channelId; legacy /chat still resolves to the same surface.
  • Remove Chat from the rail; home/onboarding/agents/palette navigate via /c.
  • Canvas state carries the open channel so conversation survives page navigation.

Out of scope for this PR (ticket follow-ups): sub-channel creation/hierarchy UI and threads (CL-5520 alignment).

Test plan

  • bun test focused: channel-path, canvas-column-state, routes, shell, rail, panel-contribution (38 pass)
  • CI green on this PR
  • Manual: open a channel from the panel → canvas shows timeline + composer
  • Manual: navigate Home → Routines with canvas open → conversation intact
  • Manual: compact layout uses main-pane chat; legacy /chat/:id still loads

Closes CL-5779 (core surface; nesting/threads follow).

Cover channel deep-link helpers, canvas open-with-channel
behavior, and route/rail expectations after retiring /chat
from the nav.
Open channels in the right-side canvas column on expanded
layouts, keep a main-pane ChatWorkspace fallback when the
canvas is unavailable, and retire Chat from the rail. Legacy
/chat links still resolve to the same surface.
Replace the /chat route row with the canvas-hosted /c deep
link and note that legacy /chat URLs still resolve.
Prettier on the new surface files; prefix unused canvas-state
reducer args so eslint accepts the fixed signatures.
@TheGreatAxios
TheGreatAxios force-pushed the cl-5779-channels-everywhere-right-side-chat-surface-retire-the-chat branch from cd5fde9 to 1c26348 Compare August 9, 2026 20:46
@TheGreatAxios
TheGreatAxios merged commit 80d250b into main Aug 9, 2026
2 checks passed
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.

1 participant