rails_error_dashboard 0.5.15 → 0.6.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3569ff68dcb977b53c8565d17e3f95de2b28b9c3abc462c77c6877277f79d101
-  data.tar.gz: b27615768c8fc87bca5f55f0deee1d4f1ec5c9c7466956480180808779708fc9
+  metadata.gz: 8730984999d2ab508eeefe889bfd320e54cb9395c6baacbe87fef1a3aa20de7a
+  data.tar.gz: 6a301193d907d990418f6a74b83ee36a54caeac1b05dc57a75ea24213e9b7f7c
 SHA512:
-  metadata.gz: 85b4d1f4017fafc58181547f3fca6c87b19d72b4b19092462dd89cdfd35cc7d84cd2203244b5273db2e7901e09c54530a8ee53baf904eccbb016f7df64b3cfbe
-  data.tar.gz: f77c96cdd3d5e322025086c889417af20668bdd8cbf582fb93b512eecaed4971596db330a81fe7912536e6bace26dab34f9d88b31f88123a887f0b0468d0a1db
+  metadata.gz: af05b90ebe19d1b90adb82b7e8d3a4cd57faf695327c3a51c2d36ced221d1951265e4d88d77e4ddb843c465582fe14d6911141e3f081b2de76ca3037a4e637f0
+  data.tar.gz: add93453ac6ebee6a3f320fc336fe30c81a952f58fc30b3b8554c410071586da7d9c25da1d886c5976b4ade28cc6504c5b355509a29dcf5025aa8b502d1fff65

data/app/controllers/rails_error_dashboard/application_controller.rb

@@ -23,23 +23,27 @@ module RailsErrorDashboard
       Rails.logger.error("Params: #{params.inspect}")
       Rails.logger.error(exception.backtrace&.first(10)&.join("\n")) if exception.backtrace
 
-      # Render user-friendly error page
-      render plain: "The Error Dashboard encountered an issue displaying this page.\n\n" \
-                    "Your application is unaffected - this is only a dashboard display error.\n\n" \
-                    "Error: #{exception.message}\n\n" \
-                    "Check Rails logs for details: [RailsErrorDashboard]",
-             status: :internal_server_error,
-             layout: false
+      render_dashboard_error(
+        icon: "bi-exclamation-triangle",
+        icon_style: "background: var(--status-warning-bg); color: var(--status-warning);",
+        title: "Something went wrong",
+        message: "The Error Dashboard encountered an issue displaying this page. Your application is unaffected.",
+        detail: exception.message,
+        status: :internal_server_error
+      )
     end
 
     # Handle record not found — return 404 instead of 500
     rescue_from ActiveRecord::RecordNotFound do |exception|
       Rails.logger.warn("[RailsErrorDashboard] Record not found: #{exception.message}")
-      render plain: "The requested error was not found.\n\n" \
-                    "It may have been deleted or the ID is invalid.\n\n" \
-                    "Error: #{exception.message}",
-             status: :not_found,
-             layout: false
+
+      render_dashboard_error(
+        icon: "bi-search",
+        title: "The requested error was not found",
+        message: "It may have been deleted or the ID is invalid.",
+        detail: exception.message,
+        status: :not_found
+      )
     end
 
     # Handle Pagy pagination errors — redirect to page 1
@@ -47,5 +51,26 @@ module RailsErrorDashboard
       Rails.logger.warn("[RailsErrorDashboard] Pagination error: #{exception.message}")
       redirect_to request.path, status: :moved_permanently
     end
