@shepai/cli 1.129.2-pr415.e8b42a3 → 1.129.2

package/dist/packages/core/src/application/ports/output/services/index.d.ts

@@ -10,7 +10,7 @@ export type { IVersionService } from './version-service.interface.js';
 export type { IWebServerService } from './web-server-service.interface.js';
 export type { IWorktreeService, WorktreeInfo } from './worktree-service.interface.js';
 export { WorktreeError, WorktreeErrorCode } from './worktree-service.interface.js';
-export type { IToolInstallerService, AvailableTerminalEntry } from './tool-installer.service.js';
+export type { IToolInstallerService } from './tool-installer.service.js';
 export type { INotificationService } from './notification-service.interface.js';
 export type { IGitPrService, CiStatus, CiStatusResult, DiffSummary, MergeStrategy, PrCreateResult, } from './git-pr-service.interface.js';
 export { GitPrError, GitPrErrorCode } from './git-pr-service.interface.js';

package/dist/packages/core/src/application/ports/output/services/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../../../../packages/core/src/application/ports/output/services/index.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAEH,YAAY,EAAE,qBAAqB,EAAE,aAAa,EAAE,MAAM,uCAAuC,CAAC;AAClG,OAAO,EACL,iBAAiB,EACjB,kBAAkB,EAClB,wBAAwB,EACxB,4BAA4B,GAC7B,MAAM,uCAAuC,CAAC;AAC/C,YAAY,EACV,uBAAuB,EACvB,qBAAqB,GACtB,MAAM,iCAAiC,CAAC;AACzC,YAAY,EAAE,eAAe,EAAE,MAAM,gCAAgC,CAAC;AACtE,YAAY,EAAE,iBAAiB,EAAE,MAAM,mCAAmC,CAAC;AAC3E,YAAY,EAAE,gBAAgB,EAAE,YAAY,EAAE,MAAM,iCAAiC,CAAC;AACtF,OAAO,EAAE,aAAa,EAAE,iBAAiB,EAAE,MAAM,iCAAiC,CAAC;AACnF,YAAY,EAAE,qBAAqB,EAAE,sBAAsB,EAAE,MAAM,6BAA6B,CAAC;AACjG,YAAY,EAAE,oBAAoB,EAAE,MAAM,qCAAqC,CAAC;AAChF,YAAY,EACV,aAAa,EACb,QAAQ,EACR,cAAc,EACd,WAAW,EACX,aAAa,EACb,cAAc,GACf,MAAM,+BAA+B,CAAC;AACvC,OAAO,EAAE,UAAU,EAAE,cAAc,EAAE,MAAM,+BAA+B,CAAC;AAC3E,YAAY,EACV,mBAAmB,EACnB,cAAc,EACd,eAAe,EACf,gBAAgB,EAChB,eAAe,GAChB,MAAM,qCAAqC,CAAC;AAC7C,YAAY,EAAE,cAAc,EAAE,WAAW,EAAE,MAAM,+BAA+B,CAAC;AACjF,YAAY,EAAE,kBAAkB,EAAE,gBAAgB,EAAE,MAAM,mCAAmC,CAAC;AAC9F,YAAY,EACV,wBAAwB,EACxB,UAAU,EACV,2BAA2B,EAC3B,YAAY,EACZ,eAAe,GAChB,MAAM,0CAA0C,CAAC;AAClD,OAAO,EACL,eAAe,EACf,gBAAgB,EAChB,mBAAmB,EACnB,mBAAmB,GACpB,MAAM,0CAA0C,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../../../../packages/core/src/application/ports/output/services/index.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAEH,YAAY,EAAE,qBAAqB,EAAE,aAAa,EAAE,MAAM,uCAAuC,CAAC;AAClG,OAAO,EACL,iBAAiB,EACjB,kBAAkB,EAClB,wBAAwB,EACxB,4BAA4B,GAC7B,MAAM,uCAAuC,CAAC;AAC/C,YAAY,EACV,uBAAuB,EACvB,qBAAqB,GACtB,MAAM,iCAAiC,CAAC;AACzC,YAAY,EAAE,eAAe,EAAE,MAAM,gCAAgC,CAAC;AACtE,YAAY,EAAE,iBAAiB,EAAE,MAAM,mCAAmC,CAAC;AAC3E,YAAY,EAAE,gBAAgB,EAAE,YAAY,EAAE,MAAM,iCAAiC,CAAC;AACtF,OAAO,EAAE,aAAa,EAAE,iBAAiB,EAAE,MAAM,iCAAiC,CAAC;AACnF,YAAY,EAAE,qBAAqB,EAAE,MAAM,6BAA6B,CAAC;AACzE,YAAY,EAAE,oBAAoB,EAAE,MAAM,qCAAqC,CAAC;AAChF,YAAY,EACV,aAAa,EACb,QAAQ,EACR,cAAc,EACd,WAAW,EACX,aAAa,EACb,cAAc,GACf,MAAM,+BAA+B,CAAC;AACvC,OAAO,EAAE,UAAU,EAAE,cAAc,EAAE,MAAM,+BAA+B,CAAC;AAC3E,YAAY,EACV,mBAAmB,EACnB,cAAc,EACd,eAAe,EACf,gBAAgB,EAChB,eAAe,GAChB,MAAM,qCAAqC,CAAC;AAC7C,YAAY,EAAE,cAAc,EAAE,WAAW,EAAE,MAAM,+BAA+B,CAAC;AACjF,YAAY,EAAE,kBAAkB,EAAE,gBAAgB,EAAE,MAAM,mCAAmC,CAAC;AAC9F,YAAY,EACV,wBAAwB,EACxB,UAAU,EACV,2BAA2B,EAC3B,YAAY,EACZ,eAAe,GAChB,MAAM,0CAA0C,CAAC;AAClD,OAAO,EACL,eAAe,EACf,gBAAgB,EAChB,mBAAmB,EACnB,mBAAmB,GACpB,MAAM,0CAA0C,CAAC"}

