@shepai/cli 1.70.0 → 1.70.2

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

@@ -0,0 +1,7 @@
+export declare function createDeploymentLogger(prefix: string): {
+    info: (...args: unknown[]) => void;
+    debug: (...args: unknown[]) => void;
+    warn: (...args: unknown[]) => void;
+    error: (...args: unknown[]) => void;
+};
+//# sourceMappingURL=deployment-logger.d.ts.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"deployment-logger.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/infrastructure/services/deployment/deployment-logger.ts"],"names":[],"mappings":"AAWA,wBAAgB,sBAAsB,CAAC,MAAM,EAAE,MAAM;oBAIvB,OAAO,EAAE;qBACR,OAAO,EAAE;oBACpB,OAAO,EAAE;qBACR,OAAO,EAAE;EAE7B"}

package/dist/packages/core/src/infrastructure/services/deployment/deployment-logger.js

@@ -0,0 +1,17 @@
+/* eslint-disable no-console */
+/**
+ * Debug-gated logger for deployment services.
+ *
+ * - `info` and `debug` only emit when `process.env.DEBUG` is set.
+ * - `warn` and `error` always emit (they indicate real problems).
+ */
+const noop = () => undefined;
+export function createDeploymentLogger(prefix) {
+    const isDebug = !!process.env.DEBUG;
+    return {
+        info: isDebug ? (...args) => console.info(prefix, ...args) : noop,
+        debug: isDebug ? (...args) => console.debug(prefix, ...args) : noop,
+        warn: (...args) => console.warn(prefix, ...args),
+        error: (...args) => console.error(prefix, ...args),
+    };
+}

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;AAE9D,OAAO,KAAK,EACV,kBAAkB,EAClB,gBAAgB,EACjB,MAAM,qEAAqE,CAAC;AAC7E,OAAO,EAAE,eAAe,EAAE,MAAM,wBAAwB,CAAC;AAgBzD,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;gBAEjC,IAAI,GAAE,OAAO,CAAC,qBAAqB,CAAM;IAIrD;;;OAGG;IACH,KAAK,CAAC,QAAQ,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,GAAG,IAAI;IAmDjD;;;OAGG;IACH,SAAS,CAAC,QAAQ,EAAE,MAAM,GAAG,gBAAgB,GAAG,IAAI;IAMpD;;OAEG;IACG,IAAI,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IA6B3C;;OAEG;IACH,OAAO,IAAI,IAAI;IAMf;;OAEG;IACH,OAAO,CAAC,WAAW;IAQnB;;OAEG;IACH,OAAO,CAAC,oBAAoB;IA0B5B;;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;AAE9D,OAAO,KAAK,EACV,kBAAkB,EAClB,gBAAgB,EACjB,MAAM,qEAAqE,CAAC;AAC7E,OAAO,EAAE,eAAe,EAAE,MAAM,wBAAwB,CAAC;AAkBzD,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;gBAEjC,IAAI,GAAE,OAAO,CAAC,qBAAqB,CAAM;IAIrD;;;OAGG;IACH,KAAK,CAAC,QAAQ,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,GAAG,IAAI;IA8EjD;;;OAGG;IACH,SAAS,CAAC,QAAQ,EAAE,MAAM,GAAG,gBAAgB,GAAG,IAAI;IAYpD;;OAEG;IACG,IAAI,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAuC3C;;OAEG;IACH,OAAO,IAAI,IAAI;IAMf;;OAEG;IACH,OAAO,CAAC,WAAW;IAQnB;;OAEG;IACH,OAAO,CAAC,oBAAoB;IAoD5B;;OAEG;YACW,aAAa;IAW3B;;OAEG;IACH,OAAO,CAAC,WAAW;CASpB"}

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

@@ -9,7 +9,9 @@
 import { spawn } from 'node:child_process';
 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';
