teamplay 0.3.8 → 0.3.10

package/index.js

@@ -14,7 +14,7 @@ export { GLOBAL_ROOT_ID } from './orm/Root.js'
 export const $ = _getRootSignal({ rootId: GLOBAL_ROOT_ID, rootFunction: universal$ })
 export default $
 export { default as sub } from './orm/sub.js'
-export { default as useSub, setUseDeferredValue as __setUseDeferredValue } from './react/useSub.js'
+export { default as useSub, useAsyncSub, setUseDeferredValue as __setUseDeferredValue } from './react/useSub.js'
 export { default as observer } from './react/observer.js'
 export { connection, setConnection, getConnection, fetchOnly, setFetchOnly, publicOnly, setPublicOnly } from './orm/connection.js'
 export { useId, useNow, useScheduleUpdate, useTriggerUpdate } from './react/helpers.js'

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "teamplay",
-  "version": "0.3.8",
+  "version": "0.3.10",
   "description": "Full-stack signals ORM with multiplayer",
   "type": "module",
   "main": "index.js",
@@ -23,12 +23,12 @@
   },
   "dependencies": {
     "@nx-js/observer-util": "^4.1.3",
-    "@teamplay/backend": "^0.3.8",
-    "@teamplay/cache": "^0.3.8",
-    "@teamplay/channel": "^0.3.8",
-    "@teamplay/debug": "^0.3.8",
-    "@teamplay/schema": "^0.3.8",
-    "@teamplay/utils": "^0.3.8",
+    "@teamplay/backend": "^0.3.10",
+    "@teamplay/cache": "^0.3.10",
+    "@teamplay/channel": "^0.3.10",
+    "@teamplay/debug": "^0.3.10",
+    "@teamplay/schema": "^0.3.10",
+    "@teamplay/utils": "^0.3.10",
     "diff-match-patch": "^1.0.5",
     "events": "^3.3.0",
     "json0-ot-diff": "^1.1.2",
@@ -63,5 +63,5 @@
     ]
   },
   "license": "MIT",
-  "gitHead": "627be4811f622fe0202480b0c6f1bd2fead83e19"
+  "gitHead": "c6f36be025ca89afbf56a43d1a0960b30494d772"
 }

package/react/useSub.js

@@ -8,70 +8,79 @@ let TEST_THROTTLING = false
 // Currently it does lead to issues with extra rerenders and requires further investigation
 let USE_DEFERRED_VALUE = false
 
