Add optional --fast-mode flag to skip heavy preflight checks - #28
Open
ARULGNANAKUMAR wants to merge 1 commit into
Open
Add optional --fast-mode flag to skip heavy preflight checks#28ARULGNANAKUMAR wants to merge 1 commit into
ARULGNANAKUMAR wants to merge 1 commit into
Conversation
Introduces --fast-mode flag to reduce wizard startup time by skipping heavy quota and network validation checks. This PR introduces an optional --fast-mode flag to reduce wizard startup time by skipping heavy quota and network validation checks. - Default behavior remains unchanged - Intended for development/testing use - No breaking changes
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
| response.read() | ||
|
|
||
| console.print(" [green]✓[/] API key is valid") | ||
| console.print(" [green]v[/] API key is valid") |
There was a problem hiding this comment.
Unicode status symbols replaced with plain ASCII letters
Medium Severity
All unicode status symbols throughout the file have been replaced with plain ASCII letters — ✓ → v, ✗ → x, and ⚠ → !. This affects ~40 lines across all cloud providers and all code paths, not just fast-mode. A green v is not a recognizable success indicator the way ✓ is, making CLI output harder to scan. This appears to be an unintentional change bundled with the --fast-mode feature, possibly caused by an editor or tool stripping unicode characters.
Additional Locations (2)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Introduces --fast-mode flag to reduce wizard startup time by skipping heavy quota and network validation checks.
This PR introduces an optional --fast-mode flag to reduce wizard startup time by skipping heavy quota and network validation checks.
Problem
Describe the purpose of this change. What problem is being solved and why?
Solution
Describe the approach you took. Link to any relevant bugs, issues, docs, or other resources.
Type of Change
Test Plan
Describe specific steps for validating this change.
Note
Medium Risk
Adds an opt-in path that skips quota/capacity validation across AWS/GCP/Azure, which could let users proceed with unsupported deployments if used unintentionally. Default behavior is unchanged, so impact is limited to callers enabling the flag.
Overview
Adds an optional
--fast-modeflag to the BYOC setup wizard that skips heavy quota/capacity preflight checks while still running basic validation (AZ/zone availability and CIDR conflict checks) for AWS, GCP, and Azure.Propagates
fast_modefromrun_setup()through each wizard into the cloud-specific*PreflightChecker, prints a prominent warning when fast mode is enabled, and standardizes console status icons from✓/✗/⚠tov/x/!.Written by Cursor Bugbot for commit fe9cc1c. This will update automatically on new commits. Configure here.