package/dist/packages/core/src/application/ports/output/services/tool-installer.service.d.ts

@@ -5,11 +5,6 @@
  * Implementations check tool availability and execute installations.
  */
 import type { ToolInstallationStatus, ToolInstallCommand } from '../../../../domain/generated/output.js';
-export interface AvailableTerminalEntry {
-    id: string;
-    name: string;
-    available: boolean;
-}
 /**
  * Service interface for tool installation management.
  */
@@ -36,13 +31,5 @@ export interface IToolInstallerService {
      * @returns ToolInstallationStatus with result of installation
      */
     executeInstall(toolName: string, onOutput?: (data: string) => void): Promise<ToolInstallationStatus>;
-    /**
-     * List all terminal emulators supported on the current platform with availability status.
-     * Uses tool metadata loaded in the Node.js context (not bundled by Next.js),
-     * so this must be called via the DI container from server actions.
-     *
-     * @returns Array of terminal entries with id, name, and availability
-     */
-    listAvailableTerminals(): Promise<AvailableTerminalEntry[]>;
 }
 //# sourceMappingURL=tool-installer.service.d.ts.map

package/dist/packages/core/src/application/ports/output/services/tool-installer.service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"tool-installer.service.d.ts","sourceRoot":"","sources":["../../../../../../../../packages/core/src/application/ports/output/services/tool-installer.service.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EACV,sBAAsB,EACtB,kBAAkB,EACnB,MAAM,wCAAwC,CAAC;AAEhD,MAAM,WAAW,sBAAsB;IACrC,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,SAAS,EAAE,OAAO,CAAC;CACpB;AAED;;GAEG;AACH,MAAM,WAAW,qBAAqB;IACpC;;;;;OAKG;IACH,iBAAiB,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,sBAAsB,CAAC,CAAC;IAErE;;;;;OAKG;IACH,iBAAiB,CAAC,QAAQ,EAAE,MAAM,GAAG,kBAAkB,GAAG,IAAI,CAAC;IAE/D;;;;;;OAMG;IACH,cAAc,CACZ,QAAQ,EAAE,MAAM,EAChB,QAAQ,CAAC,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,IAAI,GAChC,OAAO,CAAC,sBAAsB,CAAC,CAAC;IAEnC;;;;;;OAMG;IACH,sBAAsB,IAAI,OAAO,CAAC,sBAAsB,EAAE,CAAC,CAAC;CAC7D"}
+{"version":3,"file":"tool-installer.service.d.ts","sourceRoot":"","sources":["../../../../../../../../packages/core/src/application/ports/output/services/tool-installer.service.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EACV,sBAAsB,EACtB,kBAAkB,EACnB,MAAM,wCAAwC,CAAC;AAEhD;;GAEG;AACH,MAAM,WAAW,qBAAqB;IACpC;;;;;OAKG;IACH,iBAAiB,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,sBAAsB,CAAC,CAAC;IAErE;;;;;OAKG;IACH,iBAAiB,CAAC,QAAQ,EAAE,MAAM,GAAG,kBAAkB,GAAG,IAAI,CAAC;IAE/D;;;;;;OAMG;IACH,cAAc,CACZ,QAAQ,EAAE,MAAM,EAChB,QAAQ,CAAC,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,IAAI,GAChC,OAAO,CAAC,sBAAsB,CAAC,CAAC;CACpC"}

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

@@ -11,7 +11,7 @@
  * - Automatic timeout handling with process termination
  * - Immutable status objects via value object factory functions
  */
-import type { IToolInstallerService, AvailableTerminalEntry } from '../../../application/ports/output/services/tool-installer.service.js';
+import type { IToolInstallerService } from '../../../application/ports/output/services/tool-installer.service.js';
 import type { ToolInstallationStatus, ToolInstallCommand } from '../../../domain/generated/output.js';
 export declare class ToolInstallerServiceImpl implements IToolInstallerService {
     /**
@@ -29,12 +29,6 @@ export declare class ToolInstallerServiceImpl implements IToolInstallerService {
      * Streams stdout/stderr to optional callback and handles timeout automatically.
      */
     executeInstall(toolName: string, onOutput?: (data: string) => void): Promise<ToolInstallationStatus>;
-    /**
-     * List all terminal emulators supported on the current platform with availability status.
-     * TOOL_METADATA is loaded in the Node.js CLI context, so this is safe to call from the
-     * DI container even when the Next.js standalone bundle cannot resolve tools/ via import.meta.url.
-     */
-    listAvailableTerminals(): Promise<AvailableTerminalEntry[]>;
     /**
      * Create installation suggestions for a tool based on its metadata
      */

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,EACV,qBAAqB,EACrB,sBAAsB,EACvB,MAAM,sEAAsE,CAAC;AAC9E,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;;;;OAIG;IACG,sBAAsB,IAAI,OAAO,CAAC,sBAAsB,EAAE,CAAC;IAuCjE;;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

@@ -21,7 +21,7 @@ import { injectable } from 'tsyringe';
 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, getTerminalEntries } from './tool-metadata.js';
+import { TOOL_METADATA } from './tool-metadata.js';
 import { checkBinaryExists } from './binary-exists.js';
 /**
  * Resolve binary name for the current platform.
@@ -172,42 +172,6 @@ let ToolInstallerServiceImpl = class ToolInstallerServiceImpl {
             });
         });
     }
-    /**
-     * List all terminal emulators supported on the current platform with availability status.
-     * TOOL_METADATA is loaded in the Node.js CLI context, so this is safe to call from the
-     * DI container even when the Next.js standalone bundle cannot resolve tools/ via import.meta.url.
-     */
-    async listAvailableTerminals() {
-        const currentPlatform = platform();
-        const entries = getTerminalEntries();
-        const results = [];
-        for (const [id, meta] of entries) {
-            if (meta.platforms &&
-                !meta.platforms.includes(currentPlatform)) {
-                continue;
-            }
-            if (id === 'system-terminal') {
-                results.unshift({ id: 'system', name: meta.name, available: true });
-                continue;
-            }
-            const binary = typeof meta.binary === 'string' ? meta.binary : meta.binary[currentPlatform];
-            if (!binary)
-                continue;
-            let available = false;
-            if (meta.verifyCommand.startsWith('test ')) {
-                available = await runVerifyCommand(meta.verifyCommand);
-            }
-            else {
-                const result = await checkBinaryExists(binary);
-                available = result.found;
-            }
-            results.push({ id, name: meta.name, available });
-        }
-        if (!results.some((r) => r.id === 'system')) {
-            results.unshift({ id: 'system', name: 'System Terminal', available: true });
-        }
-        return results;
-    }
     /**
      * Create installation suggestions for a tool based on its metadata
      */

package/dist/src/presentation/web/app/actions/get-available-terminals.d.ts

@@ -4,14 +4,9 @@ export interface AvailableTerminal {
     available: boolean;
 }
 /**
- * Returns the list of terminal entries from the DI container's IToolInstallerService,
- * filtered to the current platform with availability checks.
- *
- * Using the DI container (not a direct import from tool-metadata) ensures that
- * TOOL_METADATA is read from the correct tools/ directory path — it is loaded once
- * in the Node.js CLI bootstrap context where import.meta.url resolves correctly.
- * Direct imports of tool-metadata from Next.js server action bundles break in
- * standalone production mode because import.meta.url points to the chunk file.
+ * Returns the list of terminal entries from tool metadata, filtered to the
+ * current platform, with availability checks. The "system-terminal" entry
+ * is always marked available and listed first.
  */
 export declare function getAvailableTerminals(): Promise<AvailableTerminal[]>;
 //# sourceMappingURL=get-available-terminals.d.ts.map

package/dist/src/presentation/web/app/actions/get-available-terminals.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"get-available-terminals.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/get-available-terminals.ts"],"names":[],"mappings":"AAKA,MAAM,WAAW,iBAAiB;IAChC,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,SAAS,EAAE,OAAO,CAAC;CACpB;AAED;;;;;;;;;GASG;AACH,wBAAsB,qBAAqB,IAAI,OAAO,CAAC,iBAAiB,EAAE,CAAC,CAO1E"}
+{"version":3,"file":"get-available-terminals.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/get-available-terminals.ts"],"names":[],"mappings":"AAMA,MAAM,WAAW,iBAAiB;IAChC,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,SAAS,EAAE,OAAO,CAAC;CACpB;AAED;;;;GAIG;AACH,wBAAsB,qBAAqB,IAAI,OAAO,CAAC,iBAAiB,EAAE,CAAC,CAkD1E"}

