fix(notifications): consolidate notification utils, update email styling

[waleedlatif1]

Summary

• consolidate notification utils, update email styling
• removed trace spans from email notifs since it is extremely large and unparseable

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)

[vercel]

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

1 Skipped Deployment

Project	Deployment	Review	Updated (UTC)
docs	Skipped		Jan 15, 2026 2:14am

[greptile-apps]

Greptile Summary

This PR consolidates notification email rendering into a shared React component, improving maintainability and consistency. The changes also remove the trace spans feature from notifications due to payload size concerns.

Key changes:

• Created WorkflowNotificationEmail component with proper type definitions for email-safe serialization
• Refactored workspace-notification-delivery.ts to use the shared template instead of inline HTML
• Added codeBlock style to base email styles for JSON/code display
• Removed includeTraceSpans field from the entire notification system (API, UI, types, and background tasks)
• Added preview routes for testing notification email templates in various states
• Properly handles type conversion from canonical RateLimitStatus (with Date) to EmailRateLimitStatus (with string) for JSON serialization

The consolidation follows established email component patterns and maintains backward compatibility for existing notification configurations.

Confidence Score: 5/5

• This PR is safe to merge with no identified issues
• The refactoring is well-structured and thorough, consolidating email rendering logic into a shared component while properly handling type conversions. All changes are consistent across API routes, UI, types, and background tasks. The removal of trace spans is cleanly implemented throughout the codebase.
• No files require special attention

Important Files Changed

Filename	Overview
apps/sim/components/emails/notifications/workflow-notification-email.tsx	Created new email component for workflow notifications with proper styling and type definitions for serialized data
apps/sim/background/workspace-notification-delivery.ts	Refactored to use shared email template, removed trace spans feature, simplified email rendering logic
apps/sim/components/emails/render.ts	Added renderWorkflowNotificationEmail function to consolidate email rendering utilities
apps/sim/app/api/workspaces/[id]/notifications/route.ts	Removed includeTraceSpans field from API schema and responses
apps/sim/app/workspace/[workspaceId]/logs/components/logs-toolbar/components/notifications/notifications.tsx	Removed trace spans option from notification settings UI

Sequence Diagram

sequenceDiagram
    participant UI as Notification UI
    participant API as API Routes
    participant BG as Background Task
    participant Email as Email Components
    participant Mailer as Email Mailer

    Note over UI,API: Configuration Phase
    UI->>API: POST /notifications (without includeTraceSpans)
    API->>API: Store subscription config
    API-->>UI: Return subscription

    Note over BG,Mailer: Notification Delivery Phase
    BG->>BG: Build payload (no trace spans)
    BG->>BG: Add rate limits if enabled
    BG->>BG: Add usage data if enabled
    BG->>Email: renderWorkflowNotificationEmail(props)
    Email->>Email: WorkflowNotificationEmail component
    Email->>Email: Apply baseStyles.codeBlock
    Email-->>BG: Return HTML string
    BG->>Mailer: sendEmail(html, text)
    Mailer-->>BG: Delivery result

Loading

[greptile-apps]

_{11 files reviewed, 5 comments}

_{Edit Code Review Agent Settings | Greptile}

[waleedlatif1]

@greptile