Fix git status pipeline issues (#4)

* Fix git status pipeline issues

- amux-0em: Avoid duplicate git fetch on cache miss by adding UpdateCache()
  method to StatusManager and using it instead of RequestRefresh()
- amux-206: Rebuild dashboard rows when GitStatusResult arrives and dirty
  filter is active, so filtered view stays accurate
- amux-pkp: Fix sidebar refresh to use messages.GitStatusResult instead of
  unhandled local StatusRefreshed type
- amux-8ix: Activate FileWatcher by wiring callback to emit events via
  channel and implementing startFileWatcher() to listen

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Fix gofmt formatting issues

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>

Author: andyrewlee

internal/app/app.go

@@ -69,6 +69,7 @@ type App struct {
 	// Git status management
 	statusManager *git.StatusManager
 	fileWatcher   *git.FileWatcher
+	fileWatcherCh chan messages.FileWatcherEvent
 
 	// Layout
 	width, height int
@@ -100,8 +101,17 @@ func New() (*App, error) {
 	// Create status manager (callback will be nil, we use it for caching only)
 	statusManager := git.NewStatusManager(nil)
 
-	// Create file watcher (may fail, that's ok)
-	fileWatcher, _ := git.NewFileWatcher(nil)
+	// Create file watcher event channel
+	fileWatcherCh := make(chan messages.FileWatcherEvent, 10)
+
+	// Create file watcher with callback that sends to channel
+	fileWatcher, _ := git.NewFileWatcher(func(root string) {
+		select {
+		case fileWatcherCh <- messages.FileWatcherEvent{Root: root}:
+		default:
+			// Channel full, drop event (will catch on next change)
+		}
+	})
 
 	return &App{
 		config:        cfg,
@@ -110,6 +120,7 @@ func New() (*App, error) {
 		scripts:       scripts,
 		statusManager: statusManager,
 		fileWatcher:   fileWatcher,
+		fileWatcherCh: fileWatcherCh,
 		layout:        layout.NewManager(),
 		dashboard:     dashboard.New(),
 		center:        center.New(cfg),
@@ -145,10 +156,12 @@ func (a *App) startGitStatusTicker() tea.Cmd {
 
 // startFileWatcher starts watching for file changes and returns events
 func (a *App) startFileWatcher() tea.Cmd {
-	if a.fileWatcher == nil {
+	if a.fileWatcher == nil || a.fileWatcherCh == nil {
 		return nil
 	}
-	return nil // File watcher runs in background, we'll handle it differently
+	return func() tea.Msg {
+		return <-a.fileWatcherCh
+	}
 }
 
 // Update handles all messages
@@ -441,6 +454,8 @@ func (a *App) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 		// File changed, invalidate cache and refresh
 		a.statusManager.Invalidate(msg.Root)
 		cmds = append(cmds, a.requestGitStatus(msg.Root))
+		// Continue listening for file changes
+		cmds = append(cmds, a.startFileWatcher())
 
 	case messages.Error:
 		a.err = msg.Err
@@ -801,9 +816,9 @@ func (a *App) loadProjects() tea.Cmd {
 func (a *App) requestGitStatus(root string) tea.Cmd {
 	return func() tea.Msg {
 		status, err := git.GetStatus(root)
-		// Update cache
+		// Update cache directly (no async refresh needed, we just fetched)
 		if a.statusManager != nil && err == nil {
-			a.statusManager.RequestRefresh(root)
+			a.statusManager.UpdateCache(root, status)
 		}
 		return messages.GitStatusResult{
 			Root:   root,

internal/git/status_manager.go

@@ -136,6 +136,16 @@ func (m *StatusManager) Invalidate(root string) {
 	delete(m.cache, root)
 }
 
+// UpdateCache directly updates the cache with a status result (no fetch)
+func (m *StatusManager) UpdateCache(root string, status *StatusResult) {
+	m.mu.Lock()
+	defer m.mu.Unlock()
+	m.cache[root] = &StatusCache{
+		Status:    status,
+		FetchedAt: time.Now(),
+	}
+}
+
 // InvalidateAll clears the entire cache
 func (m *StatusManager) InvalidateAll() {
 	m.mu.Lock()

internal/ui/dashboard/model.go

@@ -149,6 +149,10 @@ func (m *Model) Update(msg tea.Msg) (*Model, tea.Cmd) {
 		if msg.Err == nil {
 			m.statusCache[msg.Root] = msg.Status
 		}
+		// Rebuild rows if dirty filter is active (status change may affect visibility)
+		if m.filterDirty {
+			m.rebuildRows()
+		}
 
 	case messages.WorktreeActivated:
 		if msg.Worktree != nil {

internal/ui/sidebar/model.go

@@ -13,6 +13,7 @@ import (
 
 	"github.com/andyrewlee/amux/internal/data"
 	"github.com/andyrewlee/amux/internal/git"
+	"github.com/andyrewlee/amux/internal/messages"
 	"github.com/andyrewlee/amux/internal/ui/common"
 )
 
@@ -96,9 +97,6 @@ func (m *Model) Update(msg tea.Msg) (*Model, tea.Cmd) {
 		case key.Matches(msg, key.NewBinding(key.WithKeys("g"))):
 			cmds = append(cmds, m.refreshStatus())
 		}
-
-	case git.StatusResult:
-		// This would come from a refresh command
 	}
 
 	return m, tea.Batch(cmds...)
@@ -428,16 +426,11 @@ func (m *Model) refreshStatus() tea.Cmd {
 
 	root := m.worktree.Root
 	return func() tea.Msg {
-		status, _ := git.GetStatus(root)
-		return StatusRefreshed{Status: status}
+		status, err := git.GetStatus(root)
+		return messages.GitStatusResult{Root: root, Status: status, Err: err}
 	}
 }
 
-// StatusRefreshed is sent when status is refreshed
-type StatusRefreshed struct {
-	Status *git.StatusResult
-}
-
 // SetSize sets the sidebar size
 func (m *Model) SetSize(width, height int) {
 	m.width = width