chore(file-viewer): final comment cleanup before merge

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/files/components/file-viewer/text-editor-state.test.ts

@@ -339,21 +339,12 @@ describe('syncTextEditorContentState — streaming finalize shortcuts', () => {
 })
 
 describe('syncTextEditorContentState — inter-session content shrink (replace mode)', () => {
-  // These tests cover the path that causes scroll jumps in the file viewer.
-  // During "session linger" (session 1 complete, session 2 not yet started),
-  // the state machine stays in 'streaming' phase with session 1's full content.
-  // When session 2's first chunk arrives it is often SHORTER than session 1's
-  // final content. The state machine must correctly replace the content rather
-  // than treating it as already-final or dropping it.
-
   it('replaces long linger content with a short first chunk from a new session', () => {
-    // State during linger: streaming with session 1's full long content
     const lingerState = streaming(
       'a very long document with many paragraphs',
       'a very long document with many paragraphs',
       ''
     )
-    // Session 2 first chunk arrives — much shorter than session 1's content
     const next = syncTextEditorContentState(lingerState, {
       canReconcileToFetchedContent: false,
       fetchedContent: undefined,
@@ -382,8 +373,6 @@ describe('syncTextEditorContentState — inter-session content shrink (replace m
   })
 
   it('does not finalize early when the new short chunk happens to equal savedContent', () => {
-    // savedContent '' and streamingContent '' — but we are in streaming phase,
-    // not ready, so content should remain in streaming phase
     const lingerState = streaming('long content', 'long content', 'old saved')
     const next = syncTextEditorContentState(lingerState, {
       canReconcileToFetchedContent: false,
@@ -427,8 +416,6 @@ describe('syncTextEditorContentState — inter-session content shrink (replace m
   })
 
   it('synthetic file (canReconcile=false) finalizes with current content when streaming ends', () => {
-    // After the new session finishes streaming, since there is no fetchedContent
-    // to reconcile with (synthetic file has no db key), it must finalize immediately.
     const finalChunk = streaming(
       '# Complete Document\n\nAll done.',
       '# Complete Document\n\nAll done.',

apps/sim/app/workspace/[workspaceId]/home/hooks/use-file-preview-sessions.test.tsx

@@ -397,7 +397,6 @@ describe('reduceFilePreviewSessions', () => {
         previewText: 'final',
       }),
     })
-    // activeSessionId is lingering on preview-1 after complete
 
     const reset = reduceFilePreviewSessions(state, { type: 'reset' })
 
@@ -417,7 +416,6 @@ describe('reduceFilePreviewSessions', () => {
   })
 
   it('hydrate merges incoming sessions into existing state without replacing non-stale sessions', () => {
-    // Existing state has preview-1 at version 3
     const existing = reduceFilePreviewSessions(INITIAL_FILE_PREVIEW_SESSIONS_STATE, {
       type: 'upsert',
       session: createSession({
@@ -429,7 +427,6 @@ describe('reduceFilePreviewSessions', () => {
       }),
     })
 
-    // Hydrate with preview-1 at older version 2 + a new preview-2
     const hydrated = reduceFilePreviewSessions(existing, {
       type: 'hydrate',
       sessions: [
@@ -450,10 +447,8 @@ describe('reduceFilePreviewSessions', () => {
       ],
     })
 
-    // preview-1 kept at version 3 (not replaced by stale version 2)
     expect(hydrated.sessions['preview-1']?.previewVersion).toBe(3)
     expect(hydrated.sessions['preview-1']?.previewText).toBe('current')
-    // preview-2 added
     expect(hydrated.sessions['preview-2']?.previewText).toBe('new')
   })
 
@@ -553,8 +548,6 @@ describe('reduceFilePreviewSessions', () => {
         previewText: 'background',
       }),
     })
-    // active is now preview-2 (later upsert); preview-1 is no longer active
-    // Complete preview-1 (the non-active session)
     const completed = reduceFilePreviewSessions(state, {
       type: 'complete',
       session: createSession({

apps/sim/hooks/use-auto-scroll.ts

@@ -49,9 +49,7 @@ export function useAutoScroll(
     const el = containerRef.current
     if (!el) return
 
-    // Only pin to bottom if the user is already near the bottom (or the
-    // container has no scrollable content yet). If they've scrolled up to
-    // read, keep their position — they can scroll down to re-engage.
+    // Don't jump if the user scrolled up — keep their position.
     const distanceFromBottom = el.scrollHeight - el.scrollTop - el.clientHeight
     const isNearBottom = distanceFromBottom <= STICK_THRESHOLD
     stickyRef.current = isNearBottom