Fix git add regression: use git reset instead of pathspec exclusion (#93)

The :!node_modules pathspec causes errors on repos that gitignore
node_modules. Instead, stage everything then unstage node_modules
if it got picked up from the worktree symlink.

Co-authored-by: unknown <that-github-user@github.com>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: 3 people

src/utils/git.ts

@@ -78,7 +78,9 @@ export async function getDiff(worktreePath: string): Promise<string> {
   try {
     // Include both staged and unstaged changes relative to HEAD
     // Exclude node_modules symlink (created by createWorktree for tool access)
-    await exec("git", ["add", "-A", "--", ".", ":!node_modules"], { cwd: worktreePath });
+    await exec("git", ["add", "-A"], { cwd: worktreePath });
+    // Unstage node_modules symlink if it got picked up (created by createWorktree)
+    await exec("git", ["reset", "HEAD", "--", "node_modules"], { cwd: worktreePath }).catch(() => {});
     const { stdout } = await exec("git", ["diff", "--cached", "HEAD"], {
       cwd: worktreePath,
     });
@@ -95,7 +97,9 @@ export async function getDiffStats(
   worktreePath: string,
 ): Promise<{ filesChanged: string[]; linesAdded: number; linesRemoved: number }> {
   try {
-    await exec("git", ["add", "-A", "--", ".", ":!node_modules"], { cwd: worktreePath });
+    await exec("git", ["add", "-A"], { cwd: worktreePath });
+    // Unstage node_modules symlink if it got picked up (created by createWorktree)
+    await exec("git", ["reset", "HEAD", "--", "node_modules"], { cwd: worktreePath }).catch(() => {});
     const { stdout } = await exec("git", ["diff", "--cached", "--stat", "HEAD"], {
       cwd: worktreePath,
     });