@solidjs/signals 0.10.6 → 0.10.8

package/dist/dev.js

@@ -1287,7 +1287,7 @@ function signal(v, options, firewall = null) {
   };
   s._name = options?.name ?? "signal";
   firewall && (firewall._child = s);
-  if (snapshotCaptureActive) {
+  if (snapshotCaptureActive && !s._pureWrite) {
     s._snapshotValue = v === undefined ? NO_SNAPSHOT : v;
     snapshotSources.add(s);
   }
@@ -1306,9 +1306,25 @@ function optimisticComputed(fn, initialValue, options) {
 function isEqual(a, b) {
   return a === b;
 }
+let strictRead = false;
+function setStrictRead(v) {
+  const prev = strictRead;
+  strictRead = v;
+  return prev;
+}
 function untrack(fn) {
-  if (!tracking) return fn();
+  if (!tracking && !strictRead) return fn();
   tracking = false;
+  if (strictRead) {
+    const prev = strictRead;
+    strictRead = false;
+    try {
+      return fn();
+    } finally {
+      tracking = true;
+      strictRead = prev;
+    }
+  }
   try {
     return fn();
   } finally {
@@ -1406,6 +1422,10 @@ function read(el) {
       return snapshot;
     }
   }
+  if (strictRead && !tracking)
+    console.warn(
+      `Untracked reactive read in ${strictRead}. This value won't update — use untrack() if intentional.`
+    );
   return !c ||
     currentOptimisticLane !== null ||
     el._pendingValue === NOT_PENDING ||
@@ -2121,6 +2141,10 @@ const storeTraps = {
         );
       }
     }
+    if (strictRead && !tracking && typeof property === "string")
+      console.warn(
+        `Untracked reactive read in ${strictRead}. This value won't update — use untrack() if intentional.`
+      );
     return isWrappable(value) ? wrap(value, target) : value;
   },
   has(target, property) {
@@ -2624,18 +2648,29 @@ function omit(props, ...keys) {
 }
 function mapArray(list, map, options) {
   const keyFn = typeof options?.keyed === "function" ? options.keyed : undefined;
+  const indexes = map.length > 1;
+  const wrappedMap = options?.name
+    ? (...args) => {
+        setStrictRead(options.name);
+        try {
+          return map(...args);
+        } finally {
+          setStrictRead(false);
+        }
+      }
+    : map;
   return createMemo(
     updateKeyedMap.bind({
       _owner: createOwner(),
       _len: 0,
       _list: list,
       _items: [],
-      _map: map,
+      _map: wrappedMap,
       _mappings: [],
       _nodes: [],
       _key: keyFn,
       _rows: keyFn || options?.keyed === false ? [] : undefined,
-      _indexes: map.length > 1 ? [] : undefined,
+      _indexes: indexes ? [] : undefined,
       _fallback: options?.fallback
     })
   );
@@ -2767,12 +2802,22 @@ function updateKeyedMap() {
   return this._mappings;
 }
 function repeat(count, map, options) {
+  const wrappedMap = options?.name
+    ? i => {
+        setStrictRead(options.name);
+        try {
+          return map(i);
+        } finally {
+          setStrictRead(false);
+        }
+      }
+    : map;
   return updateRepeat.bind({
     _owner: createOwner(),
     _len: 0,
     _offset: 0,
     _count: count,
-    _map: map,
+    _map: wrappedMap,
     _nodes: [],
     _mappings: [],
     _from: options?.from,
@@ -3030,6 +3075,7 @@ export {
   runWithOwner,
   setContext,
   setSnapshotCapture,
+  setStrictRead,
   snapshot,
   untrack
 };