+
+    private
+
+    def render_dashboard_error(icon:, title:, message:, detail: nil, icon_style: nil, status: :internal_server_error)
+      set_common_view_variables
+      error_html = <<~ERB
+        <div class="red-empty-state" style="margin-top: var(--space-6);">
+          <div class="red-empty-state-icon"#{icon_style ? " style=\"#{icon_style}\"" : ""}><i class="bi #{icon}"></i></div>
+          <div class="red-empty-state-title">#{ERB::Util.html_escape(title)}</div>
+          <div class="red-empty-state-message">#{ERB::Util.html_escape(message)}</div>
+          #{"<div style=\"font-size: 12px; color: var(--text-tertiary); margin-top: var(--space-2); font-family: var(--font-mono);\">" + ERB::Util.html_escape(detail) + "</div>" if detail}
+          <a href="#{errors_path}" class="red-empty-state-cta" style="margin-top: var(--space-4);"><i class="bi bi-arrow-left"></i> Back to errors</a>
+        </div>
+      ERB
+      render html: error_html.html_safe, status: status, layout: "rails_error_dashboard"
+    end
+
+    def set_common_view_variables
+      @applications = Application.ordered_by_name.pluck(:name, :id) rescue []
+      @default_credentials_warning = RailsErrorDashboard.configuration.default_credentials? rescue false
+    end
   end
 end

data/app/controllers/rails_error_dashboard/errors_controller.rb

@@ -98,49 +98,49 @@ module RailsErrorDashboard
         resolution_reference: params[:resolution_reference]
       )
 
-      redirect_to error_path(@error)
+      redirect_to error_path(@error, **app_context_params)
     end
 
     # Phase 3: Workflow Integration Actions (via Commands)
 
     def assign
       @error = Commands::AssignError.call(params[:id], assigned_to: params[:assigned_to])
-      redirect_to error_path(@error)
+      redirect_to error_path(@error, **app_context_params)
     end
 
     def unassign
       @error = Commands::UnassignError.call(params[:id])
-      redirect_to error_path(@error)
+      redirect_to error_path(@error, **app_context_params)
     end
 
     def update_priority
       @error = Commands::UpdateErrorPriority.call(params[:id], priority_level: params[:priority_level])
-      redirect_to error_path(@error)
+      redirect_to error_path(@error, **app_context_params)
     end
 
     def snooze
       @error = Commands::SnoozeError.call(params[:id], hours: params[:hours].to_i, reason: params[:reason])
-      redirect_to error_path(@error)
+      redirect_to error_path(@error, **app_context_params)
     end
 
     def unsnooze
       @error = Commands::UnsnoozeError.call(params[:id])
-      redirect_to error_path(@error)
+      redirect_to error_path(@error, **app_context_params)
     end
 
     def mute
       @error = Commands::MuteError.call(params[:id], muted_by: params[:muted_by], reason: params[:reason])
-      redirect_to error_path(@error)
+      redirect_to error_path(@error, **app_context_params)
     end
 
     def unmute
       @error = Commands::UnmuteError.call(params[:id])
-      redirect_to error_path(@error)
+      redirect_to error_path(@error, **app_context_params)
     end
 
     def update_status
       result = Commands::UpdateErrorStatus.call(params[:id], status: params[:status], comment: params[:comment])
-      redirect_to error_path(result[:error])
+      redirect_to error_path(result[:error], **app_context_params)
     end
 
     def create_issue
@@ -153,7 +153,7 @@ module RailsErrorDashboard
       else
         flash[:alert] = "Failed to create issue: #{result[:error]}"
       end
-      redirect_to error_path(params[:id], anchor: "issue-tracking")
+      redirect_to error_path(params[:id], anchor: "issue-tracking", **app_context_params)
     end
 
     def link_issue
@@ -164,7 +164,7 @@ module RailsErrorDashboard
       else
         flash[:alert] = "Failed to link issue: #{result[:error]}"
       end
-      redirect_to error_path(params[:id], anchor: "issue-tracking")
+      redirect_to error_path(params[:id], anchor: "issue-tracking", **app_context_params)
     end
 
     def analytics
@@ -205,7 +205,7 @@ module RailsErrorDashboard
       # Check if feature is enabled
       unless RailsErrorDashboard.configuration.enable_platform_comparison
         flash[:alert] = "Platform Comparison is not enabled. Enable it in config/initializers/rails_error_dashboard.rb"
-        redirect_to errors_path
+        redirect_to errors_path(**app_context_params)
         return
       end
 
