Skip to content

Refactor chat and component boundaries - #5

Merged
roprgm merged 2 commits into
mainfrom
agents/refactor-chat-ui
Jul 24, 2026
Merged

Refactor chat and component boundaries#5
roprgm merged 2 commits into
mainfrom
agents/refactor-chat-ui

Conversation

@roprgm

@roprgm roprgm commented Jul 24, 2026

Copy link
Copy Markdown
Owner

Summary

  • organize reusable chat primitives under components/chat
  • move the app header and sidebar out of components/session
  • keep session-owned rendering and controls in a flat components/session directory
  • merge single-consumer helpers into their owning components
  • remove mutable JSX variables and update the architecture map

Why

The component tree mixed reusable chat presentation, session behavior, and app-shell navigation under components/session. Several small helpers also lived in separate files even though they had a single owner.

This refactor makes those ownership boundaries explicit while preserving the existing behavior, styling, streaming, and session contracts.

Validation

  • bun run check
  • bun run test — 51 tests passed
  • bun run build

@vercel

vercel Bot commented Jul 24, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
eve-code Ready Ready Preview, Comment Jul 24, 2026 5:18pm

Request Review

@roprgm
roprgm marked this pull request as ready for review July 24, 2026 11:24
@roprgm roprgm changed the title Refactor chat UI primitives Refactor chat and component boundaries Jul 24, 2026
@roprgm
roprgm force-pushed the agents/refactor-chat-ui branch from fbf67ea to 0e48869 Compare July 24, 2026 17:17
@roprgm
roprgm merged commit 138f383 into main Jul 24, 2026
3 checks passed
@roprgm
roprgm deleted the agents/refactor-chat-ui branch July 26, 2026 20:36
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