rails-informant 0.2.1 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a3921be9c6e9f434524b03c140e94ca5de03128bbd9b27c9a9df67477ed79aa7
-  data.tar.gz: 90feef10dc4d4ada45acaefa06b65ea3cdc5543ba86beb5952f78b3167ec7948
+  metadata.gz: 13adbf148b2adf2f831fdf34c73814362fc2e08dab9564183fb69f193c935a34
+  data.tar.gz: 77f6661494c46d98e72697fe6816de893e1f66fcf9a06f5bfffa7d9f61095da9
 SHA512:
-  metadata.gz: caef8697007fb7a2c678f925db8366ebb6dd7d493f489cc87e7a099f57b8852238a5ca7949d133a11120be6dd6a9eda3535d4d62f60bf31a7ea81187d9b004f7
-  data.tar.gz: bc56bb6ca81812f771196081a70fade906ca1c4a8c4f047f6d63c326c87f67e941bbd05c0cc5c355898726ea6d6591b86fdbed72293bccfbb20dfffca6102c5b
+  metadata.gz: c2dbb4db63a9749d40d0b1c8cd46020e400912f409810d266b483c7916384083cdf1ac51a43732fdf5675fe19b09e02b13ec7c31cd49a832a4eeaf4671ff8464
+  data.tar.gz: a359ac2d32fa026e29d9c53e17551af84cd13b7e4702b8141612ac0b92a0cab41f555ea54419849244f4b62c62d2bae49c6fb198a04bb6471afef7021d50fed9

data/app/jobs/rails_informant/notify_job.rb

@@ -3,9 +3,16 @@ module RailsInformant
     queue_as :default
 
     retry_on ::Net::OpenTimeout, ::Net::ReadTimeout, ::SocketError, RailsInformant::NotifierError, attempts: 5, wait: 15.seconds
-    discard_on ActiveRecord::RecordNotFound
+    discard_on ActiveRecord::RecordNotFound,
+               ArgumentError,
+               Errno::ECONNREFUSED,
+               Errno::ECONNRESET,
+               Errno::EHOSTUNREACH,
+               OpenSSL::SSL::SSLError
 
     def perform(group)
+      Current.delivering_notification = true
+
       occurrence = group.occurrences.order(created_at: :desc).first
       failures = []
 
@@ -17,12 +24,16 @@ module RailsInformant
         failures << e
       end
 
-      group.update_column(:last_notified_at, Time.current) if failures.empty?
-
-      if failures.any?
+      if failures.empty?
+        group.update_column(:last_notified_at, Time.current)
+        Notifiers::CircuitBreaker.record_success
+      else
+        Notifiers::CircuitBreaker.record_failure
         failures.drop(1).each { |e| Rails.logger.error "[RailsInformant] Notifier failed: #{e.class}: #{e.message}" }
         raise failures.first
       end
+    ensure
+      Current.delivering_notification = false
     end
 
     private

data/lib/rails_informant/current.rb

@@ -1,5 +1,5 @@
 module RailsInformant
   class Current < ActiveSupport::CurrentAttributes
-    attribute :breadcrumbs, :user_context, :custom_context
+    attribute :breadcrumbs, :custom_context, :delivering_notification, :user_context
   end
 end

data/lib/rails_informant/error_recorder.rb

@@ -6,18 +6,27 @@ module RailsInformant
     class << self
       def record(error, severity: "error", context: {}, source: nil, env: nil)
         return unless RailsInformant.initialized?
+        return if self_caused_error?(error)
+
         now = Time.current
         attrs = ContextBuilder.group_attributes(error, severity:, context:, env:, now:)
         group = ErrorGroup.find_or_create_for(Fingerprint.generate(error), attrs)
         group.detect_regression!
         store_occurrence(group, error, env:, context:) if should_store_occurrence?(group)
-        notify(group, error)
+        notify(group)
       rescue StandardError => e
         Rails.logger.error "[RailsInformant] Capture failed: #{e.class}: #{e.message}\n#{e.backtrace&.first(5)&.join("\n")}"
       end
 
       private
 
