fix: fix crash when evaluating Symbol.prototype in getStringIfConstant (#182)

Co-authored-by: Yosuke Ota <[email protected]>

Author: fisker

src/get-string-if-constant.mjs

@@ -18,5 +18,15 @@ export function getStringIfConstant(node, initialScope = null) {
     }
 
     const evaluated = getStaticValue(node, initialScope)
-    return evaluated && String(evaluated.value)
+
+    if (evaluated) {
+        // `String(Symbol.prototype)` throws error
+        try {
+            return String(evaluated.value)
+        } catch {
+            // No op
+        }
+    }
+
+    return null
 }

test/get-string-if-constant.mjs

@@ -60,13 +60,17 @@ describe("The 'getStringIfConstant' function", () => {
             { code: "let id = 'abc'; id = 'foo'; id", expected: null },
             { code: "var id = 'abc'; id = 'foo'; id", expected: null },
             { code: "const id = otherId; id", expected: null },
+            { code: "Symbol.prototype", expected: null },
         ]) {
             it(`should return ${JSON.stringify(expected)} from ${code}`, () => {
                 const linter = newCompatLinter()
 
                 let actual = null
                 linter.verify(code, {
-                    languageOptions: { ecmaVersion: 2020 },
+                    languageOptions: {
+                        ecmaVersion: 2020,
+                        globals: { Symbol: "readonly" },
+                    },
                     rules: { "test/test": "error" },
                     plugins: {
                         test: {