testprune 0.1.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d3484c32e59411fd94af756cfaec78df0a07c6ce0f7f7dfaf16cf82c6745d84f
-  data.tar.gz: 47e74f078c45818060d393c08473a66275fd5a002e750a24ed02ca6b47d8f412
+  metadata.gz: e0524c23f51b9afdbb9731dee285f81be5571abcff4730b09194865edcb01d0d
+  data.tar.gz: 648d2e72e49dc16a083b7ca116889e4694fdaa72e6e1337207bdc7da7376260f
 SHA512:
-  metadata.gz: 0c9cc56ac84e87e994165c3abce1238a84924f3ecf980042ee876b0c0b6fa551021780d23a903494298f37bd248c546d1655c282285c98f6a2111ba37a9ab67b
-  data.tar.gz: f6c4748cde9a7e89dbaaeb67cc609e8c17de9173b75c723d0f0c4f22f84e77034b52a51187233418778920ddec58953585942002b10771a8ce726201632522fd
+  metadata.gz: 762a0cf8e298644d06094814b81e9579312c16247a649013da282207623e88349a0918a95657f9a33125fb9b0a029369630071806b484b8c5869f4979dafc858
+  data.tar.gz: a245409289d680cfe06bc08ad2afdc004ae1d542f9e2895e70d0840834852f8ebf75d95215129f1afd7c58d84b34925864d5af02ae34ae933b70c34aa35a0a75

data/README.md

@@ -11,9 +11,12 @@
 ```sh
 gem install testprune   # no Gemfile change required
 
-testprune run           # runs your suite once, records per-test coverage
+testprune scan          # runs your suite once, records per-test coverage
 testprune report        # see what's redundant — read-only
 testprune apply         # approve removals → .patch file → git apply
+
+# or all at once:
+testprune prune         # scan + apply in a single step
 ```
 
 Works with **Minitest** and **RSpec**. No config files. No changes to your project required.
@@ -54,7 +57,7 @@ end
 
 ```sh
 bundle install
-bundle exec testprune run
+bundle exec testprune scan
 ```
 
 ---
@@ -64,20 +67,46 @@ bundle exec testprune run
 ### Step 1 — Capture
 
 ```sh
-# Autodetect: runs `rspec` if spec/ exists, `rake test` otherwise
-testprune run
+# No args: autodetects framework; prompts to exclude slow/integration folders
+# (any test/ subdirectory matching: selenium, request, piper, integration)
+testprune scan
+
+# Positional paths — no `--` needed
+testprune scan test/api test/services test/models
+testprune scan test/models test/jobs -s app -s lib
 
 # Explicit command (pass after --)
-testprune run -- bundle exec rspec spec/models
-testprune run -- bundle exec ruby -Itest test/my_test.rb
-testprune run -- bundle exec rails test test/controllers/
+testprune scan -- bundle exec rspec spec/models
+testprune scan -- bundle exec ruby -Itest test/my_test.rb
+testprune scan -- bundle exec rails test test/controllers/
 
 # Restrict which source files are analyzed (-s is repeatable)
-testprune run -s app -s lib -s packs
+testprune scan -s app -s lib -s packs
+```
+
+When run with no arguments, `testprune scan` checks your `test/` directory and flags any subdirectories whose names contain `selenium`, `request`, `piper`, or `integration` — folders that tend to be slow, browser-driven, or external and are generally poor candidates for coverage analysis. You'll be prompted to exclude them before the run starts:
+
+```
+testprune: found folders that may be slow or integration-heavy:
+  test/integration
+  test/selenium
+Include them in this run? [y/N]:
 ```
 
+Answering `N` (the default) scopes the run to the remaining folders using `rails test`. Answering `y` proceeds with the normal autodetected command.
+
 Writes `tmp/.testprune/run.json` — per-test coverage deltas and wall times. This is the only step that boots your suite.
 
+### One-step workflow
+
+```sh
+testprune prune          # scan + apply in sequence — boots suite, then prompts for approval
+testprune prune test/models test/jobs -s app -s lib
+testprune prune -- bundle exec rails test test/models/
+```
+
+`prune` runs `scan` to capture coverage data, then immediately runs `apply` — which prints the report and prompts before writing any patch. Use this when you want the full workflow without running three separate commands.
+
 ### Step 2 — Report
 
 ```sh
@@ -154,7 +183,7 @@ Removed tests are **commented out** (not deleted) with a reason annotation — y
        │
        ▼
 ┌─────────────────────────────────────────────────────────────────────┐
-│  CAPTURE   (testprune run)                                          │
+│  CAPTURE   (testprune scan)                                         │
 │                                                                     │
 │  Coverage.setup(lines:, branches:, methods:)                        │
 │       │                                                             │
@@ -318,37 +347,47 @@ removal** — testprune can't tell what it uniquely exercises.
 ### Minitest project (standard layout)
 
 ```sh
