Remove refmt.exe / Reason syntax support (#5683)

* Remove bsrefmt

* Remove refmt.exe

* Remove "refmt": 3

* Remove ast_reason_pp module and re/rei file extensions

* Remove leftover js_refmt_compiler

* Remove refmt_main3.ml

* Fix moveArtifacts script

* Clean up literals.ml

* Update CHANGELOG

Author: cknitt

.gitattributes

@@ -1,5 +1,4 @@
 jscomp/js_cmj_datasets.ml binary
 jscomp/bin/reactjs_ppx_v2.ml binary
-jscomp/bin/refmt_main3.ml binary
 docs/Manual.html binary
 docs/Manual.pdf binary

.gitignore

@@ -73,9 +73,6 @@ jscomp/bin/*.exe
 # so that `npm install && npm intall -g .`
 # the second install would use the cached ninja.exe (instead of being ignored)
 #
-!bin/bsrefmt
-
-
 #ninja
 .ninja_log
 .ninja_deps

.vscode/tasks.json

@@ -74,19 +74,6 @@
         "fileLocation": "autoDetect",
         "owner": "globalUnused"
       }
-    },
-    {
-      "label": "check unused refmt",
-      "command": "node",
-      "options": {
-        "cwd": "${workspaceRoot}/lib"
-      },
-      "args": ["../scripts/checkUnused.js", "refmt_main3"],
-      "problemMatcher": {
-        "base": "$ocamlc",
-        "fileLocation": "autoDetect",
-        "owner": "globalUnused"
-      }
     }
   ]
 }

CHANGELOG.md

@@ -10,7 +10,14 @@
 > - :house: [Internal]
 > - :nail_care: [Polish]
 
-# 10.1.0-rc.1
+# 11.0.0-alpha.1
+
+#### :boom: Breaking Change
+
+- Remove support for the legacy Reason syntax. Existing Reason code can be converted to ReScript syntax using ReScript 9 as follows:
+  - `npm i -g rescript@9`
+  - `rescript convert <reason files>`
+- Remove obsolete built-in project templates and the "rescript init" functionality. This will be replaced by the create-rescript-app project that is maintained separately.
 
 # 10.1.0-alpha.2
 

bsrefmt

@@ -354,19 +354,6 @@
       "additionalProperties": false,
       "required": ["version"]
     },
-    "refmt-specs": {
-      "oneOf": [
-        {
-          "type": "number",
-          "enum": [2, 3],
-          "description": "Reason syntax version to use. Prefer 3. Default: 2, for backward-compatiblity"
-        },
-        {
-          "type": "string",
-          "description": "(Usually not needed) Alternative path to `refmt`, the Reason syntax binary. Default: path to the `refmt.exe` ReScript provides out of the box"
-        }
-      ]
-    },
     "bsc-flags": {
       "oneOf": [
         {

darwin/refmt.exe

@@ -81,14 +81,10 @@ let add_basename ~(dir : string) (map : t) ?error_on_invalid_suffix basename : t
     (match () with
     | _ when file_suffix = Literals.suffix_ml -> ()
     | _ when file_suffix = Literals.suffix_res -> syntax_kind := Res
-    | _ when file_suffix = Literals.suffix_re -> syntax_kind := Reason
     | _ when file_suffix = Literals.suffix_mli -> info := Intf
     | _ when file_suffix = Literals.suffix_resi ->
         info := Intf;
         syntax_kind := Res
-    | _ when file_suffix = Literals.suffix_rei ->
-        info := Intf;
-        syntax_kind := Reason
     | _ -> invalid_suffix := true);
     let info = !info in
     let syntax_kind = !syntax_kind in

darwinarm64/refmt.exe

@@ -40,8 +40,6 @@ let handle_generators oc (group : Bsb_file_groups.file_group) custom_rules =
 
 type suffixes = { impl : string; intf : string }
 
-let re_suffixes = { impl = Literals.suffix_re; intf = Literals.suffix_rei }
-
 let ml_suffixes = { impl = Literals.suffix_ml; intf = Literals.suffix_mli }
 
 let res_suffixes = { impl = Literals.suffix_res; intf = Literals.suffix_resi }
@@ -52,7 +50,6 @@ let emit_module_build (rules : Bsb_ninja_rule.builtin)
   let has_intf_file = module_info.info = Impl_intf in
   let config, ast_rule =
     match module_info.syntax_kind with
-    | Reason -> (re_suffixes, rules.build_ast_from_re)
     | Ml -> (ml_suffixes, rules.build_ast)
     | Res -> (res_suffixes, rules.build_ast_from_re)
     (* FIXME: better names *)