even_better_specs 0.1.0 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1654cb014ec4bfbe5ac31e7f75259ff4f161041cd1fb15e2658a279b4ceb1f01
-  data.tar.gz: f5f45ee95234cf0631612cb9a4f48946fff2df2415fcd80aceb5b295b619dba3
+  metadata.gz: d351b776d1b0ae7a8ade491c02fc817eb10fa9609bced214c22f3eadbf98b44a
+  data.tar.gz: 23429a0a67b2ff0213e79d951821c98132d8d53dfe8467b22a44036215c61451
 SHA512:
-  metadata.gz: 672498e77b41332d9f1d8510505b5edde0bfb227cd5168fc858beaa21a0d506d2e6dd536d06fce90d24c2399e52fed993ac84e6a46250a7e5d7894878e9887c8
-  data.tar.gz: 6c38c8e78159a86f6358dae83faacd41fe246c70104ba0700b7d2a5b9ccfd3db36cbd90acf1c4ca636e5f86a43ec4f4ab447f1c69d0f23dace9873a938adc8b4
+  metadata.gz: fcb236582d722386c56855e48e7300c51fdf4c513ab6b19d1dd5275add4f11740587e77c01ad1f6282396dbd659728340709d62a76314bcc48877814391f3def
+  data.tar.gz: a87a96aa03ffcf87e0fe6426e5b750db360140f33f0b77b1b9247e53128f7b61956c101d32b4401cffb6dbc8a48a5fa7000508781187d25d7e94c2540d3583ba

data/CHANGELOG.md

