@shepai/cli 1.103.1 → 1.104.0-pr299.a2cc06c

package/dist/packages/core/src/application/use-cases/upgrade/upgrade-cli.use-case.d.ts

@@ -0,0 +1,27 @@
+/**
+ * UpgradeCliUseCase
+ *
+ * Executes a self-upgrade of @shepai/cli to the latest published version.
+ * Mirrors the CLI upgrade command logic but designed for use from the web layer.
+ *
+ * Flow:
+ *   1. Check latest version from npm registry (fail-open)
+ *   2. Compare with current version — return early if up to date
+ *   3. Run `npm i -g @shepai/cli@latest` with streamed output
+ *   4. Return result with old/new version info
+ */
+import type { IVersionService } from '../../ports/output/services/version-service.interface.js';
+export interface UpgradeResult {
+    status: 'up-to-date' | 'upgraded' | 'error';
+    currentVersion: string;
+    latestVersion: string | null;
+    errorMessage?: string;
+}
+export declare class UpgradeCliUseCase {
+    private readonly versionService;
+    constructor(versionService: IVersionService);
+    execute(onOutput?: (data: string) => void): Promise<UpgradeResult>;
+    private getLatestVersion;
+    private runNpmInstall;
+}
+//# sourceMappingURL=upgrade-cli.use-case.d.ts.map

package/dist/packages/core/src/application/use-cases/upgrade/upgrade-cli.use-case.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"upgrade-cli.use-case.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/application/use-cases/upgrade/upgrade-cli.use-case.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAIH,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,0DAA0D,CAAC;AAEhG,MAAM,WAAW,aAAa;IAC5B,MAAM,EAAE,YAAY,GAAG,UAAU,GAAG,OAAO,CAAC;IAC5C,cAAc,EAAE,MAAM,CAAC;IACvB,aAAa,EAAE,MAAM,GAAG,IAAI,CAAC;IAC7B,YAAY,CAAC,EAAE,MAAM,CAAC;CACvB;AAID,qBACa,iBAAiB;IAG1B,OAAO,CAAC,QAAQ,CAAC,cAAc;gBAAd,cAAc,EAAE,eAAe;IAG5C,OAAO,CAAC,QAAQ,CAAC,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,IAAI,GAAG,OAAO,CAAC,aAAa,CAAC;IAkCxE,OAAO,CAAC,gBAAgB;IA6CxB,OAAO,CAAC,aAAa;CAuBtB"}

package/dist/packages/core/src/application/use-cases/upgrade/upgrade-cli.use-case.js

@@ -0,0 +1,128 @@
+/**
+ * UpgradeCliUseCase
+ *
+ * Executes a self-upgrade of @shepai/cli to the latest published version.
+ * Mirrors the CLI upgrade command logic but designed for use from the web layer.
+ *
+ * Flow:
+ *   1. Check latest version from npm registry (fail-open)
+ *   2. Compare with current version — return early if up to date
+ *   3. Run `npm i -g @shepai/cli@latest` with streamed output
+ *   4. Return result with old/new version info
+ */
+var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {
+    var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
+    if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
+    else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
+    return c > 3 && r && Object.defineProperty(target, key, r), r;
+};
+var __metadata = (this && this.__metadata) || function (k, v) {
+    if (typeof Reflect === "object" && typeof Reflect.metadata === "function") return Reflect.metadata(k, v);
+};
+var __param = (this && this.__param) || function (paramIndex, decorator) {
+    return function (target, key) { decorator(target, key, paramIndex); }
+};
+import { injectable, inject } from 'tsyringe';
+import { spawn } from 'node:child_process';
+const VERSION_CHECK_TIMEOUT_MS = 10_000;
+let UpgradeCliUseCase = class UpgradeCliUseCase {
+    versionService;
+    constructor(versionService) {
+        this.versionService = versionService;
+    }
+    async execute(onOutput) {
+        const { version: currentVersion } = this.versionService.getVersion();
+        // 1. Check latest version (fail-open)
+        const latestVersion = await this.getLatestVersion(onOutput);
+        // 2. Already up to date
+        if (latestVersion && latestVersion === currentVersion) {
+            onOutput?.(`Already up to date (v${currentVersion})\n`);
+            return { status: 'up-to-date', currentVersion, latestVersion };
+        }
+        // 3. Run upgrade
+        const target = latestVersion ? `v${latestVersion}` : 'latest';
+        onOutput?.(`Upgrading from v${currentVersion} to ${target}...\n`);
+        try {
+            const exitCode = await this.runNpmInstall(onOutput);
+            if (exitCode === 0) {
+                onOutput?.(`Successfully upgraded to ${target}\n`);
+                return { status: 'upgraded', currentVersion, latestVersion };
+            }
+            else {
+                const msg = `npm install exited with code ${exitCode}`;
+                onOutput?.(`Error: ${msg}\n`);
+                return { status: 'error', currentVersion, latestVersion, errorMessage: msg };
+            }
+        }
+        catch (error) {
+            const msg = error instanceof Error ? error.message : 'Installation failed';
+            onOutput?.(`Error: ${msg}\n`);
+            return { status: 'error', currentVersion, latestVersion, errorMessage: msg };
+        }
+    }
+    getLatestVersion(onOutput) {
+        return new Promise((resolve) => {
+            let output = '';
+            let settled = false;
+            const child = spawn('npm', ['view', '@shepai/cli', 'version'], {
+                stdio: ['ignore', 'pipe', 'pipe'],
+            });
+            const timeout = setTimeout(() => {
+                if (!settled) {
+                    settled = true;
+                    child.kill();
+                    onOutput?.('Version check timed out, proceeding with upgrade...\n');
+                    resolve(null);
+                }
+            }, VERSION_CHECK_TIMEOUT_MS);
+            child.stdout?.on('data', (data) => {
+                output += data.toString();
+            });
+            child.on('close', (code) => {
+                if (!settled) {
+                    settled = true;
+                    clearTimeout(timeout);
+                    if (code === 0 && output.trim()) {
+                        resolve(output.trim());
+                    }
+                    else {
+                        resolve(null);
+                    }
+                }
+            });
+            child.on('error', () => {
+                if (!settled) {
+                    settled = true;
+                    clearTimeout(timeout);
+                    onOutput?.('Could not check latest version\n');
+                    resolve(null);
+                }
+            });
+        });
+    }
+    runNpmInstall(onOutput) {
+        return new Promise((resolve, reject) => {
+            const child = spawn('npm', ['i', '-g', '@shepai/cli@latest'], {
+                stdio: ['ignore', 'pipe', 'pipe'],
+            });
+            child.stdout?.on('data', (data) => {
+                onOutput?.(data.toString());
+            });
+            child.stderr?.on('data', (data) => {
+                onOutput?.(data.toString());
+            });
+            child.on('close', (code) => {
+                resolve(code ?? 1);
+            });
+            child.on('error', (err) => {
+                reject(err);
+            });
+        });
+    }
+};
+UpgradeCliUseCase = __decorate([
+    injectable(),
+    __param(0, inject('IVersionService')),
+    __metadata("design:paramtypes", [Object])
+], UpgradeCliUseCase);
+export { UpgradeCliUseCase };

