use_packs 0.0.14 → 0.0.16

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fb4b5cfe33a3c5881971d27c78fefe591378e9728baf6513beffc14058ef23d6
-  data.tar.gz: 15144da46c38e628af6bfc3c5243d8d0c42a46b2fa739b66339fb120e31c6d36
+  metadata.gz: e985f265dd9fc9a580f8ebe329bb89a41b3445a67ca4b4416cb627c7adf3ce6e
+  data.tar.gz: 31da3a703da54e26119c1075fb27b24aed4dd0b76c93af12c8b8de827b229d2e
 SHA512:
-  metadata.gz: ce3dbd0c9c3b103887e0ff3e84421396cc3e711b4cac3626e34e622d403ab806eb63ec50e3bb4daf64b647d7f991d940e5973e60cd28e1a8b1a4880635be2525
-  data.tar.gz: 9f38e2e00c87dc857a279a726bd7539d5344a0a51dbd5acfeac4153194dcaef5bcab1e577c9a11417963602f652d7559e33a9dba5a77f6372bec066e8ac058c2
+  metadata.gz: 4ead72b2296e414cbc512c476ff11e0df9ac53b7407920534742d9039e917998d2e329758666c3a201a339b1c6359189303b8b5b1661265441246c67425d6ea1
+  data.tar.gz: e54e4771949e4d9ff229fd1c440e09a9ba042deebf1636a0e3bf1e8a2205078243dcb265d07f48357cb6e44babbd55dc1b6f8ce7013316eec7427f63ab8fb039

data/README.md

