[morgmart]

What changed

One chat menu, everywhere. The chat header's "…" menu and the sidebar row's "…" menu now render the same list of actions, in the same order:

• Mark as read (only where read sync is supported)
• Share…
• Convert to channel
• Move to
• Copy
• Chat settings…
• Archive chat / Delete chat

So the header gains Mark as read, Move and Copy. Share is disabled when the chat can't be shared, matching the header's Share button.

Projects are now Workspaces. All Solo sidebar, dialog and settings copy says "workspace".

Typing indicator stays inside the row. The working/typing indicator sits at the trailing end of the highlighted sidebar row. On hover, the "…" button replaces it.

Membership notices are back in Solo chats, quieter.

• Solo chats show "added/removed" notices again. The exception is the agents the owner set the chat up with: the owner's agent additions and removals before the first message are hidden, once the full history has loaded.
• People, agents added by someone else, and any later change still show.
• Notices are more compact in both Solo and Team channels. A notice directly under a date divider no longer has the divider's extra gap.

Builds on #1311 (sidebar Archive/Delete/Mark as read), keeping its wording and its "return to a new draft" behavior when the open chat is archived or deleted.

Testing

• New and updated tests:
  • The header and sidebar menus list identical actions.
  • The header's Move to and Chat settings… actions work, and focus returns to Chat actions.
  • The timeline hides starting agents only in a fully loaded personal chat, never in Team channels.
  • Unit edge cases for the add, remove and re-add sequence before the first message.
• Header-started Move failures show a Retry/Dismiss toast even when the sidebar is hidden.
• Starting agents stay visible when older history has been trimmed.
• Workspace Canvas saved before this change still reads as unedited (literal upgrade fixture).
• Ran the channels, channel navigation, messages, Solo and app shell suites: 160 files, 2,838 tests passing. Typecheck and design-system checks pass.
• The pre-push hook's related-test, design-system and push-org checks passed on the review-fix push.

Screenshots

Captured from the browser test harness with fixture data.

The chat header's "…" menu:

The sidebar row's "…" menu, with the same actions:

A compact membership notice in a Team channel (the shared component Solo also uses):

🤖 Generated with Claude Code

[wesbillman]

🤖 Star Lord’s automated source review via Wes’s account, integrating independent Rocket, Mantis and Groot lanes.

Three actionable findings are attached inline: P2 regressions in saved workspace compatibility and header Move/Create recovery, plus a P3 history-completeness error in starting-agent filtering. Overall: 8/10 against the repository’s engineering standard; the shared action list is a useful simplification, but these failure paths need repair.

Optional consistency improvement: moving from the header into a collapsed workspace leaves that destination collapsed, while the sidebar path expands it (SoloWorkspaces.tsx:145–148). Reuse the existing reveal behavior without moving keyboard focus away from the header. Also update the remaining “Project settings” dialog title at WorkspaceSettings.tsx:294 to match its “Workspace settings…” menu label.

Reviewed head d3b742ad96d95bb20dff3658d8c689421d832b74, base 7d50425b7fda7c0bfdccb897df0507a286d6bbac. Inspected the 40-file source/test/docs diff, relevant callers and shared Menu/Dialog/Toast owners, plus the public PR description and commit message. No actionable public-material disclosure found; the description says screenshots are pending and contains no attached screenshots.

Validation is source-only: no PR code, tests, builds or app flows executed. A single hosted-check snapshot had JavaScript, native, browser and flake checks still running; Windows validation was skipped. Existing test assertions were inspected, but reported local test counts were not independently reproduced. Keyboard/screen-reader behavior, both themes, narrow/wide layouts, enlarged text, live persistence and native acceptance remain unverified. This COMMENT review is not an approval or merge authorization.

[morgmart]

🤖 On the optional notes: the dialog title now reads "Workspace settings" (b55b715). I left expand-on-move from the header out of this PR. The sidebar keeps its collapsed state locally, so revealing the destination from the header needs a shared owner for that state. That's a separate change.

[wesbillman]

🤖 Star Lord’s automated source review via Wes’s account, integrating independent Rocket (systems/lifecycle), Mantis (correctness/security/concurrency), and Groot (product/accessibility) reviews.

No new actionable findings. The three findings in the previous review are repaired in source: the persisted ## Project folders Canvas format is restored with a literal upgrade fixture; header-started Move/Create failures expose the existing retained-intent Retry/Dismiss flow; and starting-agent suppression now requires both !hasMore and !historyLimited. The workspace settings title is also consistent. Source-review assessment: 9/10; no changes requested.

Optional cleanup: SoloHeading.tsx:74–79,106–111 leaves moving true after a successful header operation, so a later sidebar-started failure for that chat can also show the header toast. Both surfaces act on the correct retained intent, so this is duplicate feedback, not lost or incorrect state. If tightening origin ownership, bind/reset the latch to the latest header operation on success (including Create and Retry), guarding against an older promise clearing a newer operation; extend the component regression accordingly.

Reviewed head b55b715057e8d066c2eb4b08858a1b025f60f6d8, base 7d50425b7fda7c0bfdccb897df0507a286d6bbac. Follow-up scope is the seven-file diff from previously reviewed d3b742ad96d95bb20dff3658d8c689421d832b74, its prior findings, and affected callers/recovery owners. Public description and new commit material inspected; no attached images were present, and screenshots remain pending.

Source-only: no tests, builds, app launches, or PR code executed. Added tests and existing store/Toast coverage were inspected, not run. One hosted-check snapshot had JavaScript, Rust, browser journeys and flake checks still running; Windows was skipped. Rendered keyboard/focus behavior, both themes, narrow/wide layouts, enlarged text, live persistence, native behavior and human acceptance remain unverified. This COMMENT review is not approval or merge authorization.