Skip to content

refactor(web): harden direct message dialog#58

Merged
RomeoCavazza merged 1 commit into
mainfrom
agent/front-l05-direct-messages
Jul 20, 2026
Merged

refactor(web): harden direct message dialog#58
RomeoCavazza merged 1 commit into
mainfrom
agent/front-l05-direct-messages

Conversation

@RomeoCavazza

Copy link
Copy Markdown
Member

What changed

  • migrates Direct Message from a manual overlay to the shared Radix Dialog shell
  • makes each roster Message button the real trigger for focus restoration
  • loads a conversation only while its dialog is open
  • keeps the thread as the only scrollable region in a bounded dvh layout
  • announces only unseen messages from the currently open peer after a silent baseline
  • coalesces message batches and deduplicates repeated query invalidations
  • announces send and load failures without exposing WebSocket transport state

Product boundaries

  • no API, PostgreSQL, RBAC or WebSocket protocol change
  • no global chat abstraction
  • existing one-to-one operational messaging remains intact

Validation

  • 86 Vitest tests
  • 40 Playwright flows
  • real Observer send and Responder to Manager HTTP plus WebSocket receive
  • Escape, focus restoration, pending close, error announcement and 320 px geometry
  • Next.js build, ESLint, Prettier, TypeScript, Stylelint and Knip
  • Rust fmt, Clippy, audit and deny
  • deterministic reset to 6 stable private messages

@RomeoCavazza
RomeoCavazza merged commit add42cb into main Jul 20, 2026
14 checks passed
@RomeoCavazza
RomeoCavazza deleted the agent/front-l05-direct-messages branch July 20, 2026 13:19
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