packs 0.1.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cedac59e21e3cfabcc0e70369a37f099d917c6092c75a9415e4935a9bec705af
-  data.tar.gz: 36307b5c6d741d2145e73b827ea66ace06cffa0123101443f4b0d2b30e09a5e9
+  metadata.gz: 221210e029bee06a127a705570b20d9856571bb3ad187f4be782521c65be5901
+  data.tar.gz: b401bf88686e51f1316e5e81330edca9dd1a3b8cf78a90f5c46f4dad1058a6bf
 SHA512:
-  metadata.gz: 9478f4344cdb4dc234abb52da988b9e63033234a78289d9bdcbdacb738f911c6fb98050b0c699d19157f100df7a0139ff0a4be453776bdefc0aafd140a71d9e8
-  data.tar.gz: 718c24457bc7b6f5d0b471f2954dbff83ebbbbb0602a32295fb8ffc6d0e9cfea44a8e49c3a9d26c938b78e34d1d67d774502b8db1b659e5a3f88cd3f7071802d
+  metadata.gz: ba96edf990cff199f9bdc26d79477724e294770d66de07520e48d7c7311c30d371cb8b00ae4529c35029ec65d4386e77f9009d62f5df07c29957c5e597c45dc1
+  data.tar.gz: 65696d3da54c5b85de8107ae76ed60c79c38167eacd4a98597026405379edfa7de8a117e7a347ed2e7c7ffbb1aa6e69c23145b878d97391e48de227d0d1152c0

data/README.md

@@ -53,6 +53,9 @@ bin/packwerk init
 ## Describe available commands or one specific command
 `bin/packs help [COMMAND]`
 
+## Print a tree of all available commands
+`bin/packs tree`
+
 ## Create pack with name packs/your_pack
 `bin/packs create packs/your_pack`
 
