reviewer 0.1.5 → 1.0.0

data/.reviewer.yml

@@ -1,32 +1,42 @@
 bundle_audit:
   name: Bundle Audit
   description: Review Gem Dependencies for Security Issues
-  tags: [critical, dependencies, ruby, dev]
+  tags: [critical, security, dependencies, ruby, dev]
   links:
     home: https://github.com/rubysec/bundler-audit
     install: https://github.com/rubysec/bundler-audit#install
   commands:
-    install: 'bundle exec gem install bundler-audit'
-    prepare: 'bundle exec bundle-audit update'
-    review: 'bundle exec bundle-audit check --no-update'
+    install: "bundle exec gem install bundler-audit"
+    prepare: "bundle exec bundle-audit update"
+    review: "bundle exec bundle-audit check --no-update"
 
 tests:
   name: Minitest
   description: Unit Tests & Coverage
   tags: [ruby, tests, dev]
+  summary:
+    pattern: '(\d+)\s+tests?'
+    label: '\1 tests'
   links:
     home:
   commands:
     review: "bundle exec rake"
+  files:
+    review: "bundle exec ruby -Itest"
+    flag: ""
+    separator: " "
+    pattern: "*_test.rb"
+    map_to_tests: "minitest"
   reports:
     open_on_fail: true
     local_file: coverage/index.html
-    local_uri: 'file:///Users/garrettdimon/Code/reviewer/coverage/index.html#_AllFiles'
+    local_uri: "file:///Users/garrettdimon/Code/reviewer/coverage/index.html#_AllFiles"
   env:
     testopts: --seed=$SEED
     # coverage: false
 
 reek:
+  # skip_in_batch: true # Reek provides guidance and is generally run solo
   name: Reek
   description: Examine Ruby Classes for Code Smells
   tags: [ruby, quality, dev]
@@ -34,14 +44,18 @@ reek:
     home: https://github.com/troessner/reek
     install: https://github.com/troessner/reek#quickstart
   commands:
-    install: 'bundle exec gem install reek'
-    review: 'bundle exec reek'
+    install: "bundle exec gem install reek"
+    review: "bundle exec reek --config .reek.yml lib/"
+  files:
+    flag: ""
+    separator: " "
+    pattern: "*.rb"
   flags:
     color:
     documentation:
 
 flog:
-  disabled: true # Flog provides guidance and is generally run solo
+  # skip_in_batch: true # Flog provides guidance and is generally run solo
   name: Flog
   description: Reports your most tortured ruby code in an easy to read pain report.
   tags: [ruby, quality, dev]
@@ -49,15 +63,15 @@ flog:
     home: https://ruby.sadi.st/Flog.html
     install: https://ruby.sadi.st/Flog.html
   commands:
-    install: 'bundle exec gem install flog'
-    review: 'bundle exec flog -g lib'
+    install: "bundle exec gem install flog"
+    review: "bundle exec flog -g lib"
   flags:
     threshold: 10 # Percentage. i.e. Show the most concerning 10% of results
     group:
     methods-only:
 
 flay:
-  disabled: true # Flay provides guidance and is generally run solo
+  # skip_in_batch: true # Flay provides guidance and is generally run solo
   name: Flay
   description: Review ruby code for structural similarities and refactoring opportunities.
   tags: [ruby, quality, dev]
@@ -65,37 +79,39 @@ flay:
     home: https://ruby.sadi.st/Flay.html
     install: https://ruby.sadi.st/Flay.html
   commands:
-    install: 'bundle exec gem install flay'
-    review: 'bundle exec flay ./lib'
+    install: "bundle exec gem install flay"
+    review: "bundle exec flay ./lib"
   flags:
     liberal:
     summary:
 
-
 rubocop:
   name: Rubocop
   description: Review Ruby Syntax & Formatting for Consistency
-  tags: [ruby, syntax]
+  tags: [ruby, syntax, dev]
+  summary:
+    pattern: '(\d+)\s+offenses?'
+    label: '\1 offenses'
   links:
     home: https://rubocop.org
     install: https://docs.rubocop.org/rubocop/1.13/installation.html
     ignore_syntax: https://docs.rubocop.org/rubocop/configuration.html#ignoredmethods
     disable_syntax: https://docs.rubocop.org/rubocop/configuration.html#disabling-cops-within-source-code
   commands:
