brainzlab 0.1.2 → 0.1.3

data/lib/brainzlab/recall/client.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-require "net/http"
-require "uri"
-require "json"
+require 'net/http'
+require 'uri'
+require 'json'
 
 module BrainzLab
   module Recall
@@ -18,14 +18,14 @@ module BrainzLab
       def send_log(log_entry)
         return unless @config.recall_enabled && @config.valid?
 
-        post("/api/v1/log", log_entry)
+        post('/api/v1/log', log_entry)
       end
 
       def send_batch(log_entries)
         return unless @config.recall_enabled && @config.valid?
         return if log_entries.empty?
 
-        post("/api/v1/logs", { logs: log_entries })
+        post('/api/v1/logs', { logs: log_entries })
       end
 
       private
@@ -33,9 +33,9 @@ module BrainzLab
       def post(path, body)
         uri = URI.join(@config.recall_url, path)
         request = Net::HTTP::Post.new(uri)
-        request["Content-Type"] = "application/json"
-        request["Authorization"] = "Bearer #{@config.secret_key}"
-        request["User-Agent"] = "brainzlab-sdk-ruby/#{BrainzLab::VERSION}"
+        request['Content-Type'] = 'application/json'
+        request['Authorization'] = "Bearer #{@config.secret_key}"
+        request['User-Agent'] = "brainzlab-sdk-ruby/#{BrainzLab::VERSION}"
         request.body = JSON.generate(body)
 
         execute_with_retry(uri, request)
@@ -48,7 +48,7 @@ module BrainzLab
         retries = 0
         begin
           http = Net::HTTP.new(uri.host, uri.port)
-          http.use_ssl = uri.scheme == "https"
+          http.use_ssl = uri.scheme == 'https'
           http.open_timeout = 5
           http.read_timeout = 10
 
@@ -56,7 +56,11 @@ module BrainzLab
 
           case response.code.to_i
           when 200..299
-            JSON.parse(response.body) rescue {}
+            begin
+              JSON.parse(response.body)
+            rescue StandardError
+              {}
+            end
           when 429, 500..599
             raise RetryableError, "Server error: #{response.code}"
           else

data/lib/brainzlab/recall/logger.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "logger"
+require 'logger'
 
 module BrainzLab
   module Recall
@@ -12,16 +12,14 @@ module BrainzLab
         @service_name = service_name
         @broadcast_to = broadcast_to
         @level = ::Logger::DEBUG
-        @formatter = proc { |severity, _time, _progname, msg| msg }
+        @formatter = proc { |_severity, _time, _progname, msg| msg }
       end
 
-      def add(severity, message = nil, progname = nil, &block)
+      def add(severity, message = nil, progname = nil)
         severity ||= ::Logger::UNKNOWN
 
         # Handle block-based messages
-        if message.nil? && block_given?
-          message = yield
-        end
+        message = yield if message.nil? && block_given?
 
         # Handle progname as message (standard Logger behavior)
         if message.nil?
@@ -52,28 +50,28 @@ module BrainzLab
         true
       end
 
-      def debug(message = nil, &block)
-        add(::Logger::DEBUG, message, &block)
+      def debug(message = nil, &)
+        add(::Logger::DEBUG, message, &)
       end
 
-      def info(message = nil, &block)
-        add(::Logger::INFO, message, &block)
+      def info(message = nil, &)
+        add(::Logger::INFO, message, &)
       end
 
-      def warn(message = nil, &block)
-        add(::Logger::WARN, message, &block)
+      def warn(message = nil, &)
+        add(::Logger::WARN, message, &)
       end
 
-      def error(message = nil, &block)
-        add(::Logger::ERROR, message, &block)
+      def error(message = nil, &)
+        add(::Logger::ERROR, message, &)
       end
 
-      def fatal(message = nil, &block)
-        add(::Logger::FATAL, message, &block)
+      def fatal(message = nil, &)
+        add(::Logger::FATAL, message, &)
       end
 
-      def unknown(message = nil, &block)
-        add(::Logger::UNKNOWN, message, &block)
+      def unknown(message = nil, &)
+        add(::Logger::UNKNOWN, message, &)
       end
 
       # Rails compatibility methods

data/lib/brainzlab/recall/provisioner.rb

@@ -1,14 +1,14 @@
 # frozen_string_literal: true
 
-require "net/http"
-require "uri"
-require "json"
-require "fileutils"
+require 'net/http'
+require 'uri'
+require 'json'
+require 'fileutils'
 
 module BrainzLab
   module Recall
     class Provisioner
