fix(typescript-estree): error throws on BigInt with numberic separator

Author: fisker

Diff for: packages/parser/tests/lib/__snapshots__/javascript.ts.snap

@@ -6815,6 +6815,56 @@ Object {
 }
 `;
 
+exports[`javascript fixtures/bigIntLiterals/numeric-separator.src 1`] = `
+Object {
+  "$id": 1,
+  "block": Object {
+    "range": Array [
+      0,
+      8,
+    ],
+    "type": "Program",
+  },
+  "childScopes": Array [
+    Object {
+      "$id": 0,
+      "block": Object {
+        "range": Array [
+          0,
+          8,
+        ],
+        "type": "Program",
+      },
+      "childScopes": Array [],
+      "functionExpressionScope": false,
+      "isStrict": true,
+      "references": Array [],
+      "throughReferences": Array [],
+      "type": "module",
+      "upperScope": Object {
+        "$ref": 1,
+      },
+      "variableMap": Object {},
+      "variableScope": Object {
+        "$ref": 0,
+      },
+      "variables": Array [],
+    },
+  ],
+  "functionExpressionScope": false,
+  "isStrict": false,
+  "references": Array [],
+  "throughReferences": Array [],
+  "type": "global",
+  "upperScope": null,
+  "variableMap": Object {},
+  "variableScope": Object {
+    "$ref": 1,
+  },
+  "variables": Array [],
+}
+`;
+
 exports[`javascript fixtures/bigIntLiterals/octal.src 1`] = `
 Object {
   "$id": 1,

Diff for: packages/shared-fixtures/fixtures/javascript/bigIntLiterals/numeric-separator.src.js

@@ -0,0 +1 @@
+1_2_3n;

Diff for: packages/typescript-estree/src/convert.ts

@@ -1931,8 +1931,14 @@ export class Converter {
       case SyntaxKind.BigIntLiteral: {
         const range = getRange(node, this.ast);
         const rawValue = this.ast.text.slice(range[0], range[1]);
-        const bigint = rawValue.slice(0, -1); // remove suffix `n`
-        const value = typeof BigInt !== 'undefined' ? BigInt(bigint) : null;
+        const bigint = rawValue.slice(0, -1); // remove suffix `n`;
+        const value =
+          typeof BigInt !== 'undefined'
+            ? BigInt(
+                // `BigInt` doesn't accept numeric separator
+                bigint.replace(/_/g, ''),
+              )
+            : null;
         return this.createNode<TSESTree.BigIntLiteral>(node, {
           type: AST_NODE_TYPES.Literal,
           raw: rawValue,

Diff for: packages/typescript-estree/tests/ast-alignment/parse.ts

@@ -29,6 +29,7 @@ function parseWithBabelParser(text: string, jsx = true): any {
     'dynamicImport',
     'estree',
     'bigInt',
+    'numericSeparator',
     'importMeta',
     'optionalChaining',
     'nullishCoalescingOperator',

Diff for: packages/typescript-estree/tests/lib/__snapshots__/javascript.ts.snap

@@ -16780,6 +16780,104 @@ Object {
 }
 `;
 
+exports[`javascript fixtures/bigIntLiterals/numeric-separator.src 1`] = `
+Object {
+  "body": Array [
+    Object {
+      "expression": Object {
+        "bigint": "123",
+        "loc": Object {
+          "end": Object {
+            "column": 6,
+            "line": 1,
+          },
+          "start": Object {
+            "column": 0,
+            "line": 1,
+          },
+        },
+        "range": Array [
+          0,
+          6,
+        ],
+        "raw": "1_2_3n",
+        "type": "Literal",
+        "value": 123n,
+      },
+      "loc": Object {
+        "end": Object {
+          "column": 7,
+          "line": 1,
+        },
+        "start": Object {
+          "column": 0,
+          "line": 1,
+        },
+      },
+      "range": Array [
+        0,
+        7,
+      ],
+      "type": "ExpressionStatement",
+    },
+  ],
+  "loc": Object {
+    "end": Object {
+      "column": 0,
+      "line": 2,
+    },
+    "start": Object {
+      "column": 0,
+      "line": 1,
+    },
+  },
+  "range": Array [
+    0,
+    8,
+  ],
+  "sourceType": "script",
+  "tokens": Array [
+    Object {
+      "loc": Object {
+        "end": Object {
+          "column": 6,
+          "line": 1,
+        },
+        "start": Object {
+          "column": 0,
+          "line": 1,
+        },
+      },
+      "range": Array [
+        0,
+        6,
+      ],
+      "type": "Identifier",
+      "value": "1_2_3n",
+    },
+    Object {
+      "loc": Object {
+        "end": Object {
+          "column": 7,
+          "line": 1,
+        },
+        "start": Object {
+          "column": 6,
+          "line": 1,
+        },
+      },
+      "range": Array [
+        6,
+        7,
+      ],
+      "type": "Punctuator",
+      "value": ";",
+    },
+  ],
+  "type": "Program",
+}
+`;
+
 exports[`javascript fixtures/bigIntLiterals/octal.src 1`] = `
 Object {
   "body": Array [

Diff for: packages/typescript-estree/tests/lib/__snapshots__/semantic-diagnostics-enabled.ts.snap

@@ -224,6 +224,8 @@ exports[`Parse all fixtures with "errorOnTypeScriptSyntacticAndSemanticIssues" e
 
 exports[`Parse all fixtures with "errorOnTypeScriptSyntacticAndSemanticIssues" enabled fixtures/javascript/bigIntLiterals/hex.src 1`] = `"TEST OUTPUT: No semantic or syntactic issues found"`;
 
+exports[`Parse all fixtures with "errorOnTypeScriptSyntacticAndSemanticIssues" enabled fixtures/javascript/bigIntLiterals/numeric-separator.src 1`] = `"TEST OUTPUT: No semantic or syntactic issues found"`;
+
 exports[`Parse all fixtures with "errorOnTypeScriptSyntacticAndSemanticIssues" enabled fixtures/javascript/bigIntLiterals/octal.src 1`] = `"TEST OUTPUT: No semantic or syntactic issues found"`;
 
 exports[`Parse all fixtures with "errorOnTypeScriptSyntacticAndSemanticIssues" enabled fixtures/javascript/binaryLiterals/invalid.src 1`] = `