rails_stats 2.1.0 → 2.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a444e6b90de2ab6eead56780299fc2b2a24811f6318877007238eb45a4be0591
-  data.tar.gz: c790a1e79d70ea1ab78476052bf27b71cc3491b63c8b7bd2a2d180e81a817aa0
+  metadata.gz: 6c0ec937a76e806cef21dd005c123dc0cea07bdf4f83d565ee8d141b3169dda2
+  data.tar.gz: e9d5aa5cf96dd02b9cd26247e2a41fa0aa44d812230176e1fd3af76e1a3bd593
 SHA512:
-  metadata.gz: b3971dca0766fd9d81aa927823111c80d5de66fe6b70e20fbc620395d5e95f1a847f9b2c6ff09dc51611423a444e797cb1d373a5f87ad7df8b58f18ca68ce50d
-  data.tar.gz: 3d250b31139d4abf61a1c835e0f4022c17b62e57d57646935548f3a2086b1712b42dfd9a69d7d447aeeff465e69842660332f75faff87eed58f6c2d27e0b024a
+  metadata.gz: fcf370eb0f981441c2ea4c7a4ae54027c10daa03be92aa23b1c0455d95b98b697477dc04b18d7f9afe16f9a6b1bbdd3d3642b5acfeb7d3dcf5f7bf1395163e5a
+  data.tar.gz: fb0d687f57af91b8bb48caf85a4c9bfbc05a49ed4fdc73f7bf4d521a70a1824a84956854af11238d278eccf785c38baeb762964c0b5eb3b5b4b49335730fc488

data/CHANGELOG.md

@@ -1,5 +1,12 @@
-# main ([unreleased](https://github.com/fastruby/rails_stats/compare/v2.1.0...main))
+# main ([unreleased](https://github.com/fastruby/rails_stats/compare/v2.2.0...main))
 
