code_ownership 2.1.0-x86_64-linux → 2.1.2-x86_64-linux

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5eefa130f8ea4807d7330296ff516bf19dfba1da29b4a305494825701622287d
-  data.tar.gz: 2af8901f13bc6bc310efb541cf1ca40592d48e48d62faca7204df2fa16b05df9
+  metadata.gz: 11870e7233492c859a41589bd1a85a43825e2fe4ab12279fe681df97ace60fa8
+  data.tar.gz: bb5e9e12b04cfa6bc8bc73e557066f04a9a00401dd0f9d462cc6b3b59b9db583
 SHA512:
-  metadata.gz: f8c91fb0c2c5950de1b189b32eb5d36b1c4fd9d55c4c5e708a8070ddbd3899e89d1b2b1e19e4d37535a3f9e78538ca0c83f66a22e011e541e20f5f1dd17f2c18
-  data.tar.gz: fdc5d26e742254450fb022b21d8280ef05d1db863923a2b45dc82cbcc1d910cc4653e793c246a25162f8025e17f740339b6e75edfffd9a0bbca25d5a03b23edc
+  metadata.gz: beefff80a9b1d0dd3a618671ebd1fc9e6d74898c06be8dced72d45e852fe8e4c3759771d02b4edd5f86d3c7b5602d73e00f89478847f1400b10df6651e565bdf
+  data.tar.gz: d0aeb2d7c0653417cb969da695111a7ce54b8186e62a3c3618d7cd5a9510bbd29f27dfc85cc8eca3a67960d3e9d8ddc9c98f156d9d23e7fab8723104fcf94f43

data/README.md

