rescript-relay 1.0.0-beta.12 → 1.0.0-beta.13

package/CHANGELOG.md

@@ -1,6 +1,6 @@
 # master
 
-# 1.0.0-beta.12
+# 1.0.0-beta.13
 
 _[Here's a commit showing a project being upgraded to this version](https://github.com/zth/rescript-relay/commit/5831c2f1f0f13eedc1cb60468c32fd32b2dc01d3)_
 
@@ -29,10 +29,20 @@ You can go ahead and remove these packages, that are no longer needed, as the co
 - Improved utils for [dealing with enums](https://rescript-relay-documentation.vercel.app/docs/enums).
 - `recordSourceRecords` is now typed as `Js.Json.t` rather than being abstract.
 
+## Breaking changes
+
+- `refetchVariables` now works as intended with regards to supplying only the variables you want _changed_ when refetching, as [detailed under `variables` here](https://relay.dev/docs/next/api-reference/use-refetchable-fragment/#return-value). This means that what was previously `makeRefetchVariables(~someValue=123, ())` should now be `makeRefetchVariables(~someValue=Some(123), ())`. More details on this in the docs before a `1.0.0` release. Thanks to [@tsnobip](https://github.com/tsnobip) for fixing this!
+
 ## Beta fix changelog
 
 ### unreleased
 
+### beta.13
+
+- `refetchVariables` now works as intended with regards to supplying only the variables you want _changed_ when refetching, as [detailed under `variables` here](https://relay.dev/docs/next/api-reference/use-refetchable-fragment/#return-value).
+- Make all object makers inlined. This should improve bundle size some.
+- Support more Linux versions in CI (like the images Vercel uses).
+
 ### beta.12
 
 - Make CLI work with `relay.config.cjs`.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "rescript-relay",
-  "version": "1.0.0-beta.12",
+  "version": "1.0.0-beta.13",
   "main": "src/RescriptRelay.res",
   "license": "MIT",
   "author": "Gabriel Nordeborn",
@@ -59,6 +59,7 @@
     "rescript": "^9.1.2"
   },
   "dependencies": {
+    "detect-libc": "^2.0.1",
     "rescript": "^9.1.2"
   },
   "resolutions": {

package/postinstall.js

@@ -10,6 +10,7 @@
 var path = require("path");
 var cp = require("child_process");
 var fs = require("fs");
+var { isNonGlibcLinux } = require("detect-libc");
 var platform = process.platform;
 
 function getRelayCompilerPlatformSuffix() {
@@ -17,6 +18,8 @@ function getRelayCompilerPlatformSuffix() {
     return "macos-x64";
   } else if (process.platform === "darwin" && process.arch === "arm64") {
     return "macos-arm64";
+  } else if (process.platform === "linux" && isNonGlibcLinux) {
+    return "linux-musl";
   } else if (process.platform === "linux" && process.arch === "x64") {
     return "linux-x64";
   }

package/src/RescriptRelay_Internal.bs.js

@@ -3,23 +3,51 @@
 
 var Curry = require("rescript/lib/js/curry.js");
 var React = require("react");
+var Js_dict = require("rescript/lib/js/js_dict.js");
+var Caml_obj = require("rescript/lib/js/caml_obj.js");
+var Belt_Array = require("rescript/lib/js/belt_Array.js");
+var Belt_Option = require("rescript/lib/js/belt_Option.js");
 var Caml_option = require("rescript/lib/js/caml_option.js");
 
-var internal_cleanObjectFromUndefinedRaw = (function (obj) {
-    if (!obj) {
-      return obj;
-    }
-
-    var newObj = {};
+function internal_keepMapFieldsRaw(record, f) {
+  return Belt_Option.map(record, (function (obj) {
+                return Js_dict.fromArray(Belt_Array.keepMap(Js_dict.entries(obj), f));
+              }));
+}
 
-    Object.keys(obj).forEach(function(key) {
-      if (typeof obj[key] !== 'undefined') {
-        newObj[key] = obj[key];
-      }
-    });
+function internal_cleanObjectFromUndefinedRaw(record) {
+  return internal_keepMapFieldsRaw(record, (function (param) {
+                var value = param[1];
+                if (value !== undefined) {
+                  return [
+                          param[0],
+                          Caml_option.valFromOption(value)
+                        ];
+                }
+                
+              }));
+}
 
-    return newObj;
-  });
+function internal_removeUndefinedAndConvertNullsRaw(record) {
+  return internal_keepMapFieldsRaw(record, (function (param) {
+                var value = param[1];
+                var key = param[0];
+                var match = Caml_obj.caml_equal(value, Caml_option.some(undefined));
+                if (value !== undefined) {
+                  return [
+                          key,
+                          Caml_option.valFromOption(value)
+                        ];
+                } else if (match) {
+                  return [
+                          key,
+                          null
+                        ];
+                } else {
+                  return ;
+                }
+              }));
+}
 
 function internal_useConvertedValue(convert, v) {
   return React.useMemo((function () {
@@ -38,5 +66,6 @@ function internal_nullableToOptionalExnHandler(x) {
 
 exports.internal_useConvertedValue = internal_useConvertedValue;
 exports.internal_cleanObjectFromUndefinedRaw = internal_cleanObjectFromUndefinedRaw;
+exports.internal_removeUndefinedAndConvertNullsRaw = internal_removeUndefinedAndConvertNullsRaw;
 exports.internal_nullableToOptionalExnHandler = internal_nullableToOptionalExnHandler;
 /* react Not a pure module */

package/src/RescriptRelay_Internal.res

@@ -1,23 +1,27 @@
-// We occasionally have to remove undefined keys from objects, something I haven't figured out how to do with pure BuckleScript
-let internal_cleanObjectFromUndefinedRaw = %raw(
-  `
-  function (obj) {
-    if (!obj) {
-      return obj;
-    }
-
-    var newObj = {};
+let internal_keepMapFieldsRaw = (record, f) =>
+  record
+  ->Obj.magic
+  ->Belt.Option.map(obj => obj->Js.Dict.entries->Belt.Array.keepMap(f)->Js.Dict.fromArray)
+  ->Obj.magic
 
-    Object.keys(obj).forEach(function(key) {
-      if (typeof obj[key] !== 'undefined') {
-        newObj[key] = obj[key];
-      }
-    });
+// we need to do this until we can use @obj on record types
+// see https://github.com/rescript-lang/rescript-compiler/pull/5253
+let internal_cleanObjectFromUndefinedRaw = record =>
+  internal_keepMapFieldsRaw(record, ((key, value)) => {
+    switch value {
+    | Some(value) => Some((key, value))
+    | None => None
+    }
+  })
 
-    return newObj;
-  }
-`
-)
+let internal_removeUndefinedAndConvertNullsRaw = record =>
+  internal_keepMapFieldsRaw(record, ((key, value)) => {
+    switch (value, value == Some(None)) {
+    | (Some(value), _) => Some((key, Js.Nullable.return(value)))
+    | (_, true) => Some((key, Js.Nullable.null))
+    | (None, _) => None
+    }
+  })
 
 let internal_useConvertedValue = (convert, v) => React.useMemo1(() => convert(v), [v])
 

package/src/RescriptRelay_Internal.resi

@@ -1,5 +1,6 @@
 let internal_useConvertedValue: ('a => 'a, 'a) => 'a
 let internal_cleanObjectFromUndefinedRaw: 't => 't
+let internal_removeUndefinedAndConvertNullsRaw: 't => 't
 let internal_nullableToOptionalExnHandler: option<option<'b> => 'a> => option<
   Js.Nullable.t<'b> => 'a,
 >