Propagate JSX settings to dependencies (#7094)

* add build test for inhert the jsx settings

* fix to inhert jsx settings to deps

* change log

* fix test

Author: mununki

CHANGELOG.md

@@ -32,6 +32,7 @@
 - Fix "rescript format" with many files. https://github.com/rescript-lang/rescript-compiler/pull/7081
 - Fix bigint max, min https://github.com/rescript-lang/rescript-compiler/pull/7088
 - Fix parsing issue with nested variant pattern type spreads. https://github.com/rescript-lang/rescript-compiler/pull/7080
+- Fix JSX settings inheritance: only 'version' propagates to dependencies, preserving their 'mode' and 'module'. https://github.com/rescript-lang/rescript-compiler/pull/7094
 
 #### :nail_care: Polish
 

compiler/bsb/bsb_config_parse.ml

@@ -261,6 +261,12 @@ let interpret_json ~(filename : string) ~(json : Ext_json_types.t)
       in
       let bsc_flags = extract_string_list map Bsb_build_schemas.bsc_flags in
       let jsx = Bsb_jsx.from_map map in
+      let jsx, bsc_flags =
+        match package_kind with
+        | Pinned_dependency x | Dependency x ->
+          ({jsx with version = x.jsx.version}, bsc_flags)
+        | Toplevel -> (jsx, bsc_flags)
+      in
       {
         pinned_dependencies;
         gentype_config;

tests/build_tests/jsx_settings_inheritance/.gitignore

@@ -0,0 +1,2 @@
+!node_modules
+node_modules/**/lib

tests/build_tests/jsx_settings_inheritance/input.js

@@ -0,0 +1,5 @@
+//@ts-check
+var cp = require("child_process");
+var { rescript_exe } = require("#cli/bin_path");
+
+cp.execSync(rescript_exe, { cwd: __dirname });