fix(surround): fix both nested code (#44) and multiple code spans (#46)

The previous fix for #44 used parentElement.querySelector() to detect
existing <code> elements in the block, but this was too broad: it
blocked wrapping any new selection as long as any <code> existed
anywhere in the block, breaking the ability to apply inline code to
multiple non-contiguous selections (#46).

The correct scope for the check is the selection itself, not the
whole block. Using range.cloneContents().querySelector() only inspects
what the user has actually selected:

- If the selection contains an existing <code> → skip wrap (prevents #44)
- If the selection is clean, even if other <code> exist elsewhere
  in the block → wrap as normal (fixes #46)

Closes #46

Author: AnthoHansen

src/index.ts

@@ -88,15 +88,20 @@ export default class InlineCode implements InlineTool {
 
     let termWrapper = this.api.selection.findParentTag(this.tag, InlineCode.CSS) as HTMLElement;
 
-    /**
-     * If the current selection is already wrapped in a code tag, unwrap it.
-     * Otherwise, wrap the selection — regardless of whether other <code> elements
-     * exist elsewhere in the same block (fixes #46).
-     */
     if (termWrapper) {
       this.unwrap(termWrapper);
     } else {
-      this.wrap(range);
+      /**
+       * Only wrap if the selection itself does not already contain a <code> element.
+       * Using cloneContents() scopes the check to the actual selection, not the
+       * whole block — this prevents nested <code> tags (#44) while still allowing
+       * multiple non-contiguous code spans in the same block (#46).
+       */
+      const selectionContainsCode = range.cloneContents().querySelector(this.tag) !== null;
+
+      if (!selectionContainsCode) {
+        this.wrap(range);
+      }
     }
   }