Make Jsx.element a private empty record to avoid unnecessary Primitive_option.some

Author: cknitt

runtime/Jsx.res

@@ -22,7 +22,10 @@
  * along with this program; if not, write to the Free Software
  * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. */
 
-type element
+// Define this as a private empty record so that the compiler does not
+// unnecessarily add `Primitive_option.some` calls for optional props.
+type element = private {}
+
 type ref
 
 @val external null: element = "null"

tests/tests/src/jsx_optional_props_test.mjs

@@ -1,6 +1,5 @@
 // Generated by ReScript, PLEASE EDIT WITH CARE
 
-import * as Primitive_option from "rescript/lib/es6/Primitive_option.js";
 import * as JsxRuntime from "react/jsx-runtime";
 
 function Jsx_optional_props_test$ComponentWithOptionalProps(props) {
@@ -14,7 +13,7 @@ let ComponentWithOptionalProps = {
 let _element = JsxRuntime.jsx(Jsx_optional_props_test$ComponentWithOptionalProps, {
   i: 1,
   s: "test",
-  element: Primitive_option.some(JsxRuntime.jsx("div", {}))
+  element: JsxRuntime.jsx("div", {})
 });
 
 export {

tests/tests/src/jsx_preserve_test.mjs

@@ -1,6 +1,5 @@
 // Generated by ReScript, PLEASE EDIT WITH CARE
 
-import * as Primitive_option from "rescript/lib/es6/Primitive_option.js";
 import * as JsxRuntime from "react/jsx-runtime";
 
 function Jsx_preserve_test$Icon(props) {
@@ -25,7 +24,7 @@ let _multiple_element_children = <div>
 </div>;
 
 let _single_element_fragment = <>
-  {Primitive_option.some(<input />)}
+  {<input />}
 </>;
 
 let _multiple_element_fragment = <>
@@ -201,7 +200,7 @@ let ComponentWithOptionalProps = {
 let _element = <Jsx_preserve_test$ComponentWithOptionalProps
   i={1}
   s="test"
-  element={Primitive_option.some(<div />)}
+  element={<div />}
 />;
 
 export {