@shepai/cli 1.108.0 → 1.109.0-pr333.db60eb7

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

@@ -14,6 +14,7 @@ const allEvents = {
     waitingApproval: true,
     agentCompleted: true,
     agentFailed: true,
+    mergeReviewReady: true,
     prMerged: true,
     prClosed: true,
     prChecksPassed: true,
@@ -26,6 +27,7 @@ const noEvents = {
     waitingApproval: false,
     agentCompleted: false,
     agentFailed: false,
+    mergeReviewReady: false,
     prMerged: false,
     prClosed: false,
     prChecksPassed: false,

package/dist/src/presentation/web/dev-server.js

@@ -19,6 +19,7 @@ import { InitializeSettingsUseCase } from '../../../packages/core/src/applicatio
 import { initializeSettings } from '../../../packages/core/src/infrastructure/services/settings.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';
 const DEFAULT_PORT = 3000;
 async function isPortAvailable(port) {
     return new Promise((resolve) => {
@@ -64,7 +65,8 @@ async function main() {
         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();
     }
     catch (error) {

package/dist/src/presentation/web/hooks/use-cli-upgrade.d.ts

@@ -1,4 +1,4 @@
-export type UpgradeStatus = 'idle' | 'upgrading' | 'upgraded' | 'up-to-date' | 'error';
+export type UpgradeStatus = 'idle' | 'upgrading' | 'restarting' | 'upgraded' | 'up-to-date' | 'error';
 export interface CliUpgradeState {
     status: UpgradeStatus;
     output: string;

package/dist/src/presentation/web/hooks/use-cli-upgrade.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"use-cli-upgrade.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/hooks/use-cli-upgrade.ts"],"names":[],"mappings":"AAIA,MAAM,MAAM,aAAa,GAAG,MAAM,GAAG,WAAW,GAAG,UAAU,GAAG,YAAY,GAAG,OAAO,CAAC;AAEvF,MAAM,WAAW,eAAe;IAC9B,MAAM,EAAE,aAAa,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,YAAY,CAAC,EAAE,MAAM,CAAC;CACvB;AAED,MAAM,WAAW,iBAAiB;IAChC,YAAY,EAAE,MAAM,IAAI,CAAC;CAC1B;AAED,wBAAgB,aAAa,IAAI,eAAe,GAAG,iBAAiB,CA6EnE"}
+{"version":3,"file":"use-cli-upgrade.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/hooks/use-cli-upgrade.ts"],"names":[],"mappings":"AAIA,MAAM,MAAM,aAAa,GACrB,MAAM,GACN,WAAW,GACX,YAAY,GACZ,UAAU,GACV,YAAY,GACZ,OAAO,CAAC;AAOZ,MAAM,WAAW,eAAe;IAC9B,MAAM,EAAE,aAAa,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC;IACf,YAAY,CAAC,EAAE,MAAM,CAAC;CACvB;AAED,MAAM,WAAW,iBAAiB;IAChC,YAAY,EAAE,MAAM,IAAI,CAAC;CAC1B;AA4BD,wBAAgB,aAAa,IAAI,eAAe,GAAG,iBAAiB,CAuFnE"}

package/dist/src/presentation/web/hooks/use-cli-upgrade.js

@@ -1,16 +1,46 @@
 'use client';
 import { useCallback, useRef, useState } from 'react';
+/** How often to poll the server during restart (ms). */
+const RESTART_POLL_MS = 500;
+/** Maximum time to wait for the server to come back after restart (ms). */
+const RESTART_TIMEOUT_MS = 30_000;
+/**
+ * Poll the server until it responds, then reload the page.
+ */
+function waitForServerAndReload(timeoutMs, pollMs) {
+    const deadline = Date.now() + timeoutMs;
+    const poll = () => {
+        if (Date.now() > deadline)
+            return;
+        fetch('/api/version', { method: 'GET', cache: 'no-store' })
+            .then((res) => {
+            if (res.ok) {
+                window.location.reload();
+            }
+            else {
+                setTimeout(poll, pollMs);
+            }
+        })
+            .catch(() => {
+            setTimeout(poll, pollMs);
+        });
+    };
+    // Brief initial delay to let the old server shut down
+    setTimeout(poll, pollMs);
+}
 export function useCliUpgrade() {
     const [status, setStatus] = useState('idle');
     const [output, setOutput] = useState('');
     const [errorMessage, setErrorMessage] = useState();
     const abortRef = useRef(null);
+    const restartingRef = useRef(false);
     const startUpgrade = useCallback(() => {
-        if (status === 'upgrading')
+        if (status === 'upgrading' || status === 'restarting')
             return;
         setStatus('upgrading');
         setOutput('');
         setErrorMessage(undefined);
+        restartingRef.current = false;
         const controller = new AbortController();
         abortRef.current = controller;
         fetch('/api/cli-upgrade', { method: 'POST', signal: controller.signal })
@@ -31,6 +61,12 @@ export function useCliUpgrade() {
                 const lines = buffer.split('\n');
                 buffer = lines.pop() ?? '';
                 for (const line of lines) {
+                    if (line.startsWith('event: restarting')) {
+                        restartingRef.current = true;
+                        setStatus('restarting');
+                        waitForServerAndReload(RESTART_TIMEOUT_MS, RESTART_POLL_MS);
+                        continue;
+                    }
                     if (line.startsWith('event: done')) {
                         // Next data line contains the final result
                         continue;
@@ -60,6 +96,9 @@ export function useCliUpgrade() {
             }
         })
             .catch((err) => {
+            // If we're in restarting state, the connection dropping is expected
+            if (restartingRef.current)
+                return;
             if (err instanceof DOMException && err.name === 'AbortError')
                 return;
             setStatus('error');

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

@@ -1 +1 @@
-{"version":3,"file":"use-notifications.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/hooks/use-notifications.ts"],"names":[],"mappings":"AA0CA,wBAAgB,gBAAgB,IAAI,IAAI,CA8CvC"}
+{"version":3,"file":"use-notifications.d.ts","sourceRoot":"","sources":["../../../../../src/presentation/web/hooks/use-notifications.ts"],"names":[],"mappings":"AA4CA,wBAAgB,gBAAgB,IAAI,IAAI,CAiDvC"}

package/dist/src/presentation/web/hooks/use-notifications.js

@@ -13,7 +13,8 @@ const SEVERITY_TO_TOAST = {
 };
 function dispatchToast(event, navigate) {
     const method = SEVERITY_TO_TOAST[event.severity] ?? 'info';
-    const isActionable = event.eventType === NotificationEventType.WaitingApproval;
+    const isActionable = event.eventType === NotificationEventType.WaitingApproval ||
+        event.eventType === NotificationEventType.MergeReviewReady;
     toast[method](event.featureName, {
         description: event.message,
         ...(isActionable &&
@@ -56,8 +57,11 @@ export function useNotifications() {
         const newEvents = events.slice(processedCountRef.current);
         processedCountRef.current = events.length;
         for (const event of newEvents) {
-            // Only notify for actionable events and completion celebrations
-            if (event.severity !== NotificationSeverity.Error &&
+            // Only notify for actionable events and completion celebrations.
+            // MergeReviewReady is Info severity but always shown as it requires user action.
+            const isAlwaysShown = event.eventType === NotificationEventType.MergeReviewReady;
+            if (!isAlwaysShown &&
+                event.severity !== NotificationSeverity.Error &&
                 event.severity !== NotificationSeverity.Warning &&
                 event.severity !== NotificationSeverity.Success) {
                 continue;

package/dist/src/presentation/web/lib/skills.js

@@ -122,7 +122,7 @@ async function getSkillsFromDirectory(dirPath, source) {
     return skills;
 }
 export async function getSkills(projectRoot, homeDir) {
-    const root = projectRoot ?? process.cwd();
+    const root = projectRoot ?? process.env.NEXT_PUBLIC_SHEP_INSTANCE_PATH ?? process.cwd();
     const home = homeDir ?? homedir();
     const projectDir = join(root, '.claude', 'skills');
     const globalDir = join(home, '.claude', 'skills');