packs 0.0.23 → 0.0.25

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b263107f82d4798835847954ad1e85d0e62312ca9a8064ebf5ba789f522e3b3f
-  data.tar.gz: 90613f4569258bbfce8bcf2ceab4ac1038fc7a05ba2610376253ac7c8bd6e264
+  metadata.gz: 6d0b7482840c04ba379ace0bb6b3af72257dee2c3538ed49212b11bc45821b53
+  data.tar.gz: a0b35ed0dd622f22000a1e472cffa1c3d2b5e87479af1fa2bef2686454203fa9
 SHA512:
-  metadata.gz: 7173bc07d22465d0dc96041adc9e22e704429b1b0cb0b480240e2af8307bf7a47fc43a94e00768903dc588db89f5cb4445d3198de59a0463447b53cf02010716
-  data.tar.gz: 3cbfbc39d85ffd72e0c9a595a9dbeb60596b60b55cf909188bbbfc15aa7a44c59a098beda1916773f7cc16334a43d6a029b479dbc6c9bdc789a45f0d4e90ab84
+  metadata.gz: 4ef79ae43f292f79c70aa3324723a0f3ae3c307207f6f133a4ed3a7caea9b861a18d9dacb96b7c8603582e481f8193eadf71a6aa4d1e28a34fbd7ed20080f110
+  data.tar.gz: 5c7c2be4ba96f3c8b3bd0bab435735414299e27b1748f31f11427f829c7ee010a9a035872b25a0e6c38259dd5680353e3b1c57b626d5249350a1f63aa1dbff97

data/lib/packs/cli.rb

@@ -10,6 +10,7 @@ module Packs
     sig { params(pack_name: String).void }
     def create(pack_name)
       Packs.create_pack!(pack_name: pack_name)
+      exit_successfully
     end
 
     desc 'add_dependency packs/from_pack packs/to_pack', 'Add packs/to_pack to packs/from_pack/package.yml list of dependencies'
@@ -27,6 +28,7 @@ module Packs
         pack_name: from_pack,
         dependency_name: to_pack
       )
+      exit_successfully
     end
 
     desc 'list_top_dependency_violations packs/your_pack', 'List the top dependency violations of packs/your_pack'
@@ -44,6 +46,7 @@ module Packs
         pack_name: pack_name,
         limit: options[:limit]
       )
+      exit_successfully
     end
 
     desc 'list_top_privacy_violations packs/your_pack', 'List the top privacy violations of packs/your_pack'
@@ -61,6 +64,7 @@ module Packs
         pack_name: pack_name,
         limit: options[:limit]
       )
+      exit_successfully
     end
 
     desc 'make_public path/to/file.rb path/to/directory', 'Make files or directories public API'
@@ -75,6 +79,7 @@ module Packs
         paths_relative_to_root: paths,
         per_file_processors: [Packs::RubocopPostProcessor.new, Packs::CodeOwnershipPostProcessor.new]
       )
+      exit_successfully
     end
 
     desc 'move packs/destination_pack path/to/file.rb path/to/directory', 'Move files or directories from one pack to another'
@@ -91,6 +96,7 @@ module Packs
         paths_relative_to_root: paths,
         per_file_processors: [Packs::RubocopPostProcessor.new, Packs::CodeOwnershipPostProcessor.new]
       )
+      exit_successfully
     end
 
     desc 'lint_package_todo_yml_files', 'Lint `package_todo.yml` files to check for formatting issues'
@@ -108,37 +114,40 @@ module Packs
     desc 'validate', 'Run bin/packwerk validate (detects cycles)'
     sig { void }
     def validate
-      system('bin/packwerk validate')
+      Private.exit_with(Packs.validate)
     end
 
     desc 'check [ packs/my_pack ]', 'Run bin/packwerk check'
     sig { params(paths: String).void }
     def check(*paths)
-      Packs.execute(['check', *paths])
+      Private.exit_with(Packs.check(paths))
     end
 
     desc 'update', 'Run bin/packwerk update-todo'
     sig { void }
     def update
-      system('bin/packwerk update-todo')
+      Private.exit_with(Packs.update)
     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))
