packwerk-extensions 0.0.6 → 0.0.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3f20892b9d7d67a2164c7c632ac760c66859cacd2fa7af97c71a220a1d317893
-  data.tar.gz: a9a7f631ed8e6398a48e91fddc83fcff0e1d923aa16b75231e0a3296c5c4aa4a
+  metadata.gz: 558107be67a748dfb9a175b9d90be2acc373913f36a66c5bccbb3401e80cd85f
+  data.tar.gz: fb98b8c64bb3043d267aab3d966d404934a5a530020dc18e3e599a13153fe121
 SHA512:
-  metadata.gz: f66866b77e2727898fa951a05f716f8b959444553082f9c8999144013ee74d90820ee802a994b11887c3132799b47cbcc74a60cc6434a6172617ebb916748216
-  data.tar.gz: b6deec3d175813666076ce5797d29cc62df65ec233c552114cdf3492c92dd74791b53412201a8b3e56758fdb4bb9a6c7bed804225c99c192d92f9566a60a4a85
+  metadata.gz: e6beb350fd3089406f7a0ec0409b79f261136b89ffe0ede267215893ca7abb9f5613a7aaf44248f1c0b53560a0596a78739568f85bd2a081edf4b3d0f69fe0d0
+  data.tar.gz: 706bc3779b3af6ce3d6780ba9106a8bb4fc84cd81ddd93c7809b62f2663de1baf4c906f961abc6a4982c8815142478d839054b27c504639e1eb5dc88e0b267e3

data/lib/packwerk/architecture/checker.rb

@@ -41,8 +41,8 @@ module Packwerk
           .returns(T::Boolean)
       end
       def invalid_reference?(reference)
-        constant_package = Package.from(reference.constant.package)
-        referencing_package = Package.from(reference.package)
+        constant_package = Package.from(reference.constant.package, layers)
+        referencing_package = Package.from(reference.package, layers)
         !referencing_package.can_depend_on?(constant_package, layers: layers)
       end
 
@@ -62,8 +62,8 @@ module Packwerk
           .returns(String)
       end
       def message(reference)
-        constant_package = Package.from(reference.constant.package)
-        referencing_package = Package.from(reference.package)
+        constant_package = Package.from(reference.constant.package, layers)
+        referencing_package = Package.from(reference.package, layers)
 
         message = <<~MESSAGE
           Architecture layer violation: '#{reference.constant.name}' belongs to '#{reference.constant.package}', whose architecture layer type is "#{constant_package.layer}."

data/lib/packwerk/architecture/layers.rb

@@ -8,12 +8,13 @@ module Packwerk
 
       sig { void }
       def initialize
-        @names = T.let(@names, T.nilable(T::Array[String]))
+        @names = T.let(@names, T.nilable(T::Set[String]))
+        @names_list = T.let(@names_list, T.nilable(T::Array[String]))
       end
 
       sig { params(layer: String).returns(Integer) }
       def index_of(layer)
-        index = names.reverse.find_index(layer)
+        index = names_list.reverse.find_index(layer)
         if index.nil?
           raise "Layer #{layer} not find, please run `bin/packwerk validate`"
         end
@@ -21,9 +22,16 @@ module Packwerk
         index
       end
 
-      sig { returns(T::Array[String]) }
+      sig { returns(T::Set[String]) }
       def names
-        @names ||= YAML.load_file('packwerk.yml')['architecture_layers'] || []
+        @names ||= Set.new(names_list)
+      end
+
+      private
+
+      sig { returns(T::Array[String]) }
+      def names_list
+        @names_list ||= YAML.load_file('packwerk.yml')['architecture_layers'] || []
       end
     end
   end

data/lib/packwerk/architecture/package.rb

@@ -30,15 +30,20 @@ module Packwerk
       class << self
         extend T::Sig
 
-        sig { params(package: ::Packwerk::Package).returns(Package) }
-        def from(package)
-          from_config(package.config)
-        end
+        sig { params(package: ::Packwerk::Package, layers: Layers).returns(Package) }
+        def from(package, layers)
+          config = package.config
+
+          # This allows the layer to be inferred based on the package root
+          package_root = package.name.split('/').first
+          if package_root && layers.names.include?(package_root)
+            layer = package_root
+          else
+            layer = config['layer']
+          end
 
-        sig { params(config: T::Hash[T.untyped, T.untyped]).returns(Package) }
-        def from_config(config)
           Package.new(
-            layer: config['layer'],
+            layer: layer,
             enforcement_setting: config['enforce_architecture'],
             config: config
           )

data/lib/packwerk/architecture/validator.rb

@@ -13,8 +13,9 @@ module Packwerk
       def call(package_set, configuration)
         results = T.let([], T::Array[Result])
 
-        package_manifests(configuration).each do |f|
-          config = YAML.load_file(File.join(f))
+        package_set.each do |package|
+          config = package.config
+          f = Pathname.new(package.name).join('package.yml').to_s
           next if !config
 
           result = check_enforce_architecture_setting(f, config['enforce_architecture'])
@@ -25,7 +26,7 @@ module Packwerk
           results << result
           next if !result.ok?
 
-          package = Package.from_config(config)
+          package = Package.from(package, layers)
           results += check_dependencies_setting(package_set, package, f)
         end
 
@@ -51,7 +52,7 @@ module Packwerk
           other_packwerk_package = package_set.fetch(dependency)
           next if other_packwerk_package.nil?
 
-          other_package = Package.from(other_packwerk_package)
+          other_package = Package.from(other_packwerk_package, layers)
           next if package.can_depend_on?(other_package, layers: layers)
 
           results << Result.new(
@@ -81,7 +82,7 @@ module Packwerk
         else
           Result.new(
             ok: false,
-            error_value: "Invalid 'layer' option in #{config_file_path.inspect}: #{layer.inspect}. Must be one of #{layers.names.inspect}"
+            error_value: "Invalid 'layer' option in #{config_file_path.inspect}: #{layer.inspect}. Must be one of #{layers.names.to_a.inspect}"
           )
         end
       end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: packwerk-extensions
 version: !ruby/object:Gem::Version
-  version: 0.0.6
+  version: 0.0.7
 platform: ruby
 authors:
 - Gusto Engineers
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-12-21 00:00:00.000000000 Z
+date: 2022-12-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: packwerk