Skip to content

chore(everything): exclude compiled tests and vitest config from dist - #4933

Closed
bluetomlee wants to merge 1 commit into
modelcontextprotocol:mainfrom
bluetomlee:fix/4930-everything-dist-test-artifacts
Closed

bluetomlee wants to merge 1 commit into
modelcontextprotocol:mainfrom
bluetomlee:fix/4930-everything-dist-test-artifacts

Conversation

@bluetomlee

Copy link
Copy Markdown

Fixes #4930

Summary

src/everything/tsconfig.json had no exclude, so tsc emitted __tests__/** and vitest.config.js into dist/, and "files": ["dist"] published them. This adds the same exclude list src/filesystem/tsconfig.json already uses, so the tarball ships only the server and its docs/, matching the other three TS servers.

Changes

  • src/everything/tsconfig.json — add exclude: **/__tests__/**, **/*.test.ts, **/*.spec.ts, vitest.config.ts (list taken verbatim from src/filesystem/tsconfig.json).

Verification (before / after)

Clean rebuild of src/everything (npm install → npm run build), original config vs. this branch:

dist files test artifacts in dist
before 50 6 (__tests__/{prompts,registrations,resources,server,tools}.test.js + vitest.config.js)
after 44 0

npm pack --dry-run after the change lists only README.md, the compiled server files under dist/, and dist/docs/* — no tests, no test config.

Tests are unaffected (Vitest compiles from source, not dist/): npx vitest run → 109 passed (5 test files), identical before and after.

Checklist

The build tsconfig had no exclude, so tsc emitted __tests__/** and
vitest.config into dist/ and "files": ["dist"] published them. Align
the exclude with src/filesystem's tsconfig so the tarball ships only the
server and its docs, like the other three TS servers.

Fixes modelcontextprotocol#4930

Signed-off-by: Kevin Li <tomliyi@qq.com>
@changeset-bot

changeset-bot Bot commented Oct 1, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: da3b379

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@cliffhall

Copy link
Copy Markdown
Member

Thanks, @bluetomlee but this was meant to be against v2/main with changesets.

@cliffhall cliffhall closed this Oct 4, 2026
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.

everything: compiled tests and vitest.config ship in the published tarball

2 participants