Skip to content

Move cloud object models to separate crate.#11164

Open
vorporeal wants to merge 1 commit into
graphite-base/11164from
david/start-moving-cloud-object-models
Open

Move cloud object models to separate crate.#11164
vorporeal wants to merge 1 commit into
graphite-base/11164from
david/start-moving-cloud-object-models

Conversation

@vorporeal
Copy link
Copy Markdown
Contributor

@vorporeal vorporeal commented May 18, 2026

Description

Moves the remaining low-level cloud object model definitions out of the app crate so they can be reused outside warp without pulling in app-specific code.

This PR moves the workflow enum and cloud environment model surfaces into the server/client cloud object layer while keeping compatibility re-exports for existing app call sites. This is a preparatory step toward extracting the shared cloud object substrate and model crates in the follow-up PRs.

Note

note from dstern: this honestly should probably just have been merged into a higher PR in the stack (which pulls these back out of cloud_objects and into a cloud_object_models crate with all of the other models), but it's here, so cest la vie

Linked Issue

None.

Testing

  • cargo check --manifest-path /Users/david/src/warp/Cargo.toml -p warp --tests

No manual app testing; this is a crate-boundary refactor with no intended product behavior changes.

Agent Mode

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

CHANGELOG-NONE

Co-Authored-By: Oz oz-agent@warp.dev

Copy link
Copy Markdown
Contributor Author

vorporeal commented May 18, 2026

@vorporeal vorporeal force-pushed the david/start-moving-cloud-object-models branch from d5b94a3 to b434157 Compare May 18, 2026 14:52
@vorporeal vorporeal force-pushed the david/move-GenericStringModel-to-warp_server_client branch from fb75c88 to 8de3209 Compare May 18, 2026 14:53
@vorporeal vorporeal force-pushed the david/start-moving-cloud-object-models branch from b434157 to bac2d6a Compare May 19, 2026 18:43
@vorporeal vorporeal force-pushed the david/move-GenericStringModel-to-warp_server_client branch from 8de3209 to 30262f4 Compare May 19, 2026 18:43
@vorporeal vorporeal marked this pull request as ready for review May 20, 2026 15:28
@oz-for-oss
Copy link
Copy Markdown
Contributor

oz-for-oss Bot commented May 20, 2026

@vorporeal

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

@vorporeal vorporeal requested a review from bnavetta May 20, 2026 15:29
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 moves the workflow enum and cloud environment model definitions into warp_server_client::cloud_object::models and re-exports them from the existing app modules to preserve current call sites. The moved models keep the same serde attributes, trait derives, constructors, and display behavior, and no approved spec context was available for additional implementation validation.

Concerns

  • No blocking correctness, security, or spec-alignment concerns found in the changed lines.

Verdict

Found: 0 critical, 0 important, 0 suggestions

Approve

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

@vorporeal vorporeal force-pushed the david/move-GenericStringModel-to-warp_server_client branch from 30262f4 to dbe2413 Compare May 20, 2026 15:56
@vorporeal vorporeal force-pushed the david/start-moving-cloud-object-models branch 2 times, most recently from 9239c96 to ec7be2a Compare May 21, 2026 14:59
@vorporeal vorporeal force-pushed the david/move-GenericStringModel-to-warp_server_client branch from 1f9d920 to f876c67 Compare May 21, 2026 16:47
@vorporeal vorporeal force-pushed the david/start-moving-cloud-object-models branch from ec7be2a to 65ec5ea Compare May 21, 2026 16:47
@vorporeal vorporeal force-pushed the david/move-GenericStringModel-to-warp_server_client branch from f876c67 to eb76b2c Compare May 22, 2026 18:51
@vorporeal vorporeal force-pushed the david/start-moving-cloud-object-models branch from 65ec5ea to 631c76e Compare May 22, 2026 18:51
@vorporeal vorporeal force-pushed the david/start-moving-cloud-object-models branch from 631c76e to d14fff2 Compare May 22, 2026 18:53
@vorporeal vorporeal force-pushed the david/move-GenericStringModel-to-warp_server_client branch from eb76b2c to 9dffd19 Compare May 22, 2026 18:53
@vorporeal vorporeal changed the base branch from david/move-GenericStringModel-to-warp_server_client to graphite-base/11164 May 22, 2026 20:27
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