Add AmbientSetupPhaseEnded variant to OrderedTerminalEventType#17
Draft
harryalbert wants to merge 2 commits into
Draft
Add AmbientSetupPhaseEnded variant to OrderedTerminalEventType#17harryalbert wants to merge 2 commits into
harryalbert wants to merge 2 commits into
Conversation
Adds an explicit shared-session-protocol marker that the sandboxed Oz AgentDriver emits when cloud-mode setup is complete but no initial LLM turn will follow (e.g. empty-prompt local-to-cloud handoff with `--skip-initial-turn`). Viewers consume the event to tear down the Cloud Mode Setup V2 "Running setup commands..." chip and clear `BlockList::is_executing_oz_environment_startup_commands` without waiting for the first `AppendedExchange`. The variant is unit-typed (no associated data) for v1. A future struct upgrade with `task_id: Option<String>` is fine to plan for as a struct variant later but is deliberately out of scope here. Mirrors the placement and idiom of the adjacent `AgentConversationReplayStarted` / `AgentConversationReplayEnded` variants. The session-sharing-server treats `OrderedTerminalEventType` opaquely (only `CommandExecutionFinished` is special-cased for block-id tracking), so no server code changes are required to route or persist the new variant. The server will pick it up via a protocol dep rev bump after this change merges. Includes a serde round-trip test asserting the unit-variant wire form. Stage 2c of empty-prompt local-to-cloud handoff. Co-Authored-By: Oz <oz-agent@warp.dev>
… missing Started half Adjacent variants come in Started/Ended pairs; AmbientSetupPhaseEnded has no matching Started because the setup-phase start is implicit in existing SetupCommandState transitions on the viewer once scrollback arrives. Co-Authored-By: Oz <oz-agent@warp.dev>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add AmbientSetupPhaseEnded variant to OrderedTerminalEventType
Adds an explicit shared-session-protocol marker that the sandboxed Oz
AgentDriver emits when cloud-mode setup is complete but no initial LLM
turn will follow (e.g. empty-prompt local-to-cloud handoff with
--skip-initial-turn). Viewers consume the event to tear down theCloud Mode Setup V2 "Running setup commands..." chip and clear
BlockList::is_executing_oz_environment_startup_commandswithoutwaiting for the first
AppendedExchange.The variant is unit-typed (no associated data) for v1. A future struct
upgrade with
task_id: Option<String>is fine to plan for as a structvariant later but is deliberately out of scope here.
Mirrors the placement and idiom of the adjacent
AgentConversationReplayStarted/AgentConversationReplayEndedvariants. The session-sharing-server treats
OrderedTerminalEventTypeopaquely (only
CommandExecutionFinishedis special-cased forblock-id tracking), so no server code changes are required to route or
persist the new variant. The server will pick it up via a protocol
dep rev bump after this change merges.
Includes a serde round-trip test asserting the unit-variant wire form.
Stage 2c of empty-prompt local-to-cloud handoff.
Co-Authored-By: Oz oz-agent@warp.dev
REMOTE-1499: add doc comment to AmbientSetupPhaseEnded explaining the missing Started half
Adjacent variants come in Started/Ended pairs; AmbientSetupPhaseEnded has no matching Started because the setup-phase start is implicit in existing SetupCommandState transitions on the viewer once scrollback arrives.
Co-Authored-By: Oz oz-agent@warp.dev