Skip to content

open local->cloud mode conversation in the same pane#9988

Open
harryalbert wants to merge 2 commits intoharry/remote-1519-make-ui-better-for-local-cloud-handofffrom
harry/remote-1557-open-new-cloud-conversation-in-same-pane-for-local-cloud
Open

open local->cloud mode conversation in the same pane#9988
harryalbert wants to merge 2 commits intoharry/remote-1519-make-ui-better-for-local-cloud-handofffrom
harry/remote-1557-open-new-cloud-conversation-in-same-pane-for-local-cloud

Conversation

@harryalbert
Copy link
Copy Markdown
Contributor

@harryalbert harryalbert commented May 3, 2026

Description

After posting a demo of local -> cloud handoff, I get some feedback (from peter, varoon, and ZL) that it would be better to just open the cloud mode session in the same pane as the local conversation. This makes sense to me, and we're still forking the conversation and opening up a new cloud mode pane so it's not like this is a destructive action (someone can just close the cloud mode view and re-open the local conversation view).

Implements REMOTE-1557

Screenshots / Videos

demo: https://www.loom.com/share/d2f67ee4e15245959210ef41d1dfe7c6

Agent Mode

  • Warp Agent Mode - This PR was created via Warp's AI Agent Mode

Copy link
Copy Markdown
Contributor Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

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

@harryalbert harryalbert marked this pull request as ready for review May 3, 2026 19:59
@harryalbert harryalbert requested a review from zachbai May 3, 2026 19:59
@oz-for-oss
Copy link
Copy Markdown
Contributor

oz-for-oss Bot commented May 3, 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

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 changes local-to-cloud handoff so the forked cloud conversation is pushed onto the source pane's navigation stack instead of opening beside it.

Concerns

  • start_cloud_mode now returns Option, but the updated normal cloud-mode entry paths ignore the must-use return value. Since this repository runs clippy with -D warnings, that warning will fail presubmit.

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

initial_prompt: Option<String>,
ctx: &mut ViewContext<Self>,
) {
) -> Option<(ViewHandle<TerminalView>, ModelHandle<AmbientAgentViewModel>)> {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [IMPORTANT] Returning Option makes the call sites at lines 653 and 667 produce an unused_must_use warning; presubmit runs clippy with -D warnings, so either handle the failure or explicitly assign the intentionally ignored result to let _ = ....

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.

1 participant