coverband 6.1.7 → 6.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e3ae4783bfea151c9e3b5d58e397467085206359b6c3446df24ad136fc067ca3
-  data.tar.gz: c7491d46394412b813447b80e21ca29e1cfe62060b54a28a14a6145a8ec5eb84
+  metadata.gz: 12802541de3827dd777fba9962270fd2f0a6c7b3debb2ef96c4bcca32f02b48d
+  data.tar.gz: 36fa21009ed2ada842ff90ca4a18195c8bfcddc81ad0951c11f613ede11f6606
 SHA512:
-  metadata.gz: 2cbd8715bf8a5ea95e5705038acec36bd8549fb9ca01b99c4266919904122fb9fd81cbc3e1488d6eaa0c0bb955875c429751523843449abc0c5d01107d620085
-  data.tar.gz: f4ac2904d9cde83162aa3f1d350aeb25e337f8c86178b79668170ae00ef64df9d564e15cdd94597cb1ea6652daa244b8454cdc8c3401e14b74662527ff7ea34e
+  metadata.gz: 03c7e496735cfe705b926b30140b3d704993b0599e470d290fb9ae672a7b204bf5cfa98adfc73d54eb70df1c71e8cbb9d91a5f85862b86d5c2a1ba1d0d3aa704
+  data.tar.gz: 6610a4ec327a69de3bd6878cb256a8f9d605fcf2a4104e6699b6b68acbec3ac9aaaeb957569eaf9ac1e949c58c5a6adbcd75055be464da56a11dde5c873dc4b6

data/.github/workflows/main.yml

@@ -28,11 +28,18 @@ jobs:
           - gemfile: 'rails_8.0'
             ruby: '3.1'
     runs-on: ${{ matrix.os }}-latest
+    services:
+      redis:
+        image: redis:${{ matrix.redis-version }}
+        ports:
+          - 6379:6379
+        options: >-
+          --health-cmd "redis-cli ping"
+          --health-interval 10s
+          --health-timeout 5s
+          --health-retries 5
     steps:
       - uses: actions/checkout@v6
-      - uses: supercharge/redis-github-action@v2
-        with:
-          redis-version: ${{ matrix.redis-version }}
       - uses: ruby/setup-ruby@v1
         with:
           ruby-version: ${{ matrix.ruby }}

data/Gemfile

@@ -8,3 +8,6 @@ gem "rails" # latest
 gem "haml"
 gem "slim"
 gem "webrick"
+
+# Required for Ruby 3.4+ (extracted from stdlib)
+gem "cgi"

data/Gemfile.rails7.0

@@ -8,3 +8,6 @@ gem 'rails', '~>7.0.0'
 gem "haml"
 gem "slim"
 gem "webrick"
+
+# Required for Ruby 3.4+ (extracted from stdlib)
+gem "cgi"

data/Gemfile.rails7.1

@@ -7,4 +7,7 @@ gemspec
 gem 'rails', '~>7.1.0'
 gem "haml"
 gem "slim"
-gem "webrick"
+gem "webrick"
+
+# Required for Ruby 3.4+ (extracted from stdlib)
+gem "cgi"

data/Gemfile.rails7.2

@@ -8,3 +8,6 @@ gem 'rails', '~>7.2.0'
 gem "haml"
 gem "slim"
 gem "webrick"
+
+# Required for Ruby 3.4+ (extracted from stdlib)
+gem "cgi"

data/Gemfile.rails8.0

@@ -8,3 +8,6 @@ gem 'rails', '~>8.0.0'
 gem "haml"
 gem "slim"
 gem "webrick"
+
+# Required for Ruby 3.4+ (extracted from stdlib)
+gem "cgi"

data/README.md