+* [FEATURE: ABC](LINK)
+
+# v2.2.0 ([commits](https://github.com/fastruby/rails_stats/compare/v2.2.0...v2.1.0))
+
+* [FEATURE: Add a `rails-stats` command line executable that accepts an absolute or relative directory](https://github.com/fastruby/rails_stats/pull/49)
+* [FEATURE: Support component-based and Packwerk (packs) applications with a per-pack and Core breakdown](https://github.com/fastruby/rails_stats/issues/23)
+* [BUGFIX: Respect an explicit path argument (e.g. `rake stats[packs/pack1]`) when running inside a booted Rails app](https://github.com/fastruby/rails_stats/issues/23)
 * [CHORE: Improve the GH Test Workflow](https://github.com/fastruby/rails_stats/pull/35)
 * [BUGFIX: Explicitly set format as text for `Bundler::Stats::CLI` on `ConsoleFormatter`](https://github.com/fastruby/rails_stats/pull/43)
 

data/README.md

@@ -6,22 +6,62 @@ There were a few things missing to the included `rake stats`
 
 RailsStats mainly adds the ability to be run from outside the project in question. This can be helpful if the app you are interested in can not be booted for some reason.
 
-### Run it outside Rails project
+### Run it as a command line tool
+
+Once the gem is installed you can point the `rails-stats` executable at any
+directory, without setting up a `Rakefile`. It accepts both absolute and
+relative paths:
+
+```bash
+$ rails-stats /path/to/app
+$ rails-stats path/to/app
+```
 
-You will need a `Rakefile` in the directory where you call `rake` and you will
-need to require `rails_stats`:
+When no path is given it defaults to the current directory:
 
-```ruby
-# Rakefile
-require "rails_stats"
+```bash
+$ rails-stats
+```
+
+You can also request JSON output by passing `json` as a second argument:
+
+```bash
+$ rails-stats path/to/app json
 ```
 
-Then you can call it:
+#### Calling it through `rake`
+
+If you prefer `rake`, there are two ways to reach the same code:
+
+- **From any directory**, add a `Rakefile` that requires `rails_stats` and call
+  the task with a path:
+
+  ```ruby
+  # Rakefile
+  require "rails_stats"
+  ```
+
+  ```bash
+  $ rake stats\[/path/to/app/\]
+  ```
+
+- **Inside your own Rails app**, add `rails_stats` to your `Gemfile` to
+  _replace_ the default `rake stats` implementation (you might need to
+  `require "rails_stats"` in your `Rakefile`), then run:
+
+  ```bash
+  $ bundle exec rake stats
+  ```
+
+### Run it outside Rails project
+
+The quickest way is the `rails-stats` executable, which does not require a
+`Rakefile`:
 
 ```bash
-$ rake stats\[/path/to/app/\]
+$ rails-stats /path/to/app
 
-Directory: /path/to/app/
+Directory: /path/to/app
 
 +-----------------------|------------|----------------+
 |                  Name | Total Deps | 1st Level Deps |
@@ -68,20 +108,57 @@ Directory: /path/to/app/
 ### Run it on many Rails engines
 
 ```bash
-$ for dir in /path/to/many/engines/*/; do bundle exec rake stats[$dir]; done
+$ for dir in /path/to/many/engines/*/; do rails-stats "$dir"; done
 ```
 
-### Within your Rails project
+### Component-based and Packwerk (packs) applications
 
-You can also include it within your Rails application to _replace_ the default `rake stats` implementation.
+RailsStats understands large Rails monoliths that are organized into
+[packs](https://github.com/Shopify/packwerk) (via
+[packs-rails](https://github.com/rubyatscale/packs-rails)) or
+[components](https://cbra.info/) (component-based Rails).
 
-Just add rails_stats to your Gemfile.
-Depending on your setup, you might need to `require rails_stats` in your Rakefile.
+When packs or components are detected, `rails-stats` automatically prints a
+breakdown per pack/component, a `Core Application` section for everything that
+lives in the main app, and a `Total (all packs)` section that aggregates the
+whole monolith:
+
+```bash
+$ rails-stats /path/to/monolith
+
+== Core Application ==
++----------------------+---------+---------+---------+---------+---------+-----+-------+
+| Name                 | Files   | Lines   |     LOC | Classes | Methods | M/C | LOC/M |
++----------------------+---------+---------+---------+---------+---------+-----+-------+
+| ...                                                                                 |
++----------------------+---------+---------+---------+---------+---------+-----+-------+
+
+== Pack: packs/pack1 ==
++----------------------+---------+---------+---------+---------+---------+-----+-------+
+| ...                                                                                 |
++----------------------+---------+---------+---------+---------+---------+-----+-------+
+
+== Total (all packs) ==
++----------------------+---------+---------+---------+---------+---------+-----+-------+
+| ...                                                                                 |
++----------------------+---------+---------+---------+---------+---------+-----+-------+
+```
+
+A directory is treated as a pack when it contains a `package.yml` file (the
+Packwerk marker) or when it lives directly under a `packs/` or `components/`
+directory and holds any of the usual code folders (`app`, `lib`, `spec`,
+`test`). Nested packs (`packs/a/packs/b`) are supported.
+
+You can also scope stats to a single pack:
+
+```bash
+$ rails-stats packs/pack1
+```
 
-Then you'll be able to just run:
+The same breakdown is available in JSON under a `"packs"` key:
 
 ```bash
-$ bundle exec rake stats
+$ rails-stats . json
 ```
 
 ### Things it knows about
@@ -101,7 +178,7 @@ Here are some open source Rails projects and their output.
 
 ```bash
 
-$ bundle exec rake stats[/users/brian/examples/redmine/]
+$ rails-stats /users/brian/examples/redmine/
 
 +-----------------------|------------|----------------+
 |                  Name | Total Deps | 1st Level Deps |
@@ -190,7 +267,7 @@ $ bundle exec rake stats[/users/brian/examples/redmine/]
 If you want to export the details using JSON, you can use this command:
 
 ```
-$ rake stats\[test/dummy,json\]
+$ rails-stats test/dummy json
 
 Directory: /Users/etagwerker/Projects/redmine
 

data/bin/rails-stats

@@ -0,0 +1,27 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+# Standalone CLI for running rails_stats against a directory without having to
+# set up a Rakefile. Accepts both absolute and relative paths:
+#
+#   rails-stats /path/to/app
+#   rails-stats path/to/app
+#   rails-stats                 # defaults to the current directory
+#   rails-stats path/to/app json
+#
+$LOAD_PATH.unshift(File.expand_path("../lib", __dir__))
+
+require "rails_stats/all"
+
+path = ARGV[0] || "."
+format = ARGV[1] || ""
+
+root_directory = File.absolute_path(path)
+
+unless File.directory?(root_directory)
+  warn "rails-stats: no such directory - #{path}"
+  exit 1
+end
+
+puts "\nDirectory: #{root_directory}\n\n"
+RailsStats::CodeStatistics.new(root_directory, format: format).to_s

data/lib/rails_stats/all.rb

@@ -1,9 +1,17 @@
+module RailsStats
+  # Vendored / generated directory names we never descend into or treat as a
+  # pack. Shared by PackFinder (which builds IGNORED_PATH from it) and Util
+  # (which matches them by basename while walking the tree).
+  IGNORED_DIRS = %w[node_modules vendor tmp log .git].freeze
+end
+
 require 'rails_stats/stats_calculator'
 require 'rails_stats/stats_formatter'
 require 'rails_stats/json_formatter'
 require 'rails_stats/console_formatter'
 require 'rails_stats/inflector'
 require 'rails_stats/code_statistics_calculator'
+require 'rails_stats/pack_finder'
 require 'rails_stats/util'
 require 'rails_stats/app_statistics'
 require 'rails_stats/test_statistics'

data/lib/rails_stats/app_statistics.rb

@@ -1,6 +1,6 @@
 module RailsStats
   class AppStatistics
-    attr_reader :statistics, :total, :test
+    attr_reader :statistics, :total, :test, :directory
 
     def initialize(directory)
       @directories = []

data/lib/rails_stats/console_formatter.rb

@@ -5,16 +5,10 @@ module RailsStats
     def to_s
       Bundler::Stats::CLI.start(['--format', 'text'])
 
-      print_header
-      sorted_keys = @statistics.keys.sort
-      sorted_keys.each { |key| print_line(key, @statistics[key]) }
-      print_splitter
-
-      if @grand_total
-        print_line("Code", @code_total)
-        print_line("Tests", @tests_total)
-        print_line("Total", @grand_total)
-        print_splitter
+      if calculator.packs?
+        print_grouped
+      else
+        print_flat
       end
 
       print_code_test_stats
@@ -24,6 +18,44 @@ module RailsStats
 
     private
 
+      def print_flat
+        print_table(@statistics)
+      end
+
+      def print_grouped
+        @statistics_by_group.each do |group, group_statistics|
+          puts "== #{group_label(group)} =="
+          print_table(group_statistics)
+          puts ""
+        end
+
+        puts "== #{grand_total_label} =="
+        print_table(@statistics)
+      end
+
+      def group_label(group)
+        group == RailsStats::StatsCalculator::CORE_GROUP ? "Core Application" : "Pack: #{group}"
+      end
+
+      def grand_total_label
+        "Total (all packs)"
+      end
+
+      # Prints a full table (header, one line per concept, and Code/Tests/Total
+      # totals) for the given {concept => CodeStatisticsCalculator} hash.
+      def print_table(statistics)
+        code, tests, grand = calculator.totals_for(statistics)
+
+        print_header
+        statistics.keys.sort.each { |key| print_line(key, statistics[key]) }
+        print_splitter
+
+        print_line("Code", code)
+        print_line("Tests", tests)
+        print_line("Total", grand)
+        print_splitter
+      end
+
       def print_header
         print_splitter
         puts "| Name                 | Files   | Lines   |     LOC | Classes | Methods | M/C | LOC/M |"

data/lib/rails_stats/cucumber_statistics.rb

@@ -1,6 +1,6 @@
 module RailsStats
   class CucumberStatistics
-    attr_reader :statistics, :total, :test
+    attr_reader :statistics, :total, :test, :directory
 
     def initialize(directory)
       @test = true

data/lib/rails_stats/gem_statistics.rb

@@ -1,6 +1,6 @@
 module RailsStats
   class GemStatistics
-    attr_reader :statistics, :total, :test
+    attr_reader :statistics, :total, :test, :directory
 
     def initialize(directory)
       @test = false

data/lib/rails_stats/json_formatter.rb

@@ -17,6 +17,8 @@ module RailsStats
       @result << stat_hash("Tests", @tests_total).merge(code_test_hash) if @tests_total
       @result << stat_hash("Total", @grand_total).merge(code_test_hash) if @grand_total
 
+      @result << { "packs" => packs_breakdown } if calculator.packs?
+
       @result << { "schema_stats" => schema_info }
       @result << { "polymorphic_stats" => print_polymorphic_stats }
 
@@ -29,6 +31,22 @@ module RailsStats
 
     private
 
+      # Per-pack / per-component breakdown, keyed by group name ("Core" plus
+      # each pack's relative path). Each group lists its concept stats followed
+      # by Code/Tests/Total rows for that group.
+      def packs_breakdown
+        @statistics_by_group.each_with_object({}) do |(group, group_statistics), out|
+          code, tests, grand = calculator.totals_for(group_statistics)
+
+          rows = group_statistics.map { |key, stats| stat_hash(key, stats) }
+          rows << stat_hash("Code", code).merge("total" => true)
+          rows << stat_hash("Tests", tests).merge("total" => true)
+          rows << stat_hash("Total", grand).merge("total" => true)
+
+          out[group] = rows
+        end
+      end
+
       def code_test_hash
         code  = calculator.code_loc
         tests = calculator.test_loc

data/lib/rails_stats/pack_finder.rb

@@ -0,0 +1,67 @@
+module RailsStats
+  # Detects "packs" (packwerk / packs-rails) and "components" (component-based
+  # Rails, à la Stephan Hagemann) inside a Rails application so that stats can
+  # be reported per pack/component plus a "Core" group for everything else.
+  #
+  # A directory is considered a pack when either:
+  #
+  #   * it contains a `package.yml` file (the packwerk marker), or
+  #   * it is a direct child of a `packs/` or `components/` directory and holds
+  #     any of the usual code folders (`app`, `lib`, `spec`, `test`).
+  #
+  # Nested packs (e.g. `packs/a/packs/b`) are supported. The root directory
+  # itself is never reported as a pack.
+  module PackFinder
+    extend self
+
+    CODE_FOLDERS   = %w[app lib spec test].freeze
+    PACK_CONTAINERS = %w[packs components].freeze
+
+    # Path regex used to filter glob results to skip vendored/generated
+    # directories. Built from the shared RailsStats::IGNORED_DIRS list.
+    IGNORED_PATH   = %r{/(#{IGNORED_DIRS.map { |dir| Regexp.escape(dir) }.join("|")})/}.freeze
+
+    # Returns the absolute paths of every detected pack/component, sorted.
+    def find(root_directory)
+      root = File.absolute_path(root_directory)
+      packs = {}
+
+      collect_packwerk_packs(root, packs)
+      collect_convention_packs(root, packs)
+
+      packs.keys.sort
+    end
+
+    private
+
+    def collect_packwerk_packs(root, packs)
+      Dir.glob(File.join(root, "**", "package.yml")).each do |marker_path|
+        next if marker_path =~ IGNORED_PATH
+
+        dir = File.absolute_path(File.dirname(marker_path))
+        next if dir == root
+
+        packs[dir] = true
+      end
+    end
+
+    def collect_convention_packs(root, packs)
+      PACK_CONTAINERS.each do |container|
+        Dir.glob(File.join(root, "**", container, "*")).each do |path|
+          next if path =~ IGNORED_PATH
+          next unless File.directory?(path)
+
+          dir = File.absolute_path(path)
+          next if dir == root
+          next unless code_folder?(dir)
+
+          packs[dir] = true
+        end
+      end
+    end
+
+    def code_folder?(dir)
+      CODE_FOLDERS.any? { |folder| File.directory?(File.join(dir, folder)) }
+    end
+  end
+end

data/lib/rails_stats/root_statistics.rb

@@ -1,6 +1,6 @@
 module RailsStats
   class RootStatistics
-    attr_reader :statistics, :total, :test
+    attr_reader :statistics, :total, :test, :directory
 
     ROOT_FOLDERS = {
       "lib" => "Libraries",

data/lib/rails_stats/spec_statistics.rb

@@ -1,6 +1,6 @@
 module RailsStats
   class SpecStatistics
-    attr_reader :statistics, :total, :test
+    attr_reader :statistics, :total, :test, :directory
 
     SPEC_FOLDERS = ['controllers',
                     'features',

data/lib/rails_stats/stats_calculator.rb

@@ -8,12 +8,14 @@ module RailsStats
                          'assets']
 
     def initialize(root_directory)
-      @root_directory = root_directory
+      @root_directory = File.absolute_path(root_directory)
       @schema_path    = File.join(@root_directory, "db", "schema.rb")
       @structure_path = File.join(@root_directory, "db", "structure.sql")
+      @pack_roots     = PackFinder.find(@root_directory)
       @key_concepts   = calculate_key_concepts
       @projects       = calculate_projects
       @statistics     = calculate_statistics
+      @statistics_by_group = calculate_statistics_by_group
       @code_loc       = calculate_code
       @test_loc       = calculate_tests
       @schema         = calculate_create_table_calls
@@ -21,7 +23,31 @@ module RailsStats
       @files_total, @code_total, @tests_total, @grand_total = calculate_totals
     end
 
-    attr_reader :code_loc, :code_total, :files_total, :grand_total, :statistics, :test_loc, :tests_total, :schema, :schema_path, :structure_path, :polymorphic
+    attr_reader :code_loc, :code_total, :files_total, :grand_total, :statistics, :test_loc, :tests_total, :schema, :schema_path, :structure_path, :polymorphic, :pack_roots, :statistics_by_group
+
+    CORE_GROUP = "Core".freeze
+
+    # Returns true when the application is split into packs/components, i.e.
+    # there is at least one group beyond the core application.
+    def packs?
+      @pack_roots.any?
+    end
+
+    # Given a {concept => CodeStatisticsCalculator} hash (such as one of the
+    # values in +statistics_by_group+ or the flat +statistics+), returns the
+    # [code, tests, grand] totals for that group.
+    def totals_for(group_statistics)
+      code  = CodeStatisticsCalculator.new
+      tests = CodeStatisticsCalculator.new
+      grand = CodeStatisticsCalculator.new
+
+      group_statistics.each_value do |stats|
+        grand.add(stats)
+        stats.test ? tests.add(stats) : code.add(stats)
+      end
+
+      [code, tests, grand]
+    end
 
     private
 
@@ -83,7 +109,23 @@ module RailsStats
       end
 
       def calculate_root_projects
-        [RootStatistics.new(@root_directory)]
+        projects = [RootStatistics.new(@root_directory)]
+        projects + calculate_pack_root_projects
+      end
+
+      # Packs that are not self-contained gems/engines (no gemspec) still have
+      # a `lib` (and sometimes `config`) directory that would otherwise be
+      # missed, since only the root RootStatistics scans the top-level lib.
+      # Gems are already covered by GemStatistics, so skip them here to avoid
+      # double-counting.
+      def calculate_pack_root_projects
+        @pack_roots.reject { |pack_root| gem_pack?(pack_root) }.collect do |pack_root|
+          RootStatistics.new(pack_root)
+        end
+      end
+
+      def gem_pack?(pack_root)
+        Dir[File.join(pack_root, "*.gemspec")].any?
       end
 
       def calculate_cucumber_projects
@@ -104,6 +146,62 @@ module RailsStats
         out
       end
 
+      # Same aggregation as +calculate_statistics+, but keeps each project's
+      # numbers grouped by the pack/component it belongs to (or "Core"). The
+      # returned hash is ordered with "Core" first followed by packs sorted by
+      # their relative path.
+      def calculate_statistics_by_group
+        grouped = {}
+        @projects.each do |project|
+          group = group_key_for(project.directory)
+          grouped[group] ||= {}
+          project.statistics.each do |key, stats|
+            grouped[group][key] ||= CodeStatisticsCalculator.new(project.test)
+            grouped[group][key].add(stats)
+          end
+        end
+
+        # Drop empty concepts so a pack with, say, only models does not list a
+        # bunch of zero-line rows.
+        grouped.each_value do |group_statistics|
+          group_statistics.delete_if { |_key, stats| stats.lines == 0 }
+        end
+        grouped.delete_if { |_group, group_statistics| group_statistics.empty? }
+
+        sort_groups(grouped)
+      end
+
+      def sort_groups(grouped)
+        ordered = {}
+        ordered[CORE_GROUP] = grouped[CORE_GROUP] if grouped.key?(CORE_GROUP)
+        grouped.keys.reject { |key| key == CORE_GROUP }.sort.each do |key|
+          ordered[key] = grouped[key]
+        end
+        ordered
+      end
+
+      # Maps a project directory to its group: the relative path of the pack it
+      # lives in, or "Core" when it belongs to the main application.
+      def group_key_for(directory)
+        pack_root = pack_root_for(directory)
+        return CORE_GROUP unless pack_root
+
+        relative = pack_root.sub(/\A#{Regexp.escape(@root_directory)}\/?/, "")
+        relative.empty? ? CORE_GROUP : relative
+      end
+
+      # Returns the most specific (longest matching) pack root that contains the
+      # given directory, or nil when it is not part of any pack.
+      def pack_root_for(directory)
+        directory = File.absolute_path(directory.to_s)
+        best = nil
+        @pack_roots.each do |pack_root|
+          next unless directory == pack_root || directory.start_with?("#{pack_root}/")
+          best = pack_root if best.nil? || pack_root.length > best.length
+        end
+        best
+      end
+
       def calculate_totals
         files_total = @statistics.sum do |k,value|
           value.files_total

data/lib/rails_stats/stats_formatter.rb

@@ -3,6 +3,7 @@ module RailsStats
     def initialize(calculator, opts = {})
       @calculator  = calculator
       @statistics  = calculator.statistics
+      @statistics_by_group = calculator.statistics_by_group
       @code_total  = calculator.code_total
       @tests_total = calculator.tests_total
       @grand_total = calculator.grand_total

data/lib/rails_stats/tasks.rb

@@ -3,8 +3,11 @@ task :stats, [:path, :format] do |t, args|
   Rake::Task["stats"].clear # clear out normal one if there
   require 'rails_stats/all'
 
+  # Respect an explicitly given path even when running inside a booted Rails
+  # app. Only fall back to Rails.root when no path was provided. This is what
+  # makes `rake stats[packs/pack1]` work from inside the app (see issue #23).
   path = args[:path]
-  path = Rails.root.to_s if defined?(Rails)
+  path ||= Rails.root.to_s if defined?(Rails)
   fmt = args[:format] || ""
   raise "no path given for stats" unless path
 

data/lib/rails_stats/test_statistics.rb

@@ -1,6 +1,6 @@
 module RailsStats
   class TestStatistics
-    attr_reader :statistics, :total, :test
+    attr_reader :statistics, :total, :test, :directory
 
     SPEC_FOLDERS = ['controllers',
                     'functional',

data/lib/rails_stats/util.rb

@@ -81,8 +81,16 @@ module RailsStats
       Dir.foreach(directory) do |file_name|
         path = "#{directory}/#{file_name}"
 
-        if File.directory?(path) && (/^\./ !~ file_name)
+        # Skip vendored / generated directories (e.g. a pack's
+        # app/webpack/node_modules). Pruning here avoids both the wasted walk
+        # and opening a directory that merely looks like a source file, such as
+        # node_modules/popper.js (which would raise Errno::EISDIR).
+        if File.directory?(path)
+          next if /^\./ =~ file_name
+          next if IGNORED_DIRS.include?(file_name)
+
           stats.add(calculate_directory_statistics(path, pattern))
+          next
         end
 
         next unless file_name =~ pattern

data/lib/rails_stats/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RailsStats
-  VERSION = "2.1.0"
+  VERSION = "2.2.0"
 end

data/rails_stats.gemspec

@@ -6,15 +6,16 @@ require 'rails_stats/version'
 Gem::Specification.new do |spec|
   spec.name          = "rails_stats"
   spec.version       = RailsStats::VERSION
-  spec.authors       = ["Brian Leonard"]
-  spec.email         = ["brian@bleonard.com"]
+  spec.authors       = ["Brian Leonard", "Ernesto Tagwerker"]
+  spec.email         = ["brian@bleonard.com", "ernesto@ombulabs.com"]
   spec.summary       = %q{Analyze a Rails project}
   spec.description   = %q{Point it to a directory and see stuff about the app}
   spec.homepage      = "https://github.com/fastruby/rails_stats"
   spec.license       = "MIT"
 
   spec.files         = `git ls-files -z`.split("\x0")
-  spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
+  spec.bindir        = "bin"
+  spec.executables   = ["rails-stats"]
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ["lib"]
 

data/test/fixtures/packwerk_app/app/controllers/core_controller.rb

@@ -0,0 +1,4 @@
+class CoreController
+  def index
+  end
+end

data/test/fixtures/packwerk_app/app/models/core_model.rb

@@ -0,0 +1,5 @@
+class CoreModel
+  def core_method
+    true
+  end
+end

data/test/fixtures/packwerk_app/components/billing/app/models/billing_model.rb

@@ -0,0 +1,4 @@
+class BillingModel
+  def charge
+  end
+end

data/test/fixtures/packwerk_app/components/billing/lib/billing.rb

@@ -0,0 +1,2 @@
+module Billing
+end

data/test/fixtures/packwerk_app/lib/core_lib.rb

@@ -0,0 +1,3 @@
+module CoreLib
+  CONST = 1
+end

data/test/fixtures/packwerk_app/package.yml

@@ -0,0 +1 @@
+enforce_dependencies: true

data/test/fixtures/packwerk_app/packs/pack1/app/controllers/pack1_controller.rb

@@ -0,0 +1,4 @@
+class Pack1Controller
+  def show
+  end
+end

data/test/fixtures/packwerk_app/packs/pack1/app/models/pack1_model.rb

@@ -0,0 +1,6 @@
+class Pack1Model
+  def m1
+  end
+  def m2
+  end
+end

data/test/fixtures/packwerk_app/packs/pack1/lib/pack1_lib.rb

@@ -0,0 +1,2 @@
+module Pack1Lib
+end

data/test/fixtures/packwerk_app/packs/pack1/package.yml

@@ -0,0 +1,3 @@
+enforce_dependencies: true
+dependencies:
+  - packs/pack2

data/test/fixtures/packwerk_app/packs/pack1/spec/models/pack1_model_spec.rb

@@ -0,0 +1,4 @@
+describe Pack1Model do
+  it "works" do
+  end
+end

data/test/fixtures/packwerk_app/packs/pack2/app/models/pack2_model.rb

@@ -0,0 +1,4 @@
+class Pack2Model
+  def only
+  end
+end

data/test/fixtures/packwerk_app/packs/pack2/test/models/pack2_model_test.rb

@@ -0,0 +1,4 @@
+class Pack2ModelTest
+  def test_it
+  end
+end

data/test/lib/rails_stats/console_formatter_test.rb

@@ -22,5 +22,27 @@ describe RailsStats::ConsoleFormatter do
 
       assert_equal ['--format', 'text'], received_args
     end
+
+    it 'prints a section per pack plus a Core and a grand total for packwerk apps' do
+      root_directory = File.absolute_path('./test/fixtures/packwerk_app')
+      calculator = RailsStats::StatsCalculator.new(root_directory)
+      formatter  = RailsStats::ConsoleFormatter.new(calculator)
+
+      original = Bundler::Stats::CLI.method(:start)
+      Bundler::Stats::CLI.define_singleton_method(:start) { |args = []| nil }
+
+      begin
+        output, _ = capture_io { formatter.to_s }
+      ensure
+        Bundler::Stats::CLI.singleton_class.remove_method(:start)
+        Bundler::Stats::CLI.define_singleton_method(:start, original)
+      end
+
+      assert_includes output, '== Core Application =='
+      assert_includes output, '== Pack: packs/pack1 =='
+      assert_includes output, '== Pack: packs/pack2 =='
+      assert_includes output, '== Pack: components/billing =='
+      assert_includes output, '== Total (all packs) =='
+    end
   end
 end

data/test/lib/rails_stats/pack_finder_test.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+require "test_helper"
+
+describe RailsStats::PackFinder do
+  describe ".find" do
+    it "detects packwerk packs and component-based components" do
+      root = File.absolute_path("./test/fixtures/packwerk_app")
+
+      packs = RailsStats::PackFinder.find(root).map { |path| path.sub("#{root}/", "") }
+
+      assert_equal ["components/billing", "packs/pack1", "packs/pack2"], packs.sort
+    end
+
+    it "does not treat the root package.yml as a pack" do
+      root = File.absolute_path("./test/fixtures/packwerk_app")
+
+      packs = RailsStats::PackFinder.find(root)
+
+      refute_includes packs, root
+    end
+
+    it "returns an empty list for a plain Rails app" do
+      root = File.absolute_path("./test/dummy")
+
+      assert_empty RailsStats::PackFinder.find(root)
+    end
+  end
+end

data/test/lib/rails_stats/stats_calculator_test.rb

@@ -0,0 +1,64 @@
+# frozen_string_literal: true
+
+require "test_helper"
+
+describe RailsStats::StatsCalculator do
+  describe "#statistics_by_group" do
+    before do
+      root = File.absolute_path("./test/fixtures/packwerk_app")
+      @calculator = RailsStats::StatsCalculator.new(root)
+    end
+
+    it "reports that the app is split into packs" do
+      assert @calculator.packs?
+    end
+
+    it "groups statistics per pack/component with Core first" do
+      groups = @calculator.statistics_by_group.keys
+
+      assert_equal "Core", groups.first
+      assert_equal ["Core", "components/billing", "packs/pack1", "packs/pack2"], groups
+    end
+
+    it "attributes each pack's files to its own group" do
+      by_group = @calculator.statistics_by_group
+
+      assert_equal 1, by_group["packs/pack1"]["Models"].files_total
+      assert_equal 1, by_group["packs/pack1"]["Controllers"].files_total
+      assert_equal 1, by_group["packs/pack1"]["Model Tests"].files_total
+      assert_equal 1, by_group["packs/pack2"]["Models"].files_total
+      assert_equal 1, by_group["components/billing"]["Models"].files_total
+    end
+
+    it "keeps core files in the Core group" do
+      core = @calculator.statistics_by_group["Core"]
+
+      assert_equal 1, core["Models"].files_total
+      assert_equal 1, core["Controllers"].files_total
+      assert_equal 1, core["Libraries"].files_total
+    end
+
+    it "includes pack lib directories that are not gems" do
+      assert_equal 1, @calculator.statistics_by_group["packs/pack1"]["Libraries"].files_total
+    end
+
+    it "has a grand total that matches the sum of every group" do
+      _code, _tests, grand = @calculator.totals_for(@calculator.statistics)
+
+      per_group_files = @calculator.statistics_by_group.sum do |_group, stats|
+        _c, _t, group_grand = @calculator.totals_for(stats)
+        group_grand.files_total
+      end
+
+      assert_equal grand.files_total, per_group_files
+    end
+  end
+
+  describe "#packs?" do
+    it "is false for a plain Rails app" do
+      calculator = RailsStats::StatsCalculator.new(File.absolute_path("./test/dummy"))
+
+      refute calculator.packs?
+    end
+  end
+end

data/test/lib/rails_stats/util_test.rb

@@ -0,0 +1,49 @@
+# frozen_string_literal: true
+
+require "test_helper"
+require "tmpdir"
+require "fileutils"
+
+describe RailsStats::Util do
+  describe ".calculate_directory_statistics" do
+    it "ignores vendored directories and never counts a directory that looks like a source file" do
+      Dir.mktmpdir do |dir|
+        # A real source file that should be counted.
+        File.write(File.join(dir, "real.rb"), "class Real\nend\n")
+
+        # A directory whose name matches the source-file pattern. It must be
+        # walked (its contents counted) but never opened as a file itself,
+        # which previously raised Errno::EISDIR.
+        weird = File.join(dir, "weird.rb")
+        FileUtils.mkdir_p(weird)
+        File.write(File.join(weird, "inner.rb"), "class Inner\nend\n")
+
+        # A vendored tree that must be skipped entirely, including a directory
+        # named like a JS file (e.g. node_modules/popper.js), the exact case
+        # that crashed on a packs app.
+        popper = File.join(dir, "node_modules", "popper.js")
+        FileUtils.mkdir_p(popper)
+        File.write(File.join(popper, "index.js"), "function noop() {}\n")
+
+        stats = RailsStats::Util.calculate_directory_statistics(dir)
+
+        # real.rb + weird.rb/inner.rb, nothing under node_modules.
+        assert_equal 2, stats.files_total
+      end
+    end
+
+    it "skips every ignored directory name" do
+      RailsStats::IGNORED_DIRS.each do |ignored|
+        Dir.mktmpdir do |dir|
+          vendored = File.join(dir, ignored)
+          FileUtils.mkdir_p(vendored)
+          File.write(File.join(vendored, "ignored.rb"), "class Ignored\nend\n")
+
+          stats = RailsStats::Util.calculate_directory_statistics(dir)
+
+          assert_equal 0, stats.files_total, "expected #{ignored}/ to be skipped"
+        end
+      end
+    end
+  end
+end

metadata

@@ -1,13 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: rails_stats
 version: !ruby/object:Gem::Version
-  version: 2.1.0
+  version: 2.2.0
 platform: ruby
 authors:
 - Brian Leonard
+- Ernesto Tagwerker
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-04-07 00:00:00.000000000 Z
+date: 2026-07-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -40,7 +42,9 @@ dependencies:
 description: Point it to a directory and see stuff about the app
 email:
 - brian@bleonard.com
-executables: []
+- ernesto@ombulabs.com
+executables:
+- rails-stats
 extensions: []
 extra_rdoc_files: []
 files:
@@ -52,6 +56,7 @@ files:
 - LICENSE.txt
 - README.md
 - Rakefile
+- bin/rails-stats
 - codecov.yml
 - lib/rails_stats.rb
 - lib/rails_stats/all.rb
@@ -63,6 +68,7 @@ files:
 - lib/rails_stats/gem_statistics.rb
 - lib/rails_stats/inflector.rb
 - lib/rails_stats/json_formatter.rb
+- lib/rails_stats/pack_finder.rb
 - lib/rails_stats/root_statistics.rb
 - lib/rails_stats/spec_statistics.rb
 - lib/rails_stats/stats_calculator.rb
@@ -137,14 +143,31 @@ files:
 - test/dummy/test/models/user_test.rb
 - test/dummy/test/support/test_helper.rb
 - test/fixtures/console-output.txt
+- test/fixtures/packwerk_app/app/controllers/core_controller.rb
+- test/fixtures/packwerk_app/app/models/core_model.rb
+- test/fixtures/packwerk_app/components/billing/app/models/billing_model.rb
+- test/fixtures/packwerk_app/components/billing/lib/billing.rb
+- test/fixtures/packwerk_app/lib/core_lib.rb
+- test/fixtures/packwerk_app/package.yml
+- test/fixtures/packwerk_app/packs/pack1/app/controllers/pack1_controller.rb
+- test/fixtures/packwerk_app/packs/pack1/app/models/pack1_model.rb
+- test/fixtures/packwerk_app/packs/pack1/lib/pack1_lib.rb
+- test/fixtures/packwerk_app/packs/pack1/package.yml
+- test/fixtures/packwerk_app/packs/pack1/spec/models/pack1_model_spec.rb
+- test/fixtures/packwerk_app/packs/pack2/app/models/pack2_model.rb
+- test/fixtures/packwerk_app/packs/pack2/test/models/pack2_model_test.rb
 - test/lib/rails_stats/code_statistics_test.rb
 - test/lib/rails_stats/console_formatter_test.rb
 - test/lib/rails_stats/json_formatter_test.rb
+- test/lib/rails_stats/pack_finder_test.rb
+- test/lib/rails_stats/stats_calculator_test.rb
+- test/lib/rails_stats/util_test.rb
 - test/test_helper.rb
 homepage: https://github.com/fastruby/rails_stats
 licenses:
 - MIT
 metadata: {}
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -159,7 +182,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.2
+rubygems_version: 3.0.3.1
+signing_key:
 specification_version: 4
 summary: Analyze a Rails project
 test_files:
@@ -227,7 +251,23 @@ test_files:
 - test/dummy/test/models/user_test.rb
 - test/dummy/test/support/test_helper.rb
 - test/fixtures/console-output.txt
+- test/fixtures/packwerk_app/app/controllers/core_controller.rb
+- test/fixtures/packwerk_app/app/models/core_model.rb
+- test/fixtures/packwerk_app/components/billing/app/models/billing_model.rb
+- test/fixtures/packwerk_app/components/billing/lib/billing.rb
+- test/fixtures/packwerk_app/lib/core_lib.rb
+- test/fixtures/packwerk_app/package.yml
+- test/fixtures/packwerk_app/packs/pack1/app/controllers/pack1_controller.rb
+- test/fixtures/packwerk_app/packs/pack1/app/models/pack1_model.rb
+- test/fixtures/packwerk_app/packs/pack1/lib/pack1_lib.rb
+- test/fixtures/packwerk_app/packs/pack1/package.yml
+- test/fixtures/packwerk_app/packs/pack1/spec/models/pack1_model_spec.rb
+- test/fixtures/packwerk_app/packs/pack2/app/models/pack2_model.rb
+- test/fixtures/packwerk_app/packs/pack2/test/models/pack2_model_test.rb
 - test/lib/rails_stats/code_statistics_test.rb
 - test/lib/rails_stats/console_formatter_test.rb
 - test/lib/rails_stats/json_formatter_test.rb
+- test/lib/rails_stats/pack_finder_test.rb
+- test/lib/rails_stats/stats_calculator_test.rb
+- test/lib/rails_stats/util_test.rb
 - test/test_helper.rb