@shepai/cli 1.109.0 → 1.110.0-pr341.5c89549

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/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;