Skip to content

feat: expose deployment group external IDs in clients - #309

Open
alongubkin wants to merge 9 commits into
mainfrom
alon/alien-440-add-project-scoped-external-ids-to-deployment-groups
Open

feat: expose deployment group external IDs in clients#309
alongubkin wants to merge 9 commits into
mainfrom
alon/alien-440-add-project-scoped-external-ids-to-deployment-groups

Conversation

@alongubkin

Copy link
Copy Markdown
Member

Summary

  • regenerate the public Platform OpenAPI schemas for deployment group external IDs
  • expose ensure, exact-read, and explicit-update operations in the TypeScript SDK
  • verify the build-time generated Rust SDK surface

Platform API: alienplatform/platform#239

Validation

  • NODE_OPTIONS=--max-old-space-size=8192 pnpm build in client-sdks/platform/typescript
  • cargo test -p alien-platform-api
  • cargo test -p alien-platform-api --test deployment_group_external_id

Linear: ALIEN-440

@greptile-apps

greptile-apps Bot commented Aug 3, 2026

Copy link
Copy Markdown

Greptile Summary

Regenerates the Platform SDKs to expose deployment-group external IDs.

  • Adds TypeScript ensure, exact-read, and explicit-update operations with required mutation bodies.
  • Includes external IDs in deployment-group request and response models.
  • Extends the Rust compile-surface test to construct the explicit-update builder.
  • Updates CLI construction for the expanded deployment-group request type.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains; omitted TypeScript mutation bodies are rejected during SDK validation, and the Rust surface test now covers the explicit-update builder.

Important Files Changed

Filename Overview
client-sdks/platform/typescript/src/models/operations/ensuredeploymentgroupbyexternalid.ts The nested ensure body is now required by both the TypeScript type and runtime Zod schema, preventing the previously reported bodyless PUT.
client-sdks/platform/typescript/src/models/operations/setdeploymentgroupexternalid.ts The explicit-update operation requires and correctly remaps its request body before request encoding.
client-sdks/platform/rust/tests/deployment_group_external_id.rs The compile-surface test now constructs the ensure, exact-read, and explicit-update operation builders.
crates/alien-cli/src/commands/onboard.rs Onboarding initializes the newly generated optional external-ID field to None.

Reviews (7): Last reviewed commit: "test: construct validated external ID re..." | Re-trigger Greptile

Comment thread client-sdks/platform/rust/tests/deployment_group_external_id.rs
@greptile-apps

greptile-apps Bot commented Aug 6, 2026

Copy link
Copy Markdown

Too many files changed for review (3000 files, 100 file limit).

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