rails-informant 0.2.1 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a3921be9c6e9f434524b03c140e94ca5de03128bbd9b27c9a9df67477ed79aa7
-  data.tar.gz: 90feef10dc4d4ada45acaefa06b65ea3cdc5543ba86beb5952f78b3167ec7948
+  metadata.gz: 71f2d05d2caf43bbaab7641118f09b86cafb4949d9f4c2be262a4773f6a88eab
+  data.tar.gz: 8f3de02212167bfcbbb412d653af259eeed2a0e5907f75b8e1191d724faf112a
 SHA512:
-  metadata.gz: caef8697007fb7a2c678f925db8366ebb6dd7d493f489cc87e7a099f57b8852238a5ca7949d133a11120be6dd6a9eda3535d4d62f60bf31a7ea81187d9b004f7
-  data.tar.gz: bc56bb6ca81812f771196081a70fade906ca1c4a8c4f047f6d63c326c87f67e941bbd05c0cc5c355898726ea6d6591b86fdbed72293bccfbb20dfffca6102c5b
+  metadata.gz: b7c2c8599fe28a7adef712a0cc4f989cdae99465a18b2120055c049dbf77136e243dee9f382fc8dee0ed6d6c7d2d6173a18b1e42a750d031a31268fd377758ff
+  data.tar.gz: 59729b6bd6f564023810acd041e3d71f7cf930e8bb65dd88833495b6392d3eb979aa920b80650d4ffe6833f91675743b928a1162ecad4bf10c75c894759843ad

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/configuration.rb

@@ -1,15 +1,18 @@
 module RailsInformant
   class Configuration
-    attr_accessor :api_token,
-                  :capture_errors,
+    attr_accessor :capture_errors,
                   :capture_user_email,
+                  :api_token,
                   :ignored_exceptions,
                   :retention_days,
                   :slack_webhook_url,
                   :webhook_url
 
+    attr_writer :app_name
+
     def initialize
       @api_token = ENV["INFORMANT_API_TOKEN"]
+      @app_name = ENV["INFORMANT_APP_NAME"]
       @capture_errors = ENV.fetch("INFORMANT_CAPTURE_ERRORS", "true") != "false"
       @capture_user_email = false
       @custom_notifiers = []
@@ -19,6 +22,10 @@ module RailsInformant
       @webhook_url = ENV["INFORMANT_WEBHOOK_URL"]
     end
 
+    def app_name
+      @app_name.presence || detect_app_name
+    end
+
     # Returns all notifiers: built-in (auto-registered from config) + custom.
     def notifiers
       @_notifiers ||= built_in_notifiers + @custom_notifiers
@@ -36,6 +43,10 @@ module RailsInformant
 
     private
 
+    def detect_app_name
+      Rails.application&.class&.module_parent_name.presence || "App"
+    end
+
     def built_in_notifiers
       [
         (Notifiers::Slack.new if slack_webhook_url.present?),

data/lib/rails_informant/context_builder.rb

@@ -50,15 +50,23 @@ module RailsInformant
       end
 
       def build_environment_context
-        @_static_env ||= {
-          rails_env: Rails.env.to_s,
-          ruby_version: RUBY_VERSION,
-          rails_version: Rails::VERSION::STRING,
-          hostname: Socket.gethostname
-        }.freeze
+        @_static_env ||= begin
+          hostname = Socket.gethostname
+          env = {
+            rails_env: Rails.env.to_s,
+            ruby_version: RUBY_VERSION,
+            rails_version: Rails::VERSION::STRING
+          }
+          env[:hostname] = hostname unless hostname == "localhost"
+          env.freeze
+        end
         @_static_env.merge(pid: Process.pid)
       end
 
+      def reset!
+        @_static_env = nil
+      end
+
       def group_attributes(error, severity:, context:, env:, now:)
         {
           error_class: error.class.name, severity:,

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/notifiers/slack.rb

@@ -17,39 +17,51 @@ module RailsInformant
       end
 
       def build_payload(error_group, occurrence)
-        regression_tag = regression?(error_group) ? " [REGRESSION]" : ""
-
         {
+          text: "#{error_group.error_class}: #{error_group.message.to_s.truncate(200)}",
           blocks: [
-            {
-              type: "header",
-              text: {
-                type: "plain_text",
-                text: "#{error_group.error_class}#{regression_tag}",
-                emoji: true
-              }
-            },
-            {
-              type: "section",
-              fields: [
-                { type: "mrkdwn", text: "*Message:*\n#{error_group.message.to_s.truncate(200)}" },
-                { type: "mrkdwn", text: "*Status:*\n#{error_group.status}" },
-                { type: "mrkdwn", text: "*Occurrences:*\n#{error_group.total_occurrences}" },
-                { type: "mrkdwn", text: "*First seen:*\n#{error_group.first_seen_at&.iso8601}" }
-              ]
-            },
-            {
-              type: "section",
-              fields: [
-                location_field(error_group),
-                { type: "mrkdwn", text: "*Severity:*\n#{error_group.severity}" }
-              ].compact
-            },
+            header_block(error_group, occurrence),
+            error_class_block(error_group),
+            fields_block(error_group),
             context_block(occurrence)
           ].compact
         }
       end
 
+      def header_block(error_group, occurrence)
+        env = occurrence&.environment_context&.dig("rails_env") || Rails.env
+        regression_tag = regression?(error_group) ? " [REGRESSION]" : ""
+        text = "🚨 #{RailsInformant.app_name} · #{env}#{regression_tag}".truncate(150)
+
+        {
+          type: "header",
+          text: { type: "plain_text", text:, emoji: true }
+        }
+      end
+
+      def error_class_block(error_group)
+        {
+          type: "section",
+          text: {
+            type: "mrkdwn",
+            text: "*#{error_group.error_class}*\n#{error_group.message.to_s.truncate(200)}"
+          }
+        }
+      end
+
+      def fields_block(error_group)
+        {
+          type: "section",
+          fields: [
+            { type: "mrkdwn", text: "*Status:*\n#{error_group.status}" },
+            { type: "mrkdwn", text: "*Occurrences:*\n#{error_group.total_occurrences}" },
+            { type: "mrkdwn", text: "*First seen:*\n#{error_group.first_seen_at&.iso8601}" },
+            { type: "mrkdwn", text: "*Severity:*\n#{error_group.severity}" },
+            location_field(error_group)
+          ].compact
+        }
+      end
+
       def location_field(error_group)
         location = error_group.controller_action || error_group.job_class || error_group.first_backtrace_line
         return unless location
@@ -58,19 +70,14 @@ module RailsInformant
       end
 
       def context_block(occurrence)
-        return unless occurrence
-
-        elements = []
-        if occurrence.git_sha
-          elements << { type: "mrkdwn", text: "Deploy: `#{occurrence.git_sha[0, 7]}`" }
-        end
-        if occurrence.environment_context&.dig("hostname")
-          elements << { type: "mrkdwn", text: "Host: `#{occurrence.environment_context["hostname"]}`" }
-        end
-
-        return if elements.empty?
+        return unless occurrence&.git_sha
 
-        { type: "context", elements: elements }
+        {
+          type: "context",
+          elements: [
+            { type: "mrkdwn", text: "Deploy: `#{occurrence.git_sha[0, 7]}`" }
+          ]
+        }
       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"
@@ -56,7 +57,8 @@ module RailsInformant
   self.config = Configuration.new
 
   class << self
-    delegate :api_token,
+    delegate :app_name,
+             :api_token,
              :capture_errors,
              :capture_user_email,
              :ignored_exceptions,

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails-informant
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.3.0
 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: []
-...