rails-informant 0.6.0 → 0.6.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 905f71f9e86e6c982710d2c1dbd8471218798e1cfa80b8baca78124180a1fef3
-  data.tar.gz: 9cc699035012351e2bfe4b0423c2b7fe0bffd7880f05250d2d54f758bf2d50ea
+  metadata.gz: 7f1d6f0b5274e2da8e271fdc47352ff3c35f181e6b799b64a8f5273853b07139
+  data.tar.gz: 165015e620935cca923dc3fbf89b049d587666d4047bb8fafc4b73049be50631
 SHA512:
-  metadata.gz: b103c2f10d32404883a51e7d9ef82daf0dda368bd7ec50165d32f88b9b9d53924aeb6264af9c4e7f10468d6abe891228a62e7e09c2584a74a0d469a619cf9dd5
-  data.tar.gz: fad95e73c07500480d703c2018e5eeeef77656f0cd16aced9b8e778d90c700a450b1f2963c54f093b76fc2b501a013148b2afb18bef81b8eeed399db1bd5d96c
+  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 }` |
@@ -199,6 +199,12 @@ The bundled `informant-mcp` executable connects Claude Code to your error data v
 
 The `rails_informant:skill` generator creates `.mcp.json` automatically. Set `INFORMANT_PRODUCTION_URL` and `INFORMANT_PRODUCTION_TOKEN` as environment variables (e.g., via `.envrc` + direnv).
 
+`informant-mcp` reads these variables when it starts, so a Claude Code session started without them loaded crashes the server on boot. To load `.envrc` on every start, launch the server through direnv. The generator and `informant:doctor` both accept this entry:
+
+```json
+"informant": { "command": "direnv", "args": ["exec", ".", "informant-mcp"] }
+```
+
 For multi-environment setups, add env vars for each environment:
 
 ```bash

data/lib/generators/rails_informant/skill_generator.rb

@@ -25,7 +25,9 @@ module RailsInformant
       if File.exist?(mcp_path)
         existing = JSON.parse(File.read(mcp_path))
         existing["mcpServers"] ||= {}
-        existing["mcpServers"]["informant"] = Content.mcp_entry
+        unless Content.accepted_mcp_entries.include?(existing["mcpServers"]["informant"])
+          existing["mcpServers"]["informant"] = Content.mcp_entry
+        end
         create_file ".mcp.json", JSON.pretty_generate(existing) + "\n", force: true
       else
         create_file ".mcp.json", JSON.pretty_generate(

data/lib/rails_informant/claude_integration_content.rb

@@ -65,6 +65,13 @@ module RailsInformant
       { "command" => "informant-mcp" }
     end
 
+    # Entries an app may keep in place of mcp_entry. The direnv form loads the
+    # app's .envrc first: informant-mcp reads its URL and token at startup, so a
+    # session started without the .envrc loaded crashes it on boot.
+    def accepted_mcp_entries
+      [ mcp_entry, { "command" => "direnv", "args" => [ "exec", ".", "informant-mcp" ] } ]
+    end
+
     # A single UserPromptSubmit hook registration for settings.json.
     def hook_registration
       {

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

@@ -126,7 +126,7 @@ module RailsInformant
         "hook" => normalize_text(read_or_empty(hook_path)),
         "skill" => @hyperdrive_managed ? "" : normalize_text(read_or_empty(skill_path)),
         "settings" => canonical_json(live_settings_registrations || {}),
-        "mcp" => canonical_json(live_mcp_entry || {})
+        "mcp" => canonical_json(Content.accepted_mcp_entries.include?(live_mcp_entry) ? Content.mcp_entry : live_mcp_entry || {})
       )
     end
 

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.0
+  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