+      exit_successfully
     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))
+      exit_successfully
     end
 
     desc 'rename', 'Rename a pack'
     sig { void }
     def rename
       puts Private.rename_pack
+      exit_successfully
     end
 
     desc 'move_to_parent packs/child_pack packs/parent_pack ', 'Set packs/child_pack as a child of packs/parent_pack'
@@ -149,6 +158,7 @@ module Packs
         pack_name: child_pack_name,
         per_file_processors: [Packs::RubocopPostProcessor.new, Packs::CodeOwnershipPostProcessor.new]
       )
+      exit_successfully
     end
 
     private
@@ -159,6 +169,11 @@ module Packs
       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
     end
   end
 end

data/lib/packs/logging.rb

@@ -1,6 +1,6 @@
 # typed: strict
 
-require 'colorized_string'
+require 'rainbow'
 
 module Packs
   module Logging
@@ -9,14 +9,14 @@ module Packs
     sig { params(title: String, block: T.proc.void).void }
     def self.section(title, &block)
       print_divider
-      out ColorizedString.new(title).green.bold
+      out Rainbow(title).green.bold
       out "\n"
       yield
     end
 
     sig { params(text: String).void }
     def self.print_bold_green(text)
-      out ColorizedString.new(text).green.bold
+      out Rainbow(text).green.bold
     end
 
     sig { params(text: String).void }

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

@@ -16,7 +16,7 @@ module Packs
 
           sig { override.params(prompt: TTY::Prompt).void }
           def perform!(prompt)
-            system('bin/packwerk check')
+            Private.exit_with(Packs.check([]))
           end
         end
       end

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

@@ -10,9 +10,9 @@ module Packs
 
           interface!
 
-          sig { params(base: Class).void }
+          sig { params(base: T::Class[T.anything]).void }
           def self.included(base)
-            @use_cases ||= T.let(@use_cases, T.nilable(T::Array[Class]))
+            @use_cases ||= T.let(@use_cases, T.nilable(T::Array[T::Class[T.anything]]))
             @use_cases ||= []
             @use_cases << base
           end

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

@@ -16,7 +16,7 @@ module Packs
 
           sig { override.params(prompt: TTY::Prompt).void }
           def perform!(prompt)
-            system('bin/packwerk update')
+            Private.exit_with(Packs.update)
           end
         end
       end

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

@@ -16,7 +16,7 @@ module Packs
 
           sig { override.params(prompt: TTY::Prompt).void }
           def perform!(prompt)
-            system('bin/packwerk validate')
+            Private.exit_with(Packs.validate)
           end
         end
       end

data/lib/packs/private.rb

@@ -2,13 +2,12 @@
 
 require 'pathname'
 require 'fileutils'
-require 'colorized_string'
+require 'rainbow'
 require 'sorbet-runtime'
 
 require 'packs/private/file_move_operation'
 require 'packs/private/pack_relationship_analyzer'
 require 'packs/private/interactive_cli'
-require 'packs/private/packwerk_wrapper'
 
 module Packs
   module Private
@@ -295,7 +294,7 @@ module Packs
         config: package.config
       )
       ParsePackwerk.write_package_yml!(new_package)
-      PackwerkWrapper.validate!
+      Packs.validate
     end
 
     sig { params(file_move_operation: FileMoveOperation, per_file_processors: T::Array[Packs::PerFileProcessorInterface]).void }
@@ -314,7 +313,7 @@ module Packs
     sig { params(origin: Pathname, destination: Pathname).void }
     def self.idempotent_mv(origin, destination)
       if origin.exist? && destination.exist?
-        Logging.print ColorizedString.new("[SKIP] Not moving #{origin}, #{destination} already exists").red
+        Logging.print Rainbow("[SKIP] Not moving #{origin}, #{destination} already exists").red
       elsif origin.exist? && !destination.exist?
         destination.dirname.mkpath
 
@@ -322,11 +321,11 @@ module Packs
         # use git mv so that git knows that it was a move
         FileUtils.mv(origin, destination)
       elsif !origin.exist? && destination.exist?
-        Logging.print ColorizedString.new("[SKIP] Not moving #{origin}, does not exist, (#{destination} already exists)").red
+        Logging.print Rainbow("[SKIP] Not moving #{origin}, does not exist, (#{destination} already exists)").red
       else
         # We could choose to print this in a `--verbose` mode. For now, we find that printing this text in red confuses folks more than it informs them.
         # This is because it's perfectly common for a spec to not exist for a file, so at best it's a warning.
-        # Logging.print ColorizedString.new("[SKIP] Not moving #{origin}, does not exist").red
+        # Logging.print Rainbow("[SKIP] Not moving #{origin}, does not exist").red
       end
     end
 
@@ -523,13 +522,13 @@ module Packs
     sig { void }
     def self.lint_package_todo_yml_files!
       contents_before = Private.get_package_todo_contents
-      Packs.execute(['update-todo'])
+      Packs.update
       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`
-        safe_exit 0
+        exit_with true
       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`.
@@ -551,7 +550,7 @@ module Packs
 
         puts output
         Packs.config.on_package_todo_lint_failure.call(output)
-        safe_exit 1
+        exit_with false
       end
     end
 
@@ -608,10 +607,16 @@ module Packs
     end
 
     # This function exists to give us something to stub in test
-    sig { params(code: Integer).void }
-    def self.safe_exit(code)
+    sig { params(code: T::Boolean).void }
+    def self.exit_with(code)
       exit code
     end
+
+    # This function exists to give us something to stub in test
+    sig { params(command: String).returns(T::Boolean) }
+    def self.system_with(command)
+      T.cast(system(command), T::Boolean)
+    end
   end
 
   private_constant :Private

data/lib/packs.rb

@@ -4,7 +4,7 @@
 require 'fileutils'
 
 # External gem requires
-require 'colorized_string'
+require 'rainbow'
 
 # Internal gem requires
 require 'packs-specification'
@@ -36,6 +36,21 @@ module Packs
     Private::InteractiveCli.start!
   end
 
