saboteur 0.8.0 → 0.8.1

data/lib/saboteur/reporter/agent_incomplete.rb

@@ -16,12 +16,15 @@ module Saboteur
             reasons = []
             shard_reason(reasons, report)
             add(reasons, "no_coverage", skipped_for_coverage(report), "subjects or mutants had no covering test")
-            load_only = report.counts[:ran_at_load] + report.skipped_subjects.values.count do |reason|
-              reason == Selection::CoverageMap::RAN_AT_LOAD
-            end
             add(reasons, "unsupported", unsupported_subjects(report),
                 "subjects were skipped as unsupported (for example defined inside a block); no mutant was judged")
-            add(reasons, "ran_at_load", load_only, "code ran only at load; no mutant was judged")
+            add(reasons, "ran_at_load", load_only_subjects(report), "code ran only at load; no mutant was judged")
+            # A file loaded before Coverage.start has no attribution at all:
+            # not unsupported (the shape is fine) and not ran_at_load (the
+            # load-only evidence does not exist) - its own code (#688).
+            add(reasons, "preloaded", preloaded_subjects(report),
+                "files loaded before coverage started; their subjects could not be attributed to tests; " \
+                "no mutant was judged")
             add(reasons, "unreachable", report.counts[:unreachable], "tests did not call the replacement definition")
             add(reasons, "mutant_cap", omitted_by_cap(report), "mutants omitted by max_mutants_per_method")
             add(reasons, "errors", report.counts[:error], "mutants errored; their verdict did not hold up")
@@ -80,9 +83,21 @@ module Saboteur
             report.skipped_subjects.values.count(NO_COVERAGE) + report.counts[:skipped]
           end
 
+          # Load-only mutants plus the subjects skipped for the same reason.
+          def load_only_subjects(report)
+            report.counts[:ran_at_load] + report.skipped_subjects.values.count do |reason|
+              reason == Selection::CoverageMap::RAN_AT_LOAD
+            end
+          end
+
+          def preloaded_subjects(report)
+            report.skipped_subjects.values.count(Selection::CoverageMap::PRELOADED)
+          end
+
           def unsupported_subjects(report)
             report.skipped_subjects.values.count do |reason|
-              reason != NO_COVERAGE && reason != Selection::CoverageMap::RAN_AT_LOAD
+              reason != NO_COVERAGE && reason != Selection::CoverageMap::RAN_AT_LOAD &&
+                reason != Selection::CoverageMap::PRELOADED
             end
           end
 

data/lib/saboteur/reporter/terminal.rb

@@ -87,9 +87,18 @@ module Saboteur
         end
       end
 
+      # An early stop runs fewer mutants than the selection: say both
+      # numbers so "4 mutants" is not read as the whole plan (#582).
+      def mutant_counts(report)
+        planned = report.early_stop&.planned_mutants
+        return "#{report.results.size} mutants" unless planned && planned > report.results.size
+
+        "#{planned} mutants selected, #{report.results.size} attempted"
+      end
+
       def summary(report)
         seed = report.seed && ", seed #{report.seed} (repeat with --seed #{report.seed})"
-        @out.puts "saboteur #{VERSION}: #{report.subjects.size} subjects, #{report.results.size} mutants, " \
+        @out.puts "saboteur #{VERSION}: #{report.subjects.size} subjects, #{mutant_counts(report)}, " \
                   "wall #{format('%.2f', report.wall_time)} s#{seed}"
         @out.puts report.shard.line if report.shard
         @out.puts status_line(report)
@@ -258,6 +267,12 @@ module Saboteur
       def budget_shortfall(stop, completed)
         return unless stop.reason == "max_duration"
 
+        # Exit 4 alone does not say whether anything was judged: headline
+        # it when the baselines and coverage map ate the whole budget (#637).
+        if completed.zero?
+          @out.puts @palette.paint(:yellow, "no mutants ran before the --max-duration budget was spent; " \
+                                            "there is no score to judge")
+        end
         line = if stop.planned_mutants
                  "partial: #{stop.planned_mutants - completed} of #{stop.planned_mutants} mutants not run"
                else

data/lib/saboteur/selection/coverage_map.rb

@@ -20,9 +20,10 @@ module Saboteur
       FORMAT = 11
 
       RAN_AT_LOAD = "ran at load: the code ran while the app loaded; a forked test cannot observe a change here"
+      PRELOADED = "loaded before coverage started"
 
       attr_reader :lines, :tests, :build_seconds, :loaded_files, :durations, :methods, :load_lines, :load_methods,
-                  :branches
+                  :branches, :preloaded
 
       # lines: { "/abs/file.rb" => { 12 => ["id1", "id2"] } }
       # methods: { "/abs/file.rb" => { 12 => ["id1", "id2"] } } keyed by the
