Skip to content

fix: hydrate cloud mode input from viewed run config#11167

Merged
captainsafia merged 1 commit into
masterfrom
fix/cloud-mode-session-config-pill
May 18, 2026
Merged

fix: hydrate cloud mode input from viewed run config#11167
captainsafia merged 1 commit into
masterfrom
fix/cloud-mode-session-config-pill

Conversation

@captainsafia
Copy link
Copy Markdown
Contributor

Summary

  • Preserve the run snapshot environment ID when viewing an existing cloud session, even if the environment object has not loaded into CloudModel yet.
  • Hydrate the pane-level Oz model override from the viewed run's agent_config_snapshot.model_id.
  • Add regression tests for viewed-session environment and model hydration.

Tests

  • cargo test --manifest-path /workspace/warp/app/Cargo.toml viewed_task_config --lib
  • cargo test --manifest-path /workspace/warp/app/Cargo.toml terminal::view::ambient_agent::model::tests --lib

Conversation: https://staging.warp.dev/conversation/9137ae95-5fb4-4bbd-99d6-810fa7297eda
Run: https://oz.staging.warp.dev/runs/019e38ab-9e8e-7a5e-91bb-6cbb7191bec4
This PR was generated with Oz.

Preserve the run snapshot environment and model when joining existing cloud sessions so the input pills do not fall back to unrelated defaults while web session state is loading.

Co-Authored-By: Oz <oz-agent@warp.dev>
@cla-bot cla-bot Bot added the cla-signed label May 18, 2026
@captainsafia captainsafia marked this pull request as ready for review May 18, 2026 02:06
@captainsafia captainsafia requested a review from harryalbert May 18, 2026 02:06
@oz-for-oss
Copy link
Copy Markdown
Contributor

oz-for-oss Bot commented May 18, 2026

@captainsafia

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

@captainsafia captainsafia requested a review from zachbai May 18, 2026 02:06
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 preserves viewed cloud-session configuration by hydrating the environment ID and Oz model override from the run's agent config snapshot, with regression coverage for both paths.

Concerns

  • ⚠️ [IMPORTANT] This changes user-facing Cloud Mode selector/model hydration behavior, but the PR description does not include screenshots or a screen recording demonstrating the viewed-session environment and model pill state end to end. For this user-facing change, please include screenshots or a screen recording demonstrating it working end to end.

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

@captainsafia captainsafia enabled auto-merge (squash) May 18, 2026 02:14
@captainsafia captainsafia merged commit e580d37 into master May 18, 2026
43 checks passed
@captainsafia captainsafia deleted the fix/cloud-mode-session-config-pill branch May 18, 2026 02:22
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