rescript-relay 3.0.0-rc.3 → 3.0.0-rc.5

package/CHANGELOG.md

@@ -4,6 +4,23 @@
 
 # **Version 3**
 
+# Unreleased
+
+# 3.0.0-rc.5
+
+- Support for Relay `@updatable` queries and fragments (https://relay.dev/docs/guided-tour/updating-data/imperatively-modifying-store-data/#optimistic-updaters-vs-updaters). https://github.com/zth/rescript-relay/pull/501
+- Attempt 2: Fix an error in postinstall.js when reinstalling rescript-relay dependency (https://github.com/zth/rescript-relay/pull/493)
+- Suppress more false positives for unused things in generated Relay files.
+
+# 3.0.0-rc.4
+
+- Set explicit peer dependencies: `@rescript/react >= 0.12.1`, `react-relay@16.0.0`, `relay-runtime@16.0.0`.
+- Clean up connection handler generators now that v3 let us use a better representation.
+- Fix issue with default args in connection id makers. https://github.com/zth/rescript-relay/pull/488
+- Fix issue with PPX not actually converting all functions to uncurried.
+- Rename `bsconfig.json` -> `rescript.json`.
+- Fix an error in postinstall.js when reinstalling rescript-relay dependency (https://github.com/zth/rescript-relay/pull/487)
+
 # 3.0.0-rc.3
 
 - Fix issue with custom scalars in arrays not being autoconverted properly.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "rescript-relay",
-  "version": "3.0.0-rc.3",
+  "version": "3.0.0-rc.5",
   "main": "src/RescriptRelay.res",
   "license": "MIT",
   "author": "Gabriel Nordeborn",
@@ -32,7 +32,7 @@
     "rescript-relay-cli": "cli/cli.js"
   },
   "scripts": {
-    "build": "rescript build -with-deps",
+    "build": "rescript",
     "build:test": "./build-compiler-dev.sh && ./rescript-relay-compiler",
     "postinstall": "node postinstall.js",
     "test": "jest",
@@ -40,7 +40,7 @@
   },
   "devDependencies": {
     "@glennsl/rescript-fetch": "^0.2.0",
-    "@rescript/react": "0.12.0",
+    "@rescript/react": "0.12.1",
     "@testing-library/jest-dom": "^5.16.3",
     "@testing-library/react": "^13.0.0-alpha.6",
     "graphql-query-test-mock": "^0.12.1",
@@ -54,8 +54,8 @@
     "rescript": "^11.0.0"
   },
   "peerDependencies": {
-    "@rescript/react": "^0.12.0",
-    "react-relay": ">=16.0.0",
+    "@rescript/react": "^0.12.1",
+    "react-relay": "16.0.0",
     "relay-runtime": "*",
     "rescript": "^11.0.0"
   },

package/postinstall.js

@@ -36,9 +36,6 @@ function getRelayCompilerPlatformSuffix() {
  */
 
 function ppxArch() {
-  /**
-   * Use Rosetta for ARM on macOS
-   */
   if (platform === "darwin" && process.arch === "arm64") {
     return "arm64";
   }
@@ -99,20 +96,13 @@ function copyPlatformBinaries(platform) {
   /**
    * Copy the PPX
    */
-  fs.copyFileSync(
-    path.join(__dirname, "ppx-" + platform),
-    path.join(__dirname, "ppx")
-  );
-  fs.chmodSync(path.join(__dirname, "ppx"), 0777);
+  const ppxFinalFilename = platform === "windows-latest" ? "ppx.exe" : "ppx";
+  const ppxFinalPath = path.join(__dirname, ppxFinalFilename);
 
-  // Windows seems to need an .exe file as well.
-  if (platform === "windows-latest") {
-    fs.copyFileSync(
-      path.join(__dirname, "ppx-" + platform),
-      path.join(__dirname, "ppx.exe")
-    );
-    fs.chmodSync(path.join(__dirname, "ppx.exe"), 0777);
+  if (!fs.existsSync(ppxFinalPath)) {
+    fs.copyFileSync(path.join(__dirname, "ppx-" + platform), ppxFinalPath);
   }
+  fs.chmodSync(ppxFinalPath, 0o777);
 
   /**
    * Copy the Relay compiler
@@ -120,22 +110,35 @@ function copyPlatformBinaries(platform) {
 
   var platformSuffix = getRelayCompilerPlatformSuffix();
 
-  fs.copyFileSync(
-    path.join(
-      __dirname,
-      "relay-compiler-" + platformSuffix,
-      platformSuffix === "win-x64" ? "relay.exe" : "relay"
-    ),
-    path.join(__dirname, "rescript-relay-compiler.exe")
+  const rescriptRelayCompilerFinalPath = path.join(
+    __dirname,
+    "rescript-relay-compiler.exe"
   );
-  fs.chmodSync(path.join(__dirname, "rescript-relay-compiler.exe"), 0777);
+
+  if (!fs.existsSync(rescriptRelayCompilerFinalPath)) {
+    fs.copyFileSync(
+      path.join(
+        __dirname,
+        "relay-compiler-" + platformSuffix,
+        platformSuffix === "win-x64" ? "relay.exe" : "relay"
+      ),
+      rescriptRelayCompilerFinalPath
+    );
+  }
+  fs.chmodSync(rescriptRelayCompilerFinalPath, 0o777);
+}
+
+function unlinkIfNotExistsSync(path) {
+  if (fs.existsSync(path)) {
+    fs.unlinkSync(path);
+  }
 }
 
 function removeInitialBinaries() {
-  fs.unlinkSync(path.join(__dirname, "ppx-macos-arm64"));
-  fs.unlinkSync(path.join(__dirname, "ppx-macos-latest"));
-  fs.unlinkSync(path.join(__dirname, "ppx-windows-latest"));
-  fs.unlinkSync(path.join(__dirname, "ppx-linux"));
+  unlinkIfNotExistsSync(path.join(__dirname, "ppx-macos-arm64"));
+  unlinkIfNotExistsSync(path.join(__dirname, "ppx-macos-latest"));
+  unlinkIfNotExistsSync(path.join(__dirname, "ppx-windows-latest"));
+  unlinkIfNotExistsSync(path.join(__dirname, "ppx-linux"));
   fs.rmSync(path.join(__dirname, "relay-compiler-linux-x64"), {
     recursive: true,
     force: true,

package/{bsconfig.json → rescript.json}

@@ -2,9 +2,6 @@
   "name": "rescript-relay",
   "version": "0.1.0",
   "namespace": false,
-  "reason": {
-    "react-jsx": 3
-  },
   "sources": [
     {
       "dir": "src"
@@ -18,6 +15,5 @@
   "suffix": ".bs.js",
   "warnings": {
     "error": "+101"
-  },
-  "refmt": 3
+  }
 }

package/src/RescriptRelay.res

@@ -9,6 +9,7 @@ type mutationNode<'node>
 type subscriptionNode<'node>
 
 type fragmentRefs<'fragments>
+type updatableFragmentRefs<'fragments>
 
 type dataId
 type recordSourceRecords = Js.Json.t

package/src/RescriptRelay.resi

@@ -30,6 +30,9 @@ type subscriptionNode<'node>
 /**This type shows all of the fragments that has been spread on this particular object.*/
 type fragmentRefs<'fragments>
 
+/**This type shows all of the updatable fragments that has been spread on this particular object.*/
+type updatableFragmentRefs<'fragments>
+
 /**The type of the id Relay uses to identify records in its store.*/
 type dataId
 

package/src/utils.js

@@ -49,6 +49,7 @@ function traverse(
   if (addFragmentOnRoot) {
     newObj = getNewObj(newObj, currentObj);
     newObj.fragmentRefs = Object.assign({}, newObj);
+    newObj.updatableFragmentRefs = newObj.fragmentRefs;
   }
 
   for (var key in currentObj) {
@@ -194,6 +195,8 @@ function traverse(
         if (shouldAddFragmentFn && typeof v === "object" && !Array.isArray(v)) {
           var objWithFragmentFn = Object.assign({}, v);
           objWithFragmentFn.fragmentRefs = Object.assign({}, objWithFragmentFn);
+          objWithFragmentFn.updatableFragmentRefs =
+            objWithFragmentFn.fragmentRefs;
           return objWithFragmentFn;
         }
 
@@ -273,6 +276,9 @@ function traverse(
         newObj = getNewObj(newObj, currentObj);
         var objWithFragmentFn = Object.assign({}, v);
         objWithFragmentFn.fragmentRefs = Object.assign({}, objWithFragmentFn);
+        objWithFragmentFn.updatableFragmentRefs =
+          objWithFragmentFn.fragmentRefs;
+
         newObj[key] = objWithFragmentFn;
       }
     }

package/src/utils.mjs

@@ -49,6 +49,7 @@ function traverse(
   if (addFragmentOnRoot) {
     newObj = getNewObj(newObj, currentObj);
     newObj.fragmentRefs = Object.assign({}, newObj);
+    newObj.updatableFragmentRefs = newObj.fragmentRefs;
   }
 
   for (var key in currentObj) {
@@ -194,6 +195,8 @@ function traverse(
         if (shouldAddFragmentFn && typeof v === "object" && !Array.isArray(v)) {
           var objWithFragmentFn = Object.assign({}, v);
           objWithFragmentFn.fragmentRefs = Object.assign({}, objWithFragmentFn);
+          objWithFragmentFn.updatableFragmentRefs =
+            objWithFragmentFn.fragmentRefs;
           return objWithFragmentFn;
         }
 
@@ -273,6 +276,9 @@ function traverse(
         newObj = getNewObj(newObj, currentObj);
         var objWithFragmentFn = Object.assign({}, v);
         objWithFragmentFn.fragmentRefs = Object.assign({}, objWithFragmentFn);
+        objWithFragmentFn.updatableFragmentRefs =
+          objWithFragmentFn.fragmentRefs;
+
         newObj[key] = objWithFragmentFn;
       }
     }