@shepai/cli 1.104.0 → 1.104.1-pr299.6a46a49

package/dist/src/presentation/cli/commands/daemon/stop-daemon.js

@@ -18,6 +18,7 @@
  *
  * @param daemonService - IDaemonService instance (injected by caller for testability, NFR-4)
  */
+import { treeKill } from '../../../../../packages/core/src/infrastructure/services/process/tree-kill.js';
 import { messages } from '../../ui/index.js';
 const POLL_INTERVAL_MS = 200;
 const MAX_WAIT_MS = 5000;
@@ -54,15 +55,15 @@ export async function stopDaemon(daemonService) {
     }
     try {
         messages.info(`Stopping Shep daemon (PID ${pid})...`);
-        // Send SIGTERM — request graceful shutdown
-        process.kill(pid, 'SIGTERM');
+        // Send SIGTERM to process tree — request graceful shutdown
+        treeKill(pid, 'SIGTERM');
         // Poll for up to 5s waiting for the process to exit
         const died = await pollUntilDead(daemonService, pid, MAX_WAIT_MS, POLL_INTERVAL_MS);
         if (!died) {
-            // Graceful shutdown timed out — force kill
+            // Graceful shutdown timed out — force kill process tree
             messages.info('Daemon did not stop gracefully, sending SIGKILL...');
             try {
-                process.kill(pid, 'SIGKILL');
+                treeKill(pid, 'SIGKILL');
             }
             catch {
                 // Process may have exited between the check and the kill — ignore

package/dist/src/presentation/cli/commands/upgrade.command.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"upgrade.command.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/cli/commands/upgrade.command.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AACpC,OAAO,EAAE,KAAK,IAAI,YAAY,EAAqB,MAAM,oBAAoB,CAAC;AAQ9E,KAAK,OAAO,GAAG,OAAO,YAAY,CAAC;AAyEnC;;GAEG;AACH,wBAAgB,oBAAoB,CAAC,OAAO,GAAE,OAAsB,GAAG,OAAO,CA+D7E"}
+{"version":3,"file":"upgrade.command.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/cli/commands/upgrade.command.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AACpC,OAAO,EAAE,KAAK,IAAI,YAAY,EAAqB,MAAM,oBAAoB,CAAC;AAQ9E,KAAK,OAAO,GAAG,OAAO,YAAY,CAAC;AA8EnC;;GAEG;AACH,wBAAgB,oBAAoB,CAAC,OAAO,GAAE,OAAsB,GAAG,OAAO,CA+D7E"}

package/dist/src/presentation/cli/commands/upgrade.command.js

@@ -13,6 +13,8 @@ import { messages } from '../ui/index.js';
 import { stopDaemon } from './daemon/stop-daemon.js';
 import { startDaemon } from './daemon/start-daemon.js';
 const VERSION_CHECK_TIMEOUT_MS = 10_000;
+/** On Windows, npm is a .cmd batch file — spawn() needs shell: true to resolve it. */
+const IS_WINDOWS = process.platform === 'win32';
 /**
  * Get the latest published version of @shepai/cli from npm registry.
  * Returns null if the check fails (fail-open).
@@ -23,6 +25,7 @@ function getLatestVersion(spawnFn) {
         let settled = false;
         const child = spawnFn('npm', ['view', '@shepai/cli', 'version'], {
             stdio: ['ignore', 'pipe', 'pipe'],
+            ...(IS_WINDOWS && { shell: true }),
         });
         const timeout = setTimeout(() => {
             if (!settled) {
@@ -65,6 +68,7 @@ function runNpmInstall(spawnFn) {
     return new Promise((resolve, reject) => {
         const child = spawnFn('npm', ['i', '-g', '@shepai/cli@latest'], {
             stdio: 'inherit',
+            ...(IS_WINDOWS && { shell: true }),
         });
         child.on('close', (code) => {
             resolve(code ?? 1);

package/dist/src/presentation/web/app/actions/deploy-repository.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"deploy-repository.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/deploy-repository.ts"],"names":[],"mappings":"AAMA,OAAO,EAAE,eAAe,EAAE,MAAM,sCAAsC,CAAC;AAIvE,wBAAsB,gBAAgB,CACpC,cAAc,EAAE,MAAM,GACrB,OAAO,CAAC;IAAE,OAAO,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAC;IAAC,KAAK,CAAC,EAAE,eAAe,CAAA;CAAE,CAAC,CAyBxE"}
+{"version":3,"file":"deploy-repository.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/deploy-repository.ts"],"names":[],"mappings":"AAOA,OAAO,EAAE,eAAe,EAAE,MAAM,sCAAsC,CAAC;AAIvE,wBAAsB,gBAAgB,CACpC,cAAc,EAAE,MAAM,GACrB,OAAO,CAAC;IAAE,OAAO,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAC;IAAC,KAAK,CAAC,EAAE,eAAe,CAAA;CAAE,CAAC,CAyBxE"}

package/dist/src/presentation/web/app/actions/deploy-repository.js

@@ -1,12 +1,13 @@
 'use server';
 import { existsSync } from 'node:fs';
+import { isAbsolute } from 'node:path';
 import { resolve } from '../../lib/server-container.js';
 import { createDeploymentLogger } from '../../../../../packages/core/src/infrastructure/services/deployment/deployment-logger.js';
 import { DeploymentState } from '../../../../../packages/core/src/domain/generated/output.js';
 const log = createDeploymentLogger('[deployRepository]');
 export async function deployRepository(repositoryPath) {
     log.info(`called — repositoryPath="${repositoryPath}"`);
-    if (!repositoryPath?.startsWith('/')) {
+    if (!repositoryPath || !isAbsolute(repositoryPath)) {
         log.warn('rejected — not an absolute path');
         return { success: false, error: 'repositoryPath must be an absolute path' };
     }

package/dist/src/presentation/web/app/actions/open-folder.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"open-folder.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/open-folder.ts"],"names":[],"mappings":"AAeA,wBAAsB,UAAU,CAC9B,cAAc,EAAE,MAAM,GACrB,OAAO,CAAC;IAAE,OAAO,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAC;IAAC,IAAI,CAAC,EAAE,MAAM,CAAA;CAAE,CAAC,CA8B9D"}
+{"version":3,"file":"open-folder.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/open-folder.ts"],"names":[],"mappings":"AAiBA,wBAAsB,UAAU,CAC9B,cAAc,EAAE,MAAM,GACrB,OAAO,CAAC;IAAE,OAAO,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAC;IAAC,IAAI,CAAC,EAAE,MAAM,CAAA;CAAE,CAAC,CA8B9D"}

package/dist/src/presentation/web/app/actions/open-folder.js

@@ -1,6 +1,7 @@
 'use server';
 import { existsSync } from 'node:fs';
 import { platform } from 'node:os';
+import { isAbsolute } from 'node:path';
 import { spawn } from 'node:child_process';
 // Use a record lookup instead of if/else to prevent the bundler from
 // tree-shaking platform branches at build time. Turbopack evaluates
@@ -9,9 +10,10 @@ import { spawn } from 'node:child_process';
 const FOLDER_COMMANDS = {
     darwin: { cmd: 'open', args: (p) => [p] },
     linux: { cmd: 'xdg-open', args: (p) => [p] },
+    win32: { cmd: 'explorer', args: (p) => [p] },
 };
 export async function openFolder(repositoryPath) {
-    if (!repositoryPath?.startsWith('/')) {
+    if (!repositoryPath || !isAbsolute(repositoryPath)) {
         return { success: false, error: 'repositoryPath must be an absolute path' };
     }
     try {
@@ -22,7 +24,7 @@ export async function openFolder(repositoryPath) {
         if (!entry) {
             return {
                 success: false,
-                error: `Unsupported platform: ${platform()}. Folder open is supported on macOS and Linux only.`,
+                error: `Unsupported platform: ${platform()}`,
             };
         }
         const child = spawn(entry.cmd, entry.args(repositoryPath), {

package/dist/src/presentation/web/app/actions/open-ide.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"open-ide.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/open-ide.ts"],"names":[],"mappings":"AAMA,UAAU,YAAY;IACpB,cAAc,EAAE,MAAM,CAAC;IACvB,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED,wBAAsB,OAAO,CAC3B,KAAK,EAAE,YAAY,GAClB,OAAO,CAAC;IAAE,OAAO,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAC;IAAC,MAAM,CAAC,EAAE,MAAM,CAAC;IAAC,IAAI,CAAC,EAAE,MAAM,CAAA;CAAE,CAAC,CAuB/E"}
+{"version":3,"file":"open-ide.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/open-ide.ts"],"names":[],"mappings":"AAOA,UAAU,YAAY;IACpB,cAAc,EAAE,MAAM,CAAC;IACvB,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED,wBAAsB,OAAO,CAC3B,KAAK,EAAE,YAAY,GAClB,OAAO,CAAC;IAAE,OAAO,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAC;IAAC,MAAM,CAAC,EAAE,MAAM,CAAC;IAAC,IAAI,CAAC,EAAE,MAAM,CAAA;CAAE,CAAC,CAuB/E"}

package/dist/src/presentation/web/app/actions/open-ide.js

@@ -1,9 +1,10 @@
 'use server';
+import { isAbsolute } from 'node:path';
 import { getSettings } from '../../../../../packages/core/src/infrastructure/services/settings.service.js';
 import { resolve } from '../../lib/server-container.js';
 export async function openIde(input) {
     const { repositoryPath, branch } = input;
-    if (!repositoryPath?.startsWith('/')) {
+    if (!repositoryPath || !isAbsolute(repositoryPath)) {
         return { success: false, error: 'repositoryPath must be an absolute path' };
     }
     const settings = getSettings();

package/dist/src/presentation/web/app/actions/open-shell.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"open-shell.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/open-shell.ts"],"names":[],"mappings":"AAiBA,UAAU,cAAc;IACtB,cAAc,EAAE,MAAM,CAAC;IACvB,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED,wBAAsB,SAAS,CAC7B,KAAK,EAAE,cAAc,GACpB,OAAO,CAAC;IAAE,OAAO,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAC;IAAC,IAAI,CAAC,EAAE,MAAM,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAA;CAAE,CAAC,CAoC9E"}
+{"version":3,"file":"open-shell.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/open-shell.ts"],"names":[],"mappings":"AAmBA,UAAU,cAAc;IACtB,cAAc,EAAE,MAAM,CAAC;IACvB,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED,wBAAsB,SAAS,CAC7B,KAAK,EAAE,cAAc,GACpB,OAAO,CAAC;IAAE,OAAO,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAC;IAAC,IAAI,CAAC,EAAE,MAAM,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAA;CAAE,CAAC,CAoC9E"}

package/dist/src/presentation/web/app/actions/open-shell.js

@@ -1,6 +1,7 @@
 'use server';
 import { existsSync } from 'node:fs';
 import { platform } from 'node:os';
+import { isAbsolute } from 'node:path';
 import { spawn } from 'node:child_process';
 import { getSettings } from '../../../../../packages/core/src/infrastructure/services/settings.service.js';
 import { computeWorktreePath } from '../../../../../packages/core/src/infrastructure/services/ide-launchers/compute-worktree-path.js';
@@ -11,10 +12,11 @@ import { computeWorktreePath } from '../../../../../packages/core/src/infrastruc
 const SHELL_COMMANDS = {
     darwin: { cmd: 'open', args: (p) => ['-a', 'Terminal', p] },
     linux: { cmd: 'x-terminal-emulator', args: (p) => [`--working-directory=${p}`] },
+    win32: { cmd: 'powershell', args: (p) => ['-NoExit', '-Command', `Set-Location "${p}"`] },
 };
 export async function openShell(input) {
     const { repositoryPath, branch } = input;
-    if (!repositoryPath?.startsWith('/')) {
+    if (!repositoryPath || !isAbsolute(repositoryPath)) {
         return { success: false, error: 'repositoryPath must be an absolute path' };
     }
     try {
@@ -28,7 +30,7 @@ export async function openShell(input) {
         if (!entry) {
             return {
                 success: false,
-                error: `Unsupported platform: ${platform()}. Shell launch is supported on macOS and Linux only.`,
+                error: `Unsupported platform: ${platform()}`,
             };
         }
         const child = spawn(entry.cmd, entry.args(targetPath), {

package/dist/src/presentation/web/components/common/merge-review/diff-view.js

@@ -26,5 +26,5 @@ function FileDiffItem({ file }) {
 export function DiffView({ fileDiffs }) {
     if (fileDiffs.length === 0)
         return null;
-    return (_jsxs("div", { className: "border-border rounded-lg border", children: [_jsx("div", { className: "px-4 py-3", children: _jsxs("div", { className: "mb-2 flex items-center gap-2", children: [_jsx(FileText, { className: "text-muted-foreground h-4 w-4" }), _jsx("span", { className: "text-foreground text-xs font-semibold", children: "Changed Files" }), _jsxs("span", { className: "text-muted-foreground text-[10px]", children: ["(", fileDiffs.length, ")"] })] }) }), _jsx("div", { className: "border-border border-t", children: fileDiffs.map((file) => (_jsx(FileDiffItem, { file: file }, file.path))) })] }));
+    return (_jsxs("div", { className: "border-border rounded-lg border", children: [_jsx("div", { className: "px-4 py-3", children: _jsxs("div", { className: "mb-2 flex items-center gap-2", children: [_jsx(FileText, { className: "text-muted-foreground h-4 w-4" }), _jsx("span", { className: "text-foreground text-xs font-semibold", children: "Changed Files" }), _jsxs("span", { className: "text-muted-foreground text-[10px]", children: ["(", fileDiffs.length, ")"] })] }) }), _jsx("div", { className: "border-border border-t", children: fileDiffs.map((file) => (_jsx(FileDiffItem, { file: file }, `${file.status}-${file.path}`))) })] }));
 }