bundler-stats 2.2.0 → 2.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fad76aec2c17866fce619ae8dbdaa4e580a58f1c0d89c53681a35dba5e266998
-  data.tar.gz: 27d5b7a9ac01d7f7c0856cd2422b127d18fcd4de1cda1b2bcb0bf5c596528ad6
+  metadata.gz: 38f866872f6ba724ee82db46ba1715c05bcf8f22272f0ae6a23a96b1760232ce
+  data.tar.gz: ee9b4655d4c8faa1da7f23d3602820ab70eeaf10aad836f4bc7c544cd10bf795
 SHA512:
-  metadata.gz: 57c12161cd51c19804cfd15fb17903c1a1a9a82fe1347cd303230f5979539c7306165b926d5db4e65434d43c19c9a8649afbe0c30ee58a41cfebc2cdc653b250
-  data.tar.gz: ee3ffeaa9580d4866a8fb11a947274ecd70e926cfdd3db83147a04500ab2a76e15ce89384268911e7e45587d483501d81d44d13e5733fe4c79f5746820887563
+  metadata.gz: e31cce716bfae8bdc92e525d1047678b96c4ea18f80aee2875718f84c690a8a76dbd34ee923b445724716f45eb0e9cdbeab69e13b0d6479284cb4a1185beeecf
+  data.tar.gz: 2644320effea69f6e117a92154e3ddf0e0a3327a23d8141262dc4138fefa8a8a84eab049e68686ded2f778d2b271cbf8a00c43a6eb2b9fa06c09d01b11ce694d

data/CHANGELOG.md

@@ -1,6 +1,13 @@
 Changelog
 =============
 
+## [2.3.0] - 2022-02-12
+
+### Added
+- Skiplist now correctly removes dependencies from 1st and onward levels
+- Wildcard ability for specifying gems for skiplist
+- Document `versions` in README
+
 ## [2.2.0] - 2022-02-09
 
 ### Added

data/README.md

@@ -34,8 +34,10 @@ Usage
         bundle-stats version          # Prints the bundler-stats version
         bundle-stats versions TARGET  # Shows versions requirements for target in other dependencies
 
+### Auditing Your Gemfile
+
 The most obvious thing to do is run the command by itself, which should help
-identify problem areas:
+identify problem areas.
 
     > bundle-stats
 
@@ -62,7 +64,7 @@ identify problem areas:
       Unpinned Versions   54
             Github Refs   0
 
-It looks like rails_admin is a huge problem. Use `show` to investigate:
+It looks like rails_admin is a huge problem. Use `show` to investigate.
 
     > bundle-stats show rails_admin
     bundle-stats for rails_admin
@@ -98,22 +100,117 @@ It looks like rails_admin is a huge problem. Use `show` to investigate:
     |                                | bundler                                |
     +--------------------------------|----------------------------------------+
 
-Removing the dep will only actually remove 9 gems. The rest are shared
-dependencies with other gems like rails. We can also omit trees we aren't going
-to remove (hi rails) by not following them:
+### Skipping Common Dependencies
+
+Let's take a look at another common gem to rails codebases. In this case,
+we have some unique dependencies, but also many dependencies on rails, and on
+its constituent gems.
+
+    > bundle-stats show compass-rails
+    bundle-stats for compass-rails
+
+    +--------------------------------|----------------------------------------+
+    |           Depended Upon By (0) |                                        |
+    |                Depends On (35) | compass, sass-rails, sprockets         |
+    |                                | chunky_png, compass-core               |
+    |                                | compass-import-once, rb-fsevent        |
+    |                                | rb-inotify, sass, multi_json, ffi      |
+    |                                | railties, sprockets-rails, tilt        |
+    |                                | actionpack, activesupport, method_source |
+    |                                | rake, thor, actionview, rack, rack-test |
+    |                                | rails-dom-testing, rails-html-sanitizer |
+    |                                | builder, erubi, concurrent-ruby, i18n  |
+    |                                | minitest, tzinfo, thread_safe, nokogiri |
+    |                                | mini_portile2, loofah, crass           |
+    |             Unique to This (3) | compass, compass-core                  |
+    |                                | compass-import-once                    |
+    +--------------------------------|----------------------------------------+
+
+We're not looking to remove rails, so there's not much point in including it
+within this output. Instead, we can use the `nofollow` flag to skip it in all
+output lists.
 
