Skip to content

feat(apps:builds:create): support ad hoc environment variables#134

Merged
robingenz merged 1 commit intomainfrom
feat/ad-hoc-environment-variables
Mar 26, 2026
Merged

feat(apps:builds:create): support ad hoc environment variables#134
robingenz merged 1 commit intomainfrom
feat/ad-hoc-environment-variables

Conversation

@robingenz
Copy link
Member

Summary

  • Add --variable flag (repeatable KEY=VALUE) to pass ad hoc environment variables when creating a build
  • Add --variable-file flag to load ad hoc environment variables from a .env file
  • Variables are sent as adHocEnvironmentVariables in the API request body

Copilot AI review requested due to automatic review settings March 26, 2026 15:15
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds support for passing ad hoc environment variables when creating an app build via the CLI, and forwards them to the API as adHocEnvironmentVariables.

Changes:

  • Extend CreateAppBuildDto with optional adHocEnvironmentVariables.
  • Add --variable (repeatable KEY=VALUE) and --variable-file (simple .env-style) flags to apps builds create.
  • Parse/merge file + inline variables and include them in the build create request body.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
src/types/app-build.ts Adds adHocEnvironmentVariables?: Record<string, string> to the build creation DTO.
src/commands/apps/builds/create.ts Introduces new CLI flags, parses variables from inline/file, and sends them to appBuildsService.create().

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@robingenz robingenz changed the title feat(apps:builds): support ad hoc environment variables in create command feat(apps:builds:create): support ad hoc environment variables Mar 26, 2026
@robingenz robingenz merged commit 80c962a into main Mar 26, 2026
7 checks passed
@robingenz robingenz deleted the feat/ad-hoc-environment-variables branch March 26, 2026 16:11
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.

2 participants