brainzlab 0.1.2 → 0.1.3

data/lib/brainzlab/devtools/middleware/debug_panel.rb

@@ -4,7 +4,7 @@ module BrainzLab
   module DevTools
     module Middleware
       class DebugPanel
-        HTML_CONTENT_TYPE = "text/html"
+        HTML_CONTENT_TYPE = 'text/html'
 
         def initialize(app)
           @app = app
@@ -38,24 +38,24 @@ module BrainzLab
           return false unless DevTools.debug_panel_enabled?
           return false unless DevTools.allowed_environment?
           return false unless DevTools.allowed_ip?(extract_ip(env))
-          return false if asset_request?(env["PATH_INFO"])
-          return false if devtools_asset_request?(env["PATH_INFO"])
+          return false if asset_request?(env['PATH_INFO'])
+          return false if devtools_asset_request?(env['PATH_INFO'])
           return false if turbo_stream_request?(env)
 
           true
         end
 
         def extract_ip(env)
-          forwarded = env["HTTP_X_FORWARDED_FOR"]
-          return forwarded.split(",").first.strip if forwarded
+          forwarded = env['HTTP_X_FORWARDED_FOR']
+          return forwarded.split(',').first.strip if forwarded
 
-          env["REMOTE_ADDR"]
+          env['REMOTE_ADDR']
         end
 
         def injectable_response?(status, headers)
           return false unless status == 200
 
-          content_type = headers["Content-Type"]
+          content_type = headers['Content-Type']
           return false unless content_type
 
           content_type.include?(HTML_CONTENT_TYPE)
@@ -78,11 +78,11 @@ module BrainzLab
         end
 
         def turbo_stream_request?(env)
-          accept = env["HTTP_ACCEPT"] || ""
-          accept.include?("text/vnd.turbo-stream.html")
+          accept = env['HTTP_ACCEPT'] || ''
+          accept.include?('text/vnd.turbo-stream.html')
         end
 
-        def inject_panel(body, env, status, headers)
+        def inject_panel(body, _env, status, headers)
           # Collect all response body parts
           full_body = collect_body(body)
 
@@ -91,25 +91,25 @@ module BrainzLab
           data[:response] = {
             status: status,
             headers: headers.to_h,
-            content_type: headers["Content-Type"]
+            content_type: headers['Content-Type']
           }
 
           # Render panel HTML
           panel_html = @renderer.render(data)
 
           # Inject before </body>
-          if full_body.include?("</body>")
-            full_body = full_body.sub("</body>", "#{panel_html}</body>")
-          else
-            # If no </body> tag, append at the end
-            full_body = "#{full_body}#{panel_html}"
-          end
+          full_body = if full_body.include?('</body>')
+                        full_body.sub('</body>', "#{panel_html}</body>")
+                      else
+                        # If no </body> tag, append at the end
+                        "#{full_body}#{panel_html}"
+                      end
 
           [full_body]
         end
 
         def collect_body(body)
-          full_body = +""
+          full_body = +''
           body.each { |part| full_body << part }
           body.close if body.respond_to?(:close)
           full_body
@@ -117,7 +117,7 @@ module BrainzLab
 
         def update_content_length(headers, body)
           headers = headers.to_h.dup
-          headers["Content-Length"] = body.sum(&:bytesize).to_s
+          headers['Content-Length'] = body.sum(&:bytesize).to_s
           headers
         end
       end

data/lib/brainzlab/devtools/middleware/error_page.rb

@@ -19,7 +19,7 @@ module BrainzLab
             if status >= 400 && html_response?(headers) && !json_request?(env)
               # Check if this looks like Rails' default error page
               body_content = collect_body(body)
-              if body_content.include?("Action Controller: Exception caught") || body_content.include?("background: #C00")
+              if body_content.include?('Action Controller: Exception caught') || body_content.include?('background: #C00')
                 # Extract exception info from the page
                 exception_info = extract_exception_from_html(body_content)
                 if exception_info
@@ -30,41 +30,41 @@ module BrainzLab
             end
 
             [status, headers, body]
-          rescue Exception => exception
+          rescue Exception => e
             # Don't intercept if request wants JSON
-            return raise_exception(exception) if json_request?(env)
+            return raise_exception(e) if json_request?(env)
 
             # Still capture to Reflex if available
