Update orchestration message transcript UI#10285
Conversation
|
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 Powered by Oz |
There was a problem hiding this comment.
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
Co-Authored-By: Oz <oz-agent@warp.dev>
Co-Authored-By: Oz <oz-agent@warp.dev>
e2e1898 to
d366b60
Compare
Co-Authored-By: Oz <oz-agent@warp.dev>
…ssage-ui Co-Authored-By: Oz <oz-agent@warp.dev>
cephalonaut
left a comment
There was a problem hiding this comment.
Nice! It might be good to include a loom in the future demonstrating changes.
Co-Authored-By: Oz <oz-agent@warp.dev>
…ssage-ui Co-Authored-By: Oz <oz-agent@warp.dev>
## 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](https://linear.app/warpdotdev/issue/QUALITY-594/send-messagereceived-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 - [x] Warp Agent Mode - This PR was created via Warp's AI Agent Mode - Conversation: https://app.warp.dev/conversation/1b48f020-21b0-4b7b-a0e5-7274863bab36 - Plan: https://app.warp.dev/drive/notebook/R5oEvNd2TWKKGqNoBVs9zE Co-Authored-By: Oz <oz-agent@warp.dev> --------- Co-authored-by: Oz <oz-agent@warp.dev>
## 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](https://linear.app/warpdotdev/issue/QUALITY-594/send-messagereceived-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 - [x] Warp Agent Mode - This PR was created via Warp's AI Agent Mode - Conversation: https://app.warp.dev/conversation/1b48f020-21b0-4b7b-a0e5-7274863bab36 - Plan: https://app.warp.dev/drive/notebook/R5oEvNd2TWKKGqNoBVs9zE Co-Authored-By: Oz <oz-agent@warp.dev> --------- Co-authored-by: Oz <oz-agent@warp.dev>
Description
Linked Issue
ready-to-specorready-to-implement.Screenshots / Videos
Not included.
Testing
cargo fmtcargo fmt --checkgit --no-pager diff --checkcargo test -p warp orchestration --libcargo test -p warp hidden_child --libcargo clippy --workspace --all-targets --all-features --tests -- -D warningsAgent Mode
Co-Authored-By: Oz oz-agent@warp.dev