fix command order for install command

[jtcorbett]

Summary by CodeRabbit

• Chores
  • Updated development environment configuration to optimize file watching.
• Refactor
  • Streamlined install command registration structure for improved maintainability.

[jtcorbett]

• fix command order for install command #586 👈 (View in Graphite)
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

The changes refactor the CLI command registration system by removing the top-level Typer app from the install command module and updating the main CLI entry point to register the install command as a direct function-based command instead of mounting it as a sub-app. Additionally, a VSCode settings file is configured to exclude target directories from file watching.

Changes

Cohort / File(s)	Summary
VSCode Configuration \.vscode/settings\.json	Added files.watcherExclude setting to ignore all target directories.
Install Command Simplification src/mcp_agent/cli/commands/install\.py	Removed top-level Typer app initialization and the @app.callback(invoke_without_command=True) decorator; the install function logic remains unchanged.
CLI Command Registration src/mcp_agent/cli/main\.py	Changed install command registration from app.add_typer(install_cmd.app, ...) to direct function command via app.command(...)(install_cmd.install).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• PR Second pass at implementing the mcp-agent CLI #462: Related CLI command registration changes in main.py affecting how subcommands are mounted.
• PR add install command to CLI #565: Related refactoring of install command wiring that removes Typer sub-app approach.

Suggested reviewers

• rholinshead
• saqadri

Poem

🐰 CLI commands now flow straight and clean,
No nested apps cluttering the scene,
Direct functions light the way,
Simpler wiring, brighter day!
Target folders hide from sight,
All the command paths feel right. ✨

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch 10-23-fix_command_order_for_install_command

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 127a2be and 583beab.

📒 Files selected for processing (3)

• .vscode/settings.json (1 hunks)
• src/mcp_agent/cli/commands/install.py (0 hunks)
• src/mcp_agent/cli/main.py (1 hunks)

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}