fix(coordinator): align agent arg tests after rebase

Author: brooksc

electron/ipc/agents.test.ts

@@ -1,41 +1,5 @@
 import { describe, expect, it } from 'vitest';
-import { getMcpConfigArgs, getSkipPermissionsArgs } from './agents.js';
-
-describe('getMcpConfigArgs', () => {
-  it('returns flag + path for claude', () => {
-    expect(getMcpConfigArgs('claude', '/tmp/config.json')).toEqual([
-      '--mcp-config',
-      '/tmp/config.json',
-    ]);
-  });
-
-  it('returns flag + path for codex', () => {
-    expect(getMcpConfigArgs('codex', '/tmp/config.json')).toEqual(['--config', '/tmp/config.json']);
-  });
-
-  it('returns empty for gemini', () => {
-    expect(getMcpConfigArgs('gemini', '/tmp/config.json')).toEqual([]);
-  });
-
-  it('returns empty for opencode', () => {
-    expect(getMcpConfigArgs('opencode', '/tmp/config.json')).toEqual([]);
-  });
-
-  it('returns empty for copilot', () => {
-    expect(getMcpConfigArgs('copilot', '/tmp/config.json')).toEqual([]);
-  });
-
-  it('handles path-qualified claude command', () => {
-    expect(getMcpConfigArgs('/usr/local/bin/claude', '/tmp/config.json')).toEqual([
-      '--mcp-config',
-      '/tmp/config.json',
-    ]);
-  });
-
-  it('handles unknown agent', () => {
-    expect(getMcpConfigArgs('unknown-agent', '/tmp/config.json')).toEqual([]);
-  });
-});
+import { getSkipPermissionsArgs } from './agents.js';
 
 describe('getSkipPermissionsArgs', () => {
   it('returns a copy of default skip-permission args', () => {

electron/ipc/agents.ts

@@ -85,7 +85,7 @@ const AGENT_CACHE_TTL = 30_000;
 export function getSkipPermissionsArgs(command: string): string[] {
   const base = path.basename(command);
   const agent = DEFAULT_AGENTS.find((a) => a.command === base || a.command === command);
-  return agent ? agent.skip_permissions_args : [];
+  return agent ? [...agent.skip_permissions_args] : [];
 }
 
 export async function listAgents(): Promise<AgentDef[]> {