danger-packwerk 0.9.0 → 0.11.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cf8895f7ae2e94e0b1d09e146176b6777ba8f952fe03381380071114fcec2bd4
-  data.tar.gz: 2e68e2096375433cfad68d85ca8d5eac7855dcb542b34f79417a53b1e4c9eb1b
+  metadata.gz: 0f8b77fc9b24c94b59f84416ca3a564d4bc83b2e0cb0e33608aa4ed08dc2fc69
+  data.tar.gz: df82d0bf21d6702747060d5d4d3e1ae8a3ed39973abdede4db7490d4e77aa2cd
 SHA512:
-  metadata.gz: bfbe7e86650ac170c89c014f7519d0d40abdb3d864b995d8cdb5ef2478ec3e4fe6f2e3afc4854dbc216e622a8adc99761bd7d9a420167553db6b70084446740f
-  data.tar.gz: b6729e1b52a5f25ae5da573d706bb1f3fc4217c750976dcd51aafd7611a5c7f53b4851727e3fd20cb66065570d2a99b0c4a4c367572c84fc08fb6918dfdfeb6b
+  metadata.gz: ab8300d69b2b95d6ee14c451d5cb266b4bea7b99b9c2e1a9691877a660128b367df2a65b6231c131591e19964aa2f75a0cb0c88277fba67a7c69f84ed9022e97
+  data.tar.gz: 97692e72e269be921be02e80094403e56ed9175089395827fd5443babbdbe7a6ae4ce47f8b75b821571c54df36734423426271f32fe293867be44e3f152e7ded

data/lib/danger-packwerk/basic_reference_offense.rb

@@ -47,7 +47,7 @@ module DangerPackwerk
 
       from_package = ParsePackwerk.package_from_path(package_todo_yml_pathname)
       from_package_name = from_package.name
-      violations = Private::PackageTodo.from(package_todo_yml_pathname).violations
+      violations = ParsePackwerk::PackageTodo.from(package_todo_yml_pathname).violations
 
       # See the larger comment below for more information on why we need this information.
       # This is a small optimization that lets us find the location of referenced files within

data/lib/danger-packwerk/danger_package_todo_yml_changes.rb

@@ -19,18 +19,24 @@ module DangerPackwerk
     DEFAULT_MAX_COMMENTS = 5
     BeforeComment = T.type_alias { T.proc.params(violation_diff: ViolationDiff, changed_package_todo_ymls: T::Array[String]).void }
     DEFAULT_BEFORE_COMMENT = T.let(->(violation_diff, changed_package_todo_ymls) {}, BeforeComment)
+    DEFAULT_VIOLATION_TYPES = T.let([
+                                      DEPENDENCY_VIOLATION_TYPE,
+                                      PRIVACY_VIOLATION_TYPE
+                                    ], T::Array[String])
 
     sig do
       params(
         offenses_formatter: T.nilable(Update::OffensesFormatter),
         before_comment: BeforeComment,
-        max_comments: Integer
+        max_comments: Integer,
+        violation_types: T::Array[String]
       ).void
     end
     def check(
       offenses_formatter: nil,
       before_comment: DEFAULT_BEFORE_COMMENT,
-      max_comments: DEFAULT_MAX_COMMENTS
+      max_comments: DEFAULT_MAX_COMMENTS,
+      violation_types: DEFAULT_VIOLATION_TYPES
     )
       offenses_formatter ||= Update::DefaultFormatter.new
       repo_link = github.pr_json[:base][:repo][:html_url]
@@ -38,7 +44,7 @@ module DangerPackwerk
 
       changed_package_todo_ymls = (git.modified_files + git.added_files + git.deleted_files).grep(PACKAGE_TODO_PATTERN)
 
-      violation_diff = get_violation_diff
+      violation_diff = get_violation_diff(violation_types)
 
       before_comment.call(
         violation_diff,
@@ -62,8 +68,8 @@ module DangerPackwerk
       end
     end
 
-    sig { returns(ViolationDiff) }
-    def get_violation_diff # rubocop:disable Naming/AccessorMethodName
+    sig { params(violation_types: T::Array[String]).returns(ViolationDiff) }
+    def get_violation_diff(violation_types)
       added_violations = T.let([], T::Array[BasicReferenceOffense])
       removed_violations = T.let([], T::Array[BasicReferenceOffense])
 
@@ -117,12 +123,18 @@ module DangerPackwerk
       end
 
       relevant_added_violations = added_violations.reject do |violation|
-        renamed_files_after.include?(violation.file) || renamed_constants.include?(violation.class_name)
+        renamed_files_after.include?(violation.file) ||
+          renamed_constants.include?(violation.class_name) ||
+          !violation_types.include?(violation.type)
+      end
+
+      relevant_removed_violations = removed_violations.select do |violation|
+        violation_types.include?(violation.type)
       end
 
       ViolationDiff.new(
         added_violations: relevant_added_violations,
-        removed_violations: removed_violations
+        removed_violations: relevant_removed_violations
       )
     end
 

data/lib/danger-packwerk/danger_packwerk.rb

@@ -21,6 +21,10 @@ module DangerPackwerk
     DEFAULT_ON_FAILURE = T.let(->(offenses) {}, OnFailure)
     DEFAULT_FAIL = false
     DEFAULT_FAILURE_MESSAGE = 'Packwerk violations were detected! Please resolve them to unblock the build.'
+    DEFAULT_VIOLATION_TYPES = T.let([
+                                      DEPENDENCY_VIOLATION_TYPE,
+                                      PRIVACY_VIOLATION_TYPE
+                                    ], T::Array[String])
 
     class CommentGroupingStrategy < ::T::Enum
       enums do
@@ -38,6 +42,7 @@ module DangerPackwerk
         fail_build: T::Boolean,
         failure_message: String,
         on_failure: OnFailure,
+        violation_types: T::Array[String],
         grouping_strategy: CommentGroupingStrategy
       ).void
     end
@@ -47,6 +52,7 @@ module DangerPackwerk
       fail_build: DEFAULT_FAIL,
       failure_message: DEFAULT_FAILURE_MESSAGE,
       on_failure: DEFAULT_ON_FAILURE,
+      violation_types: DEFAULT_VIOLATION_TYPES,
       grouping_strategy: CommentGroupingStrategy::PerConstantPerLocation
     )
       offenses_formatter ||= Check::DefaultFormatter.new
@@ -88,11 +94,13 @@ module DangerPackwerk
 
       renamed_files = git.renamed_files.map { |before_after_file| before_after_file[:after] }
 
-      # Ignore references that have been renamed
       packwerk_reference_offenses_to_care_about = packwerk_reference_offenses.reject do |packwerk_reference_offense|
         constant_name = packwerk_reference_offense.reference.constant.name
         filepath_that_defines_this_constant = Private.constant_resolver.resolve(constant_name)&.location
-        renamed_files.include?(filepath_that_defines_this_constant)
+        # Ignore references that have been renamed
+        renamed_files.include?(filepath_that_defines_this_constant) ||
+          # Ignore violations that are not in the allow-list of violation types to leave comments for
+          !violation_types.include?(packwerk_reference_offense.violation_type)
       end
 
       # We group by the constant name, line number, and reference path. Any offenses with these same values should only differ on what type of violation

data/lib/danger-packwerk/packwerk_wrapper.rb

@@ -56,7 +56,9 @@ module DangerPackwerk
         ''
       end
 
