visualize_packs 0.5.5 → 0.5.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b4ade528fc794c71afb2470d457f31d72b2456b85a484f2b328ca1c73e7fc2cf
-  data.tar.gz: 239feaba7ced4136ac77625e6ed1632a031492e86ca6d1c0d025976b23e31300
+  metadata.gz: 698ac9ebae377a99e2b9b5fe62c3646bcaa8b4ce81439feee7354d81d80682c1
+  data.tar.gz: 15335a5ce4cc6eb90926af902da548c268bc6c86caa8ae6fd9df5127cfb7e071
 SHA512:
-  metadata.gz: 18a67a3b983c99f4d273a8e32e390a63bb0fa0398c7da51d4552fee73eedda0e77debfe45a3e196cae5b8d0e370124f7eeb82d202f4daba7b68bb90fb5b15708
-  data.tar.gz: 60e06eaacba3c796ee9045223d47203f63b19e83664c58b9db69696c591f68169f217bc8172287fcb9cc61befaa2008c4a337e486febd0477b6a0b8d14100520
+  metadata.gz: 79f8106bfafe0a7cd9c17ff6f886871b4a24990e562a389b5dab45ff5838a3a0a22fc225d44ea7afcdeb8d8a20ff98e2037b74ed95cd170964027b8b53d40176
+  data.tar.gz: 5e5d466148f46ee4876baa8ae184187115bc5d77f66d36d8396867c4fb61066f47ac6b32e22781fcbd4d98c7b387f60f271a8b0cc1a89f91ba383462ca48fcec

data/README.md

@@ -16,7 +16,7 @@ This will generate a local dependency diagram for every pack in your app
 find . -iname 'package.yml' | sed 's/\/package.yml//g' | sed 's/\.\///' | xargs -I % sh -c "bundle exec visualize_packs --only=% > %/packs.dot && dot %/packs.dot -Tpng -o %/packs.png"
 ```
 
-If your app is large and has many packages and violations, the above graphs will likely be too big. Try this version to get only the edges to and from the focus package for each diagram:
+If your app is large and has many packages and todos, the above graphs will likely be too big. Try this version to get only the edges to and from the focus package for each diagram:
 
 ```
 find . -iname 'package.yml' | sed 's/\/package.yml//g' | sed 's/\.\///' | xargs -I % sh -c "bundle exec visualize_packs --only=% --only-edges-to-focus > %/packs.dot && dot %/packs.dot -Tpng -o %/packs.png"

data/bin/visualize_packs

@@ -35,9 +35,9 @@ OptionParser.new do |opt|
   opt.on('--focus-on=PACKAGE', "Focus on a specific package") { |o| options.focus_package = o }
   opt.on('--only-edges-to-focus', "If focus is set, this shows only the edges to/from the focus node instead of all edges in the focussed graph. This only has effect when --focus-on is set.") { |o| options.show_only_edges_to_focus_package = true }
 
-  opt.on('--roll_nested_todos_into_top_level', "Don't show nested packages (not counting root). Connect edges to top-level package instead") { |o| options.roll_nested_todos_into_top_level = true }
+  opt.on('--roll-nested-into-parent-packs', "Don't show nested packages (not counting root). Connect edges to top-level package instead") { |o| options.roll_nested_into_parent_packs = true }
   opt.on('--focus_folder=FOLDER', "Draw package diagram only for packages in FOLDER") { |o| options.focus_folder = o }
-  opt.on('--no_nested_relationships', "Don't draw nested package relationships") { |o| options.show_nested_relationships = false }
+  opt.on('--no_nested_relationships', "Don't draw relationships between parents and nested packs") { |o| options.show_nested_relationships = false }
 
   opt.on('--exclude-packs=pack1,pack2,etc', "Exclude these packs from diagram") { |o| options.exclude_packs = o.to_s.split(",") }
 

data/lib/graph.dot.erb