-    install: 'bundle exec gem install rubocop'
-    review: 'bundle exec rubocop --parallel'
-    format: 'bundle exec rubocop --auto-correct'
+    install: "bundle exec gem install rubocop"
+    review: "bundle exec rubocop --parallel"
+    format: "bundle exec rubocop --auto-correct"
   files:
-    flag: ''
-    separator: ' '
+    flag: ""
+    separator: " "
+    pattern: "*.rb"
   flags:
     color:
 
-
 # The YARD Parser can also come in handy:
 #   https://yardoc.org/types
 inch:
-  disabled: true
+  # skip_in_batch: true
   tags: [docs, ruby, dev]
   name: Inch
   description: Review Ruby Documentation Quality
@@ -103,12 +119,11 @@ inch:
     home: https://rrrene.org/inch/
     usage: https://www.rubydoc.info/gems/yard/file/docs/Tags.md
   commands:
-    install: 'bundle exec gem install inch'
-    review: 'bundle exec inch'
-
+    install: "bundle exec gem install inch"
+    review: "bundle exec inch"
 
 yard:
-  disabled: true
+  # skip_in_batch: true
   tags: [docs, ruby, dev]
   name: Yard
   description: Generates Documentation
@@ -117,24 +132,135 @@ yard:
     usage: https://www.rubydoc.info/gems/yard/file/docs/GettingStarted.md#using
     install: https://github.com/lsegal/yard#installing
   commands:
-    install: 'bundle exec gem install yard'
+    install: "bundle exec gem install yard"
     prepare: "yard doc 'lib/**/*.rb'"
-    review: 'yard stats'
-    serve: 'yard server --reload && open http://0.0.0.0:8808'
+    review: "yard stats"
+    serve: "yard server --reload && open http://0.0.0.0:8808"
     generate:
-    quiet_flag: '--quiet'
-
+    quiet_flag: "--quiet"
 
 alex:
-  disabled: true
+  skip_in_batch: true
   tags: [language, docs]
-  name: Yard
-  description: Generates Documentation
+  name: Alex
+  description: Catch insensitive, inconsiderate writing
   links:
