Skip to content

Flatten chat message rows (no inner bubble data-own) - #411

Merged
TheGreatAxios merged 3 commits into
mainfrom
cl-6466-chat-page-parity-with-the-mock-tool-use-message-layout-no
Aug 25, 2026
Merged

TheGreatAxios merged 3 commits into
mainfrom
cl-6466-chat-page-parity-with-the-mock-tool-use-message-layout-no

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

Summary

  • Own-message alignment stays on .chat-message-group / .chat-bubble-row; the inner .chat-bubble no longer carries data-own.
  • Tests lock the flatten: viewer-authored messages still right-align; leftover bubble CSS stays chrome-free.
  • @corbits/chat migrations test now expects 0020–0023 and the agent_turns table so bun run check is green.

Test plan

  • bun test in packages/chat-ui for own-message-alignment + chat-bubble-flat (from the package cwd)
  • bun run check in the worktree

CL-6466: https://linear.app/abklabs/issue/CL-6466/chat-page-parity-with-the-mock-tool-use-message-layout-no-bubbles

Own-message alignment must live on the group and row. The inner
.chat-bubble must not carry data-own, and leftover bubble CSS must
stay chrome-free.
Drop data-own from the inner .chat-bubble so own-message right-align
is a single surface (group + row), matching the mock's no-bubble layout.
applyChatMigrations was still frozen at 0019 after 0020-0023 landed,
so the DB-gated suite failed on a fresh apply.
@TheGreatAxios
TheGreatAxios merged commit 260a5cc into main Aug 25, 2026
5 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