Fix bug when replacing and then not

Backports: syntax-tree/mdast-util-find-and-replace@b40a41f.

Author: wooorm

lib/index.js

@@ -117,6 +117,7 @@ export function findAndReplace(tree, find, replace, options) {
     const replace = pairs[pairIndex][1]
     let start = 0
     const index = parent.children.indexOf(node)
+    let change = false
     /** @type {Array<Content>} */
     let nodes = []
     /** @type {number|undefined} */
@@ -142,9 +143,7 @@ export function findAndReplace(tree, find, replace, options) {
       }
 
       // It wasn’t a match after all.
-      if (value === false) {
-        position = undefined
-      } else {
+      if (value !== false) {
         if (start !== position) {
           nodes.push({type: 'text', value: node.value.slice(start, position)})
         }
@@ -156,6 +155,7 @@ export function findAndReplace(tree, find, replace, options) {
         }
 
         start = position + match[0].length
+        change = true
       }
 
       if (!find.global) {
@@ -165,14 +165,14 @@ export function findAndReplace(tree, find, replace, options) {
       match = find.exec(node.value)
     }
 
-    if (position === undefined) {
-      nodes = [node]
-    } else {
+    if (change) {
       if (start < node.value.length) {
         nodes.push({type: 'text', value: node.value.slice(start)})
       }
 
       parent.children.splice(index, 1, ...nodes)
+    } else {
+      nodes = [node]
     }
 
     return index + nodes.length

test.js

@@ -215,6 +215,18 @@ test('findAndReplace', (t) => {
     'should not be order-sensitive with strings'
   )
 
+  t.deepEqual(
+    findAndReplace(h('p', 'aaa bbb'), [
+      [
+        /\b\w+\b/g,
+        function (/** @type {string} */ value) {
+          return value === 'aaa' ? h('strong', value) : false
+        }
+      ]
+    ]),
+    h('p', [h('strong', 'aaa'), ' bbb']),
+    'should support a match, and then a `false`'
+  )
   t.deepEqual(
     findAndReplace(h('p', 'Some emphasis, importance, and code.'), [
       [