Skip to content

Phase 1: extract chat bubbles and tool renderers from SidebarChat#75

Merged
Pterjudin merged 1 commit into
mainfrom
feat/phase1-chat-tool-split
Jul 22, 2026
Merged

Phase 1: extract chat bubbles and tool renderers from SidebarChat#75
Pterjudin merged 1 commit into
mainfrom
feat/phase1-chat-tool-split

Conversation

@Pterjudin

Copy link
Copy Markdown

Summary

  • Extract ChatBubble, UserMessageComponent, and prose wrappers into sidebar-tsx/chat/
  • Extract ToolHeader, ToolRenderers (builtin/MCP tools, streaming edit preview) into sidebar-tsx/tools/
  • Shrink SidebarChat.tsx from ~3,912 → ~1,257 lines (~68% reduction)
  • SidebarChat now owns composer shell: tabs, thread header, command bar, input, landing page

Module layout

sidebar-tsx/
├── SidebarChat.tsx          (~1,257 lines — shell only)
├── chat/
│   ├── ChatBubble.tsx       (messages, plan, review, checkpoint)
│   ├── UserMessageComponent.tsx
│   └── proseWrappers.tsx
└── tools/
    ├── ToolHeader.tsx
    └── ToolRenderers.tsx

Test plan

  • npm run buildreact — build succeeds
  • npm run test-phase0-qa — 88 unit tests passing
  • Manual: send chat message — user/assistant bubbles render
  • Manual: agent tool calls (read file, edit file) display correctly
  • Manual: plan/review messages render with approve/reject flows
  • Manual: landing page + composer input unchanged

Made with Cursor

Move message components, plan/review UI, builtin tool renderers, and tool headers into dedicated chat/ and tools/ modules so SidebarChat becomes a thin composer shell.

Co-authored-by: Cursor <cursoragent@cursor.com>
@cursor

cursor Bot commented Jul 22, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@Pterjudin
Pterjudin merged commit 0b5a05b into main Jul 22, 2026
14 of 26 checks passed
@Pterjudin
Pterjudin deleted the feat/phase1-chat-tool-split branch July 22, 2026 21:22
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