visualize_packs 0.5.19 → 0.5.20

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 21c174ca9487c1d11560c585972f1da5ff9b8ddc53148537753af82d1fb66a25
-  data.tar.gz: 2b910b38ad69c3df0631a98b21ec4ca6651190ab973bfc4ba4692e0f1fb7d740
+  metadata.gz: b8efefa896a6d002c7333dad53f2a8fa0be10d8c0004895682239c7a7f6a1105
+  data.tar.gz: 0046a3a518c25de488b4b2e0d89c04ced6e45c1ff5d90e9e85d71d46b5256e89
 SHA512:
-  metadata.gz: 883ddfb826a9f9051b0431d289fd98b6a92d41c41a42f4e560c0fae89a9432e741c5d9fddd2802a9eae64c9c1a6073d541f26ca8269c40d7f27b946908881088
-  data.tar.gz: 6136452d922cefa21486b40c18b2df6fda1838f25030d66019edfd291101dcae503777f7ecc9581ccf07d0c894f5800adc44248a18b931359d2ab2e1eb65d727
+  metadata.gz: 5282d8a1af4103a3d6c38675c378adee512029eb4320a29874b91463c2fc4aa7380a76165306f8dc2d8163b67e60875b2496aeacca209b40ebdb09217744888c
+  data.tar.gz: 99f84713ec95080f82668184bd69ce4e67dc30fbc739803f67efe8ebbe51728dbd4717d22eec946a2e24ce1128cd646d3cca47413a09a481718b76c029c47ae7

data/bin/visualize_packs

@@ -34,6 +34,13 @@ OptionParser.new do |opt|
 
   opt.on('--title=STRING', "Set a custom diagram title") { |o| options.title = o }
 
+  opt.on('-V', '--version', "Show version") do
+    spec_path = File.expand_path("../visualize_packs.gemspec", __dir__)
+    spec = Gem::Specification::load(spec_path)
+    puts "Version #{spec.version}"
+    exit
+  end
+
   opt.on_tail("-h", "--help", "Show this message") do
     puts opt
     exit

data/lib/visualize_packs.rb

@@ -70,25 +70,31 @@ module VisualizePacks
   def self.diagram_title(options, max_todo_count)
     return "<<b>#{options.title}</b>>" if options.title
 
-    app_name = File.basename(Dir.pwd)
-    focus_edge_info = options.focus_pack && options.show_only_edges_to_focus_pack != FocusPackEdgeDirection::All ? "showing only edges to/from focus pack" : "showing all edges between visible packs"
-    focus_info = options.focus_pack ? "Focus on #{limited_sentence(options.focus_pack)} (#{focus_edge_info})" : "All packs"
-    skipped_info = 
-    [
-      options.show_legend ? nil : "hiding legend",
-      options.show_layers ? nil : "hiding layers",
-      options.show_dependencies ? nil : "hiding dependencies",
-      options.show_todos ? nil : "hiding todos",
-      EdgeTodoTypes.values.size == options.only_todo_types.size ? nil : "only #{limited_sentence(options.only_todo_types.map &:serialize)} todos",
-      options.show_privacy ? nil : "hiding privacy",
-      options.show_teams ? nil : "hiding teams",
-      options.show_visibility ? nil : "hiding visibility",
-      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)}",
+    focus_info = if options.focus_pack
+      "Focus on #{limited_sentence(options.focus_pack)} (Edge mode: #{options.show_only_edges_to_focus_pack.serialize})"
+    else 
+      "All packs"
+    end
+
+    hidden_aspects = [
+      options.show_legend ? nil : "legend",
+      options.show_layers ? nil : "layers",
+      options.show_dependencies ? nil : "dependencies",
+      options.show_todos ? nil : "todos",
+      options.show_privacy ? nil : "privacy",
+      options.show_teams ? nil : "teams",
+      options.show_visibility ? nil : "visibility",
+      options.roll_nested_into_parent_packs ? "nested packs" : nil,
+      options.show_nested_relationships ? nil : "nested relationships",
     ].compact.join(', ').strip
-    main_title = "#{app_name}: #{focus_info}#{skipped_info != '' ? ' - ' + skipped_info : ''}"
-    sub_title = ""
+    hidden_aspects_title = hidden_aspects != '' ? "Hiding #{hidden_aspects}" : nil
+
+    todo_types = EdgeTodoTypes.values.size == options.only_todo_types.size ? nil : "Only #{options.only_todo_types.map &:serialize} todos",
+
+    exclusions = options.exclude_packs.empty? ? nil : "Excluding pack#{options.exclude_packs.size > 1 ? 's' : ''}: #{limited_sentence(options.exclude_packs)}",
+
+    main_title = [focus_info, hidden_aspects_title, todo_types, exclusions].compact.join('. ')
+
     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
@@ -115,6 +121,8 @@ module VisualizePacks
         case options.show_only_edges_to_focus_pack
         when FocusPackEdgeDirection::All then
           true
+        when FocusPackEdgeDirection::None then
+          match_packs?(start_node, options.focus_pack) && match_packs?(end_node, options.focus_pack)
         when FocusPackEdgeDirection::InOut then
           match_packs?(start_node, options.focus_pack) || match_packs?(end_node, options.focus_pack)
         when FocusPackEdgeDirection::In then
