-
Notifications
You must be signed in to change notification settings - Fork 170
Add interactive wizard for thv run command (#3050)
#3353
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Add `--interactive` / `-i` flag that launches a step-by-step TUI wizard using bubbletea to guide users through configuring an MCP server. The wizard reduces the barrier to entry for advanced features without requiring users to memorize numerous flags. Features: - 5-step guided flow: Server Source → Selection → Transport → Advanced → Preview - Support for 4 server sources: registry, container image, protocol schemes, remote URL - Searchable registry server list with fuzzy filtering - Command preview before execution - Vim-style (j/k) and arrow key navigation Signed-off-by: carlos <21148423+carlos-gn@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Large PR Detected
This PR exceeds 1000 lines of changes and requires justification before it can be reviewed.
How to unblock this PR:
Add a section to your PR description with the following format:
## Large PR Justification
[Explain why this PR must be large, such as:]
- Generated code that cannot be split
- Large refactoring that must be atomic
- Multiple related changes that would break if separated
- Migration or data transformationAlternative:
Consider splitting this PR into smaller, focused changes (< 1000 lines each) for easier review and reduced risk.
See our Contributing Guidelines for more details.
This review will be automatically dismissed once you add the justification section.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3353 +/- ##
==========================================
+ Coverage 64.82% 64.84% +0.01%
==========================================
Files 375 375
Lines 36626 36626
==========================================
+ Hits 23744 23751 +7
+ Misses 11011 11002 -9
- Partials 1871 1873 +2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Signed-off-by: carlos <21148423+carlos-gn@users.noreply.github.com>
bb9b11a to
7076d45
Compare
|
Thanks for the PR @carlos-gn! I'm going to close this PR and the original spike. |
|
Hi @eleftherias I agree. No worries. Let me know if you want me to bring this to the app or work on something else with higher priority. |
|
@carlos-gn This issue about persisting the dynamic OAuth client might be interesting to you #3335. Or let me know if something else catches your eye. |
Summary
remote URLs
Test plan
Closes #3050