@@ -252,14 +252,14 @@ module RailsErrorDashboard
         flash[:alert] = "Batch operation failed: #{result[:errors].join(', ')}"
       end
 
-      redirect_to errors_path
+      redirect_to errors_path(**app_context_params)
     end
 
     def correlation
       # Check if feature is enabled
       unless RailsErrorDashboard.configuration.enable_error_correlation
         flash[:alert] = "Error Correlation is not enabled. Enable it in config/initializers/rails_error_dashboard.rb"
-        redirect_to errors_path
+        redirect_to errors_path(**app_context_params)
         return
       end
 
@@ -299,7 +299,7 @@ module RailsErrorDashboard
     def deprecations
       unless RailsErrorDashboard.configuration.enable_breadcrumbs
         flash[:alert] = "Breadcrumbs are not enabled. Enable them in config/initializers/rails_error_dashboard.rb"
-        redirect_to errors_path
+        redirect_to errors_path(**app_context_params)
         return
       end
 
@@ -319,7 +319,7 @@ module RailsErrorDashboard
     def n_plus_one_summary
       unless RailsErrorDashboard.configuration.enable_breadcrumbs
         flash[:alert] = "Breadcrumbs are not enabled. Enable them in config/initializers/rails_error_dashboard.rb"
-        redirect_to errors_path
+        redirect_to errors_path(**app_context_params)
         return
       end
 
@@ -339,7 +339,7 @@ module RailsErrorDashboard
     def cache_health_summary
       unless RailsErrorDashboard.configuration.enable_breadcrumbs
         flash[:alert] = "Breadcrumbs are not enabled. Enable them in config/initializers/rails_error_dashboard.rb"
-        redirect_to errors_path
+        redirect_to errors_path(**app_context_params)
         return
       end
 
@@ -360,7 +360,7 @@ module RailsErrorDashboard
     def job_health_summary
       unless RailsErrorDashboard.configuration.enable_system_health
         flash[:alert] = "System health is not enabled. Enable it in config/initializers/rails_error_dashboard.rb"
-        redirect_to errors_path
+        redirect_to errors_path(**app_context_params)
         return
       end
 
@@ -380,7 +380,7 @@ module RailsErrorDashboard
     def database_health_summary
       unless RailsErrorDashboard.configuration.enable_system_health
         flash[:alert] = "System health is not enabled. Enable it in config/initializers/rails_error_dashboard.rb"
-        redirect_to errors_path
+        redirect_to errors_path(**app_context_params)
         return
       end
 
@@ -416,7 +416,7 @@ module RailsErrorDashboard
         else
           flash[:alert] = "Swallowed exception detection is not enabled. Enable it in config/initializers/rails_error_dashboard.rb"
         end
-        redirect_to errors_path
+        redirect_to errors_path(**app_context_params)
         return
       end
 
@@ -437,7 +437,7 @@ module RailsErrorDashboard
       unless RailsErrorDashboard.configuration.enable_rack_attack_tracking &&
              RailsErrorDashboard.configuration.enable_breadcrumbs
         flash[:alert] = "Rack Attack tracking is not enabled. Enable enable_rack_attack_tracking and enable_breadcrumbs in config/initializers/rails_error_dashboard.rb"
-        redirect_to errors_path
+        redirect_to errors_path(**app_context_params)
         return
       end
 
@@ -458,7 +458,7 @@ module RailsErrorDashboard
       unless RailsErrorDashboard.configuration.enable_actioncable_tracking &&
              RailsErrorDashboard.configuration.enable_breadcrumbs
         flash[:alert] = "ActionCable tracking is not enabled. Enable enable_actioncable_tracking and enable_breadcrumbs in config/initializers/rails_error_dashboard.rb"
-        redirect_to errors_path
+        redirect_to errors_path(**app_context_params)
         return
       end
 
@@ -479,7 +479,7 @@ module RailsErrorDashboard
       unless RailsErrorDashboard.configuration.enable_activestorage_tracking &&
              RailsErrorDashboard.configuration.enable_breadcrumbs
         flash[:alert] = "ActiveStorage tracking is not enabled. Enable enable_activestorage_tracking and enable_breadcrumbs in config/initializers/rails_error_dashboard.rb"
