worktree: faster refresh + handle new staged files

Author: bohutang

packages/desktop/src/features/git/DiffManager.ts

@@ -332,6 +332,7 @@ export class GitDiffManager {
       argv: ['git', 'cat-file', '-e', object],
       op: 'read',
       recordTimeline: false,
+      throwOnError: false,
       meta: { source: 'gitDiff', operation: 'cat-file-exists', object },
       timeoutMs: 15_000,
     });

packages/desktop/src/features/git/__tests__/DiffManager.newFile.test.ts

@@ -0,0 +1,45 @@
+import { describe, it, expect, vi } from 'vitest';
+import { GitDiffManager } from '../DiffManager';
+
+describe('GitDiffManager - new file detection', () => {
+  it('does not throw when HEAD:path is missing for staged added file', async () => {
+    const gitExecutor = {
+      run: vi.fn(),
+    } as any;
+
+    const manager = new GitDiffManager(gitExecutor);
+
+    // status-groups (porcelain) => staged added file
+    vi.mocked(gitExecutor.run).mockResolvedValueOnce({
+      exitCode: 0,
+      stdout: 'A  dd.txt\n',
+      stderr: '',
+    });
+
+    // numstat-staged
+    vi.mocked(gitExecutor.run).mockResolvedValueOnce({
+      exitCode: 0,
+      stdout: '1\t0\tdd.txt\n',
+      stderr: '',
+    });
+
+    // numstat-unstaged
+    vi.mocked(gitExecutor.run).mockResolvedValueOnce({
+      exitCode: 0,
+      stdout: '',
+      stderr: '',
+    });
+
+    // cat-file-exists HEAD:dd.txt should fail but must not throw
+    vi.mocked(gitExecutor.run).mockResolvedValueOnce({
+      exitCode: 128,
+      stdout: '',
+      stderr: "fatal: path 'dd.txt' exists on disk, but not in 'HEAD'\n",
+    });
+
+    const groups = await manager.getWorkingTreeGroups('/repo', 's1');
+    expect(groups.staged).toHaveLength(1);
+    expect(groups.staged[0]).toMatchObject({ path: 'dd.txt', type: 'added', isNew: true });
+  });
+});
+

packages/ui/src/components/layout/RightPanel.tsx

@@ -700,7 +700,7 @@ export const RightPanel: React.FC<RightPanelProps> = React.memo(({
       if (selectedIsUncommitted) {
         fetchFiles();
       }
-    }, 650);
+    }, 200);
   }, [session.id, fetchCommits, fetchFiles, selectedIsUncommitted]);
 
   // Refresh once after a run completes (running -> waiting/error/etc).
@@ -720,8 +720,12 @@ export const RightPanel: React.FC<RightPanelProps> = React.memo(({
   // Only refresh on stable, state-changing signals. `state`/`lastChecked` can churn even when nothing
   // meaningful changed (e.g. transient errors or polling), which would cause refresh loops.
   const gitSig = [
+    String(session.gitStatus?.state ?? ''),
     String(Number(session.gitStatus?.ahead ?? 0)),
     String(Number(session.gitStatus?.behind ?? 0)),
+    String(Number(session.gitStatus?.filesChanged ?? 0)),
+    String(Number(session.gitStatus?.additions ?? 0)),
+    String(Number(session.gitStatus?.deletions ?? 0)),
     String(Boolean(session.gitStatus?.hasUncommittedChanges)),
     String(Boolean(session.gitStatus?.hasUntrackedFiles)),
   ].join('|');