@@ -3,47 +3,90 @@
 UsePacks is a gem that helps in creating and maintaining packs. It exists to help perform some basic operations needed for pack setup and configuration. It provides a basic ruby file packager utility for [`packwerk`](https://github.com/Shopify/packwerk/). It assumes you are using [`stimpack`](https://github.com/rubyatscale/stimpack) to organize your packages.
 
 ## Usage
-### General Help
-`bin/packs --help` or just `bin/packs` to enter interactive mode.
+Make sure to run `bundle binstub use_packs` to generate `bin/packs` within your application.
 
-### Pack Creation
-`bin/packs create packs/your_pack_name_here`
+## CLI Documentation
+## Describe available commands or one specific command
+`bin/packs help [COMMAND]`
 
-### Moving files to packs
-`bin/packs move packs/your_pack_name_here path/to/file.rb path/to/directory`
-This is used for moving files into a pack (the pack must already exist).
-Note this works for moving files to packs from the monolith or from other packs
+## Create pack with name packs/your_pack
+`bin/packs create packs/your_pack`
 
-Make sure there are no spaces between the comma-separated list of paths of directories.
+## Add packs/to_pack to packs/from_pack/package.yml list of dependencies
+`bin/packs add_dependency packs/from_pack packs/to_pack`
 
-### Moving a file to public API
-`bin/packs make_public path/to/file.rb,path/to/directory`
-This moves a file or directory to public API (that is -- the `app/public` folder).
+Use this to add a dependency between packs.
 
-Make sure there are no spaces between the comma-separated list of paths of directories.
+When you use bin/packs add_dependency packs/from_pack packs/to_pack, this command will
+modify packs/from_pack/package.yml's list of dependencies and add packs/to_pack.
+
+This command will also sort the list and make it unique.
+
+## List the top dependency violations of packs/your_pack
+`bin/packs list_top_dependency_violations packs/your_pack`
+
+Want to see who is depending on you? Not sure how your pack's code is being used in an unstated way
+
+You can use this command to list the top dependency violations.
+
+If no pack name is passed in, this will list out violations across all packs.
+
+## List the top privacy violations of packs/your_pack
+`bin/packs list_top_privacy_violations packs/your_pack`
 
-### Listing top privacy violations
-`bin/packs list_top_privacy_violations packs/my_pack`
 Want to create interfaces? Not sure how your pack's code is being used?
 
 You can use this command to list the top privacy violations.
 
 If no pack name is passed in, this will list out violations across all packs.
 
-### Listing top dependency violations
-`bin/packs list_top_dependency_violations packs/my_pack`
-Want to see who is depending on you? Not sure how your pack's code is being used in an unstated way
+## Make files or directories public API
+`bin/packs make_public path/to/file.rb path/to/directory`
 
-You can use this command to list the top dependency violations.
+This moves a file or directory to public API (that is -- the `app/public` folder).
 
-If no pack name is passed in, this will list out violations across all packs.
+Make sure there are no spaces between the comma-separated list of paths of directories.
+
+## Move files or directories from one pack to another
+`bin/packs move packs/destination_pack path/to/file.rb path/to/directory`
+
+This is used for moving files into a pack (the pack must already exist).
+Note this works for moving files to packs from the monolith or from other packs
+
+Make sure there are no spaces between the comma-separated list of paths of directories.
+
+## Lint `package_todo.yml` files to check for formatting issues
+`bin/packs lint_package_todo_yml_files`
+
+## Lint `package.yml` files
+`bin/packs lint_package_yml_files [ packs/my_pack packs/my_other_pack ]`
+
+## Run bin/packwerk validate (detects cycles)
+`bin/packs validate`
+
+## Run bin/packwerk check
+`bin/packs check [ packs/my_pack ]`
+
+## Run bin/packwerk update-todo
+`bin/packs update`
+
+## Regenerate packs/*/package_rubocop_todo.yml for one or more packs
+`bin/packs regenerate_rubocop_todo [ packs/my_pack packs/my_other_pack ]`
+
+## Get info about size and violations for packs
+`bin/packs get_info [ packs/my_pack packs/my_other_pack ]`
+
+## Visualize packs
+`bin/packs visualize [ packs/my_pack packs/my_other_pack ]`
+
+## Rename a pack
+`bin/packs rename`
 
-### Adding a dependency
-`bin/packs add_dependency packs/my_pack packs/dependency_pack_name`
+## Set packs/child_pack as a child of packs/parent_pack
+`bin/packs move_to_parent packs/child_pack packs/parent_pack `
 
-This can be used to quickly modify a `package.yml` file and add a dependency. It also cleans up the list of dependencies to sort the list and remove redundant entries.
 
-### Releasing
+## Releasing
 Releases happen automatically through github actions once a version update is committed to `main`.
 
 ## Discussions, Issues, Questions, and More

data/lib/use_packs/cli.rb

@@ -30,6 +30,13 @@ module UsePacks
     end
 
     desc 'list_top_dependency_violations packs/your_pack', 'List the top dependency violations of packs/your_pack'
+    long_desc <<~LONG_DESC
+      Want to see who is depending on you? Not sure how your pack's code is being used in an unstated way
+
+      You can use this command to list the top dependency violations.
+
+      If no pack name is passed in, this will list out violations across all packs.
+    LONG_DESC
     option :limit, type: :numeric, default: 10, aliases: :l, banner: 'Specify the limit of constants to analyze'
     sig { params(pack_name: String).void }
     def list_top_dependency_violations(pack_name)
@@ -40,6 +47,13 @@ module UsePacks
     end
 
     desc 'list_top_privacy_violations packs/your_pack', 'List the top privacy violations of packs/your_pack'
+    long_desc <<~LONG_DESC
+      Want to create interfaces? Not sure how your pack's code is being used?
+
+      You can use this command to list the top privacy violations.
+
+      If no pack name is passed in, this will list out violations across all packs.
+    LONG_DESC
     option :limit, type: :numeric, default: 10, aliases: :l, banner: 'Specify the limit of constants to analyze'
     sig { params(pack_name: String).void }
     def list_top_privacy_violations(pack_name)
@@ -49,7 +63,12 @@ module UsePacks
       )
     end
 
-    desc 'make_public path/to/file.rb path/to/directory', 'Pass in a space-separated list of file or directory paths to make public'
+    desc 'make_public path/to/file.rb path/to/directory', 'Make files or directories public API'
+    long_desc <<~LONG_DESC
+      This moves a file or directory to public API (that is -- the `app/public` folder).
+
+      Make sure there are no spaces between the comma-separated list of paths of directories.
+    LONG_DESC
     sig { params(paths: String).void }
     def make_public(*paths)
       UsePacks.make_public!(
@@ -58,7 +77,13 @@ module UsePacks
       )
     end
 
-    desc 'move packs/destination_pack path/to/file.rb path/to/directory', 'Pass in a destination pack and a space-separated list of file or directory paths to move to the destination pack'
+    desc 'move packs/destination_pack path/to/file.rb path/to/directory', 'Move files or directories from one pack to another'
+    long_desc <<~LONG_DESC
+      This is used for moving files into a pack (the pack must already exist).
+      Note this works for moving files to packs from the monolith or from other packs
+
+      Make sure there are no spaces between the comma-separated list of paths of directories.
+    LONG_DESC
     sig { params(pack_name: String, paths: String).void }
     def move(pack_name, *paths)
       UsePacks.move_to_pack!(
@@ -68,17 +93,7 @@ module UsePacks
       )
     end
 
