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

greptile-apps Bot commented Aug 29, 2026

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

Filename Overview
crates/alien-deploy-cli/src/commands/up.rs Adds the setup-item flag and consistently forwards its selection to deployment metadata lookup and manager initialization.
crates/alien-manager/src/routes/sync.rs Extends the initialize request and passes the optional selection through the deployment-store creation boundary.
crates/alien-manager/src/traits/deployment_store.rs Exposes the optional setup-item selection to replaceable deployment-store implementations.
crates/alien-manager/src/routes/deployments.rs Updates direct deployment creation to explicitly omit setup-item selection where that API does not provide one.
crates/alien-manager/openapi.json Regenerates the manager API schema with the optional setupItem initialization property.
client-sdks/manager/openapi.json Keeps the distributable manager client contract aligned with the service schema.
client-sdks/manager/rust/openapi-3.0.json Keeps 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 main Aug 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 free to 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