Optimize debug check by caching environment variable lookup

Copilot · thedave42 · Copilot · commit fb55e2ed12af · 2025-08-22T16:19:27.000Z
Co-authored-by: thedave42 &lt;50186003+thedave42@users.noreply.github.com&gt;
diff --git a/api/ruby/find-inactive-members/find_inactive_members.rb b/api/ruby/find-inactive-members/find_inactive_members.rb
@@ -15,6 +15,7 @@ def initialize(app, options = {})
     @hour_start_time = Time.now
     @last_request_time = Time.now
     @mutex = Mutex.new
+    @debug_enabled = !ENV['THROTTLE_DEBUG'].nil? && !ENV['THROTTLE_DEBUG'].empty?
   end
 
   def call(env)
@@ -60,9 +61,11 @@ def throttle_request
 
   def log_throttle_status
     # This method can be called for detailed debugging if needed
+    return unless @debug_enabled
+    
     elapsed_hour = Time.now - @hour_start_time
     rate_per_hour = elapsed_hour > 0 ? (@request_count / elapsed_hour * 3600).round(1) : 0
-    $stderr.print "Throttle debug: #{@request_count} requests in last #{elapsed_hour.round(1)}s (#{rate_per_hour}/hour rate)\n" if ENV['THROTTLE_DEBUG']
+    $stderr.print "Throttle debug: #{@request_count} requests in last #{elapsed_hour.round(1)}s (#{rate_per_hour}/hour rate)\n"
   end
 end
 
