activerabbit-ai 0.4.2 → 0.4.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 48326a10836bf940aea7ebf3b6eca91f60086eb69e51e5e94266db5012d47ad9
-  data.tar.gz: 9244e314ba69d85fd9652604014520f8cbc6a4ab2c6da046b28a7f9c540a402e
+  metadata.gz: 86853d97a830e8d411ba9052ff231b29a2cddb526799e6b5c89e848ce026c7b2
+  data.tar.gz: 1faeb9a59503089d03b0ecda80fab72cd8028b3bfd49f4da5c5d25ff9cfb3139
 SHA512:
-  metadata.gz: 866f2f2f296c7a254526691e02e6c182aade30274985de6ca69f8e715b9af4036b489437eb2742840bf9fe1a582a0eaafae3ad0605342da4acde3dc6ee4d752a
-  data.tar.gz: f59a2b3dcfd170f048f96751f870cc238333fc576fbf200668ef040db9a88b97de8b1a0f60cf8c6490c3ee3f8c4ee1941318571d39d644921e0f04e2a2ae29c8
+  metadata.gz: 760d010f75113d98d6e2f83fd3768d37d7cf67e307c24418717caa8c07ea489337ed7923ef876ed7bb95e4556e6780466efaec374d2ffc62d00b7b16a37f6fda
+  data.tar.gz: 441b28d1cc42c02a13c3efe4d387674932fceafca7c8571c6b24792ac69134338050ad030ebe65253ee48387e108b8b7d85014866bd42fed1e56f1c9fa356647

data/CHANGELOG.md

@@ -2,6 +2,26 @@
 
 All notable changes to this project will be documented in this file.
 
+## [0.4.4] - 2025-10-22
+
+### Improved
+- **Time-based error deduplication**: Changed from "once per server lifecycle" to time-window based
+- Added `dedupe_window` configuration option (default: 300 seconds / 5 minutes)
+- Set `dedupe_window` to `0` to disable deduplication (useful for development/testing)
+- Automatic memory cleanup of old dedupe entries (keeps last hour only)
+- Better logging when errors are deduplicated
+
+### Fixed
+- Error deduplication no longer prevents same error from being reported after time window expires
+- Memory leak prevention by cleaning old deduplication entries
+
+## [0.4.3] - 2025-10-22
+
+### Fixed
+- **Critical bug fix**: Removed reference to non-existent `ActiveRabbit::Client::Dedupe` class in error reporter
+- Error tracking now works properly via Rails error reporter integration
+- Errors are successfully captured and sent to ActiveRabbit API
+
 ## [0.4.2] - 2025-01-04
 
 ### Fixed

data/README.md

@@ -236,6 +236,65 @@ ActiveRabbit::Client.configure do |config|
 end
 ```
 
+### Recommended Exceptions to Capture
+
+Below is a practical list of exceptions APMs should capture by default in a Rails app. Some are optional/noisy and typically excluded unless needed.
+
+- Core (Ruby/Stdlib)
+  - `StandardError`, `RuntimeError`, `NoMethodError`, `NameError`, `ArgumentError`, `TypeError`, `IndexError`, `KeyError`
+  - `Timeout::Error`, `JSON::ParserError`, `OpenSSL::SSL::SSLError`, `SocketError`, `Errno::ECONNREFUSED`/`ETIMEDOUT`/`EHOSTUNREACH`
+
+- ActionPack / Controllers
+  - `ActionController::ParameterMissing`
+  - `ActionController::BadRequest`
+  - `ActionController::InvalidAuthenticityToken` (optional; can be noisy)
+  - `ActionController::UnknownFormat`
+  - `ActionController::NotImplemented`
+
+- Routing (optional/noisy)
+  - `ActionController::RoutingError` (commonly excluded; enable only if needed)
+
+- Views / Templates
+  - `ActionView::Template::Error`
+  - `ActionView::MissingTemplate`
+  - `Encoding::UndefinedConversionError` (template rendering)
+
+- ActiveRecord / Database
+  - `ActiveRecord::RecordInvalid`
+  - `ActiveRecord::RecordNotFound` (optional; may be expected business logic)
+  - `ActiveRecord::StatementInvalid` (includes `PG::Error` subclasses)
+  - `ActiveRecord::Deadlocked`, `ActiveRecord::LockWaitTimeout`
+  - `ActiveRecord::RecordNotUnique`
+  - `ActiveRecord::ConnectionTimeoutError`
+  - `ActiveRecord::SerializationFailure`
+
+- Background Jobs (ActiveJob/Sidekiq)
+  - `ActiveJob::DeserializationError`
+  - Any unhandled exception raised in job `perform`
+
+- Networking/HTTP Clients
+  - `Net::OpenTimeout`, `Net::ReadTimeout`
+  - `Faraday::TimeoutError`, `Faraday::ConnectionFailed`
+  - `HTTP::Error` (http.rb), `RestClient::Exception`
+
+- Caching/Redis
+  - `Redis::BaseError`, `Redis::TimeoutError`, `Redis::CannotConnectError`
+
+- ActiveStorage
+  - `ActiveStorage::IntegrityError`
+  - `ActiveStorage::FileNotFoundError`
+
+- ActionCable
+  - `ActionCable::Connection::Authorization::UnauthorizedError` (if applicable)
+
+- Security/Crypto
+  - `ActiveSupport::MessageEncryptor::InvalidMessage`
+  - `ActiveSupport::MessageVerifier::InvalidSignature`
+
+Notes:
+- Optional/noisy: `RoutingError`, `RecordNotFound`, `InvalidAuthenticityToken`. Consider monitoring via metrics or targeted capture.
+- If exceptions are rescued by your app, enable reporting of rescued exceptions (`before_send_exception`/custom middleware) so they are still tracked when appropriate.
+
 ### Callbacks
 
 ```ruby

