rails_tracepoint_stack 0.3.4 → 0.3.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a3045ee154f570e6a780af9aff0edfcd18822b3394a0f268ee9a4e07e6d5769f
-  data.tar.gz: 9865f094f0a70ffcb474148aaab7ee6712ac63450326bf7e3385dfc55e1c123f
+  metadata.gz: f67c982828a7ad060513cbb50daf9f88399229a534a43cf12a674571b6ae11ae
+  data.tar.gz: fd80c178bf9de58408e769742d7640b0ccdbcc99a4722ec9ccbd6e68d115ff51
 SHA512:
-  metadata.gz: 6c5482941d4e5119b88133eca986cdb87c490d0ed6e69ed0df5a09ebe1d2efebff0c4abbf9b554abf44cbab80038b96b3bf53cb8c86b1052b967d7ad307573d8
-  data.tar.gz: 699757a6965a6101ee9b80e5039580840a5e34de1271538a286d95c6ea5d2f0e678e666c8f098b2a93df5896868fc821182258f4d07616684bace6857087e7b8
+  metadata.gz: 57b7125a7a242bf59f15096fffeef7de3c8a7543162fa7851566a07256d2ed790d39ff9c9dfcd1f3c3f8ca54714b797ed5f5015f2eb0d6f0adbe57e3ad582412
+  data.tar.gz: cd33cf3e53fe35fe62ba3867cfe8ff56047cceae59e2139c55898fcb0ed0e44d8dccf57a0b66954f1543797c1cb1070b6c68f479f76ce80f0c15282518c50110

data/lib/rails_tracepoint_stack/filter/custom_trace_selector_filter.rb

@@ -4,7 +4,7 @@ module RailsTracepointStack
       def is_a_trace_required_to_watch_by_the_custom_configs?(trace:)
         return false unless RailsTracepointStack.configuration.file_path_to_filter_patterns.any?
 
-        !filter_match_a_custom_pattern_to_be_not_ignored?(trace)
+        filter_match_a_custom_pattern_to_be_not_ignored?(trace)
       end
 
       private

data/lib/rails_tracepoint_stack/filter/rb_config.rb

@@ -2,7 +2,7 @@ module RailsTracepointStack
   module Filter
     class RbConfig
       def self.ruby_lib_path
-        @ruby_lib_path ||= ::RbConfig::CONFIG['rubylibdir']
+        @ruby_lib_path ||= ::RbConfig::CONFIG["rubylibdir"]
       end
     end
   end

data/lib/rails_tracepoint_stack/log_formatter.rb

@@ -1,3 +1,5 @@
+require "json"
+
 module RailsTracepointStack
   module LogFormatter
     def self.message(trace)
@@ -10,17 +12,183 @@ module RailsTracepointStack
     end
 
     def self.text(trace)
-      "called: #{trace.class_name}##{trace.method_name} in #{trace.file_path}:#{trace.line_number} with params: #{trace.params}"
+      "called: #{trace.class_name}##{trace.method_name} in #{trace.file_path}:#{trace.line_number} with params: #{text_value(trace.params)}"
     end
 
     def self.json(trace)
