fix: handle npx with package@version syntax on Windows for MCP servers

[roomote]

This PR attempts to address Issue #9167. Feedback and guidance are welcome.

Problem

The chrome-devtools-mcp server was not being detected when configured with npx chrome-devtools-mcp@latest on Windows systems. This was due to how the command was being wrapped with cmd.exe for Windows compatibility.

Solution

Added special handling for npx commands that include @ in the package name (e.g., chrome-devtools-mcp@latest):

• On Windows, when npx is used with a package containing @, we now combine npx and the package name into a single command string
• This ensures proper package resolution when wrapped with cmd.exe
• The fix maintains backward compatibility with other command types

Changes

• Modified McpHub.ts to detect and handle npx with versioned packages
• Added comprehensive test coverage for the new behavior
• Tests verify correct handling of:
  • npx package@version syntax
  • npx package@version with additional arguments
  • Non-Windows platforms (no change in behavior)
  • Already wrapped commands (no double-wrapping)

Testing

• All existing tests pass
• Added new tests specifically for this scenario
• Verified the fix handles various package@version formats

Fixes #9167

---

Important

Fixes handling of npx with package@version syntax on Windows in McpHub.ts, ensuring proper command execution with cmd.exe.

• Behavior:
  • Fixes handling of npx commands with package@version syntax on Windows in McpHub.ts.
  • Combines npx and package name into a single command string for Windows compatibility.
  • Ensures proper command execution when wrapped with cmd.exe.
• Testing:
  • Adds tests in McpHub.spec.ts for npx package@version syntax handling on Windows.
  • Tests include scenarios with additional arguments and non-Windows platforms.
  • Verifies no double-wrapping of commands already using cmd.exe.

^{This description was created by for 14a8021. You can customize this summary. It will automatically update as commits are pushed.}

[roomote]

Rooviewer   See task on Roo Cloud

Review completed. No issues found.

This PR successfully addresses Issue #9167 by adding special handling for npx commands with package@version syntax on Windows. The implementation is clean, well-tested, and maintains backward compatibility.

Key improvements:

• Properly handles npx package@version syntax on Windows by combining them into a single command string
• Comprehensive test coverage for various scenarios
• No breaking changes to existing functionality

_{Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.}