data/lib/active_rabbit/client/configuration.rb

@@ -14,6 +14,7 @@ module ActiveRabbit
       attr_accessor :ignored_exceptions, :ignored_user_agents, :ignore_404
       attr_accessor :release, :server_name, :logger
       attr_accessor :before_send_event, :before_send_exception
+      attr_accessor :dedupe_window  # Time window in seconds for error deduplication (0 = disabled)
 
       def initialize
         @api_url = ENV.fetch("active_rabbit_API_URL", "https://api.activerabbit.ai")
@@ -60,6 +61,9 @@ module ActiveRabbit
           /Twitterbot/i
         ]
 
+        # Deduplication (0 = disabled, time in seconds for same error to be considered duplicate)
+        @dedupe_window = 300  # 5 minutes by default
+
         # Metadata
         @release = detect_release
         @server_name = detect_server_name

data/lib/active_rabbit/client/error_reporter.rb

@@ -10,21 +10,32 @@ module ActiveRabbit
           begin
             Rails.logger.info "[ActiveRabbit] Error reporter caught: #{exception.class}: #{exception.message}" if defined?(Rails.logger)
 
-            # Initialize de-dup set
-            $reported_errors ||= Set.new
-
+            # Time-based deduplication: track errors with timestamps
+            $reported_errors ||= {}
+            
             # Generate a unique key for this error
             error_key = "#{exception.class.name}:#{exception.message}:#{exception.backtrace&.first}"
+            
+            # Get dedupe window from config (default 5 minutes, 0 = disabled)
+            dedupe_window = defined?(ActiveRabbit::Client.configuration.dedupe_window) ? 
+                            ActiveRabbit::Client.configuration.dedupe_window : 300
+            
+            current_time = Time.now.to_i
+            last_seen = $reported_errors[error_key]
+            
+            # Report if: never seen before, OR dedupe disabled (0), OR outside dedupe window
+            should_report = last_seen.nil? || dedupe_window == 0 || (current_time - last_seen) > dedupe_window
+            
+            if should_report
+              $reported_errors[error_key] = current_time
+              
+              # Clean old entries to prevent memory leak (keep last hour)
+              $reported_errors.delete_if { |_, timestamp| current_time - timestamp > 3600 }
 
-            # Only report if we haven't seen this error before and not deduped in short window
-            unless $reported_errors.include?(error_key)
-              $reported_errors.add(error_key)
-
-              dedupe_context = { request_id: (context && (context[:request_id] || context[:request]&.[](:request_id))) }
-              unless ActiveRabbit::Client::Dedupe.seen_recently?(exception, dedupe_context)
-                enriched = build_enriched_context(exception, handled: handled, severity: severity, context: context)
-                ActiveRabbit::Client.track_exception(exception, handled: handled, context: enriched)
-              end
+              enriched = build_enriched_context(exception, handled: handled, severity: severity, context: context)
+              ActiveRabbit::Client.track_exception(exception, handled: handled, context: enriched)
+            else
+              Rails.logger.debug "[ActiveRabbit] Error deduplicated (last seen #{current_time - last_seen}s ago)" if defined?(Rails.logger)
             end
           rescue => e
             Rails.logger.error "[ActiveRabbit] Error in ErrorReporter::Subscriber#report: #{e.class} - #{e.message}" if defined?(Rails.logger)

data/lib/active_rabbit/client/version.rb

@@ -2,6 +2,6 @@
 
 module ActiveRabbit
   module Client
-    VERSION = "0.4.2"
+    VERSION = "0.4.4"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: activerabbit-ai
 version: !ruby/object:Gem::Version
-  version: 0.4.2
+  version: 0.4.4
 platform: ruby
 authors:
 - Alex Shapalov
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-10-21 00:00:00.000000000 Z
+date: 2025-10-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: concurrent-ruby