rails_error_dashboard 0.1.15 → 0.1.17

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: da18321511a53167fa248ae72974c6c15807a725396aabd27af1059296ecbc46
-  data.tar.gz: 84f348d386b6b07375797275dedf97e5a9504522270447bebd6b56307f5e4a7d
+  metadata.gz: 66993309911427e7fdbddadebfea4a8603fa851ce47c7d5ac6189510c741965b
+  data.tar.gz: 86d88a89e817adcb790a9b83791decfaa13ffeac4d1b744b846d072e6610ebb5
 SHA512:
-  metadata.gz: b5b3d045f7a0b7bcae61823aa57e373111b6dab59e5a651ae5a33df9a3f043598e6469195b9cb508e79ad6bda62182877572888dba3cea37a4039715eee79349
-  data.tar.gz: 460491a0614445e91f098dbb792d86d9d7c8616f0456ad88cd6b14fdf27aba4d87990b080bd6c289eafe03befccceec94b6b23dcce29d850fd05b3bda946d39d
+  metadata.gz: 6581ea39ea6340b0df0dcb9835c597367f5c37459462f3ebe6f79315f99771ade90770fd04605477edef545b75aef32ae6d04acffe4caf896390b219ae11380e
+  data.tar.gz: 1dce7070b16addd615deb960d93e3226a448cb776526a17d1f353a1fdd8d25e90c8a2916bba21482b23637bcebf5a34c153b9227113ea6a23eeba79891f6acd7

data/app/controllers/rails_error_dashboard/application_controller.rb

@@ -2,6 +2,12 @@ module RailsErrorDashboard
   class ApplicationController < ActionController::Base
     include Pagy::Backend
 
+    # Enable features that are disabled in API-only mode
+    # These are ONLY enabled for Error Dashboard routes, not the entire app
+    include ActionController::Cookies
+    include ActionController::Flash
+    include ActionController::RequestForgeryProtection
+
     layout "rails_error_dashboard"
 
     protect_from_forgery with: :exception

data/app/models/rails_error_dashboard/error_log.rb

@@ -518,7 +518,9 @@ module RailsErrorDashboard
 
     # Turbo Stream broadcasting methods
     def broadcast_new_error
+      # Skip broadcasting in API-only mode or if Turbo is not available
       return unless defined?(Turbo)
+      return unless broadcast_available?
 
       platforms = ErrorLog.distinct.pluck(:platform).compact
       show_platform = platforms.size > 1
@@ -531,11 +533,14 @@ module RailsErrorDashboard
       )
       broadcast_replace_stats
     rescue => e
-      Rails.logger.error("Failed to broadcast new error: #{e.message}")
+      Rails.logger.error("[RailsErrorDashboard] Failed to broadcast new error: #{e.class} - #{e.message}")
+      Rails.logger.debug("[RailsErrorDashboard] Backtrace: #{e.backtrace&.first(3)&.join("\n")}")
     end
 
     def broadcast_error_update
+      # Skip broadcasting in API-only mode or if Turbo is not available
       return unless defined?(Turbo)
+      return unless broadcast_available?
 
       platforms = ErrorLog.distinct.pluck(:platform).compact
       show_platform = platforms.size > 1
@@ -548,13 +553,19 @@ module RailsErrorDashboard
       )
       broadcast_replace_stats
     rescue => e
-      Rails.logger.error("Failed to broadcast error update: #{e.message}")
+      Rails.logger.error("[RailsErrorDashboard] Failed to broadcast error update: #{e.class} - #{e.message}")
+      Rails.logger.debug("[RailsErrorDashboard] Backtrace: #{e.backtrace&.first(3)&.join("\n")}")
     end
 
     def broadcast_replace_stats
+      # Skip broadcasting in API-only mode or if Turbo is not available
       return unless defined?(Turbo)
+      return unless broadcast_available?
 
       stats = Queries::DashboardStats.call
+      # Safety check: ensure stats is not nil before broadcasting
+      return unless stats.is_a?(Hash) && stats.present?
+
       Turbo::StreamsChannel.broadcast_replace_to(
         "error_list",
         target: "dashboard_stats",
@@ -562,7 +573,26 @@ module RailsErrorDashboard
         locals: { stats: stats }
       )
     rescue => e
