rails-informant 0.6.1 → 0.6.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 69a62b35d2fcbee6378fff16208c4f1091f87c44ffd41baa638f59a42d4caefd
-  data.tar.gz: 97ecbe4eb82e39be5b42c42b6bb50d257d157eb507201bde373bca8e477222d2
+  metadata.gz: 7f1d6f0b5274e2da8e271fdc47352ff3c35f181e6b799b64a8f5273853b07139
+  data.tar.gz: 165015e620935cca923dc3fbf89b049d587666d4047bb8fafc4b73049be50631
 SHA512:
-  metadata.gz: fe94a07f7029f360b1517c2ce8f3b0b2a27872ce08ba5d6a5c9a631030e53865ad58fd1a289b67d61db66ad650c22c21b5863b8628103c568c92a5f5a0730a4e
-  data.tar.gz: f5356e08df971271b046a8f5a7387a3219b53fe64b3090ab63a9a89f3f1c7b1c2246f5dcd1b62ff107a25c3ca4b47fbf09a3feb4c4964e56b25a9994f17af0d4
+  metadata.gz: aa70bfe035cf0cb73d4aa8a9976c558ab5977ca858fdb526613e55e727f72261b0410143678a6eb523071fce99e4758f598f307727b2e51336adf4c8336a214e
+  data.tar.gz: a613e453aa984669d16c5c74ddbd8772faacbde476f341f2f94c405d6dea27db3b6a6bdc7115a2a54c38a74da26e33da74fe5412c8faf87cc294743e0c62c6db

data/README.md

@@ -93,7 +93,7 @@ Every option can be set via an environment variable. The initializer takes prece
 | `capture_user_email` | _(none)_ | `false` | Capture email from detected user (PII -- opt-in) |
 | `ignored_exceptions` | `INFORMANT_IGNORED_EXCEPTIONS` | `[]` | Exception classes to skip (walks cause chain) |
 | `ignored_paths` | `INFORMANT_IGNORED_PATHS` | `[]` | Request paths to skip (exact or segment match) |
-| `job_attempt_threshold` | `INFORMANT_JOB_ATTEMPT_THRESHOLD` | `nil` | Suppress job errors until Nth retry |
+| `job_attempt_threshold` | `INFORMANT_JOB_ATTEMPT_THRESHOLD` | `nil` | Suppress job failures until the Nth attempt (handled reports are always recorded) |
 | `retention_days` | `INFORMANT_RETENTION_DAYS` | `nil` | Auto-purge resolved errors after N days |
 | `slack_webhook_url` | `INFORMANT_SLACK_WEBHOOK_URL` | `nil` | Slack incoming webhook URL |
 | `spike_protection` | _(none)_ | `nil` | Rate-limit per error group: `{ threshold: 50, window: 1.minute }` |

data/lib/rails_informant/context_builder.rb

@@ -113,7 +113,11 @@ module RailsInformant
       end
 
       def extract_job_class(context)
-        context.dig(:job, :class) || context[:job_class]
+        case context[:job]
+        in ActiveJob::Base => job then job.class.name
+        in { class: String => job_class } then job_class
+        else context[:job_class]
+        end
       end
 
       def filtered_url(url)

data/lib/rails_informant/engine.rb

@@ -11,11 +11,17 @@ module RailsInformant
     initializer "rails_informant.context_middleware" do
       next unless RailsInformant.capture_errors
 
-      Rails.error.add_middleware ->(error, context) {
+      Rails.error.add_middleware ->(error, context:, **) {
         context.merge deploy_sha: RailsInformant.current_git_sha
       }
     end
 
+    initializer "rails_informant.job_execution_context" do
+      next unless RailsInformant.capture_errors
+
+      ActiveSupport.on_load(:active_job) { prepend RailsInformant::JobExecutionContext }
+    end
+
     initializer "rails_informant.middleware" do |app|
       next unless RailsInformant.capture_errors
 

data/lib/rails_informant/error_subscriber.rb

@@ -7,7 +7,7 @@ module RailsInformant
       return if handled && severity == :info
       return if source && SKIP_SOURCES.match?(source)
       return if RailsInformant::Current.silenced
-      return if below_job_attempt_threshold?(context)
+      return if !handled && below_job_attempt_threshold?(context)
       return if RailsInformant.ignored_exception?(error)
       return if RailsInformant.already_captured?(error)
 
@@ -25,6 +25,8 @@ module RailsInformant
       case context
       in job: { executions: Integer => executions }
         executions < threshold
+      in job: ActiveJob::Base => job
+        job.executions < threshold
       in executions: Integer => executions
         executions < threshold
       else

data/lib/rails_informant/job_execution_context.rb

@@ -0,0 +1,11 @@
+module RailsInformant
+  # Rails leaves ExecutionContext[:job] set after perform_now returns, so a job
+  # performed inline would claim every later report in the same request or job.
+  # A failing job stays in place so the executor's report still names it.
+  module JobExecutionContext
+    def perform_now
+      previous = ActiveSupport::ExecutionContext.to_h[:job]
+      super.tap { ActiveSupport::ExecutionContext[:job] = previous }
+    end
+  end
+end

data/lib/rails_informant.rb

@@ -35,6 +35,7 @@ module RailsInformant
   autoload :Event, "rails_informant/event"
   autoload :Fingerprint, "rails_informant/fingerprint"
   autoload :Integration, "rails_informant/integration"
+  autoload :JobExecutionContext, "rails_informant/job_execution_context"
   autoload :StructuredEventSubscriber, "rails_informant/structured_event_subscriber"
 
   module Middleware
@@ -173,7 +174,7 @@ module RailsInformant
       else
         head
       end
-    rescue Errno::ENOENT, Errno::EACCES, ArgumentError
+    rescue Errno::ENOENT, Errno::EACCES, Errno::ENOTDIR, ArgumentError
       nil
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails-informant
 version: !ruby/object:Gem::Version
-  version: 0.6.1
+  version: 0.6.2
 platform: ruby
 authors:
 - Daniel López Prat
@@ -147,6 +147,7 @@ files:
 - lib/rails_informant/event.rb
 - lib/rails_informant/fingerprint.rb
 - lib/rails_informant/integration.rb
+- lib/rails_informant/job_execution_context.rb
 - lib/rails_informant/mcp.rb
 - lib/rails_informant/mcp/base_tool.rb
 - lib/rails_informant/mcp/client.rb