-testprune run -s app -s lib
+testprune scan -s app -s lib
 testprune report -s app -s lib
 testprune apply -s app -s lib
+
+# or all at once:
+testprune prune -s app -s lib
 ```
 
 ### RSpec project
 
 ```sh
-testprune run -s app -s lib -- bundle exec rspec
+testprune scan -s app -s lib -- bundle exec rspec
 testprune report -s app -s lib
 ```
 
 ### Rails app
 
 ```sh
-# Run a specific directory (never run the whole suite without a path)
-testprune run -s app -s lib -s packs -- bundle exec rails test test/models/
+# Scan with no args — testprune will prompt to exclude selenium/integration folders
+testprune scan -s app -s lib
+
+# Scope to specific directories (positional args, no -- needed)
+testprune scan -s app -s lib test/models
+testprune scan -s app -s lib test/models test/jobs test/services
 
-# Multiple passes — capture incrementally by domain, analyze together
-testprune run -s app -- bundle exec rails test test/models/
-testprune run -s app -- bundle exec rails test test/controllers/
-# run.json is overwritten on each `testprune run`; analyze each pass separately
+# Explicit command if you need full control
+testprune scan -s app -s lib -s packs -- bundle exec rails test test/models/
+
+# Full workflow in one step
+testprune prune -s app -s lib test/models test/jobs
 ```
 
+> **Note:** `run.json` is overwritten on each `testprune scan`. Run report/apply after each capture pass.
+
 ### Rails app with Spring
 
 Spring preloads your app but forks the test process — the forked child doesn't
 inherit `RUBYOPT` where testprune's autostart lives. Disable it for the run:
 
 ```sh
-DISABLE_SPRING=1 testprune run -s app -s lib -- bundle exec rails test test/models/
+DISABLE_SPRING=1 testprune scan -s app -s lib -- bundle exec rails test test/models/
 ```
 
 ### Projects using SimpleCov (or other coverage gems)
@@ -379,11 +418,11 @@ rv run --ruby 3.2 env RUBYOPT="-I$(gem contents testprune | grep autostart | xar
 
 # Simpler: install testprune under the managed ruby so RUBYOPT is not needed
 rv run --ruby 3.2 gem install testprune
-rv run --ruby 3.2 bundle exec testprune run
+rv run --ruby 3.2 bundle exec testprune scan
 ```
 
 The easiest path is always to install testprune under the same Ruby that runs
-your suite. If `TESTPRUNE_DEBUG=1 testprune run` prints nothing from `[testprune-debug]`,
+your suite. If `TESTPRUNE_DEBUG=1 testprune scan` prints nothing from `[testprune-debug]`,
 the autostart never loaded — this is the cause.
 
 ### Monorepo / packs
