refactor(engines): remove profile parameter from engine interfaces and executions

The profile parameter was redundant as the agentId is sufficient for authentication purposes. This simplifies the API surface and reduces unnecessary configuration options across all engine providers (Claude, Cursor, Codex) and their execution flows.

Fixing json codex starting bug "can't find 'git-commit' profile"

Author: moazbuilds

src/agents/execution/runner.ts

@@ -18,10 +18,6 @@ export interface ExecuteAgentOptions {
    */
   model?: string;
 
-  /**
-   * Engine profile to use (defaults to agentId)
-   */
-  profile?: string;
 
   /**
    * Working directory for execution
@@ -97,7 +93,7 @@ export async function executeAgent(
   prompt: string,
   options: ExecuteAgentOptions,
 ): Promise<string> {
-  const { workingDir, projectRoot, engine: engineOverride, model: modelOverride, profile: profileOverride, logger, stderrLogger, abortSignal, timeout } = options;
+  const { workingDir, projectRoot, engine: engineOverride, model: modelOverride, logger, stderrLogger, abortSignal, timeout } = options;
 
   // Load agent config to determine engine and model
   const agentConfig = await loadAgentConfig(agentId, projectRoot ?? workingDir);
@@ -136,10 +132,8 @@ export async function executeAgent(
     console.log(`ℹ️  No engine specified for agent '${agentId}', using ${foundEngine.metadata.name} (${engineType})`);
   }
 
-  const profile = profileOverride ?? agentId;
-
   // Ensure authentication
-  await ensureEngineAuth(engineType, profile);
+  await ensureEngineAuth(engineType, agentId);
 
   // Get engine module for defaults
   const engineModule = registry.get(engineType);
@@ -164,7 +158,6 @@ export async function executeAgent(
 
   let totalStdout = '';
   const result = await engine.run({
-    profile,
     prompt: compositePrompt,
     workingDir,
     model,

src/cli/commands/agent.command.ts

@@ -3,7 +3,6 @@ import type { Command } from 'commander';
 import { executeAgent } from '../../agents/execution/index.js';
 
 type AgentCommandOptions = {
-  profile?: string;
   model?: string;
 };
 
@@ -21,7 +20,6 @@ async function registerMainAgentCommand(program: Command): Promise<void> {
     .description(`Execute agent with engine from config (defaults to ${defaultEngineName})`)
     .argument('<id>', 'Agent id from config/sub.agents.js or config/main.agents.js')
     .argument('<prompt...>', 'User request to send to the agent')
-    .option('--profile <profile>', 'Engine profile to use (defaults to the agent id)')
     .option('--model <model>', 'Model to use (overrides agent config)')
     .action(async (id: string, promptParts: string[], options: AgentCommandOptions) => {
       const prompt = promptParts.join(' ').trim();
@@ -31,7 +29,6 @@ async function registerMainAgentCommand(program: Command): Promise<void> {
 
       await executeAgent(id, prompt, {
         workingDir: process.cwd(),
-        profile: options.profile,
         model: options.model,
       });
     });
@@ -54,7 +51,6 @@ function registerEngineAgentCommands(program: Command): void {
         .description(`Execute ${engine.metadata.name} with an agent wrapper`)
         .argument('<id>', 'Agent id from config/sub.agents.js or config/main.agents.js')
         .argument('<prompt...>', 'User request to send to the agent')
-        .option('--profile <profile>', `${engine.metadata.name} profile to use (defaults to the agent id)`)
         .option('--model <model>', 'Model to use (overrides agent config)')
         .action(async (id: string, promptParts: string[], options: AgentCommandOptions) => {
           const prompt = promptParts.join(' ').trim();
@@ -65,7 +61,6 @@ function registerEngineAgentCommands(program: Command): void {
           await executeAgent(id, prompt, {
             engine: engine.metadata.id,
             workingDir: process.cwd(),
-            profile: options.profile,
             model: options.model,
           });
         });

src/infra/engines/core/types.ts

@@ -9,7 +9,6 @@
 export type EngineType = string;
 
 export interface EngineRunOptions {
-  profile: string;
   prompt: string;
   workingDir: string;
   model?: string;

src/infra/engines/providers/claude/execution/commands.ts

@@ -1,5 +1,4 @@
 export interface ClaudeCommandOptions {
-  profile: string;
   workingDir: string;
   prompt: string;
   model?: string;

src/infra/engines/providers/claude/execution/executor.ts

@@ -28,7 +28,6 @@ export async function runClaudePrompt(options: {
   }
 
   await runClaude({
-    profile: options.agentId,
     prompt: options.prompt,
     workingDir: options.cwd,
     model: options.model,
@@ -79,7 +78,6 @@ export async function runAgent(
 
   let buffered = '';
   const result = await runClaude({
-    profile: agentId,
     prompt,
     workingDir: cwd,
     model: options.model,

src/infra/engines/providers/claude/execution/runner.ts

@@ -8,7 +8,6 @@ import { expandHomeDir } from '../../../../../shared/utils/index.js';
 import { logger } from '../../../../../shared/logging/index.js';
 
 export interface RunClaudeOptions {
-  profile: string;
   prompt: string;
   workingDir: string;
   model?: string;
@@ -68,11 +67,7 @@ function formatStreamJsonLine(line: string): string | null {
 }
 
 export async function runClaude(options: RunClaudeOptions): Promise<RunClaudeResult> {
-  const { profile, prompt, workingDir, model, env, onData, onErrorData, abortSignal, timeout = 600000 } = options;
-
-  if (!profile) {
-    throw new Error('runClaude requires a profile.');
-  }
+  const { prompt, workingDir, model, env, onData, onErrorData, abortSignal, timeout = 600000 } = options;
 
   if (!prompt) {
     throw new Error('runClaude requires a prompt.');
@@ -82,7 +77,7 @@ export async function runClaude(options: RunClaudeOptions): Promise<RunClaudeRes
     throw new Error('runClaude requires a working directory.');
   }
 
-  // Set up CLAUDE_CONFIG_DIR (shared for authentication, profile is for agent data only)
+  // Set up CLAUDE_CONFIG_DIR for authentication
   const claudeConfigDir = process.env.CLAUDE_CONFIG_DIR
     ? expandHomeDir(process.env.CLAUDE_CONFIG_DIR)
     : path.join(homedir(), '.codemachine', 'claude');
@@ -120,7 +115,7 @@ export async function runClaude(options: RunClaudeOptions): Promise<RunClaudeRes
     return result;
   };
 
-  const { command, args } = buildClaudeExecCommand({ profile, workingDir, prompt, model });
+  const { command, args } = buildClaudeExecCommand({ workingDir, prompt, model });
 
   logger.debug(`Claude runner - prompt length: ${prompt.length}, lines: ${prompt.split('\n').length}`);
   logger.debug(`Claude runner - args count: ${args.length}, model: ${model ?? 'default'}`);

src/infra/engines/providers/codex/execution/commands.ts

@@ -1,5 +1,4 @@
 export interface CodexCommandOptions {
-  profile: string;
   workingDir: string;
   prompt: string;
   model?: string;
@@ -12,13 +11,11 @@ export interface CodexCommand {
 }
 
 export function buildCodexExecCommand(options: CodexCommandOptions): CodexCommand {
-  const { profile, workingDir, model, modelReasoningEffort } = options;
+  const { workingDir, model, modelReasoningEffort } = options;
 
   const args = [
     'exec',
     '--json',
-    '--profile',
-    profile,
     '--skip-git-repo-check',
     '--sandbox',
     'danger-full-access',

src/infra/engines/providers/codex/execution/executor.ts

@@ -26,7 +26,6 @@ export async function runCodexPrompt(options: {
   }
 
   await runCodex({
-    profile: options.agentId,
     prompt: options.prompt,
     workingDir: options.cwd,
     onData: (chunk) => {
@@ -76,7 +75,6 @@ export async function runAgent(
 
   let buffered = '';
   const result = await runCodex({
-    profile: agentId,
     prompt,
     workingDir: cwd,
     abortSignal: options.abortSignal,

src/infra/engines/providers/codex/execution/runner.ts

@@ -8,7 +8,6 @@ import { expandHomeDir } from '../../../../../shared/utils/index.js';
 import { logger } from '../../../../../shared/logging/index.js';
 
 export interface RunCodexOptions {
-  profile: string;
   prompt: string;
   workingDir: string;
   model?: string;
@@ -79,11 +78,7 @@ function formatCodexStreamJsonLine(line: string): string | null {
 }
 
 export async function runCodex(options: RunCodexOptions): Promise<RunCodexResult> {
-  const { profile, prompt, workingDir, model, modelReasoningEffort, env, onData, onErrorData, abortSignal, timeout = 600000 } = options;
-
-  if (!profile) {
-    throw new Error('runCodex requires a profile.');
-  }
+  const { prompt, workingDir, model, modelReasoningEffort, env, onData, onErrorData, abortSignal, timeout = 600000 } = options;
 
   if (!prompt) {
     throw new Error('runCodex requires a prompt.');
@@ -96,7 +91,7 @@ export async function runCodex(options: RunCodexOptions): Promise<RunCodexResult
   // Prefer calling the real Codex CLI directly, mirroring runner-prompts spec
   // Example (Linux/Mac):
   //   CODEX_HOME="$HOME/.codemachine/codex" codex exec \
-  //     --profile <profile> --skip-git-repo-check \
+  //     --skip-git-repo-check \
   //     --sandbox danger-full-access --dangerously-bypass-approvals-and-sandbox \
   //     -C <workingDir> "<composite prompt>"
 
@@ -133,7 +128,7 @@ export async function runCodex(options: RunCodexOptions): Promise<RunCodexResult
     return result;
   };
 
-  const { command, args } = buildCodexExecCommand({ profile, workingDir, prompt, model, modelReasoningEffort });
+  const { command, args } = buildCodexExecCommand({ workingDir, prompt, model, modelReasoningEffort });
 
   logger.debug(`Codex runner - prompt length: ${prompt.length}, lines: ${prompt.split('\n').length}`);
   logger.debug(`Codex runner - args count: ${args.length}`);

src/infra/engines/providers/cursor/execution/commands.ts

@@ -1,5 +1,4 @@
 export interface CursorCommandOptions {
-  profile: string;
   workingDir: string;
   prompt: string;
   model?: string;