Merge pull request #154 from illusionalsagacity/optional-records

BREAKING: convert @obj and @deriving(abstract) to use optional record fields

Author: jeddeloh

.gitattributes

@@ -1,2 +1,4 @@
 *.re linguist-language=Reason
 *.rei linguist-language=Reason
+*.res linguist-language=ReScript
+*.resi linguist-language=ReScript

.gitignore

@@ -8,3 +8,4 @@ yarn-error.log
 .vscode/
 lib/
 node_modules/
+.yarn/install-state.gz

EXAMPLES/bsconfig.json

@@ -20,10 +20,6 @@
     "version": 4,
     "mode": "automatic"
   },
-  "reason": {
-    "react-jsx": 3
-  },
-  "refmt": 3,
   "sources": [
     {
       "dir": "src",

EXAMPLES/package.json

@@ -11,7 +11,7 @@
     "start": "rescript build -with-deps -w"
   },
   "devDependencies": {
-    "@reasonml-community/graphql-ppx": "1.2.3",
+    "@reasonml-community/graphql-ppx": "1.2.4-1345e061.0",
     "graphql-client-example-server": "1.5.2",
     "html-webpack-plugin": "5.5.0",
     "rescript": "10.1.2",

bsconfig.json

@@ -6,10 +6,6 @@
       "in-source": true
     }
   ],
-  "reason": {
-    "react-jsx": 3
-  },
-  "refmt": 3,
   "jsx": {
     "version": 4,
     "mode": "automatic"

package.json

@@ -35,15 +35,11 @@
     "@apollo/client": "^3.5.0",
     "@reasonml-community/graphql-ppx": "^1.0.0",
     "@rescript/react": "~0.10. || ~0.11.0",
-    "bs-platform": "^8.2.0 || ^9.0.0",
-    "rescript": "^9.0.0 || ^10.0.0"
+    "rescript": "^10.0.0"
   },
   "peerDependenciesMeta": {
     "rescript": {
       "optional": true
-    },
-    "bs-platform": {
-      "optional": true
     }
   },
   "workspaces": [

src/@apollo/client/cache/core/ApolloClient__Cache_Core_Cache.res

@@ -188,9 +188,9 @@ module ApolloCache = {
         ~options={
           id,
           fragment: Fragment.query,
-          fragmentName,
-          optimistic,
-          canonizeResults,
+          ?fragmentName,
+          ?optimistic,
+          ?canonizeResults,
         },
         ~optimistic?,
         (),
@@ -218,11 +218,11 @@ module ApolloCache = {
       ->Js_.readQuery(
         ~options=DataProxy.ReadQueryOptions.toJs(
           {
-            id,
+            ?id,
             query: Operation.query,
             variables,
-            optimistic,
-            canonizeResults,
+            ?optimistic,
+            ?canonizeResults,
           },
           ~mapJsVariables,
           ~serializeVariables=Operation.serializeVariables,
@@ -247,12 +247,12 @@ module ApolloCache = {
       js->Js_.writeFragment(
         ~options=DataProxy.WriteFragmentOptions.toJs(
           {
-            broadcast,
+            ?broadcast,
             data,
             id,
             fragment: Fragment.query,
-            fragmentName,
-            overwrite,
+            ?fragmentName,
+            ?overwrite,
           },
           ~serialize=Fragment.serialize,
         ),
@@ -275,12 +275,12 @@ module ApolloCache = {
       js->Js_.writeQuery(
         ~options=DataProxy.WriteQueryOptions.toJs(
           {
-            broadcast,
+            ?broadcast,
             data,
-            id,
+            ?id,
             query: Operation.query,
             variables,
-            overwrite,
+            ?overwrite,
           },
           ~mapJsVariables,
           ~serialize=Operation.serialize,
@@ -310,13 +310,13 @@ module ApolloCache = {
       ->Js_.updateQuery(
         ~options=DataProxy.UpdateQueryOptions.toJs(
           {
-            optimistic,
-            canonizeResults,
-            broadcast,
-            id,
+            ?optimistic,
+            ?canonizeResults,
+            ?broadcast,
+            ?id,
             query: Operation.query,
             variables,
-            overwrite,
+            ?overwrite,
           },
           ~mapJsVariables,
           ~serializeVariables=Operation.serializeVariables,
@@ -350,13 +350,13 @@ module ApolloCache = {
       js
       ->Js_.updateFragment(
         ~options=DataProxy.UpdateFragmentOptions.toJs({
-          optimistic,
-          canonizeResults,
-          broadcast,
+          ?optimistic,
+          ?canonizeResults,
+          ?broadcast,
           id,
           fragment: Fragment.query,
-          fragmentName,
-          overwrite,
+          ?fragmentName,
+          ?overwrite,
         }),
         ~update=jsData =>
           jsData