fast_cov 0.1.3 → 0.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 26155209a44b2c77a0d73271bb7f5c4ad9c1788a17db7d28dd328a02e42fda9d
-  data.tar.gz: 9866bb9a6dbf135bb21b9c69658b557bdec4501a6fef688f1dbc38ac6e50d1f5
+  metadata.gz: 19e4e4403a0d37409a7c3c71ab93499a288f7868d68756d7e906dfd8c8033ec3
+  data.tar.gz: 8b1984d267cade376fea5c949f98d031d586b11ebf2e17096d749957a1d99af7
 SHA512:
-  metadata.gz: 3736dbfa1c4c09e06e9e853f9940809412c1a3e4db72a85fd83ed8ce9cbd967bcd81640ac4cb802d8f49ac786f07bf6ffc02f1973329610dfcdb093758792d5d
-  data.tar.gz: 854c246d4ae9b47ed3c83a5570b130c4c24f2b4b8b64d9c87528c36b790ce315927a4df5009d5cd81ca89734565dae1511295e0bb95b1be7b0432e363edc2019
+  metadata.gz: 7f1329d14f6e96f054db4e36b24addd60db315f63029b3be39d295667079b41fca240fb36fbdf6bc3cc7f5e80851e4db23557370b34728fb1b49e11cfb17d1a3
+  data.tar.gz: 41af227e9377d3ac3277f658405561ca7d1ec4ae86c9a1306029fdc34dc7e315a63504f9503305d110d3efaac4503b6e90bcbe0d3f8d864e929322fff38b9a2f

data/ext/fast_cov/fast_cov.c

@@ -23,9 +23,6 @@ static ID id_keys;
 
 // Cache infrastructure
 VALUE fast_cov_cache_hash; // process-level cache (non-static for access from utils)
-static VALUE cDigest;             // Digest::MD5
-static ID id_file;
-static ID id_hexdigest;
 static ID id_clear;
 static ID id_merge_bang;
 
@@ -231,52 +228,23 @@ static void on_newobj_event(VALUE tracepoint_data, void *raw_data) {
 
 // ---- Constant reference resolution (cached) -----------------------------
 
-// Computes MD5 hexdigest of a file's contents.
-static VALUE compute_file_digest_body(VALUE filename) {
-  VALUE digest_obj = rb_funcall(cDigest, id_file, 1, filename);
-  return rb_funcall(digest_obj, id_hexdigest, 0);
-}
-
-static VALUE compute_file_digest(VALUE filename) {
-  int exception_state;
-  VALUE result =
-      rb_protect(compute_file_digest_body, filename, &exception_state);
-  if (exception_state != 0) {
-    rb_set_errinfo(Qnil);
-    return Qnil;
-  }
-  return result;
-}
-
 // Parse file with Prism and extract constant names.
 static VALUE extract_const_names_body(VALUE filename) {
   return rb_funcall(cConstantExtractor, id_extract, 1, filename);
 }
 
 // Returns an array of constant name strings for a file, using the cache.
+// Cache is keyed by filename only - once parsed, results are cached for
+// the lifetime of the process. No digest/mtime validation needed since
+// files don't change during a test suite run.
 static VALUE get_const_refs_for_file(VALUE filename) {
   VALUE const_refs_hash =
       rb_hash_lookup(fast_cov_cache_hash, ID2SYM(rb_intern("const_refs")));
 
-  VALUE cached_entry = rb_hash_lookup(const_refs_hash, filename);
-
-  VALUE current_digest = compute_file_digest(filename);
-  if (NIL_P(current_digest)) {
-    if (!NIL_P(cached_entry)) {
-      rb_hash_delete(const_refs_hash, filename);
-    }
-    return Qnil;
-  }
-
-  // Cache hit: digest matches
-  if (!NIL_P(cached_entry) && RB_TYPE_P(cached_entry, T_HASH)) {
-    VALUE cached_digest =
-        rb_hash_lookup(cached_entry, ID2SYM(rb_intern("digest")));
-
-    if (!NIL_P(cached_digest) &&
-        rb_str_equal(cached_digest, current_digest) == Qtrue) {
-      return rb_hash_lookup(cached_entry, ID2SYM(rb_intern("refs")));
-    }
+  // Cache hit: return cached refs
+  VALUE cached = rb_hash_lookup(const_refs_hash, filename);
+  if (cached != Qnil) {
+    return cached;
   }
 
   // Cache miss: parse with Prism and extract constant names
@@ -285,17 +253,11 @@ static VALUE get_const_refs_for_file(VALUE filename) {
       rb_protect(extract_const_names_body, filename, &exception_state);
   if (exception_state != 0) {
     rb_set_errinfo(Qnil);
-    if (!NIL_P(cached_entry)) {
-      rb_hash_delete(const_refs_hash, filename);
-    }
     return Qnil;
   }
 
-  // Store in cache
-  VALUE new_entry = rb_hash_new();
-  rb_hash_aset(new_entry, ID2SYM(rb_intern("digest")), current_digest);
-  rb_hash_aset(new_entry, ID2SYM(rb_intern("refs")), const_names);
-  rb_hash_aset(const_refs_hash, filename, new_entry);
+  // Store in cache (filename -> refs)
+  rb_hash_aset(const_refs_hash, filename, const_names);
 
   return const_names;
 }
@@ -569,16 +531,10 @@ static VALUE fast_cov_stop(VALUE self) {
 void Init_fast_cov(void) {
   id_extract = rb_intern("extract");
   id_keys = rb_intern("keys");
-  id_file = rb_intern("file");
-  id_hexdigest = rb_intern("hexdigest");
   id_clear = rb_intern("clear");
   id_merge_bang = rb_intern("merge!");
 
-  rb_require("digest/md5");
   rb_require("fast_cov/constant_extractor");
-  VALUE mDigest = rb_const_get(rb_cObject, rb_intern("Digest"));
-  cDigest = rb_const_get(mDigest, rb_intern("MD5"));
-  rb_gc_register_address(&cDigest);
 
   // Initialize process-level cache
   fast_cov_cache_hash = rb_hash_new();

data/lib/fast_cov/compiler.rb

@@ -11,6 +11,8 @@ module FastCov
     FAST_COV_DIR = File.expand_path(".", __dir__) # lib/fast_cov/
 
     def self.compile!
+      clean_ext_dir!
+
       Dir.chdir(EXT_DIR) do
         system(RbConfig.ruby, "extconf.rb") || raise("FastCov: extconf.rb failed")
         system("make") || raise("FastCov: make failed")
@@ -20,6 +22,16 @@ module FastCov
       write_digest
     end
 
+    def self.clean_ext_dir!
+      # Clean stale build artifacts to prevent issues when switching Ruby versions
+      FileUtils.rm_f(Dir.glob(File.join(EXT_DIR, "*.o")))
+      FileUtils.rm_f(Dir.glob(File.join(EXT_DIR, "*.bundle")))
+      FileUtils.rm_f(Dir.glob(File.join(EXT_DIR, "*.so")))
+      FileUtils.rm_f(File.join(EXT_DIR, "Makefile"))
+      FileUtils.rm_f(File.join(EXT_DIR, "mkmf.log"))
+      FileUtils.rm_rf(Dir.glob(File.join(EXT_DIR, "*.dSYM")))
+    end
+
     def self.stale?
       return true unless extension_exists?
 

data/lib/fast_cov/version.rb

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fast_cov
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.4
 platform: ruby
 authors:
 - Ngan Pham