🤖 Fix: Remove config bypassing in renameWorkspace tests

The renameWorkspace tests were bypassing IPC by manually manipulating config,
creating duplicate workspace entries. This caused rename operations to fail in CI.

Fixed by removing all manual config.saveConfig() calls - setupWorkspace() already
creates workspaces via WORKSPACE_CREATE IPC, which properly adds them to config.

_Generated with `cmux`_

Author: kylecarbs

tests/ipcMain/renameWorkspace.test.ts

@@ -26,19 +26,7 @@ describeIntegration("IpcMain rename workspace integration tests", () => {
       const { env, workspaceId, workspacePath, tempGitRepo, branchName, cleanup } =
         await setupWorkspace("anthropic");
       try {
-        // Add project and workspace to config via IPC
-        await env.mockIpcRenderer.invoke(IPC_CHANNELS.PROJECT_CREATE, tempGitRepo);
-        // Manually add workspace to the project (normally done by WORKSPACE_CREATE)
-        const projectsConfig = env.config.loadConfigOrDefault();
-        const projectConfig = projectsConfig.projects.get(tempGitRepo);
-        if (projectConfig) {
-          projectConfig.workspaces.push({
-            path: workspacePath,
-            id: workspaceId,
-            name: branchName,
-          });
-          env.config.saveConfig(projectsConfig);
-        }
+        // setupWorkspace already creates the workspace and adds it to config via IPC
         const oldSessionDir = env.config.getSessionDir(workspaceId);
         const oldMetadataResult = await env.mockIpcRenderer.invoke(
           IPC_CHANNELS.WORKSPACE_GET_INFO,
@@ -167,19 +155,7 @@ describeIntegration("IpcMain rename workspace integration tests", () => {
       const { env, workspaceId, workspacePath, tempGitRepo, branchName, cleanup } =
         await setupWorkspace("anthropic");
       try {
-        // Add project and workspace to config via IPC
-        await env.mockIpcRenderer.invoke(IPC_CHANNELS.PROJECT_CREATE, tempGitRepo);
-        // Manually add workspace to the project (normally done by WORKSPACE_CREATE)
-        const projectsConfig = env.config.loadConfigOrDefault();
-        const projectConfig = projectsConfig.projects.get(tempGitRepo);
-        if (projectConfig) {
-          projectConfig.workspaces.push({
-            path: workspacePath,
-            id: workspaceId,
-            name: branchName,
-          });
-          env.config.saveConfig(projectsConfig);
-        }
+        // setupWorkspace already creates the workspace and adds it to config via IPC
 
         // Get current metadata
         const oldMetadata = await env.mockIpcRenderer.invoke(
@@ -277,19 +253,7 @@ describeIntegration("IpcMain rename workspace integration tests", () => {
       const { env, workspaceId, workspacePath, tempGitRepo, branchName, cleanup } =
         await setupWorkspace("anthropic");
       try {
-        // Add project and workspace to config via IPC
-        await env.mockIpcRenderer.invoke(IPC_CHANNELS.PROJECT_CREATE, tempGitRepo);
-        // Manually add workspace to the project (normally done by WORKSPACE_CREATE)
-        const projectsConfig = env.config.loadConfigOrDefault();
-        const projectConfig = projectsConfig.projects.get(tempGitRepo);
-        if (projectConfig) {
-          projectConfig.workspaces.push({
-            path: workspacePath,
-            id: workspaceId,
-            name: branchName,
-          });
-          env.config.saveConfig(projectsConfig);
-        }
+        // setupWorkspace already creates the workspace and adds it to config via IPC
         // Send a message to create some history
         env.sentEvents.length = 0;
         const result = await sendMessageWithModel(env.mockIpcRenderer, workspaceId, "What is 2+2?");
@@ -340,19 +304,7 @@ describeIntegration("IpcMain rename workspace integration tests", () => {
       const { env, workspaceId, workspacePath, tempGitRepo, branchName, cleanup } =
         await setupWorkspace("anthropic");
       try {
-        // Add project and workspace to config via IPC
-        await env.mockIpcRenderer.invoke(IPC_CHANNELS.PROJECT_CREATE, tempGitRepo);
-        // Manually add workspace to the project (normally done by WORKSPACE_CREATE)
-        const projectsConfig = env.config.loadConfigOrDefault();
-        const projectConfig = projectsConfig.projects.get(tempGitRepo);
-        if (projectConfig) {
-          projectConfig.workspaces.push({
-            path: workspacePath,
-            id: workspaceId,
-            name: branchName,
-          });
-          env.config.saveConfig(projectsConfig);
-        }
+        // setupWorkspace already creates the workspace and adds it to config via IPC
 
         // Send a message to create history before rename
         env.sentEvents.length = 0;
@@ -441,19 +393,7 @@ describeIntegration("IpcMain rename workspace integration tests", () => {
       const { env, workspaceId, tempGitRepo, branchName, cleanup } =
         await setupWorkspace("anthropic");
       try {
-        // Add project and workspace to config via IPC
-        await env.mockIpcRenderer.invoke(IPC_CHANNELS.PROJECT_CREATE, tempGitRepo);
-        const projectsConfig = env.config.loadConfigOrDefault();
-        const projectConfig = projectsConfig.projects.get(tempGitRepo);
-        if (projectConfig) {
-          const workspacePath = env.config.getWorkspacePath(tempGitRepo, branchName);
-          projectConfig.workspaces.push({
-            path: workspacePath,
-            id: workspaceId,
-            name: branchName,
-          });
-          env.config.saveConfig(projectsConfig);
-        }
+        // setupWorkspace already creates the workspace and adds it to config via IPC
 
         // Start a stream (don't await - we want it running)
         sendMessageWithModel(