use_packs 0.0.11 → 0.0.13

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 56ce2bde968a560b111d1be1dde1ea510e11cdf2fd5207ee61200719a535af2b
-  data.tar.gz: 369878a0a5e538739ed19f2050176bbb58397a98e1e8a12f653fd44a65a7a4a8
+  metadata.gz: e860d861797457083f35ab124de8f1b2417ab092772f4bb094622dfacda64cef
+  data.tar.gz: 7349fe62972a5423fd14a99b367eaf8eede053085a3be7afe08ae2de8556e804
 SHA512:
-  metadata.gz: 760dcede5ac85d7c11f9f8494b299060f11b1dab1518b431f337f5e196595213d21cc49fba3847c9ca116df68afea838f21a0986ce6d8166567765c15d94230f
-  data.tar.gz: 9d846a10c935462ebb2c36db38afc26702b6191dafdba359e5401d8511e8c1b1c9901f39e7edd7d4bfff0d112f38770b228772cd5c28243db187d6033d842c14
+  metadata.gz: 8a41d04d25654c0d31ef45ff37fc3a87ff9373e52d29f0b2848dc4cb0fcd55a4425d5a6de91bab60484258b0093ba1e463785cbba972435076756501ac0259e3
+  data.tar.gz: 0b285e3a349ee94ac6983e1546a17d09633ed6a49ac72e55b608cdc7d1a7f01a345ebaf1f8c0204922199c93abc9c4590b25a30745aa17995ef08b7902b29a2c

data/lib/use_packs/cli.rb

@@ -102,10 +102,10 @@ module UsePacks
       UsePacks.execute(['check', *paths])
     end
 
-    desc 'update [ packs/my_pack ]', 'Run bin/packwerk update-todo'
-    sig { params(paths: String).void }
-    def update(*paths)
-      system("bin/packwerk update-todo #{paths.join(' ')}")
+    desc 'update', 'Run bin/packwerk update-todo'
+    sig { void }
+    def update
+      system('bin/packwerk update-todo')
     end
 
     desc 'regenerate_rubocop_todo [ packs/my_pack packs/my_other_pack ]', "Regenerate packs/*/#{RuboCop::Packs::PACK_LEVEL_RUBOCOP_TODO_YML} for one or more packs"
@@ -118,9 +118,9 @@ module UsePacks
 
     # This is used by thor to know that these private methods are not intended to be CLI commands
     no_commands do
-      sig { params(pack_names: T::Array[String]).returns(T::Array[ParsePackwerk::Package]) }
+      sig { params(pack_names: T::Array[String]).returns(T::Array[Packs::Pack]) }
       def parse_pack_names(pack_names)
-        pack_names.empty? ? ParsePackwerk.all : pack_names.map { |p| ParsePackwerk.find(p.gsub(%r{/$}, '')) }.compact
+        pack_names.empty? ? Packs.all : pack_names.map { |p| Packs.find(p.gsub(%r{/$}, '')) }.compact
       end
     end
   end

data/lib/use_packs/code_ownership_post_processor.rb

@@ -34,7 +34,8 @@ module UsePacks
         @teams << 'Unknown'
       end
 
-      if !CodeOwnership.for_package(file_move_operation.destination_pack).nil?
+      pack = Packs.find(file_move_operation.destination_pack.name)
+      if pack && !CodeOwnership.for_package(pack).nil?
         CodeOwnership.remove_file_annotation!(relative_path_to_origin.to_s)
         @did_move_files = true
       end

data/lib/use_packs/private/interactive_cli/pack_selector.rb

@@ -6,9 +6,9 @@ module UsePacks
       class PackSelector
         extend T::Sig
 
-        sig { params(prompt: TTY::Prompt, question_text: String).returns(ParsePackwerk::Package) }
+        sig { params(prompt: TTY::Prompt, question_text: String).returns(Packs::Pack) }
         def self.single_pack_select(prompt, question_text: 'Please use space to select a pack')