-        redirect_to errors_path
+        redirect_to errors_path(**app_context_params)
         return
       end
 
@@ -499,7 +499,7 @@ module RailsErrorDashboard
     def diagnostic_dumps
       unless RailsErrorDashboard.configuration.enable_diagnostic_dump
         flash[:alert] = "Diagnostic dumps are not enabled. Enable them in config/initializers/rails_error_dashboard.rb"
-        redirect_to errors_path
+        redirect_to errors_path(**app_context_params)
         return
       end
 
@@ -513,7 +513,7 @@ module RailsErrorDashboard
     def create_diagnostic_dump
       unless RailsErrorDashboard.configuration.enable_diagnostic_dump
         flash[:alert] = "Diagnostic dumps are not enabled."
-        redirect_to errors_path
+        redirect_to errors_path(**app_context_params)
         return
       end
 
@@ -542,7 +542,7 @@ module RailsErrorDashboard
     def enable_coverage
       unless RailsErrorDashboard.configuration.enable_coverage_tracking
         flash[:alert] = "Coverage tracking is not enabled in configuration."
-        redirect_to errors_path
+        redirect_to errors_path(**app_context_params)
         return
       end
 
@@ -560,6 +560,23 @@ module RailsErrorDashboard
       redirect_back fallback_location: errors_path
     end
 
+    def test_error
+      exception = RailsErrorDashboard::TestError.new(
+        "[RED Test] This is a test error sent from the dashboard to verify " \
+        "that error capture and notification delivery are working correctly. " \
+        "It is safe to resolve or delete this error."
+      )
+      exception.set_backtrace(caller)
+
+      Commands::LogError.call(exception, { request: request, source: "dashboard.test_error" })
+
+      flash[:notice] = "Test error logged successfully. Check your notification channels (Slack, Discord, email, etc.) to confirm delivery."
+      redirect_to errors_path(**app_context_params)
+    rescue => e
+      flash[:alert] = "Failed to log test error: #{e.message}"
+      redirect_to settings_path(**app_context_params)
+    end
+
     def settings
       @config = RailsErrorDashboard.configuration
     end
@@ -595,6 +612,11 @@ module RailsErrorDashboard
       @applications = Application.ordered_by_name.pluck(:name, :id)
     end
 
+    # Preserves the application_id param across redirects
+    def app_context_params
+      @current_application_id.present? ? { application_id: @current_application_id } : {}
+    end
+
     def fetch_platform_issue(error)
       return nil unless error.external_issue_url.present? && error.external_issue_number.present?
       return nil unless RailsErrorDashboard.configuration.enable_issue_tracking

data/app/helpers/rails_error_dashboard/application_helper.rb

@@ -26,15 +26,15 @@ module RailsErrorDashboard
     def severity_color_var(severity)
       case severity&.to_sym
       when :critical
-        "var(--ctp-red)"
+        "var(--status-critical)"
       when :high
-        "var(--ctp-peach)"
+        "var(--status-warning)"
       when :medium
-        "var(--ctp-blue)"
+        "var(--status-info)"
       when :low
-        "var(--ctp-overlay1)"
+        "var(--text-tertiary)"
       else
-        "var(--ctp-overlay1)"
+        "var(--text-tertiary)"
       end
     end
 
@@ -74,6 +74,13 @@ module RailsErrorDashboard
       end
     end
 
+    # Returns the current application context param for preserving app selection across navigation.
+    # Use this in link helpers: errors_path(app_context) or error_path(error, **app_context)
+    # @return [Hash] { application_id: X } if an app is selected, empty hash otherwise
+    def app_context
+      params[:application_id].present? ? { application_id: params[:application_id] } : {}
+    end
+
     # Returns a sanitized hash of filter params safe for query links
     # @param extra_keys [Array<Symbol>] Additional permitted keys for specific contexts
     # @return [Hash] Whitelisted params for building URLs