-    desc 'move_to_parent packs/parent_pack packs/child_pack', 'Pass in a parent pack and another pack to be made as a child to the parent pack!'
-    sig { params(parent_name: String, pack_name: String).void }
-    def move_to_parent(parent_name, pack_name)
-      UsePacks.move_to_parent!(
-        parent_name: parent_name,
-        pack_name: pack_name,
-        per_file_processors: [UsePacks::RubocopPostProcessor.new, UsePacks::CodeOwnershipPostProcessor.new]
-      )
-    end
-
-    desc 'lint_package_todo_yml_files', 'Ensures `package_todo.yml` files are up to date'
+    desc 'lint_package_todo_yml_files', 'Lint `package_todo.yml` files to check for formatting issues'
     sig { void }
     def lint_package_todo_yml_files
       UsePacks.lint_package_todo_yml_files!
@@ -114,6 +129,34 @@ module UsePacks
       RuboCop::Packs.regenerate_todo(packs: parse_pack_names(pack_names))
     end
 
+    desc 'get_info [ packs/my_pack packs/my_other_pack ]', 'Get info about size and violations for packs'
+    sig { params(pack_names: String).void }
+    def get_info(*pack_names)
+      Private.get_info(packs: parse_pack_names(pack_names))
+    end
+
+    desc 'visualize [ packs/my_pack packs/my_other_pack ]', 'Visualize packs'
+    sig { params(pack_names: String).void }
+    def visualize(*pack_names)
+      Private.visualize(packs: parse_pack_names(pack_names))
+    end
+
+    desc 'rename', 'Rename a pack'
+    sig { void }
+    def rename
+      puts Private.rename_pack
+    end
+
+    desc 'move_to_parent packs/child_pack packs/parent_pack ', 'Set packs/child_pack as a child of packs/parent_pack'
+    sig { params(child_pack_name: String, parent_pack_name: String).void }
+    def move_to_parent(child_pack_name, parent_pack_name)
+      UsePacks.move_to_parent!(
+        parent_name: parent_pack_name,
+        pack_name: child_pack_name,
+        per_file_processors: [UsePacks::RubocopPostProcessor.new, UsePacks::CodeOwnershipPostProcessor.new]
+      )
+    end
+
     private
 
     # This is used by thor to know that these private methods are not intended to be CLI commands

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

@@ -17,7 +17,6 @@ module UsePacks
           sig { override.params(prompt: TTY::Prompt).void }
           def perform!(prompt)
             team_or_pack = prompt.select('Do you want info by team or by pack?', ['By team', 'By pack'])
-
             if team_or_pack == 'By team'
               teams = TeamSelector.multi_select(prompt)
               selected_packs = Packs.all.select do |p|
@@ -27,45 +26,9 @@ module UsePacks
               selected_packs = PackSelector.single_or_all_pack_multi_select(prompt, question_text: 'What pack(s) would you like info on?')
             end
 
-            inbound_violations = {}
-            outbound_violations = {}
-            ParsePackwerk.all.each do |p|
-              p.violations.each do |violation|
-                outbound_violations[p.name] ||= []
-                outbound_violations[p.name] << violation
-                inbound_violations[violation.to_package_name] ||= []
-                inbound_violations[violation.to_package_name] << violation
-              end
-            end
-
             puts "You've selected #{selected_packs.count} packs. Wow! Here's all the info."
-            all_inbound = T.let([], T::Array[ParsePackwerk::Violation])
-            all_outbound = T.let([], T::Array[ParsePackwerk::Violation])
-            selected_packs.each do |pack|
-              all_inbound += inbound_violations[pack.name] || []
-              all_outbound += outbound_violations[pack.name] || []
-            end
 