-            capture_to_reflex(exception)
+            capture_to_reflex(e)
 
             # Collect debug data and render branded error page
-            data = collect_debug_data(env, exception)
-            render_error_page(exception, data)
+            data = collect_debug_data(env, e)
+            render_error_page(e, data)
           end
         end
 
         def html_response?(headers)
           # Handle both uppercase and lowercase header names
-          content_type = headers["Content-Type"] || headers["content-type"] || ""
-          content_type.to_s.downcase.include?("text/html")
+          content_type = headers['Content-Type'] || headers['content-type'] || ''
+          content_type.to_s.downcase.include?('text/html')
         end
 
         def extract_exception_from_html(body)
           # Try to extract exception class and message from Rails error page
-          if match = body.match(/<h1>([^<]+)<\/h1>/)
+          if (match = body.match(%r{<h1>([^<]+)</h1>}))
             error_title = match[1]
             # Extract the exception message from the page
-            if msg_match = body.match(/<pre[^>]*>([^<]+)<\/pre>/)
+            if (msg_match = body.match(%r{<pre[^>]*>([^<]+)</pre>}))
               error_message = msg_match[1]
             end
 
             # Try to extract backtrace from Rails 8 format
             # Format: <a class="trace-frames ...">path/to/file.rb:123:in 'method'</a>
             backtrace = []
