Rewatch: warn about deprecated package specs es6/es6-global (#8146)

* Rewatch: warn about deprecated package specs es6/es6-global

* Fix tests

* Use "esmodule" in (most) rewatch tests

* fix test

* CHANGELOG

# Conflicts:
#	CHANGELOG.md

Author: cknitt

CHANGELOG.md

@@ -25,6 +25,7 @@
 
 - Fix rewatch swallowing parse warnings (%todo). https://github.com/rescript-lang/rescript/pull/8135
 - Rewatch: log errors and warnings to `stderr`. https://github.com/rescript-lang/rescript/pull/8147 https://github.com/rescript-lang/rescript/pull/8148
+- Rewatch: warn about deprecated package specs `es6`/`es6-global`. https://github.com/rescript-lang/rescript/pull/8146
 
 #### :memo: Documentation
 

rewatch/src/build.rs

@@ -428,6 +428,12 @@ fn log_config_warnings(build_state: &BuildCommandState) {
                     config::DeprecationWarning::BscFlags => {
                         log_deprecated_config_field(&package.name, "bsc-flags", "compiler-flags");
                     }
+                    config::DeprecationWarning::PackageSpecsEs6 => {
+                        log_deprecated_package_specs_module("es6");
+                    }
+                    config::DeprecationWarning::PackageSpecsEs6Global => {
+                        log_deprecated_package_specs_module("es6-global");
+                    }
                 },
             );
 
@@ -454,6 +460,11 @@ fn log_deprecated_config_field(package_name: &str, field_name: &str, new_field_n
     eprintln!("\n{}", style(warning).yellow());
 }
 
+fn log_deprecated_package_specs_module(module_name: &str) {
+    let warning = format!("deprecated: Option \"{module_name}\" is deprecated. Use \"esmodule\" instead.");
+    eprintln!("\n{}", style(warning).yellow());
+}
+
 fn log_unsupported_config_field(package_name: &str, field_name: &str) {
     let warning = format!(
         "The field '{field_name}' found in the package config of '{package_name}' is not supported by ReScript 12's new build system."

rewatch/src/config.rs

@@ -224,6 +224,8 @@ pub enum DeprecationWarning {
     BsDependencies,
     BsDevDependencies,
     BscFlags,
+    PackageSpecsEs6,
+    PackageSpecsEs6Global,
 }
 
 #[derive(Debug, Clone, Eq, PartialEq, Hash)]
@@ -746,6 +748,23 @@ impl Config {
             self.deprecation_warnings.push(DeprecationWarning::BscFlags);
         }
 
+        let (has_es6, has_es6_global) = match &self.package_specs {
+            None => (false, false),
+            Some(OneOrMore::Single(spec)) => (spec.module == "es6", spec.module == "es6-global"),
+            Some(OneOrMore::Multiple(specs)) => (
+                specs.iter().any(|spec| spec.module == "es6"),
+                specs.iter().any(|spec| spec.module == "es6-global"),
+            ),
+        };
+        if has_es6 {
+            self.deprecation_warnings
+                .push(DeprecationWarning::PackageSpecsEs6);
+        }
+        if has_es6_global {
+            self.deprecation_warnings
+                .push(DeprecationWarning::PackageSpecsEs6Global);
+        }
+
         Ok(())
     }
 }
@@ -992,7 +1011,7 @@ pub mod tests {
             },
             "package-specs": [
                 {
-                "module": "es6",
+                "module": "esmodule",
                 "in-source": true
                 }
             ],
@@ -1017,7 +1036,7 @@ pub mod tests {
             },
             "package-specs": [
                 {
-                "module": "es6",
+                "module": "esmodule",
                 "in-source": true
                 }
             ],
@@ -1042,7 +1061,7 @@ pub mod tests {
             },
             "package-specs": [
                 {
-                "module": "es6",
+                "module": "esmodule",
                 "in-source": true
                 }
             ],
@@ -1067,7 +1086,7 @@ pub mod tests {
             },
             "package-specs": [
                 {
-                "module": "es6",
+                "module": "esmodule",
                 "in-source": true
                 }
             ],
@@ -1081,6 +1100,55 @@ pub mod tests {
         assert!(config.get_deprecations().is_empty());
     }
 
+    #[test]
+    fn test_package_specs_es6_global_deprecation() {
+        let json = r#"
+        {
+            "name": "testrepo",
+            "sources": {
+                "dir": "src",
+                "subdirs": true
+            },
+            "package-specs": [
+                {
+                "module": "es6-global",
+                "in-source": true
+                }
+            ],
+            "suffix": ".mjs"
+        }
+        "#;
+
+        let config = Config::new_from_json_string(json).expect("a valid json string");
+        assert_eq!(
+            config.get_deprecations(),
+            [DeprecationWarning::PackageSpecsEs6Global]
+        );
+    }
+
+    #[test]
+    fn test_package_specs_es6_deprecation() {
+        let json = r#"
+        {
+            "name": "testrepo",
+            "sources": {
+                "dir": "src",
+                "subdirs": true
+            },
+            "package-specs": [
+                {
+                "module": "es6",
+                "in-source": true
+                }
+            ],
+            "suffix": ".mjs"
+        }
+        "#;
+
+        let config = Config::new_from_json_string(json).expect("a valid json string");
+        assert_eq!(config.get_deprecations(), [DeprecationWarning::PackageSpecsEs6]);
+    }
+
     #[test]
     fn test_unknown_fields_are_collected() {
         let json = r#"
@@ -1150,7 +1218,7 @@ pub mod tests {
             },
             "package-specs": [
                 {
-                "module": "es6",
+                "module": "esmodule",
                 "in-source": true
                 }
             ],
@@ -1185,7 +1253,7 @@ pub mod tests {
             },
             "package-specs": [
                 {
-                "module": "es6",
+                "module": "esmodule",
                 "in-source": true
                 }
             ],

rewatch/testrepo/packages/dep01/rescript.json

@@ -5,7 +5,7 @@
     "subdirs": true
   },
   "package-specs": {
-    "module": "es6",
+    "module": "esmodule",
     "in-source": true
   },
   "suffix": ".bs.js",

rewatch/testrepo/packages/dep02/rescript.json

@@ -5,7 +5,7 @@
     "subdirs": true
   },
   "package-specs": {
-    "module": "es6",
+    "module": "esmodule",
     "in-source": true
   },
   "suffix": ".bs.js",

rewatch/testrepo/packages/deprecated-config/rescript.json

@@ -6,7 +6,7 @@
     "subdirs": true
   },
   "package-specs": {
-    "module": "es6",
+    "module": "esmodule",
     "in-source": true
   },
   "suffix": ".mjs",

rewatch/testrepo/packages/file-casing-no-namespace/rescript.json

@@ -8,7 +8,7 @@
   ],
   "package-specs": [
     {
-      "module": "es6",
+      "module": "esmodule",
       "in-source": true
     }
   ],

rewatch/testrepo/packages/file-casing/rescript.json

@@ -9,7 +9,7 @@
   ],
   "package-specs": [
     {
-      "module": "es6",
+      "module": "esmodule",
       "in-source": true
     }
   ],

rewatch/testrepo/packages/namespace-casing/rescript.json

@@ -9,7 +9,7 @@
   ],
   "package-specs": [
     {
-      "module": "es6",
+      "module": "esmodule",
       "in-source": true
     }
   ],

rewatch/testrepo/packages/new-namespace/rescript.json

@@ -7,7 +7,7 @@
     "subdirs": true
   },
   "package-specs": {
-    "module": "es6",
+    "module": "esmodule",
     "in-source": true
   },
   "dependencies": ["@testrepo/dep01"],