Fix: linkify included elements on mutations

Author: eight04

demo/demo-no-script.html

@@ -11,6 +11,15 @@
 			node.textContent = document.querySelector(".testcase").textContent;
 			document.querySelector(".main").appendChild(node);
 		}
+    function dupBlocked() {
+      const html = `
+<p class="brush:">
+  http://example.com
+  <span class="bbcode_code">http://example.com</span>
+</p>
+      `
+      document.querySelector(".main").insertAdjacentHTML("beforeend", html);
+    }
 		function interval() {
 			var p = document.createElement("p");
 			p.className = "leaker";
@@ -909,6 +918,7 @@ <h3>Bad TLDs</h3>
 		<p>
 			<button class="btn-default" onclick="dup()">Dynamic inserted content</button>
 			<button class="btn-default" onclick="interval()">Leak test</button>
+      <button class="btn-default" onclick="dupBlocked()">Insert blocked text</button>
 		</p>
 	</div>
 </body>

dist/linkify-plus-plus.user.js

@@ -3832,15 +3832,25 @@ async function enable(options) {
           if (processes >= MAX_PROCESSES) {
             throw new Error("Too many processes");
           }
-          if (!validRoot(node, options.validator)) {
-            continue;
+          const linkifyIncludedElements = () => {
+            if (options.includeElement) {
+              for (const el of node.querySelectorAll(options.includeElement)) {
+                processedNodes.add(el);
+                return linkify({...options, root: el, recursive: true});
+              }
+            }
+          };
+          if (validRoot(node, options.validator)) {
+            processedNodes.add(node);
+            processes++;
+            linkify({...options, root: node, recursive: true})
+              .then(linkifyIncludedElements)
+              .finally(() => {
+                processes--;
+              });
+          } else {
+            linkifyIncludedElements();
           }
-          processedNodes.add(node);
-          processes++;
-          linkify({...options, root: node, recursive: true})
-            .finally(() => {
-              processes--;
-            });
         }
       }
     }

rollup.config.mjs

@@ -46,7 +46,7 @@ export default async () => [
         {
           test: /(options|dialog)\.js$/,
           target: "dist-extension/$1.html",
-          handle: (content, {htmlScripts}) => content.replace("</body>", `${htmlScripts}</body>`)
+          handle: (content, {htmlScripts}) => content.replace(/.*<\/body>/, `${htmlScripts}</body>`)
         },
         {
           test: /background\.js$/,

src/lib/triggers/mutation.mjs

@@ -28,15 +28,25 @@ async function enable(options) {
           if (processes >= MAX_PROCESSES) {
             throw new Error("Too many processes");
           }
-          if (!validRoot(node, options.validator)) {
-            continue;
+          const linkifyIncludedElements = () => {
+            if (options.includeElement) {
+              for (const el of node.querySelectorAll(options.includeElement)) {
+                processedNodes.add(el);
+                return linkify({...options, root: el, recursive: true});
+              }
+            }
+          }
+          if (validRoot(node, options.validator)) {
+            processedNodes.add(node);
+            processes++;
+            linkify({...options, root: node, recursive: true})
+              .then(linkifyIncludedElements)
+              .finally(() => {
+                processes--;
+              });
+          } else {
+            linkifyIncludedElements();
           }
-          processedNodes.add(node);
-          processes++;
-          linkify({...options, root: node, recursive: true})
-            .finally(() => {
-              processes--;
-            });
         }
       }
     }