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

The PR exposes deployment-group external IDs across the generated client SDKs.

  • Regenerates the Platform OpenAPI schemas with external-ID fields and operations.
  • Adds TypeScript ensure, exact-read, and explicit-update methods with required mutation bodies.
  • Extends the Rust public-surface test to construct the explicit-update request and operation builder.
  • Updates CLI onboarding Git handling to retain the repository handle while reading remote metadata.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
client-sdks/platform/typescript/src/models/operations/ensuredeploymentgroupbyexternalid.ts The ensure operation now requires its request body in both the TypeScript type and outbound validation schema, resolving the prior bodyless-PUT issue.
client-sdks/platform/typescript/src/models/operations/setdeploymentgroupexternalid.ts The explicit-update operation requires its body before request encoding, including when clearing the external ID with null.
client-sdks/platform/rust/tests/deployment_group_external_id.rs The compile-time public-surface test now constructs the explicit-update body and operation builder, resolving the prior coverage finding.
client-sdks/platform/typescript/src/sdk/deploymentgroups.ts Adds public SDK entry points for ensuring, reading, and updating deployment groups by external ID.
crates/alien-cli/src/git_utils.rs Refactors repository discovery so the repository remains available while origin metadata is read.

Reviews (12): Last reviewed commit: "fix: align external ID client schema wit..." | Re-trigger Greptile

Comment thread client-sdks/platform/rust/tests/deployment_group_external_id.rs
@alongubkin
alongubkin force-pushed the alon/alien-440-add-project-scoped-external-ids-to-deployment-groups branch from 9c0ef35 to 85ab675 Compare August 14, 2026 03:40
@alongubkin
alongubkin force-pushed the alon/alien-440-add-project-scoped-external-ids-to-deployment-groups branch from f26274b to 5d8947e Compare August 14, 2026 14: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