@shepai/cli 1.175.1-pr537.fc6a92c → 1.176.0

package/dist/packages/core/src/infrastructure/repositories/agent-run.repository.js

@@ -130,14 +130,6 @@ let SQLiteAgentRunRepository = class SQLiteAgentRunRepository {
             updated_at: toTimestamp(updates.updatedAt) ?? Date.now(),
         });
     }
-    async findByIds(ids) {
-        if (ids.length === 0)
-            return [];
-        const placeholders = ids.map(() => '?').join(', ');
-        const stmt = this.db.prepare(`SELECT * FROM agent_runs WHERE id IN (${placeholders})`);
-        const rows = stmt.all(...ids);
-        return rows.map(fromDatabase);
-    }
     async findRunningByPid(pid) {
         const stmt = this.db.prepare('SELECT * FROM agent_runs WHERE pid = ? AND status = ?');
         const rows = stmt.all(pid, 'running');

package/dist/packages/core/src/infrastructure/repositories/sqlite-phase-timing.repository.d.ts

@@ -17,7 +17,6 @@ export declare class SQLitePhaseTimingRepository implements IPhaseTimingReposito
     update(id: string, updates: Partial<Pick<PhaseTiming, 'completedAt' | 'durationMs' | 'inputTokens' | 'outputTokens' | 'cacheCreationInputTokens' | 'cacheReadInputTokens' | 'costUsd' | 'numTurns' | 'durationApiMs' | 'exitCode' | 'errorMessage'>>): Promise<void>;
     updateApprovalWait(id: string, updates: Partial<Pick<PhaseTiming, 'waitingApprovalAt' | 'approvalWaitMs'>>): Promise<void>;
     findByRunId(agentRunId: string): Promise<PhaseTiming[]>;
-    findByRunIds(runIds: string[]): Promise<PhaseTiming[]>;
     findByFeatureId(featureId: string): Promise<PhaseTiming[]>;
 }
 //# sourceMappingURL=sqlite-phase-timing.repository.d.ts.map

