Skip to content

fix: change module type to "module" - #72

Merged
ocavue merged 1 commit into
mainfrom
ocavue/esm-first
Feb 19, 2026
Merged

fix: change module type to "module"#72
ocavue merged 1 commit into
mainfrom
ocavue/esm-first

Conversation

@ocavue

@ocavue ocavue commented Feb 19, 2026

Copy link
Copy Markdown
Collaborator

Closes #71

Update package.json to matches ProseMirror's official packages (e.g. prosemirror-history/package.json):

  • Add "type": "module"
    • Before: index.js is Common JS, index.mjs is ESM
    • After: index.js is ESM, index.cjs is Common JS
  • Add "exports": { ... }
  • Add "sideEffects": false. Good for tree shaking.
  • Move prosemirror-* from peerDependencies to dependencies.

Update the build tools from Rollup to tsdown. The configuration is simpler now.

@github-actions

github-actions Bot commented Feb 19, 2026

Copy link
Copy Markdown
Contributor

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 11.49% 621 / 5402
🔵 Statements 11.49% 621 / 5402
🔵 Functions 47.72% 21 / 44
🔵 Branches 62.75% 91 / 145
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
tsdown.config.mjs 0% 0% 0% 0% 1-11
Generated in workflow #172 for commit 07b6fc0 by the Vitest Coverage Report Action

@ocavue
ocavue marked this pull request as ready for review February 19, 2026 12:19
@ocavue
ocavue merged commit 050a9cc into main Feb 19, 2026
6 of 7 checks passed
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.

Type unmatched when using "moduleResolution": "nodenext" in tsconfig.json

2 participants