-      sig { override.params(offense_collection: Packwerk::OffenseCollection, for_files: T::Set[String]).returns(String) }
+      # T.untyped should be Packwerk::OffenseCollection, but is currently private until
+      # https://github.com/Shopify/packwerk/pull/289 merges
+      sig { override.params(offense_collection: T.untyped, for_files: T::Set[String]).returns(String) }
       def show_stale_violations(offense_collection, for_files)
         ''
       end

data/lib/danger-packwerk/private.rb

@@ -1,6 +1,5 @@
 # typed: strict
 
-require 'danger-packwerk/private/package_todo'
 require 'danger-packwerk/private/ownership_information'
 require 'constant_resolver'
 

data/lib/danger-packwerk/version.rb

@@ -2,5 +2,5 @@
 # frozen_string_literal: true
 
 module DangerPackwerk
-  VERSION = '0.9.0'
+  VERSION = '0.11.0'
 end

data/sorbet/rbi/gems/{packwerk@2.2.1-7e8e7a50705833b41fe40c201eadb85ff9a1a422.rbi → packwerk@2.2.1-8476bb3cd5452765ad452d36aa45ae724f1d44f5.rbi}

@@ -67,7 +67,7 @@ class Packwerk::ApplicationValidator
       .params(
         package_set: Packwerk::PackageSet,
         configuration: ::Packwerk::Configuration
-      ).returns(::Packwerk::ApplicationValidator::Result)
+      ).returns(::Packwerk::Validator::Result)
   end
   def call(package_set, configuration); end
 
@@ -76,24 +76,24 @@ class Packwerk::ApplicationValidator
     params(
       package_set: Packwerk::PackageSet,
       configuration: ::Packwerk::Configuration
-    ).returns(::Packwerk::ApplicationValidator::Result)
+    ).returns(::Packwerk::Validator::Result)
   end
   def check_all(package_set, configuration); end
 
   # source://packwerk//lib/packwerk/application_validator.rb#69
-  sig { params(configuration: ::Packwerk::Configuration).returns(::Packwerk::ApplicationValidator::Result) }
+  sig { params(configuration: ::Packwerk::Configuration).returns(::Packwerk::Validator::Result) }
   def check_application_structure(configuration); end
 
   # source://packwerk//lib/packwerk/application_validator.rb#84
-  sig { params(configuration: ::Packwerk::Configuration).returns(::Packwerk::ApplicationValidator::Result) }
+  sig { params(configuration: ::Packwerk::Configuration).returns(::Packwerk::Validator::Result) }
   def check_package_manifest_paths(configuration); end
 
   # source://packwerk//lib/packwerk/application_validator.rb#41
-  sig { params(configuration: ::Packwerk::Configuration).returns(::Packwerk::ApplicationValidator::Result) }
+  sig { params(configuration: ::Packwerk::Configuration).returns(::Packwerk::Validator::Result) }
   def check_package_manifest_syntax(configuration); end
 
   # source://packwerk//lib/packwerk/application_validator.rb#105
-  sig { params(configuration: ::Packwerk::Configuration).returns(::Packwerk::ApplicationValidator::Result) }
+  sig { params(configuration: ::Packwerk::Configuration).returns(::Packwerk::Validator::Result) }
   def check_root_package_exists(configuration); end
 
   # source://packwerk//lib/packwerk/application_validator.rb#34
@@ -111,21 +111,6 @@ class Packwerk::ApplicationValidator
   def relative_paths(configuration, paths); end
 end
 
-# source://packwerk//lib/packwerk/application_validator/result.rb#6
-class Packwerk::ApplicationValidator::Result < ::T::Struct
-  const :ok, T::Boolean
-  const :error_value, T.nilable(::String)
-
-  # source://packwerk//lib/packwerk/application_validator/result.rb#13
-  sig { returns(T::Boolean) }
-  def ok?; end
-
-  class << self
-    # source://sorbet-runtime/0.5.10588/lib/types/struct.rb#13
-    def inherited(s); end
-  end
-end
-
 # Extracts the implicit constant reference from an active record association
 #
 # source://packwerk//lib/packwerk/association_inspector.rb#6
@@ -287,6 +272,8 @@ end
 #
 # source://packwerk//lib/packwerk/cli.rb#8
 class Packwerk::Cli