@@ -10,7 +10,7 @@ There is also a [companion VSCode Extension](https://github.com/rubyatscale/code
 
 ## Getting started
 
-To get started there's a few things you should do.
+To get started there are a few things you should do.
 
 1. Create a `config/code_ownership.yml` file and declare where your files live. Here's a sample to start with:
 
@@ -42,7 +42,7 @@ There are five ways to declare code ownership using this gem:
 
 ### Directory-Based Ownership
 
-Directory based ownership allows for all files in that directory and all its sub-directories to be owned by one team. To define this, add a `.codeowner` file inside that directory with the name of the team as the contents of that file.
+Directory-based ownership allows for all files in that directory and all its sub-directories to be owned by one team. To define this, add a `.codeowner` file inside that directory with the name of the team as the contents of that file.
 
 ```
 Team
@@ -58,7 +58,7 @@ File annotations are a last resort if there is no clear home for your code. File
 
 ### Package-Based Ownership
 
-Package based ownership integrates [`packwerk`](https://github.com/Shopify/packwerk) and has ownership defined per package. To define that all files within a package are owned by one team, configure your `package.yml` like this:
+Package-based ownership integrates [`packwerk`](https://github.com/Shopify/packwerk) and has ownership defined per package. To define that all files within a package are owned by one team, configure your `package.yml` like this:
 
 ```yml
 enforce_dependency: true
@@ -92,7 +92,7 @@ unowned_globs:
 
 ### Javascript Package Ownership
 
-Javascript package based ownership allows you to specify an ownership key in a `package.json`. To use this, configure your `package.json` like this:
+JavaScript package-based ownership allows you to specify an ownership key in a `package.json`. To use this, configure your `package.json` like this:
 
 ```json
 {
@@ -112,10 +112,10 @@ js_package_paths:
   - frontend/other_location_for_packages/*
 ```
 
-This defaults `**/`, which makes it look for `package.json` files across your application.
+This defaults to `**/`, which makes it look for `package.json` files across your application.
 
 > [!NOTE]
-> Javscript package ownership does not respect `unowned_globs`. If you wish to disable usage of this feature you can set `js_package_paths` to an empty list.
+> JavaScript package ownership does not respect `unowned_globs`. If you wish to disable usage of this feature you can set `js_package_paths` to an empty list.
 
 ```yml
 js_package_paths: []
@@ -184,7 +184,7 @@ If `codeowners_path` is set in `code_ownership.yml` codeowners will use that pat
 CodeOwnership comes with a validation function to ensure the following things are true:
 
 1. Only one mechanism is defining file ownership. That is -- you can't have a file annotation on a file owned via package-based or glob-based ownership. This helps make ownership behavior more clear by avoiding concerns about precedence.
-2. All teams referenced as an owner for any file or package is a valid team (i.e. it's in the list of `CodeTeams.all`).
+2. All teams referenced as an owner for any file or package are valid teams (i.e. they're in the list of `CodeTeams.all`).
 3. All files have ownership. You can specify in `unowned_globs` to represent a TODO list of files to add ownership to.
 4. The `.github/CODEOWNERS` file is up to date. This is automatically corrected and staged unless specified otherwise with `bin/codeownership validate --skip-autocorrect --skip-stage`. You can turn this validation off by setting `skip_codeowners_validation: true` in `config/code_ownership.yml`.
 
@@ -223,7 +223,7 @@ codeownership validate --diff
 
 ## Development
 
-Please add to `CHANGELOG.md` and this `README.md` when you make make changes.
+Please add to `CHANGELOG.md` and this `README.md` when you make changes.
 
 ## Running specs
 

data/lib/code_ownership/cli.rb

@@ -1,4 +1,5 @@
 # typed: true
+# frozen_string_literal: true
 
 require 'optparse'
 require 'pathname'

data/lib/code_ownership/private/file_path_finder.rb

@@ -1,29 +1,41 @@
 # frozen_string_literal: true
-
 # typed: strict
 
 module CodeOwnership
   module Private
     module FilePathFinder
-      module_function
-
       extend T::Sig
-      extend T::Helpers
+
+      sig { returns(String) }
+      def self.pwd_prefix
+        @pwd_prefix ||= T.let("#{Dir.pwd}/", T.nilable(String))
+      end
+
+      sig { returns(Pathname) }
+      def self.pwd
+        @pwd ||= T.let(Pathname.pwd, T.nilable(Pathname))
+      end
 
       # Returns a string version of the relative path to a Rails constant,
       # or nil if it can't find anything
-      sig { params(klass: T.nilable(T.any(T::Class[T.anything], Module))).returns(T.nilable(String)) }
-      def path_from_klass(klass)
+      sig { params(klass: T.nilable(T.any(T::Class[T.anything], T::Module[T.anything]))).returns(T.nilable(String)) }
+      def self.path_from_klass(klass)
         if klass
           path = Object.const_source_location(klass.to_s)&.first
-          (path && Pathname.new(path).relative_path_from(Pathname.pwd).to_s) || nil
+          return nil unless path
+
+          if path.start_with?(pwd_prefix)
+            path.delete_prefix(pwd_prefix)
+          else
+            Pathname.new(path).relative_path_from(pwd).to_s
+          end
         end
       rescue NameError
         nil
       end
 
       sig { params(backtrace: T.nilable(T::Array[String])).returns(T::Enumerable[String]) }
-      def from_backtrace(backtrace)
+      def self.from_backtrace(backtrace)
         return [] unless backtrace
 
         # The pattern for a backtrace hasn't changed in forever and is considered
@@ -34,7 +46,7 @@ module CodeOwnership
         #   ./app/controllers/some_controller.rb:43:in `block (3 levels) in create'
         #
         backtrace_line = if RUBY_VERSION >= '3.4.0'
-                           %r{\A(#{Pathname.pwd}/|\./)?
+                           %r{\A(#{pwd}/|\./)?
                                (?<file>.+)       # Matches 'app/controllers/some_controller.rb'
                                :
                                (?<line>\d+)      # Matches '43'
@@ -42,7 +54,7 @@ module CodeOwnership
                                '(?<function>.*)' # Matches "`block (3 levels) in create'"
                              \z}x
                          else
-                           %r{\A(#{Pathname.pwd}/|\./)?
+                           %r{\A(#{pwd}/|\./)?
                                (?<file>.+)       # Matches 'app/controllers/some_controller.rb'
                                :
                                (?<line>\d+)      # Matches '43'

data/lib/code_ownership/private/file_path_team_cache.rb

@@ -1,37 +1,33 @@
 # frozen_string_literal: true
-
 # typed: strict
 
 module CodeOwnership
   module Private
     module FilePathTeamCache
-      module_function
-
       extend T::Sig
-      extend T::Helpers
 
       sig { params(file_path: String).returns(T.nilable(CodeTeams::Team)) }
-      def get(file_path)
+      def self.get(file_path)
         cache[file_path]
       end
 
       sig { params(file_path: String, team: T.nilable(CodeTeams::Team)).void }
-      def set(file_path, team)
+      def self.set(file_path, team)
         cache[file_path] = team
       end
 
       sig { params(file_path: String).returns(T::Boolean) }
-      def cached?(file_path)
+      def self.cached?(file_path)
         cache.key?(file_path)
       end
 
       sig { void }
-      def bust_cache!
+      def self.bust_cache!
         @cache = nil
       end
 
       sig { returns(T::Hash[String, T.nilable(CodeTeams::Team)]) }
-      def cache
+      def self.cache
         @cache ||= T.let(@cache,
           T.nilable(T::Hash[String, T.nilable(CodeTeams::Team)]))
         @cache ||= {}

data/lib/code_ownership/private/for_file_output_builder.rb

@@ -1,5 +1,4 @@
 # frozen_string_literal: true
-
 # typed: strict
 
 module CodeOwnership

data/lib/code_ownership/private/team_finder.rb

@@ -1,27 +1,20 @@
 # frozen_string_literal: true
-
 # typed: strict
 
 module CodeOwnership
   module Private
     module TeamFinder
-      module_function
-
       extend T::Sig
-      extend T::Helpers
-
-      requires_ancestor { Kernel }
 
       sig { params(file_path: String, allow_raise: T::Boolean).returns(T.nilable(CodeTeams::Team)) }
-      def for_file(file_path, allow_raise: false)
+      def self.for_file(file_path, allow_raise: false)
         return nil if file_path.start_with?('./')
 
         return FilePathTeamCache.get(file_path) if FilePathTeamCache.cached?(file_path)
 
         result = T.let(RustCodeOwners.for_file(file_path), T.nilable(T::Hash[Symbol, String]))
-        return if result.nil?
 
-        if result[:team_name].nil?
+        if result.nil? || result[:team_name].nil?
           FilePathTeamCache.set(file_path, nil)
         else
           FilePathTeamCache.set(file_path, T.let(find_team!(T.must(result[:team_name]), allow_raise: allow_raise), T.nilable(CodeTeams::Team)))
@@ -31,7 +24,7 @@ module CodeOwnership
       end
 
       sig { params(files: T::Array[String], allow_raise: T::Boolean).returns(T::Hash[String, T.nilable(CodeTeams::Team)]) }
-      def teams_for_files(files, allow_raise: false)
+      def self.teams_for_files(files, allow_raise: false)
         result = {}
 
         # Collect cached results and identify non-cached files
@@ -57,8 +50,8 @@ module CodeOwnership
         result
       end
 
-      sig { params(klass: T.nilable(T.any(T::Class[T.anything], Module))).returns(T.nilable(::CodeTeams::Team)) }
-      def for_class(klass)
+      sig { params(klass: T.nilable(T.any(T::Class[T.anything], T::Module[T.anything]))).returns(T.nilable(::CodeTeams::Team)) }
+      def self.for_class(klass)
         file_path = FilePathFinder.path_from_klass(klass)
         return nil if file_path.nil?
 
@@ -66,7 +59,7 @@ module CodeOwnership
       end
 
       sig { params(package: Packs::Pack).returns(T.nilable(::CodeTeams::Team)) }
-      def for_package(package)
+      def self.for_package(package)
         owner_name = package.raw_hash['owner'] || package.metadata['owner']
         return nil if owner_name.nil?
 
@@ -74,12 +67,12 @@ module CodeOwnership
       end
 
       sig { params(backtrace: T.nilable(T::Array[String]), excluded_teams: T::Array[::CodeTeams::Team]).returns(T.nilable(::CodeTeams::Team)) }
-      def for_backtrace(backtrace, excluded_teams: [])
+      def self.for_backtrace(backtrace, excluded_teams: [])
         first_owned_file_for_backtrace(backtrace, excluded_teams: excluded_teams)&.first
       end
 
       sig { params(backtrace: T.nilable(T::Array[String]), excluded_teams: T::Array[::CodeTeams::Team]).returns(T.nilable([::CodeTeams::Team, String])) }
-      def first_owned_file_for_backtrace(backtrace, excluded_teams: [])
+      def self.first_owned_file_for_backtrace(backtrace, excluded_teams: [])
         FilePathFinder.from_backtrace(backtrace).each do |file|
           team = for_file(file)
           if team && !excluded_teams.include?(team)
@@ -91,7 +84,7 @@ module CodeOwnership
       end
 
       sig { params(team_name: String, allow_raise: T::Boolean).returns(T.nilable(CodeTeams::Team)) }
-      def find_team!(team_name, allow_raise: false)
+      def self.find_team!(team_name, allow_raise: false)
         team = CodeTeams.find(team_name)
         if team.nil? && allow_raise
           raise(StandardError, "Could not find team with name: `#{team_name}`. Make sure the team is one of `#{CodeTeams.all.map(&:name).sort}`")

data/lib/code_ownership/version.rb

@@ -1,6 +1,6 @@
-# typed: false
+# typed: strict
 # frozen_string_literal: true
 
 module CodeOwnership
-  VERSION = '2.1.0'
+  VERSION = '2.1.2'
 end

data/lib/code_ownership.rb

@@ -1,5 +1,4 @@
 # frozen_string_literal: true
-
 # typed: strict
 
 require 'code_teams'
@@ -25,17 +24,13 @@ if defined?(Packwerk)
 end
 
 module CodeOwnership
-  module_function
-
   extend T::Sig
-  extend T::Helpers
 
-  requires_ancestor { Kernel }
   GlobsToOwningTeamMap = T.type_alias { T::Hash[String, CodeTeams::Team] }
 
   # Returns the version of the code_ownership gem and the codeowners-rs gem.
   sig { returns(T::Array[String]) }
-  def version
+  def self.version
     ["code_ownership version: #{VERSION}",
      "codeowners-rs version: #{::RustCodeOwners.version}"]
   end
@@ -65,7 +60,7 @@ module CodeOwnership
   #   # => raises exception if no owner found
   #
   sig { params(file: String, from_codeowners: T::Boolean, allow_raise: T::Boolean).returns(T.nilable(CodeTeams::Team)) }
-  def for_file(file, from_codeowners: true, allow_raise: false)
+  def self.for_file(file, from_codeowners: true, allow_raise: false)
     if from_codeowners
       teams_for_files_from_codeowners([file], allow_raise: allow_raise).values.first
     else
@@ -116,7 +111,7 @@ module CodeOwnership
   # @see #validate! for ensuring CODEOWNERS file is up-to-date
   #
   sig { params(files: T::Array[String], allow_raise: T::Boolean).returns(T::Hash[String, T.nilable(CodeTeams::Team)]) }
-  def teams_for_files_from_codeowners(files, allow_raise: false)
+  def self.teams_for_files_from_codeowners(files, allow_raise: false)
     Private::TeamFinder.teams_for_files(files, allow_raise: allow_raise)
   end
 
@@ -160,12 +155,12 @@ module CodeOwnership
   # @see CLI#for_file for the command-line interface that uses this method
   #
   sig { params(file: String).returns(T.nilable(T::Hash[Symbol, String])) }
-  def for_file_verbose(file)
+  def self.for_file_verbose(file)
     ::RustCodeOwners.for_file(file)
   end
 
   sig { params(team: T.any(CodeTeams::Team, String)).returns(T::Array[String]) }
-  def for_team(team)
+  def self.for_team(team)
     team = T.must(CodeTeams.find(team)) if team.is_a?(String)
     ::RustCodeOwners.for_team(team.name)
   end
@@ -226,7 +221,7 @@ module CodeOwnership
       files: T.nilable(T::Array[String])
     ).void
   end
-  def validate!(
+  def self.validate!(
     autocorrect: true,
     stage_changes: true,
     files: nil
@@ -269,7 +264,7 @@ module CodeOwnership
   # @note Leading newlines after the annotation are also removed to maintain clean formatting.
   #
   sig { params(filename: String).void }
-  def remove_file_annotation!(filename)
+  def self.remove_file_annotation!(filename)
     filepath = Pathname.new(filename)
 
     begin
@@ -292,24 +287,24 @@ module CodeOwnership
   # Given a backtrace from either `Exception#backtrace` or `caller`, find the
   # first line that corresponds to a file with assigned ownership
   sig { params(backtrace: T.nilable(T::Array[String]), excluded_teams: T::Array[::CodeTeams::Team]).returns(T.nilable(::CodeTeams::Team)) }
-  def for_backtrace(backtrace, excluded_teams: [])
+  def self.for_backtrace(backtrace, excluded_teams: [])
     Private::TeamFinder.for_backtrace(backtrace, excluded_teams: excluded_teams)
   end
 
   # Given a backtrace from either `Exception#backtrace` or `caller`, find the
   # first owned file in it, useful for figuring out which file is being blamed.
   sig { params(backtrace: T.nilable(T::Array[String]), excluded_teams: T::Array[::CodeTeams::Team]).returns(T.nilable([::CodeTeams::Team, String])) }
-  def first_owned_file_for_backtrace(backtrace, excluded_teams: [])
+  def self.first_owned_file_for_backtrace(backtrace, excluded_teams: [])
     Private::TeamFinder.first_owned_file_for_backtrace(backtrace, excluded_teams: excluded_teams)
   end
 
-  sig { params(klass: T.nilable(T.any(T::Class[T.anything], Module))).returns(T.nilable(::CodeTeams::Team)) }
-  def for_class(klass)
+  sig { params(klass: T.nilable(T.any(T::Class[T.anything], T::Module[T.anything]))).returns(T.nilable(::CodeTeams::Team)) }
+  def self.for_class(klass)
     Private::TeamFinder.for_class(klass)
   end
 
   sig { params(package: Packs::Pack).returns(T.nilable(::CodeTeams::Team)) }
-  def for_package(package)
+  def self.for_package(package)
     Private::TeamFinder.for_package(package)
   end
 
@@ -320,5 +315,7 @@ module CodeOwnership
   sig { void }
   def self.bust_caches!
     Private::FilePathTeamCache.bust_cache!
+    Private::FilePathFinder.instance_variable_set(:@pwd, nil)
+    Private::FilePathFinder.instance_variable_set(:@pwd_prefix, nil)
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: code_ownership
 version: !ruby/object:Gem::Version
-  version: 2.1.0
+  version: 2.1.2
 platform: x86_64-linux
 authors:
 - Gusto Engineers
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2025-11-13 00:00:00.000000000 Z
+date: 2026-04-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: code_teams
@@ -44,14 +44,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.5.11249
+        version: 0.6.12763
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.5.11249
+        version: 0.6.12763
 - !ruby/object:Gem::Dependency
   name: debug
   requirement: !ruby/object:Gem::Requirement
@@ -176,9 +176,9 @@ files:
 - README.md
 - bin/codeownership
 - lib/code_ownership.rb
-- lib/code_ownership/3.2/code_ownership.so
 - lib/code_ownership/3.3/code_ownership.so
 - lib/code_ownership/3.4/code_ownership.so
+- lib/code_ownership/4.0/code_ownership.so
 - lib/code_ownership/cli.rb
 - lib/code_ownership/private/file_path_finder.rb
 - lib/code_ownership/private/file_path_team_cache.rb
@@ -203,10 +203,10 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '3.2'
+      version: '3.3'
   - - "<"
     - !ruby/object:Gem::Version
-      version: 3.5.dev
+      version: 4.1.dev
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="