@solidjs/signals 0.8.5 → 0.8.6

package/dist/dev.js

@@ -337,16 +337,19 @@ function notifySubs(node) {
   }
 }
 function recompute(el, create = false) {
-  deleteFromHeap(el, el._flags & REACTIVE_ZOMBIE ? zombieQueue : dirtyQueue);
-  if (el._pendingValue !== NOT_PENDING || el._pendingFirstChild || el._pendingDisposal)
-    disposeChildren(el);
-  else {
-    markDisposal(el);
-    globalQueue._pendingNodes.push(el);
-    el._pendingDisposal = el._disposal;
-    el._pendingFirstChild = el._firstChild;
-    el._disposal = null;
-    el._firstChild = null;
+  const honoraryOptimistic = el._type && el._transition != activeTransition;
+  if (!create) {
+    if (el._transition && activeTransition !== el._transition && !honoraryOptimistic)
+      globalQueue.initTransition(el);
+    deleteFromHeap(el, el._flags & REACTIVE_ZOMBIE ? zombieQueue : dirtyQueue);
+    if (el._transition) disposeChildren(el);
+    else {
+      markDisposal(el);
+      el._pendingDisposal = el._disposal;
+      el._pendingFirstChild = el._firstChild;
+      el._disposal = null;
+      el._firstChild = null;
+    }
   }
   const oldcontext = context;
   context = el;
@@ -373,16 +376,17 @@ function recompute(el, create = false) {
   }
   el._flags = REACTIVE_NONE;
   context = oldcontext;
-  const depsTail = el._depsTail;
-  let toRemove = depsTail !== null ? depsTail._nextDep : el._deps;
-  if (toRemove !== null) {
-    do {
-      toRemove = unlinkSubs(toRemove);
-    } while (toRemove !== null);
-    if (depsTail !== null) depsTail._nextDep = null;
-    else el._deps = null;
+  if (!(el._statusFlags & STATUS_PENDING)) {
+    const depsTail = el._depsTail;
+    let toRemove = depsTail !== null ? depsTail._nextDep : el._deps;
+    if (toRemove !== null) {
+      do {
+        toRemove = unlinkSubs(toRemove);
+      } while (toRemove !== null);
+      if (depsTail !== null) depsTail._nextDep = null;
+      else el._deps = null;
+    }
   }
-  const honoraryOptimistic = el._type && el._transition != activeTransition;
   const valueChanged =
     !el._equals ||
     !el._equals(
@@ -396,11 +400,8 @@ function recompute(el, create = false) {
   if (valueChanged || statusFlagsChanged) {
     if (valueChanged) {
       if (create || el._optimistic || honoraryOptimistic) el._value = value;
-      else {
-        if (el._pendingValue === NOT_PENDING) globalQueue._pendingNodes.push(el);
-        el._pendingValue = value;
-      }
-      if (el._pendingSignal) el._pendingSignal._set(value);
+      else el._pendingValue = value;
+      if (el._pendingSignal) el._pendingSignal._pendingValue = value;
     }
     for (let s = el._subs; s !== null; s = s._nextSub) {
       const queue = s._sub._flags & REACTIVE_ZOMBIE ? zombieQueue : dirtyQueue;
@@ -412,6 +413,13 @@ function recompute(el, create = false) {
       insertIntoHeapHeight(s._sub, s._sub._flags & REACTIVE_ZOMBIE ? zombieQueue : dirtyQueue);
     }
   }
+  if (
+    (!create || el._statusFlags & STATUS_PENDING) &&
+    !el._optimistic &&
+    !honoraryOptimistic &&
+    !el._transition
+  )
+    globalQueue._pendingNodes.push(el);
   if (el._transition && honoraryOptimistic) runInTransition(el, recompute);
 }
 function updateIfNecessary(el) {
@@ -444,7 +452,7 @@ function unlinkSubs(link) {
     dep._subs = nextSub;
     if (nextSub === null) {
       dep._unobserved?.();
-      dep._fn && unobserved(dep);
+      dep._fn && !dep._preventAutoDisposal && unobserved(dep);
     }
   }
   return nextDep;
@@ -456,7 +464,7 @@ function unobserved(el) {
     dep = unlinkSubs(dep);
   }
   el._deps = null;
-  runDisposal(el);
+  disposeChildren(el, true);
 }
 function link(dep, sub) {
   const prevDep = sub._depsTail;
@@ -712,6 +720,7 @@ function read(el) {
   let c = context;
   if (c?._root) c = c._parentComputed;
   if (c && tracking && !pendingCheck && !pendingValueCheck) {
+    if (el._fn && el._flags & REACTIVE_DISPOSED) recompute(el);
     link(el, c);
     const owner = el._firewall || el;
     if (owner._fn) {
@@ -1947,6 +1956,7 @@ function boundaryComputed(fn, propagationMask) {
     }
   });
   node._propagationMask = propagationMask;
+  node._preventAutoDisposal = true;
   return node;
 }
 function createBoundChildren(owner, fn, queue, mask) {