Skip to content

Update orchestration message transcript UI#10285

Open
kjankov wants to merge 2 commits intomasterfrom
katarina/quality-594-send-messagereceived-message-ui
Open

Update orchestration message transcript UI#10285
kjankov wants to merge 2 commits intomasterfrom
katarina/quality-594-send-messagereceived-message-ui

Conversation

@kjankov
Copy link
Copy Markdown
Contributor

@kjankov kjankov commented May 6, 2026

Description

  • Updates successful orchestration send/receive messages to render as transcript-style rows with the agent avatar beside the message content instead of boxed status cards.
  • Uses the orchestrator avatar for the lead/parent agent, existing harness/provider logos for known child agents, and distinct fallback child avatars for Oz/unknown child agents.
  • Persists child harness metadata so orchestration avatars can be restored consistently.

Linked Issue

  • Linear: QUALITY-594 Send message/received message UI
  • The linked issue is labeled ready-to-spec or ready-to-implement.
  • Where appropriate, screenshots or a short video of the implementation are included below (especially for user-visible or UI changes).

Screenshots / Videos

Not included.

Testing

  • cargo fmt
  • cargo fmt --check
  • git --no-pager diff --check
  • cargo test -p warp orchestration --lib
  • cargo test -p warp hidden_child --lib
  • cargo clippy --workspace --all-targets --all-features --tests -- -D warnings

Agent Mode

Co-Authored-By: Oz oz-agent@warp.dev

@cla-bot cla-bot Bot added the cla-signed label May 6, 2026
@oz-for-oss
Copy link
Copy Markdown
Contributor

oz-for-oss Bot commented May 6, 2026

@kjankov

I'm starting a first review of this pull request.

You can view the conversation on Warp.

I completed the review and no human review was requested for this pull request.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

Copy link
Copy Markdown
Contributor

@oz-for-oss oz-for-oss Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overview

This PR updates successful orchestration send/receive messages to render as transcript rows with participant avatars and persists child harness metadata for avatar restoration.

Concerns

  • Multiple received-message transcript rows reuse the same collapsible state, so every row chevron controls the whole message group instead of its own row.
  • For faster review, please upload screenshots or a video of the UI change working end to end.

Verdict

Found: 0 critical, 1 important, 1 suggestion

Request changes

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

recipients: &recipients,
subject: &msg.subject,
body: &msg.message_body,
message_id,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [IMPORTANT] Every received-message row passes the same output message_id, so multiple row chevrons share one hover/collapse state and clicking any row collapses or expands every message body; keep a single group-level chevron or register per-message state before rendering per-row toggles.

@kjankov kjankov requested a review from cephalonaut May 6, 2026 19:21
kjankov and others added 2 commits May 6, 2026 15:30
Co-Authored-By: Oz <oz-agent@warp.dev>
Co-Authored-By: Oz <oz-agent@warp.dev>
@kjankov kjankov force-pushed the katarina/quality-594-send-messagereceived-message-ui branch from e2e1898 to d366b60 Compare May 6, 2026 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant