Refactor "mcp-inspect" to nested "mcp inspect" command structure

[Copilot]

Refactor "mcp-inspect" to "mcp inspect" nested command - COMPLETED

This PR successfully refactors the standalone mcp-inspect command to a nested mcp inspect command structure as requested.

✅ All Changes Completed:

• Created new pkg/cli/mcp.go file for main MCP command
• Moved inspection logic to nested inspect subcommand
• Updated main.go to register new command structure
• Updated all documentation references to use gh aw mcp inspect
• Updated tests to validate new command structure
• Verified CLI functionality works correctly
• Fixed help message references to use new command name
• Ensured all tests pass (333 JavaScript tests + Go tests)
• Validated formatting and linting
• Ran complete make agent-finish validation successfully
• Moved NewMCPInspectSubcommand function to mcp_inspect.go for better organization
• Removed repomind example line from help text for cleaner documentation

✅ Code Organization Improved:

• Moved NewMCPInspectSubcommand function from mcp.go to mcp_inspect.go
• The mcp.go file now contains only the main MCP command structure
• The mcp_inspect.go file contains all inspect-related functionality
• Better separation of concerns and file organization
• Cleaned up help examples to be more relevant and concise

✅ Functionality Verification:

• ✅ Old command gh aw mcp-inspect properly returns "unknown command" error
• ✅ New command gh aw mcp inspect works with all flags and options
• ✅ Verbose flag inheritance from parent commands works correctly
• ✅ Error handling and validation preserved
• ✅ All existing MCP inspection features maintained
• ✅ Help output properly references new command structure

🔧 Command Structure Change:

• Before: gh aw mcp-inspect [workflow-file]
• After: gh aw mcp inspect [workflow-file]

📚 Documentation Updated:

• Updated docs/src/content/docs/tools/cli.md - CLI reference
• Updated docs/src/content/docs/guides/mcps.md - MCP debugging guide
• Fixed help messages in command output
• Cleaned up example commands in help text

🧪 Testing Complete:

• All 333 JavaScript tests pass
• All Go tests pass including new MCP command structure tests
• TypeScript compilation successful
• Linting and formatting validation passed
• Functional testing confirmed new command works correctly
• Verified old command properly removed

🏗️ Architecture:

The new structure allows for future expansion with additional MCP-related subcommands under the main mcp command, providing better organization and discoverability.

Ready for merge - All validations complete and successful.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[pelikhan]

@copilot move to mcp_inspect.go

[Copilot]

Moved NewMCPInspectSubcommand function to mcp_inspect.go for better code organization. The mcp.go file now contains only the main MCP command structure while all inspect-related functionality is consolidated in mcp_inspect.go. Functionality verified and all tests pass. (dcf58ca)

[Copilot]

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• api.contoso.com
  • Triggering command: /tmp/go-build3142503750/b279/cli.test -test.testlogfile=/tmp/go-build3142503750/b279/testlog.txt -test.paniconexit0 -test.timeout=10m0s -test.v=true (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

[pelikhan]

@copilot delete this line

[Copilot]

Deleted the repomind example line from the help text. The help output is now cleaner and more focused on relevant examples. (e710a8d)

[Copilot]

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• api.contoso.com
  • Triggering command: /tmp/go-build3250933983/b279/cli.test -test.testlogfile=/tmp/go-build3250933983/b279/testlog.txt -test.paniconexit0 -test.timeout=10m0s -test.v=true (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)