-    > bundle-stats show sass-rails --nofollow="railties,activesupport,actionpack"
-    bundle-stats for sass-rails
+    > bundle-stats show compass-rails --nofollow="railties,activeupport,actionview,actionpack"
+    bundle-stats for compass-rails
 
     +--------------------------------|----------------------------------------+
-    |           Depended Upon By (2) | compass-rails, rails_admin             |
-    |                 Depends On (9) | railties, sass, sprockets              |
+    |           Depended Upon By (0) |                                        |
+    |                Depends On (20) | compass, sass-rails, sprockets         |
+    |                                | chunky_png, compass-core               |
+    |                                | compass-import-once, rb-fsevent        |
+    |                                | rb-inotify, sass, multi_json, ffi      |
     |                                | sprockets-rails, tilt, concurrent-ruby |
-    |                                | rack, actionpack, activesupport        |
-    |             Unique to This (0) |                                        |
+    |                                | rack, activesupport, i18n, minitest    |
+    |                                | tzinfo, thread_safe                    |
+    |             Unique to This (3) | compass, compass-core                  |
+    |                                | compass-import-once                    |
     +--------------------------------|----------------------------------------+
 
-To consume information with a build job or somesuch, all commands can emit JSON:
+This is better, but for other codebases it's common for gems to depend on each
+of the _many many_ child gems of rails individually. Rather than specifying each
+by itself, we can use wildcards to remove them in bulk.
+
+    > bundle-stats show compass-rails --nofollow="rail*,action*,active*"
+    bundle-stats for compass-rails
+
+    +--------------------------------|----------------------------------------+
+    |           Depended Upon By (0) |                                        |
+    |                Depends On (15) | compass, sass-rails, sprockets         |
+    |                                | chunky_png, compass-core               |
+    |                                | compass-import-once, rb-fsevent        |
+    |                                | rb-inotify, sass, multi_json, ffi      |
+    |                                | sprockets-rails, tilt, concurrent-ruby |
+    |                                | rack                                   |
+    |             Unique to This (3) | compass, compass-core                  |
+    |                                | compass-import-once                    |
+    +--------------------------------|----------------------------------------+
+
+### Showing Required Versions
+
+Sometimes you try to upgrade a gem, you ask bundler _very nicely_, but you
+find that the version hasn't changed. When that happens, sometimes it's not
+clear why the gem version isn't changing.
+
+In that case, use `versions` to list all required dependencies on your gem.
+
+    > bundle-stats versions actionpack
+    version dependencies for actionpack
+
+    Depended Upon By   24
+    Resolved Version   5.2.0
+
+    +----------------------------|------------------+
+    |                       Name | Required Version |
+    +----------------------------|------------------+
+    |                  webpacker | = 5.2.0          |
+    |               actionmailer | = 5.2.0          |
+    |              activestorage | = 5.2.0          |
+    |               coffee-rails | = 5.2.0          |
+    |              compass-rails | = 5.2.0          |
+    |                     devise | = 5.2.0          |
+    |          factory_bot_rails | = 5.2.0          |
+    |         font-awesome-rails | = 5.2.0          |
+    |                rails_admin | = 5.2.0          |
+    |                   railties | = 5.2.0          |
+    |                 sass-rails | = 5.2.0          |
+    |                     scenic | = 5.2.0          |
+    |                 versionist | = 5.2.0          |
+    |                actioncable | = 5.2.0          |
+    |               jquery-rails | = 5.2.0          |
+    |            jquery-ui-rails | = 5.2.0          |
+    | jquery_file_download-rails | = 5.2.0          |
+    |                      rails | = 5.2.0          |
+    |                rspec-rails | >= 3.0           |
+    |            sprockets-rails | >= 4.0           |
+    |                 haml-rails | >= 4.0.1         |
+    |                 responders | >= 4.2.0, < 5.3  |
+    |                simple_form | >= 5.0           |
+    |   rails-controller-testing | ~> 5.x, >= 5.0.1 |
+    +----------------------------|------------------+
+
+### Output Formats
+
+To consume information with a build job or somesuch, all commands can emit JSON.
 
     > bundle-stats show sass-rails --nofollow="railties,activesupport,actionpack" -f json
     {

data/lib/bundler/stats/calculator.rb

@@ -5,7 +5,7 @@ module Bundler
     class Calculator
       attr_reader :parser, :tree, :gemfile, :remover
 
-      def initialize(gemfile_path, lockfile_path, options = {})
+      def initialize(gemfile_path, lockfile_path, skiplist = nil)
         raise ArgumentError unless File.readable?(lockfile_path)
         raise ArgumentError unless File.readable?(gemfile_path)
 
@@ -17,9 +17,7 @@ module Bundler
         lock_contents = File.read(lockfile_path)
         @parser = Bundler::LockfileParser.new(lock_contents)
 
-        skiplist = options.fetch(:skiplist, [])
         @tree = Bundler::Stats::Tree.new(@parser, skiplist: skiplist)
-
         @remover = Bundler::Stats::Remover.new(@tree, @gemfile)
       end
 

data/lib/bundler/stats/cli.rb

@@ -99,7 +99,7 @@ module Bundler
 
       def draw_versions(stats, target)
         dependers = stats[:top_level_dependencies] # they do the depending
-        say "bundle-stats for #{target}"
+        say "version dependencies for #{target}"
         say Printer.new(
           headers: nil,
           borders: false,
@@ -121,13 +121,8 @@ module Bundler
       end
 
       def build_calculator(options)
-        if !options[:nofollow].nil?
-          skiplist = options[:nofollow].gsub(/\s+/, '').split(",")
-        else
-          skiplist = []
-        end
-
-        @calculator ||= Bundler::Stats::Calculator.new(gemfile_path, lockfile_path, skiplist: skiplist)
+        skiplist = Bundler::Stats::Skiplist.new(options[:nofollow])
+        @calculator ||= Bundler::Stats::Calculator.new(gemfile_path, lockfile_path, skiplist)
       end
 
       def gemfile_path

data/lib/bundler/stats/skiplist.rb

@@ -0,0 +1,32 @@
+class Bundler::Stats::Skiplist
+  attr_reader :regex_list
+
+  def initialize(list = '')
+    @regex_list = if list.respond_to?(:regex_list)
+      list.regex_list
+    elsif list.respond_to?(:split)
+      list
+        .gsub(/\s+/, '')
+        .split(",")
+        .map { |str| glob_to_regex(str) }
+    else
+      (list || []).map { |str| glob_to_regex(str) }
+    end
+  end
+
+  def filter(gem_list)
+    gem_list.reject { |gem| self.include?(gem) }
+  end
+
+  def include?(target)
+    @regex_list.any? { |pattern| pattern =~ target.name }
+  end
+
+  private
+
+  def glob_to_regex(str)
+    str.gsub!('*','.*') # weird globbish behavior
+
+    /^#{str}$/
+  end
+end

data/lib/bundler/stats/tree.rb

@@ -1,20 +1,21 @@
 class Bundler::Stats::Tree
-  attr_accessor :tree
+  attr_accessor :tree, :skiplist
 
   ERR_MESSAGE = "The dependency `%s` wasn't found. It may not be present in " \
                 "your Gemfile.lock. This often happens when a dependency isn't " \
                 "installed on your platform."
 
-  def initialize(parser, skiplist: [])
+  def initialize(parser, skiplist: '')
     raise ArgumentError unless parser.respond_to?(:specs)
 
     @parser   = parser
     @tree     = specs_as_tree(@parser.specs)
-    @skiplist = skiplist
+    @skiplist = Bundler::Stats::Skiplist.new(skiplist)
   end
 
   def summarize(target)
     transitive_dependencies = transitive_dependencies(target)
+
     { name: target,
       total_dependencies: transitive_dependencies.count,
       first_level_dependencies: first_level_dependencies(target).count,
@@ -51,12 +52,12 @@ class Bundler::Stats::Tree
       return []
     end
 
-    top_level = @tree[target].dependencies
+    top_level = skiplist.filter(@tree[target].dependencies)
     all_level = top_level + top_level.inject([]) do |arr, dep|
       # turns out bundler refuses to include itself in the dependency tree,
       # which is sneaky
       next arr if dep.name == "bundler"
-      next arr if @skiplist.include? dep.name
+      next arr if skiplist.include? dep
 
       arr += transitive_dependencies(dep.name)
     end

data/lib/bundler/stats/version.rb

@@ -1,5 +1,5 @@
 module Bundler
   module Stats
-    VERSION = '2.2.0'
+    VERSION = '2.3.0'
   end
 end

data/lib/bundler/stats.rb

@@ -1,5 +1,6 @@
 require_relative 'stats/calculator'
 require_relative 'stats/printer'
 require_relative 'stats/remover'
+require_relative 'stats/skiplist'
 require_relative 'stats/tree'
 require_relative 'stats/version'

data/spec/lib/bundler/stats/calculator_spec.rb

@@ -38,9 +38,9 @@ describe Bundler::Stats::Calculator do
       # this sucks. break this dependency further
       allow(Bundler::LockfileParser).to receive(:new) { "parser" }
 
-      target = subject.new(gemfile_path, lockfile_path, skiplist: "abc")
+      target = subject.new(gemfile_path, lockfile_path, "skiplist")
 
-      expect(Bundler::Stats::Tree).to have_received(:new).with("parser", skiplist: "abc")
+      expect(Bundler::Stats::Tree).to have_received(:new).with("parser", skiplist: "skiplist")
     end
   end
 

data/spec/lib/bundler/stats/tree_spec.rb

@@ -140,20 +140,24 @@ describe Bundler::Stats::Tree do
   end
 
   context "skip lists" do
-    it "still includes the skipped entry" do
+    it "skips processing the requested entry, plus children of skipped entries" do
       tree = subject.new(parser, skiplist: ["depth-three"])
 
       target = tree.transitive_dependencies("depth-one")
 
-      expect(target.map(&:name)).to include("depth-three")
+      expect(target.map(&:name)).to include("depth-two")
+      expect(target.map(&:name)).not_to include("depth-three")
+      expect(target.map(&:name)).not_to include("depth-four")
     end
 
-    it "stops processing the children of skipped entries" do
-      tree = subject.new(parser, skiplist: ["depth-three"])
+    it "allows skiplist wildcards" do
+      tree = subject.new(parser, skiplist: ["depth-t*"])
 
       target = tree.transitive_dependencies("depth-one")
 
-      expect(target.map(&:name)).not_to include("depth-four")
+      expect(target.map(&:name)).not_to include("depth-two")
+      expect(target.map(&:name)).not_to include("depth-three")
+      expect(target.map(&:name)).not_to include("depth-four") # because transitive
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bundler-stats
 version: !ruby/object:Gem::Version
-  version: 2.2.0
+  version: 2.3.0
 platform: ruby
 authors:
 - Joseph Mastey
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-02-09 00:00:00.000000000 Z
+date: 2022-02-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -150,6 +150,7 @@ files:
 - lib/bundler/stats/cli.rb
 - lib/bundler/stats/printer.rb
 - lib/bundler/stats/remover.rb
+- lib/bundler/stats/skiplist.rb
 - lib/bundler/stats/tree.rb
 - lib/bundler/stats/version.rb
 - spec/lib/bundler/stats/calculator_spec.rb