Revert "Merge pull request #76 from moazbuilds/fix/log-scroll-and-dublication"

This reverts commit b56d157, reversing
changes made to 1e1f816.

Author: moazbuilds

src/cli/tui/routes/workflow/components/output/output-window.tsx

@@ -111,17 +111,10 @@ export function OutputWindow(props: OutputWindowProps) {
         // Trigger load when near the top (within 3 lines) - skip if already loading
         if (scrollTop <= 3 && props.hasMoreAbove && props.onLoadMore && !props.isLoadingEarlier) {
           debug('[OutputWindow] Loading earlier lines...')
-          const scrollHeightBefore = ref.scrollHeight
           const linesLoaded = props.onLoadMore()
           debug('[OutputWindow] Lines loaded: %d', linesLoaded)
           if (linesLoaded > 0) {
-            // SolidJS updates DOM synchronously, so we can measure and adjust immediately
-            const scrollHeightAfter = ref.scrollHeight
-            const heightAdded = scrollHeightAfter - scrollHeightBefore
-            // Maintain view position with small slack to prevent immediate re-trigger
-            const slack = 5
-            ref.scrollTop = scrollTop + heightAdded + slack
-            debug('[OutputWindow] Scroll adjusted: heightAdded=%d, slack=%d, newScrollTop=%d', heightAdded, slack, ref.scrollTop)
+            ref.scrollTop = linesLoaded  // Maintain view position
           }
         }
       }

src/cli/tui/routes/workflow/hooks/useLogStream.ts

@@ -109,8 +109,7 @@ function readLogFileIncremental(
     if (currentSize < state.lastFileSize) {
       logDebug('File truncated, doing full read. Old size: %d, new size: %d', state.lastFileSize, currentSize)
       const content = readFileSync(path, "utf-8")
-      // Filter empty lines to match incremental read behavior
-      const lines = content.split("\n").filter(line => line !== '')
+      const lines = content.split("\n")
       state.lastFileSize = currentSize
       state.lastLineCount = lines.length
       return {
@@ -124,9 +123,8 @@ function readLogFileIncremental(
     // For small files, just do a full read (simpler and fast enough)
     if (currentSize < INCREMENTAL_THRESHOLD_BYTES) {
       const content = readFileSync(path, "utf-8")
-      // Filter empty lines to match incremental read behavior
-      const lines = content.split("\n").filter(line => line !== '')
-      const newLinesCount = Math.max(0, lines.length - state.lastLineCount)
+      const lines = content.split("\n")
+      const newLinesCount = lines.length - state.lastLineCount
       state.lastFileSize = currentSize
       state.lastLineCount = lines.length
       return {
@@ -182,15 +180,14 @@ function readLogFileIncremental(
 
 /**
  * Read entire log file (used for initial load or after truncation)
- * Filters empty lines to match incremental read behavior
  */
 function readLogFileFull(path: string): string[] {
   try {
     if (!existsSync(path)) {
       return []
     }
     const content = readFileSync(path, "utf-8")
-    return content.split("\n").filter(line => line !== '')
+    return content.split("\n")
   } catch {
     return []
   }
@@ -502,9 +499,8 @@ export function useLogStream(
           const currentFileSize = getFileSize(logPath)
 
           // Update incremental state
-          // IMPORTANT: Track RAW line count, not filtered, to match readLogFileIncremental
           incrementalState.lastFileSize = currentFileSize
-          incrementalState.lastLineCount = fileLines.length
+          incrementalState.lastLineCount = filteredLines.length
 
           // Update windowed state
           const trimCount = Math.max(0, filteredLines.length - maxWindow)