Fix scroll behavior for programmatic updates

brandonkachen · codebuff-team · brandonkachen · commit 55f4baa896c3 · 2025-10-10T11:08:46.000-07:00
Set programmaticScrollRef flag when adjusting scroll position to prevent auto-scroll from fighting user intent. Also use direct scrollTop assignment instead of scrollToLatest() for consistency.

🤖 Generated with Codebuff
Co-Authored-By: Codebuff &lt;noreply@codebuff.com&gt;
diff --git a/cli/src/hooks/use-scroll-management.ts b/cli/src/hooks/use-scroll-management.ts
@@ -160,9 +160,11 @@ export const useChatScrollbox = (
         )
 
         if (scrollbox.scrollTop > maxScroll) {
+          programmaticScrollRef.current = true
           scrollbox.scrollTop = maxScroll
         } else if (autoScrollEnabledRef.current) {
-          scrollToLatest()
+          programmaticScrollRef.current = true
+          scrollbox.scrollTop = maxScroll
         }
       }, 50)
 

Original file line number	Diff line number	Diff line change
`@@ -160,9 +160,11 @@ export const useChatScrollbox = (`
`160`	`160`	`)`
`161`	`161`
`162`	`162`	`if (scrollbox.scrollTop > maxScroll) {`
	`163`	`+ programmaticScrollRef.current = true`
`163`	`164`	`scrollbox.scrollTop = maxScroll`
`164`	`165`	`} else if (autoScrollEnabledRef.current) {`
`165`		`- scrollToLatest()`
	`166`	`+ programmaticScrollRef.current = true`
	`167`	`+ scrollbox.scrollTop = maxScroll`
`166`	`168`	`}`
`167`	`169`	`}, 50)`
`168`	`170`