fix: skip ascii symbol in process (#31)

* fix: skip ascii symbol in process * fix: skip printable ASCII symbol * move printable ASCII symbol check after case fold
mathiasbynens · Sep 29, 2023 · 55f3e5e · 55f3e5e
1 parent b278ed4
commit 55f3e5e
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/rewrite-pattern.js b/rewrite-pattern.js
@@ -685,6 +685,10 @@ const processTerm = (item, regenerateOptions, groups) => {
 			const codePoint = item.codePoint;
 			const set = regenerate(codePoint);
 			const folded = maybeFold(codePoint);
+			if (folded.length === 1 && item.kind === "symbol" && folded[0] >= 0x20 && folded[0] <= 0x7E) {
+				// skip regenerate when it is a printable ASCII symbol
+				break;
+			}
 			set.add(folded);
 			update(item, set.toString(regenerateOptions));
 			break;

diff --git a/tests/tests.js b/tests/tests.js
@@ -267,6 +267,12 @@ describe('unicodePropertyEscapes', () => {
 			'\\u03B8'
 		);
 	});
+	it('should not replace `-` symbol when not in character class range', () => {
+		assert.equal(
+			rewritePattern('-'),
+			'-'
+		)
+	})
 });
 
 describe('dotAllFlag', () => {