-export default function useSub (signal, params) {
+export function useAsyncSub (signal, params, options) {
+  return useSub(signal, params, { ...options, async: true })
+}
+
+export default function useSub (signal, params, options) {
   if (USE_DEFERRED_VALUE) {
-    return useSubDeferred(signal, params) // eslint-disable-line react-hooks/rules-of-hooks
+    return useSubDeferred(signal, params, options) // eslint-disable-line react-hooks/rules-of-hooks
   } else {
-    return useSubClassic(signal, params) // eslint-disable-line react-hooks/rules-of-hooks
+    return useSubClassic(signal, params, options) // eslint-disable-line react-hooks/rules-of-hooks
   }
 }
 
 // version of sub() which works as a react hook and throws promise for Suspense
-export function useSubDeferred (signal, params) {
+export function useSubDeferred (signal, params, { async = false } = {}) {
+  const $signalRef = useRef() // eslint-disable-line react-hooks/rules-of-hooks
+  const scheduleUpdate = useScheduleUpdate()
   signal = useDeferredValue(signal)
   params = useDeferredValue(params ? JSON.stringify(params) : undefined)
   params = params != null ? JSON.parse(params) : undefined
   const promiseOrSignal = params != null ? sub(signal, params) : sub(signal)
   // 1. if it's a promise, throw it so that Suspense can catch it and wait for subscription to finish
   if (promiseOrSignal.then) {
-    if (TEST_THROTTLING) {
-      // simulate slow network
-      throw new Promise((resolve, reject) => {
-        setTimeout(() => {
-          promiseOrSignal.then(resolve, reject)
-        }, TEST_THROTTLING)
-      })
+    const promise = maybeThrottle(promiseOrSignal)
+    if (async) {
+      scheduleUpdate(promise)
+      return
     }
-    throw promiseOrSignal
-  }
+    throw promise
   // 2. if it's a signal, we save it into ref to make sure it's not garbage collected while component exists
-  const $signalRef = useRef() // eslint-disable-line react-hooks/rules-of-hooks
-  if ($signalRef.current !== promiseOrSignal) $signalRef.current = promiseOrSignal
-  return promiseOrSignal
+  } else {
+    const $signal = promiseOrSignal
+    if ($signalRef.current !== $signal) $signalRef.current = $signal
+    return $signal
+  }
 }
 
 // classic version which initially throws promise for Suspense
 // but if we get a promise second time, we return the last signal and wait for promise to resolve
-export function useSubClassic (signal, params) {
+export function useSubClassic (signal, params, { async = false } = {}) {
   const $signalRef = useRef()
   const activePromiseRef = useRef()
   const scheduleUpdate = useScheduleUpdate()
   const promiseOrSignal = params != null ? sub(signal, params) : sub(signal)
   // 1. if it's a promise, throw it so that Suspense can catch it and wait for subscription to finish
   if (promiseOrSignal.then) {
-    let promise
-    if (TEST_THROTTLING) {
-      // simulate slow network
-      promise = new Promise((resolve, reject) => {
-        setTimeout(() => {
-          promiseOrSignal.then(resolve, reject)
-        }, TEST_THROTTLING)
-      })
-    } else {
-      promise = promiseOrSignal
-    }
+    const promise = maybeThrottle(promiseOrSignal)
     // first time we just throw the promise to be caught by Suspense
-    if (!$signalRef.current) throw promise
+    if (!$signalRef.current) {
+      // if we are in async mode, we just return nothing and let the user
+      // handle appearance of signal on their own.
+      // We manually schedule an update when promise resolves since we can't
+      // rely on Suspense in this case to automatically trigger component's re-render
+      if (async) {
+        scheduleUpdate(promise)
+        return
+      }
+      // in regular mode we throw the promise to be caught by Suspense
+      // this way we guarantee that the signal with all the data
+      // will always be there when component is rendered
+      throw promise
+    }
     // if we already have a previous signal, we return it and wait for new promise to resolve
     scheduleUpdate(promise)
     return $signalRef.current
-  }
   // 2. if it's a signal, we save it into ref to make sure it's not garbage collected while component exists
-  if ($signalRef.current !== promiseOrSignal) {
-    activePromiseRef.current = undefined
-    $signalRef.current = promiseOrSignal
+  } else {
+    const $signal = promiseOrSignal
+    if ($signalRef.current !== $signal) {
+      activePromiseRef.current = undefined
+      $signalRef.current = $signal
+    }
+    return $signal
   }
-  return promiseOrSignal
 }
 
 export function setTestThrottling (ms) {
@@ -86,3 +95,13 @@ export function resetTestThrottling () {
 export function setUseDeferredValue (value) {
   USE_DEFERRED_VALUE = value
 }
+
+// throttle to simulate slow network
+function maybeThrottle (promise) {
+  if (!TEST_THROTTLING) return promise
+  return new Promise((resolve, reject) => {
+    setTimeout(() => {
+      promise.then(resolve, reject)
+    }, TEST_THROTTLING)
+  })
+}

package/react/wrapIntoSuspense.js

@@ -19,21 +19,21 @@ export default function wrapIntoSuspense ({
         stateVersion: Symbol(), // eslint-disable-line symbol-description
         onStoreChange: undefined,
         scheduledUpdatePromise: undefined,
+        scheduleUpdate: promise => {
+          if (!promise?.then) throw Error('scheduleUpdate() expects a promise')
+          if (adm.scheduledUpdatePromise === promise) return
+          adm.scheduledUpdatePromise = promise
+          promise.then(() => {
+            if (adm.scheduledUpdatePromise !== promise) return
+            adm.scheduledUpdatePromise = undefined
+            adm.onStoreChange?.()
+          })
+        },
         subscribe (onStoreChange) {
           adm.onStoreChange = () => {
             adm.stateVersion = Symbol() // eslint-disable-line symbol-description
             onStoreChange()
           }
-          adm.scheduleUpdate = promise => {
-            if (!promise?.then) throw Error('scheduleUpdate() expects a promise')
-            if (adm.scheduledUpdatePromise === promise) return
-            adm.scheduledUpdatePromise = promise
-            promise.then(() => {
-              if (adm.scheduledUpdatePromise !== promise) return
-              adm.scheduledUpdatePromise = undefined
-              adm.onStoreChange?.()
-            })
-          }
           return () => {
             adm.onStoreChange = undefined
             adm.scheduledUpdatePromise = undefined