rails_error_dashboard 0.5.10 → 0.5.12

data/lib/rails_error_dashboard/services/digest_builder.rb

@@ -0,0 +1,158 @@
+# frozen_string_literal: true
+
+module RailsErrorDashboard
+  module Services
+    # Pure algorithm: Build a digest summary of error activity for a time period.
+    # Aggregates stats from existing queries into a single hash suitable for email templates.
+    #
+    # @example
+    #   DigestBuilder.call(period: :daily)
+    #   # => { period: :daily, stats: { new_errors: 12, ... }, top_errors: [...], ... }
+    class DigestBuilder
+      PERIODS = {
+        daily: { days: 1, label: "Last 24 hours" },
+        weekly: { days: 7, label: "Last 7 days" }
+      }.freeze
+
+      def self.call(period: :daily, application_id: nil)
+        new(period: period, application_id: application_id).call
+      end
+
+      def initialize(period: :daily, application_id: nil)
+        @period = PERIODS.key?(period) ? period : :daily
+        @days = PERIODS[@period][:days]
+        @application_id = application_id
+        @start_date = @days.days.ago
+      end
+
+      def call
+        {
+          period: @period,
+          period_label: PERIODS[@period][:label],
+          generated_at: Time.current,
+          stats: build_stats,
+          top_errors: top_errors,
+          critical_unresolved: critical_unresolved,
+          comparison: build_comparison
+        }
+      rescue => e
+        Rails.logger.error("[RailsErrorDashboard] DigestBuilder failed: #{e.class}: #{e.message}")
+        empty_result
+      end
+
+      private
+
+      def base_scope
+        scope = ErrorLog.where("occurred_at >= ?", @start_date)
+        scope = scope.where(application_id: @application_id) if @application_id.present?
+        scope
+      end
+
+      def build_stats
+        scope = base_scope
+
+        new_errors = scope.where("occurrence_count <= 1").count
+        total_occurrences = scope.sum(:occurrence_count)
+        resolved = scope.where(resolved: true).count
+        unresolved = scope.where(resolved: false).count
+        # Severity is computed from error_type via SeverityClassifier (not a DB column).
+        # Count critical+high by matching known error type patterns via SQL WHERE IN.
+        critical_types = Services::SeverityClassifier::CRITICAL_ERROR_TYPES +
+                         Services::SeverityClassifier::HIGH_SEVERITY_ERROR_TYPES
+        critical_high = scope.where(error_type: critical_types).count
+
+        total = resolved + unresolved
+        resolution_rate = total > 0 ? (resolved.to_f / total * 100).round(1) : 0
+
+        {
+          new_errors: new_errors,
+          total_occurrences: total_occurrences,
+          resolved: resolved,
+          unresolved: unresolved,
+          critical_high: critical_high,
+          resolution_rate: resolution_rate
+        }
+      rescue => e
+        Rails.logger.error("[RailsErrorDashboard] DigestBuilder.build_stats failed: #{e.class}: #{e.message}")
+        { new_errors: 0, total_occurrences: 0, resolved: 0, unresolved: 0, critical_high: 0, resolution_rate: 0 }
+      end
+
+      def top_errors
+        base_scope
+          .where(resolved: false)
+          .group(:error_type)
+          .order("count_all DESC")
+          .limit(5)
+          .count
+          .map do |error_type, count|
+            sample = base_scope.where(error_type: error_type).order(occurred_at: :desc).first
+            {
+              error_type: error_type,
+              message: sample&.message.to_s.truncate(100),
+              count: count,
+              id: sample&.id
+            }
+          end
+      rescue => e
+        Rails.logger.error("[RailsErrorDashboard] DigestBuilder.top_errors failed: #{e.class}: #{e.message}")
+        []
+      end
+
+      def critical_unresolved
+        critical_types = Services::SeverityClassifier::CRITICAL_ERROR_TYPES +
+                         Services::SeverityClassifier::HIGH_SEVERITY_ERROR_TYPES
+        base_scope
+          .where(resolved: false)
+          .where(error_type: critical_types)
+          .order(occurred_at: :desc)
+          .limit(5)
+          .map do |error|
+            {
+              error_type: error.error_type,
+              message: error.message.to_s.truncate(100),
+              severity: error.severity,
+              id: error.id
+            }
+          end
+      rescue => e
+        Rails.logger.error("[RailsErrorDashboard] DigestBuilder.critical_unresolved failed: #{e.class}: #{e.message}")
+        []
+      end
+
+      def build_comparison
+        previous_start = (@days * 2).days.ago
+        previous_end = @start_date
+
+        current_count = base_scope.count
+        previous_scope = ErrorLog.where("occurred_at >= ? AND occurred_at < ?", previous_start, previous_end)
+        previous_scope = previous_scope.where(application_id: @application_id) if @application_id.present?
+        previous_count = previous_scope.count
+
+        delta = current_count - previous_count
+        percentage = previous_count > 0 ? ((delta.to_f / previous_count) * 100).round(1) : nil
+
+        {
+          current_count: current_count,
+          previous_count: previous_count,
+          error_delta: delta,
+          error_delta_percentage: percentage
+        }
+      rescue => e
+        Rails.logger.error("[RailsErrorDashboard] DigestBuilder.build_comparison failed: #{e.class}: #{e.message}")
+        { current_count: 0, previous_count: 0, error_delta: 0, error_delta_percentage: nil }
+      end
+
+      def empty_result
+        {
+          period: @period,
+          period_label: PERIODS.dig(@period, :label) || "Unknown",
+          generated_at: Time.current,
+          stats: { new_errors: 0, total_occurrences: 0, resolved: 0, unresolved: 0, critical_high: 0, resolution_rate: 0 },
+          top_errors: [],
+          critical_unresolved: [],
+          comparison: { current_count: 0, previous_count: 0, error_delta: 0, error_delta_percentage: nil }
+        }
+      end
+    end
+  end
+end