@@ -81,32 +81,28 @@ digraph package_diagram {
   <%- end -%>
   <%- if options.show_todos -%>
     <%- all_packages.each do |package| -%>
-      <%- filtered_violations = package.violations -%>
+      <%- filtered_todos = package.violations -%>
       <%- if options.only_todo_types.any? -%>
-        <%- filtered_violations = filtered_violations.select { options.only_todo_types.include?(_1.type) } -%>
+        <%- filtered_todos = filtered_todos.select { options.only_todo_types.include?(_1.type) } -%>
       <%- end -%>
-      <%- violations_by_package = filtered_violations.group_by(&:to_package_name) -%>
-      <%- violations_by_package.keys.each do |violations_to_package| -%>
-        <%- todo_types = violations_by_package[violations_to_package].group_by(&:type) -%>
-        <%- todo_types.keys.each do |violation_type| -%>
-          <%- if show_edge.call(package.name, violations_to_package) -%>
-            "<%= package.name -%>" -> "<%= violations_to_package -%>"<%= violation_type == 'privacy' ? ':private' : '' -%> [ color=darkred style=dashed
+      <%- todos_by_package = filtered_todos.group_by(&:to_package_name) -%>
+      <%- todos_by_package.keys.each do |todos_to_package| -%>
+        <%- todo_types = todos_by_package[todos_to_package].group_by(&:type) -%>
+        <%- todo_types.keys.each do |todo_type| -%>
+          <%- if show_edge.call(package.name, todos_to_package) -%>
+            "<%= package.name -%>" -> "<%= todos_to_package -%>"<%= todo_type == 'privacy' ? ':private' : '' -%> [ color=darkred style=dashed
               constraint=false
-              # headlabel="<%= violation_type -%>"
-              <%- if violation_type == 'privacy' -%>
+              # headlabel="<%= todo_type -%>"
+              <%- if todo_type == 'privacy' -%>
                 arrowhead=crow
-              <%- elsif violation_type == 'architecture' -%>
+              <%- elsif todo_type == 'architecture' -%>
                 arrowhead=invodot
-              <%- elsif violation_type == 'visibility' -%>
+              <%- elsif todo_type == 'visibility' -%>
                 arrowhead=obox
-              <%- elsif violation_type == 'dependency' -%>
+              <%- elsif todo_type == 'dependency' -%>
                 arrowhead=odot
               <%- end -%>
-              <%-
-                max_edge_width = 10
-                edge_width = (todo_types[violation_type].count / max_violation_count.to_f * max_edge_width).to_i
-              -%>
-              penwidth=<%= edge_width -%>
+              penwidth=<%= todo_edge_width(todo_types[todo_type].count, max_todo_count) -%>
             ]
           <%- end -%>
         <%- end -%>
@@ -139,7 +135,7 @@ digraph package_diagram {
       K [ fontsize=12 shape=box label="package"]
       L [ fontsize=12 shape=box label="package"]
       A -> B [label="accepted dependency" color=darkgreen]
-      C -> D [label="privac todo" color=darkred style=dashed arrowhead=crow]
+      C -> D [label="privacy todo" color=darkred style=dashed arrowhead=crow]
       E -> F [label="architecture todo" color=darkred style=dashed arrowhead=invodot]
       G -> H [label="visibility todo" color=darkred style=dashed arrowhead=obox]
       I -> J [label="dependency todo" color=darkred style=dashed arrowhead=odot]

data/lib/options.rb

@@ -15,7 +15,7 @@ class Options < T::Struct
   prop :focus_package, T.nilable(String)
   prop :show_only_edges_to_focus_package, T::Boolean, default: false
 
-  prop :roll_nested_todos_into_top_level, T::Boolean, default: false
+  prop :roll_nested_into_parent_packs, T::Boolean, default: false
   prop :focus_folder, T.nilable(String)
   prop :show_nested_relationships, T::Boolean, default: true
 

data/lib/visualize_packs.rb

@@ -14,13 +14,13 @@ module VisualizePacks
     all_packages = filtered(packages, options.focus_package, options.focus_folder, options.exclude_packs).sort_by {|x| x.name }
     all_package_names = all_packages.map &:name
 
-    all_packages = remove_nested_packs(all_packages) if options.roll_nested_todos_into_top_level
+    all_packages = remove_nested_packs(all_packages) if options.roll_nested_into_parent_packs
 
     show_edge = show_edge_builder(options, all_package_names)
     node_color = node_color_builder()
-    max_violation_count = max_violation_count(all_packages, show_edge)
+    max_todo_count = max_todo_count(all_packages, show_edge)
 
-    title = diagram_title(options, max_violation_count)
+    title = diagram_title(options, max_todo_count)
 
     architecture_layers = (raw_config['architecture_layers'] || []) + ["NotInLayer"]
     grouped_packages = architecture_layers.inject({}) do |result, key|
@@ -52,7 +52,7 @@ module VisualizePacks
     package.config.dig("metadata", "owner") || package.config["owner"]
   end
 
-  def self.diagram_title(options, max_violation_count)
+  def self.diagram_title(options, max_todo_count)
     app_name = File.basename(Dir.pwd)
     focus_edge_info = options.focus_package && options.show_only_edges_to_focus_package ? "showing only edges to/from focus pack" : "showing all edges between visible packs"
     focus_info = options.focus_package || options.focus_folder ? "Focus on #{[options.focus_package, options.focus_folder].compact.join(' and ')} (#{focus_edge_info})" : "All packs"
@@ -65,14 +65,14 @@ module VisualizePacks
       options.only_todo_types.empty? ? nil : "only #{limited_sentence(options.only_todo_types)} todos",
       options.show_privacy ? nil : "hiding privacy",
       options.show_teams ? nil : "hiding teams",
-      options.roll_nested_todos_into_top_level ? "hiding nested packs" : nil,
+      options.roll_nested_into_parent_packs ? "hiding nested packs" : nil,
       options.show_nested_relationships ? nil : "hiding nested relationships",
       options.exclude_packs.empty? ? nil : "excluding pack#{options.exclude_packs.size > 1 ? 's' : ''}: #{limited_sentence(options.exclude_packs)}",
     ].compact.join(', ').strip
     main_title = "#{app_name}: #{focus_info}#{skipped_info != '' ? ' - ' + skipped_info : ''}"
     sub_title = ""
-    if options.show_todos && max_violation_count
-      sub_title = "<br/><font point-size='12'>Widest todo edge is #{max_violation_count} violation#{max_violation_count > 1 ? 's' : ''}</font>"
+    if options.show_todos && max_todo_count
+      sub_title = "<br/><font point-size='12'>Widest todo edge is #{max_todo_count} todo#{max_todo_count > 1 ? 's' : ''}</font>"
     end
     "<<b>#{main_title}</b>#{sub_title}>"
   end
@@ -113,24 +113,43 @@ module VisualizePacks
     end
   end
 
-  def self.max_violation_count(all_packages, show_edge)
-    violation_counts = {}
+  def self.max_todo_count(all_packages, show_edge)
+    todo_counts = {}
     all_packages.each do |package|
-      violations_by_package = package.violations.group_by(&:to_package_name)
-      violations_by_package.keys.each do |violations_to_package|
-        todo_types = violations_by_package[violations_to_package].group_by(&:type)
-        todo_types.keys.each do |violation_type|
-          if show_edge.call(package.name, violations_to_package)
-            key = "#{package.name}->#{violations_to_package}:#{violation_type}"
-            violation_counts[key] = todo_types[violation_type].count
-            # violation_counts[key] += 1
+      todos_by_package = package.violations.group_by(&:to_package_name)
+      todos_by_package.keys.each do |todos_to_package|
+        todo_types = todos_by_package[todos_to_package].group_by(&:type)
+        todo_types.keys.each do |todo_type|
+          if show_edge.call(package.name, todos_to_package)
+            key = "#{package.name}->#{todos_to_package}:#{todo_type}"
+            todo_counts[key] = todo_types[todo_type].count
+            # todo_counts[key] += 1
           end
         end
       end
     end
-    violation_counts.values.max
+    todo_counts.values.max
   end
 
+  def self.todo_edge_width(todo_count, max_count)
+    # Limits
+    min_width = 1
+    max_width = 10
+    min_count = 1 # Number of todos equivalent to min_width
+
+    # Ensure safe values
+    return 0 if todo_count < min_count
+    return max_width if todo_count > max_count
+
+    todo_range = max_count - min_count
+    width_range = max_width - min_width
+    count_delta = todo_count - min_count
+
+    width_delta = count_delta / todo_range.to_f * width_range
+    edge_width = min_width + width_delta
+    edge_width.round(2)
+ end
+
   def self.filtered(packages, filter_package, filter_folder, exclude_packs)
     return packages unless filter_package || filter_folder || exclude_packs.any?
 
@@ -159,13 +178,12 @@ module VisualizePacks
   def self.all_nested_packages(all_package_names)
     all_package_names.reject { |p| p == '.' }.inject({}) do |result, package|
       package_map_tally = all_package_names.map { |other_package| Pathname.new(package).parent.to_s.include?(other_package) }
-      if package_map_tally.tally[true].nil?
-        #nothing to do
-      elsif package_map_tally.tally[true] > 1
-        raise "Can't handle multiple levels of nesting atm"
-      elsif package_map_tally.tally[true] == 1
-        result[package] = all_package_names[package_map_tally.find_index(true)]
-      end
+
+      acc = []
+      all_package_names.each_with_index { |pack, idx| acc << pack if package_map_tally[idx] }
+      acc.sort_by(&:length)
+      result[package] = acc.first unless acc.empty?
+
       result
     end
   end
@@ -201,7 +219,7 @@ module VisualizePacks
           nested_packages[d] || d
         end.uniq.reject { |p| p == package.name }
 
-        morphed_violations = package.violations.map do |v|
+        morphed_todos = package.violations.map do |v|
           ParsePackwerk::Violation.new(
             type: v.type, 
             to_package_name: nested_packages[v.to_package_name] || v.to_package_name, 
@@ -219,10 +237,10 @@ module VisualizePacks
           metadata: package.metadata,
           dependencies: morphed_dependencies,
           config: package.config,
-          violations: morphed_violations
+          violations: morphed_todos
         )
         # add dependencies TO nested packages to top-level package
-        # add violations TO nested packages to top-level package
+        # add todos TO nested packages to top-level package
       end
     end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: visualize_packs
 version: !ruby/object:Gem::Version
-  version: 0.5.5
+  version: 0.5.7
 platform: ruby
 authors:
 - Gusto Engineers
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-08-25 00:00:00.000000000 Z
+date: 2023-08-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler