parse_packwerk 0.20.0 → 0.21.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 54e72896efeae11785e34d5c4445255a7418499095d6f8f10443047437e49ad4
-  data.tar.gz: e722da109f4ef007f2b017d2fdbc4aba4cc6b09d94d8efb031cb7f4f7a1dae40
+  metadata.gz: 7665b37cbab9d74a91785449408830963feeef02bd1816827c54c39f3504228c
+  data.tar.gz: e80b45a3f6c801aa5c17e7951030bbb8f976632a8c9e4f799cbde23e1ed34988
 SHA512:
-  metadata.gz: 993a09365ac56439efba8c66c0892275a184969be156e9b33cd7e8d218ab00222f55f7488d888348bb73805b91b7b3a223d46e58236695c567628a175fbe4508
-  data.tar.gz: 7d7b919458f16a8c53038405a3e777552a92e1660a23312a288d4b0653ba59648314fd861be63b4c27f89b50975cb4bf24756538e40cadfbe803e5160c72f5d2
+  metadata.gz: 6e2ee542e063f014e7e278f089742f9318b4569208e88acd8b3eed9b7358f1000de57e4a812ec8d52c75e11ab8bf5af961ae58f2b4ccf6b09cf28f6a6dbae248
+  data.tar.gz: 1bf86e70a1ad52ef21ddde492788ab72d90c479baca3dec887c774c45c14971a4b12fe07e58b383fd229a16ab941a9f8a2c68ddebcdfde811c634c6ce193b502

data/lib/parse_packwerk/configuration.rb

@@ -12,12 +12,12 @@ module ParsePackwerk
     sig { returns(Configuration) }
     def self.fetch
       packwerk_yml_filename = Pathname.new(PACKWERK_YML_NAME)
-      if !File.exist?(packwerk_yml_filename)
-        raw_packwerk_config = {}
-      else
+      if File.exist?(packwerk_yml_filename)
         # when the YML file is empty or only contains comment, it gets parsed
         # as the boolean `false` for some reason. this handles that case
         raw_packwerk_config = YAML.load_file(packwerk_yml_filename) || {}
+      else
+        raw_packwerk_config = {}
       end
 
       Configuration.new(
@@ -32,22 +32,22 @@ module ParsePackwerk
     def self.excludes(config_hash)
       specified_exclude = config_hash['exclude']
       excludes = if specified_exclude.nil?
-        DEFAULT_EXCLUDE_GLOBS.dup
-      else
-        Array(specified_exclude)
-      end
+                   DEFAULT_EXCLUDE_GLOBS.dup
+                 else
+                   Array(specified_exclude)
+                 end
 
-      excludes.push Bundler.bundle_path.join("**").to_s
+      excludes.push Bundler.bundle_path.join('**').to_s
     end
 
     sig { params(config_hash: T::Hash[T.untyped, T.untyped]).returns(T::Array[String]) }
     def self.package_paths(config_hash)
       specified_package_paths = config_hash['package_paths']
       package_paths = if specified_package_paths.nil?
-        DEFAULT_PACKAGE_PATHS.dup
-      else
-        Array(specified_package_paths)
-      end
+                        DEFAULT_PACKAGE_PATHS.dup
+                      else
+                        Array(specified_package_paths)
+                      end
 
       # We add the root package path always
       package_paths.push '.'

data/lib/parse_packwerk/constants.rb

@@ -19,7 +19,7 @@ module ParsePackwerk
     T::Hash[T.untyped, T.untyped]
   end
 
-  DEFAULT_EXCLUDE_GLOBS = T.let(["{bin,node_modules,script,tmp,vendor}/**/*"], T::Array[String])
+  DEFAULT_EXCLUDE_GLOBS = T.let(['{bin,node_modules,script,tmp,vendor}/**/*'], T::Array[String])
   DEFAULT_PACKAGE_PATHS = T.let(['**/'], T::Array[String])
   DEFAULT_PUBLIC_PATH = T.let('app/public', String)
 end

data/lib/parse_packwerk/package.rb

@@ -5,20 +5,20 @@ module ParsePackwerk
     extend T::Sig
 
     const :name, String
-    const :enforce_dependencies, T.any(T::Boolean, String)
+    const :enforce_dependencies, T.nilable(T.any(T::Boolean, String))
     const :enforce_privacy, T.any(T::Boolean, String), default: false
     const :public_path, String, default: DEFAULT_PUBLIC_PATH
     const :metadata, MetadataYmlType
     const :dependencies, T::Array[String]
     const :config, T::Hash[T.untyped, T.untyped]
-    const :violations, T.nilable(T::Array[Violation])
+    const :violations, T::Array[Violation]
 
     sig { params(pathname: Pathname).returns(Package) }
     def self.from(pathname)
       package_loaded_yml = YAML.load_file(pathname)
       if package_loaded_yml.nil? || package_loaded_yml == false
         message = "Failed to parse `#{pathname}`. Please fix any issues with this package.yml OR add its containing folder to packwerk.yml `exclude`"
-        raise PackageParseError.new(message)
+        raise PackageParseError, message
       end
       package_name = pathname.dirname.cleanpath.to_s
 
@@ -46,7 +46,7 @@ module ParsePackwerk
 
     sig { returns(Pathname) }
     def directory
-      self.class.directory(self.name)
+      self.class.directory(name)
     end
 
     sig { returns(Pathname) }
@@ -54,7 +54,7 @@ module ParsePackwerk
       directory.join(public_path).cleanpath
     end
 
-    sig { returns(T.any(T::Boolean, String)) }
+    sig { returns(T.nilable(T.any(T::Boolean, String))) }
     def enforces_dependencies?
       enforce_dependencies
     end

data/lib/parse_packwerk/package_set.rb

@@ -1,7 +1,7 @@
 # typed: strict
 # frozen_string_literal: true
 
-require "pathname"
+require 'pathname'
 require 'bundler'
 
 module ParsePackwerk

data/lib/parse_packwerk/package_todo.rb

@@ -9,17 +9,12 @@ module ParsePackwerk
 
     sig { params(package: Package).returns(PackageTodo) }
     def self.for(package)
-      PackageTodo.from(self.yml(package.directory))
+      PackageTodo.from(yml(package.directory))
     end
 
     sig { params(pathname: Pathname).returns(PackageTodo) }
     def self.from(pathname)
-      if !pathname.exist?
-        new(
-          pathname: pathname.cleanpath,
-          violations: []
-        )
-      else
+      if pathname.exist?
         package_todo_loaded_yml = YAML.load_file(pathname)
 
         all_violations = []
@@ -39,9 +34,14 @@ module ParsePackwerk
           pathname: pathname.cleanpath,
           violations: all_violations
         )
+      else
+        new(
+          pathname: pathname.cleanpath,
+          violations: []
+        )
       end
     end
-    
+
     sig { params(dirname: Pathname).returns(Pathname) }
     def self.yml(dirname)
       dirname.join(PACKAGE_TODO_YML_NAME).cleanpath

data/lib/parse_packwerk.rb

@@ -62,11 +62,11 @@ module ParsePackwerk
   sig { params(package: ParsePackwerk::Package).void }
   def self.write_package_yml!(package)
     FileUtils.mkdir_p(package.directory)
-    
+
     File.open(package.yml, 'w') do |file|
       merged_config = package.config
 
-      merged_config.merge!('enforce_dependencies' => package.enforce_dependencies,)
+      merged_config.merge!('enforce_dependencies' => package.enforce_dependencies)
 
       if Extensions.privacy_extension_installed?
         merged_config.merge!('enforce_privacy' => package.enforce_privacy)
@@ -87,11 +87,11 @@ module ParsePackwerk
       end
 
       sorted_keys = key_sort_order
-      merged_config = merged_config.to_a.sort_by{|key, value| T.unsafe(sorted_keys).index(key) || 1000 }.to_h
+      merged_config = merged_config.to_a.sort_by { |key, _value| T.unsafe(sorted_keys).index(key) || 1000 }.to_h
 
       raw_yaml = YAML.dump(merged_config)
       # Add indentation for dependencies
-      raw_yaml.gsub!(/^- /,"  - ")
+      raw_yaml.gsub!(/^- /, '  - ')
       stylized_yaml = raw_yaml.gsub("---\n", '')
       file.write(stylized_yaml)
     end
@@ -99,7 +99,7 @@ module ParsePackwerk
 
   sig { returns(T::Array[String]) }
   def self.key_sort_order
-    %w(
+    %w[
       enforce_dependencies
       enforce_privacy
       enforce_visibility
@@ -111,7 +111,7 @@ module ParsePackwerk
       ignored_dependencies
       visible_to
       metadata
-    )
+    ]
   end
 
   # We memoize packages_by_name for fast lookup.
@@ -124,7 +124,7 @@ module ParsePackwerk
       # We want to match more specific paths first
       # Packwerk does this too and is necessary for package_from_path to work correctly.
       sorted_packages = all_packages.sort_by { |package| -package.name.length }
-      sorted_packages.map{|p| [p.name, p]}.to_h
+      sorted_packages.to_h { |p| [p.name, p] }
     end
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: parse_packwerk
 version: !ruby/object:Gem::Version
-  version: 0.20.0
+  version: 0.21.0
 platform: ruby
 authors:
 - Gusto Engineers
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-08-16 00:00:00.000000000 Z
+date: 2023-12-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sorbet-runtime
@@ -24,6 +24,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: awesome_print
+  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: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -39,7 +53,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '2.2'
 - !ruby/object:Gem::Dependency
-  name: pry
+  name: hashdiff
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -53,7 +67,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: pry
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -67,35 +81,35 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: sorbet
+  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.0'
 - !ruby/object:Gem::Dependency
-  name: tapioca
+  name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -109,7 +123,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: hashdiff
+  name: sorbet
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -123,7 +137,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: awesome_print
+  name: tapioca
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="