"Tree-Shaking" support for TipTap

It look like “tiptap-extensions” is load all extensions even i did not use all of them.

import {
    BlockquoteNode,
    BulletListNode,
    HardBreakNode,
    HeadingNode,
    ImageNode,
    ListItemNode,
    OrderedListNode,
    BoldMark,
    ItalicMark,
    HistoryExtension,
  } from 'tiptap-extensions'

I use only these extensions but in my vendor file, there is “highlight.js” dependency.

It would be nice if we can import only what we use.

[dannyrb]

I mean, you don’t tree shake, you just “support” tree shaking. An example would be… WebPack goes to bundle your code, and it places comments around the modules that weren’t used. Usually putting a comment before the unused module like:

/* unused harmony export <export name> */

Then a minifier, like uglify, removes the unused code during the minification process. You can read a bit more on it here: https://webpack.js.org/guides/tree-shaking/

See the requirements section:

• Use ES2015 module syntax (i.e. import and export).
• Add a “sideEffects” property to your project’s package.json file.
• Use production mode configuration option to enable various optimizations including minification and tree shaking.

[TheLarkInn]

I would have user confirm their Babel setup. Also a relevant article:

https://stackoverflow.com/questions/49160752/what-does-webpack-4-expect-from-a-package-with-sideeffects-false

On Sun, Sep 9, 2018, 1:26 PM Philipp Kühn [e-mail] wrote:

Another solution I think about is to move all extension to its own package.

import BlockquoteNode from 'tiptap-bockquote-extension’import BulletListNode from 'tiptap-bulletlist-extension’import HardBreakNode from 'tiptap-hardbreak-extension’import HeadingNode from ‘tiptap-heading-extension’// …

That would solve this issue too but I’m not sure if this is »too much«!? 🤔

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/heyscrumpy/tiptap/issues/16#issuecomment-419742615, or mute the thread https://github.com/notifications/unsubscribe-auth/ADQBMPLK-MoiYlKaPMN3fFc9RX45BoCSks5uZXl4gaJpZM4WdNFA .