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
| recipients: &recipients, | ||
| subject: &msg.subject, | ||
| body: &msg.message_body, | ||
| message_id, |
There was a problem hiding this comment.
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.
Co-Authored-By: Oz <oz-agent@warp.dev>
Co-Authored-By: Oz <oz-agent@warp.dev>
e2e1898 to
d366b60
Compare
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