fast_cov 0.3.0 → 0.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4a8e169598d457f54b52e872c65d55a9e87164bcac66654d7b21d9497aeba43a
-  data.tar.gz: 678150e27c7fb664bc5786764b063c1abb75218048f226b39495bc97c012ce75
+  metadata.gz: ba4073b332a0c7f0837d5cf954b9ad98353721e6350c24969541819a42a27c65
+  data.tar.gz: c85de804a17e0f1bd7b21a370e61dbe0d59dc2bb5ce02d84f827c500d6816b5a
 SHA512:
-  metadata.gz: 68e671eab2e3583313b60e408f179006d77aa0d571d1b258b971916b38642fcedae05b8b728ff0588000bd69ada0183a3502b8f6b4e90a37fda3847be3dd6fcd
-  data.tar.gz: d4531b99a2ca2cfb1a566f63bed51444a91bedfd0356a33d50101f18aafa0020ed40d652199de538cb351c6ac2e5e6a7185c1a3aaef316bc444779767a19ad23
+  metadata.gz: f080c26c0a0813a10555885ac162e7662519641501d5d499bebbb1315414ba3469770ac66d95576d6209cc7e9c8b4181fa43c6a729c3d4ccd2c282a1394a86f6
+  data.tar.gz: 0ff5f9c3053015028416a2d5190785542abf1bfe0ce6fe2c0d8dae5ab16e555419f5a8b350980119ebf87065b1166b5f700f6131c582c4df9c36b5eb3a0b5f8e

data/ext/fast_cov/fast_cov.c

@@ -140,80 +140,6 @@ static void on_line_event(rb_event_flag_t event, VALUE self_data, VALUE self,
   record_impacted_file(data, filename);
 }
 