-          packs = ParsePackwerk.all.to_h { |t| [t.name, t] }
+          packs = Packs.all.to_h { |t| [t.name, t] }
 
           pack_selection = T.let(prompt.select(
             question_text,
@@ -16,7 +16,7 @@ module UsePacks
             filter: true,
             per_page: 10,
             show_help: :always
-          ), T.nilable(ParsePackwerk::Package))
+          ), T.nilable(Packs::Pack))
 
           while pack_selection.nil?
             prompt.error(
@@ -29,15 +29,15 @@ module UsePacks
           pack_selection
         end
 
-        sig { params(prompt: TTY::Prompt, question_text: String).returns(T::Array[ParsePackwerk::Package]) }
+        sig { params(prompt: TTY::Prompt, question_text: String).returns(T::Array[Packs::Pack]) }
         def self.single_or_all_pack_multi_select(prompt, question_text: 'Please use space to select one or more packs')
           pack_selection = T.let(prompt.multi_select(
             question_text,
-            ParsePackwerk.all.to_h { |t| [t.name, t] },
+            Packs.all.to_h { |t| [t.name, t] },
             filter: true,
             per_page: 10,
             show_help: :always
-          ), T::Array[ParsePackwerk::Package])
+          ), T::Array[Packs::Pack])
 
           while pack_selection.empty?
             prompt.error(

data/lib/use_packs/private/interactive_cli/use_cases/get_info.rb

@@ -20,7 +20,7 @@ module UsePacks
 
             if team_or_pack == 'By team'
               teams = TeamSelector.multi_select(prompt)
-              selected_packs = ParsePackwerk.all.select do |p|
+              selected_packs = Packs.all.select do |p|
                 teams.map(&:name).include?(CodeOwnership.for_package(p)&.name)
               end
             else
@@ -51,12 +51,13 @@ module UsePacks
             puts "There are #{all_outbound.select(&:privacy?).sum { |v| v.files.count }} total outbound privacy violations"
             puts "There are #{all_outbound.select(&:dependency?).sum { |v| v.files.count }} total outbound dependency violations"
 
-            selected_packs.sort_by { |p| -p.directory.glob('**/*.rb').count }.each do |pack|
+            selected_packs.sort_by { |p| -p.relative_path.glob('**/*.rb').count }.each do |pack|
               puts "\n=========== Info about: #{pack.name}"
+
               owner = CodeOwnership.for_package(pack)
               puts "Owned by: #{owner.nil? ? 'No one' : owner.name}"
-              puts "Size: #{pack.directory.glob('**/*.rb').count} ruby files"
-              puts "Public API: #{pack.directory.join('app/public')}"
+              puts "Size: #{pack.relative_path.glob('**/*.rb').count} ruby files"
+              puts "Public API: #{pack.relative_path.join('app/public')}"
 
               inbound_for_pack = inbound_violations[pack.name] || []
               outbound_for_pack = outbound_violations[pack.name] || []

data/lib/use_packs/private/interactive_cli/use_cases/visualize.rb

@@ -22,14 +22,17 @@ module UsePacks
               by_name_or_by_owner = prompt.select('Do you select packs by name or by owner?', ['By name', 'By owner'])
               if by_name_or_by_owner == 'By owner'
                 teams = TeamSelector.multi_select(prompt)
-                selected_packs = ParsePackwerk.all.select do |p|
+                selected_packs = Packs.all.select do |p|
                   teams.map(&:name).include?(CodeOwnership.for_package(p)&.name)
                 end
               else
                 selected_packs = PackSelector.single_or_all_pack_multi_select(prompt)
               end
 
-              VisualizePackwerk.package_graph!(selected_packs)
+              packwerk_packages = selected_packs.map do |pack|
+                T.must(ParsePackwerk.find(pack.name))
+              end
+              VisualizePackwerk.package_graph!(packwerk_packages)
             end
           end
 

data/lib/use_packs/private.rb

@@ -378,7 +378,8 @@ module UsePacks
         )
 
         ParsePackwerk.write_package_yml!(package)
-        RuboCop::Packs.set_default_rubocop_yml(packs: [package])
+        pack = Packs.find(package.name)
+        RuboCop::Packs.set_default_rubocop_yml(packs: [pack].compact)
 
         current_contents = package.yml.read
         new_contents = current_contents.gsub('MyTeam', 'MyTeam # specify your team here, or delete this key if this package is not owned by one team')
@@ -452,6 +453,22 @@ module UsePacks
         temp_package_todo_yml.write(contents)
       end
     end
+
+    sig { params(packages: T::Array[ParsePackwerk::Package]).returns(T::Array[Packs::Pack]) }
+    def self.packwerk_packages_to_packs(packages)
+      packs = []
+      packages.each do |package|
+        pack = Packs.find(package.name)
+        packs << pack if !pack.nil?
+      end
+
+      packs
+    end
+
+    sig { params(package: ParsePackwerk::Package).returns(T.nilable(Packs::Pack)) }
+    def self.packwerk_package_to_pack(package)
+      Packs.find(package.name)
+    end
   end
 
   private_constant :Private

data/lib/use_packs.rb

@@ -282,16 +282,19 @@ module UsePacks
     end
   end
 
-  sig { params(packs: T::Array[ParsePackwerk::Package]).void }
+  sig { params(packs: T::Array[Packs::Pack]).void }
   def self.lint_package_yml_files!(packs)
     packs.each do |p|
+      packwerk_package = ParsePackwerk.find(p.name)
+      next if packwerk_package.nil?
+
       new_package = ParsePackwerk::Package.new(
-        name: p.name,
-        enforce_privacy: p.enforce_privacy,
-        enforce_dependencies: p.enforce_dependencies,
-        dependencies: p.dependencies.uniq.sort,
-        metadata: p.metadata,
-        config: p.config
+        name: packwerk_package.name,
+        enforce_privacy: packwerk_package.enforce_privacy,
+        enforce_dependencies: packwerk_package.enforce_dependencies,
+        dependencies: packwerk_package.dependencies.uniq.sort,
+        metadata: packwerk_package.metadata,
+        config: packwerk_package.config
       )
       ParsePackwerk.write_package_yml!(new_package)
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: use_packs
 version: !ruby/object:Gem::Version
-  version: 0.0.11
+  version: 0.0.13
 platform: ruby
 authors:
 - Gusto Engineers
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-12-27 00:00:00.000000000 Z
+date: 2022-12-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: code_ownership
@@ -38,6 +38,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: packs
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: packwerk
   requirement: !ruby/object:Gem::Requirement