cleo_codeowners 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9ab9ff74ae21e83712d2331afa49e4a58fb361afb592489c608662d190adfaac
-  data.tar.gz: af9b4ec93a30218aef50319329d3594ca97cff39c4b203282abf7ab763ed968b
+  metadata.gz: 95dabb73fd9d44328c2c7baed5ffbf4f91996ef2bad026d2f79175b98ba252f9
+  data.tar.gz: 4a65dca6d6efef99d01587612451b72293283a30aa0a99f3ada7f192b05added
 SHA512:
-  metadata.gz: bcf8aebdba6312e71130c7ddcd9cf66f39c6e90c7b63d66fb57dd59d0410391938a676e895f9f82245543acfec10fa44acc4db30b49cdc794619c7bfa4cb0ee6
-  data.tar.gz: 87892f76e750c5ebbbead2dabf13ac1727e6c6fc1104e5ea237b06f4c22e05fb0038f6bbfe66bb73b1ae1a22384c2a756e162927f6fd114f9d405cd678f13601
+  metadata.gz: 55b1e75f9bfbf6affb08442adb631c5fb96ab8a7ac3a1a31a433fc44235f5875964a98d4d77038965cd7c20db7cf022c5a8989db3c7a1acdfe80f32a864c792a
+  data.tar.gz: c69046a48d695beb1129a005fff5b87163b0cd4935fbb625017a123b1bc2f1ed291ee02d833cf6a2ee4ef0448d14a388a2f5324547cb7fb5fe79bfbe680b3f98

data/lib/cleo_codeowners/version.rb

@@ -3,5 +3,5 @@
 ##
 # Root module for this gem
 module CleoCodeowners
-  VERSION = '0.3.0'
+  VERSION = '0.4.0'
 end

data/lib/codeowners/generator.rb

@@ -1,6 +1,8 @@
 # typed: false
 # frozen_string_literal: true
 
+require 'pathname'
+
 module Codeowners
   class Generator
     DEFAULT_OUTPUT_PATH = '.github/CODEOWNERS'
@@ -105,32 +107,40 @@ module Codeowners
       calculated_owners.push({ path:, owner: })
     end
 
+    # GitHub applies the *last* matching CODEOWNERS rule, so rules are listed
+    # least-specific first: by number of path segments (`/app/` is broader than
+    # `/app/models/user.rb`), then by path so a wildcard such as `/foo/*` sorts
+    # ahead of the `/foo/bar/` it would otherwise shadow.
     def output_in_machine_format
-      file_rules, folder_rules = calculated_owners.partition { |owner| owner[:path].include?('.') }
-      group_by_path_segments_and_output(folder_rules)
-      group_by_path_segments_and_output(file_rules)
+      merge_owners_by_path(calculated_owners)
+        .sort_by { |path, _owners| [path_depth(path), path] }
+        .each { |path, rule_owners| output_file << "#{path} #{rule_owners.join(' ')}\n" }
     end
 
-    def group_by_path_segments_and_output(rules)
-      grouped_by_path_segments_number = rules
-                                        .group_by { |owner| sanitised_path(owner[:path]).split('/').reject(&:empty?).size }
-                                        .each_value do |rule|
-        rule.sort_by! do |owner|
-          owner[:path]
-        end
+    # Combines the owners of rules that share a path onto one line.
+    def merge_owners_by_path(rules)
+      rules_by_path = rules.group_by { |owner| sanitised_path(owner[:path]) }
+      reject_trailing_slash_conflicts(rules_by_path.keys)
+      rules_by_path.map do |path, owners_for_path|
+        [path, owners_for_path.map { |owner| owner[:owner] }.uniq.sort]
       end
-                                                                                                                                        .sort
-                                                                                                                                        .to_h
-
-      grouped_by_path_segments_number
-        .each_value
-        .map { |files_and_owners| files_and_owners.group_by { |owner| owner[:path] } }
-        .each do |files_and_owners|
-          files_and_owners.each do |file, owners_list|
-            rule_owners = owners_list.map { |owner| owner[:owner] }.uniq.sort
-            output_file << "#{sanitised_path(file)} #{rule_owners.join(' ')}\n"
-          end
-        end
+    end
+
+    # A path written both with and without a trailing slash is rejected rather
+    # than guessed at: `/foo` and `/foo/` are effectively same rule, but on disk
+    # they differ (a directory needs its slash to match recursively, a file must
+    # not have one), so the definitions must pick one.
+    def reject_trailing_slash_conflicts(paths)
+      paths.group_by { |path| path.chomp('/') }.each_value do |variants|
+        next if variants.uniq.size == 1
+
+        raise "#{variants.min} is declared both with and without a trailing slash " \
+              "(#{variants.uniq.sort.join(', ')}); pick one so it is clear whether it is a file or a directory."
+      end
+    end
+
+    def path_depth(path)
+      Pathname.new(path).each_filename.count
     end
 
     def blurb

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: cleo_codeowners
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - "@agentAngelope"