+const log = createDeploymentLogger('[DeploymentService]');
 const POLL_INTERVAL_MS = 200;
 const MAX_WAIT_MS = 5000;
 const defaultDeps = {
@@ -37,20 +39,24 @@ export class DeploymentService {
      * If a deployment already exists for this target, it is stopped first.
      */
     start(targetId, targetPath) {
+        log.info(`start() called — targetId="${targetId}", targetPath="${targetPath}"`);
         // Stop any existing deployment for this target
         const existing = this.deployments.get(targetId);
         if (existing) {
+            log.info(`Stopping existing deployment for "${targetId}" (pid=${existing.pid})`);
             this.killProcess(existing);
             this.deployments.delete(targetId);
         }
         // Detect the dev script
         const detection = this.deps.detectDevScript(targetPath);
         if (!detection.success) {
+            log.error(`Dev script detection failed: ${detection.error}`);
             throw new Error(detection.error);
         }
         // Build spawn args based on package manager
-        const { packageManager, scriptName } = detection;
+        const { packageManager, scriptName, command } = detection;
         const args = packageManager === 'npm' ? ['run', scriptName] : [scriptName];
+        log.info(`Spawning dev server: command="${command}", packageManager="${packageManager}", scriptName="${scriptName}", cwd="${targetPath}"`);
         const child = this.deps.spawn(packageManager, args, {
             shell: true,
             cwd: targetPath,
@@ -58,8 +64,10 @@ export class DeploymentService {
             stdio: ['ignore', 'pipe', 'pipe'],
         });
         if (!child.pid) {
+            log.error('spawn() returned no PID — process failed to start');
             throw new Error('Failed to spawn dev server: no PID returned');
         }
+        log.info(`Process spawned — pid=${child.pid}`);
         const entry = {
             pid: child.pid,
             child: child,
@@ -73,8 +81,19 @@ export class DeploymentService {
         // Attach stdout/stderr listeners for port detection
         this.attachOutputListener(entry, 'stdout');
         this.attachOutputListener(entry, 'stderr');
+        // Handle spawn errors (command not found, permission denied, etc.)
+        child.on('error', (err) => {
+            log.error(`Child process error for "${targetId}" (pid=${entry.pid}): ${err.message}`);
+            entry.state = DeploymentState.Stopped;
+            this.deployments.delete(targetId);
+        });
         // Clean up on process exit
-        child.on('exit', () => {
+        child.on('exit', (code, signal) => {
+            const wasBooting = entry.state === DeploymentState.Booting;
+            log.info(`Process exited for "${targetId}" (pid=${entry.pid}) — code=${code}, signal=${signal}, wasBooting=${wasBooting}`);
+            if (wasBooting) {
+                log.warn('Process exited while still in Booting state — dev server likely crashed on startup. Check stderr output above.');
+            }
             this.deployments.delete(targetId);
         });
     }
@@ -84,8 +103,11 @@ export class DeploymentService {
      */
     getStatus(targetId) {
         const entry = this.deployments.get(targetId);
-        if (!entry)
+        if (!entry) {
+            log.debug(`getStatus("${targetId}") — no deployment found`);
             return null;
+        }
+        log.debug(`getStatus("${targetId}") — state=${entry.state}, url=${entry.url}, pid=${entry.pid}`);
         return { state: entry.state, url: entry.url };
     }
     /**
@@ -93,20 +115,24 @@ export class DeploymentService {
      */
     async stop(targetId) {
         const entry = this.deployments.get(targetId);
-        if (!entry)
+        if (!entry) {
+            log.info(`stop("${targetId}") — no deployment found, nothing to stop`);
             return;
+        }
+        log.info(`stop("${targetId}") — sending SIGTERM to process group (pid=${entry.pid})`);
         // Send SIGTERM to process group
         try {
             this.deps.kill(-entry.pid, 'SIGTERM');
         }
         catch {
-            // Process may already be dead
+            log.info(`stop("${targetId}") — process already dead on SIGTERM`);
             this.deployments.delete(targetId);
             return;
         }
         // Wait for the process to exit
         const died = await this.pollUntilDead(entry.pid, MAX_WAIT_MS, POLL_INTERVAL_MS);
         if (!died) {
+            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');
@@ -115,6 +141,9 @@ export class DeploymentService {
                 // Process may have exited between check and kill
             }
         }
+        else {
+            log.info(`stop("${targetId}") — process exited gracefully`);
+        }
         // Wait for the exit event to clean up the map
         await this.waitForExit(entry.child);
     }
@@ -143,26 +172,50 @@ export class DeploymentService {
     attachOutputListener(entry, stream) {
         const bufferKey = stream === 'stdout' ? 'stdoutBuffer' : 'stderrBuffer';
         const childStream = entry.child[stream];
-        if (!childStream)
+        if (!childStream) {
+            log.warn(`[${entry.targetId}] No ${stream} stream available — cannot attach listener`);
             return;
+        }
         childStream.on('data', (chunk) => {
+            const text = chunk.toString();
             // Append chunk to buffer
-            entry[bufferKey] += chunk.toString();
+            entry[bufferKey] += text;
             // Process complete lines
             const lines = entry[bufferKey].split('\n');
             // Keep the last element (incomplete line) in the buffer
             entry[bufferKey] = lines.pop() ?? '';
             for (const line of lines) {
+                if (!line.trim())
+                    continue;
+                log.debug(`[${entry.targetId}] ${stream}: ${line}`);
                 if (entry.state !== DeploymentState.Booting)
                     break;
                 const url = parsePort(line);
                 if (url) {
+                    log.info(`[${entry.targetId}] Port detected — url="${url}" (from ${stream})`);
                     entry.state = DeploymentState.Ready;
                     entry.url = url;
                     break;
                 }
             }
         });
+        childStream.on('end', () => {
+            // Flush remaining buffer content
+            const remaining = entry[bufferKey].trim();
+            if (remaining) {
+                log.debug(`[${entry.targetId}] ${stream} (flush): ${remaining}`);
+                if (entry.state === DeploymentState.Booting) {
+                    const url = parsePort(remaining);
+                    if (url) {
+                        log.info(`[${entry.targetId}] Port detected in flushed buffer — url="${url}"`);
+                        entry.state = DeploymentState.Ready;
+                        entry.url = url;
+                    }
+                }
+                entry[bufferKey] = '';
+            }
+            log.debug(`[${entry.targetId}] ${stream} stream ended`);
+        });
     }
     /**
      * Poll until a process is dead or timeout expires.

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;AAeH,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;AAElF;;;;;GAKG;AACH,wBAAgB,eAAe,CAAC,OAAO,EAAE,MAAM,GAAG,qBAAqB,CA4BtE"}
+{"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"}

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

@@ -7,6 +7,7 @@
  */
 import { readFileSync, existsSync } from 'node:fs';
 import { join } from 'node:path';
+import { createDeploymentLogger } from './deployment-logger.js';
 /** Script names to search for, in priority order */
 const SCRIPT_PRIORITY = ['dev', 'start', 'serve'];
 /** Lockfile-to-package-manager mapping, checked in order */
@@ -15,6 +16,7 @@ const LOCKFILE_MANAGERS = [
     { lockfile: 'yarn.lock', manager: 'yarn' },
     { lockfile: 'package-lock.json', manager: 'npm' },
 ];
+const log = createDeploymentLogger('[detectDevScript]');
 /**
  * Detect the dev script and package manager for a project directory.
  *
@@ -22,28 +24,33 @@ const LOCKFILE_MANAGERS = [
  * @returns Detection result with command info, or an error
  */
 export function detectDevScript(dirPath) {
+    log.info(`scanning dirPath="${dirPath}"`);
     // Read and parse package.json
     let packageJson;
     try {
         const raw = readFileSync(join(dirPath, 'package.json'), 'utf-8');
         packageJson = JSON.parse(raw);
     }
-    catch {
-        return { success: false, error: `No package.json found in ${dirPath}` };
+    catch (err) {
+        const msg = `No package.json found in ${dirPath}`;
+        log.error(msg, err);
+        return { success: false, error: msg };
     }
     // Find the first matching script in priority order
     const scripts = packageJson.scripts ?? {};
+    const availableScripts = Object.keys(scripts);
+    log.info(`available scripts: [${availableScripts.join(', ')}], looking for: [${SCRIPT_PRIORITY.join(', ')}]`);
     const scriptName = SCRIPT_PRIORITY.find((name) => name in scripts);
     if (!scriptName) {
-        return {
-            success: false,
-            error: `No dev script found in package.json. Expected one of: ${SCRIPT_PRIORITY.join(', ')}`,
-        };
+        const msg = `No dev script found in package.json. Expected one of: ${SCRIPT_PRIORITY.join(', ')}`;
+        log.warn(msg);
+        return { success: false, error: msg };
     }
     // Detect package manager from lockfile
     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 };
 }
 /**

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

@@ -1 +1 @@
-{"version":3,"file":"deploy-feature.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/deploy-feature.ts"],"names":[],"mappings":"AAOA,OAAO,EAAE,eAAe,EAAE,MAAM,sCAAsC,CAAC;AAEvE,wBAAsB,aAAa,CACjC,SAAS,EAAE,MAAM,GAChB,OAAO,CAAC;IAAE,OAAO,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAC;IAAC,KAAK,CAAC,EAAE,eAAe,CAAA;CAAE,CAAC,CA2BxE"}
+{"version":3,"file":"deploy-feature.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/deploy-feature.ts"],"names":[],"mappings":"AAQA,OAAO,EAAE,eAAe,EAAE,MAAM,sCAAsC,CAAC;AAIvE,wBAAsB,aAAa,CACjC,SAAS,EAAE,MAAM,GAChB,OAAO,CAAC;IAAE,OAAO,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAC;IAAC,KAAK,CAAC,EAAE,eAAe,CAAA;CAAE,CAAC,CAwCxE"}

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

@@ -1,28 +1,39 @@
 'use server';
 import { existsSync } from 'node:fs';
 import { resolve } from '../../lib/server-container.js';
+import { createDeploymentLogger } from '../../../../../packages/core/src/infrastructure/services/deployment/deployment-logger.js';
 import { computeWorktreePath } from '../../../../../packages/core/src/infrastructure/services/ide-launchers/compute-worktree-path.js';
 import { DeploymentState } from '../../../../../packages/core/src/domain/generated/output.js';
+const log = createDeploymentLogger('[deployFeature]');
 export async function deployFeature(featureId) {
+    log.info(`called — featureId="${featureId}"`);
     if (!featureId?.trim()) {
+        log.warn('rejected — featureId is empty');
         return { success: false, error: 'featureId is required' };
     }
     try {
         const featureRepo = resolve('IFeatureRepository');
         const feature = await featureRepo.findById(featureId);
         if (!feature) {
+            log.warn(`feature not found in repository: "${featureId}"`);
             return { success: false, error: `Feature not found: ${featureId}` };
         }
+        log.info(`feature found — repositoryPath="${feature.repositoryPath}", branch="${feature.branch}"`);
         const worktreePath = computeWorktreePath(feature.repositoryPath, feature.branch);
+        log.info(`computed worktreePath="${worktreePath}"`);
         if (!existsSync(worktreePath)) {
+            log.warn(`worktree path does not exist on disk: "${worktreePath}"`);
             return { success: false, error: `Worktree path does not exist: ${worktreePath}` };
         }
+        log.info('worktree path exists, calling deploymentService.start()');
         const deploymentService = resolve('IDeploymentService');
         deploymentService.start(featureId, worktreePath);
+        log.info('start() returned successfully — state=Booting');
         return { success: true, state: DeploymentState.Booting };
     }
     catch (error) {
         const message = error instanceof Error ? error.message : 'Failed to deploy feature';
+        log.error(`error: ${message}`, error);
         return { success: false, error: message };
     }
 }

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":"AAKA,OAAO,EAAE,eAAe,EAAE,MAAM,sCAAsC,CAAC;AAEvE,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,CAkBxE"}
+{"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"}

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

@@ -1,21 +1,29 @@
 'use server';
 import { existsSync } from 'node:fs';
 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('/')) {
+        log.warn('rejected — not an absolute path');
         return { success: false, error: 'repositoryPath must be an absolute path' };
     }
     try {
         if (!existsSync(repositoryPath)) {
+            log.warn(`directory does not exist: "${repositoryPath}"`);
             return { success: false, error: `Directory does not exist: ${repositoryPath}` };
         }
+        log.info('directory exists, calling deploymentService.start()');
         const deploymentService = resolve('IDeploymentService');
         deploymentService.start(repositoryPath, repositoryPath);
+        log.info('start() returned successfully — state=Booting');
         return { success: true, state: DeploymentState.Booting };
     }
     catch (error) {
         const message = error instanceof Error ? error.message : 'Failed to deploy repository';
+        log.error(`error: ${message}`, error);
         return { success: false, error: message };
     }
 }

package/dist/src/presentation/web/components/common/base-drawer/base-drawer.js

@@ -44,7 +44,7 @@ export function BaseDrawer({ open, onClose, modal = false, size, header, childre
     return (_jsxs(Drawer, { direction: "right", modal: modal, handleOnly: true, open: open, onOpenChange: (isOpen) => {
             if (!isOpen)
                 onClose();
-        }, children: [modal ? _jsx(DrawerOverlay, {}) : null, _jsxs(DrawerContent, { ref: contentRef, direction: "right", showCloseButton: false, className: cn(drawerVariants({ size }), className), "data-testid": testId, children: [_jsxs("button", { type: "button", "aria-label": "Close", onClick: onClose, className: "ring-offset-background focus:ring-ring absolute top-4 right-4 rounded-xs opacity-70 transition-opacity hover:opacity-100 focus:ring-2 focus:ring-offset-2 focus:outline-hidden", "data-testid": testId ? `${testId}-close-button` : undefined, children: [_jsx(XIcon, { className: "size-4" }), _jsx("span", { className: "sr-only", children: "Close" })] }), header ? _jsx(DrawerHeader, { className: "shrink-0", children: header }) : null, header ? _jsx(Separator, {}) : null, featureFlags.envDeploy && deployTarget ? _jsx(DeployBar, { deployTarget: deployTarget }) : null, _jsx("div", { className: "flex min-h-0 flex-1 flex-col", children: children }), footer ? _jsx(DrawerFooter, { className: "shrink-0", children: footer }) : null] })] }));
+        }, children: [modal ? _jsx(DrawerOverlay, {}) : null, _jsxs(DrawerContent, { ref: contentRef, direction: "right", showCloseButton: false, className: cn(drawerVariants({ size }), className), "data-testid": testId, children: [_jsxs("button", { type: "button", "aria-label": "Close", onClick: onClose, className: "ring-offset-background focus:ring-ring absolute top-4 right-4 rounded-xs opacity-70 transition-opacity hover:opacity-100 focus:ring-2 focus:ring-offset-2 focus:outline-hidden", "data-testid": testId ? `${testId}-close-button` : undefined, children: [_jsx(XIcon, { className: "size-4" }), _jsx("span", { className: "sr-only", children: "Close" })] }), header ? _jsx(DrawerHeader, { className: "shrink-0", children: header }) : null, header ? _jsx(Separator, {}) : null, featureFlags.envDeploy && deployTarget ? _jsx(DeployBar, { deployTarget: deployTarget }) : null, _jsx("div", { className: "flex min-h-0 flex-1 flex-col overflow-hidden", children: children }), footer ? _jsx(DrawerFooter, { className: "shrink-0", children: footer }) : null] })] }));
 }
 function DeployBar({ deployTarget }) {
     const deployAction = useDeployAction(deployTarget);

package/dist/src/presentation/web/components/common/base-drawer/base-drawer.stories.js

@@ -75,5 +75,5 @@ export const WithDevServerBar = {
  * Try scrolling the content below to see the fixed header behavior.
  */
 export const ScrollableContent = {
-    render: () => (_jsx(DrawerTrigger, { header: _jsxs(_Fragment, { children: [_jsx(DrawerTitle, { children: "Scrollable Content" }), _jsx(DrawerDescription, { children: "Content below overflows and scrolls" })] }), children: _jsx("div", { className: "flex flex-col gap-4 p-4", children: Array.from({ length: 30 }, (_, i) => (_jsxs("div", { className: "border-border rounded-md border p-3", children: [_jsxs("h4", { className: "text-sm font-medium", children: ["Item ", i + 1] }), _jsx("p", { className: "text-muted-foreground text-xs", children: "This is a scrollable content item to demonstrate overflow handling." })] }, i))) }) })),
+    render: () => (_jsx(DrawerTrigger, { header: _jsxs(_Fragment, { children: [_jsx(DrawerTitle, { children: "Scrollable Content" }), _jsx(DrawerDescription, { children: "Content below overflows and scrolls" })] }), children: _jsx("div", { className: "flex flex-col gap-4 overflow-y-auto p-4", children: Array.from({ length: 30 }, (_, i) => (_jsxs("div", { className: "border-border rounded-md border p-3", children: [_jsxs("h4", { className: "text-sm font-medium", children: ["Item ", i + 1] }), _jsx("p", { className: "text-muted-foreground text-xs", children: "This is a scrollable content item to demonstrate overflow handling." })] }, i))) }) })),
 };

package/dist/src/presentation/web/components/common/control-center-drawer/control-center-drawer.js

@@ -273,7 +273,7 @@ export function ControlCenterDrawer({ view, onClose, onDelete, isDeleting, onCre
         body = mergeData ? (_jsx(MergeReview, { data: mergeData, onApprove: handleMergeApprove, onReject: handleMergeReject, isProcessing: isLoadingMerge, isRejecting: isRejecting })) : (_jsx("div", { className: "flex items-center justify-center p-8", children: _jsx(Loader2, { className: "text-muted-foreground h-6 w-6 animate-spin" }) }));
     }
     else if (view?.type === 'repository' && repoData?.repositoryPath) {
-        body = (_jsxs(_Fragment, { children: [_jsx(Separator, {}), _jsxs("div", { className: "flex flex-col gap-3 p-4", children: [_jsx("div", { className: "text-muted-foreground text-xs font-semibold tracking-wider", children: "OPEN WITH" }), _jsxs("div", { className: "flex flex-col gap-2", children: [_jsx(ActionButton, { label: "Open in IDE", onClick: repoActions.openInIde, loading: repoActions.ideLoading, error: !!repoActions.ideError, icon: Code2, variant: "outline", size: "sm" }), _jsx(ActionButton, { label: "Open in Shell", onClick: repoActions.openInShell, loading: repoActions.shellLoading, error: !!repoActions.shellError, icon: Terminal, variant: "outline", size: "sm" }), _jsx(ActionButton, { label: "Open Folder", onClick: repoActions.openFolder, loading: repoActions.folderLoading, error: !!repoActions.folderError, icon: FolderOpen, variant: "outline", size: "sm" })] })] })] }));
+        body = (_jsxs("div", { className: "flex-1 overflow-y-auto", children: [_jsx(Separator, {}), _jsxs("div", { className: "flex flex-col gap-3 p-4", children: [_jsx("div", { className: "text-muted-foreground text-xs font-semibold tracking-wider", children: "OPEN WITH" }), _jsxs("div", { className: "flex flex-col gap-2", children: [_jsx(ActionButton, { label: "Open in IDE", onClick: repoActions.openInIde, loading: repoActions.ideLoading, error: !!repoActions.ideError, icon: Code2, variant: "outline", size: "sm" }), _jsx(ActionButton, { label: "Open in Shell", onClick: repoActions.openInShell, loading: repoActions.shellLoading, error: !!repoActions.shellError, icon: Terminal, variant: "outline", size: "sm" }), _jsx(ActionButton, { label: "Open Folder", onClick: repoActions.openFolder, loading: repoActions.folderLoading, error: !!repoActions.folderError, icon: FolderOpen, variant: "outline", size: "sm" })] })] })] }));
     }
     // ── Render ──────────────────────────────────────────────────────────────
     return (_jsxs(_Fragment, { children: [_jsx(BaseDrawer, { open: view !== null && !isCreateView, onClose: onClose, size: "md", modal: false, header: header, deployTarget: repoDeployTarget, "data-testid": view?.type === 'feature'

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

@@ -9,7 +9,7 @@ function PhaseList({ phases }) {
 }
 export function MergeReview({ data, onApprove, onReject, isProcessing = false, isRejecting = false, }) {
     const { pr, diffSummary, phases, branch, warning } = data;
-    return (_jsxs("div", { className: "flex flex-1 flex-col", children: [_jsxs("div", { className: "flex-1 space-y-4 overflow-y-auto p-4", children: [_jsxs("div", { className: "flex items-start gap-3", children: [_jsx("div", { className: "mt-1.5 h-2.5 w-2.5 shrink-0 rounded-full bg-emerald-500" }), _jsxs("div", { className: "flex-1", children: [_jsx("h2", { className: "text-foreground text-sm font-bold", children: "Merge Review" }), _jsx("p", { className: "text-muted-foreground mt-1 text-xs leading-relaxed", children: pr
+    return (_jsxs("div", { className: "flex min-h-0 flex-1 flex-col", children: [_jsxs("div", { className: "flex-1 space-y-4 overflow-y-auto p-4", children: [_jsxs("div", { className: "flex items-start gap-3", children: [_jsx("div", { className: "mt-1.5 h-2.5 w-2.5 shrink-0 rounded-full bg-emerald-500" }), _jsxs("div", { className: "flex-1", children: [_jsx("h2", { className: "text-foreground text-sm font-bold", children: "Merge Review" }), _jsx("p", { className: "text-muted-foreground mt-1 text-xs leading-relaxed", children: pr
                                             ? 'Review the pull request details and approve to merge.'
                                             : 'Review the changes and approve to merge.' })] })] }), branch ? (_jsx("div", { className: "border-border rounded-lg border", children: _jsxs("div", { className: "flex items-center gap-2 px-4 py-3", children: [_jsx(GitBranch, { className: "text-muted-foreground h-4 w-4 shrink-0" }), _jsx(Badge, { variant: "secondary", className: "font-mono text-[11px]", children: branch.source }), _jsx(ArrowRight, { className: "text-muted-foreground h-3.5 w-3.5 shrink-0" }), _jsx(Badge, { variant: "outline", className: "font-mono text-[11px]", children: branch.target })] }) })) : null, pr ? (_jsx("div", { className: "border-border rounded-lg border", children: _jsxs("div", { className: "space-y-3 px-4 py-3", children: [_jsxs("div", { className: "flex items-center justify-between", children: [_jsxs("a", { href: pr.url, target: "_blank", rel: "noopener noreferrer", className: "text-primary flex items-center gap-1.5 text-sm font-semibold underline underline-offset-2", children: ["PR #", pr.number, _jsx(ExternalLink, { className: "h-3.5 w-3.5" })] }), _jsx(Badge, { variant: "outline", className: "text-xs", children: pr.status })] }), pr.ciStatus ? (_jsxs("div", { className: "flex items-center justify-between", children: [_jsx("span", { className: "text-muted-foreground text-xs font-medium", children: "CI Status" }), _jsx(CiStatusBadge, { status: pr.ciStatus })] })) : null, pr.commitHash ? (_jsxs("div", { className: "flex items-center justify-between", children: [_jsx("span", { className: "text-muted-foreground text-xs font-medium", children: "Commit" }), _jsxs("div", { className: "flex items-center gap-1.5", children: [_jsx(GitCommitHorizontal, { className: "text-muted-foreground h-3.5 w-3.5" }), _jsx("code", { className: "bg-muted text-foreground rounded-md px-1.5 py-0.5 font-mono text-[11px]", children: pr.commitHash.slice(0, 7) })] })] })) : null] }) })) : null, diffSummary ? (_jsx("div", { className: "border-border rounded-lg border", children: _jsxs("div", { className: "px-4 py-3", children: [_jsxs("div", { className: "mb-3 flex items-center gap-2", children: [_jsx(FileDiff, { className: "text-muted-foreground h-4 w-4" }), _jsx("span", { className: "text-foreground text-xs font-semibold", children: "Changes" })] }), _jsxs("div", { className: "grid grid-cols-4 gap-3", children: [_jsxs("div", { className: "text-center", children: [_jsx("div", { className: "text-foreground text-sm font-bold", children: diffSummary.filesChanged }), _jsx("div", { className: "text-muted-foreground text-[10px]", children: "files" })] }), _jsxs("div", { className: "text-center", children: [_jsxs("div", { className: "text-sm font-bold text-green-600", children: ["+", diffSummary.additions] }), _jsx("div", { className: "text-muted-foreground text-[10px]", children: "additions" })] }), _jsxs("div", { className: "text-center", children: [_jsxs("div", { className: "text-sm font-bold text-red-600", children: ["-", diffSummary.deletions] }), _jsx("div", { className: "text-muted-foreground text-[10px]", children: "deletions" })] }), _jsxs("div", { className: "text-center", children: [_jsx("div", { className: "text-foreground text-sm font-bold", children: diffSummary.commitCount }), _jsx("div", { className: "text-muted-foreground text-[10px]", children: "commits" })] })] })] }) })) : warning ? (_jsx("div", { className: "border-border rounded-lg border", children: _jsxs("div", { className: "flex items-center gap-2 px-4 py-3", children: [_jsx(AlertTriangle, { className: "text-muted-foreground h-4 w-4 shrink-0" }), _jsx("span", { className: "text-muted-foreground text-xs", children: warning })] }) })) : null, phases && phases.length > 0 ? _jsx(PhaseList, { phases: phases }) : null] }), _jsx(DrawerActionBar, { onReject: onReject, onApprove: onApprove, approveLabel: "Approve Merge", approveIcon: isProcessing ? (_jsx(Loader2, { className: "mr-1.5 h-4 w-4 animate-spin" })) : (_jsx(GitMerge, { className: "mr-1.5 h-4 w-4" })), revisionPlaceholder: "Ask AI to revise before merging...", isProcessing: isProcessing, isRejecting: isRejecting })] }));
 }

package/dist/src/presentation/web/components/common/prd-questionnaire/prd-questionnaire.js

@@ -27,7 +27,7 @@ export function PrdQuestionnaire({ data, selections, onSelect, onApprove, onReje
     }, [onSelect, isLastStep, selectSound]);
     if (total === 0)
         return null;
-    return (_jsxs("div", { className: "flex flex-1 flex-col", children: [_jsxs("div", { className: "flex-1 space-y-4 overflow-y-auto p-4", children: [showHeader ? (_jsxs("div", { className: "border-border flex items-start gap-3 border-b pb-3", children: [_jsx("div", { className: "mt-1.5 h-2 w-2 shrink-0 rounded-full bg-amber-500" }), _jsxs("div", { className: "flex-1", children: [_jsx("h3", { className: "text-foreground mb-1.5 text-sm font-bold", children: question }), _jsx("p", { className: "text-muted-foreground text-xs leading-relaxed", children: context })] })] })) : null, _jsxs("div", { className: "space-y-3", children: [_jsxs("div", { className: "flex items-start gap-3", children: [_jsx("label", { className: "text-foreground min-w-0 flex-1 text-sm font-semibold", children: currentQuestion.question }), _jsx("div", { className: "mt-1.5 flex shrink-0 gap-1", children: questions.map((q, idx) => (_jsx("button", { type: "button", "aria-label": `Go to question ${idx + 1}`, className: cn('h-1.5 rounded-full transition-all duration-200', idx === currentStep ? 'bg-primary w-4' : 'w-1.5', idx !== currentStep && selections[q.id] ? 'bg-primary/50' : '', idx !== currentStep && !selections[q.id] ? 'bg-muted-foreground/25' : ''), onClick: () => {
+    return (_jsxs("div", { className: "flex min-h-0 flex-1 flex-col", children: [_jsxs("div", { className: "flex-1 space-y-4 overflow-y-auto p-4", children: [showHeader ? (_jsxs("div", { className: "border-border flex items-start gap-3 border-b pb-3", children: [_jsx("div", { className: "mt-1.5 h-2 w-2 shrink-0 rounded-full bg-amber-500" }), _jsxs("div", { className: "flex-1", children: [_jsx("h3", { className: "text-foreground mb-1.5 text-sm font-bold", children: question }), _jsx("p", { className: "text-muted-foreground text-xs leading-relaxed", children: context })] })] })) : null, _jsxs("div", { className: "space-y-3", children: [_jsxs("div", { className: "flex items-start gap-3", children: [_jsx("label", { className: "text-foreground min-w-0 flex-1 text-sm font-semibold", children: currentQuestion.question }), _jsx("div", { className: "mt-1.5 flex shrink-0 gap-1", children: questions.map((q, idx) => (_jsx("button", { type: "button", "aria-label": `Go to question ${idx + 1}`, className: cn('h-1.5 rounded-full transition-all duration-200', idx === currentStep ? 'bg-primary w-4' : 'w-1.5', idx !== currentStep && selections[q.id] ? 'bg-primary/50' : '', idx !== currentStep && !selections[q.id] ? 'bg-muted-foreground/25' : ''), onClick: () => {
                                                 navigateSound.play();
                                                 setCurrentStep(idx);
                                             } }, q.id))) })] }), _jsx("div", { className: "space-y-2", children: currentQuestion.options.map((opt, optIdx) => {

package/dist/src/presentation/web/components/common/tech-decisions-review/tech-decisions-review.js

@@ -38,5 +38,5 @@ export function TechDecisionsReview({ data, onApprove, onReject, isProcessing =
     const { summary, decisions } = data;
     if (decisions.length === 0)
         return null;
-    return (_jsxs("div", { className: "flex flex-1 flex-col", children: [_jsxs("div", { className: "flex-1 space-y-4 overflow-y-auto p-4", children: [_jsxs("div", { className: "flex items-start gap-3", children: [_jsx("div", { className: "mt-1.5 h-2.5 w-2.5 shrink-0 rounded-full bg-blue-500" }), _jsxs("div", { className: "flex-1", children: [_jsx("h2", { className: "text-foreground text-sm font-bold", children: "Technical Implementation Plan Review" }), summary ? (_jsx("p", { className: "text-muted-foreground mt-1 text-xs leading-relaxed", children: summary })) : null] })] }), _jsxs("div", { className: "flex items-center gap-2 pt-1", children: [_jsx(GitCompareArrows, { className: "text-primary h-4 w-4" }), _jsx("h3", { className: "text-foreground text-sm font-bold", children: "Technical Decisions" })] }), decisions.map((decision, i) => (_jsx(DecisionCard, { decision: decision, index: i }, decision.title)))] }), _jsx(DrawerActionBar, { onReject: onReject, onApprove: onApprove, approveLabel: "Approve Plan", approveIcon: _jsx(Check, { className: "mr-1.5 h-4 w-4" }), revisionPlaceholder: "Ask AI to revise the plan...", isProcessing: isProcessing, isRejecting: isRejecting })] }));
+    return (_jsxs("div", { className: "flex min-h-0 flex-1 flex-col", children: [_jsxs("div", { className: "flex-1 space-y-4 overflow-y-auto p-4", children: [_jsxs("div", { className: "flex items-start gap-3", children: [_jsx("div", { className: "mt-1.5 h-2.5 w-2.5 shrink-0 rounded-full bg-blue-500" }), _jsxs("div", { className: "flex-1", children: [_jsx("h2", { className: "text-foreground text-sm font-bold", children: "Technical Implementation Plan Review" }), summary ? (_jsx("p", { className: "text-muted-foreground mt-1 text-xs leading-relaxed", children: summary })) : null] })] }), _jsxs("div", { className: "flex items-center gap-2 pt-1", children: [_jsx(GitCompareArrows, { className: "text-primary h-4 w-4" }), _jsx("h3", { className: "text-foreground text-sm font-bold", children: "Technical Decisions" })] }), decisions.map((decision, i) => (_jsx(DecisionCard, { decision: decision, index: i }, decision.title)))] }), _jsx(DrawerActionBar, { onReject: onReject, onApprove: onApprove, approveLabel: "Approve Plan", approveIcon: _jsx(Check, { className: "mr-1.5 h-4 w-4" }), revisionPlaceholder: "Ask AI to revise the plan...", isProcessing: isProcessing, isRejecting: isRejecting })] }));
 }

package/dist/src/presentation/web/hooks/use-deploy-action.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"use-deploy-action.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/hooks/use-deploy-action.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,sCAAsC,CAAC;AAM5E,MAAM,WAAW,iBAAiB;IAChC,QAAQ,EAAE,MAAM,CAAC;IACjB,UAAU,EAAE,SAAS,GAAG,YAAY,CAAC;IACrC,cAAc,EAAE,MAAM,CAAC;IACvB,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,iBAAiB;IAChC,MAAM,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IAC5B,IAAI,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IAC1B,aAAa,EAAE,OAAO,CAAC;IACvB,WAAW,EAAE,OAAO,CAAC;IACrB,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,MAAM,EAAE,eAAe,GAAG,IAAI,CAAC;IAC/B,GAAG,EAAE,MAAM,GAAG,IAAI,CAAC;CACpB;AAKD,wBAAgB,eAAe,CAAC,KAAK,EAAE,iBAAiB,GAAG,IAAI,GAAG,iBAAiB,CAmIlF"}
+{"version":3,"file":"use-deploy-action.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/hooks/use-deploy-action.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,sCAAsC,CAAC;AAM5E,MAAM,WAAW,iBAAiB;IAChC,QAAQ,EAAE,MAAM,CAAC;IACjB,UAAU,EAAE,SAAS,GAAG,YAAY,CAAC;IACrC,cAAc,EAAE,MAAM,CAAC;IACvB,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB;AAED,MAAM,WAAW,iBAAiB;IAChC,MAAM,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IAC5B,IAAI,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;IAC1B,aAAa,EAAE,OAAO,CAAC;IACvB,WAAW,EAAE,OAAO,CAAC;IACrB,WAAW,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,MAAM,EAAE,eAAe,GAAG,IAAI,CAAC;IAC/B,GAAG,EAAE,MAAM,GAAG,IAAI,CAAC;CACpB;AAeD,wBAAgB,eAAe,CAAC,KAAK,EAAE,iBAAiB,GAAG,IAAI,GAAG,iBAAiB,CAgKlF"}

package/dist/src/presentation/web/hooks/use-deploy-action.js

@@ -1,9 +1,19 @@
 'use client';
+/* eslint-disable no-console */
 import { useState, useCallback, useRef, useEffect } from 'react';
 import { deployFeature } from '../app/actions/deploy-feature.js';
 import { deployRepository } from '../app/actions/deploy-repository.js';
 import { stopDeployment } from '../app/actions/stop-deployment.js';
 import { getDeploymentStatus } from '../app/actions/get-deployment-status.js';
+const PREFIX = '[useDeployAction]';
+const isDebug = !!process.env.NEXT_PUBLIC_DEBUG;
+const noop = () => undefined;
+const log = {
+    info: isDebug ? (...args) => console.info(PREFIX, ...args) : noop,
+    debug: isDebug ? (...args) => console.debug(PREFIX, ...args) : noop,
+    warn: (...args) => console.warn(PREFIX, ...args),
+    error: (...args) => console.error(PREFIX, ...args),
+};
 const ERROR_CLEAR_DELAY = 5000;
 const POLL_INTERVAL = 3000;
 export function useDeployAction(input) {
@@ -33,12 +43,14 @@ export function useDeployAction(input) {
     }, []);
     const stopPolling = useCallback(() => {
         if (pollIntervalRef.current) {
+            log.debug('stopping polling');
             clearInterval(pollIntervalRef.current);
             pollIntervalRef.current = null;
         }
     }, []);
     const startPolling = useCallback((targetId) => {
         stopPolling();
+        log.debug(`starting polling for "${targetId}" every ${POLL_INTERVAL}ms`);
         pollIntervalRef.current = setInterval(async () => {
             if (!mountedRef.current) {
                 stopPolling();
@@ -48,19 +60,30 @@ export function useDeployAction(input) {
             if (!mountedRef.current)
                 return;
             if (!result || result.state === 'Stopped') {
+                log.info(`poll result: ${result ? `state=${result.state}` : 'null (deployment gone)'} — stopping poll`);
                 setStatus(null);
                 setUrl(null);
                 stopPolling();
             }
             else {
+                if (result.state !== status) {
+                    log.info(`poll state changed: ${status} → ${result.state}, url=${result.url}`);
+                }
                 setStatus(result.state);
                 setUrl(result.url);
             }
         }, POLL_INTERVAL);
-    }, [stopPolling]);
+    }, [stopPolling, status]);
     const handleDeploy = useCallback(async () => {
-        if (!input || deployLoading)
+        if (!input) {
+            log.warn('deploy() called but input is null — no-op');
             return;
+        }
+        if (deployLoading) {
+            log.warn('deploy() called but already loading — no-op');
+            return;
+        }
+        log.info(`deploy() — targetType="${input.targetType}", targetId="${input.targetId}", repositoryPath="${input.repositoryPath}"`);
         if (errorTimerRef.current)
             clearTimeout(errorTimerRef.current);
         setDeployLoading(true);
@@ -69,14 +92,19 @@ export function useDeployAction(input) {
             const result = input.targetType === 'feature'
                 ? await deployFeature(input.targetId)
                 : await deployRepository(input.repositoryPath);
-            if (!mountedRef.current)
+            log.info('server action result:', result);
+            if (!mountedRef.current) {
+                log.warn('component unmounted after deploy — discarding result');
                 return;
+            }
             if (!result.success) {
                 const errorMessage = result.error ?? 'An unexpected error occurred';
+                log.error(`deploy failed: ${errorMessage}`);
                 setDeployError(errorMessage);
                 errorTimerRef.current = setTimeout(() => setDeployError(null), ERROR_CLEAR_DELAY);
             }
             else {
+                log.info(`deploy succeeded — initial state=${result.state}, starting polling`);
                 setStatus(result.state ?? null);
                 setUrl(null);
                 startPolling(input.targetId);
@@ -86,6 +114,7 @@ export function useDeployAction(input) {
             if (!mountedRef.current)
                 return;
             const errorMessage = err instanceof Error ? err.message : 'An unexpected error occurred';
+            log.error(`deploy threw exception: ${errorMessage}`, err);
             setDeployError(errorMessage);
             errorTimerRef.current = setTimeout(() => setDeployError(null), ERROR_CLEAR_DELAY);
         }
@@ -98,9 +127,11 @@ export function useDeployAction(input) {
     const handleStop = useCallback(async () => {
         if (!input || stopLoading)
             return;
+        log.info(`stop() — targetId="${input.targetId}"`);
         setStopLoading(true);
         try {
             const result = await stopDeployment(input.targetId);
+            log.info('stop result:', result);
             if (!mountedRef.current)
                 return;
             if (result.success) {
@@ -109,8 +140,8 @@ export function useDeployAction(input) {
                 setUrl(null);
             }
         }
-        catch {
-            // Stop errors are non-critical; process may have already exited
+        catch (err) {
+            log.warn('stop error (non-critical):', err);
         }
         finally {
             if (mountedRef.current) {