+      # Detect errors caused by RailsInformant itself to prevent feedback loops.
+      # Primary: CurrentAttributes flag set during notification delivery.
+      # Fallback: backtrace heuristic for cross-execution scenarios (e.g. queue retries).
+      def self_caused_error?(error)
+        Current.delivering_notification || error.backtrace&.any? { it.include?("rails_informant/notifiers") }
+      end
+
       def should_store_occurrence?(group)
         return true if group.total_occurrences <= 1
         return true unless group.last_occurrence_stored_at
@@ -38,15 +47,11 @@ module RailsInformant
         keep_ids = group.occurrences.order(created_at: :desc).limit(MAX_OCCURRENCES_PER_GROUP).select(:id)
         Occurrence.where(error_group_id: group.id).where.not(id: keep_ids).delete_all
       end
-      def notify(group, error)
-        return if notifier_error?(error)
+      def notify(group)
+        return if Notifiers::CircuitBreaker.open?
         return unless RailsInformant.config.notifiers.any? { it.should_notify?(group) }
         RailsInformant::NotifyJob.perform_later group
       end
-
-      def notifier_error?(error)
-        error.backtrace&.any? { it.include?("rails_informant/notifiers") }
-      end
     end
   end
 end

data/lib/rails_informant/notifiers/circuit_breaker.rb

@@ -0,0 +1,40 @@
+module RailsInformant
+  module Notifiers
+    class CircuitBreaker
+      FAILURE_THRESHOLD = 5
+      RESET_TIMEOUT = 10.minutes
+
+      class << self
+        def open?
+          return false if failure_count < FAILURE_THRESHOLD
+
+          last_failure_at > RESET_TIMEOUT.ago
+        end
+
+        def record_failure
+          @failure_count = failure_count + 1
+          @last_failure_at = Time.current
+        end
+
+        def record_success
+          reset!
+        end
+
+        def reset!
+          @failure_count = 0
+          @last_failure_at = nil
+        end
+
+        private
+
+        def failure_count
+          @failure_count || 0
+        end
+
+        def last_failure_at
+          @last_failure_at || Time.at(0)
+        end
+      end
+    end
+  end
+end

data/lib/rails_informant.rb

@@ -47,6 +47,7 @@ module RailsInformant
   end
 
   module Notifiers
+    autoload :CircuitBreaker, "rails_informant/notifiers/circuit_breaker"
     autoload :NotificationPolicy, "rails_informant/notifiers/notification_policy"
     autoload :Slack, "rails_informant/notifiers/slack"
     autoload :Webhook, "rails_informant/notifiers/webhook"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails-informant
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
 platform: ruby
 authors:
 - Daniel López Prat
@@ -162,6 +162,7 @@ files:
 - lib/rails_informant/mcp/tools/verify_pending_fixes.rb
 - lib/rails_informant/middleware/error_capture.rb
 - lib/rails_informant/middleware/rescued_exception_interceptor.rb
+- lib/rails_informant/notifiers/circuit_breaker.rb
 - lib/rails_informant/notifiers/notification_policy.rb
 - lib/rails_informant/notifiers/slack.rb
 - lib/rails_informant/notifiers/webhook.rb
@@ -176,16 +177,6 @@ metadata:
   changelog_uri: https://github.com/6temes/rails-informant/releases
   rubygems_mfa_required: 'true'
   source_code_uri: https://github.com/6temes/rails-informant
-post_install_message: |2+
-
-  Rails Informant: SKILL.md has been simplified — workflow guidance now
-  auto-updates via MCP server instructions. Regenerate your skill file:
-
-    bin/rails generate rails_informant:skill
-
-  This is a one-time migration. Future gem updates will not require
-  regenerating SKILL.md.
-
 rdoc_options: []
 require_paths:
 - lib
@@ -204,4 +195,3 @@ rubygems_version: 4.0.3
 specification_version: 4
 summary: Self-hosted error monitoring for Rails with MCP server for agentic workflows
 test_files: []
-...