activerabbit-ai 0.6.0 → 0.6.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f74f9015abfef72171776a4ffc3b2773ba9c065b988d9e4f8ee23e9cdd2b282e
-  data.tar.gz: 4b28180c68924cc06116b19b21bc9b33f0d97f8ac3ed4caa68072c8efdff9fcd
+  metadata.gz: c71eb72533f7d6588444722f3ca8502230006f75bd12cee11b7f585e0a00aa1a
+  data.tar.gz: e187bb60976eda2eccd743589737240f40930897f6063c9ff6b061e517786135
 SHA512:
-  metadata.gz: 0fb79f1162dc1648c9ffe8d6cc05e813b20f3674143acbf935ecc657c7e1273a4ba35a8f873bf87e1e4e491f579b22568d0f45898de1106947ad41973b7317a9
-  data.tar.gz: f2bb325c0780e79160b6acbc4e6e10f98ca427459b58448562166d8c88cb5776e38c10fe3620d713945585ef0206f3a9465a04bdbec26c78ec47e0f78c1ba011
+  metadata.gz: 97ffc354514d50a73b8f32dd086e6d2a095bea00e0ce095464c7ea0a389fc8b1304359125a798abfabf4f456095aec51f3a086a2f6cb1d4c0d3eb5367d1744f0
+  data.tar.gz: dafed4a9c5c2de391bdc1527de3092340a093afdde7cc336a04c533c7d4898d5fad4987992fa3861dbbf806dd792eaa8e2a1037cc0a00133e235cd86747a4e3e

data/CHANGELOG.md

@@ -2,6 +2,26 @@
 
 All notable changes to this project will be documented in this file.
 
+## [0.6.2] - 2026-01-26
+
+### Fixed
+- **ActionMailer deliver_later RuntimeError**: Fixed critical bug where `deliver_later` would raise `RuntimeError: You've accessed the message before asking to deliver it later`
+  - The patch was incorrectly accessing `message.subject` and `message.to` before calling `super`
+  - Rails requires that `message` is NOT accessed before `deliver_later` because only mailer method arguments are passed to the job
+  - Now only tracks safe metadata: `@mailer_class`, `@action`, and argument types
+  - Added error handling to ensure tracking failures don't break email delivery
+
+### Added
+- **ActionMailer patch tests**: 18 comprehensive tests for the ActionMailer integration
+  - Verifies `deliver_later` does NOT access the message object
+  - Verifies `deliver_now` correctly tracks message details
+  - Tests error handling and edge cases
+
+## [0.6.1] - 2026-01-09
+
+### Fixed
+- **Docker/Production path detection**: Fixed `in_app_frame?` to correctly detect app frames in Docker containers and production environments with absolute paths like `/app/app/controllers/...`
+
 ## [0.6.0] - 2026-01-09
 
 ### Added

data/lib/active_rabbit/client/action_mailer_patch.rb

@@ -14,7 +14,8 @@ module ActiveRabbit
           ActiveRabbit::Client.track_event(
             "email_sent",
             {
-              mailer: self.class.name,
+              mailer_class: @mailer_class.name,
+              action: @action,
               message_id: (message.message_id rescue nil),
               subject: (message.subject rescue nil),
               to: (Array(message.to).first rescue nil),
@@ -24,11 +25,26 @@ module ActiveRabbit
         end
       end
 
-      def deliver_later
-        ActiveRabbit::Client.track_event(
-          "email_enqueued",
-          { mailer: self.class.name, subject: (message.subject rescue nil), to: (Array(message.to).first rescue nil) }
-        ) if ActiveRabbit::Client.configured?
+      def deliver_later(...)
+        # IMPORTANT: Do NOT access `message` here!
+        # Rails raises RuntimeError if you access the message before deliver_later
+        # because only mailer method arguments are passed to the job.
+        # We can only safely access metadata that doesn't touch the message object.
+        if ActiveRabbit::Client.configured?
+          begin
+            ActiveRabbit::Client.track_event(
+              "email_enqueued",
+              {
+                mailer_class: @mailer_class.name,
+                action: @action,
+                args: @args&.map { |a| a.class.name }
+              }
+            )
+          rescue => e
+            # Don't let tracking failures break email delivery
+            Rails.logger.error "[ActiveRabbit] Failed to track email_enqueued: #{e.message}" if defined?(Rails) && Rails.logger
+          end
+        end
         super
       end
     end

data/lib/active_rabbit/client/source_code_reader.rb

@@ -156,16 +156,30 @@ module ActiveRabbit
         def in_app_frame?(file)
           return false if blank?(file)
 
-          # In-app if it's in app/, lib/, or similar app directories
-          # and NOT in gems or ruby stdlib
-          (file.start_with?("app/") ||
-           file.start_with?("lib/") ||
-           file.start_with?("config/") ||
-           (file.include?("/app/") && !file.include?("/gems/"))) &&
-            !file.include?("/gems/") &&
-            !file.include?("/ruby/") &&
-            !file.include?("/rubygems/") &&
-            !file.include?("/.bundle/")
+          # Exclude gem/library paths first (these are never in-app)
+          return false if file.include?("/gems/")
+          return false if file.include?("/bundle/")
+          return false if file.include?("/.bundle/")
+          return false if file.include?("/rubygems/")
+          # Exclude Ruby stdlib but not app paths that happen to have "ruby" in them
+          return false if file =~ %r{/ruby/\d+\.\d+\.\d+/}
+
+          # In-app patterns (common Rails app structures)
+          # Relative paths
+          return true if file.start_with?("app/")
+          return true if file.start_with?("lib/")
+          return true if file.start_with?("config/")
+
+          # Absolute paths in Docker/production (e.g., /app/app/controllers/...)
+          # Match paths that contain /app/ followed by typical app directories
+          return true if file =~ %r{/app/(controllers|models|services|jobs|views|helpers|mailers|workers|channels)/}
+          return true if file =~ %r{/lib/[^/]+\.rb$}
+          return true if file =~ %r{/config/}
+
+          # Generic: path contains /app/ but not in excluded paths (already checked above)
+          return true if file.include?("/app/") && file.end_with?(".rb")
+
+          false
         end
 
         def classify_frame(file)

data/lib/active_rabbit/client/version.rb

@@ -1,5 +1,5 @@
 module ActiveRabbit
   module Client
-    VERSION = "0.6.0"
+    VERSION = "0.6.2"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: activerabbit-ai
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 0.6.2
 platform: ruby
 authors:
 - Alex Shapalov
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-01-10 00:00:00.000000000 Z
+date: 2026-01-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: concurrent-ruby
@@ -81,7 +81,6 @@ files:
 - README.md
 - Rakefile
 - TESTING_GUIDE.md
-- activerabbit-ai-0.4.5.gem
 - check_api_data.rb
 - examples/rails_app_testing.rb
 - examples/rails_integration.rb