-            puts "There are #{all_inbound.select(&:privacy?).sum { |v| v.files.count }} total inbound privacy violations"
-            puts "There are #{all_inbound.select(&:dependency?).sum { |v| v.files.count }} total inbound dependency violations"
-            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.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.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] || []
-              puts "There are #{inbound_for_pack.select(&:privacy?).sum { |v| v.files.count }} inbound privacy violations"
-              puts "There are #{inbound_for_pack.flatten.select(&:dependency?).sum { |v| v.files.count }} inbound dependency violations"
-              puts "There are #{outbound_for_pack.select(&:privacy?).sum { |v| v.files.count }} outbound privacy violations"
-              puts "There are #{outbound_for_pack.flatten.select(&:dependency?).sum { |v| v.files.count }} outbound dependency violations"
-            end
+            Private.get_info(packs: selected_packs)
           end
         end
       end

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

@@ -0,0 +1,25 @@
+# typed: strict
+
+module UsePacks
+  module Private
+    module InteractiveCli
+      module UseCases
+        class LintPackageYmlTodoFiles
+          extend T::Sig
+          extend T::Helpers
+          include Interface
+
+          sig { override.params(prompt: TTY::Prompt).void }
+          def perform!(prompt)
+            Private.lint_package_todo_yml_files!
+          end
+
+          sig { override.returns(String) }
+          def user_facing_name
+            'Lint .package_todo.yml files'
+          end
+        end
+      end
+    end
+  end
+end

data/lib/use_packs/private/interactive_cli/use_cases/{lint_package_yml.rb → lint_package_yml_files.rb}

@@ -4,7 +4,7 @@ module UsePacks
   module Private
     module InteractiveCli
       module UseCases
-        class LintPackageYml
+        class LintPackageYmlFiles
           extend T::Sig
           extend T::Helpers
           include Interface

data/lib/use_packs/private/interactive_cli/use_cases/{nest.rb → move_to_parent.rb}

@@ -4,14 +4,14 @@ module UsePacks
   module Private
     module InteractiveCli
       module UseCases
-        class Nest
+        class MoveToParent
           extend T::Sig
           extend T::Helpers
           include Interface
 
           sig { override.params(prompt: TTY::Prompt).void }
           def perform!(prompt)