-      CACHE_DIR = ENV.fetch("BRAINZLAB_CACHE_DIR") { File.join(Dir.home, ".brainzlab") }
+      CACHE_DIR = ENV.fetch('BRAINZLAB_CACHE_DIR') { File.join(Dir.home, '.brainzlab') }
 
       def initialize(config)
         @config = config
@@ -38,19 +38,19 @@ module BrainzLab
 
       def should_provision?
         if @config.debug
-          log_debug("Checking provision conditions:")
+          log_debug('Checking provision conditions:')
           log_debug("  recall_auto_provision: #{@config.recall_auto_provision}")
           log_debug("  app_name: '#{@config.app_name}'")
-          log_debug("  secret_key set: #{@config.secret_key.to_s.strip.length > 0}")
-          log_debug("  recall_master_key set: #{@config.recall_master_key.to_s.strip.length > 0}")
+          log_debug("  secret_key set: #{@config.secret_key.to_s.strip.length.positive?}")
+          log_debug("  recall_master_key set: #{@config.recall_master_key.to_s.strip.length.positive?}")
         end
 
         return false unless @config.recall_auto_provision
-        return false unless @config.app_name.to_s.strip.length > 0
-        return false if @config.secret_key.to_s.strip.length > 0
-        return false unless @config.recall_master_key.to_s.strip.length > 0
+        return false unless @config.app_name.to_s.strip.length.positive?
+        return false if @config.secret_key.to_s.strip.length.positive?
+        return false unless @config.recall_master_key.to_s.strip.length.positive?
 
-        log_debug("Will provision Recall project") if @config.debug
+        log_debug('Will provision Recall project') if @config.debug
         true
       end
 
@@ -65,9 +65,9 @@ module BrainzLab
       def provision_project
         uri = URI.parse("#{@config.recall_url}/api/v1/projects/provision")
         request = Net::HTTP::Post.new(uri)
-        request["Content-Type"] = "application/json"
-        request["X-Master-Key"] = @config.recall_master_key
-        request["User-Agent"] = "brainzlab-sdk-ruby/#{BrainzLab::VERSION}"
+        request['Content-Type'] = 'application/json'
+        request['X-Master-Key'] = @config.recall_master_key
+        request['User-Agent'] = "brainzlab-sdk-ruby/#{BrainzLab::VERSION}"
         request.body = JSON.generate({ name: @config.app_name })
 
         response = execute(uri, request)
@@ -114,7 +114,7 @@ module BrainzLab
 
       def execute(uri, request)
         http = Net::HTTP.new(uri.host, uri.port)
-        http.use_ssl = uri.scheme == "https"
+        http.use_ssl = uri.scheme == 'https'
         http.open_timeout = 5
         http.read_timeout = 10
         http.request(request)

data/lib/brainzlab/recall.rb

@@ -1,9 +1,9 @@
 # frozen_string_literal: true
 
-require_relative "recall/client"
-require_relative "recall/buffer"
-require_relative "recall/logger"
-require_relative "recall/provisioner"
+require_relative 'recall/client'
+require_relative 'recall/buffer'
+require_relative 'recall/logger'
+require_relative 'recall/provisioner'
 
 module BrainzLab
   module Recall
@@ -44,9 +44,7 @@ module BrainzLab
 
       def ensure_provisioned!
         config = BrainzLab.configuration
-        if config.debug
-          puts "[BrainzLab::Debug] Recall.ensure_provisioned! called, @provisioned=#{@provisioned}"
-        end
+        puts "[BrainzLab::Debug] Recall.ensure_provisioned! called, @provisioned=#{@provisioned}" if config.debug
 
         return if @provisioned
 
@@ -63,7 +61,7 @@ module BrainzLab
         result = yield
         duration_ms = ((Process.clock_gettime(Process::CLOCK_MONOTONIC) - start) * 1000).round(1)
 
-        info("#{label} (#{duration_ms}ms)", **data.merge(duration_ms: duration_ms))
+        info("#{label} (#{duration_ms}ms)", **data, duration_ms: duration_ms)
         result
       end
 
@@ -131,11 +129,11 @@ module BrainzLab
         case obj
         when Hash
           obj.each_with_object({}) do |(key, value), result|
-            if should_scrub?(key, fields)
-              result[key] = "[FILTERED]"
-            else
-              result[key] = deep_scrub(value, fields)
-            end
+            result[key] = if should_scrub?(key, fields)
+                            '[FILTERED]'
+                          else
+                            deep_scrub(value, fields)
+                          end
           end
         when Array
           obj.map { |item| deep_scrub(item, fields) }

data/lib/brainzlab/reflex/breadcrumbs.rb

