danger-packwerk 0.1.3 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e8ad610d58f87b5c05d4f55688a43be6fe566a24a987d29dbd43bc1397e42e2b
-  data.tar.gz: d3db9e2c065272e5d1a249b81edd5ab5b4ac253fd0281df5e8cc5f5fc4ce92a9
+  metadata.gz: 1a50f4eb29e8b1b964d365ccdc30749842efe091d659a5d62a7252d5af487fc2
+  data.tar.gz: ad82d90f8db9e0a416dff61ab76ab2a9084ccd39335e1a95e2b2c9c9b566cd18
 SHA512:
-  metadata.gz: d707148404ee53fa7ca6b7fceb595ed8e97a746e57b2f6b3ba93d6e7f8f7a435524c641c89884d3b5e243fb2f19f71119b17689429d77dd5a280efd4c212c854
-  data.tar.gz: 8927da7749c65108bc30cfa1790e2dd2eb620c38a3895c201fad152f72c420c76950a5b5ee3f6ba5329990cb2c339af2717ebd5fc19c604e412636526b9e3c00
+  metadata.gz: db4b2cb2d0c4a3cb8e24667fc60e32f2172abb33f486c0613a4977a73412d15f10136a977724257a707af203314c5569375bb43447848fc27eb8e2b20ee721ef
+  data.tar.gz: ceb901dc21ad848fd0fabcbb8b299f2d7a3e101d3d295a5fbbbbdd49b086206f196b5996e4bf64ea40deadae893a3add09fc84df1890e80f5086ad44fa7e0dcf

data/lib/danger-packwerk/danger_deprecated_references_yml_changes.rb

@@ -45,7 +45,14 @@ module DangerPackwerk
 
       current_comment_count = 0
 
-      violation_diff.added_violations.group_by(&:class_name).each do |_class_name, violations|
+      # The format for git.renamed_files is a T::Array[{after: "some/path/new", before: "some/path/old"}]
+      renamed_files = git.renamed_files.map { |before_after_file| before_after_file[:after] }
+
+      violations_to_comment_on = violation_diff.added_violations.reject do |violation|
+        renamed_files.include?(violation.file)
+      end
+
+      violations_to_comment_on.group_by(&:class_name).each do |_class_name, violations|
         break if current_comment_count >= max_comments
 
         location = T.must(violations.first).file_location

data/lib/danger-packwerk/danger_packwerk.rb

@@ -3,6 +3,7 @@
 
 require 'danger'
 require 'packwerk'
+require 'parse_packwerk'
 require 'sorbet-runtime'
 require 'danger-packwerk/packwerk_wrapper'
 
@@ -23,13 +24,23 @@ module DangerPackwerk
     DEFAULT_FAIL = false
     DEFAULT_FAILURE_MESSAGE = 'Packwerk violations were detected! Please resolve them to unblock the build.'
 
+    class CommentGroupingStrategy < ::T::Enum
+      enums do
+        PerConstantPerLocation = new
+        PerConstantPerPack = new
+      end
+    end
+
+    PerConstantPerPackGrouping = CommentGroupingStrategy::PerConstantPerPack
+
     sig do
       params(
         max_comments: Integer,
         offenses_formatter: OffensesFormatter,
         fail_build: T::Boolean,
         failure_message: String,
-        on_failure: OnFailure
+        on_failure: OnFailure,
+        grouping_strategy: CommentGroupingStrategy
       ).void
     end
     def check(
@@ -37,7 +48,8 @@ module DangerPackwerk
       offenses_formatter: DEFAULT_OFFENSES_FORMATTER,
       fail_build: DEFAULT_FAIL,
       failure_message: DEFAULT_FAILURE_MESSAGE,
-      on_failure: DEFAULT_ON_FAILURE
+      on_failure: DEFAULT_ON_FAILURE,
+      grouping_strategy: CommentGroupingStrategy::PerConstantPerLocation
     )
       # This is important because by default, Danger will leave a concantenated list of all its messages if it can't find a commentable place in the
       # diff to leave its message. This is an especially bad UX because it will be a huge wall of text not connected to the source of the issue.
@@ -75,11 +87,21 @@ module DangerPackwerk
       # 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
       # they are (privacy or dependency). We put privacy and dependency violation messages in the same comment since they would occur on the same line.
       packwerk_reference_offenses.group_by do |packwerk_reference_offense|
-        [
-          packwerk_reference_offense.reference.constant.name,
-          packwerk_reference_offense.location.line,
-          packwerk_reference_offense.reference.relative_path
-        ]
+        case grouping_strategy
+        when CommentGroupingStrategy::PerConstantPerLocation
+          [
+            packwerk_reference_offense.reference.constant.name,
+            packwerk_reference_offense.location.line,
+            packwerk_reference_offense.reference.relative_path
+          ]
+        when CommentGroupingStrategy::PerConstantPerPack
+          [
+            packwerk_reference_offense.reference.constant.name,
+            ParsePackwerk.package_from_path(packwerk_reference_offense.reference.relative_path)
+          ]
+        else
+          T.absurd(grouping_strategy)
+        end
       end.each do |_group, unique_packwerk_reference_offenses|
         break if current_comment_count >= max_comments
 

data/lib/danger-packwerk/version.rb

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

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

@@ -0,0 +1,146 @@
+# 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(T.nilable(::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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: danger-packwerk
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.4.0
 platform: ruby
 authors:
 - Gusto Engineers
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-04-25 00:00:00.000000000 Z
+date: 2022-09-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: danger-plugin-api
@@ -38,6 +38,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: parse_packwerk
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: sorbet-runtime
   requirement: !ruby/object:Gem::Requirement
@@ -186,15 +200,16 @@ files:
 - sorbet/config
 - sorbet/rbi/gems/danger@8.5.0.rbi
 - sorbet/rbi/gems/packwerk@1.3.2.rbi
+- sorbet/rbi/gems/parse_packwerk@0.12.0.rbi
 - sorbet/rbi/todo.rbi
 - sorbet/tapioca/require.rb
-homepage: https://github.com/bigrails/danger-packwerk
+homepage: https://github.com/rubyatscale/danger-packwerk
 licenses:
 - MIT
 metadata:
-  homepage_uri: https://github.com/bigrails/danger-packwerk
-  source_code_uri: https://github.com/bigrails/danger-packwerk
-  changelog_uri: https://github.com/bigrails/danger-packwerk/releases
+  homepage_uri: https://github.com/rubyatscale/danger-packwerk
+  source_code_uri: https://github.com/rubyatscale/danger-packwerk
+  changelog_uri: https://github.com/rubyatscale/danger-packwerk/releases
   allowed_push_host: https://rubygems.org
 post_install_message: 
 rdoc_options: []