-            child_pack = PackSelector.single_pack_select(prompt, question_text: 'Please select the pack that will be nested')
+            child_pack = PackSelector.single_pack_select(prompt, question_text: 'Please select the child pack that will be nested')
             parent_pack = PackSelector.single_pack_select(prompt, question_text: 'Please select the pack that will be the parent')
             UsePacks.move_to_parent!(
               parent_name: parent_pack.name,
@@ -22,7 +22,7 @@ module UsePacks
 
           sig { override.returns(String) }
           def user_facing_name
-            'Nest one pack under another'
+            'Move a child pack to be nested under a parent pack'
           end
         end
       end

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

@@ -16,16 +16,7 @@ module UsePacks
 
           sig { override.params(prompt: TTY::Prompt).void }
           def perform!(prompt)
-            prompt.warn(<<~WARNING)
-              We do not yet have an automated API for this.
-
-              Follow these steps:
-              1. Rename the `packs/your_pack` directory to the name of the new pack, `packs/new_pack_name
-              2. Replace references to `- packs/your_pack` in `package.yml` files with `- packs/new_pack_name`
-              3. Rerun `bin/packwerk update-todo` to update violations
-              4. Run `bin/codeownership validate` to update ownership information
-              5. Please let us know if anything is missing.
-            WARNING
+            prompt.warn(Private.rename_pack)
           end
         end
       end

data/lib/use_packs/private/interactive_cli/use_cases/{update_todo.rb → update.rb}

@@ -4,19 +4,19 @@ module UsePacks
   module Private
     module InteractiveCli
       module UseCases
-        class UpdateTodo
+        class Update
           extend T::Sig
           extend T::Helpers
           include Interface
 
           sig { override.returns(String) }
           def user_facing_name
-            'Run bin/packwerk update-todo'
+            'Run bin/packwerk update'
           end
 
           sig { override.params(prompt: TTY::Prompt).void }
           def perform!(prompt)
-            system('bin/packwerk update-todo')
+            system('bin/packwerk update')
           end
         end
       end

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

@@ -29,10 +29,7 @@ module UsePacks
                 selected_packs = PackSelector.single_or_all_pack_multi_select(prompt)
               end
 
-              packwerk_packages = selected_packs.map do |pack|
-                T.must(ParsePackwerk.find(pack.name))
-              end
-              VisualizePackwerk.package_graph!(packwerk_packages)
+              Private.visualize(packs: selected_packs)
             end
           end
 

data/lib/use_packs/private/interactive_cli.rb

@@ -13,13 +13,13 @@ require 'use_packs/private/interactive_cli/use_cases/add_dependency'
 require 'use_packs/private/interactive_cli/use_cases/get_info'
 require 'use_packs/private/interactive_cli/use_cases/query'
 require 'use_packs/private/interactive_cli/use_cases/make_public'
-require 'use_packs/private/interactive_cli/use_cases/nest'
+require 'use_packs/private/interactive_cli/use_cases/move_to_parent'
 require 'use_packs/private/interactive_cli/use_cases/rename'
 require 'use_packs/private/interactive_cli/use_cases/check'
-require 'use_packs/private/interactive_cli/use_cases/update_todo'
+require 'use_packs/private/interactive_cli/use_cases/update'
 require 'use_packs/private/interactive_cli/use_cases/validate'
 require 'use_packs/private/interactive_cli/use_cases/regenerate_rubocop_todo'
-require 'use_packs/private/interactive_cli/use_cases/lint_package_yml'
+require 'use_packs/private/interactive_cli/use_cases/lint_package_yml_files'
 require 'use_packs/private/interactive_cli/use_cases/visualize'
 
 module UsePacks

data/lib/use_packs/private.rb

@@ -477,6 +477,106 @@ module UsePacks
     def self.packwerk_package_to_pack(package)
       Packs.find(package.name)
     end
+
+    sig { params(packs: T::Array[Packs::Pack]).void }
+    def self.get_info(packs: Packs.all)
+      inbound_violations = {}
+      outbound_violations = {}
+      ParsePackwerk.all.each do |p|
+        p.violations.each do |violation|
+          outbound_violations[p.name] ||= []
+          outbound_violations[p.name] << violation
+          inbound_violations[violation.to_package_name] ||= []
+          inbound_violations[violation.to_package_name] << violation
+        end
+      end
+
+      all_inbound = T.let([], T::Array[ParsePackwerk::Violation])
+      all_outbound = T.let([], T::Array[ParsePackwerk::Violation])
+      packs.each do |pack|
+        all_inbound += inbound_violations[pack.name] || []
+        all_outbound += outbound_violations[pack.name] || []
+      end
+
+      puts "There are #{all_inbound.select(&:privacy?).sum { |v| v.files.count }} total inbound privacy violations"
+      puts "There are #{all_inbound.select(&:dependency?).sum { |v| v.files.count }} total inbound dependency violations"
+      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"
+
+      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.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] || []
+        puts "There are #{inbound_for_pack.select(&:privacy?).sum { |v| v.files.count }} inbound privacy violations"
+        puts "There are #{inbound_for_pack.flatten.select(&:dependency?).sum { |v| v.files.count }} inbound dependency violations"
+        puts "There are #{outbound_for_pack.select(&:privacy?).sum { |v| v.files.count }} outbound privacy violations"
+        puts "There are #{outbound_for_pack.flatten.select(&:dependency?).sum { |v| v.files.count }} outbound dependency violations"
+      end
+    end
+
+    sig { void }
+    def self.lint_package_todo_yml_files!
+      contents_before = Private.get_package_todo_contents
+      UsePacks.execute(['update-todo'])
+      contents_after = Private.get_package_todo_contents
+      diff = Private.diff_package_todo_yml(contents_before, contents_after)
+
+      if diff == ''
+        # No diff generated by `update-todo`
+        exit 0
+      else
+        output = <<~OUTPUT
+          All `package_todo.yml` files must be up-to-date and that no diff is generated when running `bin/packwerk update-todo`.
+          This helps ensure a high quality signal in other engineers' PRs when inspecting new violations by ensuring there are no unrelated changes.
+
+          There are three main reasons there may be a diff:
+          1) Most likely, you may have stale violations, meaning there are old violations that no longer apply.
+          2) You may have some sort of auto-formatter set up somewhere (e.g. something that reformats YML files) that is, for example, changing double quotes to single quotes. Ensure this is turned off for these auto-generated files.
+          3) You may have edited these files manually. It's recommended to use the `bin/packwerk update-todo` command to make changes to `package_todo.yml` files.
+
+          In all cases, you can run `bin/packwerk update-todo` to update these files.
+
+          Here is the diff generated after running `update-todo`:
+          ```
+          #{diff}
+          ```
+
+        OUTPUT
+
+        puts output
+        UsePacks.config.on_package_todo_lint_failure.call(output)
+
+        exit 1
+      end
+    end
+
+    sig { params(packs: T::Array[Packs::Pack]).void }
+    def self.visualize(packs: Packs.all)
+      packwerk_packages = packs.map do |pack|
+        T.must(ParsePackwerk.find(pack.name))
+      end
+      VisualizePackwerk.package_graph!(packwerk_packages)
+    end
+
+    sig { returns(String) }
+    def self.rename_pack
+      <<~WARNING
+        We do not yet have an automated API for this.
+
+        Follow these steps:
+        1. Rename the `packs/your_pack` directory to the name of the new pack, `packs/new_pack_name
+        2. Replace references to `- packs/your_pack` in `package.yml` files with `- packs/new_pack_name`
+        3. Rerun `bin/packwerk update-todo` to update violations
+        4. Run `bin/codeownership validate` to update ownership information
+        5. Please let us know if anything is missing.
+      WARNING
+    end
   end
 
   private_constant :Private

data/lib/use_packs.rb

@@ -248,38 +248,7 @@ module UsePacks
 
   sig { void }
   def self.lint_package_todo_yml_files!
-    contents_before = Private.get_package_todo_contents
-    UsePacks.execute(['update-todo'])
-    contents_after = Private.get_package_todo_contents
-    diff = Private.diff_package_todo_yml(contents_before, contents_after)
-
-    if diff == ''
-      # No diff generated by `update-todo`
-      exit 0
-    else
-      output = <<~OUTPUT
-        All `package_todo.yml` files must be up-to-date and that no diff is generated when running `bin/packwerk update-todo`.
-        This helps ensure a high quality signal in other engineers' PRs when inspecting new violations by ensuring there are no unrelated changes.
-
-        There are three main reasons there may be a diff:
-        1) Most likely, you may have stale violations, meaning there are old violations that no longer apply.
-        2) You may have some sort of auto-formatter set up somewhere (e.g. something that reformats YML files) that is, for example, changing double quotes to single quotes. Ensure this is turned off for these auto-generated files.
-        3) You may have edited these files manually. It's recommended to use the `bin/packwerk update-todo` command to make changes to `package_todo.yml` files.
-
-        In all cases, you can run `bin/packwerk update-todo` to update these files.
-
-        Here is the diff generated after running `update-todo`:
-        ```
-        #{diff}
-        ```
-
-      OUTPUT
-
-      puts output
-      UsePacks.config.on_package_todo_lint_failure.call(output)
-
-      exit 1
-    end
+    Private.lint_package_todo_yml_files!
   end
 
   sig { params(packs: T::Array[Packs::Pack]).void }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: use_packs
 version: !ruby/object:Gem::Version