package/dist/src/presentation/web/app/actions/get-available-terminals.js

@@ -1,21 +1,53 @@
 'use server';
-import { resolve } from '../../lib/server-container.js';
+import { platform } from 'node:os';
+import { getTerminalEntries } from '../../../../../packages/core/src/infrastructure/services/tool-installer/tool-metadata.js';
+import { checkBinaryExists } from '../../../../../packages/core/src/infrastructure/services/tool-installer/binary-exists.js';
 /**
- * Returns the list of terminal entries from the DI container's IToolInstallerService,
- * filtered to the current platform with availability checks.
- *
- * Using the DI container (not a direct import from tool-metadata) ensures that
- * TOOL_METADATA is read from the correct tools/ directory path — it is loaded once
- * in the Node.js CLI bootstrap context where import.meta.url resolves correctly.
- * Direct imports of tool-metadata from Next.js server action bundles break in
- * standalone production mode because import.meta.url points to the chunk file.
+ * Returns the list of terminal entries from tool metadata, filtered to the
+ * current platform, with availability checks. The "system-terminal" entry
+ * is always marked available and listed first.
  */
 export async function getAvailableTerminals() {
-    try {
-        const service = resolve('IToolInstallerService');
-        return await service.listAvailableTerminals();
+    const currentPlatform = platform();
+    const entries = getTerminalEntries();
+    const results = [];
+    for (const [id, meta] of entries) {
+        // Skip terminals not supported on this platform
+        if (meta.platforms &&
+            !meta.platforms.includes(currentPlatform)) {
+            continue;
+        }
+        // System terminal is always available
+        if (id === 'system-terminal') {
+            results.unshift({ id: 'system', name: meta.name, available: true });
+            continue;
+        }
+        // Check binary availability
+        const binary = typeof meta.binary === 'string' ? meta.binary : meta.binary[currentPlatform];
+        if (!binary)
+            continue;
+        // For app-bundle terminals (Warp, iTerm2) that use verifyCommand instead of binary check
+        let available = false;
+        if (meta.verifyCommand.startsWith('test ')) {
+            // Use verifyCommand for app-bundle checks (e.g., "test -d /Applications/Warp.app")
+            try {
+                const { execSync } = await import('node:child_process');
+                execSync(meta.verifyCommand, { stdio: 'ignore' });
+                available = true;
+            }
+            catch {
+                available = false;
+            }
+        }
+        else {
+            const result = await checkBinaryExists(binary);
+            available = result.found;
+        }
+        results.push({ id, name: meta.name, available });
     }
-    catch {
-        return [{ id: 'system', name: 'System Terminal', available: true }];
+    // Ensure system terminal is always present even if not in tool metadata
+    if (!results.some((r) => r.id === 'system')) {
+        results.unshift({ id: 'system', name: 'System Terminal', available: true });
     }
+    return results;
 }