Skip to content

Conversation

@olaservo
Copy link
Member

@olaservo olaservo commented Feb 14, 2025

Motivation and Context

See: #134

How Has This Been Tested?

This workaround has been tested on a Windows 10 machine.

Breaking Changes

None

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

There are proposed fixes for TSX, but it appears this would introduce a breaking change for TSX, and there is no known timeline for the next major version: privatenumber/tsx#639 (comment)

Copy link
Member

@jspahrsummers jspahrsummers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the suggestion!

Perhaps we should turn this into a package.json script of its own? e.g., npm run dev:windows (or something like that) might be easier and less error-prone than asking users to do it themselves.

@olaservo
Copy link
Member Author

@jspahrsummers yes that is a better idea! I made this change.

Copy link
Member

@jspahrsummers jspahrsummers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thank you!

@jspahrsummers jspahrsummers merged commit 2788097 into modelcontextprotocol:main Feb 19, 2025
2 checks passed
@olaservo olaservo deleted the add-note-for-dev-mode-on-windows branch February 22, 2025 03:06
IgnacioC44 referenced this pull request in MCPJam/inspector Jun 21, 2025
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.

2 participants