@@ -200,36 +208,32 @@ module VisualizePacks
     result = T.let([], T::Array[T.nilable(String)])
     result = packages.map { |pack| pack.name }
 
-    if focus_pack && !focus_pack.empty?
+    if focus_pack
       result = []
-      result += packages.map { |pack| pack.name }.select { |p| match_packs?(p, focus_pack) }
-      if options.show_dependencies
-        result += packages.select { |p| p.dependencies.any? { |d| match_packs?(d, focus_pack) }}.map { |pack| pack.name }
-      end
-      if options.show_todos && [FocusPackEdgeDirection::All, FocusPackEdgeDirection::In, FocusPackEdgeDirection::InOut].include?(options.show_only_edges_to_focus_pack)
-        result += packages.select do
-          |p| (p.violations || []).inject([]) do |res, todo|
-            res << todo.to_package_name if options.only_todo_types.include?(EdgeTodoTypes.deserialize(todo.type))
-            res
-          end.any? { |v| match_packs?(v, focus_pack) }
-        end.map { |pack| pack.name }
+      focus_pack_name = packages.map { |pack| pack.name }.select { |p| match_packs?(p, focus_pack) }
+      result += focus_pack_name
+
+      dependents = options.show_dependencies ? dependents_on(packages, focus_pack_name) : []
+      dependencies = options.show_dependencies ? dependencies_of(packages, focus_pack_name) : []
+      todos_out = options.show_todos ? todos_out(packages, focus_pack_name, options) : []
+      todos_in = options.show_todos ? todos_in(packages, focus_pack_name, options) : []
+
+      case options.show_only_edges_to_focus_pack
+      when FocusPackEdgeDirection::All, FocusPackEdgeDirection::InOut then
+        result += dependents + dependencies + todos_out + todos_in
+      when FocusPackEdgeDirection::In then
+        result += dependents + todos_in
+      when FocusPackEdgeDirection::Out then
+        result += dependencies + todos_out 
+      when FocusPackEdgeDirection::None then
+        # nothing to do
       end
-      packages.map { |pack| pack.name }.select { |p| match_packs?(p, focus_pack) }.each do |p|
-        if options.show_dependencies
-          result += packages_by_name[p].dependencies
-        end
-        if options.show_todos && [FocusPackEdgeDirection::All, FocusPackEdgeDirection::Out, FocusPackEdgeDirection::InOut].include?(options.show_only_edges_to_focus_pack)
-          result += (packages_by_name[p].violations || []).inject([]) do |res, todo|
-            res << todo.to_package_name if options.only_todo_types.include?(EdgeTodoTypes.deserialize(todo.type))
-            res
-          end
-        end
-      end
-      result = result.uniq
+
       parent_packs = result.inject([]) do |res, package_name|
         res << nested_packages[package_name]
         res
       end
+
       result = (result + parent_packs).uniq.compact
     end
 
@@ -318,4 +322,39 @@ module VisualizePacks
   def self.match_packs?(pack, packs_name_with_wildcards)
     !packs_name_with_wildcards || packs_name_with_wildcards.any? {|p| File.fnmatch(p, pack)}
   end
+
+  sig { params(all_packages: T::Array[ParsePackwerk::Package], focus_packs_names: T::Array[String]).returns(T::Array[String]) }
+  def self.dependencies_of(all_packages, focus_packs_names)
+    focus_packs = all_packages.select { focus_packs_names.include?(_1.name)}
+    
+    focus_packs.inject([]) do |result, pack|
+      result += pack.dependencies
+      result
+    end.uniq
+  end
+
+  sig { params(all_packages: T::Array[ParsePackwerk::Package], focus_packs_names: T::Array[String]).returns(T::Array[String]) }
+  def self.dependents_on(all_packages, focus_packs_names)
+    all_packages.select { |pack| pack.dependencies.any? { focus_packs_names.include?(_1) }}.map &:name
+  end
+
+  sig { params(all_packages: T::Array[ParsePackwerk::Package], focus_packs_names: T::Array[String], options: Options).returns(T::Array[String]) }
+  def self.todos_in(all_packages, focus_packs_names, options)
+    all_packages.select do |p|
+      (p.violations || []).inject([]) do |res, todo|
+        res << todo.to_package_name if options.only_todo_types.include?(EdgeTodoTypes.deserialize(todo.type))
+        res
+      end.any? { |v| focus_packs_names.include?(v) }
+    end.map { |pack| pack.name }
+  end
+
+  sig { params(all_packages: T::Array[ParsePackwerk::Package], focus_packs_names: T::Array[String], options: Options).returns(T::Array[String]) }
+  def self.todos_out(all_packages, focus_packs_names, options)
+    all_packages.inject([]) do |result, p|
+      focus_packs_names.include?(p.name) && (p.violations || []).each do |todo|
+        result << todo.to_package_name if options.only_todo_types.include?(EdgeTodoTypes.deserialize(todo.type))
+      end
+      result
+    end
+  end
 end

metadata

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