teamplay 0.4.0-alpha.97 → 0.4.0-alpha.98

package/orm/Compat/SignalCompat.js

@@ -29,7 +29,7 @@ import {
 } from '../dataTree.js'
 import { on as onCustomEvent, removeListener as removeCustomEventListener } from './eventsCompat.js'
 import { waitForImperativeQueryReady } from './queryReadiness.js'
-import { normalizePattern, onModelEvent, removeModelListener } from './modelEvents.js'
+import { isModelEventsEnabled, normalizePattern, onModelEvent, removeModelListener } from './modelEvents.js'
 import { setRefLink, removeRefLink, getAllRefLinks } from './refRegistry.js'
 import { REF_TARGET, resolveRefSignalSafe, resolveRefSegmentsSafe } from './refFallback.js'
 import { runInBatch } from '../batchScheduler.js'
@@ -1061,7 +1061,9 @@ function setReplacePrivateCompatSync ($signal, value) {
     value = normalizeIdFields(value, idFields, segments[1])
   }
   setReplacePrivateData(getOwningRootId($signal), segments, value)
-  if (isSilentContextActive()) mirrorRefMutationFromTarget(segments, value)
+  if (shouldMirrorPrivateRefMutationLocally()) {
+    mirrorRefMutationFromTarget(segments, value)
+  }
 }
 
 function delPrivateCompatSync ($signal, options) {
@@ -1123,7 +1125,9 @@ async function setReplaceOnSignal ($signal, value) {
   }
   if (isPrivateMutationForbidden()) throw Error(ERRORS.publicOnly)
   const result = setReplacePrivateData(getOwningRootId($signal), segments, value)
-  if (isSilentContextActive()) mirrorRefMutationFromTarget(segments, value)
+  if (shouldMirrorPrivateRefMutationLocally()) {
+    mirrorRefMutationFromTarget(segments, value)
+  }
   return result
 }
 
@@ -1271,6 +1275,11 @@ function shouldMirrorPublicRefMutationLocally (segments) {
   return !docSubscriptions.hasRuntime(transportHash)
 }
 
+function shouldMirrorPrivateRefMutationLocally () {
+  if (isSilentContextActive()) return true
+  return !isModelEventsEnabled()
+}
+
 function shallowCopy (value) {
   const rawValue = raw(value)
   if (Array.isArray(rawValue)) return rawValue.slice()

package/orm/dataTree.js

@@ -326,6 +326,13 @@ export async function setPublicDocReplace (segments, value) {
   }
 
   const relativePath = segments.slice(2)
+  // json0 direct replace ops require every ancestor container to already exist.
+  // Racer-like compat set, however, materializes missing/primitive parents while
+  // descending into the path. Fall back to the older diff-based path when the
+  // direct op would target a non-existent/non-object ancestor.
+  if (!canApplyDirectReplaceOp(docState.snapshot || {}, relativePath)) {
+    return setPublicDoc(segments, value)
+  }
   const previous = getRaw(segments)
   const normalizedPrevious = normalizeUndefined(
     relativePath.length === 0 ? stripIdFields(previous, idFields) : previous
@@ -444,6 +451,16 @@ function normalizeUndefined (value) {
   return value === undefined ? null : value
 }
 
+function canApplyDirectReplaceOp (docSnapshot, relativePath) {
+  if (relativePath.length === 0) return true
+  let node = docSnapshot
+  for (let i = 0; i < relativePath.length - 1; i++) {
+    if (node == null || typeof node !== 'object') return false
+    node = node[relativePath[i]]
+  }
+  return node != null && typeof node === 'object'
+}
+
 function normalizeValueForOp (value) {
   let result = raw(value)
   if (result != null && typeof result === 'object') result = JSON.parse(JSON.stringify(result))

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "teamplay",
-  "version": "0.4.0-alpha.97",
+  "version": "0.4.0-alpha.98",
   "description": "Full-stack signals ORM with multiplayer",
   "type": "module",
   "main": "index.js",
@@ -83,5 +83,5 @@
     ]
   },
   "license": "MIT",
-  "gitHead": "633e39dc754332bcd3d95ddc4cfb3522f0bfd694"
+  "gitHead": "ae4e0e6ee39e395fa9eb48a2099cc1d5b2a4630d"
 }

package/react/useSub.js

@@ -44,8 +44,8 @@ export function useSubDeferred (signal, params, { async = false, defer, batch =
   // 1. if it's a promise, throw it so that Suspense can catch it and wait for subscription to finish
   if (promiseOrSignal.then) {
     const promise = maybeThrottle(promiseOrSignal)
+    const hasPreviousSignal = !!$signalRef.current
     if (batch) {
-      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) {
@@ -61,6 +61,11 @@ export function useSubDeferred (signal, params, { async = false, defer, batch =
       scheduleUpdate(promise)
       return
     }
+    // Keep previous snapshot during update re-subscribe and refresh in background.
+    if (hasPreviousSignal) {
+      scheduleUpdate(promise)
+      return $signalRef.current
+    }
     if (compatAttemptCleanup) registerCompatAttemptCleanup(signal, params)
     throw promise
   // 2. if it's a signal, we save it into ref to make sure it's not garbage collected while component exists