cogsbox-state 0.5.375 → 0.5.377

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cogsbox-state",
-  "version": "0.5.375",
+  "version": "0.5.377",
   "description": "React state management library with form controls and server sync",
   "type": "module",
   "main": "./dist/index.js",

package/src/CogsState.tsx

@@ -1827,8 +1827,7 @@ function createProxyHandler<T>(
               const isProgrammaticScroll = useRef(false);
               const prevTotalCountRef = useRef(0);
               const prevDepsRef = useRef(dependencies);
-              const lastScrollTopRef = useRef(0);
-
+              const lastUpdateAtScrollTop = useRef(0);
               const [shadowUpdateTrigger, setShadowUpdateTrigger] = useState(0);
 
               useEffect(() => {
@@ -2001,71 +2000,49 @@ function createProxyHandler<T>(
                 const container = containerRef.current;
                 if (!container) return;
 
-                // Define our threshold. Updating every half-item feels very smooth.
-                // You can adjust this value. Using a full itemHeight also works well.
-                const scrollThreshold = itemHeight / 2;
+                // The scroll distance threshold. One item's height is a great default.
+                const scrollThreshold = itemHeight;
 
                 const handleUserScroll = () => {
-                  // Exit early for programmatic scrolls. This is still essential.
+                  // Essential guard for our own programmatic scrolls.
                   if (isProgrammaticScroll.current) {
                     return;
                   }
 
                   const scrollTop = container.scrollTop;
 
-                  // --- Part 1: Handle state changes immediately (this is cheap) ---
-                  const isAtBottom =
-                    container.scrollHeight -
-                      scrollTop -
-                      container.clientHeight <
-                    1;
-
-                  if (!isAtBottom) {
-                    if (status !== "IDLE_NOT_AT_BOTTOM") {
-                      console.log(
-                        "USER ACTION: Scrolled up -> IDLE_NOT_AT_BOTTOM"
-                      );
-                      setStatus("IDLE_NOT_AT_BOTTOM");
-                    }
-                    shouldNotScroll.current = true;
-                  } else {
-                    if (status === "IDLE_NOT_AT_BOTTOM") {
-                      console.log(
-                        "USER ACTION: Scrolled back to bottom -> LOCKED_AT_BOTTOM"
-                      );
-                      setStatus("LOCKED_AT_BOTTOM");
-                    }
-                    shouldNotScroll.current = false;
-                  }
-
-                  // --- Part 2: The "Smarter" Threshold Check ---
-
-                  // Check if the scroll distance is greater than our threshold.
-                  // Math.abs() handles both scrolling up and down.
+                  // --- THE CORE LOGIC YOU REQUESTED ---
+                  // Is the user just wiggling the scrollbar? If so, exit.
+                  // This is a very cheap check that runs on every scroll event.
                   if (
-                    Math.abs(scrollTop - lastScrollTopRef.current) <
+                    Math.abs(scrollTop - lastUpdateAtScrollTop.current) <
                     scrollThreshold
                   ) {
-                    // Not scrolled far enough, do nothing.
                     return;
                   }
 
-                  // If we've passed the threshold, update our reference for the next check.
-                  lastScrollTopRef.current = scrollTop;
+                  // --- IF WE ARE HERE, WE HAVE SCROLLED A "DECENT AMOUNT" ---
 
-                  console.log("Threshold passed: Updating virtual range."); // For debugging
+                  console.log(
+                    `Threshold passed at ${scrollTop}px. Recalculating range...`
+                  );
 
-                  // --- Part 3: Run the expensive calculation ---
-                  // This code now only runs when it's actually needed.
+                  // NOW we do the expensive work.
                   const { clientHeight } = container;
-                  let low = 0,
-                    high = totalCount - 1;
+                  let high = totalCount - 1;
+                  let low = 0;
+                  let topItemIndex = 0;
                   while (low <= high) {
                     const mid = Math.floor((low + high) / 2);
-                    if (positions[mid]! < scrollTop) low = mid + 1;
-                    else high = mid - 1;
+                    if (positions[mid]! < scrollTop) {
+                      topItemIndex = mid;
+                      low = mid + 1;
+                    } else {
+                      high = mid - 1;
+                    }
                   }
-                  const startIndex = Math.max(0, high - overscan);
+
+                  const startIndex = Math.max(0, topItemIndex - overscan);
                   let endIndex = startIndex;
                   const visibleEnd = scrollTop + clientHeight;
                   while (
@@ -2074,10 +2051,14 @@ function createProxyHandler<T>(
                   ) {
                     endIndex++;
                   }
+
                   setRange({
                     startIndex,
                     endIndex: Math.min(totalCount, endIndex + overscan),
                   });
+
+                  // Finally, we record that we did the work at THIS scroll position.
+                  lastUpdateAtScrollTop.current = scrollTop;
                 };
 
                 container.addEventListener("scroll", handleUserScroll, {
@@ -2085,7 +2066,7 @@ function createProxyHandler<T>(
                 });
                 return () =>
                   container.removeEventListener("scroll", handleUserScroll);
-              }, [totalCount, positions, status, itemHeight]); // Added itemHeight to deps
+              }, [totalCount, positions, itemHeight, overscan, status]);
 
               const scrollToBottom = useCallback(() => {
                 console.log(