peak_flow_utils 0.1.24 → 0.1.27

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '09b4cc6c4746170bc4be78f58c447be7c24fb98fe4ab2616c1efe5e3199021be'
-  data.tar.gz: 9046266b37ad81f316ff7ea2b87c21518ad575dc12b62b0a4dc172cbf5e9775d
+  metadata.gz: d3e98cbd0f73efee4f3fe4bb4402c959359be3f711d935e9060f86f20ebae56f
+  data.tar.gz: e63aa1314f76a6b3eca83356275f9a8d5e635977baf56ea7bdb488b5c087dfcd
 SHA512:
-  metadata.gz: eabf75ca1e26aedf3b2e4a22e56513317a12160757f284ce80c8f65d54dd7cb41f9515616a9a1a07453eb99cf6457d2b5e17c5e9f0cf171ba6d850c209ddf34a
-  data.tar.gz: 752b031ddbe701f07a7360ee5342fe28ebf71b2d6aa3b9b0ea4b1c695766a1fbc8e529452c67c6367e0ff616d3ddaee0b96269e5c1da4eef989de836c2291557
+  metadata.gz: 3c48c62df3d0432f2e37cd1f02b6f314b5c7f3bbfebf6409a0ce607aba86b93fc858b0b42767d62519a627c9c2241c181e80b6582eb9e7925f28bd3525f439f8
+  data.tar.gz: 75df8dd79b7db0614f70bd90627d93c80b9ff1814425af09e83f0318e853d95d29bba6c5d80b31c460ecb68e52ec120b0fcca01ca0016bfd6eeaefc30be4b61e

data/README.md

@@ -32,6 +32,8 @@ PeakFlowUtils::Notifier.configure(auth_token: "your-token")
 PeakFlowUtils::Notifier.notify(error: error)
 ```
 
+Errors that escape a `PeakFlowUtils::Notifier.with_parameters` block keep a snapshot of the scoped parameters. Later framework handlers, such as Sidekiq error handlers, can report the same error after the block has unwound and still include the captured context.
+
 ### Reporting Rails errors
 
 Add this to `config/peakflow.rb`:

data/lib/peak_flow_utils/engine.rb

@@ -2,6 +2,6 @@ require "rails"
 
 module PeakFlowUtils; end
 
-class PeakFlowUtils::Engine < Rails::Engine
+class PeakFlowUtils::Engine < Rails::Engine # rubocop:disable Style/OneClassPerFile
   isolate_namespace PeakFlowUtils
 end

data/lib/peak_flow_utils/inherited_local_var.rb

@@ -49,7 +49,9 @@ Thread.class_eval do
 
   def self.inherited_local_vars_set(values)
     inherited_local_vars_mutex.synchronize do
-      _inherited_local_vars.merge!(values)
+      current_vars = _inherited_local_vars
+      new_vars = PeakFlowUtils::DeepMerger.execute!(hashes: [current_vars, values])
+      Thread.current.instance_variable_set(:@_inherited_local_vars, new_vars)
     end
   end
 end

data/lib/peak_flow_utils/notifier.rb

@@ -3,6 +3,8 @@ class PeakFlowUtils::Notifier
   class NotConfiguredError < RuntimeError; end
   class NotifyMessageError < RuntimeError; end
 
+  CAPTURED_PARAMETERS_INSTANCE_VARIABLE = :@peak_flow_utils_captured_parameters
+
   attr_reader :auth_token, :mutex, :parameters
 
   def self.configure(auth_token:)
@@ -37,22 +39,21 @@ class PeakFlowUtils::Notifier
     random_id = ::SecureRandom.hex(16)
 
     ::PeakFlowUtils::Notifier.current.mutex.synchronize do
-      raise "'parameters' was nil?" if ::PeakFlowUtils::Notifier.current.parameters.value.nil?
-
-      parameters_with = ::PeakFlowUtils::Notifier.current.parameters.value.clone
-      parameters_with[random_id] = parameters
+      current_parameters = ::PeakFlowUtils::Notifier.current.parameters.value
+      raise "'parameters' was nil?" if current_parameters.nil?
 
-      ::PeakFlowUtils::Notifier.current.parameters.value = parameters_with
+      current_parameters[random_id] = parameters
     end
 
     begin
       yield
+    rescue StandardError => e
+      ::PeakFlowUtils::Notifier.current.capture_parameters_for_error(e)
+      raise
     ensure
       ::PeakFlowUtils::Notifier.current.mutex.synchronize do
-        parameters_without = ::PeakFlowUtils::Notifier.current.parameters.value.clone
-        parameters_without.delete(random_id)
-
-        ::PeakFlowUtils::Notifier.current.parameters.value = parameters_without
+        current_parameters = ::PeakFlowUtils::Notifier.current.parameters.value
+        current_parameters.delete(random_id)
       end
     end
   end
@@ -64,8 +65,22 @@ class PeakFlowUtils::Notifier
     @parameters = ::PeakFlowUtils::InheritedLocalVar.new({})
   end
 
-  def current_parameters(parameters: nil)
+  def capture_parameters_for_error(error)
+    return if error.instance_variable_defined?(CAPTURED_PARAMETERS_INSTANCE_VARIABLE)
+
+    error.instance_variable_set(CAPTURED_PARAMETERS_INSTANCE_VARIABLE, current_parameters)
+  end
+
+  def captured_parameters_for_error(error)
+    return unless error&.instance_variable_defined?(CAPTURED_PARAMETERS_INSTANCE_VARIABLE)
+
+    error.instance_variable_get(CAPTURED_PARAMETERS_INSTANCE_VARIABLE)
+  end
+
+  def current_parameters(error: nil, parameters: nil)
     hashes = current_parameters_hashes
+    captured_parameters = captured_parameters_for_error(error)
+    hashes << captured_parameters if captured_parameters
     hashes << parameters if parameters
 
     ::PeakFlowUtils::DeepMerger.execute!(hashes: hashes)
@@ -94,7 +109,7 @@ class PeakFlowUtils::Notifier
       error: error
     )
 
-    merged_parameters = current_parameters(parameters: parameters)
+    merged_parameters = current_parameters(error: error, parameters: parameters)
 
     uri = URI("https://www.peakflow.io/errors/reports")
 

data/lib/peak_flow_utils/version.rb

@@ -1,3 +1,3 @@
 module PeakFlowUtils
-  VERSION = "0.1.24".freeze
+  VERSION = "0.1.27".freeze
 end

data/lib/tasks/release.rake

@@ -94,7 +94,8 @@ private
       )
     )
 
-    run!("git", "add", VERSION_FILE.to_s)
+    run!("bundle", "install")
+    run!("git", "add", VERSION_FILE.to_s, "Gemfile.lock")
   end
 
   def commit!(next_version)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: peak_flow_utils
 version: !ruby/object:Gem::Version
-  version: 0.1.24
+  version: 0.1.27
 platform: ruby
 authors:
 - kaspernj
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-03-30 00:00:00.000000000 Z
+date: 2026-05-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails