rails_error_dashboard 0.1.22 → 0.1.23

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3b04467a3dfb4cb2500b3dd7cdcc8ab157e54dccbd9630f1333a2aae81e0d714
-  data.tar.gz: ffadf9eefd995500948a560f04c394adf1c005b892f7e4f23cd80ac5a7b9f4c8
+  metadata.gz: e8ebc7a75413a15752df2dc4a425f0c2dc046480e92d394d68212646360787e9
+  data.tar.gz: c4174fc14f0e8c9ca2da761bf7895f0ff5d3cf85d99470b31705dc748d9083f2
 SHA512:
-  metadata.gz: 46997287677b84461147c6ac344927a4e38d90f52da72de5069e48afe3072bc6b04d88936dc28e3c8b7c50fe47c065980286e8cbf98b3388756a8ae29a57b8f0
-  data.tar.gz: 93f7d23fe4bdf2a6fd5d62d488108e63423167b77af171e4ab42a9203c9d220010f26a34c339edda553efdcd083d54761c4d8bb0b303400b5c4a9c615775a2e3
+  metadata.gz: 33ab7875fcef261cc93f8af9ac7a7203e5b1c49a708baf247e63c7c2c9e40932d1fa21eb25c34f8b1f0d4e4f63dff3f1fdbb43f0f2b7b252575de6aaccd9cd08
+  data.tar.gz: 9213caf3f27b27b6a04e0dc198da8e45c7561734f6c5f14f8ccbaf150f17015b3e6693c05a14b4c73e47674ca6d12ae90f75097167e65ec80d47f03e7de9e507

data/app/models/rails_error_dashboard/application.rb

@@ -1,6 +1,6 @@
 module RailsErrorDashboard
   class Application < ActiveRecord::Base
-    self.table_name = 'rails_error_dashboard_applications'
+    self.table_name = "rails_error_dashboard_applications"
 
     # Associations
     has_many :error_logs, dependent: :restrict_with_error
@@ -12,10 +12,23 @@ module RailsErrorDashboard
     scope :ordered_by_name, -> { order(:name) }
 
     # Class method for finding or creating with caching
+    # Only caches successful finds, not creates (to avoid caching nil on creation failures)
     def self.find_or_create_by_name(name)
-      Rails.cache.fetch("error_dashboard/application/#{name}", expires_in: 1.hour) do
-        find_or_create_by!(name: name)
+      # Try to find in cache or database first
+      cached = Rails.cache.read("error_dashboard/application/#{name}")
+      return cached if cached
+
+      # Try to find existing
+      found = find_by(name: name)
+      if found
+        Rails.cache.write("error_dashboard/application/#{name}", found, expires_in: 1.hour)
+        return found
       end
+
+      # Create if not found (don't cache until successful)
+      created = create!(name: name)
+      Rails.cache.write("error_dashboard/application/#{name}", created, expires_in: 1.hour)
+      created
     end
 
     # Instance methods

data/lib/rails_error_dashboard/version.rb

@@ -1,3 +1,3 @@
 module RailsErrorDashboard
-  VERSION = "0.1.22"
+  VERSION = "0.1.23"
 end

data/lib/tasks/error_dashboard.rake

@@ -10,11 +10,11 @@ namespace :error_dashboard do
     # Use single SQL query with aggregates to avoid N+1 queries
     # This fetches all app data + error counts in one query instead of 6N queries
     apps = RailsErrorDashboard::Application
-             .select('rails_error_dashboard_applications.*')
-             .select('COUNT(rails_error_dashboard_error_logs.id) as total_errors')
-             .select('COALESCE(SUM(CASE WHEN NOT rails_error_dashboard_error_logs.resolved THEN 1 ELSE 0 END), 0) as unresolved_errors')
-             .joins('LEFT JOIN rails_error_dashboard_error_logs ON rails_error_dashboard_error_logs.application_id = rails_error_dashboard_applications.id')
-             .group('rails_error_dashboard_applications.id')
+             .select("rails_error_dashboard_applications.*")
+             .select("COUNT(rails_error_dashboard_error_logs.id) as total_errors")
+             .select("COALESCE(SUM(CASE WHEN NOT rails_error_dashboard_error_logs.resolved THEN 1 ELSE 0 END), 0) as unresolved_errors")
+             .joins("LEFT JOIN rails_error_dashboard_error_logs ON rails_error_dashboard_error_logs.application_id = rails_error_dashboard_applications.id")
+             .group("rails_error_dashboard_applications.id")
              .order(:name)
 
     if apps.empty?
@@ -27,9 +27,9 @@ namespace :error_dashboard do
     puts "\n#{apps.count} application(s) registered:\n\n"
 
     # Calculate column widths using aggregated data (no additional queries)