-      {
-        class: trace.class_name,
-        method_name: trace.method_name,
-        path: trace.file_path,
+      JSON.generate(
+        class: stringify(trace.class_name),
+        method_name: stringify(trace.method_name),
+        path: stringify(trace.file_path),
         line: trace.line_number,
-        params: trace.params
-      }.to_json
+        params: safe_value(trace.params)
+      )
+    end
+
+    def self.stringify(value)
+      return nil if value.nil?
+
+      value.to_s
+    rescue SystemStackError, StandardError => error
+      inspect_fallback(value, error)
+    end
+
+    def self.safe_value(value, ancestry = {})
+      case value
+      when nil, true, false, Numeric, String
+        value
+      when Symbol
+        value.to_s
+      when Array
+        safe_array(value, ancestry)
+      when Hash
+        safe_hash(value, ancestry)
+      else
+        safe_json_object(value)
+      end
+    rescue SystemStackError, StandardError => error
+      inspect_fallback(value, error)
+    end
+
+    def self.safe_array(value, ancestry)
+      object_id = value.__id__
+      return recursive_placeholder(value) if ancestry.key?(object_id)
+
+      ancestry[object_id] = true
+      value.map { |item| safe_value(item, ancestry) }
+    ensure
+      ancestry.delete(object_id)
+    end
+
+    def self.safe_hash(value, ancestry)
+      object_id = value.__id__
+      return recursive_placeholder(value) if ancestry.key?(object_id)
+
+      ancestry[object_id] = true
+      value.each_with_object({}) do |(key, item), result|
+        result[safe_hash_key(key, ancestry)] = safe_value(item, ancestry)
+      end
+    ensure
+      ancestry.delete(object_id)
+    end
+
+    def self.safe_hash_key(key, ancestry = {})
+      case key
+      when String
+        key
+      when Symbol, Numeric
+        key.to_s
+      when true, false, nil
+        key.inspect
+      else
+        stringify_hash_key(key, ancestry)
+      end
+    end
+
+    def self.safe_json_object(value)
+      json_value = JSON.parse(JSON.generate(value))
+
+      return json_value unless default_string_representation?(value, json_value)
+
+      safe_object_string(value)
+    rescue SystemStackError, StandardError
+      safe_object_string(value)
+    end
+
+    def self.default_string_representation?(value, json_value)
+      json_value.is_a?(String) &&
+        value.method(:to_s).owner == Kernel &&
+        json_value == value.to_s
+    rescue SystemStackError, StandardError
+      false
+    end
+
+    def self.stringify_hash_key(key, ancestry)
+      normalized_key = safe_value(key, ancestry)
+
+      case normalized_key
+      when String
+        normalized_key
+      when nil
+        "null"
+      when true, false, Numeric
+        normalized_key.to_s
+      else
+        JSON.generate(normalized_key)
+      end
+    rescue SystemStackError, StandardError
+      safe_object_string(key)
+    end
+
+    def self.safe_object_string(value)
+      value.inspect
+    rescue SystemStackError, StandardError => error
+      inspect_fallback(value, error)
+    end
+
+    def self.inspect_fallback(value, error)
+      class_name = value.nil? ? "NilClass" : value.class.to_s
+      "#<#{class_name} unserializable: #{error.class}: #{error.message}>"
+    end
+
+    def self.recursive_placeholder(value)
+      "[recursive #{value.class}]"
+    end
+
+    def self.text_value(value, ancestry = {})
+      case value
+      when nil
+        "nil"
+      when true, false, Numeric
+        value.to_s
+      when String
+        value.inspect
+      when Symbol
+        ":#{value}"
+      when Array
+        text_array(value, ancestry)
+      when Hash
+        text_hash(value, ancestry)
+      else
+        safe_object_string(value)
+      end
+    rescue SystemStackError, StandardError => error
+      inspect_fallback(value, error)
+    end
+
+    def self.text_array(value, ancestry)
+      object_id = value.__id__
+      return recursive_placeholder(value) if ancestry.key?(object_id)
+
+      ancestry[object_id] = true
+      "[#{value.map { |item| text_value(item, ancestry) }.join(", ")}]"
+    ensure
+      ancestry.delete(object_id)
+    end
+
+    def self.text_hash(value, ancestry)
+      object_id = value.__id__
+      return recursive_placeholder(value) if ancestry.key?(object_id)
+
+      ancestry[object_id] = true
+      pairs = value.map do |key, item|
+        "#{text_hash_key(key, ancestry)}=>#{text_value(item, ancestry)}"
+      end
+
+      "{#{pairs.join(", ")}}"
+    ensure
+      ancestry.delete(object_id)
+    end
+
+    def self.text_hash_key(key, ancestry = {})
+      case key
+      when Symbol
+        ":#{key}"
+      when String
+        key.inspect
+      else
+        text_value(key, ancestry)
+      end
     end
   end
 end

data/lib/rails_tracepoint_stack/logger.rb

@@ -5,7 +5,7 @@ module RailsTracepointStack
         RailsTracepointStack.configuration.logger.info(msg)
       else
         # TODO: Add the support to Rails.env for the default filename
-        File.open("log/rails_tracepoint_stack.log", 'a') do |f|
+        File.open("log/rails_tracepoint_stack.log", "a") do |f|
           f.puts msg
         end
       end

