pocky 1.3.0 → 1.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: fce51337b66240934ecb460959861226aafedb75
-  data.tar.gz: 75f97f5c6c17fade8d7e70b1119256e70bddd833
+SHA256:
+  metadata.gz: 612ee34127a910813870f48cc2e84a09da9fcb548e7c81a447205c9a3b84ef8b
+  data.tar.gz: 98b7be34c21ac045bafafa8ceb4c01fbcb0461495d19807ec1ab26970a11e4b8
 SHA512:
-  metadata.gz: c71ef6721a70df6cbe50e76faf256faf8021255e08406fcca9f9d1d0a22acee48c1cdf44b074db6703b183746ac5513a27620975936302f4199218511ba79b21
-  data.tar.gz: 1a3ee14c5c8f52150ec2e46bff9208d5f82c45d3163098117904ecf4994e52ecb8f36ae1dd6e09347195266a821a603d5f6624ff64a68088eb5ce26151ba32f6
+  metadata.gz: '07728e53d8c4e6950af51520abe37c0818fabbd25d84cf57cf8cffb7ee925ef10c87d68e7e2bc113cb5c2cc7bf1d7357ee0c3fa608f8aac83df503ea1c68d02c'
+  data.tar.gz: 68bb11723dff8b526aaccc0c9153a6a2879b1b6dc2efdb223d4b5246ee3c9e14d210e3ec5a49b8c168237ca823e4a687ec54bfbf1f697307caef4bae67ed20a3

data/README.md

@@ -6,18 +6,41 @@ Pocky generates dependency graphs for your packwerk packages. The gem is named a
 
 ## Usage
 
+Invoke from irb or code, only `root_path` is required.
 ```ruby
-Pocky::Graph.generate(
+Pocky::Packwerk.generate(
   root_path: 'path/to/app/packages',
-  default_package: 'The Monolith', # The default package listed as "." in deprecated_references.yml
-  package_prefix: 'app/packages', # this is for matching package names listed in deprecated_references.yml
-  output_filename: 'pocky-graph.png'
+  default_package: 'Default',    # The default package listed as "." in deprecated_references.yml
+  filename: 'packwerk-viz.png', # Name of output file
+  dpi: 150                     # Output file resolution
 )
 ```
 
 ![pocky-graph](https://user-images.githubusercontent.com/138784/103251690-a6299b80-492e-11eb-92f1-205752d850d8.png)
 
-Note that the the wider edges indicate more references.
+Note that the the bold edges indicate heavier dependencies.
+
+Invoke as a rake task:
+
+    $ rake pocky:generate"[path/to/app/packages,Monolith,packages.png,100]"
+
+
+#### `root_path` as an array
+`root_path` can also be an array in case your packages are organized in multiple directories. Alternatively, you can also provide paths to individual packages to generate more focused graphs for your package subsystems.
+
+```ruby
+Pocky::Packwerk.generate(
+  root_path: [
+    'path/to/app/packages/a',
+    'path/to/app/packages/z',
+  ]
+)
+```
+
+Generate the same graph using the rake task:
+
+    $ rake pocky:generate"[path/to/app/packages/a path/to/app/packages/z]"
+
 
 ## Installation
 
@@ -43,7 +66,7 @@ To install this gem onto your local machine, run `bundle exec rake install`. To
 
 ## Contributing
 
-Bug reports and pull requests are welcome on GitHub at https://github.com/[USERNAME]/pocky.
+Bug reports and pull requests are welcome on GitHub at https://github.com/mquan/pocky.
 
 ## License
 

data/Rakefile

@@ -3,6 +3,6 @@
 require "bundler/gem_tasks"
 require "rspec/core/rake_task"
 
-RSpec::Core::RakeTask.new(:spec)
-
-task default: :spec
+spec = Gem::Specification.find_by_name 'pocky'
+rakefile = "#{spec.gem_dir}/lib/pocky/Rakefile"
+load rakefile

data/lib/pocky.rb

@@ -1,7 +1,8 @@
 # frozen_string_literal: true
 
 require_relative 'pocky/version'
-require_relative 'pocky/graph'
+require_relative 'pocky/packwerk'
 
 module Pocky
+  require 'pocky/railtie' if defined?(Rails)
 end

data/lib/pocky/Rakefile

@@ -0,0 +1,4 @@
+require 'pocky'
+
+path = File.expand_path(__dir__)
+Dir.glob("#{path}/tasks/**/*.rake").each { |f| import f }

data/lib/pocky/packwerk.rb

@@ -0,0 +1,103 @@
+# frozen_string_literal: true
+
+require 'yaml'
+require 'ruby-graphviz'
+
+module Pocky
+  class InvalidRootPathError < StandardError
+  end
+
+  class Packwerk
+    REFERENCE_FILE_NAME = 'deprecated_references.yml'
+    MAX_EDGE_WIDTH = 5
+
+    def self.generate(params)
+      new(**params).generate
+    end
+
+    private_class_method :new
+    def initialize(
+      root_path:,
+      default_package: 'Default',
+      filename: 'packwerk-viz.png',
+      dpi: 150
+    )
+      @root_paths = [*root_path]
+      raise InvalidRootPathError, 'root_path is required' if @root_paths.empty?
+
+      @default_package = default_package
+      @filename = filename
+      @dpi = dpi.to_i
+      @deprecated_references = {}
+      @nodes = {}
+    end
+
+    def generate
+      load_package_references
+      build_directed_graph
+    end
+
+    private
+
+    def build_directed_graph
+      graph = GraphViz.new(:G, type: :digraph, dpi: @dpi)
+      @deprecated_references.each do |package, references|
+        @nodes[package] ||= graph.add_nodes(package)
+        references.each do |provider, dependencies|
+          provider_package = package_name_for_dependency(provider)
+          @nodes[provider_package] ||= graph.add_nodes(provider_package)
+
+          graph.add_edges(
+            @nodes[provider_package],
+            @nodes[package],
+            penwidth: edge_width(dependencies.length)
+          )
+        end
+      end
+
+      graph.output(png: @filename)
+    end
+
+    def edge_width(count)
+      [
+        [(count / 5).to_i, 1].max,
+        MAX_EDGE_WIDTH
+      ].min
+    end
+
+    def package_references
+      @package_references ||= @root_paths.flat_map do |path|
+        Dir["#{path}/**/#{REFERENCE_FILE_NAME}"]
+      end
+    end
+
+    def load_package_references
+      if package_references.empty?
+        raise InvalidRootPathError, "Cannot find any #{REFERENCE_FILE_NAME} in provided root_path"
+      end
+
+      package_references.each do |filename|
+        package = parse_package_name(filename)
+        @deprecated_references[package] ||= YAML.load_file(filename) || {}
+      end
+    end
+
+    def parse_package_name(filename)
+      File.basename(File.dirname(filename))
+    end
+
+    def package_name_for_dependency(name)
+      return @default_package if name == '.'
+
+      reference_filename = package_references.find do |ref|
+        ref.match(/#{name}\/#{REFERENCE_FILE_NAME}$/)
+      end
+
+      if reference_filename
+        parse_package_name(reference_filename)
+      else
+        name
+      end
+    end
+  end
+end

data/lib/pocky/railtie.rb

@@ -0,0 +1,13 @@
+require 'pocky'
+require 'rails'
+
+module Pocky
+  class Railtie < Rails::Railtie
+    railtie_name :pocky
+
+    rake_tasks do
+      path = File.expand_path(__dir__)
+      Dir.glob("#{path}/tasks/**/*.rake").each { |f| load f }
+    end
+  end
+end

data/lib/pocky/tasks/generate.rake

@@ -0,0 +1,10 @@
+require 'pocky'
+
+namespace :pocky do
+  desc 'Generate dependency graph for packwerk packages'
+  task :generate, [:root_path, :default_package, :filename, :dpi] do |_task, args|
+    Pocky::Packwerk.generate(args.merge(
+      root_path: arg[:root_path].split(/\s+/)
+    ))
+  end
+end

data/lib/pocky/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Pocky
-  VERSION = "1.3.0"
+  VERSION = "1.8.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pocky
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.8.0
 platform: ruby
 authors:
 - Quan Nguyen
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-12-29 00:00:00.000000000 Z
+date: 2020-12-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ruby-graphviz
@@ -41,7 +41,10 @@ files:
 - bin/console
 - bin/setup
 - lib/pocky.rb
-- lib/pocky/graph.rb
+- lib/pocky/Rakefile
+- lib/pocky/packwerk.rb
+- lib/pocky/railtie.rb
+- lib/pocky/tasks/generate.rake
 - lib/pocky/version.rb
 - pocky.gemspec
 homepage: https://github.com/mquan/pocky
@@ -66,8 +69,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.4.5
+rubygems_version: 3.1.4
 signing_key: 
 specification_version: 4
 summary: A ruby gem that generates dependency graph for your packwerk packages

data/lib/pocky/graph.rb

@@ -1,92 +0,0 @@
-# frozen_string_literal: true
-
-require 'yaml'
-require 'ruby-graphviz'
-
-module Pocky
-  class Graph
-    MAX_EDGE_WIDTH = 5
-
-    def self.generate(params)
-      new(**params).generate
-    end
-
-    private_class_method :new
-    def initialize(
-      root_path:,
-      default_package: 'Default',
-      package_prefix: '',
-      output_filename: 'pocky-graph.png'
-    )
-      @root_path = root_path
-      @default_package = default_package
-      @package_prefix = package_prefix
-      @output_filename = output_filename
-
-      @deprecated_references = {}
-      @nodes = {}
-    end
-
-    def generate
-      load_package_dependencies
-      build_directed_graph
-    end
-
-    private
-
-    def build_directed_graph
-      graph = GraphViz.new(:G, type: :digraph, dpi: 300)
-      @deprecated_references.each do |package, references|
-        @nodes[package] ||= graph.add_nodes(package)
-        references.each do |provider, dependencies|
-          provider_package = package_name_for_dependency(provider)
-          @nodes[provider_package] ||= graph.add_nodes(provider_package)
-
-          graph.add_edges(
-            @nodes[provider_package],
-            @nodes[package],
-            penwidth: edge_width(dependencies.length)
-          )
-        end
-      end
-
-      graph.output(png: @output_filename)
-    end
-
-    def edge_width(count)
-      [
-        [(count / 5).to_i, 1].max,
-        MAX_EDGE_WIDTH
-      ].min
-    end
-
-    def load_package_dependencies
-      Dir.each_child(@root_path) do |elem|
-        if Dir.exist?(File.join(@root_path, elem))
-          load_deprecated_references_for_package(elem)
-        end
-      end
-    end
-
-    def load_deprecated_references_for_package(package)
-      @deprecated_references[package] ||= begin
-        filename = deprecated_references_file_for(package)
-        if File.exist?(filename)
-          YAML.load_file(filename) || {}
-        else
-          {}
-        end
-      end
-    end
-
-    def deprecated_references_file_for(package)
-      File.join(@root_path, package, 'deprecated_references.yml')
-    end
-
-    def package_name_for_dependency(name)
-      return @default_package if name == '.'
-
-      name.gsub(@package_prefix, '').gsub(/^\//, '')
-    end
-  end
-end