code_ownership 2.1.1-x86_64-linux → 2.1.2-x86_64-linux

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: adae3a62f615f131fac8f1ecd05a6888cd12d32136c65ba1016640d56d5a08ea
-  data.tar.gz: 52aacacd5d5f190336353cc4fe4aca00fc8b79fc1c2de95e3cf791f3a69e5d1d
+  metadata.gz: 11870e7233492c859a41589bd1a85a43825e2fe4ab12279fe681df97ace60fa8
+  data.tar.gz: bb5e9e12b04cfa6bc8bc73e557066f04a9a00401dd0f9d462cc6b3b59b9db583
 SHA512:
-  metadata.gz: 581f3954eac7da7cf5079c9871bd653668873655c3fd24db3746ba2378291a6a36763e74faa47c93c63a67152f6cb8fea59c53bdec743f4bad57528f73f85e12
-  data.tar.gz: cbbf56062481c954cc32af8375a388bdd3e79d562f9d2d8069aa86e5984539d56c8285ae1ea56e7e964de004ce9fcdf08a2194521ef0e2d56841125201febf1b
+  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/private/file_path_finder.rb

@@ -6,13 +6,29 @@ module CodeOwnership
     module FilePathFinder
       extend T::Sig
 
+      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], 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
@@ -30,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'
@@ -38,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/team_finder.rb

@@ -13,9 +13,8 @@ module CodeOwnership
         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)))

data/lib/code_ownership/version.rb

@@ -2,5 +2,5 @@
 # frozen_string_literal: true
 
 module CodeOwnership
-  VERSION = '2.1.1'
+  VERSION = '2.1.2'
 end

data/lib/code_ownership.rb

@@ -315,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.1
+  version: 2.1.2
 platform: x86_64-linux
 authors:
 - Gusto Engineers
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2026-01-26 00:00:00.000000000 Z
+date: 2026-04-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: code_teams
@@ -176,7 +176,6 @@ 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
@@ -204,7 +203,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '3.2'
+      version: '3.3'
   - - "<"
     - !ruby/object:Gem::Version
       version: 4.1.dev