@shepai/cli 1.129.4-pr412.cbe0d37 → 1.130.0-pr413.0773bb9

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

@@ -26,7 +26,7 @@ import { injectable, inject } from 'tsyringe';
  */
 const AGENT_BINARY_MAP = {
     'claude-code': 'claude',
-    cursor: 'agent',
+    cursor: 'cursor-agent',
     'gemini-cli': 'gemini',
 };
 /**

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

@@ -2,7 +2,7 @@
  * Cursor Executor Service
  *
  * Infrastructure implementation of IAgentExecutor for the Cursor agent.
- * Executes prompts via the `agent` CLI subprocess with JSON and stream-json
+ * Executes prompts via the `cursor-agent` CLI subprocess with JSON and stream-json
  * output formats.
  *
  * Uses constructor dependency injection for the spawn function
@@ -13,7 +13,7 @@ import type { IAgentExecutor, AgentExecutionOptions, AgentExecutionResult, Agent
 import type { SpawnFunction } from '../types.js';
 /**
  * Executor service for Cursor agent.
- * Uses subprocess spawning to interact with the `agent` CLI.
+ * Uses subprocess spawning to interact with the `cursor-agent` CLI.
  */
 export declare class CursorExecutorService implements IAgentExecutor {
     private readonly spawn;
@@ -35,13 +35,13 @@ export declare class CursorExecutorService implements IAgentExecutor {
     /**
      * Spawn the agent process, handling Windows specially via PowerShell.
      *
-     * On Windows, cursor CLI ships as `agent.cmd` which requires `shell: true`,
+     * On Windows, cursor CLI ships as `cursor-agent.cmd` which requires `shell: true`,
      * but cmd.exe mangles long `-p` arguments (8191-char limit + special chars).
-     * Solution: write prompt to a temp file, invoke agent via PowerShell which
+     * Solution: write prompt to a temp file, invoke cursor-agent via PowerShell which
      * reads the file and passes the content as `-p`. PowerShell handles long
      * strings natively (32K limit) and doesn't mangle arguments.
      *
-     * On Linux/macOS, spawn `agent` directly — no shell needed.
+     * On Linux/macOS, spawn `cursor-agent` directly — no shell needed.
      */
     private spawnAgent;
     private parseStreamLine;

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

@@ -2,7 +2,7 @@
  * Cursor Executor Service
  *
  * Infrastructure implementation of IAgentExecutor for the Cursor agent.
- * Executes prompts via the `agent` CLI subprocess with JSON and stream-json
+ * Executes prompts via the `cursor-agent` CLI subprocess with JSON and stream-json
  * output formats.
  *
  * Uses constructor dependency injection for the spawn function
@@ -32,7 +32,7 @@ function toCursorModelName(model) {
 const SUPPORTED_FEATURES = new Set(['session-resume', 'streaming']);
 /**
  * Executor service for Cursor agent.
- * Uses subprocess spawning to interact with the `agent` CLI.
+ * Uses subprocess spawning to interact with the `cursor-agent` CLI.
  */
 export class CursorExecutorService {
     spawn;
@@ -312,13 +312,13 @@ export class CursorExecutorService {
     /**
      * Spawn the agent process, handling Windows specially via PowerShell.
      *
-     * On Windows, cursor CLI ships as `agent.cmd` which requires `shell: true`,
+     * On Windows, cursor CLI ships as `cursor-agent.cmd` which requires `shell: true`,
      * but cmd.exe mangles long `-p` arguments (8191-char limit + special chars).
-     * Solution: write prompt to a temp file, invoke agent via PowerShell which
+     * Solution: write prompt to a temp file, invoke cursor-agent via PowerShell which
      * reads the file and passes the content as `-p`. PowerShell handles long
      * strings natively (32K limit) and doesn't mangle arguments.
      *
-     * On Linux/macOS, spawn `agent` directly — no shell needed.
+     * On Linux/macOS, spawn `cursor-agent` directly — no shell needed.
      */
     spawnAgent(prompt, args, options) {
         const { CLAUDECODE: _, ...cleanEnv } = process.env;
@@ -330,7 +330,7 @@ export class CursorExecutorService {
             // Build the agent args WITHOUT -p and the prompt (they go via temp file)
             const agentFlags = args.filter((a) => a !== '-p' && a !== prompt).join(' ');
             const safePath = tmpFile.replace(/'/g, "''");
-            const psCmd = `$p = Get-Content -Raw '${safePath}'; & agent ${agentFlags} -p $p`;
+            const psCmd = `$p = Get-Content -Raw '${safePath}'; & cursor-agent ${agentFlags} -p $p`;
             this.log(`Windows PowerShell mode: wrote ${prompt.length} chars to ${tmpFile}`);
             this.log(`PS command: ${psCmd.replace(prompt, `<${prompt.length} chars>`)}`);
             const proc = this.spawn('powershell.exe', ['-NoProfile', '-NonInteractive', '-Command', psCmd], {
@@ -351,9 +351,9 @@ export class CursorExecutorService {
         };
         if (cwd)
             spawnOpts.cwd = cwd;
-        this.log(`Spawning: agent ${args.map((a) => (a.length > 80 ? `${a.slice(0, 77)}...` : a)).join(' ')}`);
+        this.log(`Spawning: cursor-agent ${args.map((a) => (a.length > 80 ? `${a.slice(0, 77)}...` : a)).join(' ')}`);
         this.log(`Spawn cwd: ${cwd ?? '(inherited)'}`);
-        const proc = this.spawn('agent', args, spawnOpts);
+        const proc = this.spawn('cursor-agent', args, spawnOpts);
         this.log(`Subprocess PID: ${proc.pid ?? 'undefined (spawn may have failed)'}`);
         if (proc.stdin)
             proc.stdin.end();

package/dist/packages/core/src/infrastructure/services/deployment/deployment.service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"deployment.service.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/infrastructure/services/deployment/deployment.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,EAAE,KAAK,EAAqB,MAAM,oBAAoB,CAAC;AAE9D,OAAO,KAAK,QAAQ,MAAM,gBAAgB,CAAC;AAO3C,OAAO,KAAK,EACV,kBAAkB,EAClB,gBAAgB,EAChB,QAAQ,EACT,MAAM,qEAAqE,CAAC;AAC7E,OAAO,EAAE,eAAe,EAAE,MAAM,wBAAwB,CAAC;AAgCzD,MAAM,WAAW,qBAAqB;IACpC,KAAK,EAAE,OAAO,KAAK,CAAC;IACpB,eAAe,EAAE,OAAO,eAAe,CAAC;IACxC,IAAI,EAAE,CAAC,GAAG,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC,OAAO,GAAG,MAAM,KAAK,IAAI,CAAC;IAC7D,OAAO,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC;CACnC;AA6BD,qBAAa,iBAAkB,YAAW,kBAAkB;IAC1D,OAAO,CAAC,QAAQ,CAAC,WAAW,CAAsC;IAClE,OAAO,CAAC,QAAQ,CAAC,IAAI,CAAwB;IAC7C,OAAO,CAAC,QAAQ,CAAC,OAAO,CAAsB;IAC9C,OAAO,CAAC,EAAE,CAAkC;gBAEhC,IAAI,GAAE,OAAO,CAAC,qBAAqB,CAAM;IAIrD;;OAEG;IACH,WAAW,CAAC,EAAE,EAAE,QAAQ,CAAC,QAAQ,GAAG,IAAI;IAIxC;;;OAGG;IACH,UAAU,IAAI,IAAI;IA4ClB;;;OAGG;IACH,KAAK,CAAC,QAAQ,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,EAAE,UAAU,SAAe,GAAG,IAAI;IAqF5E;;;;OAIG;IACH,SAAS,CAAC,QAAQ,EAAE,MAAM,GAAG,gBAAgB,GAAG,IAAI;IA4CpD;;OAEG;IACG,IAAI,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IA0D3C;;OAEG;IACH,OAAO,IAAI,IAAI;IAef;;OAEG;IACH,OAAO,CAAC,QAAQ,EAAE,MAAM,GAAG,QAAQ,EAAE,GAAG,IAAI;IAM5C;;OAEG;IACH,EAAE,CAAC,KAAK,EAAE,KAAK,EAAE,OAAO,EAAE,CAAC,KAAK,EAAE,QAAQ,KAAK,IAAI,GAAG,IAAI;IAI1D;;OAEG;IACH,GAAG,CAAC,KAAK,EAAE,KAAK,EAAE,OAAO,EAAE,CAAC,KAAK,EAAE,QAAQ,KAAK,IAAI,GAAG,IAAI;IAM3D,OAAO,CAAC,QAAQ;IAuBhB,OAAO,CAAC,aAAa;IAWrB,OAAO,CAAC,QAAQ;IAShB,OAAO,CAAC,MAAM;IAed,OAAO,CAAC,WAAW;IAQnB;;;OAGG;IACH,OAAO,CAAC,oBAAoB;YA4Dd,aAAa;IAW3B,OAAO,CAAC,WAAW;CASpB"}
+{"version":3,"file":"deployment.service.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/infrastructure/services/deployment/deployment.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,EAAE,KAAK,EAAmC,MAAM,oBAAoB,CAAC;AAE5E,OAAO,KAAK,QAAQ,MAAM,gBAAgB,CAAC;AAK3C,OAAO,KAAK,EACV,kBAAkB,EAClB,gBAAgB,EAChB,QAAQ,EACT,MAAM,qEAAqE,CAAC;AAC7E,OAAO,EAAE,eAAe,EAAE,MAAM,wBAAwB,CAAC;AAiCzD,MAAM,WAAW,qBAAqB;IACpC,KAAK,EAAE,OAAO,KAAK,CAAC;IACpB,eAAe,EAAE,OAAO,eAAe,CAAC;IACxC,IAAI,EAAE,CAAC,GAAG,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC,OAAO,GAAG,MAAM,KAAK,IAAI,CAAC;IAC7D,OAAO,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC;CACnC;AA0CD,qBAAa,iBAAkB,YAAW,kBAAkB;IAC1D,OAAO,CAAC,QAAQ,CAAC,WAAW,CAAsC;IAClE,OAAO,CAAC,QAAQ,CAAC,IAAI,CAAwB;IAC7C,OAAO,CAAC,QAAQ,CAAC,OAAO,CAAsB;IAC9C,OAAO,CAAC,EAAE,CAAkC;gBAEhC,IAAI,GAAE,OAAO,CAAC,qBAAqB,CAAM;IAIrD;;OAEG;IACH,WAAW,CAAC,EAAE,EAAE,QAAQ,CAAC,QAAQ,GAAG,IAAI;IAIxC;;;OAGG;IACH,UAAU,IAAI,IAAI;IA4ClB;;;OAGG;IACH,KAAK,CAAC,QAAQ,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,EAAE,UAAU,SAAe,GAAG,IAAI;IA0G5E;;;;OAIG;IACH,SAAS,CAAC,QAAQ,EAAE,MAAM,GAAG,gBAAgB,GAAG,IAAI;IA4CpD;;OAEG;IACG,IAAI,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IA0D3C;;OAEG;IACH,OAAO,IAAI,IAAI;IAef;;OAEG;IACH,OAAO,CAAC,QAAQ,EAAE,MAAM,GAAG,QAAQ,EAAE,GAAG,IAAI;IAM5C;;OAEG;IACH,EAAE,CAAC,KAAK,EAAE,KAAK,EAAE,OAAO,EAAE,CAAC,KAAK,EAAE,QAAQ,KAAK,IAAI,GAAG,IAAI;IAI1D;;OAEG;IACH,GAAG,CAAC,KAAK,EAAE,KAAK,EAAE,OAAO,EAAE,CAAC,KAAK,EAAE,QAAQ,KAAK,IAAI,GAAG,IAAI;IAM3D,OAAO,CAAC,QAAQ;IAuBhB,OAAO,CAAC,aAAa;IAWrB,OAAO,CAAC,QAAQ;IAShB,OAAO,CAAC,MAAM;IAed,OAAO,CAAC,WAAW;IAQnB;;;OAGG;IACH,OAAO,CAAC,oBAAoB;YA4Dd,aAAa;IAW3B,OAAO,CAAC,WAAW;CASpB"}

package/dist/packages/core/src/infrastructure/services/deployment/deployment.service.js

@@ -9,18 +9,17 @@
  * On startup, call `recoverAll()` to reconcile the DB with actual
  * process state (clean up dead PIDs, re-adopt live ones).
  */
-import { spawn } from 'node:child_process';
+import { spawn, execFileSync } from 'node:child_process';
 import { EventEmitter } from 'node:events';
-// NOTE: We intentionally do NOT use tree-kill here. tree-kill traverses
-// /proc to find child processes which can accidentally kill parent processes
-// (including the Shep web UI). Instead, we use process.kill(-pid) to send
-// signals to the process GROUP (since we spawn with detached: true, the
-// child gets its own process group via setsid()).
+// On Unix we use process.kill(-pid) to send signals to the process GROUP
+// (detached: true puts the child in its own group via setsid()).
+// On Windows, negative PIDs are not supported — use taskkill /T for tree kill.
 import { DeploymentState } from '../../../domain/generated/output.js';
 import { detectDevScript } from './detect-dev-script.js';
 import { createDeploymentLogger } from './deployment-logger.js';
 import { parsePort } from './parse-port.js';
 import { LogRingBuffer } from './log-ring-buffer.js';
+import { IS_WINDOWS } from '../../platform.js';
 const log = createDeploymentLogger('[DeploymentService]');
 const POLL_INTERVAL_MS = 200;
 const MAX_WAIT_MS = 5000;
@@ -28,6 +27,20 @@ const defaultDeps = {
     spawn,
     detectDevScript,
     kill: (pid, signal) => {
+        if (IS_WINDOWS) {
+            // On Windows, negative PIDs are not supported. Use taskkill /T to
+            // kill the entire process tree (shell + child dev server).
+            try {
+                execFileSync('taskkill', ['/F', '/T', '/PID', String(pid)], {
+                    stdio: 'ignore',
+                    windowsHide: true,
+                });
+            }
+            catch {
+                // Process already dead
+            }
+            return;
+        }
         try {
             // Kill the entire process group (negative PID) — safe because
             // detached: true puts the child in its own group via setsid().
@@ -132,13 +145,34 @@ export class DeploymentService {
             throw new Error(detection.error);
         }
         // Build spawn args based on package manager
-        const { packageManager, scriptName, command } = detection;
+        const { packageManager, scriptName, command, needsInstall } = detection;
         const args = packageManager === 'npm' ? ['run', scriptName] : [scriptName];
+        // Install dependencies if node_modules is missing (e.g. fresh worktree)
+        if (needsInstall) {
+            log.info(`node_modules missing in "${targetPath}" — running ${packageManager} install`);
+            try {
+                execFileSync(packageManager, ['install'], {
+                    cwd: targetPath,
+                    shell: true,
+                    stdio: 'ignore',
+                    ...(IS_WINDOWS ? { windowsHide: true } : {}),
+                });
+                log.info('Dependencies installed successfully');
+            }
+            catch (err) {
+                log.error(`Dependency install failed: ${err}`);
+                throw new Error(`Failed to install dependencies: ${err}`);
+            }
+        }
         log.info(`Spawning dev server: command="${command}", packageManager="${packageManager}", scriptName="${scriptName}", cwd="${targetPath}"`);
         const child = this.deps.spawn(packageManager, args, {
             shell: true,
             cwd: targetPath,
-            detached: true,
+            // On Unix, detached: true creates a process group via setsid() so we can
+            // kill the entire tree with process.kill(-pid). On Windows this flag causes
+            // CREATE_NEW_CONSOLE which opens a visible terminal window and disconnects
+            // stdout/stderr — we don't need it because taskkill /F /T handles tree kill.
+            ...(IS_WINDOWS ? { windowsHide: true } : { detached: true }),
             stdio: ['ignore', 'pipe', 'pipe'],
         });
         if (!child.pid) {

package/dist/packages/core/src/infrastructure/services/deployment/detect-dev-script.d.ts

@@ -10,6 +10,7 @@ export interface DetectDevScriptSuccess {
     packageManager: string;
     scriptName: string;
     command: string;
+    needsInstall: boolean;
 }
 export interface DetectDevScriptError {
     success: false;

package/dist/packages/core/src/infrastructure/services/deployment/detect-dev-script.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"detect-dev-script.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/infrastructure/services/deployment/detect-dev-script.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAgBH,MAAM,WAAW,sBAAsB;IACrC,OAAO,EAAE,IAAI,CAAC;IACd,cAAc,EAAE,MAAM,CAAC;IACvB,UAAU,EAAE,MAAM,CAAC;IACnB,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,oBAAoB;IACnC,OAAO,EAAE,KAAK,CAAC;IACf,KAAK,EAAE,MAAM,CAAC;CACf;AAED,MAAM,MAAM,qBAAqB,GAAG,sBAAsB,GAAG,oBAAoB,CAAC;AAIlF;;;;;GAKG;AACH,wBAAgB,eAAe,CAAC,OAAO,EAAE,MAAM,GAAG,qBAAqB,CAuCtE"}
+{"version":3,"file":"detect-dev-script.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/infrastructure/services/deployment/detect-dev-script.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAgBH,MAAM,WAAW,sBAAsB;IACrC,OAAO,EAAE,IAAI,CAAC;IACd,cAAc,EAAE,MAAM,CAAC;IACvB,UAAU,EAAE,MAAM,CAAC;IACnB,OAAO,EAAE,MAAM,CAAC;IAChB,YAAY,EAAE,OAAO,CAAC;CACvB;AAED,MAAM,WAAW,oBAAoB;IACnC,OAAO,EAAE,KAAK,CAAC;IACf,KAAK,EAAE,MAAM,CAAC;CACf;AAED,MAAM,MAAM,qBAAqB,GAAG,sBAAsB,GAAG,oBAAoB,CAAC;AAIlF;;;;;GAKG;AACH,wBAAgB,eAAe,CAAC,OAAO,EAAE,MAAM,GAAG,qBAAqB,CAwCtE"}

package/dist/packages/core/src/infrastructure/services/deployment/detect-dev-script.js

@@ -50,8 +50,9 @@ export function detectDevScript(dirPath) {
     const packageManager = detectPackageManager(dirPath);
     // Build the command — pnpm/yarn use `<pm> <script>`, npm uses `npm run <script>`
     const command = packageManager === 'npm' ? `npm run ${scriptName}` : `${packageManager} ${scriptName}`;
-    log.info(`detected — packageManager="${packageManager}", scriptName="${scriptName}", command="${command}"`);
-    return { success: true, packageManager, scriptName, command };
+    const needsInstall = !existsSync(join(dirPath, 'node_modules'));
+    log.info(`detected — packageManager="${packageManager}", scriptName="${scriptName}", command="${command}", needsInstall=${needsInstall}`);
+    return { success: true, packageManager, scriptName, command, needsInstall };
 }
 /**
  * Detect the package manager by checking for lockfile presence.

package/dist/src/presentation/web/app/actions/check-agent-auth.js

@@ -21,7 +21,7 @@ const AGENT_TOOL_MAP = {
 };
 const AGENT_BINARY_MAP = {
     'claude-code': 'claude',
-    cursor: 'agent',
+    cursor: 'cursor-agent',
     'gemini-cli': 'gemini',
 };
 /**