[Bug] Doesn't work with Yarn PNP

[xanderim]

Hi, thank you for the package!

• After installing and importing the package in monorepo project with Yarn PNP enabled

- After adding the missing dependency manually

the code:

import { useState } from 'react';
import RichTextEditor, { BaseKit } from 'reactjs-tiptap-editor';

// Import CSS
import 'reactjs-tiptap-editor/style.css';

const extensions = [
  BaseKit.configure({
    // Show placeholder
    placeholder: {
      showOnlyCurrent: true,
    },

    // Character count
    characterCount: {
      limit: 50_000,
    },
  }),
];

const DEFAULT = '';

const Editor = () => {
  const [content, setContent] = useState(DEFAULT);

  const onChangeContent = (value: any) => {
    setContent(value);
  };

  return (
    <RichTextEditor
      output='html'
      content={content}
      onChangeContent={onChangeContent}
      extensions={extensions}
    />
  );
};

export default Editor;

I understand that it's not related to reactjs-tiptap-editor directly, but I don't use the mermaid extension and don't plan to. Is it possible to excluded extensions that I don't use from the build? Thanks.

similar issue facebook/docusaurus#10645

[hunghg255]

Hi, thank you for the package!

• After installing and importing the package in monorepo project with Yarn PNP enabled

- After adding the missing dependency manually
the code:

import { useState } from 'react';
import RichTextEditor, { BaseKit } from 'reactjs-tiptap-editor';

// Import CSS
import 'reactjs-tiptap-editor/style.css';

const extensions = [
  BaseKit.configure({
    // Show placeholder
    placeholder: {
      showOnlyCurrent: true,
    },

    // Character count
    characterCount: {
      limit: 50_000,
    },
  }),
];

const DEFAULT = '';

const Editor = () => {
  const [content, setContent] = useState(DEFAULT);

  const onChangeContent = (value: any) => {
    setContent(value);
  };

  return (
    <RichTextEditor
      output='html'
      content={content}
      onChangeContent={onChangeContent}
      extensions={extensions}
    />
  );
};

export default Editor;

I understand that it's not related to reactjs-tiptap-editor directly, but I don't use the mermaid extension and don't plan to. Is it possible to excluded extensions that I don't use from the build? Thanks.

similar issue facebook/docusaurus#10645

this lib has some problem with yarn install, I am not sure. can you try use npm or pnpm

[xanderim]

@hunghg255 thank you for the quick reply. I think the main issue here is that unused extensions should not be bundled into the main package. It would make the package size smaller and avoid issues with external dependencies. I am not an expert on tree-shaking but I assume all extensions should be tree-shakeable.

[condorheroblog]

Yes, I think so too. Some dependencies should be marked as external dependencies and let users install them themselves, such as docx, mermaid, etc. These dependencies are large and heavy.