data/lib/rails_error_dashboard/services/notification_helpers.rb

@@ -14,7 +14,8 @@ module RailsErrorDashboard
       # @return [String] Full URL to the error detail page
       def dashboard_url(error_log)
         base_url = RailsErrorDashboard.configuration.dashboard_base_url || "http://localhost:3000"
-        "#{base_url}/error_dashboard/errors/#{error_log.id}"
+        mount_path = RailsErrorDashboard.configuration.engine_mount_path
+        "#{base_url}#{mount_path}/errors/#{error_log.id}"
       end
 
       # Truncate a message to a maximum length

data/lib/rails_error_dashboard/subscribers/issue_tracker_subscriber.rb

@@ -13,7 +13,8 @@ module RailsErrorDashboard
         # Called when a new error is first logged
         def on_error_logged(error_log)
           return unless should_auto_create?(error_log)
-          CreateIssueJob.perform_later(error_log.id)
+          dashboard_url = Services::NotificationHelpers.dashboard_url(error_log)
+          CreateIssueJob.perform_later(error_log.id, dashboard_url: dashboard_url)
         rescue => e
           nil
         end

data/lib/rails_error_dashboard/value_objects/error_context.rb

@@ -110,6 +110,11 @@ module RailsErrorDashboard
         # Additional context (from mobile apps, etc.)
         params.merge!(@context[:additional_context]) if @context[:additional_context]
 
+        # Pre-serialized params (from async logging or double-ErrorContext path).
+        # LogError creates a second ErrorContext from error_context.to_h which
+        # has :request_params as a JSON string but no :request object.
+        return @context[:request_params] if params.empty? && @context[:request_params].present?
+
         params.to_json
       end
 

data/lib/rails_error_dashboard/version.rb

@@ -1,3 +1,3 @@
 module RailsErrorDashboard
