Fix/472 (#518)

* remove unnecessary escapes in ranges

* fix lintstagedrc

* size-linit upd

Author: dangreen

.lintstagedrc

@@ -1,6 +1,3 @@
 {
-	"src/**/*.{ts,tsx}": [
-		"trigen-scripts lint:ts",
-		"git add"
-	]
+	"src/**/*.{ts,tsx}": "trigen-scripts lint:ts"
 }

.size-limit

@@ -6,7 +6,7 @@
 	},
 	{
 		"path": "lib/cli.js",
-		"limit": "1.6 KB",
+		"limit": "1.7 KB",
 		"webpack": false
 	}
 ]

package.json

@@ -45,7 +45,7 @@
     "useragent": "^2.3.0"
   },
   "devDependencies": {
-    "@size-limit/preset-small-lib": "^3.0.0",
+    "@size-limit/preset-small-lib": "^4.6.0",
     "@trigen/scripts": "^6.0.0",
     "@trigen/scripts-plugin-babel": "^6.1.1",
     "@trigen/scripts-plugin-eslint": "^6.0.0",

src/regexp/optimize.spec.ts

@@ -26,5 +26,14 @@ describe('UserAgentRegExp', () => {
 				'Family(foo)?(?=NotFamily)'
 			);
 		});
+
+		it('should remove unnecessary escapes in ranges', () => {
+
+			expect(
+				optimize('[\\.\\[]')
+			).toBe(
+				'[.[]'
+			);
+		});
 	});
 });

src/regexp/optimize.ts

@@ -6,9 +6,12 @@ import {
 } from './util';
 
 export const OPTIMIZABLE_GROUP = /^\([\s\w\d_\-/!]+\)$/;
+export const CHARCLASS_UNESCAPES = /[/.$*+?[{}|()]/;
 
 /**
- * Optimize RegExp string: remove useless braces.
+ * Optimize RegExp string:
+ * - remove unnecessary braces;
+ * - remove unnecessary escapes in ranges.
  * @param  regExpStr - RegExp string to optimize.
  * @return Optimized RegExp string.
  */
@@ -19,6 +22,7 @@ export function optimize(regExpStr: string) {
 	let skip = false;
 	let char = '';
 	let prevChar = '';
+	let nextChar = '';
 	let postfix = '';
 	let groupAccum = '';
 	let optimizedRegExpStr = '';
@@ -27,6 +31,7 @@ export function optimize(regExpStr: string) {
 
 		char = regExpStr[i];
 		prevChar = regExpStr[i - 1];
+		nextChar = regExpStr[i + 1];
 		skip = skipSquareBraces(skip, prevChar, char);
 
 		if (!skip
@@ -42,6 +47,14 @@ export function optimize(regExpStr: string) {
 			groupAccum = '';
 		}
 
+		if (skip
+			&& char === ESCAPE_SYMBOL
+			&& CHARCLASS_UNESCAPES.test(nextChar)
+		) {
+			i++;
+			char = nextChar;
+		}
+
 		if (inGroup) {
 			groupAccum += char;
 		} else {