@@ -6,6 +6,12 @@ and this project follows [Semantic Versioning](https://semver.org/).
 
 ## [Unreleased]
 
+## [0.2.0]
+
+### Added
+
+- `--diff[=BASE]` option: report only offenses on lines changed since a git revision (default `HEAD`).
+
 ## [0.1.0] - 2026-10-30
 
 ### Added

data/README.md

@@ -12,11 +12,19 @@ gem install even_better_specs
 even_better_specs            # checks every *_spec.rb
 even_better_specs spec/models
 even_better_specs -a         # autocorrect what can be autocorrected
+even_better_specs --diff     # only lines changed since HEAD, plus untracked specs
+even_better_specs --diff=develop   # only lines changed since develop (e.g. a whole branch)
 ```
 
 Do not use `bundle exec`, because the gem is not in the Gemfile. The CLI accepts all `rubocop` options.
 It runs with its own config only, so the project's `.rubocop.yml` (if any) is ignored.
 
+`--diff[=BASE]` reports only offenses that start on a line added or changed since `BASE` (default `HEAD`),
+so you are judged on what you wrote, not on the file's legacy. Without paths it inspects the changed spec
+files; with paths it inspects those and still filters by the diff. The exit code counts only the reported
+offenses. An offense that starts on an unchanged line (e.g. an existing `describe` around your new code) is
+not reported. It cannot be combined with autocorrect, which would rewrite lines outside the diff.
+
 ## Rules
 
 | Guideline | Cop |

data/exe/even_better_specs

@@ -3,5 +3,4 @@
 
 require "even_better_specs"
 
-# --config makes RuboCop ignore the project's own .rubocop.yml (if any).
-exit RuboCop::CLI.new.run(["--config", EvenBetterSpecs::CONFIG, *ARGV])
+exit EvenBetterSpecs::CLI.run(ARGV)

data/lib/even_better_specs/cli.rb

@@ -0,0 +1,47 @@
+# frozen_string_literal: true
+
+require "pathname"
+
+module EvenBetterSpecs
+  # Runs RuboCop with the gem's config, plus the `--diff[=BASE]` option.
+  module CLI
+    AUTOCORRECT_FLAGS = %w[-a -A -x --autocorrect --autocorrect-all --fix-layout].freeze
+
+    def self.run(argv)
+      args = argv.dup
+      flag = args.find { |arg| arg == "--diff" || arg.start_with?("--diff=") }
+      return rubocop(args) unless flag
+
+      args.delete(flag)
+      if args.intersect?(AUTOCORRECT_FLAGS)
+        warn "--diff cannot be combined with autocorrect: it would rewrite lines outside the diff."
+        return 2
+      end
+
+      base = flag.include?("=") ? flag.split("=", 2).last : "HEAD"
+      Diff.changed_lines = Diff.compute(base)
+      if Diff.changed_lines.empty?
+        warn "No changes since #{base}."
+        return 0
+      end
+
+      # Without explicit paths, inspect only the changed files that the config's Include matches.
+      if args.none? { |arg| File.exist?(arg) }
+        cwd = Pathname.new(File.realpath(Dir.pwd))
+        files = Diff.changed_lines.keys.map { |file| Pathname.new(file).relative_path_from(cwd).to_s }
+        args += ["--only-recognized-file-types", *files]
+      end
+      rubocop(args)
+    rescue ArgumentError => e
+      warn e.message
+      2
+    ensure
+      Diff.changed_lines = nil
+    end
+
+    def self.rubocop(args)
+      # --config makes RuboCop ignore the project's own .rubocop.yml (if any).
+      RuboCop::CLI.new.run(["--config", CONFIG, *args])
+    end
+  end
+end

data/lib/even_better_specs/diff.rb

@@ -0,0 +1,70 @@
+# frozen_string_literal: true
+
+require "open3"
+
+module EvenBetterSpecs
+  # Limits a run to the lines added or changed since a git revision (`--diff`).
+  module Diff
+    HUNK = /\A@@ -\S+ \+(\d+)(?:,(\d+))? @@/
+    WHOLE_FILE = [1..].freeze
+
+    class << self
+      # { absolute path => [line ranges] } while a `--diff` run is active, nil otherwise.
+      attr_accessor :changed_lines
+
+      # Changed line ranges per file since `base`. Untracked files count as changed in full,
+      # like RuboCop's `--changed`.
+      def compute(base)
+        root = git("rev-parse", "--show-toplevel").chomp
+        lines = parse(git("diff", "-U0", "--no-color", "--no-ext-diff", "--src-prefix=a/", "--dst-prefix=b/", base), root)
+        git("ls-files", "--others", "--exclude-standard", "--full-name", chdir: root).each_line do |path|
+          lines[File.join(root, path.chomp)] = WHOLE_FILE
+        end
+        lines
+      end
+
+      def parse(diff, root)
+        file = nil
+        diff.each_line.with_object({}) do |line, lines|
+          if line.start_with?("+++ ")
+            path = line.chomp.chomp("\t").delete_prefix("+++ ")
+            file = path == "/dev/null" ? nil : File.join(root, path.delete_prefix("b/"))
+          elsif file && (hunk = HUNK.match(line))
+            start = hunk[1].to_i
+            count = (hunk[2] || 1).to_i
+            (lines[file] ||= []) << (start...start + count) if count.positive?
+          end
+        end
+      end
+
+      def changed?(file, line)
+        # git reports real paths; RuboCop may hold a symlinked one (e.g. /tmp on macOS).
+        changed_lines.fetch(File.realpath(file), []).any? { |range| range.cover?(line) }
+      end
+
+      private
+
+      def git(*args, chdir: Dir.pwd)
+        output, error, status = Open3.capture3("git", *args, chdir: chdir)
+        raise ArgumentError, "git #{args.first} failed: #{error.strip}" unless status.success?
+
+        output
+      end
+    end
+
+    # Drops offenses outside the changed lines. Hooked after RuboCop's result cache, so the cache
+    # keeps full results, and before reporting, so every formatter and the exit code agree.
+    module RunnerFilter
+      private
+
+      def process_file(file)
+        offenses = super
+        return offenses unless Diff.changed_lines
+
+        offenses.select { |offense| Diff.changed?(file, offense.line) }.freeze
+      end
+    end
+  end
+end
+
+RuboCop::Runner.prepend(EvenBetterSpecs::Diff::RunnerFilter)

data/lib/even_better_specs/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module EvenBetterSpecs
-  VERSION = "0.1.0"
+  VERSION = "0.2.0"
 end

data/lib/even_better_specs.rb

@@ -3,6 +3,8 @@
 require "rubocop"
 require "rubocop-rspec"
 require_relative "even_better_specs/version"
+require_relative "even_better_specs/diff"
+require_relative "even_better_specs/cli"
 
 module EvenBetterSpecs
   CONFIG = File.expand_path("../config/default.yml", __dir__)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: even_better_specs
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - Glauco Custodio
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-10-30 00:00:00.000000000 Z
+date: 2026-11-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -52,6 +52,8 @@ files:
 - config/default.yml
 - exe/even_better_specs
 - lib/even_better_specs.rb
+- lib/even_better_specs/cli.rb
+- lib/even_better_specs/diff.rb
 - lib/even_better_specs/version.rb
 - lib/rubocop/cop/even_better_specs/all_possible_cases.rb
 - lib/rubocop/cop/even_better_specs/create_only_needed_data.rb