chore: remove VPC feature from CLI by tejaskash · Pull Request #466 · aws/agentcore-cli · GitHub
Skip to content

chore: remove VPC feature from CLI#466

Merged
tejaskash merged 3 commits into
mainfrom
chore/remove-vpc-feature
Feb 27, 2026
Merged

chore: remove VPC feature from CLI#466
tejaskash merged 3 commits into
mainfrom
chore/remove-vpc-feature

Conversation

@tejaskash

Copy link
Copy Markdown
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
Copy Markdown
Contributor

@tejaskash tejaskash merged commit 3e8a72f into main Feb 27, 2026
15 of 18 checks passed
@tejaskash tejaskash deleted the chore/remove-vpc-feature branch February 27, 2026 15:59
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.

2 participants