fix(notifications): auto-dismiss info-level workflow notifications

[waleedlatif1]

Summary

• Info-level workflow notifications (e.g. credential connected) now auto-dismiss with a countdown ring after 10s
• Previously only error-level notifications were auto-dismissable

Type of Change

• Bug fix

Testing

Tested manually

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[cursor]

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.}

[vercel]

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-apps]

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.

• ChangedisAutoDismissable 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

Filename	Overview
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/notifications/notifications.tsx	Removes the n.level === 'error' guard from isAutoDismissable, so all workflow-scoped notifications (both 'info' and 'error') now auto-dismiss after 10 s. The JSDoc is updated to match. Change is minimal, correct, and consistent with the existing auto-dismiss infrastructure.

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

Loading

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