PR Summary

Low Risk
Low risk UI behavior change limited to notification dismissal timing. Main risk is unintentionally auto-dismissing workflow info/warn messages users expected to persist.

Overview
Workflow-scoped notifications are now considered auto-dismissible based solely on presence of workflowId, so non-error workflow notifications (e.g., info) will also show the countdown ring and dismiss after AUTO_DISMISS_MS.

Updates the inline doc comment to reflect the broader auto-dismiss behavior (from workflow error to workflow-scoped notifications).

^{Written by Cursor Bugbot for commit a667c48. Configure here.}

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Mar 25, 2026 11:03pm

Greptile Summary

This PR broadens the auto-dismiss behaviour in notifications.tsx so that all workflow-scoped notifications (those with a workflowId) auto-dismiss after 10 s, instead of only error-level ones. The change is a single-line predicate update in isAutoDismissable plus a matching JSDoc update.

• Changed isAutoDismissable from return n.level === 'error' && !!n.workflowId → return !!n.workflowId, enabling auto-dismiss for 'info'-level workflow notifications (e.g. "credential connected").
• The Notification type only has two severity levels ('info' | 'error'), so the simplified predicate covers both without any gaps.
• The countdown ring, pause-on-click, and cleanup logic are all unaffected — they already operated on whatever isAutoDismissable returned.
• Global notifications (no workflowId) still never auto-dismiss, preserving that distinction.

Confidence Score: 5/5

• Safe to merge — single-predicate change with no side effects on existing logic.
• The diff is a one-line predicate simplification. The Notification type only has 'info' | 'error' levels, so dropping the level guard cleanly covers the intended new case. All surrounding timer, animation, and pause logic is untouched and already handles both levels correctly. No regressions are expected for error notifications, which behaved the same way before.
• No files require special attention.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant Store as NotificationStore
    participant Comp as Notifications Component
    participant Timer as setTimeout

    Store->>Comp: notification arrives (workflowId set, level='info')
    Note over Comp: isAutoDismissable → true (was false before)
    Comp->>Timer: schedule dismiss in 10 000 ms
    Note over Comp: CountdownRing rendered
    alt User clicks ring
        Comp->>Timer: clearTimeout (pause all)
        Note over Comp: isPaused = true, ring hidden
    else Timer fires
        Timer->>Comp: setExitingIds (exit animation)
        Comp->>Store: removeNotification after 200 ms
    end

_{Reviews (1): Last reviewed commit: "fix(notifications): auto-dismiss info-le..." | Re-trigger Greptile}