spectracer 1.2.1 → 1.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 69f91e79d4902946517d095ed052d4f5780e7d6be4a433fea8c82bee46d54f78
-  data.tar.gz: fe1abc2750f426935d646d8f3000f37b759f7a0ec7d1a24ae7ffb6fc19501a7f
+  metadata.gz: c01031e82f15d06eeef77717e0d31ec34a5bb73940323deb023c6bf3ec13eb13
+  data.tar.gz: 77395beec7e9113f050495f1ea036e8f019814514da08fb2a90654d25f0a6a10
 SHA512:
-  metadata.gz: be6ecf26e9ebfc694d2df58b16fd8f86cf86a0ecc7d06cef002464cbc082d77833d6c64ecb03b3073564114d22dc0e3a1fe16068a598885dd852eaab346f617e
-  data.tar.gz: 5d6aaa61ab88c7501cfce960aae3be010dc04f9202b741ff40a4035e3aea5359fe822ba96a1bccc26ee90f8d5f09e6148a4cd1bf8191abdec093850e6bb320c6
+  metadata.gz: 30dd806f8a0428ea5890a4f39b2c315ae198c740b27fa33c629f1be9b6279d28b71227afceb349994bf6dfea58e05e5b65fa574623d63131e5b338e51ba9a13e
+  data.tar.gz: 0741cbb623678895406c7a233124654c6b54f8a96743e3bb8ab05de3e4447cafc1b8fa4affc3ea0ded7466f0aa3a2b749048e9ff96a52982f09e7e8472969669

data/CHANGELOG.md

@@ -1,5 +1,23 @@
 # Changelog
 
+## [1.4.0] - 2026-02-04
+
+### Changed
+
+- Remove `./` prefix from traced paths for bktec/zzglob compatibility
+  - zzglob's `path.Clean()` normalizes `./spec/` to `spec/`, causing pattern mismatches
+  - Paths are now stored without the `./` prefix (e.g., `spec/models/user_spec.rb`)
+  - Output is compatible with both `bktec --test-file-pattern` and `rspec --pattern`
+
+## [1.3.0] - 2026-02-02
+
+### Added
+
+- GlobFactorizer to reduce redundant spec patterns before running tests
+  - Removes concrete files already matched by broader globs
+  - Eliminates narrower globs subsumed by broader ones
+  - Example: `{spec/user/**/*_spec.rb, spec/user/**/one_spec.rb}` → `{spec/user/**/*_spec.rb}`
+
 ## [1.2.0] - 2026-02-02
 
 ### Added

data/lib/spectracer/core/glob_factorizer.rb

@@ -0,0 +1,44 @@
+# frozen_string_literal: true
+
+module Spectracer
+  module Core
+    class GlobFactorizer
+      def call(patterns)
+        return patterns.to_a if patterns.size <= 1
+
+        patterns_array = patterns.to_a.uniq
+        globs, files = patterns_array.partition { |p| glob_pattern?(p) }
+
+        # Remove files that are matched by any glob
+        remaining_files = files.reject do |file|
+          globs.any? { |glob| File.fnmatch?(glob, file, File::FNM_PATHNAME) }
+        end
+
+        # Remove globs subsumed by other globs (broader patterns win)
+        remaining_globs = globs.reject do |glob|
+          globs.any? { |other| other != glob && subsumes?(other, glob) }
+        end
+
+        remaining_globs + remaining_files
+      end
+
+      private
+
+      def glob_pattern?(pattern)
+        pattern.include?("*") || pattern.include?("?") || pattern.include?("[")
+      end
+
+      def subsumes?(broader, narrower)
+        return false if broader == narrower
+
+        # Convert the narrower glob to a concrete-ish path by replacing wildcards
+        # then check if the broader glob would match it
+        test_path = narrower
+          .gsub("**", "any/nested/path")
+          .gsub("*", "placeholder")
+
+        File.fnmatch?(broader, test_path, File::FNM_PATHNAME)
+      end
+    end
+  end
+end

data/lib/spectracer/core/paths.rb

@@ -34,8 +34,8 @@ module Spectracer
       end
 
       def normalize(file_path, repo_root:)
