Skip to content

chore: excluded tests from src folder in npm package#11958

Merged
lgrammel merged 6 commits intomainfrom
lg/0bRlswPL
Jan 22, 2026
Merged

chore: excluded tests from src folder in npm package#11958
lgrammel merged 6 commits intomainfrom
lg/0bRlswPL

Conversation

@lgrammel
Copy link
Copy Markdown
Collaborator

@lgrammel lgrammel commented Jan 22, 2026

Background

The src folders in the npm packages, which are published primarily as grep targets for coding agents, include tests and related files. This increases package size and makes searching by coding agents less efficient.

Summary

Exclude tests files, test fixtures, test snapshots from src in npm packages.

Manual Verification

  • verify that snapshot does not include test files

Related Issues

Sources in npm packages were introduced in #11950 and #11790

@vercel-ai-sdk vercel-ai-sdk bot added the maintenance CI, internal documentation, automations, etc label Jan 22, 2026
@lgrammel lgrammel marked this pull request as ready for review January 22, 2026 15:14
"dist/**/*",
"docs/**/*",
"src",
"!src/**/*.test.ts",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The 'ai' package is missing the '!src/**/fixtures' exclusion from its npm package files configuration, creating an inconsistency with all other core packages

Fix on Vercel

@lgrammel lgrammel merged commit 4de5a1d into main Jan 22, 2026
19 checks passed
@lgrammel lgrammel deleted the lg/0bRlswPL branch January 22, 2026 16:02
B-Step62 pushed a commit to B-Step62/ai that referenced this pull request Jan 28, 2026
## Background

The `src` folders in the npm packages, which are published primarily as
grep targets for coding agents, include tests and related files. This
increases package size and makes searching by coding agents less
efficient.

## Summary

Exclude tests files, test fixtures, test snapshots from `src` in npm
packages.

## Manual Verification

- [x] verify that snapshot does not include test files

## Related Issues

Sources in npm packages were introduced in vercel#11950 and vercel#11790
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance CI, internal documentation, automations, etc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants