packwerk-extensions 0.0.9 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a4cd61088407945a3e4f58a0a5a3959759683e9186646a4fca4959b79bcbb9b3
-  data.tar.gz: 855d5ce12ce1d3dba10f06e67e8c25e519d46073fc7eaaa1df67a9fc2777fa09
+  metadata.gz: 9f3568d8998c39960fd6e4233ea768c315a1f08731805eb3db487fb7409793e7
+  data.tar.gz: 76aa9b02df3ecced40da162a43339c0672ec7290ba8eed35b053958573606863
 SHA512:
-  metadata.gz: 7efea5e86a203158a431c43acb5bf422257e5d9d1ec4ddddc13000b733ca8f000c713dbe72bf30cf7a735c5292a736de2570909cd7cc96182e61cc4af8c92804
-  data.tar.gz: a265c58f6976133334b818847154d5fe3663144c1e956b607b6ec7d315de7c7d9086968329a438eb0dac9384b380f942b8bac8e36ca41345cbf08cc0f7029245
+  metadata.gz: 1eeacb48b733d694284681b6370e8080e88269504bdd6f793eb7e4c68fefc8732cc1f0225272f250c201fc669bf63d0086d36cc2476dc8b79ec9cd741d3f092f
+  data.tar.gz: fc41db9e4dc2563cafff6a98fa168a823816e2b68383ce0b8399fbee872d221774fe0189356412c4f7b6962baf6655b77ed5cb0c125ab8a5e916965d980646e3

data/README.md

@@ -12,6 +12,23 @@ Currently, it ships the following checkers to help improve the boundaries betwee
 - A `visibility` checker that allows packages to be private except to an explicit group of other packages.
 - An experimental `architecture` checker that allows packages to specify their "layer" and requires that each layer only communicate with layers below it.
 
