Ensure defaultNode has highest priority (#4985)

Author: myovchev

modules/@apostrophecms/rich-text-widget/ui/apos/components/AposRichTextWidgetEditor.vue

@@ -435,6 +435,8 @@ export default {
       .filter(Boolean)
       .concat(this.aposTiptapExtensions());
 
+    this.ensureExtensionsPriority(extensions);
+
     this.editor = new Editor({
       content: this.initialContent,
       autofocus: this.autofocus,
@@ -670,6 +672,21 @@ export default {
           types: this.tiptapTypes
         }));
     },
+    // Find the `defaultNode` extension and ensure it's registered first.
+    // Any other priority related logic should be handled here.
+    // Why sorting is important?
+    // See https://github.com/ProseMirror/prosemirror/issues/1534#issuecomment-2984216986
+    // related with list item issues and `defaultNode`.
+    // NOTE: this handler mutates the input array for performance reasons.
+    ensureExtensionsPriority(extensions) {
+      const defaultNodeIndex = extensions.findIndex(ext => ext.name === 'defaultNode');
+      if (defaultNodeIndex > 0) {
+        const defaultNode = extensions.splice(defaultNodeIndex, 1)[0];
+        extensions.unshift(defaultNode);
+      }
+
+      return extensions;
+    },
     showFloatingMenu({
       state, oldState
     }) {

package.json

@@ -55,7 +55,6 @@
     "@tiptap/extension-underline": "^2.0.3",
     "@tiptap/starter-kit": "^2.0.3",
     "@tiptap/vue-3": "^2.0.3",
-    "prosemirror-model": "1.25.0",
     "@vue/compiler-sfc": "^3.3.8",
     "autoprefixer": "^10.4.1",
     "bluebird": "^3.7.2",
@@ -143,4 +142,4 @@
   "browserslist": [
     "ie >= 10"
   ]
-}
+}