Skip to content

fix: keep runtime files in mcpb packages#253

Open
he-yufeng wants to merge 1 commit into
modelcontextprotocol:mainfrom
he-yufeng:fix/runtime-safe-default-excludes
Open

fix: keep runtime files in mcpb packages#253
he-yufeng wants to merge 1 commit into
modelcontextprotocol:mainfrom
he-yufeng:fix/runtime-safe-default-excludes

Conversation

@he-yufeng
Copy link
Copy Markdown

Summary

Fixes #241.

This narrows the default source map excludes and keeps node_modules/.bin in packaged bundles. The old defaults could drop runtime files in two common cases:

  • package directories whose names end in .map, such as Iterator.prototype.map/
  • CLI shims under node_modules/.bin that packages probe at module load time

The default still excludes normal source map files such as *.js.map, *.cjs.map, *.mjs.map, *.css.map, and *.ts.map.

To verify

  • yarn test test/mcpbignore.test.ts
  • yarn lint
  • yarn build
  • git diff --check

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.

Default EXCLUDE_PATTERNS in mcpb pack is too broad

1 participant