-    home:
-    usage:
-    install:
+    home: https://alexjs.com
+  commands:
+    install: "yarn global add alex"
+    review: "alex ."
+
+# New tools for expanded testing coverage
+
+brakeman:
+  skip_in_batch: true # Requires Rails application, not applicable to gems
+  name: Brakeman
+  description: Static analysis security scanner for Ruby
+  tags: [security, ruby, dev]
+  links:
+    home: https://brakemanscanner.org
+    install: https://brakemanscanner.org/docs/install/
+  commands:
+    install: "bundle exec gem install brakeman"
+    review: "bundle exec brakeman --no-pager -q"
+
+fasterer:
+  name: Fasterer
+  description: Suggest performance improvements for Ruby code
+  tags: [ruby, quality, performance, dev]
+  links:
+    home: https://github.com/DamirSvrtan/fasterer
+  commands:
+    install: "bundle exec gem install fasterer"
+    review: "bundle exec fasterer"
+  files:
+    flag: ""
+    separator: " "
+    pattern: "*.rb"
+
+debride:
+  skip_in_batch: true # Broken on Ruby 3.4 (missing require for Racc, Timeout)
+  name: Debride
+  description: Find potentially unused methods and classes
+  tags: [ruby, quality, dev]
+  links:
+    home: https://github.com/seattlerb/debride
+  commands:
+    install: "bundle exec gem install debride"
+    review: "bundle exec debride lib/"
+
+rubycritic:
+  skip_in_batch: true # Generates HTML reports, best run manually
+  name: RubyCritic
+  description: Code quality analysis with HTML reports
+  tags: [ruby, quality, dev]
+  links:
+    home: https://github.com/whitesmith/rubycritic
+  commands:
+    install: "bundle exec gem install rubycritic"
+    review: "bundle exec rubycritic lib/ --no-browser -f console"
+
+metric_fu:
+  skip_in_batch: true # Meta-tool that runs many others, slow
+  name: MetricFu
+  description: Aggregate code metrics from multiple tools
+  tags: [ruby, quality, dev]
+  links:
+    home: https://github.com/metricfu/metric_fu
+  commands:
+    install: "bundle exec gem install metric_fu"
+    review: "bundle exec metric_fu"
+
+yardstick:
+  # skip_in_batch: true # Documentation coverage measurement
+  name: Yardstick
+  description: Measure YARD documentation coverage
+  tags: [docs, ruby, dev]
+  links:
+    home: https://github.com/dkubb/yardstick
+  commands:
+    install: "bundle exec gem install yardstick"
+    review: "bundle exec yardstick lib/"
+
+standard:
+  skip_in_batch: true # Conflicts with existing RuboCop configuration
+  name: Standard
+  description: Zero-configuration Ruby linter
+  tags: [ruby, syntax, dev]
+  links:
+    home: https://github.com/standardrb/standard
+  commands:
+    install: "bundle exec gem install standard"
+    review: "bundle exec standardrb"
+    format: "bundle exec standardrb --fix"
+
+rufo:
+  skip_in_batch: true # Alternative formatter to RuboCop
+  name: Rufo
+  description: Ruby code formatter
+  tags: [ruby, syntax, dev]
+  links:
+    home: https://github.com/ruby-formatter/rufo
+  commands:
+    install: "bundle exec gem install rufo"
+    review: "bundle exec rufo --check lib/"
+    format: "bundle exec rufo lib/"
+
+license_finder:
+  skip_in_batch: true # Slow - run manually with: rvw license_finder
+  name: License Finder
+  description: Check dependency licenses for compliance
+  tags: [dependencies, compliance, slow, dev]
+  links:
+    home: https://github.com/pivotal/LicenseFinder
+  commands:
+    install: "bundle exec gem install license_finder"
+    review: "bundle exec license_finder --decisions-file=./dependency_decisions.yml"
+
+notes:
+  name: Notes
+  description: Find TODO, FIXME, HACK, and OPTIMIZE comments
+  tags: [quality, dev]
   commands:
-    install: 'yarn global add alex'
-    review: 'alex .'
-    quiet_flag: '--quiet'
+    review: "grep -rn --include='*.rb' -E '(TODO|FIXME|HACK|OPTIMIZE|XXX):?' lib/ test/ || true"
+  max_exit_status: 1

data/.rubocop.yml

@@ -1,6 +1,7 @@
 AllCops:
-  TargetRubyVersion: 2.5
+  TargetRubyVersion: 3.2
   NewCops: enable
+  SuggestExtensions: false
   UseCache: true
   CacheRootDirectory: './'
   Exclude:
@@ -12,9 +13,40 @@ Layout/LineLength:
 
 # One case statement in a single method isn't complex.
 Metrics/CyclomaticComplexity:
-  IgnoredMethods: ['case']
+  AllowedMethods: ['case']
+
+# Development dependencies in gemspec is fine for this gem
+Gemspec/DevelopmentDependencies:
+  Enabled: false
+
+# Keyword arguments are self-documenting at call sites, so they don't
+# create the readability problem that long positional lists do.
+Metrics/ParameterLists:
+  CountKeywordArgs: false
+
+# The CLI entry point is a routing method with one guard clause per line.
+# Each branch is a single method call — the structure is as clear as it gets.
+Metrics/AbcSize:
+  AllowedMethods: ['handle_early_exits']
 
 # 10 is a good goal but a little draconian
 Metrics/MethodLength:
   CountAsOne: ['array', 'hash', 'heredoc']
   Max: 15
+
+# Test classes naturally grow with coverage (see test/.rubocop.yml for higher limit)
+Metrics/ClassLength:
+  CountAsOne: ['array', 'hash', 'heredoc']
+  Max: 150
+
+Metrics/ModuleLength:
+  CountAsOne: ['array', 'hash', 'heredoc']
+  Max: 150
+
+# Fasterer prefers yield over block.call for performance
+Style/ExplicitBlockArgument:
+  Enabled: false
+
+# Fasterer prefers fetch with block over second argument for performance
+Style/RedundantFetchBlock:
+  Enabled: false

data/CHANGELOG.md

@@ -1,7 +1,47 @@
 ## [Unreleased]
 
