Skip to content

fix: pass setup item through generated deploy CLI - #521

Merged
alongubkin merged 1 commit into
mainfrom
alon/alien-601-generated-deployment-cli-cannot-select-a-setup-item
Aug 29, 2026
Merged

fix: pass setup item through generated deploy CLI#521
alongubkin merged 1 commit into
mainfrom
alon/alien-601-generated-deployment-cli-cannot-select-a-setup-item

Conversation

@alongubkin

Copy link
Copy Markdown
Member

Summary

  • add --setup-item to generated deployment CLIs
  • carry the selection through deployment metadata lookup and manager initialization
  • expose the selection on the manager deployment-store contract for embedded managers
  • regenerate the manager OpenAPI and Rust SDK inputs

Test plan

  • cargo nextest run -p alien-manager -p alien-deploy-cli (469 passed; 1 skipped)

Linear: ALIEN-601

@greptile-apps

Copy link
Copy Markdown

Greptile Summary

The PR carries a generated deployment CLI’s setup-item selection through platform metadata lookup and manager initialization.

  • Adds the --setup-item deployment option and includes it in both deployment-info and initialize requests.
  • Extends the manager initialization and deployment-store creation contracts for embedded manager implementations.
  • Regenerates the manager OpenAPI and Rust SDK inputs and updates affected tests.

Confidence Score: 5/5

The PR appears safe to merge, with the setup-item selection consistently carried across the changed CLI and manager contract boundaries.

No concrete changed-code failure remains after checking argument forwarding, request serialization, manager initialization, and the deployment-store interface.

Important Files Changed

FilenameOverview
crates/alien-deploy-cli/src/commands/up.rsAdds the setup-item flag and consistently forwards its selection to deployment metadata lookup and manager initialization.
crates/alien-manager/src/routes/sync.rsExtends the initialize request and passes the optional selection through the deployment-store creation boundary.
crates/alien-manager/src/traits/deployment_store.rsExposes the optional setup-item selection to replaceable deployment-store implementations.
crates/alien-manager/src/routes/deployments.rsUpdates direct deployment creation to explicitly omit setup-item selection where that API does not provide one.
crates/alien-manager/openapi.jsonRegenerates the manager API schema with the optional setupItem initialization property.
client-sdks/manager/openapi.jsonKeeps the distributable manager client contract aligned with the service schema.
client-sdks/manager/rust/openapi-3.0.jsonKeeps the Rust manager SDK generation input aligned with the initialization contract.

Sequence Diagram

sequenceDiagram
participant G as Generated deploy CLI
participant P as Platform API
participant M as Manager API
participant S as Deployment store
G->>P: "GET /v1/deployment-info?platform=...&setupItem=..."
P-->>G: Setup metadata and inputs
G->>M: POST /v1/initialize with setupItem
M->>S: create_deployment(setup_item)
S-->>M: Deployment record
M-->>G: Deployment ID, model, and token
Loading

Reviews (1): Last reviewed commit: "fix: pass setup item through generated d..." | Re-trigger Greptile

@alongubkin
alongubkin merged commit 04e2e73 into mainAug 29, 2026
24 checks passed
@alongubkin
alongubkin deleted the alon/alien-601-generated-deployment-cli-cannot-select-a-setup-item branch August 29, 2026 03:18
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@alongubkin