+  sig { returns(T::Boolean) }
+  def self.update
+    Private.system_with('bin/packwerk update-todo')
+  end
+
+  sig { returns(T::Boolean) }
+  def self.validate
+    Private.system_with('bin/packwerk validate')
+  end
+
+  sig { params(files: T::Array[String]).returns(T::Boolean) }
+  def self.check(files)
+    Private.system_with("bin/packwerk check #{files.join(' ')}")
+  end
+
   sig do
     params(
       pack_name: String,
@@ -222,32 +237,6 @@ module Packs
     Specification.bust_cache!
   end
 
-  #
-  # execute_command is like `run` except it does not `exit`
-  #
-  sig { params(argv: T.untyped, formatter: T.nilable(Packwerk::OffensesFormatter)).void }
-  def self.execute(argv, formatter = nil)
-    Private::PackwerkWrapper.with_safe_exit_if_no_files_found do
-      Private::PackwerkWrapper.packwerk_cli(formatter).execute_command(argv)
-    end
-  end
-
-  sig { params(files: T::Array[String]).returns(T::Array[Packwerk::ReferenceOffense]) }
-  def self.get_offenses_for_files(files)
-    formatter = Private::PackwerkWrapper::OffensesAggregatorFormatter.new
-    Private::PackwerkWrapper.packwerk_cli_execute_safely(['check', *files], formatter)
-    formatter.aggregated_offenses.compact
-  end
-
-  sig { params(files: T::Array[String]).returns(T::Array[Packwerk::ReferenceOffense]) }
-  def self.get_offenses_for_files_by_package(files)
-    packages = Private::PackwerkWrapper.package_names_for_files(files)
-    argv = ['check', '--packages', packages.join(',')]
-    formatter = Private::PackwerkWrapper::OffensesAggregatorFormatter.new
-    Private::PackwerkWrapper.packwerk_cli_execute_safely(argv, formatter)
-    formatter.aggregated_offenses.compact
-  end
-
   sig { void }
   def self.lint_package_todo_yml_files!
     Private.lint_package_todo_yml_files!

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: packs
 version: !ruby/object:Gem::Version
-  version: 0.0.23
+  version: 0.0.25
 platform: ruby
 authors:
 - Gusto Engineers
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-08-10 00:00:00.000000000 Z
+date: 2023-08-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: code_ownership
@@ -25,7 +25,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 1.33.0
 - !ruby/object:Gem::Dependency
-  name: colorize
+  name: packs-specification
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -39,7 +39,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: packs-specification
+  name: packwerk
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -53,7 +53,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: packwerk
+  name: parse_packwerk
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -67,7 +67,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: parse_packwerk
+  name: rainbow
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -276,6 +276,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: spoom
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.2.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.2.1
 - !ruby/object:Gem::Dependency
   name: tapioca
   requirement: !ruby/object:Gem::Requirement
@@ -331,7 +345,6 @@ files:
 - lib/packs/private/interactive_cli/use_cases/validate.rb
 - lib/packs/private/interactive_cli/use_cases/visualize.rb
 - lib/packs/private/pack_relationship_analyzer.rb
-- lib/packs/private/packwerk_wrapper.rb
 - lib/packs/private/packwerk_wrapper/offenses_aggregator_formatter.rb
 - lib/packs/rubocop_post_processor.rb
 - lib/packs/user_event_logger.rb

data/lib/packs/private/packwerk_wrapper.rb

@@ -1,70 +0,0 @@
-# typed: strict
-
-require 'packwerk'
-require 'packs/private/packwerk_wrapper/offenses_aggregator_formatter'
-
-module Packs
-  module Private
-    module PackwerkWrapper
-      extend T::Sig
-
-      #
-      # execute_command is like `run` except it does not `exit`
-      #
-      sig { params(argv: T.untyped, formatter: T.nilable(Packwerk::OffensesFormatter)).void }
-      def self.packwerk_cli_execute_safely(argv, formatter = nil)
-        with_safe_exit_if_no_files_found do
-          packwerk_cli(formatter).execute_command(argv)
-        end
-      end
-
-      sig { params(block: T.proc.returns(T.untyped)).void }
-      def self.with_safe_exit_if_no_files_found(&block)
-        block.call
-      rescue SystemExit => e
-        # Packwerk should probably exit positively here rather than raising an error -- there should be no
-        # errors if the user has excluded all files being checked.
-        unless e.message == 'No files found or given. Specify files or check the include and exclude glob in the config file.'
-          raise
-        end
-      end
-
-      sig { params(formatter: T.nilable(Packwerk::OffensesFormatter)).returns(Packwerk::Cli) }
-      def self.packwerk_cli(formatter)
-        # This is mostly copied from exe/packwerk within the packwerk gem, but we use our own formatters
-        # Note that packwerk does not allow you to pass in your own progress formatter currently
-        ENV['RAILS_ENV'] = 'test'
-
-        style = Packwerk::OutputStyles::Coloured.new
-        Packwerk::Cli.new(style: style, offenses_formatter: formatter)
-      end
-
-      sig { params(files: T::Array[String]).returns(T::Array[Packwerk::ReferenceOffense]) }
-      def self.get_offenses_for_files(files)
-        formatter = OffensesAggregatorFormatter.new
-        packwerk_cli_execute_safely(['check', *files], formatter)
-        formatter.aggregated_offenses.compact
-      end
-
-      sig { void }
-      def self.validate!
-        formatter = OffensesAggregatorFormatter.new
-        packwerk_cli_execute_safely(['validate'], formatter)
-      end
-
-      sig { params(files: T::Array[String]).returns(T::Array[Packwerk::ReferenceOffense]) }
-      def self.get_offenses_for_files_by_package(files)
-        packages = package_names_for_files(files)
-        argv = ['check', '--packages', packages.join(',')]
-        formatter = OffensesAggregatorFormatter.new
-        packwerk_cli_execute_safely(argv, formatter)
-        formatter.aggregated_offenses.compact
-      end
-
-      sig { params(files: T::Array[String]).returns(T::Array[String]) }
-      def self.package_names_for_files(files)
-        files.map { |f| ParsePackwerk.package_from_path(f).name }.compact.uniq
-      end
-    end
-  end
-end