rescript-relay 0.0.0-test-rust-compiler-66f73645 → 0.0.0-test-rust-compiler-145baf5d

package/CHANGELOG.md

@@ -1,5 +1,36 @@
 # master
 
+# 1.0.0-beta.2
+
+_[Here's a commit showing a project being upgraded to this version](https://github.com/zth/rescript-relay/commit/5831c2f1f0f13eedc1cb60468c32fd32b2dc01d3)_
+
+The time has finally come - RescriptRelay `1.0.0` is in beta! The one, big major thing this release brings is that the ReScript type generation for the Relay compiler has been completely rewritten, and fully integrated into the new Relay Rust compiler. The RescriptRelay fork of the compiler is available and maintained [here])(https://github.com/zth/relay/tree/rescript-relay).
+
+## Remove Packages
+
+You can go ahead and remove these packages, that are no longer needed, as the compiler is now shipped in the main package:
+
+- `relay-config`
+- `relay-compiler`
+- `graphql` (if you don't use it for anything else)
+
+## Breaking Changes
+
+- The compiler expects the `__generated__` folder to always exist, so if you're not committing your artifacts to source control, make sure you add a `.gitkeep` to the generated folder so git keeps it around.
+
+## Improvements
+
+- The compiler itself should be _much_ faster than the old one. An order of magnitude faster. Especially for incremental compilation in watch mode.
+- There's no longer any need to manually select `__typename` on interfaces and unions for RescriptRelay's sake, unless you actually want to use it yourself.
+- We now support the `@required` directive from Relay, which is a new directive that lets you force non-nullability for nullable fields on the client side. You can then choose to throw an error if null values are encountered, or let the null value bubble up. Docs are coming soon, and you'll need to do `RescriptRelay.relayFeatureFlags.enableRequiredDirective = true` somewhere in your code to enable it for now.
+- The errors reported by the compiler is now quite a lot better.
+
+## Beta fix changelog
+
+### beta.2
+
+- Fix issue with recursive input objects not being converted correctly.
+
 # 0.23.0
 
 _[Here's a commit showing a project being upgraded to this version](https://github.com/zth/rescript-relay/commit/6e96dfafaec918b1d4e9519d3fcbf5e5c46be6c0)_

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "rescript-relay",
-  "version": "0.0.0-test-rust-compiler-66f73645",
+  "version": "0.0.0-test-rust-compiler-145baf5d",
   "main": "src/RescriptRelay.res",
   "license": "MIT",
   "author": "Gabriel Nordeborn",

package/src/RescriptRelay.res

@@ -39,6 +39,8 @@ type featureFlags = {
   mutable enableRelayContainersSuspense: bool,
   @as("ENABLE_PRECISE_TYPE_REFINEMENT")
   mutable enablePrecisTypeRefinement: bool,
+  @as("ENABLE_REQUIRED_DIRECTIVES")
+  mutable enableRequiredDirective: bool,
 }
 
 @module("relay-runtime")

package/src/RescriptRelay.resi

@@ -98,6 +98,8 @@ type featureFlags = {
   mutable enableRelayContainersSuspense: bool,
   @as("ENABLE_PRECISE_TYPE_REFINEMENT")
   mutable enablePrecisTypeRefinement: bool,
+  @as("ENABLE_REQUIRED_DIRECTIVES")
+  mutable enableRequiredDirective: bool,
 }
 
 @ocaml.doc(

package/src/experimental-router/RescriptRelayRouter.bs.js

@@ -315,6 +315,7 @@ function RescriptRelayRouter$Link(Props) {
   var className = Props.className;
   var classNameDynamic = Props.classNameDynamic;
   var browserTarget = Props.target;
+  var tabIndex = Props.tabIndex;
   var mode = Props.mode;
   var render = Props.render;
   var preloadOnHover = Props.preloadOnHover;
@@ -393,6 +394,9 @@ function RescriptRelayRouter$Link(Props) {
   if (id !== undefined) {
     tmp.id = Caml_option.valFromOption(id);
   }
+  if (tabIndex !== undefined) {
+    tmp.tabIndex = Caml_option.valFromOption(tabIndex);
+  }
   if (title !== undefined) {
     tmp.title = Caml_option.valFromOption(title);
   }

package/src/experimental-router/RescriptRelayRouter.res

@@ -306,6 +306,7 @@ module Link = {
     ~className=?,
     ~classNameDynamic=?,
     ~target as browserTarget=?,
+    ~tabIndex=?,
     ~mode=?,
     ~render=?,
     ~preloadOnHover=?,
@@ -351,6 +352,7 @@ module Link = {
         }}
         ?title
         ?id
+        ?tabIndex
         className={className->Belt.Option.getWithDefault("") ++
           switch classNameDynamic {
           | Some(f) => " " ++ f(url, to_->Url.make)

package/src/experimental-router/RescriptRelayRouter.resi

@@ -56,6 +56,7 @@ module Link: {
     ~className: string=?,
     ~classNameDynamic: (RescriptReactRouter.url, RescriptReactRouter.url) => string=?,
     ~target: [#self | #blank]=?,
+    ~tabIndex: int=?,
     ~mode: [#push | #replace]=?,
     ~render: (
       ~preload: unit => unit,

package/src/utils.js

@@ -270,7 +270,7 @@ function traverse(
  */
 function traverser(
   root,
-  _instructionMaps,
+  instructionMaps_,
   theConverters,
   nullableValue,
   rootObjectKey
@@ -279,7 +279,7 @@ function traverser(
     return nullableValue;
   }
 
-  var instructionMaps = _instructionMaps || {};
+  var instructionMaps = instructionMaps_ || {};
   var instructionMap = instructionMaps[rootObjectKey || "__root"];
 
   // No instructions
@@ -305,7 +305,15 @@ function traverser(
         return nullableValue;
       }
 
-      var n = unionRootConverter != null ? [v.__typename] : [];
+      var n = [];
+
+      // Since a root level union is treated as a "new root level", we'll need
+      // to do a separate check here of whether there's a fragment on the root
+      // we need to account for, or not.
+      if (unionRootConverter != null) {
+        n = [v.__typename];
+        fragmentsOnRoot = (instructionMap[v.__typename] || {}).f === "";
+      }
 
       var traversedObj = traverse(
         instructionMaps,
@@ -326,7 +334,14 @@ function traverser(
 
   var newObj = Object.assign({}, root);
 
-  var n = unionRootConverter != null ? [newObj.__typename] : [];
+  var n = [];
+
+  // Same as in the union array check above - if there's a fragment in the new
+  // root created by the union, we need to account for that separately here.
+  if (unionRootConverter != null) {
+    n = [newObj.__typename];
+    fragmentsOnRoot = (instructionMap[newObj.__typename] || {}).f === "";
+  }
 
   var v = traverse(
     instructionMaps,