-      Rails.logger.error("Failed to broadcast stats update: #{e.message}")
+      Rails.logger.error("[RailsErrorDashboard] Failed to broadcast stats update: #{e.class} - #{e.message}")
+      Rails.logger.debug("[RailsErrorDashboard] Backtrace: #{e.backtrace&.first(3)&.join("\n")}")
+    end
+
+    # Check if broadcast functionality is available and properly configured
+    # In API-only apps, ActionCable might not be configured or Rails.cache might not be available
+    def broadcast_available?
+      # Check if ActionCable is available (required for Turbo Streams)
+      return false unless defined?(ActionCable)
+
+      # Check if Rails.cache is configured and working
+      # This prevents errors when cache is not available in API-only mode
+      begin
+        Rails.cache.write("rails_error_dashboard_broadcast_test", true, expires_in: 1.second)
+        Rails.cache.delete("rails_error_dashboard_broadcast_test")
+        true
+      rescue => e
+        Rails.logger.debug("[RailsErrorDashboard] Broadcast not available: #{e.message}")
+        false
+      end
     end
 
     # Enhanced Metrics: Release/Version Tracking

data/app/views/layouts/rails_error_dashboard.html.erb

@@ -3,8 +3,12 @@
   <head>
     <title><%= content_for?(:page_title) ? "#{content_for(:page_title)} | " : "" %><%= Rails.application.class.module_parent_name %> - Error Dashboard</title>
     <meta name="viewport" content="width=device-width,initial-scale=1">
-    <%= csrf_meta_tags %>
-    <%= csp_meta_tag %>
+    <% if respond_to?(:csrf_meta_tags) %>
+      <%= csrf_meta_tags %>
+    <% end %>
+    <% if respond_to?(:csp_meta_tag) %>
+      <%= csp_meta_tag %>
+    <% end %>
 
     <!-- Bootstrap CSS -->
     <link href="https://cdn.jsdelivr.net/npm/bootstrap@5.3.0/dist/css/bootstrap.min.css" rel="stylesheet">
@@ -1170,17 +1174,19 @@
         };
 
         // Show flash messages as toasts
-        <% if flash[:notice] %>
-          showToast('<%= j flash[:notice] %>', 'success');
-        <% end %>
-        <% if flash[:alert] %>
-          showToast('<%= j flash[:alert] %>', 'danger');
-        <% end %>
-        <% if flash[:success] %>
-          showToast('<%= j flash[:success] %>', 'success');
-        <% end %>
-        <% if flash[:error] %>
-          showToast('<%= j flash[:error] %>', 'danger');
+        <% if defined?(flash) && flash.present? %>
+          <% if flash[:notice] %>
+            showToast('<%= j flash[:notice] %>', 'success');
+          <% end %>
+          <% if flash[:alert] %>
+            showToast('<%= j flash[:alert] %>', 'danger');
+          <% end %>
+          <% if flash[:success] %>
+            showToast('<%= j flash[:success] %>', 'success');
+          <% end %>
+          <% if flash[:error] %>
+            showToast('<%= j flash[:error] %>', 'danger');
+          <% end %>
         <% end %>
       });
     </script>

data/app/views/rails_error_dashboard/errors/analytics.html.erb

