Skip to content

perf(editor): reduce default editor bundle size - #19

Merged
cdskill merged 1 commit into
mainfrom
bench/refresh-bundle-size
Jun 24, 2026
Merged

perf(editor): reduce default editor bundle size#19
cdskill merged 1 commit into
mainfrom
bench/refresh-bundle-size

Conversation

@cdskill

@cdskill cdskill commented Jun 24, 2026

Copy link
Copy Markdown
Owner

Summary

  • move the table plugin to the @qalma/editor/table secondary entry point so prosemirror-tables stays out of the main entry
  • annotate plugin factory exports and add a guard test so unused plugins remain tree-shakeable
  • refresh bundle-size docs and benchmark results for the local 0.1.0-beta.2 candidate

Validation

  • pnpm nx build editor
  • pnpm nx test editor --runInBand
  • pnpm exec tsc -p libs/editor/tsconfig.lib.json --noEmit
  • pnpm exec tsc -p apps/docs/tsconfig.json --noEmit
  • pnpm guard:protected-test-changes
  • npm run bench from bench/bundle-size

Move the table plugin to a secondary entry point so prosemirror-tables stays out of the main package entry, annotate plugin factories for tree-shaking, and refresh the bundle-size benchmark/docs for the beta.2 candidate.
@cdskill cdskill changed the title [codex] reduce default editor bundle size chore(editor): reduce default editor bundle size Jun 24, 2026
@cdskill
cdskill marked this pull request as ready for review June 24, 2026 23:04
@cdskill cdskill added the approved-test-change Human-approved change to tests, CI, or the test-change guard label Jun 24, 2026
@cdskill cdskill changed the title chore(editor): reduce default editor bundle size perf(editor): reduce default editor bundle size Jun 24, 2026
@cdskill
cdskill merged commit ef86289 into main Jun 24, 2026
3 of 4 checks passed
@cdskill
cdskill deleted the bench/refresh-bundle-size branch June 24, 2026 23:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved-test-change Human-approved change to tests, CI, or the test-change guard

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant