revert changes that unrelated to the main issue

Author: cometkim

.github/workflows/ci.yml

@@ -83,7 +83,6 @@ jobs:
         uses: actions/setup-node@v4
         with:
           node-version: 18
-          cache: npm
 
       - name: Copy exes to platform bin dirs
         run: node ./scripts/copyExes.js
@@ -171,7 +170,6 @@ jobs:
         uses: actions/setup-node@v4
         with:
           node-version: 18
-          cache: npm
 
       - name: Install npm packages
         run: npm ci --ignore-scripts
@@ -282,7 +280,6 @@ jobs:
         uses: actions/setup-node@v4
         with:
           node-version: 18
-          cache: npm
 
       - name: NPM install
         run: npm ci --ignore-scripts
@@ -346,7 +343,6 @@ jobs:
         uses: actions/setup-node@v4
         with:
           node-version: 18
-          cache: npm
 
       - name: Download artifacts
         uses: actions/download-artifact@v4
@@ -380,7 +376,6 @@ jobs:
         with:
           node-version: 18
           registry-url: https://registry.npmjs.org # Needed to make auth work for publishing
-          cache: npm
 
       - name: Download artifacts
         uses: actions/download-artifact@v4

CHANGELOG.md

@@ -44,7 +44,6 @@
 - Turn off transformation for closures inside loops when capturing loop variables, now that `let` is emitted instead of `var`. https://github.com/rescript-lang/rescript-compiler/pull/6480
 - Fix formatter eats comments on the first argument of a uncurried function. https://github.com/rescript-lang/rescript-compiler/pull/6763 
 - Fix formatter removes parens in pipe operator with anonymous uncurried function. https://github.com/rescript-lang/rescript-compiler/pull/6766
-- Allow [future reserved words in older standards](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Lexical_grammar#future_reserved_words_in_older_standards) as identifiers. https://github.com/rescript-lang/rescript-compiler/pull/6658
 
 # 11.1.0
 

jscomp/ext/ext_ident.ml

@@ -174,19 +174,15 @@ let name_mangle name =
      Ext_ident.convert "^";;
      - : string = "$caret"
    ]}
-   [convert name] if [name] is a js keyword, and also is not explicitly used as exotic identifier, add "$$"
+   [convert name] if [name] is a js keyword,add "$$"
    otherwise do the name mangling to make sure ocaml identifier it is
    a valid js identifier
 *)
 let convert (name : string) =
-  if is_exotic name then
-     let name = unwrap_exotic name in
-     if Js_reserved_map.is_js_keyword name then "$$" ^ name
-     else name_mangle name
-  else
-   if Js_reserved_map.is_js_keyword name || Js_reserved_map.is_js_special_word name || Js_reserved_map.is_reserved name then
-      "$$" ^ name
-   else name_mangle name
+  let name = unwrap_exotic name in
+  if Js_reserved_map.is_reserved name then
+    "$$" ^ name
+  else name_mangle name
 
 (** keyword could be used in property *)