memhealth 0.1.2 → 0.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5fda50a609f167feffba18ed772f70c528fe8ae883605520a8b51283ea9e45af
-  data.tar.gz: 30bb76d54747594b6fe1b2b9432759a53f201b2ed8b6e5852a78fe7775f654a1
+  metadata.gz: 6937ace9f7a2407974057577e378f9433bb891c4d848779f80da32e4eb32703d
+  data.tar.gz: 70534a9390c32d9e0cdc1e318caee705ca82a3a93042625876978b72ed8c3079
 SHA512:
-  metadata.gz: 2d69b6f4a6f3341f1eb02f64301d690479788fbd6de6e8ef65e9da88ce40c709af7743c685d290d42e020003d67bd093318ed6189ebe722fa1a37eb25017a800
-  data.tar.gz: 9160e5176c77f8d06e3f6ade37ecc39b0ba171829370eb93f33b332b51c775096c403aba3cd52e977b17912fc33123d8f9ba2b9d47da73aaf47138bab8c98072
+  metadata.gz: 2d10a9cee1ea390ee7b55a121fdc94f4f38086eb5ab1cdbad53e1cbf71d2e1b7355d8689d30f7774ce494df25c261e5f27cfce357d594255d279e5d9149307c9
+  data.tar.gz: 6aea016d4c158e2554317c531a44314bf650b87623bea24b604e252cf74b3bc3cb10339b5a6209c7d952e2c494270159c6d120f4b9f7a1f74c853e9429dae86a

data/lib/mem_health/job_tracking_middleware.rb

@@ -30,8 +30,14 @@ module MemHealth
     private
 
     def extract_job_class(worker, job)
-      # For ActiveJob jobs, extract the actual job class from the wrapper
-      if worker.class.name == 'ActiveJob::QueueAdapters::SidekiqAdapter::JobWrapper'
+      # For ActiveJob jobs, extract the actual job class from the wrapper.
+      # Detect via the payload's "wrapped" key rather than the wrapper's class
+      # name: on Sidekiq 7+/Rails 7.1+ the old
+      # ActiveJob::QueueAdapters::SidekiqAdapter::JobWrapper constant is just an
+      # alias whose #name resolves to "Sidekiq::ActiveJob::Wrapper", so the old
+      # string comparison never matched and every job was mislabeled as the
+      # wrapper. The "wrapped" key is what the sibling extractors key off too.
+      if job['wrapped'] # ActiveJob
         job['wrapped'] || job['args']&.first&.dig('job_class') || worker.class.name
       else
         worker.class.name

data/lib/mem_health/middleware.rb

@@ -4,6 +4,7 @@ module MemHealth
 
     @@request_count = 0
     @@skipped_requests_count = 0
+    @@counter_mutex = Mutex.new
 
     def self.redis_tracked_requests_key
       "#{MemHealth.configuration.redis_key_prefix}:tracked_requests"
@@ -14,12 +15,13 @@ module MemHealth
     end
 
     def call(env)
-      @@request_count += 1
+      @@counter_mutex.synchronize { @@request_count += 1 }
 
       request = Rack::Request.new(env)
 
+      status = headers = response = nil
       metrics = measure_memory do
-        @status, @headers, @response = @app.call(env)
+        status, headers, response = @app.call(env)
       end
 
       # Skip the first few requests as they have large memory jumps due to class loading
@@ -40,15 +42,17 @@ module MemHealth
                            metadata.merge(execution_time: metrics[:execution_time]), type: :web)
         end
       else
-        @@skipped_requests_count += 1
+        @@counter_mutex.synchronize { @@skipped_requests_count += 1 }
       end
 
-      [@status, @headers, @response]
+      [status, headers, response]
     end
 
     def self.reset_data
-      @@request_count = 0
-      @@skipped_requests_count = 0
+      @@counter_mutex.synchronize do
+        @@request_count = 0
+        @@skipped_requests_count = 0
+      end
       MemHealth.configuration.redis.del(redis_tracked_requests_key)
     end
 

data/lib/mem_health/version.rb

@@ -1,3 +1,3 @@
 module MemHealth
-  VERSION = '0.1.2'
+  VERSION = '0.1.4'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: memhealth
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.4
 platform: ruby
 authors:
 - Klemen Nagode
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-10-01 00:00:00.000000000 Z
+date: 2026-06-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails