Skip to content

chore: remove VPC feature from CLI#466

Open
tejaskash wants to merge 3 commits intomainfrom
chore/remove-vpc-feature
Open

chore: remove VPC feature from CLI#466
tejaskash wants to merge 3 commits intomainfrom
chore/remove-vpc-feature

Conversation

@tejaskash
Copy link
Contributor

Summary

  • Reverts 3 VPC feature commits that were merged to main — the related feature-branch PRs (TUI wizard, template conditionals) were already discarded
  • Removes VPC CLI flags from create and add commands, VPC info messages from dev and invoke, and VPC schema additions
  • Deletes src/cli/commands/shared/vpc-utils.ts entirely
  • Keeps expected pre-existing references: VpcConfigSchema in deployed-state.ts (CDK state tracking) and NetworkModeSchema = ['PUBLIC', 'PRIVATE'] in constants.ts

Approach

Three individual git revert commits (kept separate for traceability):

  1. Revert c75f4cd — VPC CLI flags (create/add commands, 14 files)
  2. Revert 7a81b02 — VPC info messages (dev/invoke, 2 files)
  3. Revert 4180646 — VPC schema changes (7 files)

Test plan

  • npm test — 149 test files passed (1756 tests, 0 failures)
  • npm run build — TypeScript compilation + CLI bundle succeeds
  • Grep verification: no residual networkConfig, NetworkConfig, or vpc-utils references in src/
  • Only expected VPC references remain (negative tests + deployed-state CDK tracking)

@tejaskash tejaskash requested a review from a team February 27, 2026 00:00
@github-actions github-actions bot added the size/l PR size: L label Feb 27, 2026
@github-actions
Copy link
Contributor

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 44.12% 2996 / 6790
🔵 Statements 43.68% 3153 / 7217
🔵 Functions 45.92% 620 / 1350
🔵 Branches 48.49% 1935 / 3990
Generated in workflow #775 for commit f2526ef by the Vitest Coverage Report Action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/l PR size: L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant