cogsbox-state 0.5.347 → 0.5.350

package/package.json

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

package/src/CogsState.tsx

@@ -1814,6 +1814,7 @@ function createProxyHandler<T>(
                 endIndex: 10,
               });
               const isLockedToBottomRef = useRef(stickToBottom);
+              const prevTotalCountRef = useRef(0);
 
               const [shadowUpdateTrigger, setShadowUpdateTrigger] = useState(0);
 
@@ -1867,31 +1868,28 @@ function createProxyHandler<T>(
                 });
               }, [range.startIndex, range.endIndex, sourceArray, totalCount]);
 
-              // --- YOUR SCROLLING ALGORITHM (UNCHANGED and WORKING) ---
-              // This effect is the entry point. It triggers when new items are added.
+              // --- YOUR WORKING SCROLL ALGORITHM - UNTOUCHED ---
               useLayoutEffect(() => {
                 const container = containerRef.current;
-                // Only run if we are supposed to be at the bottom.
+                const hasNewItems = totalCount > prevTotalCountRef.current;
+
                 if (
                   !container ||
+                  !stickToBottom ||
                   !isLockedToBottomRef.current ||
-                  totalCount === 0
+                  !hasNewItems
                 ) {
                   return;
                 }
 
-                // STEP 1: Set the range to the end so the last items are rendered.
-                const visibleCount = 10;
+                // STEP 1: Set range to the end to render the items we need to measure.
                 setRange({
-                  startIndex: Math.max(0, totalCount - visibleCount - overscan),
+                  startIndex: Math.max(0, totalCount - 10 - overscan),
                   endIndex: totalCount,
                 });
 
-                // STEP 2: Start the LOOP.
-                let loopCount = 0;
+                // STEP 2: The LOOP.
                 const intervalId = setInterval(() => {
-                  loopCount++;
-                  // The Check: Get the last item's height FROM THE SHADOW OBJECT.
                   const lastItemIndex = totalCount - 1;
                   const shadowArray =
                     getGlobalStore
@@ -1901,33 +1899,24 @@ function createProxyHandler<T>(
                     shadowArray[lastItemIndex]?.virtualizer?.itemHeight || 0;
 
                   if (lastItemHeight > 0) {
-                    // EXIT CONDITION MET
-                    clearInterval(intervalId); // Stop the loop.
-
+                    clearInterval(intervalId);
                     // STEP 3: Scroll.
                     container.scrollTo({
                       top: container.scrollHeight,
                       behavior: "smooth",
                     });
-                  } else {
-                    if (loopCount > 20) {
-                      // Safety break
-                      clearInterval(intervalId);
-                    }
                   }
                 }, 100);
 
-                // Cleanup: Stop the loop if the component unmounts.
                 return () => clearInterval(intervalId);
-              }, [totalCount]); // This whole process triggers ONLY when totalCount changes.
+              }, [totalCount]);
 
               // --- THE FIX IS HERE ---
-              // This effect now correctly handles user scrolling AND updates the view.
+              // This effect now correctly updates when data changes.
               useEffect(() => {
                 const container = containerRef.current;
                 if (!container) return;
 
-                // This function now always has the LATEST totalCount and positions.
                 const updateVirtualRange = () => {
                   const { scrollTop, clientHeight } = container;
                   let low = 0,
@@ -1961,24 +1950,28 @@ function createProxyHandler<T>(
                   if (!isAtBottom) {
                     isLockedToBottomRef.current = false;
                   }
-                  // This always calls the fresh version of updateVirtualRange.
                   updateVirtualRange();
                 };
 
                 container.addEventListener("scroll", handleUserScroll, {
                   passive: true,
                 });
-                updateVirtualRange(); // Update range on initial load and when data changes.
+                updateVirtualRange(); // Always update range on render.
 
-                // This cleanup is crucial. It removes the old listener before adding a new one.
                 return () =>
                   container.removeEventListener("scroll", handleUserScroll);
-              }, [totalCount, positions]); // Its dependency array now includes totalCount and positions.
+              }, [totalCount, positions]); // FIX: This now has dependencies.
 
+              // Simple effect to track previous item count for the scroll algorithm.
+              useEffect(() => {
+                prevTotalCountRef.current = totalCount;
+              });
               const scrollToBottom = useCallback(
                 (behavior: ScrollBehavior = "smooth") => {
                   if (containerRef.current) {
                     isLockedToBottomRef.current = true;
+                    console.log("USER ACTION: Scroll lock ENABLED.");
+                    // This is a manual trigger, so we don't need the loop. Just scroll.
                     containerRef.current.scrollTo({
                       top: containerRef.current.scrollHeight,
                       behavior,
@@ -1992,6 +1985,7 @@ function createProxyHandler<T>(
                 (index: number, behavior: ScrollBehavior = "smooth") => {
                   if (containerRef.current && positions[index] !== undefined) {
                     isLockedToBottomRef.current = false;
+                    console.log("USER ACTION: Scroll lock DISABLED.");
                     containerRef.current.scrollTo({
                       top: positions[index],
                       behavior,
@@ -2018,6 +2012,7 @@ function createProxyHandler<T>(
                   },
                 },
               };
+
               return {
                 virtualState,
                 virtualizerProps,