teamplay 0.4.0-alpha.47 → 0.4.0-alpha.49

package/orm/Compat/SignalCompat.js

@@ -225,6 +225,31 @@ class SignalCompat extends Signal {
     return setReplaceOnSignal($target, value)
   }
 
+  async create (path, value) {
+    const forwarded = forwardRef(this, 'create', arguments)
+    if (forwarded) return forwarded
+    if (arguments.length > 2) throw Error('Signal.create() expects zero to two arguments')
+    let segments = []
+    if (arguments.length === 2) {
+      segments = parseAtSubpath(path, 1, 'Signal.create()')
+    } else if (arguments.length === 1) {
+      if (typeof path === 'string' || typeof path === 'number') {
+        segments = parseAtSubpath(path, 1, 'Signal.create()')
+        value = {}
+      } else {
+        value = path
+      }
+    } else {
+      value = {}
+    }
+    const $target = resolveSignal(this, segments)
+    ensureCreateTarget($target, 'Signal.create()')
+    if ($target.get() != null) {
+      throw Error(`Signal.create() may only be used on a non-existing document path. Path: ${$target.path()}`)
+    }
+    return setReplaceOnSignal($target, value)
+  }
+
   async setDiffDeep (path, value) {
     const forwarded = forwardRef(this, 'setDiffDeep', arguments)
     if (forwarded) return forwarded
@@ -873,6 +898,15 @@ function ensureValueTarget ($signal) {
   return segments
 }
 
+function ensureCreateTarget ($signal, methodName) {
+  const segments = $signal[SEGMENTS]
+  if ($signal[IS_QUERY]) throw Error(`${methodName} can't be used on a query signal`)
+  if (segments.length !== 2) {
+    throw Error(`${methodName} may only be used on a document path`)
+  }
+  return segments
+}
+
 async function arrayPushOnSignal ($signal, value) {
   const segments = ensureArrayTarget($signal)
   const idFields = getIdFieldsForSegments(segments)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "teamplay",
-  "version": "0.4.0-alpha.47",
+  "version": "0.4.0-alpha.49",
   "description": "Full-stack signals ORM with multiplayer",
   "type": "module",
   "main": "index.js",
@@ -82,5 +82,5 @@
     ]
   },
   "license": "MIT",
-  "gitHead": "6e0aef0801b18f7e11dcbdc83768eeffe9573dc8"
+  "gitHead": "a0220c29390a8ecd47d012dc0e701113fb0907ba"
 }

package/react/promiseBatcher.js

@@ -24,12 +24,12 @@ export function getPromiseAll () {
   const pendingPromises = promises
   const pendingChecks = Array.from(checks.values())
   const hasPromises = pendingPromises.length > 0
-  const hasChecks = pendingChecks.length > 0
-  const result = !hasPromises && !hasChecks
-    ? null
-    : hasPromises || !areChecksReady(pendingChecks)
-      ? waitForBatchReady(pendingPromises, pendingChecks)
-      : null
+  // Checks are a materialization barrier for initial batch subscriptions.
+  // If there were no subscription promises in this render, we are in update mode
+  // and should not suspend the whole subtree.
+  const result = hasPromises
+    ? waitForBatchReady(pendingPromises, pendingChecks)
+    : null
   reset()
   return result
 }
@@ -66,11 +66,6 @@ async function waitForChecksReady (pendingChecks) {
   }
 }
 
-function areChecksReady (pendingChecks) {
-  if (pendingChecks.length === 0) return true
-  return getNotReadyChecks(pendingChecks).length === 0
-}
-
 function getNotReadyChecks (pendingChecks) {
   const notReady = []
   for (const check of pendingChecks) {

package/react/useSub.js

@@ -41,7 +41,14 @@ export function useSubDeferred (signal, params, { async = false, defer, batch =
   if (promiseOrSignal.then) {
     const promise = maybeThrottle(promiseOrSignal)
     if (batch) {
-      promiseBatcher.add(promise)
+      const hasPreviousSignal = !!$signalRef.current
+      // Batch suspense must block only on initial load.
+      // On resubscribe we keep rendering previous signal and refresh in background.
+      if (!hasPreviousSignal) {
+        promiseBatcher.add(promise)
+      } else {
+        scheduleUpdate(promise)
+      }
       if (async) scheduleUpdate(promise)
       return $signalRef.current
     }
@@ -71,9 +78,16 @@ export function useSubClassic (signal, params, { async = false, batch = false }
   if (promiseOrSignal.then) {
     const promise = maybeThrottle(promiseOrSignal)
     if (batch) {
-      promiseBatcher.add(promise)
+      const hasPreviousSignal = cache.has(id)
+      // Batch suspense must block only on initial load.
+      // On resubscribe we keep rendering previous signal and refresh in background.
+      if (!hasPreviousSignal) {
+        promiseBatcher.add(promise)
+      } else {
+        scheduleUpdate(promise)
+      }
       if (async) scheduleUpdate(promise)
-      if (cache.has(id)) return cache.get(id)
+      if (hasPreviousSignal) return cache.get(id)
       return
     }
     // first time we just throw the promise to be caught by Suspense