-    name_width = [apps.map(&:name).map(&:length).max, 20].max
-    total_width = [apps.map(&:total_errors).map(&:to_s).map(&:length).max, 5].max
-    unresolved_width = [apps.map(&:unresolved_errors).map(&:to_s).map(&:length).max, 10].max
+    name_width = [ apps.map(&:name).map(&:length).max, 20 ].max
+    total_width = [ apps.map(&:total_errors).map(&:to_s).map(&:length).max, 5 ].max
+    unresolved_width = [ apps.map(&:unresolved_errors).map(&:to_s).map(&:length).max, 10 ].max
 
     # Print header
     printf "%-#{name_width}s  %#{total_width}s  %#{unresolved_width}s  %s\n",
@@ -62,9 +62,9 @@ namespace :error_dashboard do
 
   desc "Backfill application_id for existing errors"
   task backfill_application: :environment do
-    app_name = ENV['APP_NAME'] || ENV['APPLICATION_NAME'] ||
+    app_name = ENV["APP_NAME"] || ENV["APPLICATION_NAME"] ||
                (defined?(Rails) && Rails.application.class.module_parent_name) ||
-               'Legacy Application'
+               "Legacy Application"
 
     puts "\n" + "=" * 80
     puts "RAILS ERROR DASHBOARD - BACKFILL APPLICATION"
@@ -94,7 +94,7 @@ namespace :error_dashboard do
     print "\nProceed with backfill? (y/N): "
     confirmation = $stdin.gets.chomp.downcase
 
-    unless confirmation == 'y' || confirmation == 'yes'
+    unless confirmation == "y" || confirmation == "yes"
       puts "\n✗ Backfill cancelled"
       puts "\n" + "=" * 80 + "\n"
       next
@@ -125,8 +125,8 @@ namespace :error_dashboard do
 
   desc "Show application statistics and health metrics"
   task app_stats: :environment do
-    app_id = ENV['APP_ID']
-    app_name = ENV['APP_NAME']
+    app_id = ENV["APP_ID"]
+    app_name = ENV["APP_NAME"]
 
     puts "\n" + "=" * 80
     puts "RAILS ERROR DASHBOARD - APPLICATION STATISTICS"
@@ -135,16 +135,16 @@ namespace :error_dashboard do
     # Find application
     app = if app_id
             RailsErrorDashboard::Application.find_by(id: app_id)
-          elsif app_name
+    elsif app_name
             RailsErrorDashboard::Application.find_by(name: app_name)
-          else
+    else
             puts "\n✗ Please specify APP_ID or APP_NAME"
             puts "\nExamples:"
             puts "  rails error_dashboard:app_stats APP_NAME='My App'"
             puts "  rails error_dashboard:app_stats APP_ID=1"
             puts "\n" + "=" * 80 + "\n"
             next
-          end
+    end
 
     unless app
       puts "\n✗ Application not found"
@@ -202,11 +202,11 @@ namespace :error_dashboard do
         time_ago = Time.current - error.occurred_at
         time_str = if time_ago < 3600
                      "#{(time_ago / 60).to_i}m ago"
-                   elsif time_ago < 86400
+        elsif time_ago < 86400
                      "#{(time_ago / 3600).to_i}h ago"
-                   else
+        else
                      "#{(time_ago / 86400).to_i}d ago"
-                   end
+        end
         puts "  • #{error.error_type} (#{time_str}) - #{error.message.truncate(50)}"
       end
     end
@@ -216,8 +216,8 @@ namespace :error_dashboard do
 
   desc "Clean up old resolved errors"
   task cleanup_resolved: :environment do
-    days = ENV['DAYS']&.to_i || 90
-    app_name = ENV['APP_NAME']
+    days = ENV["DAYS"]&.to_i || 90
+    app_name = ENV["APP_NAME"]
 
     puts "\n" + "=" * 80
     puts "RAILS ERROR DASHBOARD - CLEANUP RESOLVED ERRORS"
@@ -225,7 +225,7 @@ namespace :error_dashboard do
     puts "\nCleaning up resolved errors older than #{days} days"
 
     scope = RailsErrorDashboard::ErrorLog.resolved
-                                         .where('resolved_at < ?', days.days.ago)
+                                         .where("resolved_at < ?", days.days.ago)
 
     if app_name
       app = RailsErrorDashboard::Application.find_by(name: app_name)
@@ -252,7 +252,7 @@ namespace :error_dashboard do
     print "\nProceed with deletion? (y/N): "
     confirmation = $stdin.gets.chomp.downcase
 
-    unless confirmation == 'y' || confirmation == 'yes'
+    unless confirmation == "y" || confirmation == "yes"
       puts "\n✗ Cleanup cancelled"
       puts "\n" + "=" * 80 + "\n"
       next

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails_error_dashboard
 version: !ruby/object:Gem::Version
-  version: 0.1.22
+  version: 0.1.23
 platform: ruby
 authors:
 - Anjan Jagirdar
@@ -386,7 +386,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.22\n━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━\n\n\U0001F195
+  \ Rails Error Dashboard v0.1.23\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