fix(core): address review feedback on paste/parse fixes

- nestedLists: walk siblings via nextSibling so meaningful (non-whitespace)
  text between bare <li>s prevents them from being merged into one <ul>.
  Whitespace text nodes still bridge consecutive orphans.
- transformPasted: bail out of retypeLeadingParagraphForEmptyTarget during
  drop events (view.dragging is set), since the slice is inserted at the
  drop point rather than the current selection.

Author: nperez0111

packages/core/src/api/parsers/html/util/__snapshots__/nestedLists.test.ts.snap

@@ -1,5 +1,7 @@
 // Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
 
+exports[`Lift nested lists > Does not merge <li> nodes separated by non-whitespace text 1`] = `"<ul><li>a</li></ul>text<ul><li>b</li></ul>"`;
+
 exports[`Lift nested lists > Leaves <li>s already inside a <ul> alone 1`] = `"<ul><li>existing</li></ul><ul><li>orphan</li></ul>"`;
 
 exports[`Lift nested lists > Lifts multiple bullet lists 1`] = `

packages/core/src/api/parsers/html/util/nestedLists.test.ts

@@ -163,6 +163,11 @@ describe("Lift nested lists", () => {
     await testHTML(html);
   });
 
+  it("Does not merge <li> nodes separated by non-whitespace text", async () => {
+    const html = `<li>a</li>text<li>b</li>`;
+    await testHTML(html);
+  });
+
   it("Wraps nested orphan <li>s as inner lists", async () => {
     const html = `<li>outer<li>inner</li></li>`;
     await testHTML(html);

packages/core/src/api/parsers/html/util/nestedLists.ts

@@ -31,11 +31,26 @@ function wrapOrphanListItems(element: HTMLElement) {
     const group: Element[] = [orphan];
     handled.add(orphan);
 
-    let next = orphan.nextElementSibling;
-    while (next && next.tagName === "LI" && orphanSet.has(next)) {
-      group.push(next);
-      handled.add(next);
-      next = next.nextElementSibling;
+    // Walk siblings via nextSibling (not nextElementSibling) so we can stop
+    // at meaningful text between orphans — only whitespace text is allowed
+    // to bridge two orphan <li>s into the same <ul>.
+    let next: Node | null = orphan.nextSibling;
+    while (next) {
+      if (isWhitespaceNode(next)) {
+        next = next.nextSibling;
+        continue;
+      }
+      if (
+        next.nodeType === 1 &&
+        (next as Element).tagName === "LI" &&
+        orphanSet.has(next as Element)
+      ) {
+        group.push(next as Element);
+        handled.add(next as Element);
+        next = next.nextSibling;
+        continue;
+      }
+      break;
     }
 
     const ul = orphan.ownerDocument.createElement("ul");

packages/core/src/editor/transformPasted.ts

@@ -204,6 +204,14 @@ function retypeLeadingParagraphForEmptyTarget(
     return null;
   }
 
+  // `transformPasted` is also called for drop events, where the slice will be
+  // inserted at the drop position rather than the current selection. In that
+  // case the selection-derived target is wrong, so bail out and let the
+  // default behavior handle drops.
+  if (view.dragging) {
+    return null;
+  }
+
   const blockInfo = getBlockInfoFromSelection(view.state);
   const target = blockInfo.isBlockContainer
     ? blockInfo.blockContent.node