@@ -90,7 +93,7 @@ Make sure there are no spaces between the comma-separated list of paths of direc
 `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
+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.
 

data/bin/rubocop

@@ -8,22 +8,9 @@
 # this file is here to facilitate running it.
 #
 
-require 'pathname'
-ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../../Gemfile',
-  Pathname.new(__FILE__).realpath)
+ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../Gemfile", __dir__)
 
-bundle_binstub = File.expand_path('bundle', __dir__)
+require "rubygems"
+require "bundler/setup"
 
-if File.file?(bundle_binstub)
-  if File.read(bundle_binstub, 300) =~ /This file was generated by Bundler/
-    load(bundle_binstub)
-  else
-    abort("Your `bin/bundle` was not generated by Bundler, so this binstub cannot run.
-Replace `bin/bundle` by running `bundle binstubs bundler --force`, then run this command again.")
-  end
-end
-
-require 'rubygems'
-require 'bundler/setup'
-
-load Gem.bin_path('rubocop', 'rubocop')
+load Gem.bin_path("rubocop", "rubocop")

data/bin/tapioca

@@ -8,22 +8,9 @@
 # this file is here to facilitate running it.
 #
 
-require 'pathname'
-ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../../Gemfile',
-  Pathname.new(__FILE__).realpath)
+ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../Gemfile", __dir__)
 
-bundle_binstub = File.expand_path('bundle', __dir__)
+require "rubygems"
+require "bundler/setup"
 
-if File.file?(bundle_binstub)
-  if File.read(bundle_binstub, 300) =~ /This file was generated by Bundler/
-    load(bundle_binstub)
-  else
-    abort("Your `bin/bundle` was not generated by Bundler, so this binstub cannot run.
-Replace `bin/bundle` by running `bundle binstubs bundler --force`, then run this command again.")
-  end
-end
-
-require 'rubygems'
-require 'bundler/setup'
-
-load Gem.bin_path('tapioca', 'tapioca')
+load Gem.bin_path("tapioca", "tapioca")

data/lib/packs/cli.rb

@@ -39,10 +39,10 @@ module Packs
       exit_successfully
     end
 
-    POSIBLE_TYPES = T.let(%w[dependency privacy layer], T::Array[String])
+    POSSIBLE_TYPES = T.let(%w(dependency privacy layer).freeze, T::Array[String])
     desc 'list_top_violations type [ packs/your_pack ]', 'List the top violations of a specific type for packs/your_pack.'
     long_desc <<~LONG_DESC
-      Possible types are: #{POSIBLE_TYPES.join(', ')}.
+      Possible types are: #{POSSIBLE_TYPES.join(', ')}.
 
       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.
 
@@ -60,7 +60,7 @@ module Packs
       ).void
     end
     def list_top_violations(type, pack_name = nil)
-      raise StandardError, "Invalid type #{type}. Possible types are: #{POSIBLE_TYPES.join(', ')}" unless POSIBLE_TYPES.include?(type)
+      raise StandardError, "Invalid type #{type}. Possible types are: #{POSSIBLE_TYPES.join(', ')}" unless POSSIBLE_TYPES.include?(type)
 
       Packs.list_top_violations(
         type: type,
@@ -85,7 +85,7 @@ module Packs
     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
+      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
@@ -135,8 +135,8 @@ module Packs
     sig { params(pack_names: String).void }
     def get_info(*pack_names)
       selected_types = options[:types].to_s.downcase.split(',')
-      invalid_types = selected_types - POSIBLE_TYPES
-      raise StandardError, "Invalid type(s): #{invalid_types.join(', ')}. Possible types are: #{POSIBLE_TYPES.join(', ')}" unless invalid_types.empty?
+      invalid_types = selected_types - POSSIBLE_TYPES
+      raise StandardError, "Invalid type(s): #{invalid_types.join(', ')}. Possible types are: #{POSSIBLE_TYPES.join(', ')}" unless invalid_types.empty?
 
       Private.get_info(
         packs: parse_pack_names(pack_names),
@@ -176,17 +176,14 @@ module Packs
 
     private
 
-    # 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[Packs::Pack]) }
-      def parse_pack_names(pack_names)
-        pack_names.empty? ? Packs.all : pack_names.map { |p| Packs.find(p.gsub(%r{/$}, '')) }.compact
-      end
-
-      sig { void }
-      def exit_successfully
-        Private.exit_with(true)
-      end
+    sig { params(pack_names: T::Array[String]).returns(T::Array[Packs::Pack]) }
+    def parse_pack_names(pack_names)
+      pack_names.empty? ? Packs.all : pack_names.filter_map { |p| Packs.find(p.delete_suffix('/')) }
+    end
+
+    sig { void }
+    def exit_successfully
+      Private.exit_with(true)
     end
   end
 end

data/lib/packs/code_ownership_post_processor.rb

@@ -25,7 +25,7 @@ module Packs
         replace_with: relative_path_to_destination
       )
 
-      team = CodeOwnership.for_file(relative_path_to_origin.to_s)
+      team = CodeOwnership.for_file(relative_path_to_origin.to_s, from_codeowners: false)
 
       if team
         @teams << team.name

data/lib/packs/configuration.rb

@@ -30,7 +30,7 @@ module Packs
     def initialize
       @enforce_dependencies = T.let(default_enforce_dependencies, T::Boolean)
       @user_event_logger = T.let(DefaultUserEventLogger.new, UserEventLogger)
-      @on_package_todo_lint_failure = T.let(->(output) {}, OnPackageTodoLintFailure)
+      @on_package_todo_lint_failure = T.let(-> (output) {}, OnPackageTodoLintFailure)
       @use_pks = T.let(false, T::Boolean)
     end
 
@@ -68,8 +68,7 @@ module Packs
     sig { returns(Configuration) }
     def config
       Private.load_client_configuration
-      @config = T.let(@config, T.nilable(Configuration))
-      @config ||= Configuration.new
+      @config ||= T.let(Configuration.new, T.nilable(Configuration))
     end
 
     sig { params(blk: T.proc.params(arg0: Configuration).void).void }

data/lib/packs/per_file_processor_interface.rb

@@ -8,7 +8,8 @@ module Packs
     abstract!
 
     sig { abstract.params(file_move_operation: Private::FileMoveOperation).void }
-    def before_move_file!(file_move_operation); end
+    def before_move_file!(file_move_operation)
+    end
 
     sig { overridable.params(file_move_operations: T::Array[Private::FileMoveOperation]).void }
     def after_move_files!(file_move_operations)

data/lib/packs/private/file_move_operation.rb

@@ -51,8 +51,8 @@ module Packs
         ).cleanpath
       end
 
-      sig { returns(FileMoveOperation) }
-      def spec_file_move_operation
+      sig { returns(T::Array[FileMoveOperation]) }
+      def spec_file_move_operations
         path_parts = filepath_without_pack_name.split('/')
         folder = T.must(path_parts[0])
         file_extension = T.must(filepath_without_pack_name.split('.').last)
@@ -67,11 +67,20 @@ module Packs
           new_destination_pathname = spec_pathname_for_non_app(destination_pathname, file_extension, folder)
         end
 
-        FileMoveOperation.new(
-          origin_pathname: new_origin_pathname,
-          destination_pathname: new_destination_pathname,
-          destination_pack: destination_pack
-        )
+        ops = [
+          FileMoveOperation.new(
+            origin_pathname: new_origin_pathname,
+            destination_pathname: new_destination_pathname,
+            destination_pack: destination_pack
+          ),
+        ]
+
+        # For controllers, also include the request spec (spec/requests/<name>_spec.rb) if it exists.
+        # Request specs are named without "controller" suffix (e.g. my_controller -> my_spec)
+        request_spec_op = request_spec_file_move_operation
+        ops << request_spec_op if request_spec_op
+
+        ops
       end
 
       sig { params(filepath: Pathname, pack: T.nilable(Packs::Pack)).returns(String) }
@@ -94,7 +103,7 @@ module Packs
       def spec_pathname_for_app(pathname, file_extension)
         pathname
           .sub('/app/', '/spec/')
-          .sub(%r{^app/}, 'spec/')
+          .sub(%r(\Aapp/), 'spec/')
           .sub(".#{file_extension}", '_spec.rb')
       end
 
@@ -102,10 +111,39 @@ module Packs
       def spec_pathname_for_non_app(pathname, file_extension, folder)
         pathname
           .sub("/#{folder}/", "/spec/#{folder}/")
-          .sub(%r{^#{folder}/}, "spec/#{folder}/")
+          .sub(%r(\A#{folder}/), "spec/#{folder}/")
           .sub(".#{file_extension}", '_spec.rb')
       end
 
+      # Maps app/controllers/<namespaces/>*_controller.rb to spec/requests/<namespaces/>*_spec.rb.
+      # Returns nil if the path is not a controller path.
+      sig { params(pathname: Pathname).returns(T.nilable(Pathname)) }
+      def pathname_to_request_spec(pathname)
+        return nil unless pathname.to_s.end_with?('_controller.rb')
+
+        pathname
+          .sub('app/controllers/', 'spec/requests/')
+          .sub('/app/', '/spec/') # if destionation doesn't have controller subdirectory
+          .sub(%r(\Aapp/), 'spec/')
+          .sub(/_controller\.rb\z/, '_spec.rb')
+          .cleanpath
+      end
+
+      sig { returns(T.nilable(FileMoveOperation)) }
+      def request_spec_file_move_operation
+        request_spec_origin = pathname_to_request_spec(origin_pathname)
+        return if request_spec_origin.nil? || !request_spec_origin.exist?
+
+        request_spec_destination = pathname_to_request_spec(destination_pathname)
+        return if request_spec_destination.nil?
+
+        FileMoveOperation.new(
+          origin_pathname: request_spec_origin,
+          destination_pathname: request_spec_destination,
+          destination_pack: destination_pack
+        )
+      end
+
       sig { params(path: Pathname).returns(FileMoveOperation) }
       def relative_to(path)
         FileMoveOperation.new(

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

@@ -10,13 +10,15 @@ module Packs
         def self.single_pack_select(prompt, question_text: 'Please use space to select a pack')
           packs = Packs.all.to_h { |t| [t.name, t] }
 
-          pack_selection = T.let(prompt.select(
-            question_text,
-            packs,
-            filter: true,
-            per_page: 10,
-            show_help: :always
-          ), T.nilable(Packs::Pack))
+          pack_selection = T.let(
+            prompt.select(
+              question_text,
+              packs,
+              filter: true,
+              per_page: 10,
+              show_help: :always
+            ), T.nilable(Packs::Pack)
+          )
 
           while pack_selection.nil?
             prompt.error(
@@ -31,13 +33,15 @@ module Packs
 
         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,
-            Packs.all.to_h { |t| [t.name, t] },
-            filter: true,
-            per_page: 10,
-            show_help: :always
-          ), T::Array[Packs::Pack])
+          pack_selection = T.let(
+            prompt.multi_select(
+              question_text,
+              Packs.all.to_h { |t| [t.name, t] },
+              filter: true,
+              per_page: 10,
+              show_help: :always
+            ), T::Array[Packs::Pack]
+          )
 
           while pack_selection.empty?
             prompt.error(

data/lib/packs/private/interactive_cli/team_selector.rb

@@ -9,15 +9,17 @@ module Packs
         sig { params(prompt: TTY::Prompt, question_text: String).returns(T.nilable(CodeTeams::Team)) }
         def self.single_select(prompt, question_text: 'Please use space to select a team owner')
           teams = CodeTeams.all.sort_by(&:name).to_h { |t| [t.name, t] }
-          return nil if teams.count == 0
-
-          team_selection = T.let(prompt.select(
-            question_text,
-            teams,
-            filter: true,
-            per_page: 10,
-            show_help: :always
-          ), T.nilable(CodeTeams::Team))
+          return nil if teams.none?
+
+          team_selection = T.let(
+            prompt.select(
+              question_text,
+              teams,
+              filter: true,
+              per_page: 10,
+              show_help: :always
+            ), T.nilable(CodeTeams::Team)
+          )
 
           while team_selection.nil?
             prompt.error(
@@ -34,13 +36,15 @@ module Packs
         def self.multi_select(prompt, question_text: 'Please use space to select team owners')
           teams = CodeTeams.all.to_h { |t| [t.name, t] }
 
-          team_selection = T.let(prompt.multi_select(
-            question_text,
-            teams,
-            filter: true,
-            per_page: 10,
-            show_help: :always
-          ), T::Array[CodeTeams::Team])
+          team_selection = T.let(
+            prompt.multi_select(
+              question_text,
+              teams,
+              filter: true,
+              per_page: 10,
+              show_help: :always
+            ), T::Array[CodeTeams::Team]
+          )
 
           while team_selection.empty?
             prompt.error(

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

@@ -26,11 +26,11 @@ module Packs
               selected_packs = PackSelector.single_or_all_pack_multi_select(prompt, question_text: 'What pack(s) would you like info on?')
             end
 
-            format = prompt.select('What output format do you want?', %w[Detail CSV])
+            format = prompt.select('What output format do you want?', %w(Detail CSV))
 
             types = prompt.multi_select(
               'What violation types do you want stats for?',
-              %w[Privacy Dependency Layer]
+              %w(Privacy Dependency Layer)
             )
 
             include_date = !prompt.no?('Should the current date be included in the report?')

data/lib/packs/private/interactive_cli/use_cases/interface.rb

@@ -23,10 +23,12 @@ module Packs
           end
 
           sig { abstract.params(prompt: TTY::Prompt).void }
-          def perform!(prompt); end
+          def perform!(prompt)
+          end
 
           sig { abstract.returns(String) }
-          def user_facing_name; end
+          def user_facing_name
+          end
         end
       end
     end

data/lib/packs/private/interactive_cli/use_cases/move_pack.rb

@@ -16,7 +16,7 @@ module Packs
               {
                 'Move a child pack to be nested under a parent pack' => :move_to_parent,
                 'Move a pack to a folder that is not a pack' =>
-                 :move_to_folder
+                 :move_to_folder,
               }
             )
 

data/lib/packs/private/interactive_cli/use_cases/query.rb

@@ -30,7 +30,7 @@ module Packs
 
             limit = prompt.ask('Specify the limit of constants to analyze', default: 10, convert: :integer)
 
-            selection = prompt.select('Are you interested in dependency, privacy, or layer violations?', %w[Dependency Privacy Layer], default: 'Privacy')
+            selection = prompt.select('Are you interested in dependency, privacy, or layer violations?', %w(Dependency Privacy Layer), default: 'Privacy')
 
             Packs.list_top_violations(
               type: selection.downcase,

data/lib/packs/private/interactive_cli.rb

@@ -28,16 +28,21 @@ module Packs
 
       sig { params(prompt: T.nilable(TTY::Prompt)).void }
       def self.start!(prompt: nil)
-        prompt ||= TTY::Prompt.new(interrupt: lambda {
-                                              puts "\n\nGoodbye! I hope you have a good day."
-                                              exit 1 })
+        prompt ||= TTY::Prompt.new(
+          interrupt: -> {
+            puts "\n\nGoodbye! I hope you have a good day."
+            exit 1
+          }
+        )
         help_text = '(Press ↑/↓ arrow to move, Enter to select and letters to filter)'
-        choice = prompt.select('Hello! What would you like to do?',
+        choice = prompt.select(
+          'Hello! What would you like to do?',
           cycle: true,
           filter: true,
           help: help_text,
           show_help: :always,
-          per_page: 15) do |menu|
+          per_page: 15
+        ) do |menu|
           menu.enum '.'
 
           UseCases::Interface.all.each do |use_case|

data/lib/packs/private.rb

@@ -22,7 +22,7 @@ module Packs
       # This results in the pack not being found, but when we write the package YML it writes to the same place,
       # causing a behaviorally confusing diff.
       # We ignore trailing slashes as an ergonomic feature to the user.
-      pack_name.gsub(%r{/$}, '')
+      pack_name.delete_suffix('/')
     end
 
     sig do
@@ -37,10 +37,12 @@ module Packs
       return if !file.exist?
 
       count = 0
-      file.write(file.read.gsub(find.to_s) do
-        count += 1
-        replace_with.to_s
-      end)
+      file.write(
+        file.read.gsub(find.to_s) do
+          count += 1
+          replace_with.to_s
+        end
+      )
       Logging.print "Replaced #{count} occurrence(s) of #{find} in #{file}" if count > 0
     end
 
@@ -69,7 +71,7 @@ module Packs
         team: team
       )
       add_public_directory(package) if package.enforce_privacy
-      add_readme_todo(package)
+      add_readme(package)
 
       Logging.section('Next steps') do
         next_steps = Packs.config.user_event_logger.after_create_pack(pack_name)
@@ -131,7 +133,7 @@ module Packs
             )
             [
               file_move_operation,
-              file_move_operation.spec_file_move_operation
+              *file_move_operation.spec_file_move_operations,
             ]
           end
           file_move_operations.each do |file_move_operation|
@@ -140,7 +142,7 @@ module Packs
         end
       end
 
-      add_readme_todo(package)
+      add_readme(package)
 
       per_file_processors.each do |processor|
         processor.after_move_files!(file_move_operations)
@@ -297,8 +299,8 @@ module Packs
         end
 
         if other_package.name == parent_name &&
-           !new_dependencies.include?(new_package_name) &&
-           !new_config['ignored_dependencies']&.include?(new_package_name)
+            !new_dependencies.include?(new_package_name) &&
+            !new_config['ignored_dependencies']&.include?(new_package_name)
           new_dependencies += [new_package_name]
         end
 
@@ -358,7 +360,7 @@ module Packs
 
             [
               file_move_operation,
-              file_move_operation.spec_file_move_operation
+              *file_move_operation.spec_file_move_operations,
             ]
           end
 
@@ -446,18 +448,22 @@ module Packs
 
       if public_directory.glob('**/**.rb').none?
         FileUtils.mkdir_p(public_directory)
-        todo_md = Packs.config.user_event_logger.on_create_public_directory_todo(package.name)
-        public_directory.join('TODO.md').write(todo_md)
+        package_name = package.directory.basename.to_s
+        FileUtils.mkdir_p(public_directory.join(package_name))
       end
     end
 
     sig { params(package: ParsePackwerk::Package).void }
-    def self.add_readme_todo(package)
+    def self.add_readme(package)
       pack_directory = package.directory
 
       if !pack_directory.join('README.md').exist?
-        readme_todo_md = Packs.config.user_event_logger.on_create_readme_todo(package.name)
-        pack_directory.join('README_TODO.md').write(readme_todo_md)
+        readme_md = Packs.config.user_event_logger.on_create_readme(package.name)
+        pack_directory.join('README.md').write(readme_md)
+
+        if pack_directory.join('README_TODO.md').exist?
+          pack_directory.join('README_TODO.md').delete
+        end
       end
     end
 
@@ -484,7 +490,7 @@ module Packs
         # but we'll need to add an API to CodeOwnership to do this
         if Pathname.new('config/code_ownership.yml').exist?
           config = {
-            'owner' => team.nil? ? 'MyTeam' : team.name
+            'owner' => team.nil? ? 'MyTeam' : team.name,
           }
         else
           config = {}
@@ -526,7 +532,7 @@ module Packs
     def self.bust_cache!
       Packs.config.bust_cache!
       # This comes explicitly after `Packs.config.bust_cache!` because
-      # otherwise `Packs.config` will attempt to reload the client configuratoin.
+      # otherwise `Packs.config` will attempt to reload the client configuration.
       @loaded_client_configuration = false
     end
 
@@ -555,7 +561,7 @@ module Packs
         write_package_todo_to_tmp_folder(before, dir_containing_contents_before)
         write_package_todo_to_tmp_folder(after, dir_containing_contents_after)
 
-        diff = `diff -r #{dir_containing_contents_before}/ #{dir_containing_contents_after}/`
+        diff = %x(diff -r #{dir_containing_contents_before}/ #{dir_containing_contents_after}/)
         # For ease of reading, sub out the tmp directory from the diff
         diff.gsub(dir_containing_contents_before, '').gsub(dir_containing_contents_after, '')
       ensure
@@ -600,13 +606,13 @@ module Packs
         include_date: T::Boolean
       ).void
     end
