remove references to npm, clean up package manager usage in workspaces by josefaidt · Pull Request #1534 · modelcontextprotocol/typescript-sdk
Motivation and Context
The main branch includes usage of both pnpm and npm, where pnpm seems more prevalent. This PR:
- removes all references to npm throughout the repo
- removes
package-lock.json, consolidates on pnpm usage - consolidates usage of
packageManagerto the repo root, it is not needed in workspace packages - removes
pnpmentries fromengines, thepackageManagerkey is sufficient - removes
manage-package-manager-versionsfrom.npmrc, versions are now consolidated to root and can be managed by corepack -- contributing guide states to use corepack - removes unused
startnpm scripts that invokenpm start server-- these files are not found in workspace packages - migrates
checknpm scripts to use pnpm as the package manager
I'd also suggest considering further cleanup of the gitignore file. I removed what I thought was unnecessary given the state of the repo and package manager of choice, but there are still plenty of entries that do not seem to be needed (like .node_repl_history and .lock-wscript)
How Has This Been Tested?
confirmed scripts still work as expected
Breaking Changes
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