-        relative = file_path.sub(/\A#{Regexp.escape(repo_root)}/, ".")
-        relative.start_with?("./") ? relative : "./#{relative}"
+        relative = file_path.sub(%r{\A#{Regexp.escape(repo_root)}/?}, "")
+        relative.sub(%r{\A\./}, "")
       end
 
       def strip_dot_prefix(path)

data/lib/spectracer/core/spec_selector.rb

@@ -5,6 +5,10 @@ module Spectracer
     class SpecSelector
       Result = Struct.new(:specs, :file_to_specs_map, keyword_init: true)
 
+      def initialize(factorizer: GlobFactorizer.new)
+        @factorizer = factorizer
+      end
+
       def call(changed_files:, inverse_deps:, globs:, on_empty:)
         spec_set = Set.new
         file_to_specs = {}
@@ -12,12 +16,12 @@ module Spectracer
         changed_files.each do |file|
           matched_specs = []
 
-          if file.end_with?("_spec.rb")
+          if file.end_with?("_spec.rb", "_test.rb")
             spec_set.add(file)
             matched_specs << file
           end
 
-          file_key = file.start_with?("./") ? file : "./#{file}"
+          file_key = file.sub(%r{\A\./}, "")
           if (specs = inverse_deps[file_key])
             spec_set.merge(specs)
             matched_specs.concat(specs)
@@ -34,7 +38,7 @@ module Spectracer
           file_to_specs[file] = matched_specs.uniq.sort unless matched_specs.empty?
         end
 
-        files = spec_set.to_a.sort
+        files = @factorizer.call(spec_set).sort
         specs_result = files.empty? ? on_empty : files.join(",")
 
         Result.new(specs: specs_result, file_to_specs_map: file_to_specs)

data/lib/spectracer/io/config_loader.rb

@@ -6,7 +6,8 @@ module Spectracer
   module IO
     class ConfigLoader
       FILE_PATH = ".spectracer.yml"
-      DEFAULT_FILE_PATH = File.expand_path("../../spectracer.default.yml", __dir__)
+      DEFAULT_RSPEC_FILE_PATH = File.expand_path("../../spectracer.default.yml", __dir__)
+      DEFAULT_MINITEST_FILE_PATH = File.expand_path("../../spectracer.default.minitest.yml", __dir__)
 
       def initialize(logger: nil)
         @logger = logger
@@ -29,7 +30,17 @@ module Spectracer
           YAML.safe_load_file(FILE_PATH)
         else
           @logger&.debug("No configuration file found at #{FILE_PATH.inspect}. Using defaults.")
-          YAML.safe_load_file(DEFAULT_FILE_PATH)
+          YAML.safe_load_file(default_file_path)
+        end
+      end
+
+      def default_file_path
+        if defined?(RSpec)
+          DEFAULT_RSPEC_FILE_PATH
+        elsif defined?(Minitest)
+          DEFAULT_MINITEST_FILE_PATH
+        else
+          DEFAULT_RSPEC_FILE_PATH
         end
       end
 

data/lib/spectracer/orchestrators/dependency_collector.rb

@@ -24,7 +24,9 @@ module Spectracer
 
         inverse_deps.each_value(&:sort!)
 
-        @store.write(inverse_deps, @paths.collected_dependencies_file)
+        output_path = @paths.collected_dependencies_file
+        @logger&.debug("Writing collected dependencies to: #{output_path}")
+        @store.write(inverse_deps, output_path)
 
         nil
       end

data/lib/spectracer/tasks/spectracer.rake

@@ -10,12 +10,39 @@ namespace :spectracer do
       exit 0
     end
 
+    framework = detect_test_framework
+    default_file_path = case framework
+    when :minitest
+      Spectracer::IO::ConfigLoader::DEFAULT_MINITEST_FILE_PATH
+    else
+      Spectracer::IO::ConfigLoader::DEFAULT_RSPEC_FILE_PATH
+    end
+
+    warn "Detected test framework: #{framework}"
     warn "Creating '#{config_path}' file."
 
-    default_content = File.read(Spectracer::IO::ConfigLoader::DEFAULT_FILE_PATH)
+    default_content = File.read(default_file_path)
     File.write(config_path, default_content)
   end
 
+  def detect_test_framework
+    gemfile_path = File.join(Dir.pwd, "Gemfile")
+    return :rspec unless File.exist?(gemfile_path)
+
+    gemfile_content = File.read(gemfile_path)
+
+    has_rspec = gemfile_content.match?(/['"]rspec['"]|['"]rspec-rails['"]/)
+    has_minitest = gemfile_content.match?(/['"]minitest['"]/) || Dir.exist?("test")
+
+    if has_rspec
+      :rspec
+    elsif has_minitest
+      :minitest
+    else
+      :rspec
+    end
+  end
+
   desc "Collect spec dependencies from tracing artifacts"
   task :collect_dependencies do
     logger = Spectracer::Logger.new(

data/lib/spectracer/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Spectracer
-  VERSION = "1.2.1"
+  VERSION = "1.4.0"
 end

data/lib/spectracer.default.minitest.yml

@@ -0,0 +1,16 @@
+# This file was generated by the 'spectracer:install' Rake task.
+# You can modify this file to customize the behavior of Spectracer.
+# Test framework: Minitest
+defaults:
+  all_specs_glob: "test/**/*_test.rb"
+  no_specs_glob: "{}"
+
+# When no spec files are detected to run, Spectracer will run the specs defined in the 'on_empty_spec_set' key.
+on_empty_spec_set: "{{no_specs_glob}}"
+
+# If a file changes and it matches the glob, the corresponding pattern will be added to the list of spec files to run.
+# These will take precedence over the 'on_empty_spec_set' key.
+globs_matcher:
+  "Gemfile": "{{all_specs_glob}}"
+  "Gemfile.lock": "{{all_specs_glob}}"
+  "config/**": "{{all_specs_glob}}"

data/lib/spectracer.default.yml

@@ -1,5 +1,6 @@
 # This file was generated by the 'spectracer:install' Rake task.
 # You can modify this file to customize the behavior of Spectracer.
+# Test framework: RSpec
 defaults:
   all_specs_glob: "spec/**/*_spec.rb"
   no_specs_glob: "{}"

data/lib/spectracer.rb

@@ -5,6 +5,7 @@ require_relative "spectracer/logger"
 
 require_relative "spectracer/core/paths"
 require_relative "spectracer/core/path_filter"
+require_relative "spectracer/core/glob_factorizer"
 require_relative "spectracer/core/spec_selector"
 
 require_relative "spectracer/io/command_runner"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: spectracer
 version: !ruby/object:Gem::Version
-  version: 1.2.1
+  version: 1.4.0
 platform: ruby
 authors:
 - Mitch Smith
@@ -36,8 +36,10 @@ files:
 - CHANGELOG.md
 - LICENSE.txt
 - README.md
+- lib/spectracer.default.minitest.yml
 - lib/spectracer.default.yml
 - lib/spectracer.rb
+- lib/spectracer/core/glob_factorizer.rb
 - lib/spectracer/core/path_filter.rb
 - lib/spectracer/core/paths.rb
 - lib/spectracer/core/spec_selector.rb