@@ -1,9 +1,9 @@
-<img src="https://raw.github.com/danmayer/coverband/master/docs/assets/logo/heads.svg?sanitize=true" width='300'>
+<img src="https://raw.github.com/danmayer/coverband/main/docs/assets/logo/heads.svg?sanitize=true" width='300'>
 
 # Coverband
 
 [![GithubCI](https://github.com/danmayer/coverband/workflows/CI/badge.svg)](https://github.com/danmayer/coverband/actions)
-[![Coverage Status](https://coveralls.io/repos/github/danmayer/coverband/badge.svg?branch=master)](https://coveralls.io/github/danmayer/coverband?branch=master)
+[![Coverage Status](https://coveralls.io/repos/github/danmayer/coverband/badge.svg?branch=main)](https://coveralls.io/github/danmayer/coverband?branch=main)
 [![Maintainability](https://api.codeclimate.com/v1/badges/1e6682f9540d75f26da7/maintainability)](https://codeclimate.com/github/danmayer/coverband/maintainability)
 [![Discord Shield](https://img.shields.io/discord/609509533999562753)](https://discord.gg/KAH38EV)
 
@@ -57,7 +57,7 @@ gem 'coverband'
 
 With older versions of Coverband, projects would report to redis using rack or sidekiq middleware. After Coverband 4.0, this should no longer be required and could cause performance issues. Reporting to redis is now automatically done within a background thread with no custom code needed.
 
-See [changelog](https://github.com/danmayer/coverband/blob/master/changes.md).
+See [changelog](https://github.com/danmayer/coverband/blob/main/changes.md).
 
 ## Rails
 
@@ -77,10 +77,12 @@ run ActionController::Dispatcher.new
 
 ## Coverband Web UI
 
-![image](https://raw.github.com/danmayer/coverband/master/docs/coverband_web_ui.png)
+![image](https://raw.github.com/danmayer/coverband/main/docs/coverband_web_ui.png)
 
 > You can check it out locally by running the [Coverband Demo App](https://github.com/danmayer/coverband_rails_example).
 
+- View a shared demo at [https://coverband-rails-example.onrender.com/](https://coverband-rails-example.onrender.com/)
+
 - View overall coverage information
 
 - Drill into individual file coverage
@@ -208,7 +210,7 @@ Take Coverband for a spin on the live Heroku deployed [Coverband Demo](https://c
 
 If you need to configure Coverband, this can be done by creating a `config/coverband.rb` file relative to your project root.
 
-- See [lib/coverband/configuration.rb](https://github.com/danmayer/coverband/blob/master/lib/coverband/configuration.rb) for all options
+- See [lib/coverband/configuration.rb](https://github.com/danmayer/coverband/blob/main/lib/coverband/configuration.rb) for all options
 - By default Coverband will try to store data to Redis \* Redis endpoint is looked for in this order: `ENV['COVERBAND_REDIS_URL']`, `ENV['REDIS_URL']`, or `localhost`
 
 Below is an example config file for a Rails 5 app:
@@ -269,7 +271,7 @@ This feature is enabled by default. To stop this feature, disable the feature in
 
 `config.track_views = false`
 
-![image](https://raw.github.com/danmayer/coverband/master/docs/coverband_view_tracker.png)
+![image](https://raw.github.com/danmayer/coverband/main/docs/coverband_view_tracker.png)
 
 ### Hiding settings
 

data/changes.md

@@ -1,3 +1,24 @@
+### 6.2.0
+
+* Fix: Correct `Rack::Static` URL handling in the web reporter by using an empty string URL prefix; thanks @alpaca-tc (#635)
+* Fix: Skip missing files during dead method scans to avoid errors in dynamic or removed-file environments; thanks @doug-hall (#632)
+* Fix: Improve MCP dead method formatting for clearer tool output; thanks @doug-hall (#633)
+* Docs: README updates (including `master` → `main` wording updates); thanks @jjb (#627)
+
+### 6.1.8
+
+* Feature: Added temporal data to GetFileCoverage tool for MCP by @fabienpiette (#626)
+* Optimization: Significant performance improvements to `FileList` avoiding array allocations and reducing redundant iterations (#619, #620, #622, #623)
+* Optimization: Optimize `HTMLFormatter` with template caching (#616, #618)
+* Optimization: Optimize memory usage by removing redundant `dup` calls across `Coverband::Utils::Result` (#617, #621)
+* Optimization: Optimize `array_add` logic with in-place modification avoiding overhead in newer Ruby (#615)
+* Optimization: Optimize `ViewTracker` unused keys and path normalization (#608, #609, #613)
+* Optimization: Optimize `AbstractTracker` to use bulk Redis hset and direct Hash lookups (#610, #612)
+* Optimization: Cache Regexp in `short_name` and optimize `RelativeFileConverter` with `Regexp.union` (#606, #607)
+* Optimization: Optimize delta collector ignore check order (#611)
+* Refactoring: Extract Line class to `Coverband::Utils::SourceFile::Line` (#605, #614)
+* Fix: Improve symlink handling in file converters (#603)
+
 ### Coverband 6.1.7
 
 * add back a running demo site

data/lib/coverband/adapters/base.rb

@@ -125,14 +125,29 @@ module Coverband
         }
       end
 
-      # TODO: This should have cases reduced
       def array_add(latest, original)
-        if latest.empty? && original.empty?
-          []
-        elsif Coverband.configuration.use_oneshot_lines_coverage
-          latest.map!.with_index { |v, i| ((v + original[i] >= 1) ? 1 : 0) if v && original[i] }
+        use_oneshot = Coverband.configuration.use_oneshot_lines_coverage
+        if latest.frozen?
+          latest.map.with_index do |v, i|
+            if v && original[i]
+              if use_oneshot
+                (v + original[i] >= 1) ? 1 : 0
+              else
+                v + original[i]
+              end
+            end
+          end
         else
-          latest.map.with_index { |v, i| (v && original[i]) ? v + original[i] : nil }
+          latest.each_with_index do |v, i|
+            latest[i] = if v && original[i]
+              if use_oneshot
+                (v + original[i] >= 1) ? 1 : 0
+              else
+                v + original[i]
+              end
+            end
+          end
+          latest
         end
       end
     end

data/lib/coverband/adapters/file_store.rb

@@ -65,8 +65,7 @@ module Coverband
       end
 
       def save_report(report)
-        data = report.dup
-        data = merge_reports(data, coverage)
+        data = merge_reports(report, coverage)
         File.write(path, JSON.dump(data))
       end
 

data/lib/coverband/adapters/hash_redis_store.rb

@@ -29,6 +29,7 @@ module Coverband
 
         @ttl = opts[:ttl]
         @relative_file_converter = opts[:relative_file_converter] || Utils::RelativeFileConverter
+        @short_name_regex = /^#{Coverband.configuration.root}/
       end
 
       def supported?
@@ -171,7 +172,7 @@ module Coverband
       end
 
       def short_name(filename)
-        filename.sub(/^#{Coverband.configuration.root}/, ".")
+        filename.sub(@short_name_regex, ".")
           .gsub(%r{^\./}, "")
       end
 

data/lib/coverband/adapters/redis_store.rb

@@ -63,8 +63,7 @@ module Coverband
       # the Coverband 2 had the same issue,
       # and the tradeoff has always been acceptable
       def save_report(report)
-        data = report.dup
-        data = merge_reports(data, coverage(nil, skip_hash_check: true))
+        data = merge_reports(report, coverage(nil, skip_hash_check: true))
         save_coverage(data)
       end
 

data/lib/coverband/adapters/web_service_store.rb

@@ -66,10 +66,9 @@ module Coverband
         # We set here vs initialize to avoid setting on the primary process vs child processes
         @pid ||= ::Process.pid
 
-        # TODO: do we need dup
         # TODO: we don't need upstream timestamps, server will track first_seen
         Thread.new do
-          data = expand_report(report.dup)
+          data = expand_report(report)
           full_package = {
             collection_type: "coverage_delta",
             collection_data: {

data/lib/coverband/collectors/abstract_tracker.rb

@@ -61,8 +61,8 @@ module Coverband
       end
 
       def unused_keys(used_keys = nil)
-        recently_used_keys = (used_keys || self.used_keys).keys
-        all_keys.reject { |k| recently_used_keys.include?(k.to_s) }
+        recently_used_keys = used_keys || self.used_keys
+        all_keys.reject { |k| recently_used_keys.key?(k.to_s) }
       end
 
       def as_json
@@ -97,8 +97,8 @@ module Coverband
         redis_store.set(tracker_time_key, Time.now.to_i) unless @one_time_timestamp || tracker_time_key_exists?
         @one_time_timestamp = true
         reported_time = Time.now.to_i
-        @keys_to_record.to_a.each do |key|
-          redis_store.hset(tracker_key, key.to_s, reported_time)
+        if @keys_to_record.any?
+          redis_store.hset(tracker_key, @keys_to_record.each_with_object({}) { |key, h| h[key.to_s] = reported_time })
         end
         @keys_to_record.clear
       rescue => e

data/lib/coverband/collectors/delta.rb

@@ -54,8 +54,8 @@ module Coverband
           # on the critical performance path, and any refactoring I come up with
           # would slow down the performance.
           ###
-          next unless @@ignore_patterns.none? { |pattern| file.match(pattern) } &&
-            file.start_with?(@@project_directory)
+          next unless file.start_with?(@@project_directory) &&
+            @@ignore_patterns.none? { |pattern| file.match(pattern) }
 
           # This handles Coverage branch support, setup by default in
           # simplecov 0.18.x
@@ -84,8 +84,8 @@ module Coverband
           # on the critical performance path, and any refactoring I come up with
           # would slow down the performance.
           ###
-          next unless @@ignore_patterns.none? { |pattern| file.match(pattern) } &&
-            file.start_with?(@@project_directory)
+          next unless file.start_with?(@@project_directory) &&
+            @@ignore_patterns.none? { |pattern| file.match(pattern) }
 
           @@stubs[file] ||= ::Coverage.line_stub(file)
           transformed_line_counts = coverage[:oneshot_lines].each_with_object(@@stubs[file].dup) { |line_number, line_counts|

data/lib/coverband/collectors/view_tracker.rb

@@ -78,8 +78,17 @@ module Coverband
         recently_used_views = used_keys.keys
         unused_views = all_keys - recently_used_views
         # since layouts don't include format we count them used if they match with ANY formats
-        unused_views = unused_views.reject { |view| view.include?("/layouts/") && recently_used_views.any? { |used_view| view.include?(used_view) } }
-        unused_views.reject { |view| @ignore_patterns.any? { |pattern| view.match?(pattern) } }
+        potential_layout_references = recently_used_views.reject { |v| v.end_with?(".erb", ".haml", ".slim") }
+
+        layout_matcher = if potential_layout_references.any?
+          Regexp.union(potential_layout_references)
+        end
+
+        unused_views.reject! do |view|
+          (layout_matcher && view.include?("/layouts/") && view.match?(layout_matcher)) ||
+            @ignore_patterns.any? { |pattern| view.match?(pattern) }
+        end
+        unused_views
       end
 
       def clear_key!(filename)
@@ -103,6 +112,7 @@ module Coverband
 
         roots.each do |root|
           normalized = normalized.gsub(root, "")
+          break if normalized.length < original_length
         end
 
         # Only remove leading slash if we actually modified the path by removing a root

data/lib/coverband/mcp/tools/get_dead_methods.rb

@@ -28,21 +28,21 @@ module Coverband
 
           if file_pattern
             dead_methods = dead_methods.select do |method|
-              File.fnmatch(file_pattern, method[:file_path], File::FNM_PATHNAME)
+              File.fnmatch(file_pattern, method.file_path, File::FNM_PATHNAME)
             end
           end
 
           # Group by file for easier reading
-          grouped = dead_methods.group_by { |m| m[:file_path] }
+          grouped = dead_methods.group_by(&:file_path)
 
           result = grouped.map do |file_path, methods|
             {
               file: file_path,
               dead_methods: methods.map do |m|
                 {
-                  class_name: m[:class_name],
-                  method_name: m[:method_name],
-                  line_number: m[:line_number]
+                  class_name: m.class_name,
+                  method_name: m.name,
+                  line_number: m.first_line_number
                 }
               end
             }

data/lib/coverband/mcp/tools/get_file_coverage.rb

@@ -52,6 +52,8 @@ module Coverband
               lines_missed: file_data["lines_missed"],
               runtime_percentage: file_data["runtime_percentage"],
               never_loaded: file_data["never_loaded"],
+              first_updated_at: file_data["first_updated_at"],
+              last_updated_at: file_data["last_updated_at"],
               coverage: file_data["coverage"]
             }
           end

data/lib/coverband/reporters/json_report.rb

@@ -72,6 +72,11 @@ module Coverband
         end
       end
 
+      def format_timestamp(timestamp)
+        return nil if timestamp.nil? || timestamp == Coverband::Utils::SourceFile::NOT_AVAILABLE
+        timestamp.is_a?(Time) ? timestamp.iso8601 : timestamp.to_s
+      end
+
       def report_as_json
         return filtered_report_files.to_json if for_merged_report
 
@@ -133,6 +138,8 @@ module Coverband
             filename: source_file.filename,
             hash: Digest::SHA1.hexdigest(source_file.filename),
             never_loaded: source_file.never_loaded,
+            first_updated_at: format_timestamp(source_file.first_updated_at),
+            last_updated_at: format_timestamp(source_file.last_updated_at),
             runtime_percentage: result.runtime_relevant_coverage(source_file),
             lines_of_code: source_file.lines.count,
             lines_covered: source_file.covered_lines.count,

data/lib/coverband/reporters/web.rb

@@ -32,9 +32,9 @@ module Coverband
 
       def init_web
         full_path = Gem::Specification.find_by_name("coverband").full_gem_path
-        @static = Rack::Static.new(self,
-          root: File.expand_path("public", full_path),
-          urls: [/.*\.css/, /.*\.js/, /.*\.gif/, /.*\.png/])
+        @file_server = Rack::Files.new(
+          File.expand_path("public", full_path)
+        )
       end
 
       def check_auth
@@ -85,7 +85,7 @@ module Coverband
           else
             case request_path_info
             when /.*\.(css|js|gif|png)/
-              @static.call(env)
+              @file_server.get(env)
             when %r{/settings}
               [200, coverband_headers, [settings]]
             when %r{/view_tracker_data}

data/lib/coverband/utils/absolute_file_converter.rb

@@ -5,7 +5,7 @@ module Coverband
     class AbsoluteFileConverter
       def initialize(roots)
         @cache = {}
-        @roots = roots.map { |root| "#{File.expand_path(root)}/" }
+        @roots = convert_roots(roots)
       end
 
       def self.instance
@@ -24,11 +24,25 @@ module Coverband
         @cache[relative_path] ||= begin
           relative_filename = relative_path
           local_filename = relative_filename
-          @roots.each do |root|
-            relative_filename = relative_filename.sub(/^#{root}/, "./")
-            # once we have a relative path break out of the loop
-            break if relative_filename.start_with? "./"
+          @roots.each do |root, root_regexp|
+            if relative_filename.match?(root_regexp)
+              relative_filename = relative_filename.sub(root_regexp, "./")
+              # once we have a relative path break out of the loop
+              break
+            end
           end
+
+          if relative_filename == local_filename && File.exist?(local_filename)
+            real_filename = File.realpath(local_filename)
+            @roots.each do |root, root_regexp|
+              if real_filename.match?(root_regexp)
+                relative_filename = real_filename.sub(root_regexp, "./")
+                # once we have a relative path break out of the loop
+                break
+              end
+            end
+          end
+
           # the filename for our reports is expected to be a full path.
           # roots.last should be roots << current_root}/
           # a fully expanded path of config.root
@@ -36,12 +50,28 @@ module Coverband
           # above only works for app files
           # we need to rethink some of this logic
           # gems aren't at project root and can have multiple locations
-          local_root = @roots.find { |root|
+          local_root = @roots.find { |root, _root_regexp|
             File.exist?(relative_filename.gsub("./", root))
-          }
+          }&.first
           local_root ? relative_filename.gsub("./", local_root) : local_filename
         end
       end
+
+      private
+
+      def convert_roots(roots)
+        roots.flat_map { |root|
+          items = []
+          expanded = "#{File.expand_path(root)}/"
+          items << [expanded, /^#{expanded}/]
+
+          if File.exist?(root)
+            real = "#{File.realpath(root)}/"
+            items << [real, /^#{Regexp.escape(real)}/]
+          end
+          items
+        }.uniq
+      end
     end
   end
 end

data/lib/coverband/utils/dead_methods.rb

@@ -43,7 +43,11 @@ module Coverband
         # and runtime phases.
         coverage = Coverband.configuration.store.get_coverage_report[Coverband::MERGED_TYPE]
         coverage.flat_map do |file_path, coverage|
+          next [] unless File.exist?(file_path)
+
           scan(file_path: file_path, coverage: coverage["data"])
+        rescue Errno::ENOENT
+          []
         end
       end
 

data/lib/coverband/utils/file_list.rb

@@ -14,28 +14,28 @@ module Coverband
       def covered_lines
         return 0.0 if empty?
 
-        map { |f| f.covered_lines.count }.inject(:+)
+        @covered_lines ||= sum(&:covered_lines_count)
       end
 
       # Returns the count of lines that have been missed
       def missed_lines
         return 0.0 if empty?
 
-        map { |f| f.missed_lines.count }.inject(:+)
+        @missed_lines ||= sum(&:missed_lines_count)
       end
 
       # Returns the count of lines that are not relevant for coverage
       def never_lines
         return 0.0 if empty?
 
-        map { |f| f.never_lines.count }.inject(:+)
+        @never_lines ||= sum(&:never_lines_count)
       end
 
       # Returns the count of skipped lines
       def skipped_lines
         return 0.0 if empty?
 
-        map { |f| f.skipped_lines.count }.inject(:+)
+        @skipped_lines ||= sum(&:skipped_lines_count)
       end
 
       # Computes the coverage based upon lines covered and lines missed for each file
@@ -46,7 +46,9 @@ module Coverband
 
       # Returns the overall amount of relevant lines of code across all files in this list
       def lines_of_code
-        covered_lines + missed_lines
+        return 0.0 if empty?
+
+        @lines_of_code ||= sum(&:lines_of_code)
       end
 
       # Computes the coverage based upon lines covered and lines missed
@@ -68,11 +70,17 @@ module Coverband
       def covered_strength
         return 0.0 if empty? || lines_of_code.zero?
 
-        Float(map { |f| f.covered_strength * f.lines_of_code }.inject(:+) / lines_of_code)
+        Float(sum { |f| f.covered_strength * f.lines_of_code } / lines_of_code)
       end
 
       def first_seen_at
-        map(&:first_updated_at).reject { |el| el.is_a?(String) }.min
+        min = nil
+        each do |f|
+          val = f.first_updated_at
+          next if val.is_a?(String)
+          min = val if min.nil? || val < min
+        end
+        min
       end
     end
   end

data/lib/coverband/utils/html_formatter.rb

@@ -73,9 +73,12 @@ module Coverband
         template("data").result(binding)
       end
 
+      TEMPLATE_CACHE = {}
+      private_constant :TEMPLATE_CACHE
+
       # Returns the an erb instance for the template of given name
       def template(name)
-        ERB.new(File.read(File.join(File.dirname(__FILE__), "../../../views/", "#{name}.erb")))
+        TEMPLATE_CACHE[name] ||= ERB.new(File.read(File.expand_path("../../../views/#{name}.erb", __dir__)))
       end
 
       def output_path
@@ -119,7 +122,7 @@ module Coverband
 
       # Returns a table containing the given source files
       def formatted_file_list(title, result, source_files, options = {})
-        title_id = title.gsub(/^[^a-zA-Z]+/, "").gsub(/[^a-zA-Z0-9\-\_]/, "")
+        title_id = title.gsub(/^[^a-zA-Z]+/, "").gsub(/[^a-zA-Z0-9\-_]/, "")
         # Silence a warning by using the following variable to assign to `_`:
         # "warning: possibly useless use of a variable in void context"
         # The variable is used by ERB via binding.

data/lib/coverband/utils/relative_file_converter.rb

@@ -17,24 +17,39 @@ module Coverband
 
       def initialize(roots)
         @cache = {}
-        @roots = normalize(roots)
+        @roots_regexp = Regexp.union(convert_roots(roots))
       end
 
       def convert(file)
         @cache[file] ||= begin
-          relative_file = file
-          @roots.each do |root|
-            relative_file = file.gsub(/^#{root}/, ".")
-            break relative_file if relative_file.start_with?(".")
+          relative_file = file.sub(@roots_regexp, "./")
+
+          if relative_file == file && !file.start_with?(".") && File.exist?(file)
+            real_file = File.realpath(file)
+            new_relative_file = real_file.sub(@roots_regexp, "./")
+            relative_file = ((new_relative_file == real_file) ? file : new_relative_file)
           end
+
           relative_file
         end
       end
 
       private
 
-      def normalize(paths)
-        paths.map { |root| File.expand_path(root) }
+      def convert_roots(roots)
+        roots.flat_map { |root|
+          items = []
+          expanded = File.expand_path(root)
+          expanded += "/" unless expanded.end_with?("/")
+          items << /^#{Regexp.escape(expanded)}/
+
+          if File.exist?(root)
+            real = File.realpath(root)
+            real += "/" unless real.end_with?("/")
+            items << /^#{Regexp.escape(real)}/
+          end
+          items
+        }.uniq
       end
     end
   end

data/lib/coverband/utils/result.rb

@@ -51,8 +51,6 @@ module Coverband
       # the line-by-line coverage to zero (if relevant) or nil (comments / whitespace etc).
       def self.add_not_loaded_files(result, tracked_files)
         if tracked_files
-          # TODO: Can we get rid of this dup it wastes memory
-          result = result.dup
           Dir[tracked_files].each do |file|
             absolute = File.expand_path(file)
             result[absolute] ||= {