@shepai/cli 1.125.0 → 1.126.0

package/dist/packages/core/src/application/ports/output/services/deployment-service.interface.d.ts

@@ -9,6 +9,7 @@
  * - Application layer depends on this interface
  * - Infrastructure layer provides concrete implementations
  */
+import type Database from 'better-sqlite3';
 import type { DeploymentState } from '../../../../domain/generated/output.js';
 /** A single log line captured from a deployment's stdout or stderr. */
 export interface LogEntry {
@@ -39,16 +40,26 @@ export interface DeploymentStatus {
  * - Support graceful shutdown (SIGTERM → SIGKILL) for individual and bulk stops
  */
 export interface IDeploymentService {
+    /**
+     * Inject the database connection for persistence.
+     * Must be called before start/stop/getStatus to enable DB persistence.
+     */
+    setDatabase(db: Database.Database): void;
+    /**
+     * Recover dev servers from the database on startup.
+     * Validates each PID is still alive; removes dead rows.
+     */
+    recoverAll(): void;
     /**
      * Start a dev server deployment for the given target.
      * If a deployment already exists for this targetId, it is stopped first.
      *
      * @param targetId - Unique identifier for the deployment target (featureId or repositoryId)
      * @param targetPath - Absolute filesystem path to the directory to run the dev server in
-     * @returns The initial deployment state (always Booting on success)
+     * @param targetType - Type of target ('feature' or 'repository')
      * @throws Error if no dev script is found in package.json or the process fails to spawn
      */
-    start(targetId: string, targetPath: string): void;
+    start(targetId: string, targetPath: string, targetType?: string): void;
     /**
      * Stop a running deployment gracefully.
      * Sends SIGTERM to the process group, then SIGKILL after a timeout.

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

@@ -1 +1 @@
-{"version":3,"file":"deployment-service.interface.d.ts","sourceRoot":"","sources":["../../../../../../../../packages/core/src/application/ports/output/services/deployment-service.interface.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,8BAA8B,CAAC;AAEpE,uEAAuE;AACvE,MAAM,WAAW,QAAQ;IACvB,2CAA2C;IAC3C,QAAQ,EAAE,MAAM,CAAC;IACjB,8CAA8C;IAC9C,MAAM,EAAE,QAAQ,GAAG,QAAQ,CAAC;IAC5B,mDAAmD;IACnD,IAAI,EAAE,MAAM,CAAC;IACb,6DAA6D;IAC7D,SAAS,EAAE,MAAM,CAAC;CACnB;AAED,+CAA+C;AAC/C,MAAM,WAAW,gBAAgB;IAC/B,iDAAiD;IACjD,KAAK,EAAE,eAAe,CAAC;IACvB,qEAAqE;IACrE,GAAG,EAAE,MAAM,GAAG,IAAI,CAAC;CACpB;AAED;;;;;;;;;GASG;AACH,MAAM,WAAW,kBAAkB;IACjC;;;;;;;;OAQG;IACH,KAAK,CAAC,QAAQ,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;IAElD;;;;;;OAMG;IACH,IAAI,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;IAEtC;;;;;OAKG;IACH,SAAS,CAAC,QAAQ,EAAE,MAAM,GAAG,gBAAgB,GAAG,IAAI,CAAC;IAErD;;;OAGG;IACH,OAAO,IAAI,IAAI,CAAC;IAIhB;;;;;OAKG;IACH,OAAO,CAAC,QAAQ,EAAE,MAAM,GAAG,QAAQ,EAAE,GAAG,IAAI,CAAC;IAE7C;;;;;OAKG;IACH,EAAE,CAAC,KAAK,EAAE,KAAK,EAAE,OAAO,EAAE,CAAC,KAAK,EAAE,QAAQ,KAAK,IAAI,GAAG,IAAI,CAAC;IAE3D;;;;;OAKG;IACH,GAAG,CAAC,KAAK,EAAE,KAAK,EAAE,OAAO,EAAE,CAAC,KAAK,EAAE,QAAQ,KAAK,IAAI,GAAG,IAAI,CAAC;CAC7D"}
+{"version":3,"file":"deployment-service.interface.d.ts","sourceRoot":"","sources":["../../../../../../../../packages/core/src/application/ports/output/services/deployment-service.interface.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,KAAK,QAAQ,MAAM,gBAAgB,CAAC;AAC3C,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,8BAA8B,CAAC;AAEpE,uEAAuE;AACvE,MAAM,WAAW,QAAQ;IACvB,2CAA2C;IAC3C,QAAQ,EAAE,MAAM,CAAC;IACjB,8CAA8C;IAC9C,MAAM,EAAE,QAAQ,GAAG,QAAQ,CAAC;IAC5B,mDAAmD;IACnD,IAAI,EAAE,MAAM,CAAC;IACb,6DAA6D;IAC7D,SAAS,EAAE,MAAM,CAAC;CACnB;AAED,+CAA+C;AAC/C,MAAM,WAAW,gBAAgB;IAC/B,iDAAiD;IACjD,KAAK,EAAE,eAAe,CAAC;IACvB,qEAAqE;IACrE,GAAG,EAAE,MAAM,GAAG,IAAI,CAAC;CACpB;AAED;;;;;;;;;GASG;AACH,MAAM,WAAW,kBAAkB;IACjC;;;OAGG;IACH,WAAW,CAAC,EAAE,EAAE,QAAQ,CAAC,QAAQ,GAAG,IAAI,CAAC;IAEzC;;;OAGG;IACH,UAAU,IAAI,IAAI,CAAC;IAEnB;;;;;;;;OAQG;IACH,KAAK,CAAC,QAAQ,EAAE,MAAM,EAAE,UAAU,EAAE,MAAM,EAAE,UAAU,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAEvE;;;;;;OAMG;IACH,IAAI,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;IAEtC;;;;;OAKG;IACH,SAAS,CAAC,QAAQ,EAAE,MAAM,GAAG,gBAAgB,GAAG,IAAI,CAAC;IAErD;;;OAGG;IACH,OAAO,IAAI,IAAI,CAAC;IAIhB;;;;;OAKG;IACH,OAAO,CAAC,QAAQ,EAAE,MAAM,GAAG,QAAQ,EAAE,GAAG,IAAI,CAAC;IAE7C;;;;;OAKG;IACH,EAAE,CAAC,KAAK,EAAE,KAAK,EAAE,OAAO,EAAE,CAAC,KAAK,EAAE,QAAQ,KAAK,IAAI,GAAG,IAAI,CAAC;IAE3D;;;;;OAKG;IACH,GAAG,CAAC,KAAK,EAAE,KAAK,EAAE,OAAO,EAAE,CAAC,KAAK,EAAE,QAAQ,KAAK,IAAI,GAAG,IAAI,CAAC;CAC7D"}

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;AAoHrC;;;;;;GAMG;AACH,wBAAsB,mBAAmB,IAAI,OAAO,CAAC,OAAO,SAAS,CAAC,CAiUrE;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;AAoHrC;;;;;;GAMG;AACH,wBAAsB,mBAAmB,IAAI,OAAO,CAAC,OAAO,SAAS,CAAC,CAmUrE;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

@@ -169,6 +169,8 @@ export async function initializeContainer() {
     container.registerSingleton(AttachmentStorageService);
     container.register('AttachmentStorageService', { useToken: AttachmentStorageService });
     const deploymentService = new DeploymentService();
+    deploymentService.setDatabase(db);
+    deploymentService.recoverAll();
     container.registerInstance('IDeploymentService', deploymentService);
     // Register agent infrastructure
     container.register('IAgentRunRepository', {

package/dist/packages/core/src/infrastructure/persistence/sqlite/migrations/040-create-dev-servers-table.d.ts

@@ -0,0 +1,21 @@
+/**
+ * Migration 040: Create dev_servers table.
+ *
+ * Persists running dev server deployments so they survive page reloads
+ * and server restarts. The DeploymentService reconciles on startup by
+ * checking PID liveness and cleaning up stale rows.
+ *
+ * Columns:
+ *  - target_id   TEXT PRIMARY KEY — featureId or repositoryPath
+ *  - target_type TEXT NOT NULL    — 'feature' | 'repository'
+ *  - pid         INTEGER NOT NULL — OS process ID
+ *  - state       TEXT NOT NULL    — DeploymentState enum value
+ *  - url         TEXT             — detected dev server URL (null while Booting)
+ *  - target_path TEXT NOT NULL    — filesystem path where the dev server runs
+ *  - started_at  INTEGER NOT NULL — unix milliseconds
+ */
+import type { MigrationParams } from 'umzug';
+import type Database from 'better-sqlite3';
+export declare function up({ context: db }: MigrationParams<Database.Database>): Promise<void>;
+export declare function down({ context: db }: MigrationParams<Database.Database>): Promise<void>;
+//# sourceMappingURL=040-create-dev-servers-table.d.ts.map

package/dist/packages/core/src/infrastructure/persistence/sqlite/migrations/040-create-dev-servers-table.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"040-create-dev-servers-table.d.ts","sourceRoot":"","sources":["../../../../../../../../packages/core/src/infrastructure/persistence/sqlite/migrations/040-create-dev-servers-table.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG;AAEH,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,OAAO,CAAC;AAC7C,OAAO,KAAK,QAAQ,MAAM,gBAAgB,CAAC;AAE3C,wBAAsB,EAAE,CAAC,EAAE,OAAO,EAAE,EAAE,EAAE,EAAE,eAAe,CAAC,QAAQ,CAAC,QAAQ,CAAC,GAAG,OAAO,CAAC,IAAI,CAAC,CAY3F;AAED,wBAAsB,IAAI,CAAC,EAAE,OAAO,EAAE,EAAE,EAAE,EAAE,eAAe,CAAC,QAAQ,CAAC,QAAQ,CAAC,GAAG,OAAO,CAAC,IAAI,CAAC,CAE7F"}

package/dist/packages/core/src/infrastructure/persistence/sqlite/migrations/040-create-dev-servers-table.js

@@ -0,0 +1,32 @@
+/**
+ * Migration 040: Create dev_servers table.
+ *
+ * Persists running dev server deployments so they survive page reloads
+ * and server restarts. The DeploymentService reconciles on startup by
+ * checking PID liveness and cleaning up stale rows.
+ *
+ * Columns:
+ *  - target_id   TEXT PRIMARY KEY — featureId or repositoryPath
+ *  - target_type TEXT NOT NULL    — 'feature' | 'repository'
+ *  - pid         INTEGER NOT NULL — OS process ID
+ *  - state       TEXT NOT NULL    — DeploymentState enum value
+ *  - url         TEXT             — detected dev server URL (null while Booting)
+ *  - target_path TEXT NOT NULL    — filesystem path where the dev server runs
+ *  - started_at  INTEGER NOT NULL — unix milliseconds
+ */
+export async function up({ context: db }) {
+    db.exec(`
+    CREATE TABLE IF NOT EXISTS dev_servers (
+      target_id   TEXT PRIMARY KEY,
+      target_type TEXT NOT NULL,
+      pid         INTEGER NOT NULL,
+      state       TEXT NOT NULL DEFAULT 'Booting',
+      url         TEXT,
+      target_path TEXT NOT NULL,
+      started_at  INTEGER NOT NULL
+    )
+  `);
+}
+export async function down({ context: db }) {
+    db.exec('DROP TABLE IF EXISTS dev_servers');
+}

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/nodes/merge/ci-watch-fix-loop.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ci-watch-fix-loop.d.ts","sourceRoot":"","sources":["../../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/merge/ci-watch-fix-loop.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,+DAA+D,CAAC;AACpG,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,yEAAyE,CAAC;AAClH,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,iEAAiE,CAAC;AAKrG,OAAO,EAAE,QAAQ,EAAE,KAAK,WAAW,EAAE,MAAM,8BAA8B,CAAC;AAC1E,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,oBAAoB,CAAC;AAErD,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,+DAA+D,CAAC;AAK3G,MAAM,WAAW,cAAc;IAC7B,QAAQ,EAAE,cAAc,CAAC;IACzB,YAAY,EAAE,aAAa,CAAC;IAC5B,iBAAiB,EAAE,IAAI,CAAC,kBAAkB,EAAE,UAAU,GAAG,QAAQ,CAAC,CAAC;CACpE;AAED,MAAM,WAAW,gBAAgB;IAC/B,GAAG,EAAE,MAAM,CAAC;IACZ,MAAM,EAAE,MAAM,CAAC;IACf,OAAO,EAAE,qBAAqB,CAAC;IAC/B,OAAO,EAAE,OAAO,CAAC,UAAU,CAAC,IAAI,CAAC,kBAAkB,EAAE,UAAU,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC;IAC/E,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;IACrB,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,gBAAgB,EAAE,MAAM,CAAC;IACzB,QAAQ,EAAE,MAAM,EAAE,CAAC;IACnB,GAAG,EAAE,UAAU,CAAC;CACjB;AAED,MAAM,MAAM,gBAAgB,GAAG,MAAM,GAAG,UAAU,GAAG,QAAQ,GAAG,SAAS,GAAG,WAAW,GAAG,SAAS,CAAC;AAEpG,MAAM,WAAW,gBAAgB;IAC/B,QAAQ,EAAE,QAAQ,CAAC;IACnB,aAAa,EAAE,MAAM,CAAC;IACtB,YAAY,EAAE,WAAW,EAAE,CAAC;IAC5B,WAAW,EAAE,gBAAgB,CAAC;CAC/B;AAED;;;;;GAKG;AACH,wBAAsB,iBAAiB,CACrC,IAAI,EAAE,cAAc,EACpB,MAAM,EAAE,gBAAgB,GACvB,OAAO,CAAC,gBAAgB,CAAC,CA8I3B"}
+{"version":3,"file":"ci-watch-fix-loop.d.ts","sourceRoot":"","sources":["../../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/merge/ci-watch-fix-loop.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,+DAA+D,CAAC;AACpG,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,yEAAyE,CAAC;AAClH,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,iEAAiE,CAAC;AAKrG,OAAO,EAAE,QAAQ,EAAE,KAAK,WAAW,EAAE,MAAM,8BAA8B,CAAC;AAC1E,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,oBAAoB,CAAC;AAErD,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,+DAA+D,CAAC;AAK3G,MAAM,WAAW,cAAc;IAC7B,QAAQ,EAAE,cAAc,CAAC;IACzB,YAAY,EAAE,aAAa,CAAC;IAC5B,iBAAiB,EAAE,IAAI,CAAC,kBAAkB,EAAE,UAAU,GAAG,QAAQ,CAAC,CAAC;CACpE;AAED,MAAM,WAAW,gBAAgB;IAC/B,GAAG,EAAE,MAAM,CAAC;IACZ,MAAM,EAAE,MAAM,CAAC;IACf,OAAO,EAAE,qBAAqB,CAAC;IAC/B,OAAO,EAAE,OAAO,CAAC,UAAU,CAAC,IAAI,CAAC,kBAAkB,EAAE,UAAU,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC;IAC/E,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;IACrB,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,gBAAgB,EAAE,MAAM,CAAC;IACzB,QAAQ,EAAE,MAAM,EAAE,CAAC;IACnB,GAAG,EAAE,UAAU,CAAC;CACjB;AAED,MAAM,MAAM,gBAAgB,GAAG,MAAM,GAAG,UAAU,GAAG,QAAQ,GAAG,SAAS,GAAG,WAAW,GAAG,SAAS,CAAC;AAEpG,MAAM,WAAW,gBAAgB;IAC/B,QAAQ,EAAE,QAAQ,CAAC;IACnB,aAAa,EAAE,MAAM,CAAC;IACtB,YAAY,EAAE,WAAW,EAAE,CAAC;IAC5B,WAAW,EAAE,gBAAgB,CAAC;CAC/B;AAED;;;;;GAKG;AACH,wBAAsB,iBAAiB,CACrC,IAAI,EAAE,cAAc,EACpB,MAAM,EAAE,gBAAgB,GACvB,OAAO,CAAC,gBAAgB,CAAC,CAuJ3B"}

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/nodes/merge/ci-watch-fix-loop.js

@@ -84,6 +84,11 @@ export async function runCiWatchFixLoop(deps, params) {
         }
         throw err;
     }
+    // Use the run URL from watchCi — it reflects the actual run watched,
+    // which may differ from the initial getCiStatus() result when multiple
+    // workflow runs exist for the same branch.
+    if (watchResult.runUrl)
+        runUrl = watchResult.runUrl;
     if (watchResult.status === 'success') {
         log.info('CI passed on first watch');
         return { ciStatus: CiStatus.Success, ciFixAttempts, ciFixHistory, ciFixStatus: 'success' };
@@ -128,6 +133,10 @@ export async function runCiWatchFixLoop(deps, params) {
             }
             throw err;
         }
+        // Update runUrl to the run that was actually watched (avoids mismatch
+        // when multiple workflow runs exist for the same branch).
+        if (fixWatchResult.runUrl)
+            runUrl = fixWatchResult.runUrl;
         const outcome = fixWatchResult.status === 'success' ? 'fixed' : 'failed';
         ciFixHistory.push({
             attempt: ciFixAttempts,

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

@@ -2,11 +2,15 @@
  * Deployment Service
  *
  * Infrastructure service that manages local dev server deployments.
- * Holds an in-memory Map of active deployments keyed by targetId.
- * Handles process spawning, stdout-based port detection, and graceful
- * shutdown (SIGTERM → poll → SIGKILL).
+ * Holds an in-memory Map of active deployments keyed by targetId AND
+ * persists deployment records to the `dev_servers` SQLite table so that
+ * running dev servers survive page reloads and server restarts.
+ *
+ * On startup, call `recoverAll()` to reconcile the DB with actual
+ * process state (clean up dead PIDs, re-adopt live ones).
  */
 import { spawn } from 'node:child_process';
+import type Database from 'better-sqlite3';
 import type { IDeploymentService, DeploymentStatus, LogEntry } from '../../../application/ports/output/services/deployment-service.interface.js';
 import { detectDevScript } from './detect-dev-script.js';
 export interface DeploymentServiceDeps {
@@ -19,15 +23,26 @@ export declare class DeploymentService implements IDeploymentService {
     private readonly deployments;
     private readonly deps;
     private readonly emitter;
+    private db;
     constructor(deps?: Partial<DeploymentServiceDeps>);
+    /**
+     * Inject the database connection. Called from DI container after DB is ready.
+     */
+    setDatabase(db: Database.Database): void;
+    /**
+     * Recover dev servers from the database on startup.
+     * Validates each PID is still alive; removes dead rows.
+     */
+    recoverAll(): void;
     /**
      * Start a deployment for the given target.
      * If a deployment already exists for this target, it is stopped first.
      */
-    start(targetId: string, targetPath: string): void;
+    start(targetId: string, targetPath: string, targetType?: string): void;
     /**
      * Get the current deployment status for a target.
-     * Returns null if no deployment exists for this target.
+     * Checks in-memory Map first, then falls back to DB for recovered deployments.
+     * Validates PID liveness — cleans up if dead.
      */
     getStatus(targetId: string): DeploymentStatus | null;
     /**
@@ -50,22 +65,17 @@ export declare class DeploymentService implements IDeploymentService {
      * Unsubscribe from real-time log events.
      */
     off(event: 'log', handler: (entry: LogEntry) => void): void;
-    /**
-     * Send SIGKILL to a process tree.
-     */
+    private dbUpsert;
+    private dbUpdateState;
+    private dbDelete;
+    private dbFind;
     private killProcess;
     /**
      * Attach a line-buffered listener on stdout or stderr that calls parsePort
      * and accumulates log entries.
      */
     private attachOutputListener;
-    /**
-     * Poll until a process is dead or timeout expires.
-     */
     private pollUntilDead;
-    /**
-     * Wait for a child process to emit 'exit', with a short timeout.
-     */
     private waitForExit;
 }
 //# sourceMappingURL=deployment.service.d.ts.map

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

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

@@ -2,13 +2,20 @@
  * Deployment Service
  *
  * Infrastructure service that manages local dev server deployments.
- * Holds an in-memory Map of active deployments keyed by targetId.
- * Handles process spawning, stdout-based port detection, and graceful
- * shutdown (SIGTERM → poll → SIGKILL).
+ * Holds an in-memory Map of active deployments keyed by targetId AND
+ * persists deployment records to the `dev_servers` SQLite table so that
+ * running dev servers survive page reloads and server restarts.
+ *
+ * On startup, call `recoverAll()` to reconcile the DB with actual
+ * process state (clean up dead PIDs, re-adopt live ones).
  */
 import { spawn } from 'node:child_process';
 import { EventEmitter } from 'node:events';
-import treeKill from 'tree-kill';
+// NOTE: We intentionally do NOT use tree-kill here. tree-kill traverses
+// /proc to find child processes which can accidentally kill parent processes
+// (including the Shep web UI). Instead, we use process.kill(-pid) to send
+// signals to the process GROUP (since we spawn with detached: true, the
+// child gets its own process group via setsid()).
 import { DeploymentState } from '../../../domain/generated/output.js';
 import { detectDevScript } from './detect-dev-script.js';
 import { createDeploymentLogger } from './deployment-logger.js';
@@ -20,7 +27,22 @@ const MAX_WAIT_MS = 5000;
 const defaultDeps = {
     spawn,
     detectDevScript,
-    kill: (pid, signal) => treeKill(pid, signal),
+    kill: (pid, signal) => {
+        try {
+            // Kill the entire process group (negative PID) — safe because
+            // detached: true puts the child in its own group via setsid().
+            process.kill(-pid, signal);
+        }
+        catch {
+            // Fallback: kill just the process itself
+            try {
+                process.kill(pid, signal);
+            }
+            catch {
+                // Process already dead
+            }
+        }
+    },
     isAlive: (pid) => {
         try {
             process.kill(pid, 0);
@@ -35,14 +57,65 @@ export class DeploymentService {
     deployments = new Map();
     deps;
     emitter = new EventEmitter();
+    db = null;
     constructor(deps = {}) {
         this.deps = { ...defaultDeps, ...deps };
     }
+    /**
+     * Inject the database connection. Called from DI container after DB is ready.
+     */
+    setDatabase(db) {
+        this.db = db;
+    }
+    /**
+     * Recover dev servers from the database on startup.
+     * Validates each PID is still alive; removes dead rows.
+     */
+    recoverAll() {
+        if (!this.db)
+            return;
+        let rows;
+        try {
+            const stmt = this.db.prepare('SELECT * FROM dev_servers');
+            rows = stmt.all() ?? [];
+        }
+        catch {
+            log.info('dev_servers table not ready — skipping recovery');
+            return;
+        }
+        if (!rows || rows.length === 0) {
+            log.info('No dev servers to recover from database');
+            return;
+        }
+        log.info(`Recovering ${rows.length} dev server(s) from database`);
+        for (const row of rows) {
+            if (this.deps.isAlive(row.pid)) {
+                log.info(`Recovered "${row.target_id}" (pid=${row.pid}, state=${row.state}, url=${row.url})`);
+                const entry = {
+                    pid: row.pid,
+                    child: null, // orphan — we don't have the ChildProcess handle
+                    state: row.state,
+                    url: row.url,
+                    targetId: row.target_id,
+                    targetPath: row.target_path,
+                    targetType: row.target_type,
+                    stdoutBuffer: '',
+                    stderrBuffer: '',
+                    logs: new LogRingBuffer(),
+                };
+                this.deployments.set(row.target_id, entry);
+            }
+            else {
+                log.info(`Stale dev server "${row.target_id}" (pid=${row.pid}) — removing from DB`);
+                this.dbDelete(row.target_id);
+            }
+        }
+    }
     /**
      * Start a deployment for the given target.
      * If a deployment already exists for this target, it is stopped first.
      */
-    start(targetId, targetPath) {
+    start(targetId, targetPath, targetType = 'repository') {
         log.info(`start() called — targetId="${targetId}", targetPath="${targetPath}"`);
         // Stop any existing deployment for this target
         const existing = this.deployments.get(targetId);
@@ -50,6 +123,7 @@ export class DeploymentService {
             log.info(`Stopping existing deployment for "${targetId}" (pid=${existing.pid})`);
             this.killProcess(existing);
             this.deployments.delete(targetId);
+            this.dbDelete(targetId);
         }
         // Detect the dev script
         const detection = this.deps.detectDevScript(targetPath);
@@ -78,11 +152,14 @@ export class DeploymentService {
             state: DeploymentState.Booting,
             url: null,
             targetId,
+            targetPath,
+            targetType,
             stdoutBuffer: '',
             stderrBuffer: '',
             logs: new LogRingBuffer(),
         };
         this.deployments.set(targetId, entry);
+        this.dbUpsert(entry);
         // Attach stdout/stderr listeners for port detection
         this.attachOutputListener(entry, 'stdout');
         this.attachOutputListener(entry, 'stderr');
@@ -91,6 +168,7 @@ export class DeploymentService {
             log.error(`Child process error for "${targetId}" (pid=${entry.pid}): ${err.message}`);
             entry.state = DeploymentState.Stopped;
             this.deployments.delete(targetId);
+            this.dbDelete(targetId);
         });
         // Clean up on process exit
         child.on('exit', (code, signal) => {
@@ -100,20 +178,52 @@ export class DeploymentService {
                 log.warn('Process exited while still in Booting state — dev server likely crashed on startup. Check stderr output above.');
             }
             this.deployments.delete(targetId);
+            this.dbDelete(targetId);
         });
     }
     /**
      * Get the current deployment status for a target.
-     * Returns null if no deployment exists for this target.
+     * Checks in-memory Map first, then falls back to DB for recovered deployments.
+     * Validates PID liveness — cleans up if dead.
      */
     getStatus(targetId) {
         const entry = this.deployments.get(targetId);
-        if (!entry) {
-            log.debug(`getStatus("${targetId}") — no deployment found`);
-            return null;
+        if (entry) {
+            // Validate the process is still alive (handles orphan crashes)
+            if (!this.deps.isAlive(entry.pid)) {
+                log.info(`getStatus("${targetId}") — pid=${entry.pid} is dead, cleaning up`);
+                this.deployments.delete(targetId);
+                this.dbDelete(targetId);
+                return null;
+            }
+            log.debug(`getStatus("${targetId}") — state=${entry.state}, url=${entry.url}, pid=${entry.pid}`);
+            return { state: entry.state, url: entry.url };
         }
-        log.debug(`getStatus("${targetId}") — state=${entry.state}, url=${entry.url}, pid=${entry.pid}`);
-        return { state: entry.state, url: entry.url };
+        // Check DB for entries not yet in memory (e.g., after hook re-mount)
+        const row = this.dbFind(targetId);
+        if (row) {
+            if (this.deps.isAlive(row.pid)) {
+                // Re-adopt into memory
+                const recovered = {
+                    pid: row.pid,
+                    child: null,
+                    state: row.state,
+                    url: row.url,
+                    targetId: row.target_id,
+                    targetPath: row.target_path,
+                    targetType: row.target_type,
+                    stdoutBuffer: '',
+                    stderrBuffer: '',
+                    logs: new LogRingBuffer(),
+                };
+                this.deployments.set(targetId, recovered);
+                return { state: recovered.state, url: recovered.url };
+            }
+            // Dead — clean up
+            this.dbDelete(targetId);
+        }
+        log.debug(`getStatus("${targetId}") — no deployment found`);
+        return null;
     }
     /**
      * Stop a deployment gracefully: SIGTERM → poll → SIGKILL.
@@ -121,7 +231,19 @@ export class DeploymentService {
     async stop(targetId) {
         const entry = this.deployments.get(targetId);
         if (!entry) {
-            log.info(`stop("${targetId}") — no deployment found, nothing to stop`);
+            // Check DB in case it's a recovered orphan
+            const row = this.dbFind(targetId);
+            if (row && this.deps.isAlive(row.pid)) {
+                log.info(`stop("${targetId}") — killing orphan process (pid=${row.pid})`);
+                try {
+                    this.deps.kill(row.pid, 'SIGKILL');
+                }
+                catch {
+                    // already dead
+                }
+            }
+            this.dbDelete(targetId);
+            log.info(`stop("${targetId}") — no in-memory deployment found, cleaned DB`);
             return;
         }
         log.info(`stop("${targetId}") — sending SIGTERM to process tree (pid=${entry.pid})`);
@@ -133,13 +255,13 @@ export class DeploymentService {
         catch {
             log.info(`stop("${targetId}") — process already dead on SIGTERM`);
             this.deployments.delete(targetId);
+            this.dbDelete(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');
             }
@@ -150,8 +272,13 @@ export class DeploymentService {
         else {
             log.info(`stop("${targetId}") — process exited gracefully`);
         }
-        // Wait for the exit event to clean up the map
-        await this.waitForExit(entry.child);
+        // Wait for the exit event to clean up the map (only if we have a ChildProcess handle)
+        if (entry.child) {
+            await this.waitForExit(entry.child);
+        }
+        // Ensure cleanup
+        this.deployments.delete(targetId);
+        this.dbDelete(targetId);
     }
     /**
      * Force-stop all tracked deployments immediately (for daemon shutdown).
@@ -161,6 +288,15 @@ export class DeploymentService {
             entry.logs.clear();
             this.killProcess(entry);
         }
+        // Also clean DB
+        if (this.db) {
+            try {
+                this.db.prepare('DELETE FROM dev_servers').run();
+            }
+            catch {
+                // table might not exist yet
+            }
+        }
     }
     /**
      * Get the accumulated log buffer for a deployment.
@@ -183,9 +319,54 @@ export class DeploymentService {
     off(event, handler) {
         this.emitter.off(event, handler);
     }
-    /**
-     * Send SIGKILL to a process tree.
-     */
+    // ─── Database helpers ───────────────────────────────────────────────
+    dbUpsert(entry) {
+        if (!this.db)
+            return;
+        try {
+            this.db
+                .prepare(`INSERT OR REPLACE INTO dev_servers
+         (target_id, target_type, pid, state, url, target_path, started_at)
+         VALUES (?, ?, ?, ?, ?, ?, ?)`)
+                .run(entry.targetId, entry.targetType, entry.pid, entry.state, entry.url, entry.targetPath, Date.now());
+        }
+        catch (err) {
+            log.warn(`dbUpsert failed for "${entry.targetId}": ${err}`);
+        }
+    }
+    dbUpdateState(targetId, state, url) {
+        if (!this.db)
+            return;
+        try {
+            this.db
+                .prepare('UPDATE dev_servers SET state = ?, url = ? WHERE target_id = ?')
+                .run(state, url, targetId);
+        }
+        catch (err) {
+            log.warn(`dbUpdateState failed for "${targetId}": ${err}`);
+        }
+    }
+    dbDelete(targetId) {
+        if (!this.db)
+            return;
+        try {
+            this.db.prepare('DELETE FROM dev_servers WHERE target_id = ?').run(targetId);
+        }
+        catch (err) {
+            log.warn(`dbDelete failed for "${targetId}": ${err}`);
+        }
+    }
+    dbFind(targetId) {
+        if (!this.db)
+            return null;
+        try {
+            return (this.db.prepare('SELECT * FROM dev_servers WHERE target_id = ?').get(targetId) ?? null);
+        }
+        catch {
+            return null;
+        }
+    }
+    // ─── Process helpers ────────────────────────────────────────────────
     killProcess(entry) {
         try {
             this.deps.kill(entry.pid, 'SIGKILL');
@@ -200,24 +381,20 @@ export class DeploymentService {
      */
     attachOutputListener(entry, stream) {
         const bufferKey = stream === 'stdout' ? 'stdoutBuffer' : 'stderrBuffer';
-        const childStream = entry.child[stream];
+        const childStream = entry.child?.[stream];
         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] += 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}`);
-                // Accumulate in log buffer and emit event
                 const logEntry = {
                     targetId: entry.targetId,
                     stream,
@@ -233,12 +410,12 @@ export class DeploymentService {
                         log.info(`[${entry.targetId}] Port detected — url="${url}" (from ${stream})`);
                         entry.state = DeploymentState.Ready;
                         entry.url = url;
+                        this.dbUpdateState(entry.targetId, entry.state, entry.url);
                     }
                 }
             }
         });
         childStream.on('end', () => {
-            // Flush remaining buffer content
             const remaining = entry[bufferKey].trim();
             if (remaining) {
                 log.debug(`[${entry.targetId}] ${stream} (flush): ${remaining}`);
@@ -248,6 +425,7 @@ export class DeploymentService {
                         log.info(`[${entry.targetId}] Port detected in flushed buffer — url="${url}"`);
                         entry.state = DeploymentState.Ready;
                         entry.url = url;
+                        this.dbUpdateState(entry.targetId, entry.state, entry.url);
                     }
                 }
                 entry[bufferKey] = '';
@@ -255,9 +433,6 @@ export class DeploymentService {
             log.debug(`[${entry.targetId}] ${stream} stream ended`);
         });
     }
-    /**
-     * Poll until a process is dead or timeout expires.
-     */
     async pollUntilDead(pid, maxMs, intervalMs) {
         const deadline = Date.now() + maxMs;
         while (Date.now() < deadline) {
@@ -268,9 +443,6 @@ export class DeploymentService {
         }
         return false;
     }
-    /**
-     * Wait for a child process to emit 'exit', with a short timeout.
-     */
     waitForExit(child) {
         return new Promise((resolve) => {
             const timeout = setTimeout(() => resolve(), 1000);

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

@@ -5,11 +5,5 @@
  * dev server stdout/stderr output. Handles common patterns from Next.js,
  * Vite, Express, and generic servers.
  */
-/**
- * Parse a line of stdout/stderr output for a localhost URL or port.
- *
- * @param line - A single line of process output
- * @returns The detected URL string, or null if no URL/port was found
- */
 export declare function parsePort(line: string): string | null;
 //# sourceMappingURL=parse-port.d.ts.map

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

@@ -1 +1 @@
-{"version":3,"file":"parse-port.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/infrastructure/services/deployment/parse-port.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAkCH;;;;;GAKG;AACH,wBAAgB,SAAS,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI,CAarD"}
+{"version":3,"file":"parse-port.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/infrastructure/services/deployment/parse-port.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AA4CH,wBAAgB,SAAS,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI,CAgBrD"}