-  version: 0.0.14
+  version: 0.0.16
 platform: ruby
 authors:
 - Gusto Engineers
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-03-30 00:00:00.000000000 Z
+date: 2023-04-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: code_ownership
@@ -320,14 +320,15 @@ files:
 - lib/use_packs/private/interactive_cli/use_cases/create.rb
 - lib/use_packs/private/interactive_cli/use_cases/get_info.rb
 - lib/use_packs/private/interactive_cli/use_cases/interface.rb
-- lib/use_packs/private/interactive_cli/use_cases/lint_package_yml.rb
+- lib/use_packs/private/interactive_cli/use_cases/lint_package_todo_yml_files.rb
+- lib/use_packs/private/interactive_cli/use_cases/lint_package_yml_files.rb
 - lib/use_packs/private/interactive_cli/use_cases/make_public.rb
 - lib/use_packs/private/interactive_cli/use_cases/move.rb
-- lib/use_packs/private/interactive_cli/use_cases/nest.rb
+- lib/use_packs/private/interactive_cli/use_cases/move_to_parent.rb
 - lib/use_packs/private/interactive_cli/use_cases/query.rb
 - lib/use_packs/private/interactive_cli/use_cases/regenerate_rubocop_todo.rb
 - lib/use_packs/private/interactive_cli/use_cases/rename.rb
-- lib/use_packs/private/interactive_cli/use_cases/update_todo.rb
+- lib/use_packs/private/interactive_cli/use_cases/update.rb
 - lib/use_packs/private/interactive_cli/use_cases/validate.rb
 - lib/use_packs/private/interactive_cli/use_cases/visualize.rb
 - lib/use_packs/private/pack_relationship_analyzer.rb