data/lib/rails_tracepoint_stack/trace.rb

@@ -1,4 +1,4 @@
-require 'forwardable'
+require "forwardable"
 
 module RailsTracepointStack
   class Trace

data/lib/rails_tracepoint_stack/trace_filter.rb

@@ -22,7 +22,7 @@ module RailsTracepointStack
       end
       if should_ignore_because_is_ruby_trace?(trace: trace)
         return true
-      end 
+      end
 
       return false
     end

data/lib/rails_tracepoint_stack/tracer.rb

@@ -1,14 +1,23 @@
-#TODO: Move to a loader file
-require 'rails_tracepoint_stack/logger'
-require 'rails_tracepoint_stack/trace_filter'
-require 'rails_tracepoint_stack/trace'
-require 'rails_tracepoint_stack/log_formatter'
+# TODO: Move to a loader file
+require "rails_tracepoint_stack/logger"
+require "rails_tracepoint_stack/trace_filter"
+require "rails_tracepoint_stack/trace"
+require "rails_tracepoint_stack/log_formatter"
 
 module RailsTracepointStack
   class Tracer
     include RailsTracepointStack::TraceFilter
-    # TODO: Tracer.new shoud return the tracer. Is weird to call Tracer.new.tracer
-    def tracer
+    extend Forwardable
+ 
+    def_delegators :@tracer, :enable, :disable
+
+    def initialize
+      generate_tracer
+    end
+
+    private
+
+    def generate_tracer
       @tracer ||= TracePoint.new(:call) do |tracepoint|
         trace = RailsTracepointStack::Trace.new(trace_point: tracepoint)
 
@@ -16,7 +25,7 @@ module RailsTracepointStack
 
         # TODO: Use proper OO
         message = RailsTracepointStack::LogFormatter.message trace
-        RailsTracepointStack::Logger.log message 
+        RailsTracepointStack::Logger.log message
       end
     end
   end

data/lib/rails_tracepoint_stack/version.rb

@@ -1,3 +1,3 @@
 module RailsTracepointStack
-  VERSION = "0.3.4"
+  VERSION = "0.3.5"
 end

data/lib/rails_tracepoint_stack.rb

@@ -20,7 +20,7 @@ module RailsTracepointStack
   def self.enable_trace
     raise ArgumentError, "Block not given to #enable_trace" unless block_given?
 
-    tracer = RailsTracepointStack::Tracer.new.tracer
+    tracer = RailsTracepointStack::Tracer.new
     tracer.enable
     yield
   ensure
@@ -28,8 +28,8 @@ module RailsTracepointStack
   end
 end
 
-if ENV.fetch('RAILS_TRACEPOINT_STACK_ENABLED', 'false') == 'true'
-  $rails_tracer_rtps = RailsTracepointStack::Tracer.new.tracer
+if ENV.fetch("RAILS_TRACEPOINT_STACK_ENABLED", "false") == "true"
+  $rails_tracer_rtps = RailsTracepointStack::Tracer.new
   $rails_tracer_rtps.enable
 
   at_exit do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rails_tracepoint_stack
 version: !ruby/object:Gem::Version
-  version: 0.3.4
+  version: 0.3.5
 platform: ruby
 authors:
 - Carlos Daniel Pohlod
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-08-03 00:00:00.000000000 Z
+date: 2026-05-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -50,6 +50,26 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 13.0.0
+- !ruby/object:Gem::Dependency
+  name: standard
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.39'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.39.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.39'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.39.1
 description: A formatted output of all methods called in your rails application of
   code created by the developer, with the complete path to the class/module, including
   passed params.
@@ -78,7 +98,7 @@ licenses:
 metadata:
   documentation_uri: https://github.com/carlosdanielpohlod/rails_tracepoint_stack/
   changelog_uri: https://github.com/carlosdanielpohlod/rails_tracepoint_stack/blob/main/changelog.md
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -93,8 +113,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.10
-signing_key: 
+rubygems_version: 3.4.19
+signing_key:
 specification_version: 4
 summary: Get a complete stack trace for your code on a Rails application.
 test_files: []