Skip to content

Feat/chat viewer bridge and bubble widget#68

Open
Nixxx19 wants to merge 8 commits intomofa-org:mainfrom
Nixxx19:feat/chat-viewer-bridge-and-bubble-widget
Open

Feat/chat viewer bridge and bubble widget#68
Nixxx19 wants to merge 8 commits intomofa-org:mainfrom
Nixxx19:feat/chat-viewer-bridge-and-bubble-widget

Conversation

@Nixxx19
Copy link
Copy Markdown
Contributor

@Nixxx19 Nixxx19 commented Mar 15, 2026

what this fixes

two connected problems, solved together:

1. silent data loss in dispatcher.rs
MofaNodeType::ChatViewer was hitting continue — every chat
message from dora was silently dropped. no error, no log, nothing.
implements ChatViewerBridge to complete the pipeline end-to-end.

2. chat bubble widget
adds ChatBubble (role-colored gpu-rendered bubble) and
ChatBubblePanel (scrollable conversation view) with full
user / assistant / system role support.

why one pr

the bridge without the widget = messages go nowhere.
the widget without the bridge = nothing feeds it.
one feature, split across two crates.

changes

  • mofa-dora-bridge/src/widgets/chat_viewer_bridge.rs (new)
  • mofa-dora-bridge/src/dispatcher.rs — replaces todo+continue
  • mofa-ui/src/widgets/chat_bubble.rs (new)
  • apps/mofa-test-app/src/main.rs — registry count 13→15
  • fix pre-existing warnings in log_panel and sidebar

verified

  • 18 tests pass, 0 regressions
  • mofa-test-app: 14/14 green, all 15 widgets confirmed
Screenshot 2026-03-15 at 7 51 01 PM

@Nixxx19
Copy link
Copy Markdown
Contributor Author

Nixxx19 commented Mar 15, 2026

cc @BH3GEI @ymote please review, thanks!

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