package/dist/packages/core/src/infrastructure/repositories/sqlite-phase-timing.repository.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"sqlite-phase-timing.repository.d.ts","sourceRoot":"","sources":["../../../../../../packages/core/src/infrastructure/repositories/sqlite-phase-timing.repository.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,QAAQ,MAAM,gBAAgB,CAAC;AAE3C,OAAO,KAAK,EAAE,sBAAsB,EAAE,MAAM,4EAA4E,CAAC;AACzH,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,kCAAkC,CAAC;AA6FpE;;GAEG;AACH,qBACa,2BAA4B,YAAW,sBAAsB;IAC5D,OAAO,CAAC,QAAQ,CAAC,EAAE;gBAAF,EAAE,EAAE,QAAQ,CAAC,QAAQ;IAE5C,IAAI,CAAC,WAAW,EAAE,WAAW,GAAG,OAAO,CAAC,IAAI,CAAC;IA0B7C,MAAM,CACV,EAAE,EAAE,MAAM,EACV,OAAO,EAAE,OAAO,CACd,IAAI,CACF,WAAW,EACT,aAAa,GACb,YAAY,GACZ,aAAa,GACb,cAAc,GACd,0BAA0B,GAC1B,sBAAsB,GACtB,SAAS,GACT,UAAU,GACV,eAAe,GACf,UAAU,GACV,cAAc,CACjB,CACF,GACA,OAAO,CAAC,IAAI,CAAC;IAsEV,kBAAkB,CACtB,EAAE,EAAE,MAAM,EACV,OAAO,EAAE,OAAO,CAAC,IAAI,CAAC,WAAW,EAAE,mBAAmB,GAAG,gBAAgB,CAAC,CAAC,GAC1E,OAAO,CAAC,IAAI,CAAC;IA2BV,WAAW,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC;IASvD,YAAY,CAAC,MAAM,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC;IAYtD,eAAe,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC;CAWjE"}
+{"version":3,"file":"sqlite-phase-timing.repository.d.ts","sourceRoot":"","sources":["../../../../../../packages/core/src/infrastructure/repositories/sqlite-phase-timing.repository.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,QAAQ,MAAM,gBAAgB,CAAC;AAE3C,OAAO,KAAK,EAAE,sBAAsB,EAAE,MAAM,4EAA4E,CAAC;AACzH,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,kCAAkC,CAAC;AA6FpE;;GAEG;AACH,qBACa,2BAA4B,YAAW,sBAAsB;IAC5D,OAAO,CAAC,QAAQ,CAAC,EAAE;gBAAF,EAAE,EAAE,QAAQ,CAAC,QAAQ;IAE5C,IAAI,CAAC,WAAW,EAAE,WAAW,GAAG,OAAO,CAAC,IAAI,CAAC;IA0B7C,MAAM,CACV,EAAE,EAAE,MAAM,EACV,OAAO,EAAE,OAAO,CACd,IAAI,CACF,WAAW,EACT,aAAa,GACb,YAAY,GACZ,aAAa,GACb,cAAc,GACd,0BAA0B,GAC1B,sBAAsB,GACtB,SAAS,GACT,UAAU,GACV,eAAe,GACf,UAAU,GACV,cAAc,CACjB,CACF,GACA,OAAO,CAAC,IAAI,CAAC;IAsEV,kBAAkB,CACtB,EAAE,EAAE,MAAM,EACV,OAAO,EAAE,OAAO,CAAC,IAAI,CAAC,WAAW,EAAE,mBAAmB,GAAG,gBAAgB,CAAC,CAAC,GAC1E,OAAO,CAAC,IAAI,CAAC;IA2BV,WAAW,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC;IASvD,eAAe,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC;CAWjE"}

package/dist/packages/core/src/infrastructure/repositories/sqlite-phase-timing.repository.js

@@ -19,7 +19,7 @@ function toDatabase(timing) {
         id: timing.id,
         agent_run_id: timing.agentRunId,
         phase: timing.phase,
-        started_at: timing.startedAt.getTime(),
+        started_at: timing.startedAt instanceof Date ? timing.startedAt.getTime() : timing.startedAt,
         completed_at: timing.completedAt instanceof Date ? timing.completedAt.getTime() : null,
         duration_ms: timing.durationMs != null ? Number(timing.durationMs) : null,
         waiting_approval_at: timing.waitingApprovalAt instanceof Date ? timing.waitingApprovalAt.getTime() : null,
@@ -36,8 +36,8 @@ function toDatabase(timing) {
         duration_api_ms: timing.durationApiMs != null ? Number(timing.durationApiMs) : null,
         exit_code: timing.exitCode ?? null,
         error_message: timing.errorMessage ?? null,
-        created_at: timing.createdAt.getTime(),
-        updated_at: timing.updatedAt.getTime(),
+        created_at: timing.createdAt instanceof Date ? timing.createdAt.getTime() : timing.createdAt,
+        updated_at: timing.updatedAt instanceof Date ? timing.updatedAt.getTime() : timing.updatedAt,
     };
 }
 function fromDatabase(row) {
@@ -182,14 +182,6 @@ let SQLitePhaseTimingRepository = class SQLitePhaseTimingRepository {
         const rows = stmt.all(agentRunId);
         return rows.map(fromDatabase);
     }
-    async findByRunIds(runIds) {
-        if (runIds.length === 0)
-            return [];
-        const placeholders = runIds.map(() => '?').join(', ');
-        const stmt = this.db.prepare(`SELECT * FROM phase_timings WHERE agent_run_id IN (${placeholders}) ORDER BY created_at`);
-        const rows = stmt.all(...runIds);
-        return rows.map(fromDatabase);
-    }
     async findByFeatureId(featureId) {
         const stmt = this.db.prepare(`
       SELECT pt.* FROM phase_timings pt

package/dist/packages/core/src/infrastructure/repositories/sqlite-settings.repository.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"sqlite-settings.repository.d.ts","sourceRoot":"","sources":["../../../../../../packages/core/src/infrastructure/repositories/sqlite-settings.repository.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,KAAK,QAAQ,MAAM,gBAAgB,CAAC;AAE3C,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,8EAA8E,CAAC;AACxH,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,kCAAkC,CAAC;AAOjE;;;GAGG;AACH,qBACa,wBAAyB,YAAW,mBAAmB;IACtD,OAAO,CAAC,QAAQ,CAAC,EAAE;gBAAF,EAAE,EAAE,QAAQ,CAAC,QAAQ;IAElD;;;;;;OAMG;IACG,UAAU,CAAC,QAAQ,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC;IAmFnD;;;;OAIG;IACG,IAAI,IAAI,OAAO,CAAC,QAAQ,GAAG,IAAI,CAAC;IAgBtC;;;;;OAKG;IACG,MAAM,CAAC,QAAQ,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC;CA4FhD"}
+{"version":3,"file":"sqlite-settings.repository.d.ts","sourceRoot":"","sources":["../../../../../../packages/core/src/infrastructure/repositories/sqlite-settings.repository.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,KAAK,QAAQ,MAAM,gBAAgB,CAAC;AAE3C,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,8EAA8E,CAAC;AACxH,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,kCAAkC,CAAC;AAOjE;;;GAGG;AACH,qBACa,wBAAyB,YAAW,mBAAmB;IACtD,OAAO,CAAC,QAAQ,CAAC,EAAE;gBAAF,EAAE,EAAE,QAAQ,CAAC,QAAQ;IAElD;;;;;;OAMG;IACG,UAAU,CAAC,QAAQ,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC;IAmFnD;;;;OAIG;IACG,IAAI,IAAI,OAAO,CAAC,QAAQ,GAAG,IAAI,CAAC;IAgBtC;;;;;OAKG;IACG,MAAM,CAAC,QAAQ,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC;CA2FhD"}

package/dist/packages/core/src/infrastructure/repositories/sqlite-settings.repository.js

@@ -48,7 +48,7 @@ let SQLiteSettingsRepository = class SQLiteSettingsRepository {
         user_name, user_email, user_github_username, user_preferred_language,
         env_default_editor, env_shell_preference, env_terminal_preference,
         sys_auto_update, sys_log_level,
-        agent_type, agent_auth_method, agent_token, agent_permission_mode,
+        agent_type, agent_auth_method, agent_token,
         notif_in_app_enabled, notif_browser_enabled, notif_desktop_enabled,
         notif_evt_agent_started, notif_evt_phase_completed, notif_evt_waiting_approval,
         notif_evt_agent_completed, notif_evt_agent_failed,
@@ -80,7 +80,7 @@ let SQLiteSettingsRepository = class SQLiteSettingsRepository {
         @user_name, @user_email, @user_github_username, @user_preferred_language,
         @env_default_editor, @env_shell_preference, @env_terminal_preference,
         @sys_auto_update, @sys_log_level,
-        @agent_type, @agent_auth_method, @agent_token, @agent_permission_mode,
+        @agent_type, @agent_auth_method, @agent_token,
         @notif_in_app_enabled, @notif_browser_enabled, @notif_desktop_enabled,
         @notif_evt_agent_started, @notif_evt_phase_completed, @notif_evt_waiting_approval,
         @notif_evt_agent_completed, @notif_evt_agent_failed,
@@ -164,7 +164,6 @@ let SQLiteSettingsRepository = class SQLiteSettingsRepository {
         agent_type = @agent_type,
         agent_auth_method = @agent_auth_method,
         agent_token = @agent_token,
-        agent_permission_mode = @agent_permission_mode,
         notif_in_app_enabled = @notif_in_app_enabled,
         notif_browser_enabled = @notif_browser_enabled,
         notif_desktop_enabled = @notif_desktop_enabled,

package/dist/packages/core/src/infrastructure/services/agents/common/agent-executor-factory.service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"agent-executor-factory.service.d.ts","sourceRoot":"","sources":["../../../../../../../../packages/core/src/infrastructure/services/agents/common/agent-executor-factory.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,KAAK,EAAE,SAAS,EAAE,WAAW,EAAE,MAAM,wCAAwC,CAAC;AACrF,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,yEAAyE,CAAC;AAC9G,OAAO,KAAK,EAAE,yBAAyB,EAAE,MAAM,qFAAqF,CAAC;AACrI,OAAO,KAAK,EACV,qBAAqB,EACrB,YAAY,EACb,MAAM,iFAAiF,CAAC;AASzF,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,YAAY,CAAC;AAEhD;;;;;GAKG;AACH,qBAAa,oBAAqB,YAAW,qBAAqB;IAMpD,OAAO,CAAC,QAAQ,CAAC,KAAK;IALlC,OAAO,CAAC,QAAQ,CAAC,KAAK,CAAqC;IAE3D;;OAEG;gBAC0B,KAAK,EAAE,aAAa;IAEjD;;;;;;;OAOG;IACH,cAAc,CAAC,SAAS,EAAE,SAAS,EAAE,WAAW,EAAE,WAAW,GAAG,cAAc;IAsC9E;;;;OAIG;IACH,kBAAkB,IAAI,SAAS,EAAE;IAYjC,UAAU,IAAI,YAAY,EAAE;IAW5B;;;;;;OAMG;IACH,kBAAkB,CAAC,SAAS,EAAE,SAAS,GAAG,MAAM,EAAE;IAmBlD;;;;;;;;OAQG;IACH,yBAAyB,CACvB,SAAS,EAAE,SAAS,EACpB,WAAW,EAAE,WAAW,GACvB,yBAAyB;IAW5B;;;;;OAKG;IACH,mBAAmB,CAAC,SAAS,EAAE,SAAS,GAAG,OAAO;CAGnD"}
+{"version":3,"file":"agent-executor-factory.service.d.ts","sourceRoot":"","sources":["../../../../../../../../packages/core/src/infrastructure/services/agents/common/agent-executor-factory.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,KAAK,EAAE,SAAS,EAAE,WAAW,EAAE,MAAM,wCAAwC,CAAC;AACrF,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,yEAAyE,CAAC;AAC9G,OAAO,KAAK,EAAE,yBAAyB,EAAE,MAAM,qFAAqF,CAAC;AACrI,OAAO,KAAK,EACV,qBAAqB,EACrB,YAAY,EACb,MAAM,iFAAiF,CAAC;AAQzF,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,YAAY,CAAC;AAEhD;;;;;GAKG;AACH,qBAAa,oBAAqB,YAAW,qBAAqB;IAMpD,OAAO,CAAC,QAAQ,CAAC,KAAK;IALlC,OAAO,CAAC,QAAQ,CAAC,KAAK,CAAqC;IAE3D;;OAEG;gBAC0B,KAAK,EAAE,aAAa;IAEjD;;;;;;;OAOG;IACH,cAAc,CAAC,SAAS,EAAE,SAAS,EAAE,WAAW,EAAE,WAAW,GAAG,cAAc;IAmC9E;;;;OAIG;IACH,kBAAkB,IAAI,SAAS,EAAE;IAWjC,UAAU,IAAI,YAAY,EAAE;IAU5B;;;;;;OAMG;IACH,kBAAkB,CAAC,SAAS,EAAE,SAAS,GAAG,MAAM,EAAE;IAiBlD;;;;;;;;OAQG;IACH,yBAAyB,CACvB,SAAS,EAAE,SAAS,EACpB,WAAW,EAAE,WAAW,GACvB,yBAAyB;IAW5B;;;;;OAKG;IACH,mBAAmB,CAAC,SAAS,EAAE,SAAS,GAAG,OAAO;CAGnD"}

package/dist/packages/core/src/infrastructure/services/agents/common/agent-executor-factory.service.js

@@ -14,7 +14,6 @@ import { DevAgentExecutorService } from './executors/dev-executor.service.js';
 import { GeminiCliExecutorService } from './executors/gemini-cli-executor.service.js';
 import { CodexCliExecutorService } from './executors/codex-cli-executor.service.js';
 import { CopilotCliExecutorService } from './executors/copilot-cli-executor.service.js';
-import { RovoDevExecutorService } from './executors/rovo-dev-executor.service.js';
 /**
  * Factory that creates and caches agent executor instances.
  *
@@ -63,9 +62,6 @@ export class AgentExecutorFactory {
             case 'copilot-cli':
                 executor = new CopilotCliExecutorService(this.spawn, _authConfig);
                 break;
-            case 'rovo-dev':
-                executor = new RovoDevExecutorService(this.spawn, _authConfig);
-                break;
             default:
                 throw new Error(`Unsupported agent type: ${agentType}. Supported: ${this.getSupportedAgents().join(', ')}`);
         }
@@ -85,7 +81,6 @@ export class AgentExecutorFactory {
             'gemini-cli',
             'codex-cli',
             'copilot-cli',
-            'rovo-dev',
         ];
     }
     getCliInfo() {
@@ -95,7 +90,6 @@ export class AgentExecutorFactory {
             { agentType: 'cursor', cmd: 'cursor', versionArgs: ['--version'] },
             { agentType: 'codex-cli', cmd: 'codex', versionArgs: ['--version'] },
             { agentType: 'copilot-cli', cmd: 'copilot', versionArgs: ['--version'] },
-            { agentType: 'rovo-dev', cmd: 'rovo', versionArgs: ['--version'] },
         ];
     }
     /**
@@ -117,8 +111,6 @@ export class AgentExecutorFactory {
                 return CODEX_CLI_MODELS;
             case 'copilot-cli':
                 return COPILOT_CLI_MODELS;
-            case 'rovo-dev':
-                return ROVO_DEV_MODELS;
             default:
                 return [];
         }
@@ -197,5 +189,3 @@ const COPILOT_CLI_MODELS = [
     'gpt-5.4',
     'gpt-5.4-mini',
 ];
-// TODO: Update with actual Rovo Dev model identifiers when available
-const ROVO_DEV_MODELS = ['rovo-agent-1', 'rovo-agent-1-mini'];

package/dist/packages/core/src/infrastructure/services/agents/common/agent-runner.service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"agent-runner.service.d.ts","sourceRoot":"","sources":["../../../../../../../../packages/core/src/infrastructure/services/agents/common/agent-runner.service.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAIH,OAAO,KAAK,EACV,YAAY,EACZ,eAAe,EAChB,MAAM,6DAA6D,CAAC;AACrE,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,+DAA+D,CAAC;AACpG,OAAO,KAAK,EAAE,sBAAsB,EAAE,MAAM,wEAAwE,CAAC;AACrH,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,qEAAqE,CAAC;AAE/G,OAAO,KAAK,EAAE,QAAQ,EAAE,aAAa,EAAE,MAAM,8BAA8B,CAAC;AAoB5E,qBAAa,kBAAmB,YAAW,YAAY;IAEnD,OAAO,CAAC,QAAQ,CAAC,QAAQ;IACzB,OAAO,CAAC,QAAQ,CAAC,gBAAgB;IACjC,OAAO,CAAC,QAAQ,CAAC,aAAa;gBAFb,QAAQ,EAAE,cAAc,EACxB,gBAAgB,EAAE,sBAAsB,EACxC,aAAa,EAAE,mBAAmB;IAG/C,QAAQ,CAAC,SAAS,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,eAAe,GAAG,OAAO,CAAC,QAAQ,CAAC;IAoBxF,cAAc,CACnB,SAAS,EAAE,MAAM,EACjB,MAAM,EAAE,MAAM,EACd,OAAO,CAAC,EAAE,eAAe,GACxB,aAAa,CAAC,aAAa,CAAC;YAwCjB,QAAQ;YAwCR,aAAa;YAUb,UAAU;CAQzB"}
+{"version":3,"file":"agent-runner.service.d.ts","sourceRoot":"","sources":["../../../../../../../../packages/core/src/infrastructure/services/agents/common/agent-runner.service.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAIH,OAAO,KAAK,EACV,YAAY,EACZ,eAAe,EAChB,MAAM,6DAA6D,CAAC;AACrE,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,+DAA+D,CAAC;AACpG,OAAO,KAAK,EAAE,sBAAsB,EAAE,MAAM,wEAAwE,CAAC;AACrH,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,qEAAqE,CAAC;AAE/G,OAAO,KAAK,EAAE,QAAQ,EAAE,aAAa,EAAE,MAAM,8BAA8B,CAAC;AAoB5E,qBAAa,kBAAmB,YAAW,YAAY;IAEnD,OAAO,CAAC,QAAQ,CAAC,QAAQ;IACzB,OAAO,CAAC,QAAQ,CAAC,gBAAgB;IACjC,OAAO,CAAC,QAAQ,CAAC,aAAa;gBAFb,QAAQ,EAAE,cAAc,EACxB,gBAAgB,EAAE,sBAAsB,EACxC,aAAa,EAAE,mBAAmB;IAG/C,QAAQ,CAAC,SAAS,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,eAAe,GAAG,OAAO,CAAC,QAAQ,CAAC;IAoBxF,cAAc,CACnB,SAAS,EAAE,MAAM,EACjB,MAAM,EAAE,MAAM,EACd,OAAO,CAAC,EAAE,eAAe,GACxB,aAAa,CAAC,aAAa,CAAC;YAyCjB,QAAQ;YAwCR,aAAa;YAUb,UAAU;CAQzB"}

package/dist/packages/core/src/infrastructure/services/agents/common/agent-runner.service.js

@@ -73,7 +73,7 @@ export class AgentRunnerService {
             yield {
                 type: event.type,
                 content: event.content,
-                timestamp: event.timestamp instanceof Date ? event.timestamp : new Date(event.timestamp),
+                timestamp: event.timestamp instanceof Date ? event.timestamp.toISOString() : event.timestamp,
             };
         }
         // Ensure graph promise cleanup completes
@@ -92,7 +92,7 @@ export class AgentRunnerService {
         const executor = await this.executorProvider.getExecutor();
         const runId = crypto.randomUUID();
         const threadId = crypto.randomUUID();
-        const now = new Date();
+        const now = new Date().toISOString();
         const agentRun = {
             id: runId,
             agentType,
@@ -112,7 +112,7 @@ export class AgentRunnerService {
         return { definition, executor, runId, threadId };
     }
     async markCompleted(runId, result) {
-        const completedAt = new Date();
+        const completedAt = new Date().toISOString();
         await this.runRepository.updateStatus(runId, AgentRunStatus.completed, {
             result: result.analysisMarkdown ?? result.result ?? JSON.stringify(result),
             completedAt,
@@ -120,7 +120,7 @@ export class AgentRunnerService {
         });
     }
     async markFailed(runId, error) {
-        const failedAt = new Date();
+        const failedAt = new Date().toISOString();
         await this.runRepository.updateStatus(runId, AgentRunStatus.failed, {
             error: error instanceof Error ? error.message : String(error),
             completedAt: failedAt,

package/dist/packages/core/src/infrastructure/services/agents/common/executors/claude-code-executor.service.d.ts

@@ -10,15 +10,19 @@
  */
 import type { AgentType, AgentFeature } from '../../../../../domain/generated/output.js';
 import type { IAgentExecutor, AgentExecutionOptions, AgentExecutionResult, AgentExecutionStreamEvent } from '../../../../../application/ports/output/agents/agent-executor.interface.js';
-import { type SpawnFunction } from '../types.js';
-import { AbstractAgentExecutor } from './abstract-agent-executor.js';
+import type { SpawnFunction } from '../types.js';
 /**
  * Executor service for Claude Code agent.
  * Uses subprocess spawning to interact with the `claude` CLI.
  */
-export declare class ClaudeCodeExecutorService extends AbstractAgentExecutor implements IAgentExecutor {
+export declare class ClaudeCodeExecutorService implements IAgentExecutor {
+    private readonly spawn;
     readonly agentType: AgentType;
+    /** When true, suppresses debug logging (set per-call via options.silent) */
+    private silent;
     constructor(spawn: SpawnFunction);
+    /** Debug logging — writes to stdout so it appears in the worker log file */
+    private log;
     execute(prompt: string, options?: AgentExecutionOptions): Promise<AgentExecutionResult>;
     executeStream(prompt: string, options?: AgentExecutionOptions): AsyncIterable<AgentExecutionStreamEvent>;
     /**

package/dist/packages/core/src/infrastructure/services/agents/common/executors/claude-code-executor.service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"claude-code-executor.service.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/common/executors/claude-code-executor.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,KAAK,EAAE,SAAS,EAAE,YAAY,EAAE,MAAM,2CAA2C,CAAC;AACzF,OAAO,KAAK,EACV,cAAc,EACd,qBAAqB,EACrB,oBAAoB,EAEpB,yBAAyB,EAC1B,MAAM,4EAA4E,CAAC;AACpF,OAAO,EAA0B,KAAK,aAAa,EAAE,MAAM,aAAa,CAAC;AACzE,OAAO,EAAE,qBAAqB,EAAE,MAAM,8BAA8B,CAAC;AAWrE;;;GAGG;AACH,qBAAa,yBAA0B,SAAQ,qBAAsB,YAAW,cAAc;IAC5F,QAAQ,CAAC,SAAS,EAAE,SAAS,CAA8B;gBAE/C,KAAK,EAAE,aAAa;IAI1B,OAAO,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,qBAAqB,GAAG,OAAO,CAAC,oBAAoB,CAAC;IAsHtF,aAAa,CAClB,MAAM,EAAE,MAAM,EACd,OAAO,CAAC,EAAE,qBAAqB,GAC9B,aAAa,CAAC,yBAAyB,CAAC;IAkG3C;;;OAGG;IACH,OAAO,CAAC,cAAc;IA0CtB,eAAe,CAAC,OAAO,EAAE,YAAY,GAAG,OAAO;IAI/C,OAAO,CAAC,SAAS;IAuBjB,OAAO,CAAC,eAAe;IAUvB,OAAO,CAAC,iBAAiB;IAIzB,OAAO,CAAC,eAAe;IA6BvB;;;OAGG;IACH,OAAO,CAAC,YAAY;IAqBpB,OAAO,CAAC,eAAe;CAkExB"}
+{"version":3,"file":"claude-code-executor.service.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/common/executors/claude-code-executor.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,KAAK,EAAE,SAAS,EAAE,YAAY,EAAE,MAAM,2CAA2C,CAAC;AACzF,OAAO,KAAK,EACV,cAAc,EACd,qBAAqB,EACrB,oBAAoB,EAEpB,yBAAyB,EAC1B,MAAM,4EAA4E,CAAC;AACpF,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AAajD;;;GAGG;AACH,qBAAa,yBAA0B,YAAW,cAAc;IAMlD,OAAO,CAAC,QAAQ,CAAC,KAAK;IALlC,QAAQ,CAAC,SAAS,EAAE,SAAS,CAA8B;IAE3D,4EAA4E;IAC5E,OAAO,CAAC,MAAM,CAAS;gBAEM,KAAK,EAAE,aAAa;IAEjD,4EAA4E;IAC5E,OAAO,CAAC,GAAG;IAML,OAAO,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,qBAAqB,GAAG,OAAO,CAAC,oBAAoB,CAAC;IAoHtF,aAAa,CAClB,MAAM,EAAE,MAAM,EACd,OAAO,CAAC,EAAE,qBAAqB,GAC9B,aAAa,CAAC,yBAAyB,CAAC;IAgG3C;;;OAGG;IACH,OAAO,CAAC,cAAc;IA0CtB,eAAe,CAAC,OAAO,EAAE,YAAY,GAAG,OAAO;IAI/C,OAAO,CAAC,SAAS;IAejB,OAAO,CAAC,eAAe;IAUvB,OAAO,CAAC,iBAAiB;IAyBzB,OAAO,CAAC,eAAe;IA6BvB;;;OAGG;IACH,OAAO,CAAC,YAAY;IAqBpB,OAAO,CAAC,eAAe;CAkExB"}

package/dist/packages/core/src/infrastructure/services/agents/common/executors/claude-code-executor.service.js

@@ -8,8 +8,8 @@
  * Uses constructor dependency injection for the spawn function
  * to enable testability without mocking node:child_process directly.
  */
-import { MAX_STDERR_BUFFER_SIZE } from '../types.js';
-import { AbstractAgentExecutor } from './abstract-agent-executor.js';
+import { getCurrentPhase, getLogPrefix } from '../../feature-agent/log-context.js';
+import { IS_WINDOWS } from '../../../../platform.js';
 /** Features supported by Claude Code CLI */
 const SUPPORTED_FEATURES = new Set([
     'session-resume',
@@ -22,10 +22,20 @@ const SUPPORTED_FEATURES = new Set([
  * Executor service for Claude Code agent.
  * Uses subprocess spawning to interact with the `claude` CLI.
  */
-export class ClaudeCodeExecutorService extends AbstractAgentExecutor {
+export class ClaudeCodeExecutorService {
+    spawn;
     agentType = 'claude-code';
+    /** When true, suppresses debug logging (set per-call via options.silent) */
+    silent = false;
     constructor(spawn) {
-        super(spawn);
+        this.spawn = spawn;
+    }
+    /** Debug logging — writes to stdout so it appears in the worker log file */
+    log(message) {
+        if (this.silent)
+            return;
+        const ts = new Date().toISOString();
+        process.stdout.write(`[${ts}] ${getCurrentPhase()}${getLogPrefix()}${message}\n`);
     }
     async execute(prompt, options) {
         this.silent = options?.silent ?? false;
@@ -90,9 +100,7 @@ export class ClaudeCodeExecutorService extends AbstractAgentExecutor {
             });
             proc.stderr?.on('data', (chunk) => {
                 const data = chunk.toString();
-                if (stderr.length < MAX_STDERR_BUFFER_SIZE) {
-                    stderr += data;
-                }
+                stderr += data;
                 this.log(`stderr: ${data.trimEnd()}`);
             });
             proc.on('error', (error) => {
@@ -178,9 +186,7 @@ export class ClaudeCodeExecutorService extends AbstractAgentExecutor {
             }
         });
         proc.stderr?.on('data', (chunk) => {
-            if (stderr.length < MAX_STDERR_BUFFER_SIZE) {
-                stderr += chunk.toString();
-            }
+            stderr += chunk.toString();
         });
         proc.on('error', (err) => {
             error = err;
@@ -266,14 +272,7 @@ export class ClaudeCodeExecutorService extends AbstractAgentExecutor {
     buildArgs(_prompt, options) {
         // Prompt is piped via stdin — not passed as a CLI argument — to avoid
         // ENAMETOOLONG on Windows when prompts exceed the ~32 KB arg-length limit.
-        const args = ['-p', '--output-format', 'json'];
-        // Permission mode: strict uses plan mode; default/autonomous use dangerously-skip-permissions
-        if (options?.permissionMode === 'strict') {
-            args.push('--permission-mode', 'plan');
-        }
-        else {
-            args.push('--dangerously-skip-permissions');
-        }
+        const args = ['-p', '--output-format', 'json', '--dangerously-skip-permissions'];
         if (options?.resumeSession)
             args.push('--resume', options.resumeSession);
         if (options?.model)
@@ -303,7 +302,24 @@ export class ClaudeCodeExecutorService extends AbstractAgentExecutor {
         return args;
     }
     buildSpawnOptions(options) {
-        return this.buildBaseSpawnOptions(options?.cwd);
+        const spawnOpts = {};
+        if (options?.cwd)
+            spawnOpts.cwd = options.cwd;
+        // Explicitly pipe stdio so streams are available even when parent disconnects
+        spawnOpts.stdio = ['pipe', 'pipe', 'pipe'];
+        // On Windows: windowsHide=true to prevent blank console windows.
+        // Do NOT use shell=true — it causes DEP0190 argument escaping issues
+        // and mangles prompts with special characters. The claude CLI is a
+        // native .exe, so spawn() finds it on PATH without shell.
+        if (IS_WINDOWS) {
+            spawnOpts.windowsHide = true;
+        }
+        // Strip CLAUDECODE env var to prevent "nested session" error when shep
+        // is invoked from within a Claude Code session. The claude CLI checks for
+        // this variable and refuses to start if it's set.
+        const { CLAUDECODE: _, ...cleanEnv } = process.env;
+        spawnOpts.env = cleanEnv;
+        return spawnOpts;
     }
     parseJsonResult(stdout) {
         const trimmed = stdout.trim();

package/dist/packages/core/src/infrastructure/services/agents/common/executors/codex-cli-executor.service.d.ts

@@ -9,16 +9,20 @@
  */
 import type { AgentType, AgentFeature, AgentConfig } from '../../../../../domain/generated/output.js';
 import type { IAgentExecutor, AgentExecutionOptions, AgentExecutionResult, AgentExecutionStreamEvent } from '../../../../../application/ports/output/agents/agent-executor.interface.js';
-import { type SpawnFunction } from '../types.js';
-import { AbstractAgentExecutor } from './abstract-agent-executor.js';
+import type { SpawnFunction } from '../types.js';
 /**
  * Executor service for OpenAI Codex CLI agent.
  * Uses subprocess spawning to interact with the `codex` CLI.
  */
-export declare class CodexCliExecutorService extends AbstractAgentExecutor implements IAgentExecutor {
+export declare class CodexCliExecutorService implements IAgentExecutor {
+    private readonly spawn;
     private readonly authConfig?;
     readonly agentType: AgentType;
+    /** When true, suppresses debug logging (set per-call via options.silent) */
+    private silent;
     constructor(spawn: SpawnFunction, authConfig?: AgentConfig | undefined);
+    /** Debug logging — writes to stdout so it appears in the worker log file */
+    private log;
     supportsFeature(feature: AgentFeature): boolean;
     execute(prompt: string, options?: AgentExecutionOptions): Promise<AgentExecutionResult>;
     executeStream(prompt: string, options?: AgentExecutionOptions): AsyncIterable<AgentExecutionStreamEvent>;

package/dist/packages/core/src/infrastructure/services/agents/common/executors/codex-cli-executor.service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"codex-cli-executor.service.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/common/executors/codex-cli-executor.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAKH,OAAO,KAAK,EACV,SAAS,EACT,YAAY,EACZ,WAAW,EACZ,MAAM,2CAA2C,CAAC;AACnD,OAAO,KAAK,EACV,cAAc,EACd,qBAAqB,EACrB,oBAAoB,EAEpB,yBAAyB,EAC1B,MAAM,4EAA4E,CAAC;AACpF,OAAO,EAA0B,KAAK,aAAa,EAAE,MAAM,aAAa,CAAC;AACzE,OAAO,EAAE,qBAAqB,EAAE,MAAM,8BAA8B,CAAC;AAsBrE;;;GAGG;AACH,qBAAa,uBAAwB,SAAQ,qBAAsB,YAAW,cAAc;IAKxF,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAC;IAJ9B,QAAQ,CAAC,SAAS,EAAE,SAAS,CAA4B;gBAGvD,KAAK,EAAE,aAAa,EACH,UAAU,CAAC,EAAE,WAAW,YAAA;IAK3C,eAAe,CAAC,OAAO,EAAE,YAAY,GAAG,OAAO;IAIzC,OAAO,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,qBAAqB,GAAG,OAAO,CAAC,oBAAoB,CAAC;IAiKtF,aAAa,CAClB,MAAM,EAAE,MAAM,EACd,OAAO,CAAC,EAAE,qBAAqB,GAC9B,aAAa,CAAC,yBAAyB,CAAC;IAmO3C;;;;OAIG;IACH,wDAAwD;IACxD,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,kBAAkB,CAAyC;IAEnF,OAAO,CAAC,cAAc;IAkJtB;;;;;OAKG;IACH,OAAO,CAAC,SAAS;IAgCjB,OAAO,CAAC,iBAAiB;IAYzB;;OAEG;IACH,OAAO,CAAC,YAAY;IAUpB;;;OAGG;IACH,OAAO,CAAC,gBAAgB;IAgBxB;;;OAGG;IACH,OAAO,CAAC,eAAe;IAkBvB;;;OAGG;IACH,OAAO,CAAC,sBAAsB;CAU/B"}
+{"version":3,"file":"codex-cli-executor.service.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/common/executors/codex-cli-executor.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAKH,OAAO,KAAK,EACV,SAAS,EACT,YAAY,EACZ,WAAW,EACZ,MAAM,2CAA2C,CAAC;AACnD,OAAO,KAAK,EACV,cAAc,EACd,qBAAqB,EACrB,oBAAoB,EAEpB,yBAAyB,EAC1B,MAAM,4EAA4E,CAAC;AACpF,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AAuBjD;;;GAGG;AACH,qBAAa,uBAAwB,YAAW,cAAc;IAO1D,OAAO,CAAC,QAAQ,CAAC,KAAK;IACtB,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAC;IAP9B,QAAQ,CAAC,SAAS,EAAE,SAAS,CAA4B;IAEzD,4EAA4E;IAC5E,OAAO,CAAC,MAAM,CAAS;gBAGJ,KAAK,EAAE,aAAa,EACpB,UAAU,CAAC,EAAE,WAAW,YAAA;IAG3C,4EAA4E;IAC5E,OAAO,CAAC,GAAG;IAMX,eAAe,CAAC,OAAO,EAAE,YAAY,GAAG,OAAO;IAIzC,OAAO,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,qBAAqB,GAAG,OAAO,CAAC,oBAAoB,CAAC;IA+JtF,aAAa,CAClB,MAAM,EAAE,MAAM,EACd,OAAO,CAAC,EAAE,qBAAqB,GAC9B,aAAa,CAAC,yBAAyB,CAAC;IAiO3C;;;;OAIG;IACH,wDAAwD;IACxD,OAAO,CAAC,MAAM,CAAC,QAAQ,CAAC,kBAAkB,CAAyC;IAEnF,OAAO,CAAC,cAAc;IAkJtB;;;;;OAKG;IACH,OAAO,CAAC,SAAS;IA4BjB,OAAO,CAAC,iBAAiB;IA0BzB;;OAEG;IACH,OAAO,CAAC,YAAY;IAUpB;;;OAGG;IACH,OAAO,CAAC,gBAAgB;IAgBxB;;;OAGG;IACH,OAAO,CAAC,eAAe;IAkBvB;;;OAGG;IACH,OAAO,CAAC,sBAAsB;CAU/B"}

package/dist/packages/core/src/infrastructure/services/agents/common/executors/codex-cli-executor.service.js

@@ -10,8 +10,7 @@
 import * as fs from 'node:fs';
 import * as os from 'node:os';
 import * as path from 'node:path';
-import { MAX_STDERR_BUFFER_SIZE } from '../types.js';
-import { AbstractAgentExecutor } from './abstract-agent-executor.js';
+import { getCurrentPhase, getLogPrefix } from '../../feature-agent/log-context.js';
 /** Features supported by Codex CLI */
 const SUPPORTED_FEATURES = new Set([
     'session-resume',
@@ -34,13 +33,23 @@ const FATAL_STDERR_PATTERNS = [
  * Executor service for OpenAI Codex CLI agent.
  * Uses subprocess spawning to interact with the `codex` CLI.
  */
-export class CodexCliExecutorService extends AbstractAgentExecutor {
+export class CodexCliExecutorService {
+    spawn;
     authConfig;
     agentType = 'codex-cli';
+    /** When true, suppresses debug logging (set per-call via options.silent) */
+    silent = false;
     constructor(spawn, authConfig) {
-        super(spawn);
+        this.spawn = spawn;
         this.authConfig = authConfig;
     }
+    /** Debug logging — writes to stdout so it appears in the worker log file */
+    log(message) {
+        if (this.silent)
+            return;
+        const ts = new Date().toISOString();
+        process.stdout.write(`[${ts}] ${getCurrentPhase()}${getLogPrefix()}${message}\n`);
+    }
     supportsFeature(feature) {
         return SUPPORTED_FEATURES.has(feature);
     }
@@ -120,9 +129,7 @@ export class CodexCliExecutorService extends AbstractAgentExecutor {
                 });
                 proc.stderr?.on('data', (chunk) => {
                     const data = chunk.toString();
-                    if (stderr.length < MAX_STDERR_BUFFER_SIZE) {
-                        stderr += data;
-                    }
+                    stderr += data;
                     this.log(`stderr: ${data.trimEnd()}`);
                 });
                 proc.on('error', (error) => {
@@ -327,9 +334,7 @@ export class CodexCliExecutorService extends AbstractAgentExecutor {
                 }
             });
             proc.stderr?.on('data', (chunk) => {
-                if (stderr.length < MAX_STDERR_BUFFER_SIZE) {
-                    stderr += chunk.toString();
-                }
+                stderr += chunk.toString();
             });
             proc.on('error', (err) => {
                 if (timeoutId)
@@ -530,14 +535,10 @@ export class CodexCliExecutorService extends AbstractAgentExecutor {
      * For resume: `codex exec resume <threadId> "prompt" --json --sandbox danger-full-access ...`
      */
     buildArgs(prompt, options, tempSchemaPath) {
-        // Permission mode: strict uses --ask-for-approval always; default/autonomous use never
-        const approvalMode = options?.permissionMode === 'strict' ? 'always' : 'never';
         const baseFlags = [
             '--json',
             '--sandbox',
             'danger-full-access',
-            '--ask-for-approval',
-            approvalMode,
             '--skip-git-repo-check',
             '--color',
             'never',
@@ -557,11 +558,23 @@ export class CodexCliExecutorService extends AbstractAgentExecutor {
         return ['exec', '-', ...baseFlags];
     }
     buildSpawnOptions(_options) {
-        const spawnOpts = this.buildBaseSpawnOptions();
+        const spawnOpts = {};
+        // Explicitly pipe stdio so streams are available even when parent disconnects
+        spawnOpts.stdio = ['pipe', 'pipe', 'pipe'];
+        // On Windows: windowsHide=true to prevent blank console windows.
+        // Codex CLI is a native Rust binary, so shell=true is NOT needed.
+        if (process.platform === 'win32') {
+            spawnOpts.windowsHide = true;
+        }
+        // Strip CLAUDECODE env var to prevent "nested session" error when shep
+        // is invoked from within a Claude Code session.
+        const { CLAUDECODE: _, ...cleanEnv } = process.env;
         // Inject CODEX_API_KEY when using token auth
         if (this.authConfig?.authMethod === 'token' && this.authConfig.token) {
-            const baseEnv = spawnOpts.env;
-            spawnOpts.env = { ...baseEnv, CODEX_API_KEY: this.authConfig.token };
+            spawnOpts.env = { ...cleanEnv, CODEX_API_KEY: this.authConfig.token };
+        }
+        else {
+            spawnOpts.env = cleanEnv;
         }
         return spawnOpts;
     }

package/dist/packages/core/src/infrastructure/services/agents/common/executors/copilot-cli-executor.service.d.ts

@@ -15,16 +15,20 @@
  */
 import type { AgentType, AgentFeature, AgentConfig } from '../../../../../domain/generated/output.js';
 import type { IAgentExecutor, AgentExecutionOptions, AgentExecutionResult, AgentExecutionStreamEvent } from '../../../../../application/ports/output/agents/agent-executor.interface.js';
-import { type SpawnFunction } from '../types.js';
-import { AbstractAgentExecutor } from './abstract-agent-executor.js';
+import type { SpawnFunction } from '../types.js';
 /**
  * Executor service for GitHub Copilot CLI agent.
  * Uses subprocess spawning to interact with the `copilot` CLI.
  */
-export declare class CopilotCliExecutorService extends AbstractAgentExecutor implements IAgentExecutor {
+export declare class CopilotCliExecutorService implements IAgentExecutor {
+    private readonly spawn;
     private readonly authConfig?;
     readonly agentType: AgentType;
+    /** When true, suppresses debug logging (set per-call via options.silent) */
+    private silent;
     constructor(spawn: SpawnFunction, authConfig?: AgentConfig | undefined);
+    /** Debug logging — writes to stdout so it appears in the worker log file */
+    private log;
     supportsFeature(feature: AgentFeature): boolean;
     execute(prompt: string, options?: AgentExecutionOptions): Promise<AgentExecutionResult>;
     executeStream(prompt: string, options?: AgentExecutionOptions): AsyncIterable<AgentExecutionStreamEvent>;

package/dist/packages/core/src/infrastructure/services/agents/common/executors/copilot-cli-executor.service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"copilot-cli-executor.service.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/common/executors/copilot-cli-executor.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAEH,OAAO,KAAK,EACV,SAAS,EACT,YAAY,EACZ,WAAW,EACZ,MAAM,2CAA2C,CAAC;AACnD,OAAO,KAAK,EACV,cAAc,EACd,qBAAqB,EACrB,oBAAoB,EAEpB,yBAAyB,EAC1B,MAAM,4EAA4E,CAAC;AACpF,OAAO,EAA0B,KAAK,aAAa,EAAE,MAAM,aAAa,CAAC;AACzE,OAAO,EAAE,qBAAqB,EAAE,MAAM,8BAA8B,CAAC;AA2DrE;;;GAGG;AACH,qBAAa,yBAA0B,SAAQ,qBAAsB,YAAW,cAAc;IAK1F,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAC;IAJ9B,QAAQ,CAAC,SAAS,EAAE,SAAS,CAA8B;gBAGzD,KAAK,EAAE,aAAa,EACH,UAAU,CAAC,EAAE,WAAW,YAAA;IAK3C,eAAe,CAAC,OAAO,EAAE,YAAY,GAAG,OAAO;IAIzC,OAAO,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,qBAAqB,GAAG,OAAO,CAAC,oBAAoB,CAAC;IA6ItF,aAAa,CAClB,MAAM,EAAE,MAAM,EACd,OAAO,CAAC,EAAE,qBAAqB,GAC9B,aAAa,CAAC,yBAAyB,CAAC;IAoL3C;;;OAGG;IACH,OAAO,CAAC,SAAS;IAkCjB;;OAEG;IACH,OAAO,CAAC,cAAc;IAiBtB;;;OAGG;IACH,OAAO,CAAC,mBAAmB;IAI3B,OAAO,CAAC,mBAAmB;YAUb,4BAA4B;IAuB1C,OAAO,CAAC,iBAAiB;IAIzB;;;OAGG;IACH,OAAO,CAAC,YAAY;IASpB;;;OAGG;IACH,OAAO,CAAC,eAAe;CAkBxB"}
+{"version":3,"file":"copilot-cli-executor.service.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/common/executors/copilot-cli-executor.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAEH,OAAO,KAAK,EACV,SAAS,EACT,YAAY,EACZ,WAAW,EACZ,MAAM,2CAA2C,CAAC;AACnD,OAAO,KAAK,EACV,cAAc,EACd,qBAAqB,EACrB,oBAAoB,EAEpB,yBAAyB,EAC1B,MAAM,4EAA4E,CAAC;AACpF,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AA4DjD;;;GAGG;AACH,qBAAa,yBAA0B,YAAW,cAAc;IAO5D,OAAO,CAAC,QAAQ,CAAC,KAAK;IACtB,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAC;IAP9B,QAAQ,CAAC,SAAS,EAAE,SAAS,CAA8B;IAE3D,4EAA4E;IAC5E,OAAO,CAAC,MAAM,CAAS;gBAGJ,KAAK,EAAE,aAAa,EACpB,UAAU,CAAC,EAAE,WAAW,YAAA;IAG3C,4EAA4E;IAC5E,OAAO,CAAC,GAAG;IAMX,eAAe,CAAC,OAAO,EAAE,YAAY,GAAG,OAAO;IAIzC,OAAO,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,qBAAqB,GAAG,OAAO,CAAC,oBAAoB,CAAC;IA2ItF,aAAa,CAClB,MAAM,EAAE,MAAM,EACd,OAAO,CAAC,EAAE,qBAAqB,GAC9B,aAAa,CAAC,yBAAyB,CAAC;IAkL3C;;;OAGG;IACH,OAAO,CAAC,SAAS;IAsBjB;;OAEG;IACH,OAAO,CAAC,cAAc;IAiBtB;;;OAGG;IACH,OAAO,CAAC,mBAAmB;IAI3B,OAAO,CAAC,mBAAmB;YAUb,4BAA4B;IAuB1C,OAAO,CAAC,iBAAiB;IAuBzB;;;OAGG;IACH,OAAO,CAAC,YAAY;IASpB;;;OAGG;IACH,OAAO,CAAC,eAAe;CAkBxB"}

package/dist/packages/core/src/infrastructure/services/agents/common/executors/copilot-cli-executor.service.js

@@ -13,8 +13,7 @@
  * Uses constructor dependency injection for the spawn function
  * to enable testability without mocking node:child_process directly.
  */
-import { MAX_STDERR_BUFFER_SIZE } from '../types.js';
-import { AbstractAgentExecutor } from './abstract-agent-executor.js';
+import { getCurrentPhase, getLogPrefix } from '../../feature-agent/log-context.js';
 import { randomUUID } from 'node:crypto';
 import { unlink, writeFile } from 'node:fs/promises';
 import { join } from 'node:path';
@@ -65,13 +64,23 @@ const LEGACY_MODEL_ALIASES = {
  * Executor service for GitHub Copilot CLI agent.
  * Uses subprocess spawning to interact with the `copilot` CLI.
  */
-export class CopilotCliExecutorService extends AbstractAgentExecutor {
+export class CopilotCliExecutorService {
+    spawn;
     authConfig;
     agentType = 'copilot-cli';
+    /** When true, suppresses debug logging (set per-call via options.silent) */
+    silent = false;
     constructor(spawn, authConfig) {
-        super(spawn);
+        this.spawn = spawn;
         this.authConfig = authConfig;
     }
+    /** Debug logging — writes to stdout so it appears in the worker log file */
+    log(message) {
+        if (this.silent)
+            return;
+        const ts = new Date().toISOString();
+        process.stdout.write(`[${ts}] ${getCurrentPhase()}${getLogPrefix()}${message}\n`);
+    }
     supportsFeature(feature) {
         return SUPPORTED_FEATURES.has(feature);
     }
@@ -145,9 +154,7 @@ export class CopilotCliExecutorService extends AbstractAgentExecutor {
             });
             proc.stderr?.on('data', (chunk) => {
                 const data = chunk.toString();
-                if (stderr.length < MAX_STDERR_BUFFER_SIZE) {
-                    stderr += data;
-                }
+                stderr += data;
                 this.log(`stderr: ${data.trimEnd()}`);
             });
             proc.on('error', (error) => {
@@ -314,9 +321,7 @@ export class CopilotCliExecutorService extends AbstractAgentExecutor {
             }
         });
         proc.stderr?.on('data', (chunk) => {
-            if (stderr.length < MAX_STDERR_BUFFER_SIZE) {
-                stderr += chunk.toString();
-            }
+            stderr += chunk.toString();
         });
         proc.on('error', (err) => {
             if (timeoutId)
@@ -368,24 +373,15 @@ export class CopilotCliExecutorService extends AbstractAgentExecutor {
      * Prompt is passed via -p flag (not stdin).
      */
     buildArgs(prompt, options) {
-        // Permission mode: strict omits --allow-all (requires confirmation); default/autonomous keep it
-        const flags = options?.permissionMode === 'strict'
-            ? BASE_FLAGS.filter((f) => f !== '--allow-all')
-            : [...BASE_FLAGS];
-        const args = ['-p', prompt, ...flags];
+        const args = ['-p', prompt, ...BASE_FLAGS];
         if (options?.model) {
             args.push('--model', this.normalizeModel(options.model));
         }
         if (options?.resumeSession)
             args.push(`--resume=${options.resumeSession}`);
-        // Copilot CLI has no --allowed-tools flag; it uses --allow-all for blanket permission.
-        // Individual tool restrictions cannot be expressed. Unlike Claude Code (--allowedTools)
-        // and Gemini CLI (--allowed-tools), Copilot only supports blanket --allow-all.
-        // Emit a visible console.warn so callers notice the limitation in logs.
+        // Unsupported options — log and ignore
         if (options?.allowedTools?.length) {
-            // eslint-disable-next-line no-console -- intentional: surface limitation visibly in all log sinks
-            console.warn(`[copilot-cli] allowedTools [${options.allowedTools.join(', ')}] cannot be passed to Copilot CLI — ` +
-                'it only supports blanket --allow-all (already set). Tool restrictions will NOT be enforced.');
+            this.log('allowedTools option is not supported by Copilot CLI — ignoring');
         }
         if (options?.systemPrompt) {
             this.log('systemPrompt option is not supported by Copilot CLI — ignoring');
@@ -449,7 +445,22 @@ export class CopilotCliExecutorService extends AbstractAgentExecutor {
         };
     }
     buildSpawnOptions(options) {
-        return this.buildBaseSpawnOptions(options?.cwd);
+        const spawnOpts = {};
+        if (options?.cwd)
+            spawnOpts.cwd = options.cwd;
+        // Explicitly pipe stdio so streams are available
+        spawnOpts.stdio = ['pipe', 'pipe', 'pipe'];
+        // On Windows: windowsHide=true to prevent blank console windows.
+        // Copilot CLI is a Node.js binary, so shell=true is NOT needed.
+        if (process.platform === 'win32') {
+            spawnOpts.windowsHide = true;
+        }
+        // Strip CLAUDECODE env var to prevent "nested session" error when shep
+        // is invoked from within a Claude Code session.
+        const { CLAUDECODE: _, ...cleanEnv } = process.env;
+        // Copilot CLI uses GitHub OAuth — no API key injection.
+        spawnOpts.env = cleanEnv;
+        return spawnOpts;
     }
     /**
      * Extract token usage from the Copilot CLI result event usage object.

package/dist/packages/core/src/infrastructure/services/agents/common/executors/cursor-executor.service.d.ts

@@ -10,15 +10,19 @@
  */
 import type { AgentType, AgentFeature } from '../../../../../domain/generated/output.js';
 import type { IAgentExecutor, AgentExecutionOptions, AgentExecutionResult, AgentExecutionStreamEvent } from '../../../../../application/ports/output/agents/agent-executor.interface.js';
-import { type SpawnFunction } from '../types.js';
-import { AbstractAgentExecutor } from './abstract-agent-executor.js';
+import type { SpawnFunction } from '../types.js';
 /**
  * Executor service for Cursor agent.
  * Uses subprocess spawning to interact with the `cursor-agent` CLI.
  */
-export declare class CursorExecutorService extends AbstractAgentExecutor implements IAgentExecutor {
+export declare class CursorExecutorService implements IAgentExecutor {
+    private readonly spawn;
     readonly agentType: AgentType;
+    /** When true, suppresses debug logging (set per-call via options.silent) */
+    private silent;
     constructor(spawn: SpawnFunction);
+    /** Debug logging — writes to stdout so it appears in the worker log file */
+    private log;
     supportsFeature(feature: AgentFeature): boolean;
     execute(prompt: string, options?: AgentExecutionOptions): Promise<AgentExecutionResult>;
     executeStream(prompt: string, options?: AgentExecutionOptions): AsyncIterable<AgentExecutionStreamEvent>;