Skip to content

feat!: flash deploy auto builds, automatic app & env management#178

Merged
KAJdev merged 10 commits intomainfrom
zeke/ae-2040-p0-merge-build-step-into-flash-deploy
Feb 6, 2026
Merged

feat!: flash deploy auto builds, automatic app & env management#178
KAJdev merged 10 commits intomainfrom
zeke/ae-2040-p0-merge-build-step-into-flash-deploy

Conversation

@KAJdev
Copy link
Contributor

@KAJdev KAJdev commented Feb 5, 2026

currently, you have to do a multi-step process of creating a build archive and flash deploy send to send it to an environment. this is mostly just busywork.

This PR focused on removing all that friction and presenting with a simple monolithic flash deploy command that will:

  1. create a build archive
  2. ensure an app and environment exist (can specify both with flags)
  3. deploy the archive to the resolved environment

we default to the current directory name for the app name. If the app has a single environment, it will just be automatically used to deploy. If no environments exist, a production environment will be created. If multiple exist, you are asked to specify one. specifying an environment or app with --env or --app that doesnt exist will automatically create it.

flash deploy also had a bunch of environment management sub-commands, these have been moved to flash env. (flash deploy new has been moved to flash env create to be more in line with other commands).

@KAJdev KAJdev marked this pull request as ready for review February 5, 2026 22:11
@KAJdev KAJdev requested review from deanq and jhcipar February 5, 2026 22:11
Copy link
Member

@DeJayDev DeJayDev left a comment

Choose a reason for hiding this comment

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

fantastic, thank you - just these but not a hard blocker

KAJdev and others added 2 commits February 5, 2026 13:49
Co-authored-by: Dj Isaac <contact@dejaydev.com>
Co-authored-by: Dj Isaac <contact@dejaydev.com>
@deanq deanq requested a review from Copilot February 6, 2026 00:37
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

This PR introduces a streamlined deployment workflow by consolidating the build and deploy steps into a single flash deploy command, while also reorganizing environment management commands under a new flash env namespace. The changes eliminate manual app and environment creation by automatically handling these resources based on flags and discovery.

Changes:

  • Unified build and deployment into flash deploy command with automatic app/environment creation
  • Moved environment management from flash deploy <subcommand> to flash env <subcommand>
  • Refactored flag naming from --app-name to --app for consistency

Reviewed changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tests/unit/cli/test_env.py New test suite covering environment management commands (list, create, info, delete)
tests/unit/cli/test_deploy.py Updated tests for new unified deploy command with automatic environment resolution
tests/unit/cli/test_apps.py Updated flag names from --app-name to --app
src/runpod_flash/cli/utils/skeleton_template/README.md Updated deployment examples to reflect new single-command workflow
src/runpod_flash/cli/utils/deployment.py Updated error message to reference flash deploy instead of flash build
src/runpod_flash/cli/main.py Restructured command registration: added flash deploy as direct command and flash env as subcommand group
src/runpod_flash/cli/commands/env.py New environment management module with list, create, info, delete commands
src/runpod_flash/cli/commands/deploy.py Completely rewritten to handle build + deploy workflow with automatic environment resolution
src/runpod_flash/cli/commands/build.py Refactored to extract core build logic into run_build() for reuse by deploy command
src/runpod_flash/cli/commands/apps.py Updated flag name from --app-name to --app

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

@KAJdev KAJdev merged commit cee5e74 into main Feb 6, 2026
6 checks passed
@KAJdev KAJdev deleted the zeke/ae-2040-p0-merge-build-step-into-flash-deploy branch February 6, 2026 00:40
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.

4 participants