-            body.scan(/<a[^>]*class="trace-frames[^"]*"[^>]*>\s*([^<]+)\s*<\/a>/m) do |trace_match|
+            body.scan(%r{<a[^>]*class="trace-frames[^"]*"[^>]*>\s*([^<]+)\s*</a>}m) do |trace_match|
               line = trace_match[0].strip
               # Decode HTML entities
-              line = line.gsub("&#39;", "'").gsub("&quot;", '"').gsub("&amp;", "&").gsub("&lt;", "<").gsub("&gt;", ">")
+              line = line.gsub('&#39;', "'").gsub('&quot;', '"').gsub('&amp;', '&').gsub('&lt;', '<').gsub('&gt;', '>')
               backtrace << line unless line.empty?
             end
 
@@ -78,12 +78,13 @@ module BrainzLab
 
         def decode_html_entities(str)
           return str unless str
-          str.gsub("&#39;", "'")
-             .gsub("&quot;", '"')
-             .gsub("&amp;", "&")
-             .gsub("&lt;", "<")
-             .gsub("&gt;", ">")
-             .gsub("&nbsp;", " ")
+
+          str.gsub('&#39;', "'")
+             .gsub('&quot;', '"')
+             .gsub('&amp;', '&')
+             .gsub('&lt;', '<')
+             .gsub('&gt;', '>')
+             .gsub('&nbsp;', ' ')
         end
 
         def collect_debug_data_from_info(env, info)
@@ -127,9 +128,9 @@ module BrainzLab
           [
             500,
             {
-              "Content-Type" => "text/html; charset=utf-8",
-              "Content-Length" => html.bytesize.to_s,
-              "X-Content-Type-Options" => "nosniff"
+              'Content-Type' => 'text/html; charset=utf-8',
+              'Content-Length' => html.bytesize.to_s,
+              'X-Content-Type-Options' => 'nosniff'
             },
             [html]
           ]
@@ -138,7 +139,7 @@ module BrainzLab
         private
 
         def collect_body(body)
-          full_body = +""
+          full_body = +''
           body.each { |part| full_body << part }
           body.close if body.respond_to?(:close)
           full_body
@@ -153,19 +154,19 @@ module BrainzLab
         end
 
         def extract_ip(env)
-          forwarded = env["HTTP_X_FORWARDED_FOR"]
-          return forwarded.split(",").first.strip if forwarded
+          forwarded = env['HTTP_X_FORWARDED_FOR']
+          return forwarded.split(',').first.strip if forwarded
 
-          env["REMOTE_ADDR"]
+          env['REMOTE_ADDR']
         end
 
         def json_request?(env)
-          accept = env["HTTP_ACCEPT"] || ""
-          content_type = env["CONTENT_TYPE"] || ""
+          accept = env['HTTP_ACCEPT'] || ''
+          content_type = env['CONTENT_TYPE'] || ''
 
-          accept.include?("application/json") ||
-            content_type.include?("application/json") ||
-            env["HTTP_X_REQUESTED_WITH"] == "XMLHttpRequest"
+          accept.include?('application/json') ||
+            content_type.include?('application/json') ||
+            env['HTTP_X_REQUESTED_WITH'] == 'XMLHttpRequest'
         end
 
         def capture_to_reflex(exception)
@@ -199,9 +200,9 @@ module BrainzLab
           request = defined?(ActionDispatch::Request) ? ActionDispatch::Request.new(env) : nil
 
           {
-            method: request&.request_method || env["REQUEST_METHOD"],
-            path: request&.path || env["PATH_INFO"],
-            url: request&.url || env["REQUEST_URI"],
+            method: request&.request_method || env['REQUEST_METHOD'],
+            path: request&.path || env['PATH_INFO'],
+            url: request&.url || env['REQUEST_URI'],
             params: scrub_params(context&.request_params || extract_params(env)),
             headers: extract_headers(env),
             session: {}
@@ -228,8 +229,8 @@ module BrainzLab
         def extract_headers(env)
           headers = {}
           env.each do |key, value|
-            if key.start_with?("HTTP_")
-              header_name = key.sub("HTTP_", "").split("_").map(&:capitalize).join("-")
+            if key.start_with?('HTTP_')
+              header_name = key.sub('HTTP_', '').split('_').map(&:capitalize).join('-')
               headers[header_name] = value
             end
           end
@@ -243,7 +244,7 @@ module BrainzLab
 
           params.transform_values.with_index do |(key, value), _|
             if scrub_fields.include?(key.to_s.downcase)
-              "[FILTERED]"
+              '[FILTERED]'
             elsif value.is_a?(Hash)
               scrub_params(value)
             else
@@ -277,14 +278,14 @@ module BrainzLab
         def in_app_frame?(file)
           return false unless file
 
-          file.include?("/app/") && !file.include?("/vendor/") && !file.include?("/gems/")
+          file.include?('/app/') && !file.include?('/vendor/') && !file.include?('/gems/')
         end
 
         def extract_source_from_backtrace(backtrace_lines)
           return nil if backtrace_lines.empty?
 
           # Find the first in-app frame
-          target_line = backtrace_lines.find { |line| in_app_frame?(line.split(":").first) }
+          target_line = backtrace_lines.find { |line| in_app_frame?(line.split(':').first) }
           target_line ||= backtrace_lines.first
 
           match = target_line.match(/\A(.+):(\d+)/)
@@ -317,7 +318,7 @@ module BrainzLab
           return nil unless exception.backtrace&.any?
 
           # Find the first in-app frame (application code, not gems/framework)
-          target_line = exception.backtrace.find { |line| in_app_frame?(line.split(":").first) }
+          target_line = exception.backtrace.find { |line| in_app_frame?(line.split(':').first) }
           # Fall back to first frame if no in-app frame found
           target_line ||= exception.backtrace.first
 
@@ -349,10 +350,10 @@ module BrainzLab
 
         def collect_environment_info
           {
-            rails_version: defined?(Rails::VERSION::STRING) ? Rails::VERSION::STRING : "N/A",
+            rails_version: defined?(::Rails::VERSION::STRING) ? ::Rails::VERSION::STRING : 'N/A',
             ruby_version: RUBY_VERSION,
             env: BrainzLab.configuration.environment,
-            server: ENV["SERVER_SOFTWARE"] || "Unknown",
+            server: ENV['SERVER_SOFTWARE'] || 'Unknown',
             pid: Process.pid
           }
         end
@@ -363,9 +364,9 @@ module BrainzLab
           [
             500,
             {
-              "Content-Type" => "text/html; charset=utf-8",
-              "Content-Length" => html.bytesize.to_s,
-              "X-Content-Type-Options" => "nosniff"
+              'Content-Type' => 'text/html; charset=utf-8',
+              'Content-Length' => html.bytesize.to_s,
+              'X-Content-Type-Options' => 'nosniff'
             },
             [html]
           ]

data/lib/brainzlab/devtools/renderers/debug_panel_renderer.rb

@@ -1,15 +1,15 @@
 # frozen_string_literal: true
 
-require "erb"
-require "cgi"
-require "json"
+require 'erb'
+require 'cgi'
+require 'json'
 
 module BrainzLab
   module DevTools
     module Renderers
       class DebugPanelRenderer
         def initialize
-          @template_path = File.join(DevTools::ASSETS_PATH, "templates", "debug_panel.html.erb")
+          @template_path = File.join(DevTools::ASSETS_PATH, 'templates', 'debug_panel.html.erb')
           # Cache compiled ERB template to avoid file I/O on every request
           @cached_erb = nil
           @template_mtime = nil
@@ -47,7 +47,7 @@ module BrainzLab
         end
 
         def json_pretty(obj)
-          return "" if obj.nil? || (obj.respond_to?(:empty?) && obj.empty?)
+          return '' if obj.nil? || (obj.respond_to?(:empty?) && obj.empty?)
 
           JSON.pretty_generate(obj)
         rescue StandardError
@@ -55,14 +55,14 @@ module BrainzLab
         end
 
         def truncate(text, length = 80)
-          return "" unless text
+          return '' unless text
 
           text = text.to_s
           text.length > length ? "#{text[0...length]}..." : text
         end
 
         def format_duration(ms)
-          return "0ms" unless ms
+          return '0ms' unless ms
 
           if ms >= 1000
             "#{(ms / 1000.0).round(2)}s"
@@ -72,78 +72,82 @@ module BrainzLab
         end
 
         def duration_class(ms)
-          return "" unless ms
+          return '' unless ms
 
           if ms > 1000
-            "very-slow"
+            'very-slow'
           elsif ms > 500
-            "slow"
+            'slow'
           elsif ms > 200
-            "moderate"
+            'moderate'
           else
-            ""
+            ''
           end
         end
 
         def query_duration_class(ms)
-          return "" unless ms
+          return '' unless ms
 
           if ms > 100
-            "very-slow"
+            'very-slow'
           elsif ms > 50
-            "slow"
+            'slow'
           elsif ms > 10
-            "moderate"
+            'moderate'
           else
-            ""
+            ''
           end
         end
 
         def status_class(status)
           case status
-          when 200..299 then "success"
-          when 300..399 then "redirect"
-          when 400..499 then "client-error"
-          when 500..599 then "server-error"
-          else ""
+          when 200..299 then 'success'
+          when 300..399 then 'redirect'
+          when 400..499 then 'client-error'
+          when 500..599 then 'server-error'
+          else ''
           end
         end
 
         def log_level_class(level)
           case level.to_s.downcase
-          when "error", "fatal" then "error"
-          when "warn", "warning" then "warning"
-          when "info" then "info"
-          when "debug" then "debug"
-          else ""
+          when 'error', 'fatal' then 'error'
+          when 'warn', 'warning' then 'warning'
+          when 'info' then 'info'
+          when 'debug' then 'debug'
+          else ''
           end
         end
 
         def format_timestamp(time)
-          return "" unless time
+          return '' unless time
 
-          time.strftime("%H:%M:%S.%L")
+          time.strftime('%H:%M:%S.%L')
         end
 
         def memory_class(delta_mb)
-          return "" unless delta_mb
+          return '' unless delta_mb
 
           if delta_mb > 50
-            "high"
+            'high'
           elsif delta_mb > 20
-            "moderate"
+            'moderate'
           else
-            ""
+            ''
           end
         end
 
         # Cache compiled ERB template, reloading only if file changed (dev mode)
         def cached_erb
-          current_mtime = File.mtime(@template_path) rescue nil
+          current_mtime = begin
+            File.mtime(@template_path)
+          rescue StandardError
+            nil
+          end
 
           if @cached_erb.nil? || (current_mtime && current_mtime != @template_mtime)
             template = File.read(@template_path)
-            @cached_erb = ERB.new(template, trim_mode: "-")
+            @cached_erb = ERB.new(template, trim_mode: '-')
             @template_mtime = current_mtime
           end
 

data/lib/brainzlab/devtools/renderers/error_page_renderer.rb

@@ -1,14 +1,14 @@
 # frozen_string_literal: true
 
-require "erb"
-require "cgi"
+require 'erb'
+require 'cgi'
 
 module BrainzLab
   module DevTools
     module Renderers
       class ErrorPageRenderer
         def initialize
-          @template_path = File.join(DevTools::ASSETS_PATH, "templates", "error_page.html.erb")
+          @template_path = File.join(DevTools::ASSETS_PATH, 'templates', 'error_page.html.erb')
           @cached_erb = nil
           @template_mtime = nil
         end
@@ -40,10 +40,10 @@ module BrainzLab
         end
 
         def format_params(params, indent = 0)
-          return "" if params.nil? || params.empty?
+          return '' if params.nil? || params.empty?
 
           lines = []
-          prefix = "  " * indent
+          prefix = '  ' * indent
 
           params.each do |key, value|
             if value.is_a?(Hash)
@@ -60,13 +60,13 @@ module BrainzLab
         end
 
         def truncate(text, length = 100)
-          return "" unless text
+          return '' unless text
 
           text.length > length ? "#{text[0...length]}..." : text
         end
 
         def time_ago(time)
-          return "unknown" unless time
+          return 'unknown' unless time
 
           seconds = Time.now.utc - time
           case seconds
@@ -78,11 +78,15 @@ module BrainzLab
 
         # Cache compiled ERB template, reloading only if file changed
         def cached_erb
-          current_mtime = File.mtime(@template_path) rescue nil
+          current_mtime = begin
+            File.mtime(@template_path)
+          rescue StandardError
+            nil
+          end
 
           if @cached_erb.nil? || (current_mtime && current_mtime != @template_mtime)
             template = File.read(@template_path)
-            @cached_erb = ERB.new(template, trim_mode: "-")
+            @cached_erb = ERB.new(template, trim_mode: '-')
             @template_mtime = current_mtime
           end
 

data/lib/brainzlab/devtools.rb

@@ -1,16 +1,16 @@
 # frozen_string_literal: true
 
-require_relative "devtools/data/collector"
-require_relative "devtools/middleware/asset_server"
-require_relative "devtools/middleware/database_handler"
-require_relative "devtools/middleware/error_page"
-require_relative "devtools/middleware/debug_panel"
-require_relative "devtools/renderers/error_page_renderer"
-require_relative "devtools/renderers/debug_panel_renderer"
+require_relative 'devtools/data/collector'
+require_relative 'devtools/middleware/asset_server'
+require_relative 'devtools/middleware/database_handler'
+require_relative 'devtools/middleware/error_page'
+require_relative 'devtools/middleware/debug_panel'
+require_relative 'devtools/renderers/error_page_renderer'
+require_relative 'devtools/renderers/debug_panel_renderer'
 
 module BrainzLab
   module DevTools
-    ASSETS_PATH = File.expand_path("devtools/assets", __dir__)
+    ASSETS_PATH = File.expand_path('devtools/assets', __dir__)
 
     class << self
       def enabled?
@@ -33,7 +33,7 @@ module BrainzLab
 
       def allowed_ip?(request_ip)
         # Skip IP checking in development - environment check is enough
-        return true if BrainzLab.configuration.environment == "development"
+        return true if BrainzLab.configuration.environment == 'development'
 
         return true if BrainzLab.configuration.devtools_allowed_ips.empty?
 
@@ -42,7 +42,7 @@ module BrainzLab
 
         # Check CIDR ranges
         allowed_ips.any? do |ip|
-          if ip.include?("/")
+          if ip.include?('/')
             ip_in_cidr?(request_ip, ip)
           else
             ip == request_ip
@@ -65,7 +65,7 @@ module BrainzLab
       private
 
       def ip_in_cidr?(ip, cidr)
-        require "ipaddr"
+        require 'ipaddr'
         IPAddr.new(cidr).include?(IPAddr.new(ip))
       rescue IPAddr::InvalidAddressError
         false

data/lib/brainzlab/flux/buffer.rb

@@ -75,7 +75,7 @@ module BrainzLab
         return if events.empty? && metrics.empty?
 
         @client.send_batch(events: events, metrics: metrics)
-      rescue => e
+      rescue StandardError => e
         BrainzLab.debug("[Flux] Batch send failed: #{e.message}")
       end
 
@@ -84,8 +84,8 @@ module BrainzLab
           loop do
             sleep FLUSH_INTERVAL
             begin
-              flush! if size > 0
-            rescue => e
+              flush! if size.positive?
+            rescue StandardError => e
               BrainzLab.debug("[Flux] Flush thread error: #{e.message}")
             end
           end