+  extend ::ActiveSupport::Autoload
+
   # source://packwerk//lib/packwerk/cli.rb#21
   sig do
     params(
@@ -327,19 +314,19 @@ class Packwerk::Cli
   sig { returns(T::Boolean) }
   def init; end
 
-  # source://packwerk//lib/packwerk/cli.rb#174
-  sig { params(result: ::Packwerk::ApplicationValidator::Result).void }
+  # source://packwerk//lib/packwerk/cli.rb#176
+  sig { params(result: ::Packwerk::Validator::Result).void }
   def list_validation_errors(result); end
 
   # source://packwerk//lib/packwerk/cli.rb#123
-  sig { params(result: ::Packwerk::Result).returns(T::Boolean) }
+  sig { params(result: ::Packwerk::Cli::Result).returns(T::Boolean) }
   def output_result(result); end
 
-  # source://packwerk//lib/packwerk/cli.rb#166
+  # source://packwerk//lib/packwerk/cli.rb#168
   sig { returns(Packwerk::PackageSet) }
   def package_set; end
 
-  # source://packwerk//lib/packwerk/cli.rb#186
+  # source://packwerk//lib/packwerk/cli.rb#188
   sig { params(args: T::Array[::String]).returns(::Packwerk::ParseRun) }
   def parse_run(args); end
 
@@ -351,11 +338,22 @@ class Packwerk::Cli
   sig { params(_paths: T::Array[::String]).returns(T::Boolean) }
   def validate(_paths); end
 
-  # source://packwerk//lib/packwerk/cli.rb#161
+  # source://packwerk//lib/packwerk/cli.rb#163
   sig { returns(::Packwerk::ApplicationValidator) }
   def validator; end
 end
 
+# source://packwerk//lib/packwerk/cli/result.rb#6
+class Packwerk::Cli::Result < ::T::Struct
+  const :message, ::String
+  const :status, T::Boolean
+
+  class << self
+    # source://sorbet-runtime/0.5.10588/lib/types/struct.rb#13
+    def inherited(s); end
+  end
+end
+
 # source://packwerk//lib/packwerk/configuration.rb#8
 class Packwerk::Configuration
   # @return [Configuration] a new instance of Configuration
@@ -462,44 +460,8 @@ class Packwerk::ConstNodeInspector
   def root_constant?(parent); end
 end
 
-# Get information about unresolved constants without loading the application code.
-# Information gathered: Fully qualified name, path to file containing the definition, package,
-# and visibility (public/private to the package).
-#
-# The implementation makes a few assumptions about the code base:
-# - `Something::SomeOtherThing` is defined in a path of either `something/some_other_thing.rb` or `something.rb`,
-#   relative to the load path. Rails' `zeitwerk` autoloader makes the same assumption.
-# - It is OK to not always infer the exact file defining the constant. For example, when a constant is inherited, we
-#   have no way of inferring the file it is defined in. You could argue though that inheritance means that another
-#   constant with the same name exists in the inheriting class, and this view is sufficient for all our use cases.
-#
-# source://packwerk//lib/packwerk/constant_discovery.rb#17
-class Packwerk::ConstantDiscovery
-  # source://packwerk//lib/packwerk/constant_discovery.rb#27
-  sig { params(constant_resolver: ::ConstantResolver, packages: Packwerk::PackageSet).void }
-  def initialize(constant_resolver:, packages:); end
-
-  # Analyze a constant via its name.
-  # If the constant is unresolved, we need the current namespace path to correctly infer its full name
-  #
-  # source://packwerk//lib/packwerk/constant_discovery.rb#60
-  sig do
-    params(
-      const_name: ::String,
-      current_namespace_path: T.nilable(T::Array[::String])
-    ).returns(T.nilable(::Packwerk::ConstantDiscovery::ConstantContext))
-  end
-  def context_for(const_name, current_namespace_path: T.unsafe(nil)); end
-
-  # Get the package that owns a given file path.
-  #
-  # source://packwerk//lib/packwerk/constant_discovery.rb#43
-  sig { params(path: ::String).returns(::Packwerk::Package) }
-  def package_from_path(path); end
-end
-
-# source://packwerk//lib/packwerk/constant_discovery.rb#20
-class Packwerk::ConstantDiscovery::ConstantContext < ::Struct
+# source://packwerk//lib/packwerk/constant_context.rb#9
+class Packwerk::ConstantContext < ::Struct
   # Returns the value of attribute location
   #
   # @return [Object] the current value of location
@@ -510,7 +472,7 @@ class Packwerk::ConstantDiscovery::ConstantContext < ::Struct
   # @param value [Object] the value to set the attribute location to.
   # @return [Object] the newly set value
   #
-  # source://packwerk//lib/packwerk/constant_discovery.rb#20
+  # source://packwerk//lib/packwerk/constant_context.rb#9
   def location=(_); end
 
   # Returns the value of attribute name
@@ -523,7 +485,7 @@ class Packwerk::ConstantDiscovery::ConstantContext < ::Struct
   # @param value [Object] the value to set the attribute name to.
   # @return [Object] the newly set value
   #
-  # source://packwerk//lib/packwerk/constant_discovery.rb#20
+  # source://packwerk//lib/packwerk/constant_context.rb#9
   def name=(_); end
 
   # Returns the value of attribute package
@@ -536,7 +498,7 @@ class Packwerk::ConstantDiscovery::ConstantContext < ::Struct
   # @param value [Object] the value to set the attribute package to.
   # @return [Object] the newly set value
   #
-  # source://packwerk//lib/packwerk/constant_discovery.rb#20
+  # source://packwerk//lib/packwerk/constant_context.rb#9
   def package=(_); end
 
   class << self
@@ -547,6 +509,42 @@ class Packwerk::ConstantDiscovery::ConstantContext < ::Struct
   end
 end
 
+# Get information about unresolved constants without loading the application code.
+# Information gathered: Fully qualified name, path to file containing the definition, package,
+# and visibility (public/private to the package).
+#
+# The implementation makes a few assumptions about the code base:
+# - `Something::SomeOtherThing` is defined in a path of either `something/some_other_thing.rb` or `something.rb`,
+#   relative to the load path. Rails' `zeitwerk` autoloader makes the same assumption.
+# - It is OK to not always infer the exact file defining the constant. For example, when a constant is inherited, we
+#   have no way of inferring the file it is defined in. You could argue though that inheritance means that another
+#   constant with the same name exists in the inheriting class, and this view is sufficient for all our use cases.
+#
+# source://packwerk//lib/packwerk/constant_discovery.rb#17
+class Packwerk::ConstantDiscovery
+  # source://packwerk//lib/packwerk/constant_discovery.rb#25
+  sig { params(constant_resolver: ::ConstantResolver, packages: Packwerk::PackageSet).void }
+  def initialize(constant_resolver:, packages:); end
+
+  # Analyze a constant via its name.
+  # If the constant is unresolved, we need the current namespace path to correctly infer its full name
+  #
+  # source://packwerk//lib/packwerk/constant_discovery.rb#58
+  sig do
+    params(
+      const_name: ::String,
+      current_namespace_path: T.nilable(T::Array[::String])
+    ).returns(T.nilable(::Packwerk::ConstantContext))
+  end
+  def context_for(const_name, current_namespace_path: T.unsafe(nil)); end
+
+  # Get the package that owns a given file path.
+  #
+  # source://packwerk//lib/packwerk/constant_discovery.rb#41
+  sig { params(path: ::String).returns(::Packwerk::Package) }
+  def package_from_path(path); end
+end
+
 # An interface describing an object that can extract a constant name from an AST node.
 #
 # @abstract Subclasses must implement the `abstract` methods below.
@@ -694,24 +692,24 @@ end
 # source://packwerk//lib/packwerk/files_for_processing.rb#8
 Packwerk::FilesForProcessing::RelativeFileSet = T.type_alias { T::Set[::String] }
 
-# source://packwerk//lib/packwerk.rb#66
+# source://packwerk//lib/packwerk.rb#51
 module Packwerk::Formatters
   extend ::ActiveSupport::Autoload
 end
 
-# source://packwerk//lib/packwerk/formatters/offenses_formatter.rb#6
-class Packwerk::Formatters::OffensesFormatter
+# source://packwerk//lib/packwerk/formatters/default_offenses_formatter.rb#6
+class Packwerk::Formatters::DefaultOffensesFormatter
   include ::Packwerk::OffensesFormatter
 
-  # source://packwerk//lib/packwerk/formatters/offenses_formatter.rb#33
+  # source://packwerk//lib/packwerk/formatters/default_offenses_formatter.rb#33
   sig { override.returns(::String) }
   def identifier; end
 
-  # source://packwerk//lib/packwerk/formatters/offenses_formatter.rb#14
+  # source://packwerk//lib/packwerk/formatters/default_offenses_formatter.rb#14
   sig { override.params(offenses: T::Array[T.nilable(::Packwerk::Offense)]).returns(::String) }
   def show_offenses(offenses); end
 
-  # source://packwerk//lib/packwerk/formatters/offenses_formatter.rb#24
+  # source://packwerk//lib/packwerk/formatters/default_offenses_formatter.rb#24
   sig do
     override
       .params(
@@ -721,31 +719,31 @@ class Packwerk::Formatters::OffensesFormatter
   end
   def show_stale_violations(offense_collection, fileset); end
 
-  # source://packwerk//lib/packwerk/formatters/offenses_formatter.rb#38
+  # source://packwerk//lib/packwerk/formatters/default_offenses_formatter.rb#38
   sig { override.params(strict_mode_violations: T::Array[::Packwerk::ReferenceOffense]).returns(::String) }
   def show_strict_mode_violations(strict_mode_violations); end
 
   private
 
-  # source://packwerk//lib/packwerk/formatters/offenses_formatter.rb#54
+  # source://packwerk//lib/packwerk/formatters/default_offenses_formatter.rb#54
   sig { params(offense: ::Packwerk::ReferenceOffense).returns(::String) }
   def format_strict_mode_violation(offense); end
 
-  # source://packwerk//lib/packwerk/formatters/offenses_formatter.rb#62
+  # source://packwerk//lib/packwerk/formatters/default_offenses_formatter.rb#62
   sig { params(offenses: T::Array[T.nilable(::Packwerk::Offense)]).returns(::String) }
   def offenses_list(offenses); end
 
-  # source://packwerk//lib/packwerk/formatters/offenses_formatter.rb#70
+  # source://packwerk//lib/packwerk/formatters/default_offenses_formatter.rb#70
   sig { params(offenses: T::Array[T.nilable(::Packwerk::Offense)]).returns(::String) }
   def offenses_summary(offenses); end
 
-  # source://packwerk//lib/packwerk/formatters/offenses_formatter.rb#49
+  # source://packwerk//lib/packwerk/formatters/default_offenses_formatter.rb#49
   sig { returns(::Packwerk::OutputStyle) }
   def style; end
 end
 
-# source://packwerk//lib/packwerk/formatters/offenses_formatter.rb#9
-Packwerk::Formatters::OffensesFormatter::IDENTIFIER = T.let(T.unsafe(nil), String)
+# source://packwerk//lib/packwerk/formatters/default_offenses_formatter.rb#9
+Packwerk::Formatters::DefaultOffensesFormatter::IDENTIFIER = T.let(T.unsafe(nil), String)
 
 # source://packwerk//lib/packwerk/formatters/progress_formatter.rb#8
 class Packwerk::Formatters::ProgressFormatter
@@ -753,58 +751,69 @@ class Packwerk::Formatters::ProgressFormatter
   sig { params(out: T.any(::IO, ::StringIO), style: ::Packwerk::OutputStyle).void }
   def initialize(out, style: T.unsafe(nil)); end
 
-  # source://packwerk//lib/packwerk/formatters/progress_formatter.rb#39
+  # source://packwerk//lib/packwerk/formatters/progress_formatter.rb#44
+  sig { void }
   def interrupted; end
 
-  # source://packwerk//lib/packwerk/formatters/progress_formatter.rb#35
+  # source://packwerk//lib/packwerk/formatters/progress_formatter.rb#39
+  sig { void }
   def mark_as_failed; end
 
-  # source://packwerk//lib/packwerk/formatters/progress_formatter.rb#31
+  # source://packwerk//lib/packwerk/formatters/progress_formatter.rb#34
+  sig { void }
   def mark_as_inspected; end
 
-  # source://packwerk//lib/packwerk/formatters/progress_formatter.rb#24
+  # source://packwerk//lib/packwerk/formatters/progress_formatter.rb#26
+  sig { params(target_files: T::Set[::String], block: T.proc.void).void }
   def started_inspection(target_files, &block); end
 
-  # source://packwerk//lib/packwerk/formatters/progress_formatter.rb#17
+  # source://packwerk//lib/packwerk/formatters/progress_formatter.rb#18
+  sig { params(block: T.proc.void).void }
   def started_validation(&block); end
 
   private
 
-  # source://packwerk//lib/packwerk/formatters/progress_formatter.rb#46
+  # source://packwerk//lib/packwerk/formatters/progress_formatter.rb#52
+  sig { params(execution_time: ::Float).void }
   def finished(execution_time); end
 
-  # source://packwerk//lib/packwerk/formatters/progress_formatter.rb#55
+  # source://packwerk//lib/packwerk/formatters/progress_formatter.rb#63
+  sig { params(target_files: T::Set[::String]).void }
   def start_inspection(target_files); end
 
-  # source://packwerk//lib/packwerk/formatters/progress_formatter.rb#51
+  # source://packwerk//lib/packwerk/formatters/progress_formatter.rb#58
+  sig { void }
   def start_validation; end
 end
 
-# source://packwerk//lib/packwerk.rb#72
+# source://packwerk//lib/packwerk.rb#87
 module Packwerk::Generators
   extend ::ActiveSupport::Autoload
 end
 
 # source://packwerk//lib/packwerk/generators/configuration_file.rb#8
 class Packwerk::Generators::ConfigurationFile
-  # source://packwerk//lib/packwerk/generators/configuration_file.rb#20
+  # source://packwerk//lib/packwerk/generators/configuration_file.rb#23
   sig { params(root: ::String, out: T.any(::IO, ::StringIO)).void }
   def initialize(root:, out: T.unsafe(nil)); end
 
-  # source://packwerk//lib/packwerk/generators/configuration_file.rb#26
+  # source://packwerk//lib/packwerk/generators/configuration_file.rb#29
   sig { returns(T::Boolean) }
   def generate; end
 
   private
 
-  # source://packwerk//lib/packwerk/generators/configuration_file.rb#43
+  # source://packwerk//lib/packwerk/generators/configuration_file.rb#47
+  sig { returns(::String) }
   def render; end
 
-  # source://packwerk//lib/packwerk/generators/configuration_file.rb#47
+  # source://packwerk//lib/packwerk/generators/configuration_file.rb#52
+  sig { returns(::String) }
   def template; end
 
   class << self
-    # source://packwerk//lib/packwerk/generators/configuration_file.rb#14
+    # source://packwerk//lib/packwerk/generators/configuration_file.rb#17
+    sig { params(root: ::String, out: T.any(::IO, ::StringIO)).returns(T::Boolean) }
     def generate(root:, out:); end
   end
 end
@@ -814,17 +823,17 @@ Packwerk::Generators::ConfigurationFile::CONFIGURATION_TEMPLATE_FILE_PATH = T.le
 
 # source://packwerk//lib/packwerk/generators/root_package.rb#6
 class Packwerk::Generators::RootPackage
-  # @return [RootPackage] a new instance of RootPackage
-  #
-  # source://packwerk//lib/packwerk/generators/root_package.rb#15
+  # source://packwerk//lib/packwerk/generators/root_package.rb#19
+  sig { params(root: ::String, out: T.any(::IO, ::StringIO)).void }
   def initialize(root:, out: T.unsafe(nil)); end
 
-  # source://packwerk//lib/packwerk/generators/root_package.rb#21
+  # source://packwerk//lib/packwerk/generators/root_package.rb#25
   sig { returns(T::Boolean) }
   def generate; end
 
   class << self
-    # source://packwerk//lib/packwerk/generators/root_package.rb#10
+    # source://packwerk//lib/packwerk/generators/root_package.rb#13
+    sig { params(root: ::String, out: T.any(::IO, ::StringIO)).returns(T::Boolean) }
     def generate(root:, out:); end
   end
 end
@@ -864,7 +873,7 @@ class Packwerk::Graph
 end
 
 # source://packwerk//lib/packwerk/node.rb#5
-class Packwerk::Node; end
+module Packwerk::Node; end
 
 # source://packwerk//lib/packwerk/node.rb#6
 class Packwerk::Node::Location < ::Struct
@@ -1279,7 +1288,7 @@ module Packwerk::OutputStyle
   def reset; end
 end
 
-# source://packwerk//lib/packwerk.rb#55
+# source://packwerk//lib/packwerk.rb#40
 module Packwerk::OutputStyles
   extend ::ActiveSupport::Autoload
 end
@@ -1503,11 +1512,11 @@ class Packwerk::ParseRun
   def initialize(relative_file_set:, configuration:, file_set_specified: T.unsafe(nil), offenses_formatter: T.unsafe(nil), progress_formatter: T.unsafe(nil)); end
 
   # source://packwerk//lib/packwerk/parse_run.rb#61
-  sig { returns(::Packwerk::Result) }
+  sig { returns(::Packwerk::Cli::Result) }
   def check; end
 
   # source://packwerk//lib/packwerk/parse_run.rb#39
-  sig { returns(::Packwerk::Result) }
+  sig { returns(::Packwerk::Cli::Result) }
   def update_todo; end
 
   private
@@ -1569,25 +1578,33 @@ module Packwerk::Parsers; end
 class Packwerk::Parsers::Erb
   include ::Packwerk::Parsers::ParserInterface
 
-  # @return [Erb] a new instance of Erb
-  #
-  # source://packwerk//lib/packwerk/parsers/erb.rb#14
+  # source://packwerk//lib/packwerk/parsers/erb.rb#17
+  sig { params(parser_class: T.untyped, ruby_parser: ::Packwerk::Parsers::Ruby).void }
   def initialize(parser_class: T.unsafe(nil), ruby_parser: T.unsafe(nil)); end
 
-  # source://packwerk//lib/packwerk/parsers/erb.rb#19
+  # source://packwerk//lib/packwerk/parsers/erb.rb#23
+  sig { override.params(io: T.any(::IO, ::StringIO), file_path: ::String).returns(T.untyped) }
   def call(io:, file_path: T.unsafe(nil)); end
 
-  # source://packwerk//lib/packwerk/parsers/erb.rb#25
+  # source://packwerk//lib/packwerk/parsers/erb.rb#30
+  sig { params(buffer: ::Parser::Source::Buffer, file_path: ::String).returns(T.nilable(::AST::Node)) }
   def parse_buffer(buffer, file_path:); end
 
   private
 
   # @yield [node]
   #
-  # source://packwerk//lib/packwerk/parsers/erb.rb#52
+  # source://packwerk//lib/packwerk/parsers/erb.rb#71
+  sig do
+    params(
+      node: T.nilable(T.any(::AST::Node, ::String)),
+      block: T.nilable(T.proc.params(arg0: ::AST::Node).void)
+    ).returns(T.nilable(T.any(T::Array[::String], T::Enumerator[::AST::Node])))
+  end
   def code_nodes(node, &block); end
 
-  # source://packwerk//lib/packwerk/parsers/erb.rb#38
+  # source://packwerk//lib/packwerk/parsers/erb.rb#49
+  sig { params(erb_ast: T.all(::AST::Node, ::Object), file_path: ::String).returns(T.nilable(::AST::Node)) }
   def to_ruby_ast(erb_ast, file_path); end
 end
 
@@ -1596,13 +1613,19 @@ class Packwerk::Parsers::Factory
   include ::Singleton
   extend ::Singleton::SingletonClassMethods
 
-  # source://packwerk//lib/packwerk/parsers/factory.rb#33
+  # source://packwerk//lib/packwerk/parsers/factory.rb#24
+  sig { void }
+  def initialize; end
+
+  # source://packwerk//lib/packwerk/parsers/factory.rb#41
+  sig { returns(::Class) }
   def erb_parser_class; end
 
-  # source://packwerk//lib/packwerk/parsers/factory.rb#37
+  # source://packwerk//lib/packwerk/parsers/factory.rb#46
+  sig { params(klass: T.nilable(::Class)).void }
   def erb_parser_class=(klass); end
 
-  # source://packwerk//lib/packwerk/parsers/factory.rb#24
+  # source://packwerk//lib/packwerk/parsers/factory.rb#31
   sig { params(path: ::String).returns(T.nilable(::Packwerk::Parsers::ParserInterface)) }
   def for_path(path); end
 end
@@ -1640,7 +1663,7 @@ module Packwerk::Parsers::ParserInterface
   # @abstract
   #
   # source://packwerk//lib/packwerk/parsers/parser_interface.rb#15
-  sig { abstract.params(io: ::File, file_path: ::String).returns(T.untyped) }
+  sig { abstract.params(io: T.any(::IO, ::StringIO), file_path: ::String).returns(T.untyped) }
   def call(io:, file_path:); end
 end
 
@@ -1648,26 +1671,26 @@ end
 class Packwerk::Parsers::Ruby
   include ::Packwerk::Parsers::ParserInterface
 
-  # @return [Ruby] a new instance of Ruby
-  #
-  # source://packwerk//lib/packwerk/parsers/ruby.rb#25
+  # source://packwerk//lib/packwerk/parsers/ruby.rb#34
+  sig { params(parser_class: T.untyped).void }
   def initialize(parser_class: T.unsafe(nil)); end
 
-  # source://packwerk//lib/packwerk/parsers/ruby.rb#30
+  # source://packwerk//lib/packwerk/parsers/ruby.rb#40
+  sig { override.params(io: T.any(::IO, ::StringIO), file_path: ::String).returns(T.nilable(::Parser::AST::Node)) }
   def call(io:, file_path: T.unsafe(nil)); end
 end
 
-# source://packwerk//lib/packwerk/parsers/ruby.rb#12
+# source://packwerk//lib/packwerk/parsers/ruby.rb#14
 class Packwerk::Parsers::Ruby::RaiseExceptionsParser < ::Parser::Ruby27
-  # @return [RaiseExceptionsParser] a new instance of RaiseExceptionsParser
-  #
-  # source://packwerk//lib/packwerk/parsers/ruby.rb#13
+  # source://packwerk//lib/packwerk/parsers/ruby.rb#18
+  sig { params(builder: T.untyped).void }
   def initialize(builder); end
 end
 
-# source://packwerk//lib/packwerk/parsers/ruby.rb#19
+# source://packwerk//lib/packwerk/parsers/ruby.rb#24
 class Packwerk::Parsers::Ruby::TolerateInvalidUtf8Builder < ::Parser::Builders::Default
-  # source://packwerk//lib/packwerk/parsers/ruby.rb#20
+  # source://packwerk//lib/packwerk/parsers/ruby.rb#28
+  sig { params(token: T.untyped).returns(T.untyped) }
   def string_value(token); end
 end
 
@@ -1738,12 +1761,12 @@ class Packwerk::Reference < ::Struct
   end
 end
 
-# source://packwerk//lib/packwerk.rb#79
+# source://packwerk//lib/packwerk.rb#96
 module Packwerk::ReferenceChecking
   extend ::ActiveSupport::Autoload
 end
 
-# source://packwerk//lib/packwerk.rb#84
+# source://packwerk//lib/packwerk.rb#101
 module Packwerk::ReferenceChecking::Checkers
   extend ::ActiveSupport::Autoload
 end
@@ -1805,7 +1828,7 @@ class Packwerk::ReferenceExtractor
   end
   def initialize(constant_name_inspectors:, root_node:, root_path:); end
 
-  # source://packwerk//lib/packwerk/reference_extractor.rb#76
+  # source://packwerk//lib/packwerk/reference_extractor.rb#79
   sig do
     params(
       node: ::Parser::AST::Node,
@@ -1817,12 +1840,17 @@ class Packwerk::ReferenceExtractor
 
   private
 
-  # @return [Boolean]
-  #
-  # source://packwerk//lib/packwerk/reference_extractor.rb#120
+  # source://packwerk//lib/packwerk/reference_extractor.rb#130
+  sig do
+    params(
+      constant_name: ::String,
+      name_location: T.nilable(::Packwerk::Node::Location),
+      namespace_path: T::Array[::String]
+    ).returns(T::Boolean)
+  end
   def local_reference?(constant_name, name_location, namespace_path); end
 
-  # source://packwerk//lib/packwerk/reference_extractor.rb#105
+  # source://packwerk//lib/packwerk/reference_extractor.rb#108
   sig do
     params(
       constant_name: ::String,
@@ -1869,17 +1897,6 @@ class Packwerk::ReferenceOffense < ::Packwerk::Offense
   def violation_type; end
 end
 
-# source://packwerk//lib/packwerk/result.rb#5
-class Packwerk::Result < ::T::Struct
-  const :message, ::String
-  const :status, T::Boolean
-
-  class << self
-    # source://sorbet-runtime/0.5.10588/lib/types/struct.rb#13
-    def inherited(s); end
-  end
-end
-
 # Holds the context of a Packwerk run across multiple files.
 #
 # source://packwerk//lib/packwerk/run_context.rb#8
@@ -2011,6 +2028,8 @@ Packwerk::VERSION = T.let(T.unsafe(nil), String)
 #
 # source://packwerk//lib/packwerk/validator.rb#9
 module Packwerk::Validator
+  extend ::ActiveSupport::Autoload
+
   abstract!
 
   # @abstract
@@ -2021,18 +2040,18 @@ module Packwerk::Validator
       .params(
         package_set: Packwerk::PackageSet,
         configuration: ::Packwerk::Configuration
-      ).returns(::Packwerk::ApplicationValidator::Result)
+      ).returns(::Packwerk::Validator::Result)
   end
   def call(package_set, configuration); end
 
   # source://packwerk//lib/packwerk/validator.rb#67
   sig do
     params(
-      results: T::Array[::Packwerk::ApplicationValidator::Result],
+      results: T::Array[::Packwerk::Validator::Result],
       separator: ::String,
       before_errors: ::String,
       after_errors: ::String
-    ).returns(::Packwerk::ApplicationValidator::Result)
+    ).returns(::Packwerk::Validator::Result)
   end
   def merge_results(results, separator: T.unsafe(nil), before_errors: T.unsafe(nil), after_errors: T.unsafe(nil)); end
 
@@ -2074,6 +2093,21 @@ module Packwerk::Validator
   end
 end
 
+# source://packwerk//lib/packwerk/validator/result.rb#6
+class Packwerk::Validator::Result < ::T::Struct
+  const :ok, T::Boolean
+  const :error_value, T.nilable(::String)
+
+  # source://packwerk//lib/packwerk/validator/result.rb#13
+  sig { returns(T::Boolean) }
+  def ok?; end
+
+  class << self
+    # source://sorbet-runtime/0.5.10588/lib/types/struct.rb#13
+    def inherited(s); end
+  end
+end
+
 # source://packwerk//lib/packwerk/validators/dependency_validator.rb#5
 module Packwerk::Validators; end
 
@@ -2087,7 +2121,7 @@ class Packwerk::Validators::DependencyValidator
       .params(
         package_set: Packwerk::PackageSet,
         configuration: ::Packwerk::Configuration
-      ).returns(::Packwerk::ApplicationValidator::Result)
+      ).returns(::Packwerk::Validator::Result)
   end
   def call(package_set, configuration); end
 
@@ -2110,15 +2144,15 @@ class Packwerk::Validators::DependencyValidator
   def build_cycle_strings(cycles); end
 
   # source://packwerk//lib/packwerk/validators/dependency_validator.rb#66
-  sig { params(package_set: Packwerk::PackageSet).returns(::Packwerk::ApplicationValidator::Result) }
+  sig { params(package_set: Packwerk::PackageSet).returns(::Packwerk::Validator::Result) }
   def check_acyclic_graph(package_set); end
 
   # source://packwerk//lib/packwerk/validators/dependency_validator.rb#34
-  sig { params(configuration: ::Packwerk::Configuration).returns(::Packwerk::ApplicationValidator::Result) }
+  sig { params(configuration: ::Packwerk::Configuration).returns(::Packwerk::Validator::Result) }
   def check_package_manifest_syntax(configuration); end
 
   # source://packwerk//lib/packwerk/validators/dependency_validator.rb#92
-  sig { params(configuration: ::Packwerk::Configuration).returns(::Packwerk::ApplicationValidator::Result) }
+  sig { params(configuration: ::Packwerk::Configuration).returns(::Packwerk::Validator::Result) }
   def check_valid_package_dependencies(configuration); end
 
   # source://packwerk//lib/packwerk/validators/dependency_validator.rb#128

data/sorbet/rbi/gems/parse_packwerk@0.16.0.rbi

@@ -0,0 +1,224 @@
+# typed: true
+
+# DO NOT EDIT MANUALLY
+# This is an autogenerated file for types exported from the `parse_packwerk` gem.
+# Please instead update this file by running `bin/tapioca gem parse_packwerk`.
+
+# source://parse_packwerk//lib/parse_packwerk/constants.rb#3
+module ParsePackwerk
+  class << self
+    # source://parse_packwerk//lib/parse_packwerk.rb#28
+    sig { returns(T::Array[::ParsePackwerk::Package]) }
+    def all; end
+
+    # source://parse_packwerk//lib/parse_packwerk.rb#112
+    sig { void }
+    def bust_cache!; end
+
+    # source://parse_packwerk//lib/parse_packwerk.rb#33
+    sig { params(name: ::String).returns(T.nilable(::ParsePackwerk::Package)) }
+    def find(name); end
+
+    # source://parse_packwerk//lib/parse_packwerk.rb#43
+    sig { params(file_path: T.any(::Pathname, ::String)).returns(::ParsePackwerk::Package) }
+    def package_from_path(file_path); end
+
+    # source://parse_packwerk//lib/parse_packwerk.rb#54
+    sig { params(package: ::ParsePackwerk::Package).void }
+    def write_package_yml!(package); end
+
+    # source://parse_packwerk//lib/parse_packwerk.rb#38
+    sig { returns(::ParsePackwerk::Configuration) }
+    def yml; end
+
+    private
+
+    # We memoize packages_by_name for fast lookup.
+    # Since Graph is an immutable value object, we can create indexes and general caching mechanisms safely.
+    #
+    # source://parse_packwerk//lib/parse_packwerk.rb#100
+    sig { returns(T::Hash[::String, ::ParsePackwerk::Package]) }
+    def packages_by_name; end
+  end
+end
+
+# source://parse_packwerk//lib/parse_packwerk/configuration.rb#4
+class ParsePackwerk::Configuration < ::T::Struct
+  const :exclude, T::Array[::String]
+  const :package_paths, T::Array[::String]
+
+  class << self
+    # source://parse_packwerk//lib/parse_packwerk/configuration.rb#28
+    sig { params(config_hash: T::Hash[T.untyped, T.untyped]).returns(T::Array[::String]) }
+    def excludes(config_hash); end
+
+    # source://parse_packwerk//lib/parse_packwerk/configuration.rb#11
+    sig { returns(::ParsePackwerk::Configuration) }
+    def fetch; end
+
+    # source://sorbet-runtime/0.5.10588/lib/types/struct.rb#13
+    def inherited(s); end
+
+    # source://parse_packwerk//lib/parse_packwerk/configuration.rb#40
+    sig { params(config_hash: T::Hash[T.untyped, T.untyped]).returns(T::Array[::String]) }
+    def package_paths(config_hash); end
+  end
+end
+
+# source://parse_packwerk//lib/parse_packwerk/constants.rb#22
+ParsePackwerk::DEFAULT_EXCLUDE_GLOBS = T.let(T.unsafe(nil), Array)
+
+# source://parse_packwerk//lib/parse_packwerk/constants.rb#23
+ParsePackwerk::DEFAULT_PACKAGE_PATHS = T.let(T.unsafe(nil), Array)
+
+# source://parse_packwerk//lib/parse_packwerk/constants.rb#24
+ParsePackwerk::DEFAULT_PUBLIC_PATH = T.let(T.unsafe(nil), String)
+
+# source://parse_packwerk//lib/parse_packwerk/constants.rb#14
+ParsePackwerk::DEPENDENCIES = T.let(T.unsafe(nil), String)
+
+# source://parse_packwerk//lib/parse_packwerk/constants.rb#10
+ParsePackwerk::DEPENDENCY_VIOLATION_TYPE = T.let(T.unsafe(nil), String)
+
+# source://parse_packwerk//lib/parse_packwerk/constants.rb#8
+ParsePackwerk::ENFORCE_DEPENDENCIES = T.let(T.unsafe(nil), String)
+
+# source://parse_packwerk//lib/parse_packwerk/constants.rb#9
+ParsePackwerk::ENFORCE_PRIVACY = T.let(T.unsafe(nil), String)
+
+# source://parse_packwerk//lib/parse_packwerk/constants.rb#13
+ParsePackwerk::METADATA = T.let(T.unsafe(nil), String)
+
+# Since this metadata is unstructured YAML, it could be any type. We leave it to clients of `ParsePackwerk::Package`
+# to add types based on their known usage of metadata.
+#
+# source://parse_packwerk//lib/parse_packwerk/constants.rb#18
+ParsePackwerk::MetadataYmlType = T.type_alias { T::Hash[T.untyped, T.untyped] }
+
+# source://parse_packwerk//lib/parse_packwerk.rb#14
+class ParsePackwerk::MissingConfiguration < ::StandardError
+  # source://parse_packwerk//lib/parse_packwerk.rb#18
+  sig { params(packwerk_file_name: ::Pathname).void }
+  def initialize(packwerk_file_name); end
+end
+
+# source://parse_packwerk//lib/parse_packwerk/constants.rb#7
+ParsePackwerk::PACKAGE_TODO_YML_NAME = T.let(T.unsafe(nil), String)
+
+# source://parse_packwerk//lib/parse_packwerk/constants.rb#5
+ParsePackwerk::PACKAGE_YML_NAME = T.let(T.unsafe(nil), String)
+
+# source://parse_packwerk//lib/parse_packwerk/constants.rb#6
+ParsePackwerk::PACKWERK_YML_NAME = T.let(T.unsafe(nil), String)
+
+# source://parse_packwerk//lib/parse_packwerk/constants.rb#11
+ParsePackwerk::PRIVACY_VIOLATION_TYPE = T.let(T.unsafe(nil), String)
+
+# source://parse_packwerk//lib/parse_packwerk/constants.rb#12
+ParsePackwerk::PUBLIC_PATH = T.let(T.unsafe(nil), String)
+
+# source://parse_packwerk//lib/parse_packwerk/package.rb#4
+class ParsePackwerk::Package < ::T::Struct
+  const :name, ::String
+  const :enforce_dependencies, T::Boolean
+  const :enforce_privacy, T::Boolean
+  const :public_path, ::String, default: T.unsafe(nil)
+  const :metadata, T::Hash[T.untyped, T.untyped]
+  const :dependencies, T::Array[::String]
+
+  # source://parse_packwerk//lib/parse_packwerk/package.rb#35
+  sig { returns(::Pathname) }
+  def directory; end
+
+  # source://parse_packwerk//lib/parse_packwerk/package.rb#45
+  sig { returns(T::Boolean) }
+  def enforces_dependencies?; end
+
+  # source://parse_packwerk//lib/parse_packwerk/package.rb#50
+  sig { returns(T::Boolean) }
+  def enforces_privacy?; end
+
+  # source://parse_packwerk//lib/parse_packwerk/package.rb#40
+  sig { returns(::Pathname) }
+  def public_directory; end
+
+  # source://parse_packwerk//lib/parse_packwerk/package.rb#55
+  sig { returns(T::Array[::ParsePackwerk::Violation]) }
+  def violations; end
+
+  # source://parse_packwerk//lib/parse_packwerk/package.rb#30
+  sig { returns(::Pathname) }
+  def yml; end
+
+  class << self
+    # source://parse_packwerk//lib/parse_packwerk/package.rb#15
+    sig { params(pathname: ::Pathname).returns(::ParsePackwerk::Package) }
+    def from(pathname); end
+
+    # source://sorbet-runtime/0.5.10588/lib/types/struct.rb#13
+    def inherited(s); end
+  end
+end
+
+# source://parse_packwerk//lib/parse_packwerk/package_set.rb#8
+class ParsePackwerk::PackageSet
+  class << self
+    # source://parse_packwerk//lib/parse_packwerk/package_set.rb#12
+    sig do
+      params(
+        package_pathspec: T::Array[::String],
+        exclude_pathspec: T::Array[::String]
+      ).returns(T::Array[::ParsePackwerk::Package])
+    end
+    def from(package_pathspec:, exclude_pathspec:); end
+
+    private
+
+    # source://parse_packwerk//lib/parse_packwerk/package_set.rb#28
+    sig { params(globs: T::Array[::String], path: ::Pathname).returns(T::Boolean) }
+    def exclude_path?(globs, path); end
+  end
+end
+
+# source://parse_packwerk//lib/parse_packwerk/package_todo.rb#4
+class ParsePackwerk::PackageTodo < ::T::Struct
+  const :pathname, ::Pathname
+  const :violations, T::Array[::ParsePackwerk::Violation]
+
+  class << self
+    # source://parse_packwerk//lib/parse_packwerk/package_todo.rb#11
+    sig { params(package: ::ParsePackwerk::Package).returns(::ParsePackwerk::PackageTodo) }
+    def for(package); end
+
+    # source://parse_packwerk//lib/parse_packwerk/package_todo.rb#17
+    sig { params(pathname: ::Pathname).returns(::ParsePackwerk::PackageTodo) }
+    def from(pathname); end
+
+    # source://sorbet-runtime/0.5.10588/lib/types/struct.rb#13
+    def inherited(s); end
+  end
+end
+
+# source://parse_packwerk//lib/parse_packwerk/constants.rb#4
+ParsePackwerk::ROOT_PACKAGE_NAME = T.let(T.unsafe(nil), String)
+
+# source://parse_packwerk//lib/parse_packwerk/violation.rb#4
+class ParsePackwerk::Violation < ::T::Struct
+  const :type, ::String
+  const :to_package_name, ::String
+  const :class_name, ::String
+  const :files, T::Array[::String]
+
+  # source://parse_packwerk//lib/parse_packwerk/violation.rb#13
+  sig { returns(T::Boolean) }
+  def dependency?; end
+
+  # source://parse_packwerk//lib/parse_packwerk/violation.rb#18
+  sig { returns(T::Boolean) }
+  def privacy?; end
+
+  class << self
+    # source://sorbet-runtime/0.5.10588/lib/types/struct.rb#13
+    def inherited(s); end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: danger-packwerk
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.11.0
 platform: ruby
 authors:
 - Gusto Engineers
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-12-14 00:00:00.000000000 Z
+date: 2022-12-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: code_ownership
@@ -209,7 +209,6 @@ files:
 - lib/danger-packwerk/packwerk_wrapper.rb
 - lib/danger-packwerk/private.rb
 - lib/danger-packwerk/private/ownership_information.rb
-- lib/danger-packwerk/private/package_todo.rb
 - lib/danger-packwerk/update/default_formatter.rb
 - lib/danger-packwerk/update/offenses_formatter.rb
 - lib/danger-packwerk/version.rb
@@ -261,9 +260,9 @@ files:
 - sorbet/rbi/gems/nokogiri@1.13.8.rbi
 - sorbet/rbi/gems/octokit@5.6.1.rbi
 - sorbet/rbi/gems/open4@1.3.4.rbi
-- sorbet/rbi/gems/packwerk@2.2.1-7e8e7a50705833b41fe40c201eadb85ff9a1a422.rbi
+- sorbet/rbi/gems/packwerk@2.2.1-8476bb3cd5452765ad452d36aa45ae724f1d44f5.rbi
 - sorbet/rbi/gems/parallel@1.22.1.rbi
-- sorbet/rbi/gems/parse_packwerk@0.12.1.rbi
+- sorbet/rbi/gems/parse_packwerk@0.16.0.rbi
 - sorbet/rbi/gems/parser@3.1.2.1.rbi
 - sorbet/rbi/gems/pry@0.14.1.rbi
 - sorbet/rbi/gems/public_suffix@5.0.0.rbi

data/lib/danger-packwerk/private/package_todo.rb

@@ -1,65 +0,0 @@
-# typed: strict
-
-module DangerPackwerk
-  module Private
-    #
-    # The `Violation` and `PackageTodo` classes come from Gusto's private `ParsePackwerk` gem.
-    # Until we decide to open source that gem, we inline these as a private implementation detail of `DangerPackwerk` for now.
-    #
-    class Violation < T::Struct
-      extend T::Sig
-
-      const :type, String
-      const :to_package_name, String
-      const :class_name, String
-      const :files, T::Array[String]
-
-      sig { returns(T::Boolean) }
-      def dependency?
-        type == 'dependency'
-      end
-
-      sig { returns(T::Boolean) }
-      def privacy?
-        type == 'privacy'
-      end
-    end
-
-    class PackageTodo < T::Struct
-      extend T::Sig
-
-      const :pathname, Pathname
-      const :violations, T::Array[Violation]
-
-      sig { params(pathname: Pathname).returns(PackageTodo) }
-      def self.from(pathname)
-        if pathname.exist?
-          package_todo_loaded_yml = YAML.load_file(pathname)
-
-          all_violations = []
-          package_todo_loaded_yml&.each_key do |to_package_name|
-            package_todo_per_package = package_todo_loaded_yml[to_package_name]
-            package_todo_per_package.each_key do |class_name|
-              symbol_usage = package_todo_per_package[class_name]
-              files = symbol_usage['files']
-              violations = symbol_usage['violations']
-              all_violations << Violation.new(type: 'dependency', to_package_name: to_package_name, class_name: class_name, files: files) if violations.include? 'dependency'
-
-              all_violations << Violation.new(type: 'privacy', to_package_name: to_package_name, class_name: class_name, files: files) if violations.include? 'privacy'
-            end
-          end
-
-          new(
-            pathname: pathname.cleanpath,
-            violations: all_violations
-          )
-        else
-          new(
-            pathname: pathname.cleanpath,
-            violations: []
-          )
-        end
-      end
-    end
-  end
-end

data/sorbet/rbi/gems/parse_packwerk@0.12.1.rbi

@@ -1,146 +0,0 @@
-# typed: true
-
-# DO NOT EDIT MANUALLY
-# This is an autogenerated file for types exported from the `parse_packwerk` gem.
-# Please instead update this file by running `bin/tapioca gem parse_packwerk`.
-
-module ParsePackwerk
-  class << self
-    sig { returns(T::Array[::ParsePackwerk::Package]) }
-    def all; end
-
-    sig { void }
-    def bust_cache!; end
-
-    sig { params(name: ::String).returns(T.nilable(::ParsePackwerk::Package)) }
-    def find(name); end
-
-    sig { params(file_path: T.any(::Pathname, ::String)).returns(::ParsePackwerk::Package) }
-    def package_from_path(file_path); end
-
-    sig { params(package: ::ParsePackwerk::Package).void }
-    def write_package_yml!(package); end
-
-    sig { returns(::ParsePackwerk::Configuration) }
-    def yml; end
-
-    private
-
-    sig { returns(T::Hash[::String, ::ParsePackwerk::Package]) }
-    def packages_by_name; end
-  end
-end
-
-class ParsePackwerk::Configuration < ::T::Struct
-  const :exclude, T::Array[::String]
-  const :package_paths, T::Array[::String]
-
-  class << self
-    sig { params(config_hash: T::Hash[T.untyped, T.untyped]).returns(T::Array[::String]) }
-    def excludes(config_hash); end
-
-    sig { returns(::ParsePackwerk::Configuration) }
-    def fetch; end
-
-    def inherited(s); end
-
-    sig { params(config_hash: T::Hash[T.untyped, T.untyped]).returns(T::Array[::String]) }
-    def package_paths(config_hash); end
-  end
-end
-
-ParsePackwerk::DEFAULT_EXCLUDE_GLOBS = T.let(T.unsafe(nil), Array)
-ParsePackwerk::DEFAULT_PACKAGE_PATHS = T.let(T.unsafe(nil), Array)
-ParsePackwerk::DEPENDENCIES = T.let(T.unsafe(nil), String)
-ParsePackwerk::DEPRECATED_REFERENCES_YML_NAME = T.let(T.unsafe(nil), String)
-
-class ParsePackwerk::DeprecatedReferences < ::T::Struct
-  const :pathname, ::Pathname
-  const :violations, T::Array[::ParsePackwerk::Violation]
-
-  class << self
-    sig { params(package: ::ParsePackwerk::Package).returns(::ParsePackwerk::DeprecatedReferences) }
-    def for(package); end
-
-    sig { params(pathname: ::Pathname).returns(::ParsePackwerk::DeprecatedReferences) }
-    def from(pathname); end
-
-    def inherited(s); end
-  end
-end
-
-ParsePackwerk::ENFORCE_DEPENDENCIES = T.let(T.unsafe(nil), String)
-ParsePackwerk::ENFORCE_PRIVACY = T.let(T.unsafe(nil), String)
-ParsePackwerk::METADATA = T.let(T.unsafe(nil), String)
-ParsePackwerk::MetadataYmlType = T.type_alias { T::Hash[T.untyped, T.untyped] }
-
-class ParsePackwerk::MissingConfiguration < ::StandardError
-  sig { params(packwerk_file_name: ::Pathname).void }
-  def initialize(packwerk_file_name); end
-end
-
-ParsePackwerk::PACKAGE_YML_NAME = T.let(T.unsafe(nil), String)
-ParsePackwerk::PACKWERK_YML_NAME = T.let(T.unsafe(nil), String)
-
-class ParsePackwerk::Package < ::T::Struct
-  const :dependencies, T::Array[::String]
-  const :enforce_dependencies, T::Boolean
-  const :enforce_privacy, T::Boolean
-  const :metadata, T::Hash[T.untyped, T.untyped]
-  const :name, ::String
-
-  sig { returns(::Pathname) }
-  def directory; end
-
-  sig { returns(T::Boolean) }
-  def enforces_dependencies?; end
-
-  sig { returns(T::Boolean) }
-  def enforces_privacy?; end
-
-  sig { returns(::Pathname) }
-  def yml; end
-
-  class << self
-    sig { params(pathname: ::Pathname).returns(::ParsePackwerk::Package) }
-    def from(pathname); end
-
-    def inherited(s); end
-  end
-end
-
-class ParsePackwerk::PackageSet
-  class << self
-    sig do
-      params(
-        package_pathspec: T::Array[::String],
-        exclude_pathspec: T::Array[::String]
-      ).returns(T::Array[::ParsePackwerk::Package])
-    end
-    def from(package_pathspec:, exclude_pathspec:); end
-
-    private
-
-    sig { params(globs: T::Array[::String], path: ::Pathname).returns(T::Boolean) }
-    def exclude_path?(globs, path); end
-  end
-end
-
-ParsePackwerk::ROOT_PACKAGE_NAME = T.let(T.unsafe(nil), String)
-
-class ParsePackwerk::Violation < ::T::Struct
-  const :class_name, ::String
-  const :files, T::Array[::String]
-  const :to_package_name, ::String
-  const :type, ::String
-
-  sig { returns(T::Boolean) }
-  def dependency?; end
-
-  sig { returns(T::Boolean) }
-  def privacy?; end
-
-  class << self
-    def inherited(s); end
-  end
-end