-    def self.get_info(packs: Packs.all, format: :detail, types: %i[privacy dependency layer], include_date: false)
+    def self.get_info(packs: Packs.all, format: :detail, types: %i(privacy dependency layer), include_date: false)
       require 'csv' if format == :csv
 
       today = Date.today.iso8601
       violations = {
         inbound: {},
-        outbound: {}
+        outbound: {},
       }
 
       package_by_name = {}
@@ -623,7 +629,7 @@ module Packs
 
       all = {
         inbound: T.let([], T::Array[ParsePackwerk::Violation]),
-        outbound: T.let([], T::Array[ParsePackwerk::Violation])
+        outbound: T.let([], T::Array[ParsePackwerk::Violation]),
       }
       packs.each do |pack|
         all[:inbound] += violations[:inbound][pack.name] || []
@@ -657,7 +663,7 @@ module Packs
           pack_name: pack.name,
           owner: owner.nil? ? 'No one' : owner.name,
           size: pack.relative_path.glob('**/*.rb').count,
-          public_api: pack.relative_path.join(package_by_name[pack.name].public_path)
+          public_api: pack.relative_path.join(package_by_name[pack.name].public_path),
         }
 
         row.delete(:date) unless include_date

data/lib/packs/update_references_post_processor.rb

@@ -19,7 +19,7 @@ module Packs
       destination_pack = T.must(file_move_operations.first).destination_pack.name
 
       if self.class.ripgrep_enabled?
