@shepai/cli 1.108.0 → 1.109.0-pr333.db60eb7

package/dist/packages/core/src/infrastructure/services/notifications/notification-watcher.service.js

@@ -12,7 +12,7 @@
  * Design decision: Uses DB polling instead of IPC because the worker
  * process is spawned detached with IPC disconnected.
  */
-import { AgentRunStatus, NotificationEventType, NotificationSeverity, } from '../../../domain/generated/output.js';
+import { AgentRunStatus, NotificationEventType, NotificationSeverity, SdlcLifecycle, } from '../../../domain/generated/output.js';
 const DEFAULT_POLL_INTERVAL_MS = 3000;
 const ACTIVE_STATUSES = new Set([
     AgentRunStatus.pending,
@@ -56,6 +56,7 @@ export class NotificationWatcherService {
     notificationService;
     pollIntervalMs;
     trackedRuns = new Map();
+    trackedFeatureLifecycles = new Map();
     intervalId = null;
     // Suppresses notifications on the first poll to avoid replaying historical state
     isBootstrapped = false;
@@ -92,6 +93,13 @@ export class NotificationWatcherService {
         catch {
             // DB not ready or query failed — skip this poll cycle
         }
+        try {
+            const features = await this.featureRepository.list();
+            this.checkFeatureLifecycles(features);
+        }
+        catch {
+            // DB not ready or query failed — skip this poll cycle
+        }
     }
     async processRuns(runs) {
         const currentRunIds = new Set();
@@ -182,6 +190,32 @@ export class NotificationWatcherService {
             }
         }
     }
+    checkFeatureLifecycles(features) {
+        for (const feature of features) {
+            const prevLifecycle = this.trackedFeatureLifecycles.get(feature.id);
+            this.trackedFeatureLifecycles.set(feature.id, feature.lifecycle);
+            // Emit notification when a feature transitions TO Review from a known previous state.
+            // Skip when prevLifecycle is undefined (first observation / bootstrap seeding) or
+            // when the watcher hasn't completed its first poll yet.
+            if (this.isBootstrapped &&
+                prevLifecycle !== undefined &&
+                feature.lifecycle === SdlcLifecycle.Review &&
+                prevLifecycle !== SdlcLifecycle.Review) {
+                const prUrl = feature.pr?.url;
+                const message = prUrl ? `Ready for merge review — PR: ${prUrl}` : 'Ready for merge review';
+                const event = {
+                    eventType: NotificationEventType.MergeReviewReady,
+                    agentRunId: feature.agentRunId ?? '',
+                    featureId: feature.id,
+                    featureName: feature.name,
+                    message,
+                    severity: NotificationSeverity.Info,
+                    timestamp: new Date().toISOString(),
+                };
+                this.notificationService.notify(event);
+            }
+        }
+    }
     async resolveFeatureIdentity(run) {
         if (run.featureId) {
             try {

package/dist/packages/core/src/infrastructure/services/notifications/notification.service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"notification.service.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/infrastructure/services/notifications/notification.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,KAAK,EACV,iBAAiB,EAElB,MAAM,qCAAqC,CAAC;AAE7C,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,8EAA8E,CAAC;AAEzH,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,uBAAuB,CAAC;AAC7D,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,uBAAuB,CAAC;AAe7D,qBAAa,mBAAoB,YAAW,oBAAoB;IAC9D,OAAO,CAAC,QAAQ,CAAC,GAAG,CAAkB;IACtC,OAAO,CAAC,QAAQ,CAAC,eAAe,CAAkB;gBAEtC,GAAG,EAAE,eAAe,EAAE,eAAe,EAAE,eAAe;IAKlE,MAAM,CAAC,KAAK,EAAE,iBAAiB,GAAG,IAAI;CAiBvC"}
+{"version":3,"file":"notification.service.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/infrastructure/services/notifications/notification.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAEH,OAAO,KAAK,EACV,iBAAiB,EAElB,MAAM,qCAAqC,CAAC;AAE7C,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,8EAA8E,CAAC;AAEzH,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,uBAAuB,CAAC;AAC7D,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,uBAAuB,CAAC;AAgB7D,qBAAa,mBAAoB,YAAW,oBAAoB;IAC9D,OAAO,CAAC,QAAQ,CAAC,GAAG,CAAkB;IACtC,OAAO,CAAC,QAAQ,CAAC,eAAe,CAAkB;gBAEtC,GAAG,EAAE,eAAe,EAAE,eAAe,EAAE,eAAe;IAKlE,MAAM,CAAC,KAAK,EAAE,iBAAiB,GAAG,IAAI;CAiBvC"}

package/dist/packages/core/src/infrastructure/services/notifications/notification.service.js

@@ -21,6 +21,7 @@ const EVENT_TYPE_TO_CONFIG_KEY = {
     [NotificationEventType.PrChecksPassed]: 'prChecksPassed',
     [NotificationEventType.PrChecksFailed]: 'prChecksFailed',
     [NotificationEventType.PrBlocked]: 'prBlocked',
+    [NotificationEventType.MergeReviewReady]: 'mergeReviewReady',
 };
 export class NotificationService {
     bus;

package/dist/packages/core/src/infrastructure/services/pr-sync/pr-sync-watcher.service.d.ts

@@ -13,6 +13,7 @@ import type { IFeatureRepository } from '../../../application/ports/output/repos
 import type { IAgentRunRepository } from '../../../application/ports/output/agents/agent-run-repository.interface.js';
 import type { IGitPrService } from '../../../application/ports/output/services/git-pr-service.interface.js';
 import type { INotificationService } from '../../../application/ports/output/services/notification-service.interface.js';
+import type Database from 'better-sqlite3';
 export declare class PrSyncWatcherService {
     private readonly featureRepo;
     private readonly agentRunRepo;
@@ -21,12 +22,22 @@ export declare class PrSyncWatcherService {
     private readonly pollIntervalMs;
     private readonly trackedFeatures;
     private readonly skippedRepos;
+    private readonly rateLimitedUntil;
+    private readonly db;
+    private readonly processId;
     private intervalId;
-    constructor(featureRepo: IFeatureRepository, agentRunRepo: IAgentRunRepository, gitPrService: IGitPrService, notificationService: INotificationService, pollIntervalMs?: number);
+    private pollCycle;
+    constructor(featureRepo: IFeatureRepository, agentRunRepo: IAgentRunRepository, gitPrService: IGitPrService, notificationService: INotificationService, pollIntervalMs?: number, db?: Database.Database | null);
     isRunning(): boolean;
     start(): void;
     stop(): void;
+    /** Attempt to acquire the cross-process poll lock. Returns true if acquired. */
+    private tryAcquireLock;
+    /** Check if a feature should be polled this cycle based on its unchanged cycles count. */
+    private shouldPollFeature;
     private poll;
+    private isRateLimited;
+    private handleRateLimitError;
     private processRepository;
     private processFeature;
     /** Mark associated agent run as completed so the UI reflects "done" state. */
@@ -39,7 +50,7 @@ export declare class PrSyncWatcherService {
  *
  * @throws Error if the watcher is already initialized
  */
-export declare function initializePrSyncWatcher(featureRepo: IFeatureRepository, agentRunRepo: IAgentRunRepository, gitPrService: IGitPrService, notificationService: INotificationService, pollIntervalMs?: number): void;
+export declare function initializePrSyncWatcher(featureRepo: IFeatureRepository, agentRunRepo: IAgentRunRepository, gitPrService: IGitPrService, notificationService: INotificationService, pollIntervalMs?: number, db?: Database.Database | null): void;
 /**
  * Get the PR sync watcher singleton.
  *

package/dist/packages/core/src/infrastructure/services/pr-sync/pr-sync-watcher.service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"pr-sync-watcher.service.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/infrastructure/services/pr-sync/pr-sync-watcher.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAWH,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,gFAAgF,CAAC;AACzH,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,4EAA4E,CAAC;AACtH,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,wEAAwE,CAAC;AAK5G,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,8EAA8E,CAAC;AAkBzH,qBAAa,oBAAoB;IAC/B,OAAO,CAAC,QAAQ,CAAC,WAAW,CAAqB;IACjD,OAAO,CAAC,QAAQ,CAAC,YAAY,CAAsB;IACnD,OAAO,CAAC,QAAQ,CAAC,YAAY,CAAgB;IAC7C,OAAO,CAAC,QAAQ,CAAC,mBAAmB,CAAuB;IAC3D,OAAO,CAAC,QAAQ,CAAC,cAAc,CAAS;IACxC,OAAO,CAAC,QAAQ,CAAC,eAAe,CAAqC;IACrE,OAAO,CAAC,QAAQ,CAAC,YAAY,CAAqB;IAClD,OAAO,CAAC,UAAU,CAA+C;gBAG/D,WAAW,EAAE,kBAAkB,EAC/B,YAAY,EAAE,mBAAmB,EACjC,YAAY,EAAE,aAAa,EAC3B,mBAAmB,EAAE,oBAAoB,EACzC,cAAc,GAAE,MAAiC;IASnD,SAAS,IAAI,OAAO;IAIpB,KAAK,IAAI,IAAI;IAcb,IAAI,IAAI,IAAI;YASE,IAAI;YAuCJ,iBAAiB;YA6CjB,cAAc;IAyK5B,8EAA8E;YAChE,gBAAgB;IAW9B,OAAO,CAAC,gBAAgB;CAoBzB;AAMD;;;;;GAKG;AACH,wBAAgB,uBAAuB,CACrC,WAAW,EAAE,kBAAkB,EAC/B,YAAY,EAAE,mBAAmB,EACjC,YAAY,EAAE,aAAa,EAC3B,mBAAmB,EAAE,oBAAoB,EACzC,cAAc,CAAC,EAAE,MAAM,GACtB,IAAI,CAYN;AAED;;;;;GAKG;AACH,wBAAgB,gBAAgB,IAAI,oBAAoB,CAQvD;AAED;;GAEG;AACH,wBAAgB,gBAAgB,IAAI,OAAO,CAE1C;AAED;;;;;GAKG;AACH,wBAAgB,kBAAkB,IAAI,IAAI,CAKzC"}
+{"version":3,"file":"pr-sync-watcher.service.d.ts","sourceRoot":"","sources":["../../../../../../../packages/core/src/infrastructure/services/pr-sync/pr-sync-watcher.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAWH,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,gFAAgF,CAAC;AACzH,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,4EAA4E,CAAC;AACtH,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,wEAAwE,CAAC;AAK5G,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,8EAA8E,CAAC;AACzH,OAAO,KAAK,QAAQ,MAAM,gBAAgB,CAAC;AAqB3C,qBAAa,oBAAoB;IAC/B,OAAO,CAAC,QAAQ,CAAC,WAAW,CAAqB;IACjD,OAAO,CAAC,QAAQ,CAAC,YAAY,CAAsB;IACnD,OAAO,CAAC,QAAQ,CAAC,YAAY,CAAgB;IAC7C,OAAO,CAAC,QAAQ,CAAC,mBAAmB,CAAuB;IAC3D,OAAO,CAAC,QAAQ,CAAC,cAAc,CAAS;IACxC,OAAO,CAAC,QAAQ,CAAC,eAAe,CAAqC;IACrE,OAAO,CAAC,QAAQ,CAAC,YAAY,CAAqB;IAClD,OAAO,CAAC,QAAQ,CAAC,gBAAgB,CAA6B;IAC9D,OAAO,CAAC,QAAQ,CAAC,EAAE,CAA2B;IAC9C,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAS;IACnC,OAAO,CAAC,UAAU,CAA+C;IACjE,OAAO,CAAC,SAAS,CAAK;gBAGpB,WAAW,EAAE,kBAAkB,EAC/B,YAAY,EAAE,mBAAmB,EACjC,YAAY,EAAE,aAAa,EAC3B,mBAAmB,EAAE,oBAAoB,EACzC,cAAc,GAAE,MAAiC,EACjD,EAAE,GAAE,QAAQ,CAAC,QAAQ,GAAG,IAAW;IAWrC,SAAS,IAAI,OAAO;IAIpB,KAAK,IAAI,IAAI;IAcb,IAAI,IAAI,IAAI;IASZ,gFAAgF;IAChF,OAAO,CAAC,cAAc;IAiBtB,0FAA0F;IAC1F,OAAO,CAAC,iBAAiB;YAUX,IAAI;IA6ClB,OAAO,CAAC,aAAa;IAYrB,OAAO,CAAC,oBAAoB;YAYd,iBAAiB;YA+CjB,cAAc;IAoL5B,8EAA8E;YAChE,gBAAgB;IAW9B,OAAO,CAAC,gBAAgB;CAoBzB;AAMD;;;;;GAKG;AACH,wBAAgB,uBAAuB,CACrC,WAAW,EAAE,kBAAkB,EAC/B,YAAY,EAAE,mBAAmB,EACjC,YAAY,EAAE,aAAa,EAC3B,mBAAmB,EAAE,oBAAoB,EACzC,cAAc,CAAC,EAAE,MAAM,EACvB,EAAE,CAAC,EAAE,QAAQ,CAAC,QAAQ,GAAG,IAAI,GAC5B,IAAI,CAaN;AAED;;;;;GAKG;AACH,wBAAgB,gBAAgB,IAAI,oBAAoB,CAQvD;AAED;;GAEG;AACH,wBAAgB,gBAAgB,IAAI,OAAO,CAE1C;AAED;;;;;GAKG;AACH,wBAAgB,kBAAkB,IAAI,IAAI,CAKzC"}

package/dist/packages/core/src/infrastructure/services/pr-sync/pr-sync-watcher.service.js

@@ -11,6 +11,8 @@
  */
 import { SdlcLifecycle, PrStatus, CiStatus, AgentRunStatus, NotificationEventType, NotificationSeverity, } from '../../../domain/generated/output.js';
 const DEFAULT_POLL_INTERVAL_MS = 30_000;
+const LOCK_TTL_MS = 60_000;
+const RATE_LIMIT_BACKOFF_MS = 5 * 60_000;
 const TAG = '[PrSyncWatcher]';
 const CI_STATUS_MAP = {
     success: CiStatus.Success,
@@ -25,13 +27,19 @@ export class PrSyncWatcherService {
     pollIntervalMs;
     trackedFeatures = new Map();
     skippedRepos = new Set();
+    rateLimitedUntil = new Map();
+    db;
+    processId;
     intervalId = null;
-    constructor(featureRepo, agentRunRepo, gitPrService, notificationService, pollIntervalMs = DEFAULT_POLL_INTERVAL_MS) {
+    pollCycle = 0;
+    constructor(featureRepo, agentRunRepo, gitPrService, notificationService, pollIntervalMs = DEFAULT_POLL_INTERVAL_MS, db = null) {
         this.featureRepo = featureRepo;
         this.agentRunRepo = agentRunRepo;
         this.gitPrService = gitPrService;
         this.notificationService = notificationService;
         this.pollIntervalMs = pollIntervalMs;
+        this.db = db;
+        this.processId = `${process.pid}-${Date.now()}`;
     }
     isRunning() {
         return this.intervalId !== null;
@@ -55,8 +63,40 @@ export class PrSyncWatcherService {
             console.log(`${TAG} Stopped`);
         }
     }
+    /** Attempt to acquire the cross-process poll lock. Returns true if acquired. */
+    tryAcquireLock() {
+        if (!this.db)
+            return true; // no DB → single-process mode, always proceed
+        const now = Date.now();
+        const expiresAt = now + LOCK_TTL_MS;
+        const stmt = this.db.prepare(`
+      INSERT OR REPLACE INTO pr_sync_lock (id, locked_by, locked_at, expires_at)
+      SELECT 1, ?, ?, ?
+      WHERE NOT EXISTS (
+        SELECT 1 FROM pr_sync_lock WHERE id = 1 AND locked_by != ? AND expires_at > ?
+      )
+    `);
+        const result = stmt.run(this.processId, now, expiresAt, this.processId, now);
+        return result.changes > 0;
+    }
+    /** Check if a feature should be polled this cycle based on its unchanged cycles count. */
+    shouldPollFeature(featureId) {
+        const tracked = this.trackedFeatures.get(featureId);
+        if (!tracked)
+            return true; // new feature, always poll
+        const uc = tracked.unchangedCycles;
+        if (uc <= 2)
+            return true; // poll every cycle
+        if (uc <= 5)
+            return this.pollCycle % 2 === 0; // poll every 2nd cycle
+        return this.pollCycle % 4 === 0; // poll every 4th cycle
+    }
     async poll() {
         try {
+            if (!this.tryAcquireLock()) {
+                return; // another process holds the lock — skip this cycle
+            }
+            this.pollCycle++;
             const allFeatures = await this.featureRepo.list({ lifecycle: SdlcLifecycle.Review });
             // Include features with a valid repositoryPath (with or without PR data)
             const features = allFeatures.filter((f) => f.repositoryPath);
@@ -89,9 +129,32 @@ export class PrSyncWatcherService {
             console.warn(`${TAG} Poll failed: ${msg}`);
         }
     }
+    isRateLimited(repoPath) {
+        const until = this.rateLimitedUntil.get(repoPath);
+        if (until === undefined)
+            return false;
+        if (Date.now() >= until) {
+            this.rateLimitedUntil.delete(repoPath);
+            // eslint-disable-next-line no-console
+            console.log(`${TAG} Rate limit backoff expired for ${repoPath}`);
+            return false;
+        }
+        return true;
+    }
+    handleRateLimitError(repoPath, error) {
+        const msg = error.message;
+        if (msg.includes('API rate limit exceeded') || msg.includes('rate limit')) {
+            const backoffUntil = Date.now() + RATE_LIMIT_BACKOFF_MS;
+            this.rateLimitedUntil.set(repoPath, backoffUntil);
+            // eslint-disable-next-line no-console
+            console.warn(`${TAG} Rate limited for ${repoPath}, backing off until ${new Date(backoffUntil).toISOString()}`);
+        }
+    }
     async processRepository(repoPath, features) {
         if (this.skippedRepos.has(repoPath))
             return;
+        if (this.isRateLimited(repoPath))
+            return;
         // Skip repos without a git remote — gh pr list will always fail
         try {
             if (!(await this.gitPrService.hasRemote(repoPath))) {
@@ -110,6 +173,8 @@ export class PrSyncWatcherService {
         }
         catch (error) {
             const msg = error instanceof Error ? error.message : String(error);
+            if (error instanceof Error)
+                this.handleRateLimitError(repoPath, error);
             // eslint-disable-next-line no-console
             console.warn(`${TAG} listPrStatuses failed for ${repoPath}: ${msg}`);
             return;
@@ -153,10 +218,15 @@ export class PrSyncWatcherService {
                 ciStatus: undefined,
                 mergeable: undefined,
                 featureName: feature.name,
+                unchangedCycles: 0,
             });
             await this.featureRepo.update(feature);
             return;
         }
+        // Check exponential backoff — skip features that haven't changed recently
+        if (!this.shouldPollFeature(feature.id)) {
+            return;
+        }
         const pr = feature.pr;
         const prStatusInfo = statusByNumber.get(pr.number);
         let needsUpdate = false;
@@ -169,6 +239,7 @@ export class PrSyncWatcherService {
                 ciStatus: pr.ciStatus,
                 mergeable: pr.mergeable,
                 featureName: feature.name,
+                unchangedCycles: 0,
             });
         }
         const tracked = this.trackedFeatures.get(feature.id);
@@ -223,12 +294,18 @@ export class PrSyncWatcherService {
         }
         catch (error) {
             const msg = error instanceof Error ? error.message : String(error);
+            if (error instanceof Error)
+                this.handleRateLimitError(feature.repositoryPath, error);
             // eslint-disable-next-line no-console
             console.warn(`${TAG} getCiStatus failed for "${feature.name}": ${msg}`);
         }
         if (needsUpdate) {
+            tracked.unchangedCycles = 0;
             await this.featureRepo.update(feature);
         }
+        else {
+            tracked.unchangedCycles++;
+        }
     }
     /** Mark associated agent run as completed so the UI reflects "done" state. */
     async completeAgentRun(feature) {
@@ -264,11 +341,11 @@ let watcherInstance = null;
  *
  * @throws Error if the watcher is already initialized
  */
-export function initializePrSyncWatcher(featureRepo, agentRunRepo, gitPrService, notificationService, pollIntervalMs) {
+export function initializePrSyncWatcher(featureRepo, agentRunRepo, gitPrService, notificationService, pollIntervalMs, db) {
     if (watcherInstance !== null) {
         throw new Error('PR sync watcher already initialized. Cannot re-initialize.');
     }
-    watcherInstance = new PrSyncWatcherService(featureRepo, agentRunRepo, gitPrService, notificationService, pollIntervalMs);
+    watcherInstance = new PrSyncWatcherService(featureRepo, agentRunRepo, gitPrService, notificationService, pollIntervalMs, db ?? null);
 }
 /**
  * Get the PR sync watcher singleton.

package/dist/packages/core/src/infrastructure/services/web-server.service.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"web-server.service.d.ts","sourceRoot":"","sources":["../../../../../../packages/core/src/infrastructure/services/web-server.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAGH,OAAO,IAAI,MAAM,MAAM,CAAC;AACxB,OAAO,IAAI,MAAM,WAAW,CAAC;AAG7B,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,yEAAyE,CAAC;AAIjH,MAAM,WAAW,aAAa;IAC5B,aAAa,EAAE,OAAO,IAAI,CAAC;IAC3B,gBAAgB,EAAE,OAAO,IAAI,CAAC,YAAY,CAAC;CAC5C;AAOD;;;;;;;;;GASG;AACH,wBAAgB,aAAa,IAAI;IAAE,GAAG,EAAE,MAAM,CAAC;IAAC,GAAG,EAAE,OAAO,CAAA;CAAE,CAsB7D;AAED,qBACa,gBAAiB,YAAW,iBAAiB;IACxD,OAAO,CAAC,GAAG,CAAwB;IACnC,OAAO,CAAC,MAAM,CAA4B;IAC1C,OAAO,CAAC,cAAc,CAAS;IAC/B,OAAO,CAAC,QAAQ,CAAC,IAAI,CAAgB;gBAEzB,IAAI,GAAE,OAAO,CAAC,aAAa,CAAM;IAI7C;;;;;OAKG;IACG,KAAK,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,EAAE,GAAG,UAAO,GAAG,OAAO,CAAC,IAAI,CAAC;IA2BjE;;;OAGG;IACG,IAAI,IAAI,OAAO,CAAC,IAAI,CAAC;CAuB5B"}
+{"version":3,"file":"web-server.service.d.ts","sourceRoot":"","sources":["../../../../../../packages/core/src/infrastructure/services/web-server.service.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAGH,OAAO,IAAI,MAAM,MAAM,CAAC;AACxB,OAAO,IAAI,MAAM,WAAW,CAAC;AAG7B,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,yEAAyE,CAAC;AAKjH,MAAM,WAAW,aAAa;IAC5B,aAAa,EAAE,OAAO,IAAI,CAAC;IAC3B,gBAAgB,EAAE,OAAO,IAAI,CAAC,YAAY,CAAC;CAC5C;AAOD;;;;;;;;;GASG;AACH,wBAAgB,aAAa,IAAI;IAAE,GAAG,EAAE,MAAM,CAAC;IAAC,GAAG,EAAE,OAAO,CAAA;CAAE,CAsB7D;AAED,qBACa,gBAAiB,YAAW,iBAAiB;IACxD,OAAO,CAAC,GAAG,CAAwB;IACnC,OAAO,CAAC,MAAM,CAA4B;IAC1C,OAAO,CAAC,cAAc,CAAS;IAC/B,OAAO,CAAC,QAAQ,CAAC,IAAI,CAAgB;gBAEzB,IAAI,GAAE,OAAO,CAAC,aAAa,CAAM;IAI7C;;;;;OAKG;IACG,KAAK,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,EAAE,GAAG,UAAO,GAAG,OAAO,CAAC,IAAI,CAAC;IA0CjE;;;OAGG;IACG,IAAI,IAAI,OAAO,CAAC,IAAI,CAAC;CAuB5B"}

package/dist/packages/core/src/infrastructure/services/web-server.service.js

@@ -22,6 +22,7 @@ import next from 'next';
 import http from 'node:http';
 import path from 'node:path';
 import fs from 'node:fs';
+import { IS_WINDOWS } from '../platform.js';
 const defaultDeps = {
     createNextApp: next,
     createHttpServer: http.createServer,
@@ -70,6 +71,20 @@ let WebServerService = class WebServerService {
      * @param dev - Whether to run in development mode (default: auto-detect)
      */
     async start(port, dir, dev = true) {
+        // On Windows, Next.js uses path.relative(CWD, dir) internally then
+        // reconstructs with path.join(CWD, relative). When CWD and dir are on
+        // different drives (e.g. D:\project vs C:\...\web), path.relative returns
+        // the absolute path (can't compute relative across drives), and path.join
+        // produces a mangled path like D:\project\C:\...\web\. Fix by ensuring
+        // CWD is on the same drive as dir.
+        if (IS_WINDOWS) {
+            const getDriveLetter = (p) => p.match(/^[a-zA-Z]:/)?.[0]?.toUpperCase();
+            const cwdDrive = getDriveLetter(process.cwd());
+            const dirDrive = getDriveLetter(dir);
+            if (cwdDrive && dirDrive && cwdDrive !== dirDrive) {
+                process.chdir(dir);
+            }
+        }
         const app = this.deps.createNextApp({
             dev,
             dir,

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

@@ -1 +1 @@
-{"version":3,"file":"ui.command.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/cli/commands/ui.command.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG;AAEH,OAAO,EAAE,OAAO,EAAwB,MAAM,WAAW,CAAC;AA+B1D;;GAEG;AACH,wBAAgB,eAAe,IAAI,OAAO,CAkFzC"}
+{"version":3,"file":"ui.command.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/cli/commands/ui.command.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG;AAEH,OAAO,EAAE,OAAO,EAAwB,MAAM,WAAW,CAAC;AAgC1D;;GAEG;AACH,wBAAgB,eAAe,IAAI,OAAO,CA0FzC"}

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

@@ -21,6 +21,7 @@ import { setVersionEnvVars } from '../../../../packages/core/src/infrastructure/
 import { resolveWebDir } from '../../../../packages/core/src/infrastructure/services/web-server.service.js';
 import { initializeNotificationWatcher, getNotificationWatcher, } from '../../../../packages/core/src/infrastructure/services/notifications/notification-watcher.service.js';
 import { initializePrSyncWatcher, getPrSyncWatcher, } from '../../../../packages/core/src/infrastructure/services/pr-sync/pr-sync-watcher.service.js';
+import { getExistingConnection } from '../../../../packages/core/src/infrastructure/persistence/sqlite/connection.js';
 import { BrowserOpenerService } from '../../../../packages/core/src/infrastructure/services/browser-opener.service.js';
 import { colors, fmt, messages } from '../ui/index.js';
 function parsePort(value) {
@@ -66,7 +67,8 @@ Examples:
             getNotificationWatcher().start();
             // Start PR sync watcher to detect PR/CI status transitions on GitHub
             const gitPrService = container.resolve('IGitPrService');
-            initializePrSyncWatcher(featureRepo, runRepo, gitPrService, notificationService);
+            const db = getExistingConnection();
+            initializePrSyncWatcher(featureRepo, runRepo, gitPrService, notificationService, undefined, db);
             getPrSyncWatcher().start();
             const url = `http://localhost:${port}`;
             messages.success(`Server ready at ${fmt.code(url)}`);

package/dist/src/presentation/web/app/actions/get-feature-drawer-data.d.ts

@@ -3,6 +3,9 @@ import type { FeatureNodeData } from '../../components/common/feature-node/index
  * Fetches full FeatureNodeData for a given feature ID.
  * Used by the drawer for targeted data syncing without triggering
  * a full router.refresh() / server component re-render.
+ *
+ * CI status and mergeable status are read from the DB (already updated
+ * by PrSyncWatcherService) instead of making duplicate GitHub API calls.
  */
 export declare function getFeatureDrawerData(featureId: string): Promise<FeatureNodeData | null>;
 //# sourceMappingURL=get-feature-drawer-data.d.ts.map

package/dist/src/presentation/web/app/actions/get-feature-drawer-data.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"get-feature-drawer-data.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/get-feature-drawer-data.ts"],"names":[],"mappings":"AAUA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kCAAkC,CAAC;AAQxE;;;;GAIG;AACH,wBAAsB,oBAAoB,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,eAAe,GAAG,IAAI,CAAC,CAsD7F"}
+{"version":3,"file":"get-feature-drawer-data.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/get-feature-drawer-data.ts"],"names":[],"mappings":"AASA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,kCAAkC,CAAC;AAExE;;;;;;;GAOG;AACH,wBAAsB,oBAAoB,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,eAAe,GAAG,IAAI,CAAC,CAgC7F"}

package/dist/src/presentation/web/app/actions/get-feature-drawer-data.js

@@ -1,16 +1,13 @@
 'use server';
 import { resolve } from '../../lib/server-container.js';
-import { CiStatus } from '../../../../../packages/core/src/domain/generated/output.js';
 import { buildFeatureNodeData } from '../build-feature-node-data.js';
-const CI_STATUS_MAP = {
-    success: CiStatus.Success,
-    failure: CiStatus.Failure,
-    pending: CiStatus.Pending,
-};
 /**
  * Fetches full FeatureNodeData for a given feature ID.
  * Used by the drawer for targeted data syncing without triggering
  * a full router.refresh() / server component re-render.
+ *
+ * CI status and mergeable status are read from the DB (already updated
+ * by PrSyncWatcherService) instead of making duplicate GitHub API calls.
  */
 export async function getFeatureDrawerData(featureId) {
     try {
@@ -23,37 +20,15 @@ export async function getFeatureDrawerData(featureId) {
         if (!feature)
             return null;
         const run = feature.agentRunId ? await agentRunRepo.findById(feature.agentRunId) : null;
-        const [repo, baseBranch, artifact, remoteUrl, liveMergeable, liveCiResult] = await Promise.all([
+        // CI status and mergeable status are read from the feature record (updated by
+        // PrSyncWatcherService every 30s) — no duplicate GitHub API calls needed.
+        // Only getDefaultBranch and getRemoteUrl are kept as they are local git operations.
+        const [repo, baseBranch, artifact, remoteUrl] = await Promise.all([
             repoRepo.findByPath(feature.repositoryPath).catch(() => null),
             gitPrService.getDefaultBranch(feature.repositoryPath).catch(() => 'main'),
             getArtifact.execute(featureId).catch(() => null),
             gitPrService.getRemoteUrl(feature.repositoryPath).catch(() => null),
-            feature.pr?.number
-                ? gitPrService
-                    .getMergeableStatus(feature.repositoryPath, feature.pr.number)
-                    .catch(() => undefined)
-                : Promise.resolve(undefined),
-            feature.branch
-                ? gitPrService.getCiStatus(feature.repositoryPath, feature.branch).catch(() => undefined)
-                : Promise.resolve(undefined),
         ]);
-        // Merge live PR status into feature data
-        if (feature.pr) {
-            const liveCiStatus = liveCiResult
-                ? (CI_STATUS_MAP[liveCiResult.status] ?? CiStatus.Pending)
-                : undefined;
-            const updates = {};
-            if (liveMergeable !== undefined)
-                updates.mergeable = liveMergeable;
-            if (liveCiStatus !== undefined)
-                updates.ciStatus = liveCiStatus;
-            if (Object.keys(updates).length > 0) {
-                feature.pr = { ...feature.pr, ...updates };
-                featureRepo.update(feature).catch(() => {
-                    /* best-effort persist */
-                });
-            }
-        }
         return buildFeatureNodeData(feature, run, {
             repositoryName: repo?.name,
             baseBranch,

package/dist/src/presentation/web/app/api/agent-events/route.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"route.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/app/api/agent-events/route.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAiBH,eAAO,MAAM,OAAO,kBAAkB,CAAC;AAqEvC,wBAAgB,GAAG,CAAC,OAAO,EAAE,OAAO,GAAG,QAAQ,CAuQ9C"}
+{"version":3,"file":"route.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/app/api/agent-events/route.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAiBH,eAAO,MAAM,OAAO,kBAAkB,CAAC;AAqEvC,wBAAgB,GAAG,CAAC,OAAO,EAAE,OAAO,GAAG,QAAQ,CA4R9C"}

package/dist/src/presentation/web/app/api/agent-events/route.js

@@ -191,9 +191,28 @@ export function GET(request) {
                             }
                             // Check for lifecycle change (agent stays "running" but moves through phases)
                             if (prev.lifecycle !== feature.lifecycle) {
+                                const prevLifecycle = prev.lifecycle;
                                 prev.lifecycle = feature.lifecycle;
                                 const nodeName = LIFECYCLE_TO_NODE[feature.lifecycle];
-                                if (nodeName) {
+                                // Emit MergeReviewReady when lifecycle transitions TO Review
+                                if (feature.lifecycle === SdlcLifecycle.Review &&
+                                    prevLifecycle !== SdlcLifecycle.Review) {
+                                    const prUrl = feature.pr?.url;
+                                    const message = prUrl
+                                        ? `Ready for merge review — PR: ${prUrl}`
+                                        : 'Ready for merge review';
+                                    emitEvent({
+                                        eventType: NotificationEventType.MergeReviewReady,
+                                        agentRunId: run.id,
+                                        featureId: feature.id,
+                                        featureName: feature.name,
+                                        phaseName: 'merge',
+                                        message,
+                                        severity: NotificationSeverity.Info,
+                                        timestamp: new Date().toISOString(),
+                                    });
+                                }
+                                else if (nodeName) {
                                     emitEvent({
                                         eventType: NotificationEventType.PhaseCompleted,
                                         agentRunId: run.id,

package/dist/src/presentation/web/app/api/cli-upgrade/route.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"route.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/app/api/cli-upgrade/route.ts"],"names":[],"mappings":"AAIA,eAAO,MAAM,OAAO,kBAAkB,CAAC;AAEvC,wBAAsB,IAAI,IAAI,OAAO,CAAC,QAAQ,CAAC,CA+B9C"}
+{"version":3,"file":"route.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/app/api/cli-upgrade/route.ts"],"names":[],"mappings":"AAIA,eAAO,MAAM,OAAO,kBAAkB,CAAC;AAEvC,wBAAsB,IAAI,IAAI,OAAO,CAAC,QAAQ,CAAC,CA4C9C"}

package/dist/src/presentation/web/app/api/cli-upgrade/route.js

@@ -11,13 +11,26 @@ export async function POST() {
                     controller.enqueue(encoder.encode(`data: ${chunk.replace(/\n/g, '\ndata: ')}\n\n`));
                 });
                 controller.enqueue(encoder.encode(`event: done\ndata: ${JSON.stringify(result)}\n\n`));
+                // If the upgrade succeeded, schedule a daemon self-restart so the
+                // new version is loaded automatically — no manual restart needed.
+                if (result.status === 'upgraded') {
+                    controller.enqueue(encoder.encode(`event: restarting\ndata: restarting\n\n`));
+                    controller.close();
+                    await useCase.scheduleDaemonRestart();
+                    return;
+                }
             }
             catch (error) {
                 const message = error instanceof Error ? error.message : 'Upgrade failed';
                 controller.enqueue(encoder.encode(`event: done\ndata: ${JSON.stringify({ status: 'error', currentVersion: '', latestVersion: null, errorMessage: message })}\n\n`));
             }
             finally {
-                controller.close();
+                try {
+                    controller.close();
+                }
+                catch {
+                    // Already closed (e.g. after restart path)
+                }
             }
         },
     });

package/dist/src/presentation/web/app/api/tools/[id]/launch/route.js

@@ -6,7 +6,7 @@ export async function POST(_request, { params }) {
         const useCase = resolve('LaunchToolUseCase');
         const result = await useCase.execute({
             toolId: id,
-            directoryPath: process.cwd(),
+            directoryPath: process.env.NEXT_PUBLIC_SHEP_INSTANCE_PATH ?? process.cwd(),
             headless: true,
         });
         if (result.ok) {

package/dist/src/presentation/web/components/common/feature-node/derive-feature-state.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"derive-feature-state.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/feature-node/derive-feature-state.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,EAIL,qBAAqB,EACtB,MAAM,+BAA+B,CAAC;AACvC,OAAO,KAAK,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,+BAA+B,CAAC;AACvE,OAAO,KAAK,EAAE,gBAAgB,EAAE,qBAAqB,EAAE,MAAM,6BAA6B,CAAC;AAE3F;;;;;;;;;;GAUG;AACH,wBAAgB,eAAe,CAC7B,OAAO,EAAE,OAAO,EAChB,QAAQ,CAAC,EAAE,QAAQ,GAAG,IAAI,EAC1B,OAAO,CAAC,EAAE;IAAE,UAAU,CAAC,EAAE,OAAO,CAAA;CAAE,GACjC,gBAAgB,CAkDlB;AAED;;;;;GAKG;AACH,wBAAgB,cAAc,CAAC,OAAO,EAAE,OAAO,GAAG,MAAM,CAYvD;AAED,oGAAoG;AACpG,wBAAgB,mBAAmB,CAAC,SAAS,EAAE,qBAAqB,GAAG,gBAAgB,CAoBtF;AAED,wEAAwE;AACxE,eAAO,MAAM,gBAAgB,EAAE,MAAM,CAAC,MAAM,EAAE,qBAAqB,CAOlE,CAAC;AAcF;;;GAGG;AACH,wBAAgB,eAAe,CAAC,OAAO,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,IAAI,GAAG,qBAAqB,CAQ7F;AAED,MAAM,WAAW,cAAc;IAC7B,SAAS,EAAE,MAAM,CAAC;IAClB,KAAK,EAAE,gBAAgB,GAAG,SAAS,CAAC;IACpC,SAAS,EAAE,qBAAqB,GAAG,SAAS,CAAC;IAC7C,SAAS,EAAE,qBAAqB,CAAC;IACjC,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED;;;;;GAKG;AACH,wBAAgB,sBAAsB,CACpC,MAAM,EAAE,SAAS;IAAE,SAAS,EAAE,MAAM,CAAC;IAAC,SAAS,EAAE,qBAAqB,CAAC;IAAC,SAAS,CAAC,EAAE,MAAM,CAAA;CAAE,EAAE,GAC7F,cAAc,EAAE,CAmBlB;AAED,wBAAgB,uBAAuB,CACrC,SAAS,EAAE,MAAM,GAAG,SAAS,GAC5B,qBAAqB,GAAG,SAAS,CAGnC"}
+{"version":3,"file":"derive-feature-state.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/feature-node/derive-feature-state.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,EAIL,qBAAqB,EACtB,MAAM,+BAA+B,CAAC;AACvC,OAAO,KAAK,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,+BAA+B,CAAC;AACvE,OAAO,KAAK,EAAE,gBAAgB,EAAE,qBAAqB,EAAE,MAAM,6BAA6B,CAAC;AAE3F;;;;;;;;;;GAUG;AACH,wBAAgB,eAAe,CAC7B,OAAO,EAAE,OAAO,EAChB,QAAQ,CAAC,EAAE,QAAQ,GAAG,IAAI,EAC1B,OAAO,CAAC,EAAE;IAAE,UAAU,CAAC,EAAE,OAAO,CAAA;CAAE,GACjC,gBAAgB,CAkDlB;AAED;;;;;GAKG;AACH,wBAAgB,cAAc,CAAC,OAAO,EAAE,OAAO,GAAG,MAAM,CAYvD;AAED,oGAAoG;AACpG,wBAAgB,mBAAmB,CAAC,SAAS,EAAE,qBAAqB,GAAG,gBAAgB,CAsBtF;AAED,wEAAwE;AACxE,eAAO,MAAM,gBAAgB,EAAE,MAAM,CAAC,MAAM,EAAE,qBAAqB,CAOlE,CAAC;AAcF;;;GAGG;AACH,wBAAgB,eAAe,CAAC,OAAO,EAAE,OAAO,EAAE,GAAG,EAAE,QAAQ,GAAG,IAAI,GAAG,qBAAqB,CAQ7F;AAED,MAAM,WAAW,cAAc;IAC7B,SAAS,EAAE,MAAM,CAAC;IAClB,KAAK,EAAE,gBAAgB,GAAG,SAAS,CAAC;IACpC,SAAS,EAAE,qBAAqB,GAAG,SAAS,CAAC;IAC7C,SAAS,EAAE,qBAAqB,CAAC;IACjC,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED;;;;;GAKG;AACH,wBAAgB,sBAAsB,CACpC,MAAM,EAAE,SAAS;IAAE,SAAS,EAAE,MAAM,CAAC;IAAC,SAAS,EAAE,qBAAqB,CAAC;IAAC,SAAS,CAAC,EAAE,MAAM,CAAA;CAAE,EAAE,GAC7F,cAAc,EAAE,CAmBlB;AAED,wBAAgB,uBAAuB,CACrC,SAAS,EAAE,MAAM,GAAG,SAAS,GAC5B,qBAAqB,GAAG,SAAS,CAGnC"}

package/dist/src/presentation/web/components/common/feature-node/derive-feature-state.js

@@ -99,6 +99,8 @@ export function mapEventTypeToState(eventType) {
             return 'action-required';
         case NotificationEventType.PrBlocked:
             return 'blocked';
+        case NotificationEventType.MergeReviewReady:
+            return 'action-required';
     }
 }
 /** Map domain SdlcLifecycle enum values to UI FeatureLifecyclePhase. */

package/dist/src/presentation/web/components/common/version-badge/version-badge.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"version-badge.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/version-badge/version-badge.tsx"],"names":[],"mappings":"AAMA,MAAM,WAAW,iBAAiB;IAChC,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,YAAY,CAAC,EAAE,MAAM,CAAC;CACvB;AAED,wBAAgB,YAAY,CAAC,EAC3B,OAAO,EACP,MAAM,EACN,UAAU,EACV,KAAa,EACb,WAA2B,EAC3B,WAAW,EACX,YAAY,GACb,EAAE,iBAAiB,2CA2EnB"}
+{"version":3,"file":"version-badge.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/version-badge/version-badge.tsx"],"names":[],"mappings":"AAMA,MAAM,WAAW,iBAAiB;IAChC,OAAO,EAAE,MAAM,CAAC;IAChB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,YAAY,CAAC,EAAE,MAAM,CAAC;CACvB;AAED,wBAAgB,YAAY,CAAC,EAC3B,OAAO,EACP,MAAM,EACN,UAAU,EACV,KAAa,EACb,WAA2B,EAC3B,WAAW,EACX,YAAY,GACb,EAAE,iBAAiB,2CAyFnB"}

package/dist/src/presentation/web/components/common/version-badge/version-badge.js

@@ -10,9 +10,14 @@ export function VersionBadge({ version, branch, commitHash, isDev = false, packa
     // In dev mode show "1.92.2-dev", in production show "v1.92.2"
     const displayVersion = isDev ? `${version}-dev` : `v${version}`;
     const isUpgrading = upgradeStatus === 'upgrading';
+    const isRestarting = upgradeStatus === 'restarting';
     const didUpgrade = upgradeStatus === 'upgraded';
     const upgradeError = upgradeStatus === 'error';
-    return (_jsx(TooltipProvider, { delayDuration: 200, children: _jsxs(Tooltip, { children: [_jsx(TooltipTrigger, { asChild: true, children: _jsxs("span", { className: "text-muted-foreground/80 hover:text-muted-foreground relative cursor-default text-[11px] leading-tight transition-colors", "data-testid": "version-label", children: [displayVersion, updateAvailable && !didUpgrade ? (_jsx("span", { className: "absolute -top-0.5 -right-1.5 size-1.5 rounded-full bg-emerald-400", "data-testid": "update-dot" })) : null] }) }), _jsxs(TooltipContent, { side: "right", className: "max-w-[280px] space-y-1 p-3 text-left", children: [_jsx("div", { className: "mb-1.5 text-xs font-semibold", children: packageName }), description ? (_jsx("div", { className: "text-[10px] leading-snug opacity-70", children: description })) : null, _jsxs("div", { className: "border-t border-white/10 pt-1.5", children: [_jsx(Row, { label: "Version", value: displayVersion }), isDev && branch ? _jsx(Row, { label: "Branch", value: branch }) : null, isDev && shortHash ? _jsx(Row, { label: "Commit", value: shortHash, mono: true }) : null, isDev && instancePath ? _jsx(Row, { label: "Path", value: instancePath, mono: true }) : null, latest ? (_jsx(Row, { label: "Latest", value: `v${latest}`, highlight: updateAvailable })) : null] }), didUpgrade ? (_jsx("div", { className: "border-t border-white/10 pt-1.5", children: _jsx("span", { className: "text-[10px] font-medium text-emerald-400", "data-testid": "upgrade-success", children: "Upgraded successfully \u2014 restart to apply" }) })) : upgradeError ? (_jsx("div", { className: "border-t border-white/10 pt-1.5", children: _jsx("span", { className: "text-[10px] font-medium text-red-400", "data-testid": "upgrade-error", children: "Upgrade failed" }) })) : updateAvailable ? (_jsx("div", { className: "border-t border-white/10 pt-1.5", children: _jsx("button", { type: "button", onClick: startUpgrade, disabled: isUpgrading, className: "inline-flex items-center gap-1 text-[10px] font-medium text-emerald-400 transition-colors hover:text-emerald-300 disabled:opacity-50", "data-testid": "upgrade-button", children: isUpgrading ? 'Upgrading...' : `Upgrade to v${latest}` }) })) : null] })] }) }));
+    return (_jsx(TooltipProvider, { delayDuration: 200, children: _jsxs(Tooltip, { children: [_jsx(TooltipTrigger, { asChild: true, children: _jsxs("span", { className: "text-muted-foreground/80 hover:text-muted-foreground relative cursor-default text-[11px] leading-tight transition-colors", "data-testid": "version-label", children: [displayVersion, updateAvailable && !didUpgrade && !isRestarting ? (_jsx("span", { className: "absolute -top-0.5 -right-1.5 size-1.5 rounded-full bg-emerald-400", "data-testid": "update-dot" })) : null] }) }), _jsxs(TooltipContent, { side: "right", className: "max-w-[280px] space-y-1 p-3 text-left", children: [_jsx("div", { className: "mb-1.5 text-xs font-semibold", children: packageName }), description ? (_jsx("div", { className: "text-[10px] leading-snug opacity-70", children: description })) : null, _jsxs("div", { className: "border-t border-white/10 pt-1.5", children: [_jsx(Row, { label: "Version", value: displayVersion }), isDev && branch ? _jsx(Row, { label: "Branch", value: branch }) : null, isDev && shortHash ? _jsx(Row, { label: "Commit", value: shortHash, mono: true }) : null, isDev && instancePath ? _jsx(Row, { label: "Path", value: instancePath, mono: true }) : null, latest ? (_jsx(Row, { label: "Latest", value: `v${latest}`, highlight: updateAvailable })) : null] }), isRestarting ? (_jsx("div", { className: "border-t border-white/10 pt-1.5", children: _jsx("span", { className: "text-[10px] font-medium text-emerald-400", "data-testid": "upgrade-restarting", children: "Restarting..." }) })) : didUpgrade ? (_jsx("div", { className: "border-t border-white/10 pt-1.5", children: _jsx("span", { className: "text-[10px] font-medium text-emerald-400", "data-testid": "upgrade-success", children: "Upgraded successfully" }) })) : upgradeError ? (_jsx("div", { className: "border-t border-white/10 pt-1.5", children: _jsx("span", { className: "text-[10px] font-medium text-red-400", "data-testid": "upgrade-error", children: "Upgrade failed" }) })) : updateAvailable ? (_jsx("div", { className: "border-t border-white/10 pt-1.5", children: _jsx("button", { type: "button", onClick: startUpgrade, disabled: isUpgrading || isRestarting, className: "inline-flex items-center gap-1 text-[10px] font-medium text-emerald-400 transition-colors hover:text-emerald-300 disabled:opacity-50", "data-testid": "upgrade-button", children: isUpgrading
+                                    ? 'Upgrading...'
+                                    : isRestarting
+                                        ? 'Restarting...'
+                                        : `Upgrade to v${latest}` }) })) : null] })] }) }));
 }
 function Row({ label, value, highlight, mono, }) {
     return (_jsxs("div", { className: "flex items-center justify-between gap-4 text-[10px]", children: [_jsx("span", { className: "opacity-60", children: label }), _jsx("span", { className: [

package/dist/src/presentation/web/components/common/version-badge/version-badge.stories.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"version-badge.stories.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/version-badge/version-badge.stories.tsx"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAa,MAAM,kBAAkB,CAAC;AAClE,OAAO,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAC;AAwD/C,QAAA,MAAM,IAAI,EAAE,IAAI,CAAC,OAAO,YAAY,CAOnC,CAAC;AAEF,eAAe,IAAI,CAAC;AACpB,KAAK,KAAK,GAAG,QAAQ,CAAC,OAAO,YAAY,CAAC,CAAC;AAE3C,mFAAmF;AACnF,eAAO,MAAM,UAAU,EAAE,KAOxB,CAAC;AAEF,wFAAwF;AACxF,eAAO,MAAM,WAAW,EAAE,KAWzB,CAAC;AAEF,mEAAmE;AACnE,eAAO,MAAM,UAAU,EAAE,KAWxB,CAAC;AAEF,yCAAyC;AACzC,eAAO,MAAM,OAAO,EAAE,KAKrB,CAAC;AAEF,sFAAsF;AACtF,eAAO,MAAM,eAAe,EAAE,KAO7B,CAAC;AAEF,oFAAoF;AACpF,eAAO,MAAM,kBAAkB,EAAE,KAWhC,CAAC;AAEF,kEAAkE;AAClE,eAAO,MAAM,UAAU,EAAE,KAOxB,CAAC"}
+{"version":3,"file":"version-badge.stories.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/version-badge/version-badge.stories.tsx"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAa,MAAM,kBAAkB,CAAC;AAClE,OAAO,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAC;AA+D/C,QAAA,MAAM,IAAI,EAAE,IAAI,CAAC,OAAO,YAAY,CAOnC,CAAC;AAEF,eAAe,IAAI,CAAC;AACpB,KAAK,KAAK,GAAG,QAAQ,CAAC,OAAO,YAAY,CAAC,CAAC;AAE3C,mFAAmF;AACnF,eAAO,MAAM,UAAU,EAAE,KAOxB,CAAC;AAEF,wFAAwF;AACxF,eAAO,MAAM,WAAW,EAAE,KAWzB,CAAC;AAEF,mEAAmE;AACnE,eAAO,MAAM,UAAU,EAAE,KAWxB,CAAC;AAEF,yCAAyC;AACzC,eAAO,MAAM,OAAO,EAAE,KAKrB,CAAC;AAEF,sFAAsF;AACtF,eAAO,MAAM,eAAe,EAAE,KAO7B,CAAC;AAEF,oFAAoF;AACpF,eAAO,MAAM,kBAAkB,EAAE,KAWhC,CAAC;AAEF,kEAAkE;AAClE,eAAO,MAAM,UAAU,EAAE,KAOxB,CAAC"}

package/dist/src/presentation/web/components/common/version-badge/version-badge.stories.js

@@ -22,6 +22,7 @@ function withNpmVersion(latest) {
                         controller.enqueue(encoder.encode(`data: Upgrading to v${latest}...\n\n`));
                         setTimeout(() => {
                             controller.enqueue(encoder.encode(`event: done\ndata: ${JSON.stringify({ status: 'upgraded', currentVersion: '1.90.0', latestVersion: latest })}\n\n`));
+                            controller.enqueue(encoder.encode(`event: restarting\ndata: restarting\n\n`));
                             controller.close();
                         }, 1000);
                     },
@@ -31,6 +32,12 @@ function withNpmVersion(latest) {
                     headers: { 'Content-Type': 'text/event-stream' },
                 });
             }
+            if (url.includes('/api/version')) {
+                return new Response(JSON.stringify({ version: latest }), {
+                    status: 200,
+                    headers: { 'Content-Type': 'application/json' },
+                });
+            }
             return originalFetch(input, init);
         });
         return _jsx(Story, {});

package/dist/src/presentation/web/components/features/settings/notification-settings-section.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"notification-settings-section.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/features/settings/notification-settings-section.tsx"],"names":[],"mappings":"AAUA,OAAO,KAAK,EAAE,uBAAuB,EAAE,MAAM,sCAAsC,CAAC;AAkBpF,MAAM,WAAW,gCAAgC;IAC/C,aAAa,EAAE,uBAAuB,CAAC;CACxC;AAED,wBAAgB,2BAA2B,CAAC,EAAE,aAAa,EAAE,EAAE,gCAAgC,2CAkJ9F"}
+{"version":3,"file":"notification-settings-section.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/features/settings/notification-settings-section.tsx"],"names":[],"mappings":"AAUA,OAAO,KAAK,EAAE,uBAAuB,EAAE,MAAM,sCAAsC,CAAC;AAmBpF,MAAM,WAAW,gCAAgC;IAC/C,aAAa,EAAE,uBAAuB,CAAC;CACxC;AAED,wBAAgB,2BAA2B,CAAC,EAAE,aAAa,EAAE,EAAE,gCAAgC,2CAkJ9F"}

package/dist/src/presentation/web/components/features/settings/notification-settings-section.js

@@ -16,6 +16,7 @@ const AGENT_EVENT_TOGGLES = [
     { key: 'agentFailed', label: 'Agent Failed' },
 ];
 const PR_EVENT_TOGGLES = [
+    { key: 'mergeReviewReady', label: 'Merge Review Ready' },
     { key: 'prMerged', label: 'PR Merged' },
     { key: 'prClosed', label: 'PR Closed' },
     { key: 'prChecksPassed', label: 'PR Checks Passed' },

package/dist/src/presentation/web/components/features/settings/notification-settings-section.stories.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"notification-settings-section.stories.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/features/settings/notification-settings-section.stories.tsx"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAQ,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AACvD,OAAO,EAAE,2BAA2B,EAAE,MAAM,iCAAiC,CAAC;AAE9E,QAAA,MAAM,IAAI;;;;;;;CAO0C,CAAC;AAErD,eAAe,IAAI,CAAC;AACpB,KAAK,KAAK,GAAG,QAAQ,CAAC,OAAO,IAAI,CAAC,CAAC;AA4BnC,eAAO,MAAM,OAAO,EAAE,KASrB,CAAC;AAEF,eAAO,MAAM,UAAU,EAAE,KASxB,CAAC;AAEF,eAAO,MAAM,WAAW,EAAE,KASzB,CAAC"}
+{"version":3,"file":"notification-settings-section.stories.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/features/settings/notification-settings-section.stories.tsx"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAQ,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AACvD,OAAO,EAAE,2BAA2B,EAAE,MAAM,iCAAiC,CAAC;AAE9E,QAAA,MAAM,IAAI;;;;;;;CAO0C,CAAC;AAErD,eAAe,IAAI,CAAC;AACpB,KAAK,KAAK,GAAG,QAAQ,CAAC,OAAO,IAAI,CAAC,CAAC;AA8BnC,eAAO,MAAM,OAAO,EAAE,KASrB,CAAC;AAEF,eAAO,MAAM,UAAU,EAAE,KASxB,CAAC;AAEF,eAAO,MAAM,WAAW,EAAE,KASzB,CAAC"}