fix(replace)!: issues with nested objects replacements (#903)

* fix(replace): issues with nested objects replacements

* fix(replace): update tests & snapshots

.github/workflows/release.yml

@@ -8,7 +8,8 @@ on:
 jobs:
   publish:
     # let's ignore release commits, otherwise it'll try to run twice
-    if: !startsWith(github.event.head_commit.message , 'chore(release):')
+    if: |
+      !startsWith(github.event.head_commit.message , 'chore(release):')
 
     runs-on: ubuntu-latest
 

packages/replace/README.md

@@ -61,9 +61,14 @@ In addition to the properties and values specified for replacement, users may al
 ### `delimiters`
 
 Type: `Array[...String, String]`<br>
-Default: `['\b', '\b']`
+Default: `['\b', '\b(?!\.)']`
 
-Specifies the boundaries around which strings will be replaced. By default, delimiters are [word boundaries](https://www.regular-expressions.info/wordboundaries.html). See [Word Boundaries](#word-boundaries) below for more information.
+Specifies the boundaries around which strings will be replaced. By default, delimiters are [word boundaries](https://www.regular-expressions.info/wordboundaries.html) and also prevent replacements of instances with nested access. See [Word Boundaries](#word-boundaries) below for more information.
+For example, if you pass `typeof window` in `values` to-be-replaced, then you could expect the following scenarios:
+
+- `typeof window` **will** be replaced
+- `typeof window.document` **will not** be replaced due to `(?!\.)` boundary
+- `typeof windowSmth` **will not** be replaced due to a `\b` boundary
 
 ### `preventAssignment`
 

packages/replace/src/index.js

@@ -46,7 +46,7 @@ export default function replace(options = {}) {
         `${escape(delimiters[0])}(${keys.join('|')})${escape(delimiters[1])}${lookahead}`,
         'g'
       )
-    : new RegExp(`\\b(${keys.join('|')})\\b${lookahead}`, 'g');
+    : new RegExp(`\\b(${keys.join('|')})\\b(?!\\.)${lookahead}`, 'g');
 
   return {
     name: 'replace',

packages/replace/test/fixtures/form/replace-nothing/_config.js

@@ -1,4 +1,6 @@
 module.exports = {
   description: 'replaces nothing',
-  options: {}
+  options: {
+    'typeof window': `"object"`
+  }
 };

packages/replace/test/fixtures/form/replace-nothing/input.js

@@ -1 +1,2 @@
 console.log('as-it'); // eslint-disable-line
+console.log(typeof window.document); // eslint-disable-line

packages/replace/test/fixtures/form/replace-nothing/output.js

@@ -1 +1,2 @@
 console.log('as-it');
+console.log(typeof window.document); // eslint-disable-line

packages/replace/test/fixtures/form/replace-strings/_config.js

@@ -1,6 +1,7 @@
 module.exports = {
   description: 'replaces strings',
   options: {
-    ANSWER: '42'
+    ANSWER: '42',
+    'typeof window': `"object"`
   }
 };

packages/replace/test/fixtures/form/replace-strings/input.js

@@ -1 +1,2 @@
 console.log(ANSWER); // eslint-disable-line
+console.log(typeof window);

packages/replace/test/fixtures/form/replace-strings/output.js

@@ -1 +1,2 @@
 console.log(42);
+console.log('object');

packages/replace/test/snapshots/form.js.md

@@ -2,7 +2,13 @@
 
 The actual snapshot is saved in `form.js.snap`.
 
-Generated by [AVA](https://ava.li).
+Generated by [AVA](https://avajs.dev).
+
+## assignment: doesn't replace lvalue in assignment
+
+> Snapshot 1
+
+    'process.env.DEBUG = \'test\';'
 
 ## delimiters: observes delimiters
 
@@ -37,13 +43,15 @@ Generated by [AVA](https://ava.li).
 
 > Snapshot 1
 
-    'console.log(\'as-it\'); // eslint-disable-line'
+    `console.log('as-it'); // eslint-disable-line␊
+    console.log(typeof window.document); // eslint-disable-line`
 
 ## replace-strings: replaces strings
 
 > Snapshot 1
 
-    'console.log(42); // eslint-disable-line'
+    `console.log(42); // eslint-disable-line␊
+    console.log("object");`
 
 ## replacement-function: allows replacement to be a function
 
@@ -58,9 +66,3 @@ Generated by [AVA](https://ava.li).
     `const one = 1; // eslint-disable-line␊
     ␊
     console.log(one);`
-
-## assignment: doesn't replace lvalue in assignment
-
-> Snapshot 1
-
-    'process.env.DEBUG = \'test\';'

packages/replace/test/snapshots/misc.js.md

@@ -2,49 +2,7 @@
 
 The actual snapshot is saved in `misc.js.snap`.
 
-Generated by [AVA](https://ava.li).
-
-## does not generate sourcemaps if disabled
-
-> Snapshot 1
-
-    'Sourcemap is likely to be incorrect: a plugin (replace) was used to transform files, but didn\'t generate a sourcemap for the transformation. Consult the plugin documentation for help'
-
-## generates sourcemaps
-
-> Snapshot 1
-
-    {
-      character: 12,
-      column: 12,
-      line: 1,
-    }
-
-> Snapshot 2
-
-    {
-      column: 15,
-      line: 1,
-      name: null,
-      source: 'other.js',
-    }
-
-> Snapshot 3
-
-    {
-      character: 17,
-      column: 0,
-      line: 3,
-    }
-
-> Snapshot 4
-
-    {
-      column: 0,
-      line: 2,
-      name: null,
-      source: 'main.js',
-    }
+Generated by [AVA](https://avajs.dev).
 
 ## no explicit setting of preventAssignment
 

packages/replace/test/snapshots/sourcemaps.js.md

@@ -2,9 +2,9 @@
 
 The actual snapshot is saved in `sourcemaps.js.snap`.
 
-Generated by [AVA](https://ava.li).
+Generated by [AVA](https://avajs.dev).
 
-## generates sourcemaps when sourcemap is used as an output argument
+## generates sourcemaps by default
 
 > Snapshot 1
 
@@ -111,75 +111,3 @@ Generated by [AVA](https://ava.li).
       name: null,
       source: 'main.js',
     }
-
-## generates sourcemaps by dfault
-
-> Snapshot 1
-
-    {
-      character: 12,
-      column: 12,
-      line: 1,
-    }
-
-> Snapshot 2
-
-    {
-      column: 15,
-      line: 1,
-      name: null,
-      source: 'other.js',
-    }
-
-> Snapshot 3
-
-    {
-      character: 17,
-      column: 0,
-      line: 3,
-    }
-
-> Snapshot 4
-
-    {
-      column: 0,
-      line: 2,
-      name: null,
-      source: 'main.js',
-    }
-
-## generates sourcemaps by default
-
-> Snapshot 1
-
-    {
-      character: 12,
-      column: 12,
-      line: 1,
-    }
-
-> Snapshot 2
-
-    {
-      column: 15,
-      line: 1,
-      name: null,
-      source: 'other.js',
-    }
-
-> Snapshot 3
-
-    {
-      character: 17,
-      column: 0,
-      line: 3,
-    }
-
-> Snapshot 4
-
-    {
-      column: 0,
-      line: 2,
-      name: null,
-      source: 'main.js',
-    }