-  VERSION = "0.5.10"
+  VERSION = "0.5.12"
 end

data/lib/rails_error_dashboard.rb

@@ -67,6 +67,9 @@ require "rails_error_dashboard/services/local_variable_capturer"
 require "rails_error_dashboard/services/swallowed_exception_tracker"
 require "rails_error_dashboard/services/crash_capture"
 require "rails_error_dashboard/services/diagnostic_dump_generator"
+require "rails_error_dashboard/services/coverage_tracker"
+require "rails_error_dashboard/services/digest_builder"
+require "rails_error_dashboard/queries/user_impact_summary"
 require "rails_error_dashboard/subscribers/breadcrumb_subscriber"
 require "rails_error_dashboard/subscribers/rack_attack_subscriber"
 require "rails_error_dashboard/subscribers/action_cable_subscriber"

data/lib/tasks/error_dashboard.rake

@@ -525,4 +525,27 @@ namespace :error_dashboard do
 
     puts "\n" + "=" * 80 + "\n"
   end
+
+  desc "Send error digest email (PERIOD=daily|weekly, APP_ID=optional)"
+  task send_digest: :environment do
+    period = ENV.fetch("PERIOD", "daily")
+    app_id = ENV["APP_ID"]
+
+    unless RailsErrorDashboard.configuration.enable_scheduled_digests
+      puts "Scheduled digests are not enabled. Set config.enable_scheduled_digests = true"
+      next
+    end
+
+    recipients = RailsErrorDashboard.configuration.digest_recipients ||
+                 RailsErrorDashboard.configuration.notification_email_recipients
+
+    if recipients.blank?
+      puts "No digest recipients configured. Set config.digest_recipients or config.notification_email_recipients"
+      next
+    end
+
+    puts "Sending #{period} error digest to #{recipients.join(', ')}..."
+    RailsErrorDashboard::ScheduledDigestJob.perform_later(period: period, application_id: app_id)
+    puts "Digest job enqueued."
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails_error_dashboard
 version: !ruby/object:Gem::Version
-  version: 0.5.10
+  version: 0.5.12
 platform: ruby
 authors:
 - Anjan Jagirdar
@@ -260,10 +260,12 @@ files:
 - app/jobs/rails_error_dashboard/pagerduty_error_notification_job.rb
 - app/jobs/rails_error_dashboard/reopen_linked_issue_job.rb
 - app/jobs/rails_error_dashboard/retention_cleanup_job.rb
+- app/jobs/rails_error_dashboard/scheduled_digest_job.rb
 - app/jobs/rails_error_dashboard/slack_error_notification_job.rb
 - app/jobs/rails_error_dashboard/swallowed_exception_flush_job.rb
 - app/jobs/rails_error_dashboard/webhook_error_notification_job.rb
 - app/mailers/rails_error_dashboard/application_mailer.rb
+- app/mailers/rails_error_dashboard/digest_mailer.rb
 - app/mailers/rails_error_dashboard/error_notification_mailer.rb
 - app/models/rails_error_dashboard/application.rb
 - app/models/rails_error_dashboard/cascade_pattern.rb
@@ -275,6 +277,8 @@ files:
 - app/models/rails_error_dashboard/error_occurrence.rb
 - app/models/rails_error_dashboard/swallowed_exception.rb
 - app/views/layouts/rails_error_dashboard.html.erb
+- app/views/rails_error_dashboard/digest_mailer/digest_summary.html.erb
+- app/views/rails_error_dashboard/digest_mailer/digest_summary.text.erb
 - app/views/rails_error_dashboard/error_notification_mailer/error_alert.html.erb
 - app/views/rails_error_dashboard/error_notification_mailer/error_alert.text.erb
 - app/views/rails_error_dashboard/errors/_breadcrumbs_group.html.erb
