visualize_packwerk 0.2.0 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c15049155fd43dca095255941ff8f996409fd2712f28008874fe3a772ba9b28b
-  data.tar.gz: 406b008fa89777838b588080aa3ee25afd417f16663124b668937ea91c3b9c17
+  metadata.gz: 4684e489b9b59adcca847e4c3261f98d487210b07c3573e9c147f94fd0044424
+  data.tar.gz: 9f3e57d74e418e25e7732822193e1b7d3fd06048c46d6d1a9da7d97a60c62a7d
 SHA512:
-  metadata.gz: b6ee92e733adf0da1062cf262849685a5adb053baef2adb34793aadedbb68c072193a19df4ff9d8eed280e491cfb0afac4bb9a1211962f6cd1d8a9b20c4bf6d8
-  data.tar.gz: 6ca30bbcadd94b1b8241456001c6d8144f6b395718cb1d765605cb0c2f1fb7b0bffe4d36c312cbb7510ed9930385013eafe598788f63d6c675abbcd8b6034539
+  metadata.gz: 51eaec81b7b964cba8e6bca64c2e1b80298f650d6fcf7fdd1c0817e016f9661a437ab86f068d9a041a903d1c83b8d31212343419a807ebefd117ed5c4dfa6832
+  data.tar.gz: e8eb48b12eb1841bd0543f660c29dd8644cf3cfd9cedf31fcab81b51f76e8353d87bfd522197b07ad6e0622cfbfaaa4649d49eee8a6b6d242f719e2510bd4e46

data/lib/visualize_packwerk/package_graph.rb

@@ -16,7 +16,7 @@ module VisualizePackwerk
     sig { params(package_nodes: T::Set[PackageNode]).void }
     def initialize(package_nodes:)
       @package_nodes = package_nodes
-      @index_by_name = T.let({}, T::Hash[String, PackageNode])
+      @index_by_name = T.let({}, T::Hash[String, T.nilable(PackageNode)])
     end
 
     sig { returns(PackageGraph) }
@@ -46,9 +46,9 @@ module VisualizePackwerk
       PackageGraph.new(package_nodes: package_nodes)
     end
 
-    sig { params(name: String).returns(PackageNode) }
+    sig { params(name: String).returns(T.nilable(PackageNode)) }
     def package_by_name(name)
-      @index_by_name[name] ||= T.must(package_nodes.find { |node| node.name == name })
+      @index_by_name[name] ||= package_nodes.find { |node| node.name == name }
     end
   end
 end

data/lib/visualize_packwerk/team_graph.rb

@@ -26,7 +26,9 @@ module VisualizePackwerk
         package_nodes_for_team.map(&:violations_by_package).each do |new_violations_by_package|
           new_violations_by_package.each do |pack_name, count|
             # We first get the pack owner of the violated package
-            other_team = package_graph.package_by_name(pack_name).team_name
+            other_package = package_graph.package_by_name(pack_name)
+            next if other_package.nil?
+            other_team = other_package.team_name
             violations_by_team[other_team] ||= 0
             # Then we add the violations on that team together
             # TODO: We may want to ignore this if team == other_team to avoid arrows pointing to self, but maybe not!
@@ -36,7 +38,9 @@ module VisualizePackwerk
 
         dependencies = Set.new
         package_nodes_for_team.map(&:dependencies).reduce(Set.new, :+).each do |dependency|
-          dependencies << package_graph.package_by_name(dependency).team_name
+          other_pack = package_graph.package_by_name(dependency)
+          next if other_pack.nil?
+          dependencies << other_pack.team_name
         end
 
         team_nodes << TeamNode.new(

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: visualize_packwerk
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.1
 platform: ruby
 authors:
 - Gusto Engineers
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-05-02 00:00:00.000000000 Z
+date: 2023-05-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sorbet-runtime