Skip to content

fix: correct rollup-plugin-copy types export#3104

Open
luuhongyii wants to merge 2 commits into
modernweb-dev:masterfrom
luuhongyii:codex/fix-rollup-copy-types-export
Open

fix: correct rollup-plugin-copy types export#3104
luuhongyii wants to merge 2 commits into
modernweb-dev:masterfrom
luuhongyii:codex/fix-rollup-copy-types-export

Conversation

@luuhongyii

Copy link
Copy Markdown

Summary

  • Point @web/rollup-plugin-copy's root export types condition to ./dist/copy.d.ts.
  • Keep the existing import and require runtime entries unchanged.

Why

The published @web/rollup-plugin-copy@0.5.1 package exports root types from ./index.d.ts, but that file is not included in the npm tarball. The package-level types field already points to dist/copy.d.ts, which is included, so this aligns the conditional export with the existing published declaration file.

Validation

  • node -e "const p=require('./packages/rollup-plugin-copy/package.json'); if(p.exports['.'].types!=='./dist/copy.d.ts') process.exit(1); console.log('manifest export types ok')"
  • git diff --check
  • Verified npm tarball evidence for @web/rollup-plugin-copy@0.5.1:
    • current exports["."].types: ./index.d.ts exists: false
    • package-level types: dist/copy.d.ts exists: true
    • proposed exports["."].types: ./dist/copy.d.ts exists: true

@changeset-bot

changeset-bot Bot commented Jun 13, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 02b7abb

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@web/rollup-plugin-copy Patch

Not sure what this means? Click here to learn what changesets are.

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

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.

1 participant