fix: avoid treating at sign within regex as current context; fixes #280

Author: brettz9

badges/tests-badge.svg

@@ -2382,8 +2382,10 @@ class JSONPathClass {
     if (!scriptCache.has(scriptCacheKey)) {
       let script = code.replaceAll('@parentProperty', '_$_parentProperty').replaceAll('@parent', '_$_parent').replaceAll('@property', '_$_property').replaceAll('@root', '_$_root')
       // Replace a bare `@` (not followed by an identifier
-      //   character) while leaving quoted string literals intact
-      .replaceAll(/('(?:\\.|[^'\\])*'|"(?:\\.|[^"\\])*")|@(?![\w$])/gv, (_, str) => str ?? '_$_v');
+      //   character) while leaving quoted string literals and
+      //   regex literals (a `/` where an operand is expected,
+      //   as opposed to division) intact
+      .replaceAll(/('(?:\\.|[^'\\])*'|"(?:\\.|[^"\\])*"|(?:^|[\(,=:\[!\|?\{\};+\-*%<>~^]|&)\s*\/(?:\\.|\[(?:\\.|[^\]\\])*\]|[^\/\\\[])+\/[dgimsuvy]*)|@(?![\w$])/gv, (_, literal) => literal ?? '_$_v');
       if (containsPath) {
         script = script.replaceAll('@path', '_$_path');
       }

dist/index-browser-esm.js

@@ -2388,8 +2388,10 @@
 	    if (!scriptCache.has(scriptCacheKey)) {
 	      let script = code.replaceAll('@parentProperty', '_$_parentProperty').replaceAll('@parent', '_$_parent').replaceAll('@property', '_$_property').replaceAll('@root', '_$_root')
 	      // Replace a bare `@` (not followed by an identifier
-	      //   character) while leaving quoted string literals intact
-	      .replaceAll(/('(?:\\.|[^'\\])*'|"(?:\\.|[^"\\])*")|@(?![\w$])/gv, (_, str) => str ?? '_$_v');
+	      //   character) while leaving quoted string literals and
+	      //   regex literals (a `/` where an operand is expected,
+	      //   as opposed to division) intact
+	      .replaceAll(/('(?:\\.|[^'\\])*'|"(?:\\.|[^"\\])*"|(?:^|[\(,=:\[!\|?\{\};+\-*%<>~^]|&)\s*\/(?:\\.|\[(?:\\.|[^\]\\])*\]|[^\/\\\[])+\/[dgimsuvy]*)|@(?![\w$])/gv, (_, literal) => literal ?? '_$_v');
 	      if (containsPath) {
 	        script = script.replaceAll('@path', '_$_path');
 	      }

dist/index-browser-esm.min.js

@@ -2386,8 +2386,10 @@ class JSONPathClass {
     if (!scriptCache.has(scriptCacheKey)) {
       let script = code.replaceAll('@parentProperty', '_$_parentProperty').replaceAll('@parent', '_$_parent').replaceAll('@property', '_$_property').replaceAll('@root', '_$_root')
       // Replace a bare `@` (not followed by an identifier
-      //   character) while leaving quoted string literals intact
-      .replaceAll(/('(?:\\.|[^'\\])*'|"(?:\\.|[^"\\])*")|@(?![\w$])/gv, (_, str) => str ?? '_$_v');
+      //   character) while leaving quoted string literals and
+      //   regex literals (a `/` where an operand is expected,
+      //   as opposed to division) intact
+      .replaceAll(/('(?:\\.|[^'\\])*'|"(?:\\.|[^"\\])*"|(?:^|[\(,=:\[!\|?\{\};+\-*%<>~^]|&)\s*\/(?:\\.|\[(?:\\.|[^\]\\])*\]|[^\/\\\[])+\/[dgimsuvy]*)|@(?![\w$])/gv, (_, literal) => literal ?? '_$_v');
       if (containsPath) {
         script = script.replaceAll('@path', '_$_path');
       }

dist/index-browser-esm.min.js.map

@@ -2384,8 +2384,10 @@ class JSONPathClass {
     if (!scriptCache.has(scriptCacheKey)) {
       let script = code.replaceAll('@parentProperty', '_$_parentProperty').replaceAll('@parent', '_$_parent').replaceAll('@property', '_$_property').replaceAll('@root', '_$_root')
       // Replace a bare `@` (not followed by an identifier
-      //   character) while leaving quoted string literals intact
-      .replaceAll(/('(?:\\.|[^'\\])*'|"(?:\\.|[^"\\])*")|@(?![\w$])/gv, (_, str) => str ?? '_$_v');
+      //   character) while leaving quoted string literals and
+      //   regex literals (a `/` where an operand is expected,
+      //   as opposed to division) intact
+      .replaceAll(/('(?:\\.|[^'\\])*'|"(?:\\.|[^"\\])*"|(?:^|[\(,=:\[!\|?\{\};+\-*%<>~^]|&)\s*\/(?:\\.|\[(?:\\.|[^\]\\])*\]|[^\/\\\[])+\/[dgimsuvy]*)|@(?![\w$])/gv, (_, literal) => literal ?? '_$_v');
       if (containsPath) {
         script = script.replaceAll('@path', '_$_path');
       }

dist/index-browser-umd.cjs

@@ -1048,10 +1048,12 @@ class JSONPathClass {
                 .replaceAll('@property', '_$_property')
                 .replaceAll('@root', '_$_root')
                 // Replace a bare `@` (not followed by an identifier
-                //   character) while leaving quoted string literals intact
+                //   character) while leaving quoted string literals and
+                //   regex literals (a `/` where an operand is expected,
+                //   as opposed to division) intact
                 .replaceAll(
-                    /('(?:\\.|[^'\\])*'|"(?:\\.|[^"\\])*")|@(?![\w$])/gv,
-                    (_, str) => str ?? '_$_v'
+                    /('(?:\\.|[^'\\])*'|"(?:\\.|[^"\\])*"|(?:^|[\(,=:\[!\|?\{\};+\-*%<>~^]|&)\s*\/(?:\\.|\[(?:\\.|[^\]\\])*\]|[^\/\\\[])+\/[dgimsuvy]*)|@(?![\w$])/gv,
+                    (_, literal) => literal ?? '_$_v'
                 );
             if (containsPath) {
                 script = script.replaceAll('@path', '_$_path');