Fix to reset lastIndex at false result

Author: wooorm

lib/index.js

@@ -171,7 +171,12 @@ export function findAndReplace(tree, list, options) {
       }
 
       // It wasn’t a match after all.
-      if (value !== false) {
+      if (value === false) {
+        // False acts as if there was no match.
+        // So we need to reset `lastIndex`, which currently being at the end of
+        // the current match, to the beginning.
+        find.lastIndex = position + 1
+      } else {
         if (start !== position) {
           nodes.push({type: 'text', value: node.value.slice(start, position)})
         }

test.js

@@ -1,3 +1,7 @@
+/**
+ * @typedef {import('hast').Root} Root
+ */
+
 import assert from 'node:assert/strict'
 import test from 'node:test'
 import {h} from 'hastscript'
@@ -290,6 +294,35 @@ test('findAndReplace', async function (t) {
     assert.deepEqual(tree, create())
   })
 
+  await t.test('should not treat `false` as a match', async function () {
+    /** @type {Root} */
+    const tree = {type: 'root', children: [{type: 'text', value: ':1:2:'}]}
+
+    findAndReplace(tree, [
+      /:(\d+):/g,
+      /**
+       * @param {string} _
+       * @param {string} $1
+       */
+      function (_, $1) {
+        return $1 === '2' ? h('strong', $1) : false
+      }
+    ])
+
+    assert.deepEqual(tree, {
+      type: 'root',
+      children: [
+        {type: 'text', value: ':1'},
+        {
+          type: 'element',
+          tagName: 'strong',
+          properties: {},
+          children: [{type: 'text', value: '2'}]
+        }
+      ]
+    })
+  })
+
   await t.test('should not be order-sensitive with strings', async function () {
     const tree = h('p', 'Some emphasis, importance, and code.')