Skip to content

Add orchestration message display setting#12219

Merged
harryalbert merged 4 commits into
masterfrom
harry/orchestration-message-display-setting
Jun 5, 2026
Merged

Add orchestration message display setting#12219
harryalbert merged 4 commits into
masterfrom
harry/orchestration-message-display-setting

Conversation

@harryalbert
Copy link
Copy Markdown
Contributor

@harryalbert harryalbert commented Jun 4, 2026

Description

Right now orchestration messages auto-collapse so users have to manually click to expand them. It makes sense to keep this as the default option (for now), but I think there should be a setting for the message expansion behavior, similar to thinking traces.

The setting has 3 options:

  • Always Collapse (default)
  • Always Show
  • Show & Collapse (expand while the message is being written, and then collapse, same as the thinking trace setting)

Testing

https://www.loom.com/share/884b5253a20f4be3bb9fc1db40834b46

Conversation: https://staging.warp.dev/conversation/350fa0b0-68bd-4f95-9c58-cfdfae17d0c5
Run: https://oz.staging.warp.dev/runs/019e9444-002f-7b9c-91ec-9ff5eb8c8420
This PR was generated with Oz.

@cla-bot cla-bot Bot added the cla-signed label Jun 4, 2026
@harryalbert harryalbert force-pushed the harry/orchestration-message-display-setting branch from 4650599 to 51a9569 Compare June 5, 2026 19:29
Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@harryalbert harryalbert marked this pull request as ready for review June 5, 2026 19:32
@oz-for-oss
Copy link
Copy Markdown
Contributor

oz-for-oss Bot commented Jun 5, 2026

@harryalbert

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

@harryalbert harryalbert requested review from advait-m and removed request for kjankov June 5, 2026 19:32
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 adds a new AI setting for orchestration message expansion behavior, wires it into settings/command-palette state, updates collapsible state handling for sent and received orchestration messages, and switches orchestration message bodies to Markdown-backed rendering.

Concerns

  • Markdown-rendered orchestration message bodies now inherit the standard executable shell-code controls, so child-agent transcript content can surface runnable commands where it was previously inert text.

Security

  • A child-agent or orchestrator message containing a fenced shell snippet can now expose the normal run action through the transcript renderer. That should be disabled or explicitly opted into for orchestration message bodies.

Verdict

Found: 0 critical, 1 important, 0 suggestions

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

Comment thread app/src/ai/blocklist/block/view_impl/orchestration.rs Outdated
@harryalbert harryalbert force-pushed the harry/orchestration-message-display-setting branch from c81a895 to e9ec603 Compare June 5, 2026 20:20
@harryalbert harryalbert force-pushed the harry/orchestration-message-display-setting branch from e9ec603 to 44d3696 Compare June 5, 2026 20:27
@harryalbert harryalbert merged commit 21a8ae4 into master Jun 5, 2026
26 checks passed
@harryalbert harryalbert deleted the harry/orchestration-message-display-setting branch June 5, 2026 21:41
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.

2 participants