@@ -315,6 +319,7 @@ files:
 - app/views/rails_error_dashboard/errors/settings/_value_badge.html.erb
 - app/views/rails_error_dashboard/errors/show.html.erb
 - app/views/rails_error_dashboard/errors/swallowed_exceptions.html.erb
+- app/views/rails_error_dashboard/errors/user_impact.html.erb
 - config/routes.rb
 - db/development.sqlite3
 - db/migrate/20251223000000_create_rails_error_dashboard_complete_schema.rb
@@ -418,6 +423,7 @@ files:
 - lib/rails_error_dashboard/queries/release_timeline.rb
 - lib/rails_error_dashboard/queries/similar_errors.rb
 - lib/rails_error_dashboard/queries/swallowed_exception_summary.rb
+- lib/rails_error_dashboard/queries/user_impact_summary.rb
 - lib/rails_error_dashboard/services/analytics_cache_manager.rb
 - lib/rails_error_dashboard/services/backtrace_parser.rb
 - lib/rails_error_dashboard/services/backtrace_processor.rb
@@ -429,10 +435,12 @@ files:
 - lib/rails_error_dashboard/services/cascade_detector.rb
 - lib/rails_error_dashboard/services/cause_chain_extractor.rb
 - lib/rails_error_dashboard/services/codeberg_issue_client.rb
+- lib/rails_error_dashboard/services/coverage_tracker.rb
 - lib/rails_error_dashboard/services/crash_capture.rb
 - lib/rails_error_dashboard/services/curl_generator.rb
 - lib/rails_error_dashboard/services/database_health_inspector.rb
 - lib/rails_error_dashboard/services/diagnostic_dump_generator.rb
+- lib/rails_error_dashboard/services/digest_builder.rb
 - lib/rails_error_dashboard/services/discord_payload_builder.rb
 - lib/rails_error_dashboard/services/environment_snapshot.rb
 - lib/rails_error_dashboard/services/error_broadcaster.rb
@@ -486,14 +494,30 @@ metadata:
   documentation_uri: https://AnjanJ.github.io/rails_error_dashboard
   bug_tracker_uri: https://github.com/AnjanJ/rails_error_dashboard/issues
   funding_uri: https://github.com/sponsors/AnjanJ
-post_install_message: "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\n
-  \ Rails Error Dashboard v0.5.10\n━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\n\n\U0001F195
-  First time? Quick start:\n   rails generate rails_error_dashboard:install\n   rails
-  db:migrate\n   # Add to config/routes.rb:\n   mount RailsErrorDashboard::Engine
-  => '/error_dashboard'\n\n\U0001F504 Upgrading from v0.1.x?\n   rails db:migrate\n
-  \  \U0001F4DD Changelog: https://github.com/AnjanJ/rails_error_dashboard/blob/main/CHANGELOG.md\n\n\U0001F3AE
-  Live demo: https://rails-error-dashboard.anjan.dev\n   (gandalf / youshallnotpass)\n\n\U0001F4D6
-  Full docs: https://github.com/AnjanJ/rails_error_dashboard\n━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\n"
+post_install_message: |
+  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+    RED (Rails Error Dashboard) v0.5.12
+  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+  First install:
+     rails generate rails_error_dashboard:install
+     rails db:migrate
+     # Route and config are set up automatically by the generator.
+
+  Upgrading from a previous version:
+     rails generate rails_error_dashboard:install
+     rails db:migrate
+     # The generator detects your existing config and only adds new migrations.
+
+  Separate database users:
+     rails generate rails_error_dashboard:install
+     rails db:migrate:error_dashboard
+     # See docs for full separate-DB setup.
+
+  Live demo: https://rails-error-dashboard.anjan.dev
+  Full docs:  https://github.com/AnjanJ/rails_error_dashboard
+  Changelog:  https://github.com/AnjanJ/rails_error_dashboard/blob/main/CHANGELOG.md
+  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 rdoc_options: []
 require_paths:
 - lib