-- TODO: Improve and streamline installation
-- TODO: Add support for targeting specific files
+## [1.0.0] - 2026-02-03
+
+### Added
+- Git-aware file targeting: `staged`, `unstaged`, `modified`, `untracked` keywords resolve files from git status and pass only relevant files to each tool
+- `failed` keyword: re-run only tools that failed in the previous run, scoped to their failed files
+- `--json` / `-j` flag for structured JSON output (CI, scripting, agent integration)
+- `--raw` / `-r` flag to force passthrough output (bypasses capturing)
+- `--format` flag with streaming, summary, and json modes
+- `--capabilities` / `-c` flag for agent discovery (outputs JSON describing tools, keywords, scenarios)
+- `skip_in_batch` config option: exclude tools from `rvw` while keeping them available via `rvw tool_name`
+- `files.pattern` config: glob pattern to filter which files are passed to each tool
+- `files.map_to_tests` config: map source files to test files (`minitest` or `rspec` conventions)
+- `files.review` / `files.format` config: alternative commands when files are scoped
+- First-run experience: interactive setup when no `.reviewer.yml` exists
+- `rvw init` command: auto-detect tools from Gemfile.lock and generate `.reviewer.yml`
+- `rvw doctor` command: diagnostics for configuration, tools, keywords, and environment
+- Keyword resolution summary: preview which tools and files will run before execution
+- Spell-check suggestions for mistyped keywords
+- Auto-detection catalog: bundler-audit, rubocop, standard, reek, flog, flay, brakeman, fasterer, minitest, rspec, eslint, prettier, stylelint, typescript, biome
+- Progress bar for captured output with timing estimates
+- PTY-based streaming capture for failed file extraction from single-tool runs
+
+### Fixed
+- Console width returns default in piped/CI contexts instead of 0
+- `rvw failed` no longer crashes on empty tag objects
+- MIT license restored in LICENSE.txt
+- `--help` and `--version` exit immediately instead of running tool suite
+- Valid JSON emitted for early exits (no matching tools, no files)
+
+### Changed
+- **Ruby 3.2+ required** (supports 3.2, 3.3, 3.4, and 4.0)
+- Architecture refactor: full dependency injection, no global state in business logic
+- Output decomposed into domain formatters (Runner, Batch, Session, Doctor, Setup, Report)
+- Session class owns run lifecycle; Reviewer module is pure wiring
+- Context struct threads shared dependencies through the call stack
+- Tool timing extracted to Tool::Timing collaborator with injected history
+- Result interpretation separated from Runner execution (Result.from_runner)
+- Tests no longer depend on global state or require reset between runs
+- Redesigned output: compact `↳ command` format, `✓ ~Xs` checkmark summaries
+- ANSI color output guarded for TTY (clean output in CI and pipes)
+- README rewritten with installation, usage, configuration, and workflow documentation
+- `disabled` config key deprecated in favor of `skip_in_batch`
 
 ## [0.1.4] - 2021-07-08
 

data/Gemfile

@@ -5,4 +5,42 @@ source 'https://rubygems.org'
 # Specify your gem's dependencies in reviewer.gemspec
 gemspec
 
-gem 'rake', '~> 13.0'
+gem 'rake', '~> 13.2'
+
+# Security auditing - always run in CI
+gem 'bundler-audit'
+
+# Self-review tools - not required for running tests
+# Install with: bundle config set --local with lint && bundle install
+group :lint, optional: true do
+  # Ruby 3.4+ requires explicit racc (no longer a default gem)
+  gem 'racc'
+
+  # Style & Linting
+  gem 'rubocop'
+  gem 'rubocop-minitest'
+  gem 'rubocop-rake'
+  gem 'standard', '>= 1.35.1'
+
+  # Code Quality & Complexity
+  gem 'debride'
+  gem 'fasterer'
+  gem 'flay'
+  gem 'flog'
+  gem 'metric_fu'
+  gem 'reek'
+  gem 'rubycritic'
+
+  # Security
+  gem 'brakeman'
+
+  # Documentation
+  gem 'inch'
+  gem 'yardstick'
+
+  # Formatting
+  gem 'rufo'
+
+  # Compliance
+  gem 'license_finder'
+end