+## Installation
+
+To register all checkers included in this gem, add the following to your `packwerk.yml`:
+
+```yaml
+require: packwerk-extensions
+```
+
+Alternatively, you can require individual checkers:
+
+```yaml
+require:
+  - packwerk/privacy/checker
+  - packwerk/visibility/checker
+  - packwerk/architecture/checker
+```
+
 ## Privacy Checker
 The privacy checker extension was originally extracted from [packwerk](https://github.com/Shopify/packwerk).
 
@@ -39,6 +56,9 @@ Example:
 public_path: my/custom/path/
 ```
 
+### Using specific private constants
+Sometimes it is desirable to only enforce privacy on a subset of constants in a package. You can do so by defining a `private_constants` list in your package.yml. Note that `enforce_privacy` must be set to `true` or `'strict'` for this to work.
+
 ### Package Privacy violation
 A constant that is private to its package has been referenced from outside of the package. Constants are declared private in their package’s `package.yml`.
 

data/lib/packwerk/architecture/checker.rb

@@ -2,6 +2,8 @@
 # frozen_string_literal: true
 
 require 'packwerk/architecture/layers'
+require 'packwerk/architecture/package'
+require 'packwerk/architecture/validator'
 
 module Packwerk
   module Architecture

data/lib/packwerk/privacy/checker.rb

@@ -1,6 +1,9 @@
 # typed: strict
 # frozen_string_literal: true
 
+require 'packwerk/privacy/package'
+require 'packwerk/privacy/validator'
+
 module Packwerk
   module Privacy
     # Checks whether a given reference references a private constant of another package.
@@ -27,7 +30,9 @@ module Packwerk
         return false if privacy_package.public_path?(reference.constant.location)
 
         privacy_option = privacy_package.enforce_privacy
-        !enforcement_disabled?(privacy_option)
+        return false if enforcement_disabled?(privacy_option)
+
+        explicitly_private_constant?(reference.constant, explicitly_private_constants: privacy_package.private_constants)
       end
 
       sig do
@@ -60,6 +65,20 @@ module Packwerk
 
       private
 
+      sig do
+        params(
+          constant: ConstantContext,
+          explicitly_private_constants: T::Array[String]
+        ).returns(T::Boolean)
+      end
+      def explicitly_private_constant?(constant, explicitly_private_constants:)
+        return true if explicitly_private_constants.empty?
+
+        explicitly_private_constants.include?(constant.name) ||
+          # nested constants
+          explicitly_private_constants.any? { |epc| constant.name.start_with?("#{epc}::") }
+      end
+
       sig do
         params(privacy_option: T.nilable(T.any(T::Boolean, String, T::Array[String])))
           .returns(T::Boolean)

data/lib/packwerk/privacy/package.rb

@@ -8,7 +8,8 @@ module Packwerk
 
       const :public_path, String
       const :user_defined_public_path, T.nilable(String)
-      const :enforce_privacy, T.nilable(T.any(T::Boolean, String, T::Array[String]))
+      const :enforce_privacy, T.nilable(T.any(T::Boolean, String))
+      const :private_constants, T::Array[String]
 
       sig { params(path: String).returns(T::Boolean) }
       def public_path?(path)
@@ -23,7 +24,8 @@ module Packwerk
           Package.new(
             public_path: public_path_for(package),
             user_defined_public_path: user_defined_public_path(package),
-            enforce_privacy: package.config['enforce_privacy']
+            enforce_privacy: package.config['enforce_privacy'],
+            private_constants: package.config['private_constants'] || []
           )
         end
 

data/lib/packwerk/privacy/validator.rb

@@ -19,6 +19,8 @@ module Packwerk
           results << check_enforce_privacy_setting(config_file_path, setting)
         end
 
+        results += verify_private_constants_setting(package_set, configuration)
+
         public_path_settings = package_manifests_settings_for(configuration, 'public_path')
         public_path_settings.each do |config_file_path, setting|
           results << check_public_path(config_file_path, setting)
@@ -29,11 +31,49 @@ module Packwerk
 
       sig { override.returns(T::Array[String]) }
       def permitted_keys
-        %w[public_path enforce_privacy]
+        %w[public_path enforce_privacy private_constants]
       end
 
       private
 
+      sig { params(package_set: PackageSet, configuration: Configuration).returns(T::Array[Result]) }
+      def verify_private_constants_setting(package_set, configuration)
+        private_constants_setting = package_manifests_settings_for(configuration, 'private_constants')
+        results = T.let([], T::Array[Result])
+        resolver = ConstantResolver.new(
+          root_path: configuration.root_path,
+          load_paths: configuration.load_paths
+        )
+
+        private_constants_setting.each do |config_file_path, setting|
+          next if setting.nil?
+
+          unless setting.is_a?(Array)
+            results << Result.new(
+              ok: false,
+              error_value: "Invalid 'private_constants' setting: #{setting.inspect}"
+            )
+            next
+          end
+
+          constants = setting
+
+          results += assert_constants_can_be_loaded(constants, config_file_path)
+
+          constant_locations = constants.map { |c| [c, resolver.resolve(c)&.location] }
+
+          constant_locations.each do |name, location|
+            results << if location
+                         check_private_constant_location(configuration, package_set, name, location, config_file_path)
+                       else
+                         private_constant_unresolvable(name, config_file_path)
+                       end
+          end
+        end
+
+        results
+      end
+
       sig do
         params(config_file_path: String, setting: T.untyped).returns(Result)
       end
@@ -52,7 +92,7 @@ module Packwerk
         params(config_file_path: String, setting: T.untyped).returns(Result)
       end
       def check_enforce_privacy_setting(config_file_path, setting)
-        if [TrueClass, FalseClass, Array, NilClass].include?(setting.class) || setting == 'strict'
+        if [TrueClass, FalseClass, NilClass].include?(setting.class) || setting == 'strict'
           Result.new(ok: true)
         else
           Result.new(
@@ -61,6 +101,55 @@ module Packwerk
           )
         end
       end
+
+      sig do
+        params(configuration: Configuration, package_set: PackageSet, name: T.untyped, location: T.untyped,
+               config_file_path: T.untyped).returns(Result)
+      end
+      def check_private_constant_location(configuration, package_set, name, location, config_file_path)
+        declared_package = package_set.package_from_path(relative_path(configuration, config_file_path))
+        constant_package = package_set.package_from_path(location)
+
+        if constant_package == declared_package
+          Result.new(ok: true)
+        else
+          Result.new(
+            ok: false,
+            error_value: "'#{name}' is declared as private in the '#{declared_package}' package but appears to be " \
+                         "defined\nin the '#{constant_package}' package. Packwerk resolved it to #{location}."
+          )
+        end
+      end
+
+      sig { params(constants: T.untyped, config_file_path: String).returns(T::Array[Result]) }
+      def assert_constants_can_be_loaded(constants, config_file_path)
+        constants.map do |constant|
+          if constant.start_with?('::')
+            constant.try(&:constantize) && Result.new(ok: true)
+          else
+            error_value = "'#{constant}', listed in the 'private_constants' option " \
+                          "in #{config_file_path}, is invalid.\nPrivate constants need to be " \
+                          'prefixed with the top-level namespace operator `::`.'
+            Result.new(
+              ok: false,
+              error_value: error_value
+            )
+          end
+        end
+      end
+
+      sig { params(name: T.untyped, config_file_path: T.untyped).returns(Result) }
+      def private_constant_unresolvable(name, config_file_path)
+        explicit_filepath = "#{(name.start_with?('::') ? name[2..] : name).underscore}.rb"
+
+        Result.new(
+          ok: false,
+          error_value: "'#{name}', listed in #{config_file_path}, could not be resolved.\n" \
+                       "This is probably because it is an autovivified namespace - a namespace module that doesn't have a\n" \
+                       "file explicitly defining it. Packwerk currently doesn't support declaring autovivified namespaces as\n" \
+                       "private. Add a #{explicit_filepath} file to explicitly define the constant."
+        )
+      end
     end
   end
 end

data/lib/packwerk/visibility/checker.rb

@@ -1,6 +1,9 @@
 # typed: strict
 # frozen_string_literal: true
 
+require 'packwerk/visibility/package'
+require 'packwerk/visibility/validator'
+
 module Packwerk
   module Visibility
     # Checks whether a given reference references a constant from a package that does not permit visibility

data/lib/packwerk-extensions.rb

@@ -5,16 +5,8 @@ require 'sorbet-runtime'
 require 'packwerk'
 
 require 'packwerk/privacy/checker'
-require 'packwerk/privacy/package'
-require 'packwerk/privacy/validator'
-
 require 'packwerk/visibility/checker'
-require 'packwerk/visibility/package'
-require 'packwerk/visibility/validator'
-
 require 'packwerk/architecture/checker'
-require 'packwerk/architecture/package'
-require 'packwerk/architecture/validator'
 
 module Packwerk
   module Extensions

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: packwerk-extensions
 version: !ruby/object:Gem::Version
-  version: 0.0.9
+  version: 0.1.1
 platform: ruby
 authors:
 - Gusto Engineers
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-12-30 00:00:00.000000000 Z
+date: 2023-02-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: packwerk
@@ -25,19 +25,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 2.2.1
 - !ruby/object:Gem::Dependency
-  name: rails
+  name: railties
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 6.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 6.0.0
 - !ruby/object:Gem::Dependency
   name: sorbet-runtime
   requirement: !ruby/object:Gem::Requirement
@@ -108,6 +108,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rake
+  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: rubocop
   requirement: !ruby/object:Gem::Requirement