🤖 Fix git unit test failures by isolating mock config per test

The tests were failing because a shared global workspace directory caused
workspace collisions and leftover state from previous test runs.

Root cause:
- mockConfig used a fixed srcDir path shared across all tests
- Multiple tests created workspaces with the same names
- Failed runs left directories behind causing 'Workspace already exists' errors

Solution:
- Created createMockConfig(tempDir) helper to generate unique configs per test
- Each test now gets its own isolated workspace directory
- Removed debug logging

All 807 tests now pass.

Author: ammar-agent

src/services/gitService.test.ts

@@ -23,21 +23,25 @@ async function createTestRepo(basePath: string): Promise<string> {
   return repoPath;
 }
 
-// Mock config for createWorktree
-const mockConfig = {
-  srcDir: path.join(__dirname, "..", "test-workspaces"),
-  getWorkspacePath: (projectPath: string, branchName: string) => {
-    return path.join(path.dirname(projectPath), "workspaces", branchName);
-  },
-} as unknown as Config;
+// Helper to create mock config with unique temp directory
+function createMockConfig(tempDir: string): Config {
+  return {
+    srcDir: path.join(tempDir, "test-workspaces"),
+    getWorkspacePath: (projectPath: string, branchName: string) => {
+      return path.join(path.dirname(projectPath), "workspaces", branchName);
+    },
+  } as unknown as Config;
+}
 
 describe("removeWorktreeSafe", () => {
   let tempDir: string;
   let repoPath: string;
   let defaultBranch: string;
+  let mockConfig: Config;
 
   beforeEach(async () => {
     tempDir = await fs.mkdtemp(path.join(__dirname, "..", "test-temp-"));
+    mockConfig = createMockConfig(tempDir);
     repoPath = await createTestRepo(tempDir);
     defaultBranch = await detectDefaultTrunkBranch(repoPath);
   });
@@ -55,9 +59,6 @@ describe("removeWorktreeSafe", () => {
     const result = await createWorktree(mockConfig, repoPath, "test-branch", {
       trunkBranch: defaultBranch,
     });
-    if (!result.success) {
-      console.error("createWorktree failed:", result.error);
-    }
     expect(result.success).toBe(true);
     const worktreePath = result.path!;
 
@@ -186,9 +187,11 @@ describe("isWorktreeClean", () => {
   let tempDir: string;
   let repoPath: string;
   let defaultBranch: string;
+  let mockConfig: Config;
 
   beforeEach(async () => {
     tempDir = await fs.mkdtemp(path.join(__dirname, "..", "test-temp-"));
+    mockConfig = createMockConfig(tempDir);
     repoPath = await createTestRepo(tempDir);
     defaultBranch = await detectDefaultTrunkBranch(repoPath);
   });