@rpcbase/client 0.110.0 → 0.112.0

package/auth/index.js

@@ -113,8 +113,11 @@ export const session_restrict = (ctx, next) => {
 export const get_uid = () => __user_id
 export const set_uid = (val) => {
   __user_id = val
-  const tenant_id_prefix = val.slice(0, 8)
-  storage.setItem(uid_storage_key(tenant_id_prefix), val)
+
+  if (typeof val === "string") {
+    const tenant_id_prefix = val.slice(0, 8)
+    storage.setItem(uid_storage_key(tenant_id_prefix), val)
+  }
 }
 
 export const get_tenant_id = () => __tenant_id

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rpcbase/client",
-  "version": "0.110.0",
+  "version": "0.112.0",
   "scripts": {
     "build-firebase": "webpack -c firebase/webpack.config.js",
     "build": "yarn build-firebase",

package/rts/getUseDocument.js

@@ -9,7 +9,7 @@ const getUseDocument = (register_query) => (...args) => {
   const res = useQuery(...args)
 
   let data
-  // cache will always return [] when there are no matching documents, but we want null instead
+  // WARNING: cache will always return [] when there are no matching documents, but we want null instead
   if (Array.isArray(res.data) && res.data.length === 0 && res.source === "cache") {
     data = null
   } else if (Array.isArray(res.data) && res.data.length  > 0) {

package/rts/getUseQuery/index.js

@@ -6,6 +6,9 @@ import debug from "debug"
 import _omit from "lodash/omit"
 import LZString from "lz-string"
 
+// TODO: remove this when stable
+import isEqual from "fast-deep-equal/react"
+
 import isEqualValues from "../../isEqualValues"
 import get_uid from "../../auth/get_uid"
 
@@ -50,6 +53,7 @@ const getUseQuery = (register_query) => (
 
   const [source, setSource] = useState()
 
+  const dataRef = useRef(null)
   const [data, setData] = useData({useStorage, storageKey, hasInitiallySetFromStorage})
 
   const [error, setError] = useState()
@@ -69,6 +73,8 @@ const getUseQuery = (register_query) => (
 
   const applyNewData = (newData, context) => {
     setData(newData)
+    // set data in a ref so that it doesn't force re-rendering ie: unsubscribe / resubscribe
+    dataRef.current = newData
 
     // we only save network queries
     if (useStorage && context.source === "network") {
@@ -158,7 +164,11 @@ const getUseQuery = (register_query) => (
         return
       }
 
-      if (!isEqualValues(data, newData)) {
+      if (isEqual(data, newData) && !isEqualValues(data, newData) && __DEV__) {
+        alert("EQUALITY MISMATCH THIS SHOULD NOT HAPPPEND!!!", data, newData)
+      }
+
+      if (!isEqualValues(dataRef.current, newData)) {
         applyContext(context)
         applyNewData(newData, context)
       } else {
@@ -170,8 +180,10 @@ const getUseQuery = (register_query) => (
       log && log("useQuery cleanup unsubscribe()")
       typeof unsubscribe === "function" && unsubscribe()
     }
+
+    // WARNING: do not change the hooks dependencies param or you risk creating infinite loops as it unsubscribes on cleanup
     // TODO: this isnt right we need to update on options change too
-  }, [JSON.stringify(query), data, key])
+  }, [JSON.stringify(query), key])
 
 
   const loadNextPage = useCallback(() => {