package/dist/packages/core/src/infrastructure/di/container.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"container.d.ts","sourceRoot":"","sources":["../../../../../../packages/core/src/infrastructure/di/container.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAEH,OAAO,kBAAkB,CAAC;AAC1B,OAAO,EAAE,SAAS,EAAE,MAAM,UAAU,CAAC;AAiHrC;;;;;;GAMG;AACH,wBAAsB,mBAAmB,IAAI,OAAO,CAAC,OAAO,SAAS,CAAC,CA0SrE;AAED;;;GAGG;AACH,wBAAgB,sBAAsB,IAAI,OAAO,CAEhD;AAED;;;GAGG;AACH,OAAO,EAAE,SAAS,EAAE,CAAC"}
+{"version":3,"file":"container.d.ts","sourceRoot":"","sources":["../../../../../../packages/core/src/infrastructure/di/container.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAEH,OAAO,kBAAkB,CAAC;AAC1B,OAAO,EAAE,SAAS,EAAE,MAAM,UAAU,CAAC;AAkHrC;;;;;;GAMG;AACH,wBAAsB,mBAAmB,IAAI,OAAO,CAAC,OAAO,SAAS,CAAC,CA8SrE;AAED;;;GAGG;AACH,wBAAgB,sBAAsB,IAAI,OAAO,CAEhD;AAED;;;GAGG;AACH,OAAO,EAAE,SAAS,EAAE,CAAC"}

package/dist/packages/core/src/infrastructure/di/container.js

@@ -77,6 +77,7 @@ import { DeleteRepositoryUseCase } from '../../application/use-cases/repositorie
 import { CheckAndUnblockFeaturesUseCase } from '../../application/use-cases/features/check-and-unblock-features.use-case.js';
 import { UpdateFeatureLifecycleUseCase } from '../../application/use-cases/features/update/update-feature-lifecycle.use-case.js';
 import { CleanupFeatureWorktreeUseCase } from '../../application/use-cases/features/cleanup-feature-worktree.use-case.js';
+import { UpgradeCliUseCase } from '../../application/use-cases/upgrade/upgrade-cli.use-case.js';
 // Session listing
 import { ClaudeCodeSessionRepository } from '../services/agents/sessions/claude-code-session.repository.js';
 import { StubSessionRepository } from '../services/agents/sessions/stub-session.repository.js';
@@ -279,6 +280,7 @@ export async function initializeContainer() {
     container.registerSingleton(CheckAndUnblockFeaturesUseCase);
     container.registerSingleton(UpdateFeatureLifecycleUseCase);
     container.registerSingleton(CleanupFeatureWorktreeUseCase);
+    container.registerSingleton(UpgradeCliUseCase);
     // Session repositories (per-AgentType string tokens)
     container.register(`IAgentSessionRepository:${AgentType.ClaudeCode}`, {
         useFactory: () => new ClaudeCodeSessionRepository(),
@@ -360,6 +362,9 @@ export async function initializeContainer() {
     container.register('CleanupFeatureWorktreeUseCase', {
         useFactory: (c) => c.resolve(CleanupFeatureWorktreeUseCase),
     });
+    container.register('UpgradeCliUseCase', {
+        useFactory: (c) => c.resolve(UpgradeCliUseCase),
+    });
     _initialized = true;
     return container;
 }

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

@@ -1 +1 @@
-{"version":3,"file":"dev-executor.service.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/common/executors/dev-executor.service.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,SAAS,EAAE,YAAY,EAAE,MAAM,2CAA2C,CAAC;AACzF,OAAO,KAAK,EACV,cAAc,EACd,qBAAqB,EACrB,oBAAoB,EACpB,yBAAyB,EAC1B,MAAM,4EAA4E,CAAC;AAmIpF,qBAAa,uBAAwB,YAAW,cAAc;IAC5D,QAAQ,CAAC,SAAS,EAAE,SAAS,CAAsB;IACnD,OAAO,CAAC,QAAQ,CAAC,OAAO,CAAS;;IAM3B,OAAO,CAAC,MAAM,EAAE,MAAM,EAAE,QAAQ,CAAC,EAAE,qBAAqB,GAAG,OAAO,CAAC,oBAAoB,CAAC;IAOvF,aAAa,CAClB,MAAM,EAAE,MAAM,EACd,OAAO,CAAC,EAAE,qBAAqB,GAC9B,aAAa,CAAC,yBAAyB,CAAC;IAK3C,eAAe,CAAC,QAAQ,EAAE,YAAY,GAAG,OAAO;IAIhD,OAAO,CAAC,QAAQ;CAmEjB"}
+{"version":3,"file":"dev-executor.service.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/common/executors/dev-executor.service.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,SAAS,EAAE,YAAY,EAAE,MAAM,2CAA2C,CAAC;AACzF,OAAO,KAAK,EACV,cAAc,EACd,qBAAqB,EACrB,oBAAoB,EACpB,yBAAyB,EAC1B,MAAM,4EAA4E,CAAC;AAwIpF,qBAAa,uBAAwB,YAAW,cAAc;IAC5D,QAAQ,CAAC,SAAS,EAAE,SAAS,CAAsB;IACnD,OAAO,CAAC,QAAQ,CAAC,OAAO,CAAS;;IAM3B,OAAO,CAAC,MAAM,EAAE,MAAM,EAAE,QAAQ,CAAC,EAAE,qBAAqB,GAAG,OAAO,CAAC,oBAAoB,CAAC;IAOvF,aAAa,CAClB,MAAM,EAAE,MAAM,EACd,OAAO,CAAC,EAAE,qBAAqB,GAC9B,aAAa,CAAC,yBAAyB,CAAC;IAK3C,eAAe,CAAC,QAAQ,EAAE,YAAY,GAAG,OAAO;IAIhD,OAAO,CAAC,QAAQ;CAmEjB"}

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

@@ -101,20 +101,22 @@ const CI_FIX_RESULT = 'Fixed CI failure: corrected import path. All checks now p
  * plan.prompt.ts:199.
  */
 function extractSpecDir(prompt) {
+    // Match both Unix absolute paths (/...) and Windows absolute paths (C:\...)
+    const ABS_PATH = String.raw `(?:\/|[A-Za-z]:[/\\])[^\s\n]+`;
     // Analyze ("Write your analysis to:"), requirements ("Update the file at:"),
     // research ("Write your research to:")
-    const fileMatch = prompt.match(/(?:Write your \w+ to|Update the file at):\s+(\/[^\s\n]+)/);
+    const fileMatch = prompt.match(new RegExp(`(?:Write your \\w+ to|Update the file at):\\s+(${ABS_PATH})`));
     if (fileMatch)
         return path.dirname(fileMatch[1]);
     // Plan: "Write to BOTH /path/plan.yaml AND /path/tasks.yaml"
-    const planMatch = prompt.match(/Write to BOTH\s+(\/[^\s\n]+)\/plan\.yaml/);
+    const planMatch = prompt.match(new RegExp(`Write to BOTH\\s+(${ABS_PATH})[/\\\\]plan\\.yaml`));
     if (planMatch)
         return planMatch[1];
     return null;
 }
 /** Validate extracted specDir: must be absolute, no ".." traversal. */
 function validateSpecDir(specDir) {
-    if (!specDir.startsWith('/'))
+    if (!path.isAbsolute(specDir))
         throw new Error(`DevAgentExecutorService: invalid specDir (not absolute): ${specDir}`);
     if (specDir.includes('..'))
         throw new Error(`DevAgentExecutorService: invalid specDir (contains ".."): ${specDir}`);

package/dist/packages/core/src/infrastructure/services/agents/sessions/claude-code-session.repository.js

@@ -311,7 +311,7 @@ let ClaudeCodeSessionRepository = class ClaudeCodeSessionRepository {
         const home = os.homedir();
         if (filePath === home)
             return '~';
-        if (filePath.startsWith(`${home}/`)) {
+        if (filePath.startsWith(`${home}${path.sep}`)) {
             return `~${filePath.slice(home.length)}`;
         }
         return filePath;

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

@@ -51,7 +51,7 @@ export declare class DeploymentService implements IDeploymentService {
      */
     off(event: 'log', handler: (entry: LogEntry) => void): void;
     /**
-     * Send SIGKILL to a process group.
+     * Send SIGKILL to a process tree.
      */
     private killProcess;
     /**

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;;;;;;;GAOG;AAEH,OAAO,EAAE,KAAK,EAAqB,MAAM,oBAAoB,CAAC;AAG9D,OAAO,KAAK,EACV,kBAAkB,EAClB,gBAAgB,EAChB,QAAQ,EACT,MAAM,qEAAqE,CAAC;AAC7E,OAAO,EAAE,eAAe,EAAE,MAAM,wBAAwB,CAAC;AAoBzD,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;AAgBD,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;gBAElC,IAAI,GAAE,OAAO,CAAC,qBAAqB,CAAM;IAIrD;;;OAGG;IACH,KAAK,CAAC,QAAQ,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,GAAG,IAAI;IA+EjD;;;OAGG;IACH,SAAS,CAAC,QAAQ,EAAE,MAAM,GAAG,gBAAgB,GAAG,IAAI;IAYpD;;OAEG;IACG,IAAI,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAyC3C;;OAEG;IACH,OAAO,IAAI,IAAI;IAOf;;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;IAI3D;;OAEG;IACH,OAAO,CAAC,WAAW;IAQnB;;;OAGG;IACH,OAAO,CAAC,oBAAoB;IA+D5B;;OAEG;YACW,aAAa;IAW3B;;OAEG;IACH,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;;;;;;;GAOG;AAEH,OAAO,EAAE,KAAK,EAAqB,MAAM,oBAAoB,CAAC;AAI9D,OAAO,KAAK,EACV,kBAAkB,EAClB,gBAAgB,EAChB,QAAQ,EACT,MAAM,qEAAqE,CAAC;AAC7E,OAAO,EAAE,eAAe,EAAE,MAAM,wBAAwB,CAAC;AAoBzD,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;AAgBD,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;gBAElC,IAAI,GAAE,OAAO,CAAC,qBAAqB,CAAM;IAIrD;;;OAGG;IACH,KAAK,CAAC,QAAQ,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,GAAG,IAAI;IA+EjD;;;OAGG;IACH,SAAS,CAAC,QAAQ,EAAE,MAAM,GAAG,gBAAgB,GAAG,IAAI;IAYpD;;OAEG;IACG,IAAI,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAyC3C;;OAEG;IACH,OAAO,IAAI,IAAI;IAOf;;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;IAI3D;;OAEG;IACH,OAAO,CAAC,WAAW;IAQnB;;;OAGG;IACH,OAAO,CAAC,oBAAoB;IA+D5B;;OAEG;YACW,aAAa;IAW3B;;OAEG;IACH,OAAO,CAAC,WAAW;CASpB"}

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

@@ -8,6 +8,7 @@
  */
 import { spawn } from 'node:child_process';
 import { EventEmitter } from 'node:events';
+import treeKill from 'tree-kill';
 import { DeploymentState } from '../../../domain/generated/output.js';
 import { detectDevScript } from './detect-dev-script.js';
 import { createDeploymentLogger } from './deployment-logger.js';
@@ -19,7 +20,7 @@ const MAX_WAIT_MS = 5000;
 const defaultDeps = {
     spawn,
     detectDevScript,
-    kill: (pid, signal) => process.kill(pid, signal),
+    kill: (pid, signal) => treeKill(pid, signal),
     isAlive: (pid) => {
         try {
             process.kill(pid, 0);
@@ -123,11 +124,11 @@ export class DeploymentService {
             log.info(`stop("${targetId}") — no deployment found, nothing to stop`);
             return;
         }
-        log.info(`stop("${targetId}") — sending SIGTERM to process group (pid=${entry.pid})`);
+        log.info(`stop("${targetId}") — sending SIGTERM to process tree (pid=${entry.pid})`);
         entry.logs.clear();
-        // Send SIGTERM to process group
+        // Send SIGTERM to process tree (tree-kill handles child processes)
         try {
-            this.deps.kill(-entry.pid, 'SIGTERM');
+            this.deps.kill(entry.pid, 'SIGTERM');
         }
         catch {
             log.info(`stop("${targetId}") — process already dead on SIGTERM`);
@@ -140,7 +141,7 @@ export class DeploymentService {
             log.warn(`stop("${targetId}") — process did not exit after ${MAX_WAIT_MS}ms, escalating to SIGKILL`);
             // Escalate to SIGKILL
             try {
-                this.deps.kill(-entry.pid, 'SIGKILL');
+                this.deps.kill(entry.pid, 'SIGKILL');
             }
             catch {
                 // Process may have exited between check and kill
@@ -183,11 +184,11 @@ export class DeploymentService {
         this.emitter.off(event, handler);
     }
     /**
-     * Send SIGKILL to a process group.
+     * Send SIGKILL to a process tree.
      */
     killProcess(entry) {
         try {
-            this.deps.kill(-entry.pid, 'SIGKILL');
+            this.deps.kill(entry.pid, 'SIGKILL');
         }
         catch {
             // Process may already be dead

package/dist/packages/core/src/infrastructure/services/ide-launchers/json-driven-ide-launcher.service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"json-driven-ide-launcher.service.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/infrastructure/services/ide-launchers/json-driven-ide-launcher.service.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAKH,OAAO,KAAK,EACV,mBAAmB,EACnB,eAAe,EAChB,MAAM,8EAA8E,CAAC;AAetF;;;;;GAKG;AACH,wBAAgB,oBAAoB,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,GAAG,MAAM,CAGnF;AAED,qBACa,4BAA6B,YAAW,mBAAmB;IACtE,OAAO,CAAC,QAAQ,CAAC,OAAO,CAAwB;;IAiB1C,MAAM,CACV,QAAQ,EAAE,MAAM,EAChB,aAAa,EAAE,MAAM,EACrB,OAAO,CAAC,EAAE;QAAE,QAAQ,CAAC,EAAE,OAAO,CAAA;KAAE,GAC/B,OAAO,CAAC,eAAe,CAAC;IA8D3B,iBAAiB,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC;CAWtD"}
+{"version":3,"file":"json-driven-ide-launcher.service.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/infrastructure/services/ide-launchers/json-driven-ide-launcher.service.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAMH,OAAO,KAAK,EACV,mBAAmB,EACnB,eAAe,EAChB,MAAM,8EAA8E,CAAC;AAetF;;;;;GAKG;AACH,wBAAgB,oBAAoB,CAAC,KAAK,EAAE,MAAM,GAAG,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,GAAG,MAAM,CAGnF;AAED,qBACa,4BAA6B,YAAW,mBAAmB;IACtE,OAAO,CAAC,QAAQ,CAAC,OAAO,CAAwB;;IAiB1C,MAAM,CACV,QAAQ,EAAE,MAAM,EAChB,aAAa,EAAE,MAAM,EACrB,OAAO,CAAC,EAAE;QAAE,QAAQ,CAAC,EAAE,OAAO,CAAA;KAAE,GAC/B,OAAO,CAAC,eAAe,CAAC;IA8DrB,iBAAiB,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC;CAQ5D"}

package/dist/packages/core/src/infrastructure/services/ide-launchers/json-driven-ide-launcher.service.js

@@ -16,8 +16,9 @@ var __metadata = (this && this.__metadata) || function (k, v) {
     if (typeof Reflect === "object" && typeof Reflect.metadata === "function") return Reflect.metadata(k, v);
 };
 import { injectable } from 'tsyringe';
-import { spawn, execFile } from 'node:child_process';
+import { spawn } from 'node:child_process';
 import { platform } from 'node:os';
+import { checkBinaryExists } from '../tool-installer/binary-exists.js';
 import { TOOL_METADATA } from '../tool-installer/tool-metadata.js';
 /**
  * Resolve a platform-specific value from a string or per-platform record.
@@ -101,16 +102,13 @@ let JsonDrivenIdeLauncherService = class JsonDrivenIdeLauncherService {
             };
         }
     }
-    checkAvailability(editorId) {
+    async checkAvailability(editorId) {
         const entry = this.editors.get(editorId);
         if (!entry)
-            return Promise.resolve(false);
+            return false;
         const binary = resolvePlatformValue(entry.binary);
-        return new Promise((resolve) => {
-            execFile('which', [binary], (err) => {
-                resolve(!err);
-            });
-        });
+        const result = await checkBinaryExists(binary);
+        return result.found;
     }
 };
 JsonDrivenIdeLauncherService = __decorate([

package/dist/packages/core/src/infrastructure/services/process/tree-kill.d.ts

@@ -0,0 +1,12 @@
+/**
+ * Cross-platform process tree termination.
+ *
+ * Re-exports the npm `tree-kill` package for use by the presentation layer,
+ * which cannot resolve packages installed in packages/core/node_modules
+ * directly (pnpm workspace isolation).
+ *
+ * Uses taskkill /T /F on Windows, kill/ps/pgrep on Unix.
+ */
+import treeKill from 'tree-kill';
+export { treeKill };
+//# sourceMappingURL=tree-kill.d.ts.map

package/dist/packages/core/src/infrastructure/services/process/tree-kill.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"tree-kill.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/infrastructure/services/process/tree-kill.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,QAAQ,MAAM,WAAW,CAAC;AAEjC,OAAO,EAAE,QAAQ,EAAE,CAAC"}

package/dist/packages/core/src/infrastructure/services/process/tree-kill.js

@@ -0,0 +1,11 @@
+/**
+ * Cross-platform process tree termination.
+ *
+ * Re-exports the npm `tree-kill` package for use by the presentation layer,
+ * which cannot resolve packages installed in packages/core/node_modules
+ * directly (pnpm workspace isolation).
+ *
+ * Uses taskkill /T /F on Windows, kill/ps/pgrep on Unix.
+ */
+import treeKill from 'tree-kill';
+export { treeKill };

package/dist/packages/core/src/infrastructure/services/spec/spec-initializer.service.js

@@ -35,7 +35,7 @@ function applyTemplate(template, vars) {
     return result;
 }
 // ─── YAML Templates ────────────────────────────────────────────────
-// These match .claude/skills/shep-kit:new-feature/templates/ exactly.
+// These match .claude/skills/shep-kit-new-feature/templates/ exactly.
 const SPEC_YAML = `# Feature Specification (YAML)
 # This is the source of truth. Markdown is auto-generated from this file.
 

package/dist/packages/core/src/infrastructure/services/tool-installer/binary-exists.d.ts

@@ -0,0 +1,25 @@
+/**
+ * Cross-platform binary existence check.
+ *
+ * Wraps the npm `which` package to provide a single cross-platform API
+ * for checking if a binary exists in the system PATH. Works on Unix
+ * (which) and Windows (where.exe + PATHEXT resolution) without platform
+ * branching.
+ */
+/**
+ * Check if a binary exists in the system PATH.
+ *
+ * Uses the npm `which` package which handles Windows PATHEXT resolution
+ * internally (.exe, .cmd, .bat extensions).
+ *
+ * Returns:
+ * - `{ found: true }` when the binary is on PATH
+ * - `{ found: false, notInPath: true }` when the binary is not on PATH
+ * - `{ found: false, error }` for genuine errors (e.g. permission denied)
+ */
+export declare const checkBinaryExists: (binary: string) => Promise<{
+    found: boolean;
+    notInPath?: boolean;
+    error?: Error;
+}>;
+//# sourceMappingURL=binary-exists.d.ts.map

package/dist/packages/core/src/infrastructure/services/tool-installer/binary-exists.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"binary-exists.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/infrastructure/services/tool-installer/binary-exists.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAIH;;;;;;;;;;GAUG;AACH,eAAO,MAAM,iBAAiB,GAC5B,QAAQ,MAAM,KACb,OAAO,CAAC;IAAE,KAAK,EAAE,OAAO,CAAC;IAAC,SAAS,CAAC,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,KAAK,CAAA;CAAE,CAUhE,CAAC"}

package/dist/packages/core/src/infrastructure/services/tool-installer/binary-exists.js

@@ -0,0 +1,32 @@
+/**
+ * Cross-platform binary existence check.
+ *
+ * Wraps the npm `which` package to provide a single cross-platform API
+ * for checking if a binary exists in the system PATH. Works on Unix
+ * (which) and Windows (where.exe + PATHEXT resolution) without platform
+ * branching.
+ */
+import which from 'which';
+/**
+ * Check if a binary exists in the system PATH.
+ *
+ * Uses the npm `which` package which handles Windows PATHEXT resolution
+ * internally (.exe, .cmd, .bat extensions).
+ *
+ * Returns:
+ * - `{ found: true }` when the binary is on PATH
+ * - `{ found: false, notInPath: true }` when the binary is not on PATH
+ * - `{ found: false, error }` for genuine errors (e.g. permission denied)
+ */
+export const checkBinaryExists = async (binary) => {
+    try {
+        const result = await which(binary, { nothrow: true });
+        if (result !== null) {
+            return { found: true };
+        }
+        return { found: false, notInPath: true };
+    }
+    catch (error) {
+        return { found: false, error: error instanceof Error ? error : new Error(String(error)) };
+    }
+};

package/dist/packages/core/src/infrastructure/services/tool-installer/tool-installer.service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"tool-installer.service.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/infrastructure/services/tool-installer/tool-installer.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAKH,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,sEAAsE,CAAC;AAClH,OAAO,KAAK,EACV,sBAAsB,EACtB,kBAAkB,EAEnB,MAAM,qCAAqC,CAAC;AAsD7C,qBACa,wBAAyB,YAAW,qBAAqB;IACpE;;;OAGG;IACG,iBAAiB,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,sBAAsB,CAAC;IAoC1E;;;OAGG;IACH,iBAAiB,CAAC,QAAQ,EAAE,MAAM,GAAG,kBAAkB,GAAG,IAAI;IAqB9D;;;OAGG;IACG,cAAc,CAClB,QAAQ,EAAE,MAAM,EAChB,QAAQ,CAAC,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,IAAI,GAChC,OAAO,CAAC,sBAAsB,CAAC;IAgFlC;;OAEG;IACH,OAAO,CAAC,6BAA6B;CA4BtC"}
+{"version":3,"file":"tool-installer.service.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/infrastructure/services/tool-installer/tool-installer.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAKH,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,sEAAsE,CAAC;AAClH,OAAO,KAAK,EACV,sBAAsB,EACtB,kBAAkB,EAEnB,MAAM,qCAAqC,CAAC;AA2B7C,qBACa,wBAAyB,YAAW,qBAAqB;IACpE;;;OAGG;IACG,iBAAiB,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,sBAAsB,CAAC;IAoC1E;;;OAGG;IACH,iBAAiB,CAAC,QAAQ,EAAE,MAAM,GAAG,kBAAkB,GAAG,IAAI;IAqB9D;;;OAGG;IACG,cAAc,CAClB,QAAQ,EAAE,MAAM,EAChB,QAAQ,CAAC,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,IAAI,GAChC,OAAO,CAAC,sBAAsB,CAAC;IAgFlC;;OAEG;IACH,OAAO,CAAC,6BAA6B;CA4BtC"}

package/dist/packages/core/src/infrastructure/services/tool-installer/tool-installer.service.js

@@ -18,10 +18,11 @@ var __decorate = (this && this.__decorate) || function (decorators, target, key,
     return c > 3 && r && Object.defineProperty(target, key, r), r;
 };
 import { injectable } from 'tsyringe';
-import { exec, execFile, spawn } from 'node:child_process';
+import { exec, spawn } from 'node:child_process';
 import { platform } from 'node:os';
 import { createAvailableStatus, createMissingStatus, createErrorStatus, } from '../../../domain/value-objects/tool-installation-status.js';
 import { TOOL_METADATA } from './tool-metadata.js';
+import { checkBinaryExists } from './binary-exists.js';
 /**
  * Resolve binary name for the current platform.
  * Supports both simple string binaries and per-platform maps.
@@ -31,30 +32,6 @@ function resolveBinary(metadata) {
         return metadata.binary;
     return metadata.binary[platform()] ?? Object.values(metadata.binary)[0];
 }
-/**
- * Check if a binary exists in the system PATH using 'which' command.
- *
- * `which` exits with code 1 when the binary is simply not in PATH.
- * Node's execFile wraps that as an Error, but it's a normal "not found"
- * result. Only truly unexpected failures (ENOENT for `which` itself,
- * EACCES, signals, etc.) are surfaced via the `error` field.
- */
-const checkBinaryExists = (binary) => {
-    return new Promise((resolve) => {
-        execFile('which', [binary], (err) => {
-            if (!err)
-                return resolve({ found: true });
-            // execFile errors from a non-zero exit code have a numeric `code` property.
-            // That's the normal "binary not found in PATH" case.
-            const code = err.code;
-            if (code === undefined || typeof code === 'number') {
-                return resolve({ found: false, notInPath: true });
-            }
-            // Anything else (ENOENT for `which` itself, EACCES, etc.) is a real error.
-            resolve({ found: false, error: err });
-        });
-    });
-};
 /**
  * Run a shell command and resolve true when it exits 0, false otherwise.
  * Used as a fallback for GUI apps whose binary isn't in PATH (e.g. Warp, iTerm2).
@@ -130,8 +107,8 @@ let ToolInstallerServiceImpl = class ToolInstallerServiceImpl {
         }
         return new Promise((resolve) => {
             let output = '';
-            const child = spawn('sh', ['-c', installCommand.command], {
-                shell: false,
+            const child = spawn(installCommand.command, [], {
+                shell: true,
                 stdio: ['pipe', 'pipe', 'pipe'],
                 env: {
                     ...process.env,

package/dist/packages/core/src/infrastructure/services/tool-installer/tools/alacritty.json

@@ -5,13 +5,14 @@
   "tags": ["terminal"],
   "author": "Alacritty Contributors",
   "website": "https://alacritty.org",
-  "platforms": ["linux", "darwin"],
+  "platforms": ["linux", "darwin", "win32"],
   "iconUrl": "https://cdn.simpleicons.org/alacritty",
   "binary": "alacritty",
   "packageManager": "brew",
   "commands": {
     "darwin": "brew install --cask alacritty",
-    "linux": "sudo apt-get update && sudo apt-get install -y alacritty"
+    "linux": "sudo apt-get update && sudo apt-get install -y alacritty",
+    "win32": "winget install --id Alacritty.Alacritty -e --source winget"
   },
   "timeout": 300000,
   "documentationUrl": "https://alacritty.org/",

package/dist/packages/core/src/infrastructure/services/tool-installer/tools/claude-code.json

@@ -5,13 +5,14 @@
   "tags": ["cli-agent"],
   "author": "Anthropic",
   "website": "https://claude.ai/code",
-  "platforms": ["linux", "darwin"],
+  "platforms": ["linux", "darwin", "win32"],
   "iconUrl": "https://cdn.simpleicons.org/claude",
   "binary": "claude",
   "packageManager": "curl",
   "commands": {
     "linux": "curl -fsSL https://claude.ai/install.sh | bash",
-    "darwin": "curl -fsSL https://claude.ai/install.sh | bash"
+    "darwin": "curl -fsSL https://claude.ai/install.sh | bash",
+    "win32": "npm install -g @anthropic-ai/claude-code"
   },
   "timeout": 300000,
   "documentationUrl": "https://code.claude.com/docs/en/overview",

package/dist/packages/core/src/infrastructure/services/tool-installer/tools/cursor-cli.json

@@ -5,13 +5,14 @@
   "tags": ["cli-agent"],
   "author": "Anysphere",
   "website": "https://cursor.com",
-  "platforms": ["linux", "darwin"],
+  "platforms": ["linux", "darwin", "win32"],
   "iconUrl": "https://cdn.simpleicons.org/cursor",
   "binary": "cursor-agent",
   "packageManager": "curl",
   "commands": {
     "linux": "curl https://cursor.com/install -fsS | bash",
-    "darwin": "curl https://cursor.com/install -fsS | bash"
+    "darwin": "curl https://cursor.com/install -fsS | bash",
+    "win32": "winget install --id Anysphere.Cursor -e --source winget"
   },
   "timeout": 300000,
   "documentationUrl": "https://www.codecademy.com/article/getting-started-with-cursor-cli",

package/dist/packages/core/src/infrastructure/services/tool-installer/tools/cursor.json

@@ -11,7 +11,8 @@
   "packageManager": "manual",
   "commands": {
     "linux": "Download from https://cursor.com/download and install the AppImage manually",
-    "darwin": "Download from https://cursor.com/download or use: brew install --cask cursor"
+    "darwin": "Download from https://cursor.com/download or use: brew install --cask cursor",
+    "win32": "winget install --id Anysphere.Cursor -e --source winget"
   },
   "timeout": 600000,
   "documentationUrl": "https://cursor.com/download",

package/dist/packages/core/src/infrastructure/services/tool-installer/tools/gh.json

@@ -11,7 +11,8 @@
   "packageManager": "apt",
   "commands": {
     "linux": "(type -p wget >/dev/null || (sudo apt update && sudo apt-get install wget -y)) && sudo mkdir -p -m 755 /etc/apt/keyrings && out=$(mktemp) && wget -nv -O$out https://cli.github.com/packages/githubcli-archive-keyring.gpg && cat $out | sudo tee /etc/apt/keyrings/githubcli-archive-keyring.gpg > /dev/null && sudo chmod go+r /etc/apt/keyrings/githubcli-archive-keyring.gpg && echo \"deb [arch=$(dpkg --print-architecture) signed-by=/etc/apt/keyrings/githubcli-archive-keyring.gpg] https://cli.github.com/packages stable main\" | sudo tee /etc/apt/sources.list.d/github-cli.list > /dev/null && sudo apt update && sudo apt install gh -y",
-    "darwin": "brew install gh"
+    "darwin": "brew install gh",
+    "win32": "winget install --id GitHub.cli -e --source winget"
   },
   "timeout": 300000,
   "documentationUrl": "https://cli.github.com/manual/",

package/dist/packages/core/src/infrastructure/services/tool-installer/tools/git.json

@@ -11,7 +11,8 @@
   "packageManager": "apt",
   "commands": {
     "linux": "sudo apt-get update && sudo apt-get install -y git",
-    "darwin": "xcode-select --install"
+    "darwin": "xcode-select --install",
+    "win32": "winget install --id Git.Git -e --source winget"
   },
   "timeout": 300000,
   "documentationUrl": "https://git-scm.com/doc",

package/dist/packages/core/src/infrastructure/services/tool-installer/tools/vscode.json

@@ -11,7 +11,8 @@
   "packageManager": "apt",
   "commands": {
     "linux": "curl -fsSL https://packages.microsoft.com/keys/microsoft.asc | sudo gpg --yes --dearmor -o /usr/share/keyrings/packages.microsoft.gpg && echo \"deb [arch=amd64 signed-by=/usr/share/keyrings/packages.microsoft.gpg] https://packages.microsoft.com/repos/code stable main\" | sudo tee /etc/apt/sources.list.d/vscode.list > /dev/null && sudo apt-get update && sudo apt-get install -y code",
-    "darwin": "brew install visual-studio-code"
+    "darwin": "brew install visual-studio-code",
+    "win32": "winget install --id Microsoft.VisualStudioCode -e --source winget"
   },
   "timeout": 300000,
   "documentationUrl": "https://code.visualstudio.com/docs/setup/linux",

package/dist/packages/core/src/infrastructure/services/tool-installer/tools/warp.json

@@ -5,12 +5,13 @@
   "tags": ["terminal"],
   "author": "Warp",
   "website": "https://www.warp.dev",
-  "platforms": ["darwin"],
+  "platforms": ["darwin", "win32"],
   "iconUrl": "https://cdn.simpleicons.org/warp",
   "binary": "warp",
   "packageManager": "brew",
   "commands": {
-    "darwin": "brew install --cask warp"
+    "darwin": "brew install --cask warp",
+    "win32": "winget install --id Warp.Warp -e --source winget"
   },
   "timeout": 300000,
   "documentationUrl": "https://docs.warp.dev/",