@@ -30,7 +31,7 @@ module Saboteur
       # so methods carry those subjects (issue #107).
       # durations: { "id1" => seconds }
       def initialize(lines:, tests:, build_seconds: nil, durations: {}, methods: {}, load_lines: {}, load_methods: {},
-                     branches: {}, loaded_files: nil)
+                     branches: {}, loaded_files: nil, preloaded: [])
         @lines = lines
         @tests = tests
         @build_seconds = build_seconds
@@ -40,6 +41,7 @@ module Saboteur
         @load_methods = load_methods
         @branches = branches
         @loaded_files = loaded_files || []
+        @preloaded = preloaded.to_set { |file| CoverageMap.canonical(file) }
       end
 
       # Source files the given tests touched, sorted and deduped: the
@@ -99,6 +101,19 @@ module Saboteur
                        loaded_files: loaded_files)
       end
 
+      # Files that loaded before Coverage.start (gemspec require, RUBYOPT
+      # -r): they can never carry hits, so their subjects are reported as
+      # loaded before coverage started instead of no coverage.
+      def with_preloaded(paths)
+        self.class.new(lines: lines, tests: tests, build_seconds: build_seconds, durations: durations,
+                       methods: methods, load_lines: load_lines, load_methods: load_methods, branches: branches,
+                       loaded_files: loaded_files, preloaded: paths)
+      end
+
+      def preloaded?(file)
+        preloaded.include?(CoverageMap.canonical(file))
+      end
+
       # A def the boot called (a def-time macro's class-body caller, say)
       # and no test or hook called again: re-evaluating it post-boot
       # changes nothing a selected test can observe, because closure

data/lib/saboteur/version.rb

@@ -2,5 +2,5 @@
 # frozen_string_literal: true
 
 module Saboteur
-  VERSION = "0.8.0"
+  VERSION = "0.8.1"
 end

data/lib/saboteur.rb

@@ -51,6 +51,7 @@ require_relative "saboteur/integration/base"
 require_relative "saboteur/subject"
 require_relative "saboteur/refinement_activations"
 require_relative "saboteur/subject_finder"
+require_relative "saboteur/attribute_finder"
 require_relative "saboteur/constant_subject"
 require_relative "saboteur/constant_subject_finder"
 require_relative "saboteur/constant_reads"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: saboteur
 version: !ruby/object:Gem::Version
-  version: 0.8.0
+  version: 0.8.1
 platform: ruby
 authors:
 - Vamsi Pavan Mahesh Gunturu
@@ -10,20 +10,6 @@ bindir: exe
 cert_chain: []
 date: 2026-10-18 00:00:00.000000000 Z
 dependencies:
-- !ruby/object:Gem::Dependency
-  name: fiddle
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '1.1'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '1.1'
 - !ruby/object:Gem::Dependency
   name: prism
   requirement: !ruby/object:Gem::Requirement
@@ -61,6 +47,7 @@ files:
 - lib/generators/saboteur/install/install_generator.rb
 - lib/saboteur.rb
 - lib/saboteur/agent_skill.rb
+- lib/saboteur/attribute_finder.rb
 - lib/saboteur/baseline/acceptance.rb
 - lib/saboteur/baseline/command.rb
 - lib/saboteur/baseline/comparison.rb
@@ -126,6 +113,7 @@ files:
 - lib/saboteur/engine/unscoped_suite_warning.rb
 - lib/saboteur/engine/verdict_check.rb
 - lib/saboteur/execution/external_runner.rb
+- lib/saboteur/execution/fiddle_loader.rb
 - lib/saboteur/execution/fork_readiness.rb
 - lib/saboteur/execution/marker_sweep.rb
 - lib/saboteur/execution/peak_memory.rb
@@ -138,6 +126,7 @@ files:
 - lib/saboteur/execution/seatbelt.rb
 - lib/saboteur/execution/stale_captures.rb
 - lib/saboteur/execution/supervisor.rb
+- lib/saboteur/execution/test_command_shape.rb
 - lib/saboteur/execution/worker_pool.rb
 - lib/saboteur/execution/wrapper.rb
 - lib/saboteur/frameworks/database_hosts.rb
@@ -153,6 +142,7 @@ files:
 - lib/saboteur/integration/focus_guard.rb
 - lib/saboteur/integration/rspec.rb
 - lib/saboteur/mutation_source/argument_removal.rb
+- lib/saboteur/mutation_source/argument_swap.rb
 - lib/saboteur/mutation_source/base.rb
 - lib/saboteur/mutation_source/builtin.rb
 - lib/saboteur/mutation_source/chain_removal.rb