Skip to content

Add --vscode flag to init CLI for MCP configuration#81

Merged
fajarhide merged 1 commit intomainfrom
fix/vscode
May 8, 2026
Merged

Add --vscode flag to init CLI for MCP configuration#81
fajarhide merged 1 commit intomainfrom
fix/vscode

Conversation

@fajarhide
Copy link
Copy Markdown
Owner

@fajarhide fajarhide commented May 7, 2026

PR Auto Describe

Summary

Added VS Code (MCP) support to the CLI init command.
Introduces a --vscode flag, updates help text, interactive menu, and default option handling to include “VS Code (MCP)”.
All logic paths now recognize the new flag and add "vscode" to the list of configuration IDs.


Key Changes

  • New --vscode CLI flag and help entry.
  • Updated interactive selection to show option 13 and parse it.
  • Expanded default option list and ID vector to include "vscode".
  • Adjusted conditional checks to account for the new flag.

Detailed Breakdown

  • Help display (print_help): added line --vscode with cyan formatting.
  • Argument parsing (run_init):
    • New boolean is_vscode set by args.iter().any(|a| a == "--vscode").
    • Updated default‑only check: && !is_vscode.
  • Interactive menu:
    • Added println!(" [{}] VS Code (MCP)", "13".cyan());.
    • Prompt text changed to [1-13, q].
    • Match arm "13" => is_vscode = true.
  • Default options list: added "vscode" to the vector of IDs when no specific flags are supplied.
  • Configuration IDs (ids vector): added if is_vscode { ids.push("vscode"); } after existing checks.

Notes

None beyond the new flag; all integration points now handle "vscode" consistently.


Breaking Changes

No breaking changes. The new flag is additive and does not alter existing behavior unless explicitly requested.

Last updated: 2026-05-07 16:45:28

@fajarhide fajarhide merged commit e3e224f into main May 8, 2026
4 checks passed
@fajarhide fajarhide deleted the fix/vscode branch May 8, 2026 07:23
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.

1 participant