-// ---- Utils module methods (FastCov::Utils) ------------------------------
-
-// Utils.path_within?(path, directory) -> true/false
-// Check if path is within directory, correctly handling:
-// - Trailing slashes on directory
-// - Sibling directories with longer names (e.g., /a/b/c vs /a/b/cd)
-static VALUE utils_path_within(VALUE self, VALUE path, VALUE directory) {
-  Check_Type(path, T_STRING);
-  Check_Type(directory, T_STRING);
-
-  // Freeze strings to prevent GC compaction from moving them
-  rb_str_freeze(path);
-  rb_str_freeze(directory);
-
-  bool result = fast_cov_is_within_root(
-      RSTRING_PTR(path), RSTRING_LEN(path),
-      RSTRING_PTR(directory), RSTRING_LEN(directory));
-
-  return result ? Qtrue : Qfalse;
-}
-
-// Utils.relativize_paths(set, root) -> set
-// Mutates set in place: converts absolute paths to relative paths from root.
-// Paths not within root are left unchanged.
-static VALUE utils_relativize_paths(VALUE self, VALUE set, VALUE root) {
-  Check_Type(root, T_STRING);
-
-  // Freeze root to prevent GC from moving it during compaction
-  rb_str_freeze(root);
-
-  const char *root_ptr = RSTRING_PTR(root);
-  long root_len = RSTRING_LEN(root);
-
-  // Normalize: strip trailing slash for offset calculation
-  long effective_root_len = root_len;
-  if (effective_root_len > 0 && root_ptr[effective_root_len - 1] == '/') {
-    effective_root_len--;
-  }
-
-  // Collect paths to transform (can't modify set while iterating)
-  VALUE paths = rb_funcall(set, rb_intern("to_a"), 0);
-  long num_paths = RARRAY_LEN(paths);
-
-  for (long i = 0; i < num_paths; i++) {
-    VALUE abs_path = rb_ary_entry(paths, i);
-    if (!RB_TYPE_P(abs_path, T_STRING)) continue;
-
-    // Freeze to prevent GC moving it
-    rb_str_freeze(abs_path);
-
-    const char *path_ptr = RSTRING_PTR(abs_path);
-    long path_len = RSTRING_LEN(abs_path);
-
-    // Use proper within_root check
-    if (!fast_cov_is_within_root(path_ptr, path_len, root_ptr, root_len)) {
-      continue;
-    }
-
-    // Calculate offset (skip root + separator)
-    long offset = effective_root_len;
-    if (offset < path_len && path_ptr[offset] == '/') offset++;
-
-    // Create relative path
-    VALUE rel_path = rb_str_substr(abs_path, offset, path_len - offset);
-
-    // Delete old path, add new path
-    rb_funcall(set, rb_intern("delete"), 1, abs_path);
-    rb_funcall(set, rb_intern("add"), 1, rel_path);
-  }
-
-  RB_GC_GUARD(paths);
-  return set;
-}
-
 // ---- Ruby instance methods ----------------------------------------------
 
 static VALUE fast_cov_initialize(int argc, VALUE *argv, VALUE self) {
@@ -366,8 +292,4 @@ void Init_fast_cov(void) {
   rb_define_method(cCoverage, "start", fast_cov_start, 0);
   rb_define_method(cCoverage, "stop", fast_cov_stop, 0);
 
-  // FastCov::Utils module (C-defined methods)
-  VALUE mUtils = rb_define_module_under(mFastCov, "Utils");
-  rb_define_module_function(mUtils, "path_within?", utils_path_within, 2);
-  rb_define_module_function(mUtils, "relativize_paths", utils_relativize_paths, 2);
 }

data/lib/fast_cov/trackers/abstract_tracker.rb

@@ -21,6 +21,10 @@ module FastCov
       @started_thread = nil
     end
 
+    def root
+      @coverage_map.root
+    end
+
     # Public API - called by FastCov framework
 
     def start
@@ -70,12 +74,12 @@ module FastCov
     class << self
       attr_accessor :active
 
-      def record(to: nil)
+      def record(path, to: nil)
         return unless active
-        return unless block_given?
+        return unless path
 
-        path = yield
-        active.record(path, to: to) if path
+        to ||= Utils.resolve_caller(caller_locations(1, 20), active.root)
+        active.record(path, to: to)
       end
 
       def reset

data/lib/fast_cov/trackers/const_get_tracker.rb

@@ -23,9 +23,8 @@ module FastCov
 
     module ConstGetPatch
       def const_get(name, inherit = true)
-        source = caller_locations(1, 1).first&.absolute_path
         result = super
-        FastCov::ConstGetTracker.record(to: source) { const_source_location(name, inherit)&.first }
+        FastCov::ConstGetTracker.record(const_source_location(name, inherit)&.first)
         result
       end
     end

data/lib/fast_cov/trackers/factory_bot_tracker.rb

@@ -42,7 +42,7 @@ module FastCov
           next unless block.is_a?(Proc)
 
           location = block.source_location
-          record { location&.first }
+          record(location&.first)
         end
       end
     end

data/lib/fast_cov/trackers/file_tracker.rb

@@ -4,21 +4,27 @@ require_relative "abstract_tracker"
 
 module FastCov
   # Tracks files read from disk during coverage (JSON, YAML, .rb templates, etc.)
-  # via File.read and File.open.
+  # via File.read, File.open, and YAML load methods.
+  #
+  # YAML methods are patched separately because Bootsnap's compile cache
+  # overrides YAML.load_file/safe_load_file to bypass File.open entirely.
   #
   # Register via: coverage_map.use(FastCov::FileTracker)
   class FileTracker < AbstractTracker
     def install
-      return if File.singleton_class.ancestors.include?(FilePatch)
+      unless File.singleton_class.ancestors.include?(FilePatch)
+        File.singleton_class.prepend(FilePatch)
+      end
 
-      File.singleton_class.prepend(FilePatch)
+      if defined?(::YAML) && !::YAML.singleton_class.ancestors.include?(YamlPatch)
+        ::YAML.singleton_class.prepend(YamlPatch)
+      end
     end
 
     module FilePatch
       def read(name, *args, **kwargs, &block)
-        source = caller_locations(1, 1).first&.absolute_path
         super.tap do
-          FastCov::FileTracker.record(to: source) { File.expand_path(name) }
+          FastCov::FileTracker.record(File.expand_path(name))
         end
       end
 
@@ -26,9 +32,28 @@ module FastCov
         mode = args[0]
         is_read = mode.nil? || (mode.is_a?(String) && mode.start_with?("r")) ||
                   (mode.is_a?(Integer) && (mode & (File::WRONLY | File::RDWR)).zero?)
-        source = caller_locations(1, 1).first&.absolute_path
         super.tap do
-          FastCov::FileTracker.record(to: source) { File.expand_path(name) } if is_read
+          FastCov::FileTracker.record(File.expand_path(name)) if is_read
+        end
+      end
+    end
+
+    module YamlPatch
+      def load_file(path, *args, **kwargs)
+        super.tap do
+          FastCov::FileTracker.record(File.expand_path(path))
+        end
+      end
+
+      def safe_load_file(path, *args, **kwargs)
+        super.tap do
+          FastCov::FileTracker.record(File.expand_path(path))
+        end
+      end
+
+      def unsafe_load_file(path, *args, **kwargs)
+        super.tap do
+          FastCov::FileTracker.record(File.expand_path(path))
         end
       end
     end

data/lib/fast_cov/utils.rb

@@ -0,0 +1,44 @@
+# frozen_string_literal: true
+
+module FastCov
+  module Utils
+    # Check if path is within directory, correctly handling:
+    # - Trailing slashes on directory
+    # - Sibling directories with longer names (e.g., /a/b/c vs /a/b/cd)
+    def self.path_within?(path, directory)
+      dir = directory.end_with?("/") ? directory.chop : directory
+      return true if path == dir
+
+      path.start_with?("#{dir}/")
+    end
+
+    # Mutates set in place: converts absolute paths to relative paths from root.
+    # Paths not within root are left unchanged.
+    def self.relativize_paths(set, root)
+      prefix = root.end_with?("/") ? root : "#{root}/"
+
+      set.to_a.each do |abs_path|
+        next unless abs_path.is_a?(String)
+        next unless abs_path.start_with?(prefix) || abs_path == root.chomp("/")
+
+        set.delete(abs_path)
+        set.add(abs_path.delete_prefix(prefix))
+      end
+
+      set
+    end
+
+    # Walk caller locations to find the first frame whose file is within root.
+    # Handles indirect calls (e.g., YAML.load_file -> File.open) where the
+    # immediate caller is a stdlib/gem file outside the project.
+    def self.resolve_caller(locations, root)
+      locations.each do |loc|
+        path = loc.absolute_path
+        next unless path
+
+        return path if path_within?(path, root)
+      end
+      nil
+    end
+  end
+end

data/lib/fast_cov/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module FastCov
-  VERSION = "0.3.0"
+  VERSION = "0.3.1"
 end

data/lib/fast_cov.rb

@@ -3,6 +3,7 @@
 require "fast_cov/fast_cov.#{RUBY_VERSION}"
 
 module FastCov
+  autoload :Utils, File.expand_path("fast_cov/utils", __dir__)
   autoload :VERSION, File.expand_path("fast_cov/version", __dir__)
   autoload :ConnectedDependencies, File.expand_path("fast_cov/connected_dependencies", __dir__)
   autoload :CoverageMap, File.expand_path("fast_cov/coverage_map", __dir__)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fast_cov
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.3.1
 platform: ruby
 authors:
 - Ngan Pham
@@ -91,6 +91,7 @@ files:
 - lib/fast_cov/trackers/const_get_tracker.rb
 - lib/fast_cov/trackers/factory_bot_tracker.rb
 - lib/fast_cov/trackers/file_tracker.rb
+- lib/fast_cov/utils.rb
 - lib/fast_cov/version.rb
 homepage: https://github.com/Gusto/fast_cov
 licenses: