fix(linter): global variable check should always check builtin variables (#3973)

closes #3374

crates/oxc_linter/src/context.rs

@@ -137,13 +137,16 @@ impl<'a> LintContext<'a> {
     }
 
     pub fn env_contains_var(&self, var: &str) -> bool {
+        if GLOBALS["builtin"].contains_key("var") {
+            return true;
+        }
         for env in self.env().iter() {
-            let env = GLOBALS.get(env).unwrap_or(&GLOBALS["builtin"]);
+            if let Some(env) = GLOBALS.get(env) {
-            if env.get(var).is_some() {
+                if env.contains_key(var) {
-                return true;
+                    return true;
+                }
             }
         }
-
         false
     }
 

crates/oxc_linter/src/rules/eslint/no_undef.rs

@@ -156,6 +156,7 @@ fn test() {
         "class C { static { a; let a; } }",
         "class C { static { function a() {} a; } }",
         "class C { static { a; function a() {} } }",
+        "String;Array;Boolean;"
     ];
 
     let fail = vec![