@@ -131,14 +131,29 @@
             <script>
               document.addEventListener('DOMContentLoaded', function() {
                 const colors = window.getChartColors();
+                // Distinct, accessible colors for 5 platforms with good contrast
+                const platformColors = [
+                  "#2563EB", // Blue - API
+                  "#10B981", // Green - Android
+                  "#F59E0B", // Amber - Background Jobs
+                  "#8B5CF6", // Purple - Web
+                  "#EF4444"  // Red - iOS
+                ];
+
                 new Chartkick.PieChart("errors-by-platform-chart", <%= raw @errors_by_platform.to_json %>, {
-                  colors: ["#000000", "#3DDC84", "#3B82F6"],
+                  colors: platformColors,
                   height: "300px",
                   legend: "bottom",
                   donut: true,
                   library: {
                     plugins: {
-                      legend: { labels: { color: colors.textColor } }
+                      legend: {
+                        labels: {
+                          color: colors.textColor,
+                          font: { size: 12, weight: 'bold' }
+                        }
+                      }
+                      // Tooltip uses global theme-aware defaults from layout
                     }
                   }
                 });

data/app/views/rails_error_dashboard/errors/platform_comparison.html.erb

@@ -291,6 +291,19 @@
 </div>
 
 <script>
+  // Platform color mapping - consistent with analytics page
+  const platformColorMap = {
+    'api': '#2563EB',           // Blue
+    'android': '#10B981',       // Green
+    'background_jobs': '#F59E0B', // Amber
+    'web': '#8B5CF6',           // Purple
+    'ios': '#EF4444'            // Red
+  };
+
+  function getPlatformColor(platform) {
+    return platformColorMap[platform.toLowerCase()] || '#6B7280'; // Gray fallback
+  }
+
   // Error Rate Chart
   const errorRateCtx = document.getElementById('errorRateChart');
   if (errorRateCtx) {
@@ -301,16 +314,8 @@
         datasets: [{
           label: 'Total Errors',
           data: <%= raw @error_rate_by_platform.values.to_json %>,
-          backgroundColor: (function() {
-            const isDark = document.body.classList.contains('dark-mode');
-            const config = getCatppuccinChartConfig(isDark);
-            return config.colors.blue.replace('rgb', 'rgba').replace(')', ', 0.5)');
-          })(),
-          borderColor: (function() {
-            const isDark = document.body.classList.contains('dark-mode');
-            const config = getCatppuccinChartConfig(isDark);
-            return config.colors.blue;
-          })(),
+          backgroundColor: 'rgba(37, 99, 235, 0.5)', // Blue with transparency
+          borderColor: '#2563EB', // Blue
           borderWidth: 1
         }]
       },
@@ -330,9 +335,6 @@
   // Daily Trend Chart
   const dailyTrendCtx = document.getElementById('dailyTrendChart');
   if (dailyTrendCtx) {
-    // Use Catppuccin colors from global theme config
-    const isDark = document.body.classList.contains('dark-mode');
-
     const datasets = <%= raw @daily_trends.map { |platform, data|
       {
         label: platform.to_s.capitalize,
@@ -342,12 +344,12 @@
       }
     }.to_json %>;
 
-    // Apply Catppuccin platform colors
+    // Apply platform-specific colors
     datasets.forEach(ds => {
-      const color = getPlatformColor(ds.platform, isDark);
+      const color = getPlatformColor(ds.platform);
       ds.borderColor = color;
       // Add transparency for background
-      ds.backgroundColor = color.replace('rgb', 'rgba').replace(')', ', 0.1)');
+      ds.backgroundColor = color + '1A'; // Add 10% alpha in hex
     });
 
     new Chart(dailyTrendCtx, {
@@ -382,16 +384,8 @@
         datasets: [{
           label: 'Hours to Resolve',
           data: times,
-          backgroundColor: (function() {
-            const isDark = document.body.classList.contains('dark-mode');
-            const config = getCatppuccinChartConfig(isDark);
-            return config.colors.orange.replace('rgb', 'rgba').replace(')', ', 0.5)');
-          })(),
-          borderColor: (function() {
-            const isDark = document.body.classList.contains('dark-mode');
-            const config = getCatppuccinChartConfig(isDark);
-            return config.colors.orange;
-          })(),
+          backgroundColor: 'rgba(245, 158, 11, 0.5)', // Amber with transparency
+          borderColor: '#F59E0B', // Amber
           borderWidth: 1
         }]
       },

data/app/views/rails_error_dashboard/errors/show.html.erb

@@ -30,9 +30,6 @@
       <button type="button" class="btn btn-outline-secondary" onclick="downloadErrorJSON()" title="Download error details as JSON">
         <i class="bi bi-download"></i> Export JSON
       </button>
-      <button type="button" class="btn btn-outline-primary" onclick="copyErrorURL()" title="Copy shareable link to this error">
-        <i class="bi bi-share"></i> Share
-      </button>
       <% if @error.resolved? %>
         <span class="badge bg-success fs-6">
           <i class="bi bi-check-circle"></i> Resolved
@@ -46,26 +43,6 @@
   </div>
 
   <script>
-    function copyErrorURL() {
-      const url = window.location.href;
-      navigator.clipboard.writeText(url).then(() => {
-        // Change button text temporarily
-        const button = event.currentTarget;
-        const originalHTML = button.innerHTML;
-        button.innerHTML = '<i class="bi bi-check"></i> Copied!';
-        button.classList.remove('btn-outline-primary');
-        button.classList.add('btn-success');
-
-        setTimeout(() => {
-          button.innerHTML = originalHTML;
-          button.classList.remove('btn-success');
-          button.classList.add('btn-outline-primary');
-        }, 2000);
-      }).catch(err => {
-        alert('Failed to copy URL: ' + err);
-      });
-    }
-
     function downloadErrorJSON() {
       const errorData = {
         id: <%= @error.id %>,

data/lib/rails_error_dashboard/engine.rb

@@ -4,6 +4,15 @@ module RailsErrorDashboard
 
     # Initialize the engine
     initializer "rails_error_dashboard.middleware" do |app|
+      # Enable Flash middleware for Error Dashboard routes in API-only apps
+      # This ensures flash messages work even when config.api_only = true
+      if app.config.api_only
+        # Insert Flash middleware ONLY for Error Dashboard routes
+        app.middleware.use ActionDispatch::Flash
+        app.middleware.use ActionDispatch::Cookies
+        app.middleware.use ActionDispatch::Session::CookieStore
+      end
+
       # Add error catching middleware if enabled
       if RailsErrorDashboard.configuration.enable_middleware
         app.config.middleware.insert_before 0, RailsErrorDashboard::Middleware::ErrorCatcher

data/lib/rails_error_dashboard/queries/dashboard_stats.rb

@@ -12,28 +12,57 @@ module RailsErrorDashboard
       def call
         # Cache dashboard stats for 1 minute to reduce database load
         # Dashboard is viewed frequently, so short cache prevents stale data
-        Rails.cache.fetch(cache_key, expires_in: 1.minute) do
+        begin
+          Rails.cache.fetch(cache_key, expires_in: 1.minute) do
+            {
+              total_today: ErrorLog.where("occurred_at >= ?", Time.current.beginning_of_day).count,
+              total_week: ErrorLog.where("occurred_at >= ?", 7.days.ago).count,
+              total_month: ErrorLog.where("occurred_at >= ?", 30.days.ago).count,
+              unresolved: ErrorLog.unresolved.count,
+              resolved: ErrorLog.resolved.count,
+              by_platform: ErrorLog.group(:platform).count,
+              top_errors: top_errors,
+              #  Trend visualizations
+              errors_trend_7d: errors_trend_7d,
+              errors_by_severity_7d: errors_by_severity_7d,
+              spike_detected: spike_detected?,
+              spike_info: spike_info,
+              # New metrics for Overview dashboard
+              error_rate: error_rate,
+              affected_users_today: affected_users_today,
+              affected_users_yesterday: affected_users_yesterday,
+              affected_users_change: affected_users_change,
+              trend_percentage: trend_percentage,
+              trend_direction: trend_direction,
+              top_errors_by_impact: top_errors_by_impact
+            }
+          end
+        rescue => e
+          # If Rails.cache or any stats query fails, return empty stats hash
+          # This prevents broadcast failures in API-only mode or when cache is unavailable
+          Rails.logger.error("[RailsErrorDashboard] DashboardStats failed: #{e.class} - #{e.message}")
+          Rails.logger.debug("[RailsErrorDashboard] Backtrace: #{e.backtrace&.first(3)&.join("\n")}")
+
+          # Return minimal stats hash to prevent nil errors in views
           {
-            total_today: ErrorLog.where("occurred_at >= ?", Time.current.beginning_of_day).count,
-            total_week: ErrorLog.where("occurred_at >= ?", 7.days.ago).count,
-            total_month: ErrorLog.where("occurred_at >= ?", 30.days.ago).count,
-            unresolved: ErrorLog.unresolved.count,
-            resolved: ErrorLog.resolved.count,
-            by_platform: ErrorLog.group(:platform).count,
-            top_errors: top_errors,
-            #  Trend visualizations
-            errors_trend_7d: errors_trend_7d,
-            errors_by_severity_7d: errors_by_severity_7d,
-            spike_detected: spike_detected?,
-            spike_info: spike_info,
-            # New metrics for Overview dashboard
-            error_rate: error_rate,
-            affected_users_today: affected_users_today,
-            affected_users_yesterday: affected_users_yesterday,
-            affected_users_change: affected_users_change,
-            trend_percentage: trend_percentage,
-            trend_direction: trend_direction,
-            top_errors_by_impact: top_errors_by_impact
+            total_today: 0,
+            total_week: 0,
+            total_month: 0,
+            unresolved: 0,
+            resolved: 0,
+            by_platform: {},
+            top_errors: {},
+            errors_trend_7d: {},
+            errors_by_severity_7d: { critical: 0, high: 0, medium: 0, low: 0 },
+            spike_detected: false,
+            spike_info: nil,
+            error_rate: 0.0,
+            affected_users_today: 0,
+            affected_users_yesterday: 0,
+            affected_users_change: 0,
+            trend_percentage: 0.0,
+            trend_direction: :stable,
+            top_errors_by_impact: []
           }
         end
       end

data/lib/rails_error_dashboard/value_objects/error_context.rb

@@ -46,7 +46,16 @@ module RailsErrorDashboard
       end
 
       def build_request_url
-        return @context[:request]&.fullpath if @context[:request]
+        # Handle both full Rails requests and API-only requests
+        if @context[:request]
+          begin
+            return @context[:request].fullpath
+          rescue NoMethodError
+            # Fallback for minimal request objects
+            return @context[:request].path rescue nil
+          end
+        end
+
         return @context[:request_url] if @context[:request_url]
         return "Background Job: #{@context[:job]&.class}" if @context[:job]
         return "Sidekiq: #{@context[:job_class]}" if @context[:job_class]
@@ -114,10 +123,17 @@ module RailsErrorDashboard
       def detect_platform
         # Check if it's from a mobile request
         user_agent = extract_user_agent
-        return Services::PlatformDetector.detect(user_agent) if @context[:request]
 
-        # Everything else is API/backend
-        "API"
+        return "API" unless user_agent.present? && @context[:request]
+
+        # Only detect platform if we have a valid user agent
+        begin
+          Services::PlatformDetector.detect(user_agent)
+        rescue => e
+          # Fallback to API if platform detection fails
+          Rails.logger.debug("[RailsErrorDashboard] Platform detection failed: #{e.message}")
+          "API"
+        end
       end
 
       def extract_controller_name
@@ -161,8 +177,8 @@ module RailsErrorDashboard
       end
 
       def extract_session_id
-        # From Rails session
-        return @context[:request]&.session&.id if @context[:request]&.session
+        # Session is only available in full Rails mode, not API-only
+        return @context[:request]&.session&.id if @context[:request]&.respond_to?(:session) && @context[:request]&.session
 
         # From explicit context
         return @context[:session_id] if @context[:session_id]

data/lib/rails_error_dashboard/version.rb

@@ -1,3 +1,3 @@
 module RailsErrorDashboard
-  VERSION = "0.1.15"
+  VERSION = "0.1.17"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails_error_dashboard
 version: !ruby/object:Gem::Version
-  version: 0.1.15
+  version: 0.1.17
 platform: ruby
 authors:
 - Anjan Jagirdar
@@ -379,7 +379,7 @@ metadata:
   source_code_uri: https://github.com/AnjanJ/rails_error_dashboard
   changelog_uri: https://github.com/AnjanJ/rails_error_dashboard/blob/main/CHANGELOG.md
 post_install_message: "━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\n
-  \ Rails Error Dashboard v0.1.15\n━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\n\n\U0001F195
+  \ Rails Error Dashboard v0.1.17\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