-        matching_files = `rg -l --hidden '#{origin_pack}' .`
+        matching_files = %x(rg -l --hidden '#{origin_pack}' .)
         matching_files.split("\n").each do |file_name|
           substitute_references!(file_name, origin_pack, destination_pack)
         end

data/lib/packs/user_event_logger.rb

@@ -124,30 +124,7 @@ module Packs
     end
 
     sig { params(pack_name: String).returns(String) }
-    def on_create_public_directory_todo(pack_name)
-      <<~MSG
-        This directory holds your public API!
-
-        Any classes, constants, or modules that you want other packs to use and you intend to support should go in here.
-        Anything that is considered private should go in other folders.
-
-        If another pack uses classes, constants, or modules that are not in your public folder, it will be considered a "privacy violation" by packwerk.
-        You can prevent other packs from using private API by using packwerk.
-
-        Want to find how your private API is being used today?
-        Try running: `bin/packs list_top_violations privacy #{pack_name}`
-
-        Want to move something into this folder?
-        Try running: `bin/packs make_public #{pack_name}/path/to/file.rb`
-
-        One more thing -- feel free to delete this file and replace it with a README.md describing your package in the main package directory.
-
-        See #{documentation_link} for more info!
-      MSG
-    end
-
-    sig { params(pack_name: String).returns(String) }
-    def on_create_readme_todo(pack_name)
+    def on_create_readme(pack_name)
       readme_template_pathname = Packs.config.readme_template_pathname
       readme_template = readme_template_pathname.read if readme_template_pathname.exist?
 

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: packs
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Gusto Engineers
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-12-18 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bigdecimal
@@ -140,16 +139,16 @@ dependencies:
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '2.2'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '2.2'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: pry
   requirement: !ruby/object:Gem::Requirement
@@ -220,6 +219,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop-gusto
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: sorbet
   requirement: !ruby/object:Gem::Requirement
@@ -252,16 +265,16 @@ dependencies:
   name: spoom
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 1.2.1
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 1.2.1
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: tapioca
   requirement: !ruby/object:Gem::Requirement
@@ -329,7 +342,6 @@ metadata:
   source_code_uri: https://github.com/rubyatscale/packs
   changelog_uri: https://github.com/rubyatscale/packs/releases
   allowed_push_host: https://rubygems.org
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -337,15 +349,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.6.0
+      version: '3.2'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.22
-signing_key:
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Provides CLI tools for working with ruby packs.
 test_files: []