Improve sidebar refresh progress bar and reduce session scan overhead

Replace broken GeometryReader+onAppear indeterminate loader with
TimelineView+Canvas that reliably animates back and forth. Remove
intermediate session publishing during refresh (was firing N partial
UI updates every 25 sessions), now publishes once at the end.

Author: nicosuave

macos/Sources/Features/Worktrunk/WorktrunkSidebarView.swift

@@ -838,38 +838,24 @@ private struct RepoPickerPopover: View {
 }
 
 private struct SidebarRefreshProgressBar: View {
-    @State private var position: CGFloat = 0
-
-    private let barWidthRatio: CGFloat = 0.25
-
     var body: some View {
-        GeometryReader { geometry in
-            ZStack(alignment: .leading) {
-                Rectangle()
-                    .fill(Color.accentColor.opacity(0.3))
-
-                Rectangle()
-                    .fill(Color.accentColor)
-                    .frame(
-                        width: geometry.size.width * barWidthRatio,
-                        height: geometry.size.height
-                    )
-                    .offset(x: position * (geometry.size.width * (1 - barWidthRatio)))
+        TimelineView(.animation) { timeline in
+            let t = timeline.date.timeIntervalSinceReferenceDate
+            let phase = (1 + sin(t * .pi)) / 2
+            Canvas { context, size in
+                context.fill(
+                    Path(CGRect(origin: .zero, size: size)),
+                    with: .color(.accentColor.opacity(0.15))
+                )
+                let barWidth = size.width * 0.25
+                let x = phase * (size.width - barWidth)
+                context.fill(
+                    Path(CGRect(x: x, y: 0, width: barWidth, height: size.height)),
+                    with: .color(.accentColor)
+                )
             }
         }
         .frame(height: 3)
-        .clipped()
         .allowsHitTesting(false)
-        .onAppear {
-            withAnimation(
-                .easeInOut(duration: 1.2)
-                .repeatForever(autoreverses: true)
-            ) {
-                position = 1
-            }
-        }
-        .onDisappear {
-            position = 0
-        }
     }
 }

macos/Sources/Features/Worktrunk/WorktrunkStore.swift

@@ -1377,11 +1377,7 @@ final class WorktrunkStore: ObservableObject {
 
     func refreshSessions() async {
         var allSessions: [String: [AISession]] = [:]
-        var updateCount = 0
         var dirtyWorktreePaths = Set<String>()
-        let publishCheckBatch = 25
-        let publishIntervalSeconds: TimeInterval = 0.2
-        var lastPublish = Date.distantPast
         let validWorktreePaths = sidebarWorktreePaths
         var index = sessionIndex.snapshot()
         var seenClaudePaths = Set<String>()
@@ -1401,36 +1397,17 @@ final class WorktrunkStore: ObservableObject {
             }
         }
 
-        func prepareForPublish() {
+        func sortDirtySessions() {
             guard !dirtyWorktreePaths.isEmpty else { return }
             for path in dirtyWorktreePaths {
                 allSessions[path]?.sort { $0.timestamp > $1.timestamp }
             }
             dirtyWorktreePaths.removeAll(keepingCapacity: true)
         }
 
-        func publishSnapshotIfNeeded(force: Bool = false) async {
-            let now = Date()
-            if !force, now.timeIntervalSince(lastPublish) < publishIntervalSeconds {
-                return
-            }
-            lastPublish = now
-            prepareForPublish()
-            let snapshot = allSessions
-            await MainActor.run {
-                if sessionsByWorktreePath != snapshot {
-                    sessionsByWorktreePath = snapshot
-                }
-            }
-        }
-
-        func noteSession(_ session: AISession, worktreePath: String) async {
+        func noteSession(_ session: AISession, worktreePath: String) {
             allSessions[worktreePath, default: []].append(session)
             dirtyWorktreePaths.insert(worktreePath)
-            updateCount += 1
-            if updateCount % publishCheckBatch == 0 {
-                await publishSnapshotIfNeeded()
-            }
         }
 
         // Scan Claude sessions with periodic UI updates
@@ -1475,7 +1452,7 @@ final class WorktrunkStore: ObservableObject {
                                 sourcePath: sessionFile.path,
                                 messageCount: cached.messageCount
                             )
-                            await noteSession(session, worktreePath: worktreePath)
+                            noteSession(session, worktreePath: worktreePath)
                             if cached.worktreePath != worktreePath {
                                 var updated = cached
                                 updated.worktreePath = worktreePath
@@ -1496,7 +1473,7 @@ final class WorktrunkStore: ObservableObject {
                             validWorktreePaths: validWorktreePaths
                         ) {
                             session.worktreePath = worktreePath
-                            await noteSession(session, worktreePath: worktreePath)
+                            noteSession(session, worktreePath: worktreePath)
                             index.claude[sessionFile.path] = SessionIndexEntry(
                                 sessionId: session.id,
                                 cwd: session.cwd,
@@ -1557,7 +1534,7 @@ final class WorktrunkStore: ObservableObject {
                             sourcePath: fileURL.path,
                             messageCount: cached.messageCount
                         )
-                        await noteSession(session, worktreePath: worktreePath)
+                        noteSession(session, worktreePath: worktreePath)
                         if cached.worktreePath != worktreePath {
                             var updated = cached
                             updated.worktreePath = worktreePath
@@ -1574,7 +1551,7 @@ final class WorktrunkStore: ObservableObject {
                         validWorktreePaths: validWorktreePaths
                     ) {
                         session.worktreePath = worktreePath
-                        await noteSession(session, worktreePath: worktreePath)
+                        noteSession(session, worktreePath: worktreePath)
                         index.codex[fileURL.path] = SessionIndexEntry(
                             sessionId: session.id,
                             cwd: session.cwd,
@@ -1678,7 +1655,7 @@ final class WorktrunkStore: ObservableObject {
                                 sourcePath: infoFile.path,
                                 messageCount: cached.messageCount
                             )
-                            await noteSession(session, worktreePath: worktreePath)
+                            noteSession(session, worktreePath: worktreePath)
                         }
                         continue
                     }
@@ -1733,7 +1710,7 @@ final class WorktrunkStore: ObservableObject {
                             sourcePath: infoFile.path,
                             messageCount: messageCount
                         )
-                        await noteSession(session, worktreePath: worktreePath)
+                        noteSession(session, worktreePath: worktreePath)
                     }
                 }
             }
@@ -1743,8 +1720,8 @@ final class WorktrunkStore: ObservableObject {
 
         index.opencode = index.opencode.filter { seenOpenCodePaths.contains($0.key) }
 
-        // Final sort and update
-        prepareForPublish()
+        // Final sort and single publish
+        sortDirtySessions()
 
         sessionCache.saveToDisk()
         sessionIndex.update(index)