@@ -392,7 +431,7 @@ Run each pack separately and analyze with its source path:
 
 ```sh
 # Capture one pack
-testprune run -s packs/tenancy/app -- \
+testprune scan -s packs/tenancy/app -- \
   bundle exec rails test packs/tenancy/test/
 
 # Report for that pack
@@ -427,7 +466,7 @@ jobs:
           ruby-version: '3.2'
           bundler-cache: true
       - run: gem install testprune
-      - run: testprune run -s app -s lib -- bundle exec rails test test/models/
+      - run: testprune scan -s app -s lib -- bundle exec rails test test/models/
       - run: testprune report -s app -s lib --json > testprune-report.json
       - uses: actions/upload-artifact@v4
         with:
@@ -462,9 +501,9 @@ testprune report -s app -s lib || true
 
 | Variable | Effect |
 |----------|--------|
-| `TESTPRUNE_ROOT` | Set the project root (default: `Dir.pwd`). Set automatically by `testprune run`. |
-| `TESTPRUNE_SOURCE_PATHS` | Colon-separated source paths. Set automatically by `testprune run`. |
-| `TESTPRUNE_OUTPUT_DIR` | Output directory. Set automatically by `testprune run`. |
+| `TESTPRUNE_ROOT` | Set the project root (default: `Dir.pwd`). Set automatically by `testprune scan`. |
+| `TESTPRUNE_SOURCE_PATHS` | Colon-separated source paths. Set automatically by `testprune scan`. |
+| `TESTPRUNE_OUTPUT_DIR` | Output directory. Set automatically by `testprune scan`. |
 | `TESTPRUNE_DEBUG` | Print adapter-load diagnostics (`[testprune-debug] autostart loaded in pid …`). Useful when capture produces no `run.json`. |
 | `DISABLE_SPRING` | Disable Spring preloader so the test process inherits testprune's instrumentation. |
 
@@ -494,7 +533,7 @@ overhead. On very large suites (10k+ tests) this is noticeable but acceptable
 it's mitigated by restricting `--source` to the paths you care about.
 
 **run.json is machine-local.** Coverage paths are absolute. Don't run
-`testprune run` on CI and `testprune report` on a laptop with a different home
+`testprune scan` on CI and `testprune report` on a laptop with a different home
 directory — the paths won't match. Always run all three commands on the same
 machine.
 

data/lib/testprune/analysis.rb

@@ -29,7 +29,7 @@ module Testprune
       end
 
       unless File.exist?(@config.run_file)
-        raise Error, "no captured data at #{@config.run_file}. Run `testprune run` first."
+        raise Error, "no coverage data found. Run `testprune run` first to capture per-test coverage."
       end
 
       run = begin

data/lib/testprune/cli.rb

@@ -9,18 +9,22 @@ module Testprune
   #   report  analyzes run.json and prints grouped candidates (read-only)
   #   apply   prompts for approval, then writes a removal patch (never edits in place)
   class CLI
+    NOISY_PATTERNS = %w[selenium request piper integration].freeze
+
     BANNER = <<~TXT
       testprune — audit a Ruby test suite for redundant coverage
 
       Usage:
-        testprune run [options] [-- <test command>]
+        testprune scan [options] [-- <test command>]
         testprune report [options]
         testprune apply [options]
+        testprune prune [options] [-- <test command>]
 
       Commands:
-        run      Run the target suite instrumented; capture per-test coverage + timing
+        scan     Run the target suite instrumented; capture per-test coverage + timing
         report   Analyze captured data and print removal candidates (read-only)
         apply    Review candidates, ask for approval, emit a git-applyable patch
+        prune    Run scan + apply in one step (the full workflow)
 
       Options:
         -s, --source PATH      Source dir to analyze (repeatable; default: app, lib)
@@ -40,9 +44,10 @@ module Testprune
       argv = argv.dup
       command = argv.shift
       case command
-      when 'run'            then cmd_run(argv)
+      when 'scan'           then cmd_scan(argv)
       when 'report'         then cmd_report(argv)
       when 'apply'          then cmd_apply(argv)
+      when 'prune'          then cmd_prune(argv)
       when '-v', '--version' then puts(Testprune::VERSION)
       when nil, '-h', '--help' then puts(BANNER)
       else
@@ -88,12 +93,46 @@ module Testprune
       end
     end
 
-    def cmd_run(argv)
+    def cmd_scan(argv)
       cmd_argv, test_command = split_test_command(argv)
-      opts, = parse_options(cmd_argv)
+      opts, paths = parse_options(cmd_argv)
       apply_config(opts)
       require_relative 'runner'
-      Runner.new(Testprune.config).call(test_command)
+      runner = Runner.new(Testprune.config)
+      if test_command.nil? && paths.empty?
+        test_command = prompt_noisy_exclusions(runner)
+      elsif test_command.nil?
+        test_command = runner.command_for_paths(paths)
+      end
+      runner.call(test_command)
+    end
+
+    def prompt_noisy_exclusions(runner)
+      test_dir = File.join(Testprune.config.root, 'test')
+      return nil unless File.directory?(test_dir)
+
+      all_subdirs = Dir.children(test_dir)
+                       .select { |d| File.directory?(File.join(test_dir, d)) }
+                       .sort
+      noisy = all_subdirs.select { |d| NOISY_PATTERNS.any? { |p| d.downcase.include?(p) } }
+      return nil if noisy.empty?
+
+      warn("testprune: found folders that may be slow or integration-heavy:")
+      noisy.each { |d| warn("  test/#{d}") }
+      $stderr.print("Include them in this run? [y/N]: ")
+      answer = $stdin.gets&.strip&.downcase
+
+      return nil if %w[y yes].include?(answer)
+
+      kept = (all_subdirs - noisy).map { |d| "test/#{d}" }
+      return nil if kept.empty?
+
+      runner.command_for_paths(kept)
+    end
+
+    def cmd_prune(argv)
+      cmd_scan(argv)
+      cmd_apply([])
     end
 
     def cmd_report(argv)

data/lib/testprune/runner.rb

@@ -39,6 +39,18 @@ module Testprune
       ok
     end
 
+    def command_for_paths(paths)
+      bundler = File.exist?(File.join(@config.root, 'Gemfile'))
+      prefix  = bundler ? %w[bundle exec] : []
+      if File.directory?(File.join(@config.root, 'spec'))
+        prefix + %w[rspec] + paths
+      elsif File.exist?(File.join(@config.root, 'bin', 'rails'))
+        prefix + %w[rails test] + paths
+      else
+        prefix + %w[rake test] + paths
+      end
+    end
+
     private
 
     def env

data/lib/testprune/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Testprune
-  VERSION = '0.1.0'
+  VERSION = '0.3.0'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: testprune
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Seth MacPherson
@@ -83,7 +83,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.9
+rubygems_version: 4.0.3
 specification_version: 4
 summary: Audits a Ruby test suite for duplicate/redundant coverage using Prism AST
   + Coverage data