@@ -10,7 +10,7 @@ module BrainzLab
         @mutex = Mutex.new
       end
 
-      def add(message:, category: "default", level: :info, data: nil)
+      def add(message:, category: 'default', level: :info, data: nil)
         crumb = {
           timestamp: Time.now.utc.iso8601(3),
           message: message.to_s,
@@ -43,7 +43,7 @@ module BrainzLab
         Context.current.breadcrumbs
       end
 
-      def add_breadcrumb(message, category: "default", level: :info, data: nil)
+      def add_breadcrumb(message, category: 'default', level: :info, data: nil)
         breadcrumbs.add(message: message, category: category, level: level, data: data)
       end
 

data/lib/brainzlab/reflex/client.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-require "net/http"
-require "uri"
-require "json"
+require 'net/http'
+require 'uri'
+require 'json'
 
 module BrainzLab
   module Reflex
@@ -17,14 +17,14 @@ module BrainzLab
       def send_error(payload)
         return unless @config.reflex_enabled && @config.reflex_valid?
 
-        post("/api/v1/errors", payload)
+        post('/api/v1/errors', payload)
       end
 
       def send_batch(payloads)
         return unless @config.reflex_enabled && @config.reflex_valid?
         return if payloads.empty?
 
-        post("/api/v1/errors/batch", { errors: payloads })
+        post('/api/v1/errors/batch', { errors: payloads })
       end
 
       private
@@ -32,9 +32,9 @@ module BrainzLab
       def post(path, body)
         uri = URI.join(@config.reflex_url, path)
         request = Net::HTTP::Post.new(uri)
-        request["Content-Type"] = "application/json"
-        request["Authorization"] = "Bearer #{@config.reflex_auth_key}"
-        request["User-Agent"] = "brainzlab-sdk-ruby/#{BrainzLab::VERSION}"
+        request['Content-Type'] = 'application/json'
+        request['Authorization'] = "Bearer #{@config.reflex_auth_key}"
+        request['User-Agent'] = "brainzlab-sdk-ruby/#{BrainzLab::VERSION}"
         request.body = JSON.generate(body)
 
         execute_with_retry(uri, request)
@@ -47,7 +47,7 @@ module BrainzLab
         retries = 0
         begin
           http = Net::HTTP.new(uri.host, uri.port)
-          http.use_ssl = uri.scheme == "https"
+          http.use_ssl = uri.scheme == 'https'
           http.open_timeout = 5
           http.read_timeout = 10
 
@@ -55,7 +55,11 @@ module BrainzLab
 
           case response.code.to_i
           when 200..299
-            JSON.parse(response.body) rescue {}
+            begin
+              JSON.parse(response.body)
+            rescue StandardError
+              {}
+            end
           when 429, 500..599
             raise RetryableError, "Server error: #{response.code}"
           else

data/lib/brainzlab/reflex/provisioner.rb

@@ -1,14 +1,14 @@
 # frozen_string_literal: true
 
-require "net/http"
-require "uri"
-require "json"
-require "fileutils"
+require 'net/http'
+require 'uri'
+require 'json'
+require 'fileutils'
 
 module BrainzLab
   module Reflex
     class Provisioner
-      CACHE_DIR = ENV.fetch("BRAINZLAB_CACHE_DIR") { File.join(Dir.home, ".brainzlab") }
+      CACHE_DIR = ENV.fetch('BRAINZLAB_CACHE_DIR') { File.join(Dir.home, '.brainzlab') }
 
       def initialize(config)
         @config = config
@@ -38,10 +38,10 @@ module BrainzLab
 
       def should_provision?
         return false unless @config.reflex_auto_provision
-        return false unless @config.app_name.to_s.strip.length > 0
+        return false unless @config.app_name.to_s.strip.length.positive?
         # Only skip if reflex_api_key is already set (not secret_key, which may be for Recall)
-        return false if @config.reflex_api_key.to_s.strip.length > 0
-        return false unless @config.reflex_master_key.to_s.strip.length > 0
+        return false if @config.reflex_api_key.to_s.strip.length.positive?
+        return false unless @config.reflex_master_key.to_s.strip.length.positive?
 
         true
       end
@@ -49,9 +49,9 @@ module BrainzLab
       def provision_project
         uri = URI.parse("#{@config.reflex_url}/api/v1/projects/provision")
         request = Net::HTTP::Post.new(uri)
-        request["Content-Type"] = "application/json"
-        request["X-Master-Key"] = @config.reflex_master_key
-        request["User-Agent"] = "brainzlab-sdk-ruby/#{BrainzLab::VERSION}"
+        request['Content-Type'] = 'application/json'
+        request['X-Master-Key'] = @config.reflex_master_key
+        request['User-Agent'] = "brainzlab-sdk-ruby/#{BrainzLab::VERSION}"
         request.body = JSON.generate({ name: @config.app_name })
 
         response = execute(uri, request)
@@ -100,7 +100,7 @@ module BrainzLab
 
       def execute(uri, request)
         http = Net::HTTP.new(uri.host, uri.port)
-        http.use_ssl = uri.scheme == "https"
+        http.use_ssl = uri.scheme == 'https'
         http.open_timeout = 5
         http.read_timeout = 10
         http.request(request)

data/lib/brainzlab/reflex.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-require_relative "reflex/client"
-require_relative "reflex/breadcrumbs"
-require_relative "reflex/provisioner"
+require_relative 'reflex/client'
+require_relative 'reflex/breadcrumbs'
+require_relative 'reflex/provisioner'
 
 module BrainzLab
   module Reflex
@@ -158,7 +158,7 @@ module BrainzLab
 
         payload = {
           timestamp: Time.now.utc.iso8601(3),
-          error_class: "Message",
+          error_class: 'Message',
           message: message.to_s,
           level: level.to_s,
 
@@ -247,17 +247,17 @@ module BrainzLab
         # - "path/to/file.rb:42"                   (no method)
         if line =~ /\A(.+):(\d+):in [`']([^']+)'?\z/
           {
-            file: $1,
-            line: $2.to_i,
-            function: $3,
-            in_app: in_app_frame?($1)
+            file: ::Regexp.last_match(1),
+            line: ::Regexp.last_match(2).to_i,
+            function: ::Regexp.last_match(3),
+            in_app: in_app_frame?(::Regexp.last_match(1))
           }
         elsif line =~ /\A(.+):(\d+)\z/
           {
-            file: $1,
-            line: $2.to_i,
+            file: ::Regexp.last_match(1),
+            line: ::Regexp.last_match(2).to_i,
             function: nil,
-            in_app: in_app_frame?($1)
+            in_app: in_app_frame?(::Regexp.last_match(1))
           }
         else
           # Still store file for display even if format is unexpected
@@ -267,14 +267,14 @@ module BrainzLab
 
       def in_app_frame?(path)
         return false if path.nil?
-        return false if path.include?("vendor/")
-        return false if path.include?("/gems/")
-        return false if path.include?("/ruby/")
+        return false if path.include?('vendor/')
+        return false if path.include?('/gems/')
+        return false if path.include?('/ruby/')
 
         # Match both relative and absolute paths containing app/ or lib/
-        path.start_with?("app/", "lib/", "./app/", "./lib/") ||
-          path.include?("/app/") ||
-          path.include?("/lib/")
+        path.start_with?('app/', 'lib/', './app/', './lib/') ||
+          path.include?('/app/') ||
+          path.include?('/lib/')
       end
 
       def filter_params(params)
@@ -288,11 +288,11 @@ module BrainzLab
         case obj
         when Hash
           obj.each_with_object({}) do |(key, value), result|
-            if should_filter?(key, fields)
-              result[key] = "[FILTERED]"
-            else
-              result[key] = deep_filter(value, fields)
-            end
+            result[key] = if should_filter?(key, fields)
+                            '[FILTERED]'
+                          else
+                            deep_filter(value, fields)
+                          end
           end
         when Array
           obj.map { |item| deep_filter(item, fields) }
@@ -363,12 +363,12 @@ module BrainzLab
 
       def in_app_line?(line)
         return false if line.nil?
-        return false if line.include?("vendor/")
-        return false if line.include?("/gems/")
+        return false if line.include?('vendor/')
+        return false if line.include?('/gems/')
 
-        line.start_with?("app/", "lib/", "./app/", "./lib/") ||
-          line.include?("/app/") ||
-          line.include?("/lib/")
+        line.start_with?('app/', 'lib/', './app/', './lib/') ||
+          line.include?('/app/') ||
+          line.include?('/lib/')
       end
 
       def normalize_frame_for_fingerprint(frame)
@@ -377,9 +377,9 @@ module BrainzLab
         # Extract file and method, normalize out line numbers
         # "app/models/user.rb:42:in `save'" -> "app/models/user.rb:in `save'"
         if frame =~ /\A(.+):\d+:in `(.+)'\z/
-          "#{$1}:in `#{$2}'"
+          "#{::Regexp.last_match(1)}:in `#{::Regexp.last_match(2)}'"
         elsif frame =~ /\A(.+):\d+\z/
-          $1
+          ::Regexp.last_match(1)
         else
           frame
         end