fix(core-utils): remove domino from the codebase (#1833)

.changeset/witty-vans-roll.md

@@ -0,0 +1,5 @@
+---
+'@remirror/core-utils': patch
+---
+
+Removes `domino` from the codebase.

packages/remirror__core-utils/__tests__/core-utils.node.spec.ts

@@ -0,0 +1,53 @@
+/**
+ * @jest-environment node
+ */
+
+import { doc, p, schema as testSchema } from 'jest-prosemirror';
+
+import { htmlToProsemirrorNode, prosemirrorNodeToDom } from '../';
+
+describe('toDOM', () => {
+  const node = doc(p('hello'));
+
+  it('allows for custom document to be passed in', async () => {
+    const { JSDOM } = await import('jsdom');
+    const customDocument = new JSDOM().window.document;
+    expect(prosemirrorNodeToDom(node, customDocument)).toBeObject();
+
+    const wrongDocument = {} as Document;
+    expect(() => prosemirrorNodeToDom(node, customDocument)).not.toThrow();
+    expect(() => prosemirrorNodeToDom(node, wrongDocument)).toThrow();
+  });
+});
+
+describe('htmlToProsemirrorNode', () => {
+  const content = `<p>Hello</p>`;
+
+  it('allows for custom document to be passed in', async () => {
+    const { JSDOM } = await import('jsdom');
+    const customDocument = new JSDOM().window.document;
+    expect(
+      htmlToProsemirrorNode({
+        content: content,
+        schema: testSchema,
+        document: customDocument,
+      }),
+    ).toEqualProsemirrorNode(doc(p('Hello')));
+
+    const wrongDocument = {} as Document;
+    expect(() =>
+      htmlToProsemirrorNode({
+        content: content,
+        schema: testSchema,
+        document: customDocument,
+      }),
+    ).not.toThrow();
+    expect(() =>
+      htmlToProsemirrorNode({
+        content: content,
+        schema: testSchema,
+        document: wrongDocument,
+      }),
+    ).toThrow();
+  });
+});

packages/remirror__core-utils/__tests__/core-utils.spec.ts

@@ -1,4 +1,7 @@
-import domino from 'domino';
+/**
+ * @jest-environment jsdom
+ */
+
 import {
   a,
   atomInline,
@@ -735,10 +738,6 @@ describe('toDOM', () => {
   it('transforms a doc into a documentFragment', () => {
     expect(prosemirrorNodeToDom(node)).toBeInstanceOf(DocumentFragment);
   });
-
-  it('allows for custom document to be passed in', () => {
-    expect(prosemirrorNodeToDom(node, domino.createDocument())).toBeObject();
-  });
 });
 
 describe('htmlToProsemirrorNode', () => {
@@ -749,16 +748,6 @@ describe('htmlToProsemirrorNode', () => {
       doc(p('Hello')),
     );
   });
-
-  it('allows for custom document to be passed in', () => {
-    expect(
-      htmlToProsemirrorNode({
-        content: content,
-        schema: testSchema,
-        document: domino.createDocument(),
-      }),
-    ).toEqualProsemirrorNode(doc(p('Hello')));
-  });
 });
 
 test('getRemirrorJSON', () => {

packages/remirror__core-utils/package.json

@@ -44,22 +44,17 @@
     "@remirror/pm": "^2.0.0-beta.14",
     "@types/jsdom": "^16.2.13",
     "@types/node": "^16.3.3",
-    "domino": "^2.1.6",
     "jsdom": "^17.0.0"
   },
   "peerDependencies": {
     "@remirror/pm": "^2.0.0-beta.14",
     "@types/node": "*",
-    "domino": "*",
     "jsdom": "*"
   },
   "peerDependenciesMeta": {
     "@types/node": {
       "optional": true
     },
-    "domino": {
-      "optional": true
-    },
     "jsdom": {
       "optional": true
     }

packages/remirror__core-utils/readme.md

@@ -20,4 +20,4 @@ This is included by default when you install the recommended `remirror` package.
 
 If you plan to support SSR and need to parse the html contents of the editor in an SSR environment then `min-document` is automatically added to all node environments.
 
-However, `min-document` can't actually parse the content properly since the implementation is intentionally underpowered. To properly parse content from a html string you will need to install either `jsdom` or `domino`. These dependencies are only included within non-browser builds and won't bloat your bundle size in the browser.
+However, `min-document` can't actually parse the content properly since the implementation is intentionally underpowered. To properly parse content from a html string you will need to install `jsdom`. These dependencies are only included within non-browser builds and won't bloat your bundle size in the browser.

packages/remirror__core-utils/src/core-utils.ts

@@ -1220,13 +1220,6 @@ export function getDocument(): Document {
     // ignore
   }
 
-  try {
-    // eslint-disable-next-line @typescript-eslint/no-var-requires
-    return require('domino').createDocument();
-  } catch {
-    // ignore
-  }
-
   try {
     // eslint-disable-next-line @typescript-eslint/no-var-requires
     return require('min-document');

support/root/.size-limit.json

@@ -114,7 +114,6 @@
     "ignore": [
       "@remirror/pm",
       "@types/node",
-      "domino",
       "jsdom",
       "@remirror/core-constants",
       "@remirror/core-helpers",