code_ownership 2.1.1 → 2.1.2

data/ext/cargo-vendor/codeowners-0.3.2/tests/validate_files_test.rs

@@ -0,0 +1,378 @@
+use assert_cmd::prelude::*;
+use predicates::prelude::*;
+use std::{error::Error, process::Command};
+
+mod common;
+
+use common::*;
+
+#[test]
+fn test_validate_with_owned_files() -> Result<(), Box<dyn Error>> {
+    run_codeowners(
+        "valid_project",
+        &["validate", "ruby/app/models/payroll.rb", "ruby/app/models/bank_account.rb"],
+        true,
+        OutputStream::Stdout,
+        predicate::eq(""),
+    )?;
+
+    Ok(())
+}
+
+#[test]
+fn test_validate_with_unowned_file() -> Result<(), Box<dyn Error>> {
+    run_codeowners(
+        "valid_project",
+        &["validate", "ruby/app/unowned.rb"],
+        false,
+        OutputStream::Stdout,
+        predicate::str::contains("ruby/app/unowned.rb").and(predicate::str::contains("Unowned")),
+    )?;
+
+    Ok(())
+}
+
+#[test]
+fn test_validate_with_mixed_files() -> Result<(), Box<dyn Error>> {
+    run_codeowners(
+        "valid_project",
+        &["validate", "ruby/app/models/payroll.rb", "ruby/app/unowned.rb"],
+        false,
+        OutputStream::Stdout,
+        predicate::str::contains("ruby/app/unowned.rb").and(predicate::str::contains("Unowned")),
+    )?;
+
+    Ok(())
+}
+
+#[test]
+fn test_validate_with_no_files() -> Result<(), Box<dyn Error>> {
+    // Existing behavior - validates entire project
+    run_codeowners("valid_project", &["validate"], true, OutputStream::Stdout, predicate::eq(""))?;
+
+    Ok(())
+}
+
+#[test]
+fn test_generate_and_validate_with_owned_files() -> Result<(), Box<dyn Error>> {
+    let fixture_root = std::path::Path::new("tests/fixtures/valid_project");
+    let temp_dir = setup_fixture_repo(fixture_root);
+    let project_root = temp_dir.path();
+    git_add_all_files(project_root);
+
+    let codeowners_path = project_root.join("tmp/CODEOWNERS");
+
+    Command::cargo_bin("codeowners")?
+        .arg("--project-root")
+        .arg(project_root)
+        .arg("--codeowners-file-path")
+        .arg(&codeowners_path)
+        .arg("--no-cache")
+        .arg("generate-and-validate")
+        .arg("ruby/app/models/payroll.rb")
+        .arg("ruby/app/models/bank_account.rb")
+        .assert()
+        .success();
+
+    Ok(())
+}
+
+#[test]
+fn test_generate_and_validate_with_unowned_file() -> Result<(), Box<dyn Error>> {
+    let fixture_root = std::path::Path::new("tests/fixtures/valid_project");
+    let temp_dir = setup_fixture_repo(fixture_root);
+    let project_root = temp_dir.path();
+    git_add_all_files(project_root);
+
+    let codeowners_path = project_root.join("tmp/CODEOWNERS");
+
+    Command::cargo_bin("codeowners")?
+        .arg("--project-root")
+        .arg(project_root)
+        .arg("--codeowners-file-path")
+        .arg(&codeowners_path)
+        .arg("--no-cache")
+        .arg("generate-and-validate")
+        .arg("ruby/app/unowned.rb")
+        .assert()
+        .failure()
+        .stdout(predicate::str::contains("ruby/app/unowned.rb"))
+        .stdout(predicate::str::contains("Unowned"));
+
+    Ok(())
+}
+
+#[test]
+fn test_validate_with_absolute_path() -> Result<(), Box<dyn Error>> {
+    let fixture_root = std::path::Path::new("tests/fixtures/valid_project");
+    let temp_dir = setup_fixture_repo(fixture_root);
+    let project_root = temp_dir.path();
+    git_add_all_files(project_root);
+
+    let file_absolute_path = project_root.join("ruby/app/models/payroll.rb").canonicalize()?;
+
+    Command::cargo_bin("codeowners")?
+        .arg("--project-root")
+        .arg(project_root)
+        .arg("--no-cache")
+        .arg("validate")
+        .arg(file_absolute_path.to_str().unwrap())
+        .assert()
+        .success();
+
+    Ok(())
+}
+
+#[test]
+fn test_validate_only_checks_codeowners_file() -> Result<(), Box<dyn Error>> {
+    // This test demonstrates that `validate` with files only checks the CODEOWNERS file
+    // It does NOT check file annotations or other ownership sources
+    //
+    // If a file has an annotation but is missing from CODEOWNERS, `validate` will report it as unowned
+    // This is why `generate-and-validate` should be used for accuracy
+
+    // ruby/app/models/bank_account.rb has @team Payments annotation and is in CODEOWNERS
+    run_codeowners(
+        "valid_project",
+        &["validate", "ruby/app/models/bank_account.rb"],
+        true,
+        OutputStream::Stdout,
+        predicate::eq(""),
+    )?;
+
+    Ok(())
+}
+
+#[test]
+fn test_validate_files_respects_owned_globs_with_excluded_extensions() -> Result<(), Box<dyn Error>> {
+    // Validates that files not matching owned_globs are silently skipped when
+    // validate is called with an explicit file list.
+    //
+    // valid_project owned_globs: "{gems,config,javascript,ruby,components}/**/*.{rb,tsx,erb}"
+    // .rbi files (Sorbet interface files) do NOT match this pattern and should be filtered.
+
+    // Setup: Create a temporary copy of valid_project fixture
+    let fixture_root = std::path::Path::new("tests/fixtures/valid_project");
+    let temp_dir = setup_fixture_repo(fixture_root);
+    let project_root = temp_dir.path();
+
+    // Create .rbi files (Sorbet interface files) that do NOT match owned_globs
+    // These files should be ignored by validate when specified in the file list
+    let bank_account_rbi = project_root.join("ruby/app/models/bank_account.rbi");
+    let payroll_rbi = project_root.join("ruby/app/models/payroll.rbi");
+
+    std::fs::write(
+        &bank_account_rbi,
+        "# typed: strict\n# RBI file for BankAccount\nclass BankAccount; end\n",
+    )?;
+    std::fs::write(&payroll_rbi, "# typed: strict\n# RBI file for Payroll\nclass Payroll; end\n")?;
+
+    git_add_all_files(project_root);
+
+    // Step 1: Generate CODEOWNERS
+    // This should ONLY include .rb files (not .rbi) because .rbi doesn't match owned_globs
+    let codeowners_path = project_root.join("tmp/CODEOWNERS");
+    Command::cargo_bin("codeowners")?
+        .arg("--project-root")
+        .arg(project_root)
+        .arg("--codeowners-file-path")
+        .arg(&codeowners_path)
+        .arg("--no-cache")
+        .arg("generate")
+        .assert()
+        .success();
+
+    // Verify: CODEOWNERS contains .rb files but NOT .rbi files
+    let codeowners_content = std::fs::read_to_string(&codeowners_path)?;
+    assert!(
+        codeowners_content.contains("bank_account.rb"),
+        "CODEOWNERS should contain .rb files (they match owned_globs)"
+    );
+    assert!(
+        !codeowners_content.contains("bank_account.rbi"),
+        "CODEOWNERS should NOT contain .rbi files (they don't match owned_globs)"
+    );
+
+    // Step 2: Run validate with BOTH .rb and .rbi files in the list.
+    // .rbi files should be silently filtered; only .rb files validated; command succeeds.
+    Command::cargo_bin("codeowners")?
+        .arg("--project-root")
+        .arg(project_root)
+        .arg("--codeowners-file-path")
+        .arg(&codeowners_path)
+        .arg("--no-cache")
+        .arg("validate")
+        // Mix .rb and .rbi files in the argument list
+        .arg("ruby/app/models/bank_account.rb") // Should be validated (matches owned_globs)
+        .arg("ruby/app/models/bank_account.rbi") // Should be SKIPPED (doesn't match)
+        .arg("ruby/app/models/payroll.rb") // Should be validated (matches owned_globs)
+        .arg("ruby/app/models/payroll.rbi") // Should be SKIPPED (doesn't match)
+        .assert()
+        .success()
+        .stdout(predicate::eq(""));
+
+    Ok(())
+}
+
+// ============================================================================
+// GLOB FILTERING TESTS: Verify validate with files respects owned_globs
+// ============================================================================
+//
+// These tests ensure that when validate is called with explicit file paths,
+// it correctly filters files based on owned_globs configuration. Files that
+// don't match owned_globs should be silently skipped, not reported as unowned.
+
+#[test]
+fn test_validate_filters_multiple_non_matching_extensions() -> Result<(), Box<dyn Error>> {
+    // Test that various file types not in owned_globs are filtered out
+    // valid_project owned_globs: "{gems,config,javascript,ruby,components}/**/*.{rb,tsx,erb}"
+    let fixture_root = std::path::Path::new("tests/fixtures/valid_project");
+    let temp_dir = setup_fixture_repo(fixture_root);
+    let project_root = temp_dir.path();
+
+    // Create files with extensions NOT in owned_globs
+    std::fs::write(project_root.join("ruby/app/models/test.rbi"), "# Sorbet RBI file")?;
+    std::fs::write(project_root.join("ruby/app/models/test.md"), "# Markdown doc")?;
+    std::fs::write(project_root.join("ruby/app/models/test.txt"), "Plain text")?;
+    std::fs::write(project_root.join("ruby/app/models/test.json"), "{}")?;
+
+    git_add_all_files(project_root);
+
+    let codeowners_path = project_root.join("tmp/CODEOWNERS");
+
+    // Generate CODEOWNERS (will only include .rb, .tsx, .erb files)
+    Command::cargo_bin("codeowners")?
+        .arg("--project-root")
+        .arg(project_root)
+        .arg("--codeowners-file-path")
+        .arg(&codeowners_path)
+        .arg("--no-cache")
+        .arg("generate")
+        .assert()
+        .success();
+
+    // Validate with a mix of matching and non-matching files
+    // All non-matching should be filtered, matching ones should succeed
+    Command::cargo_bin("codeowners")?
+        .arg("--project-root")
+        .arg(project_root)
+        .arg("--codeowners-file-path")
+        .arg(&codeowners_path)
+        .arg("--no-cache")
+        .arg("validate")
+        .arg("ruby/app/models/payroll.rb") // matches owned_globs, is owned
+        .arg("ruby/app/models/test.rbi") // doesn't match owned_globs
+        .arg("ruby/app/models/test.md") // doesn't match owned_globs
+        .arg("ruby/app/models/test.txt") // doesn't match owned_globs
+        .arg("ruby/app/models/test.json") // doesn't match owned_globs
+        .assert()
+        .success()
+        .stdout(predicate::eq(""));
+
+    Ok(())
+}
+
+#[test]
+fn test_validate_filters_files_outside_owned_directories() -> Result<(), Box<dyn Error>> {
+    // Test that files in directories not matching owned_globs are filtered
+    // valid_project owned_globs: "{gems,config,javascript,ruby,components}/**/*.{rb,tsx,erb}"
+    let fixture_root = std::path::Path::new("tests/fixtures/valid_project");
+    let temp_dir = setup_fixture_repo(fixture_root);
+    let project_root = temp_dir.path();
+
+    // Create .rb files OUTSIDE the owned directories
+    std::fs::create_dir_all(project_root.join("scripts"))?;
+    std::fs::write(project_root.join("scripts/deploy.rb"), "# Deploy script")?;
+    std::fs::create_dir_all(project_root.join("bin"))?;
+    std::fs::write(project_root.join("bin/run.rb"), "# Run script")?;
+
+    git_add_all_files(project_root);
+
+    let codeowners_path = project_root.join("tmp/CODEOWNERS");
+
+    // Generate CODEOWNERS
+    Command::cargo_bin("codeowners")?
+        .arg("--project-root")
+        .arg(project_root)
+        .arg("--codeowners-file-path")
+        .arg(&codeowners_path)
+        .arg("--no-cache")
+        .arg("generate")
+        .assert()
+        .success();
+
+    // Validate with files both inside and outside owned directories
+    Command::cargo_bin("codeowners")?
+        .arg("--project-root")
+        .arg(project_root)
+        .arg("--codeowners-file-path")
+        .arg(&codeowners_path)
+        .arg("--no-cache")
+        .arg("validate")
+        .arg("ruby/app/models/payroll.rb") // inside ruby/, matches owned_globs
+        .arg("scripts/deploy.rb") // outside owned dirs, filtered
+        .arg("bin/run.rb") // outside owned dirs, filtered
+        .assert()
+        .success()
+        .stdout(predicate::eq(""));
+
+    Ok(())
+}
+
+#[test]
+fn test_validate_respects_unowned_globs() -> Result<(), Box<dyn Error>> {
+    // Test that files matching unowned_globs are filtered out even if they match owned_globs
+    let fixture_root = std::path::Path::new("tests/fixtures/valid_project");
+    let temp_dir = setup_fixture_repo(fixture_root);
+    let project_root = temp_dir.path();
+
+    // Read and modify the config to add unowned_globs
+    let config_path = project_root.join("config/code_ownership.yml");
+    let config_content = std::fs::read_to_string(&config_path)?;
+    let updated_config = config_content.replace("unowned_globs:", "unowned_globs:\n  - ruby/app/models/ignored_*.rb");
+    std::fs::write(&config_path, updated_config)?;
+
+    // Create a file that matches owned_globs but also matches unowned_globs
+    std::fs::write(
+        project_root.join("ruby/app/models/ignored_test.rb"),
+        "# This file should be ignored via unowned_globs",
+    )?;
+
+    git_add_all_files(project_root);
+
+    let codeowners_path = project_root.join("tmp/CODEOWNERS");
+
+    // Generate CODEOWNERS (ignored_test.rb should NOT be included)
+    Command::cargo_bin("codeowners")?
+        .arg("--project-root")
+        .arg(project_root)
+        .arg("--codeowners-file-path")
+        .arg(&codeowners_path)
+        .arg("--no-cache")
+        .arg("generate")
+        .assert()
+        .success();
+
+    // Verify the ignored file is NOT in CODEOWNERS
+    let codeowners_content = std::fs::read_to_string(&codeowners_path)?;
+    assert!(
+        !codeowners_content.contains("ignored_test.rb"),
+        "ignored_test.rb should not be in CODEOWNERS"
+    );
+
+    // Validate with the ignored file - should be filtered by unowned_globs
+    Command::cargo_bin("codeowners")?
+        .arg("--project-root")
+        .arg(project_root)
+        .arg("--codeowners-file-path")
+        .arg(&codeowners_path)
+        .arg("--no-cache")
+        .arg("validate")
+        .arg("ruby/app/models/payroll.rb") // owned, should validate
+        .arg("ruby/app/models/ignored_test.rb") // matches unowned_globs, should be filtered
+        .assert()
+        .success()
+        .stdout(predicate::eq(""));
+
+    Ok(())
+}

data/ext/cargo-vendor/unicode-ident-1.0.19/.cargo-checksum.json

@@ -1 +1 @@
-{"files":{".cargo_vcs_info.json":"5416fcff6fa0c6611cec99144bd5690f506c7875b8263ddeae6f123e22c3b50c",".github/FUNDING.yml":"b017158736b3c9751a2d21edfce7fe61c8954e2fced8da8dd3013c2f3e295bd9",".github/workflows/ci.yml":"3f09084970623b248ee41aaa7928e8e1e4298a712a6621334580951a3227495b","Cargo.lock":"2b727ff15f388012e929c089fdb62ead00fdb88a720a34c0e5c88b98748dc186","Cargo.toml":"92d59ecf8e5c840a4cad498e6fa36ccb34ae1aa3c2ceec2abd258b129fae67d1","Cargo.toml.orig":"afbf880657a9fd530a2c7e47f3e91e388a5a4b313580e82ad0618a3f0d043823","LICENSE-APACHE":"62c7a1e35f56406896d7aa7ca52d0cc0d272ac022b5d2796e7d6905db8a3636a","LICENSE-MIT":"23f18e03dc49df91622fe2a76176497404e46ced8a715d9d2b67a7446571cca3","LICENSE-UNICODE":"f7db81051789b729fea528a63ec4c938fdcb93d9d61d97dc8cc2e9df6d47f2a1","README.md":"14a5b144df5ffd729c605286a8b2abe4733f147197b5142e60d6bc77ea44854c","benches/xid.rs":"7eb058c1140a253f7177af8868a95aabb1b92c52dc7eee5abbfeadb507d7845d","src/lib.rs":"9fdfc52c5695b4bdf6a777afe6705f18f50a9d05939f17fbede0e84aeef11c64","src/tables.rs":"9afe268f3a6087a96d4217aaf2fe9ebd5418f133bd733193009df7dccebf154b","tests/compare.rs":"f2311271aa1db7380e5bf153ef83ee99777e14579e4f28c2b1a3e21877ffe715","tests/fst/.gitignore":"2cd419079c0a08bb15766520880998651dd1c72c55347a31f43357595b16ac10","tests/fst/mod.rs":"69a3aaf59acd8bca962ecc6234be56be8c0934ab79b253162f10eb881523901f","tests/fst/xid_continue.fst":"b58be4f0c498253e7a5ac664046096f15f249da66131347d4b822097623549a2","tests/fst/xid_start.fst":"aec7eecdacfce308d2e6210f47d28ed3aad5c8b048efbfbc11fc22e60fbf435b","tests/roaring/mod.rs":"f5c6d55463a7f53e92a493cf046d717149250fbafc0e0fe94bdb531377bf8b11","tests/static_size.rs":"3a3467bad2bbb0cef8f0fe953967e16bc0b0a865ec52fd7eb34d620e9b3a11d4","tests/tables/mod.rs":"e6949172d10fc4b2431ce7546269bfd4f9146454c8c3e31faf5e5d80c16a8ab6","tests/tables/tables.rs":"302d87306100b6280f8db93e167dc70c47f724045cf1312b7354683656c3f36b","tests/trie/mod.rs":"d4acbb716bcbaf80660039797f45e138ed8bbd66749fa3b19b1a971574679cc9","tests/trie/trie.rs":"f7b9edc1e8a98e3be42b653bba27bb4eb5fc48a559d6d8d1c6a4db4b5425b0d5"},"package":"f63a545481291138910575129486daeaf8ac54aee4387fe7906919f7830c7d9d"}
+{"files":{".cargo_vcs_info.json":"5416fcff6fa0c6611cec99144bd5690f506c7875b8263ddeae6f123e22c3b50c",".github/FUNDING.yml":"b017158736b3c9751a2d21edfce7fe61c8954e2fced8da8dd3013c2f3e295bd9",".github/workflows/ci.yml":"3f09084970623b248ee41aaa7928e8e1e4298a712a6621334580951a3227495b","Cargo.lock":"2b727ff15f388012e929c089fdb62ead00fdb88a720a34c0e5c88b98748dc186","Cargo.toml":"92d59ecf8e5c840a4cad498e6fa36ccb34ae1aa3c2ceec2abd258b129fae67d1","Cargo.toml.orig":"afbf880657a9fd530a2c7e47f3e91e388a5a4b313580e82ad0618a3f0d043823","LICENSE-APACHE":"62c7a1e35f56406896d7aa7ca52d0cc0d272ac022b5d2796e7d6905db8a3636a","LICENSE-MIT":"23f18e03dc49df91622fe2a76176497404e46ced8a715d9d2b67a7446571cca3","LICENSE-UNICODE":"f7db81051789b729fea528a63ec4c938fdcb93d9d61d97dc8cc2e9df6d47f2a1","README.md":"14a5b144df5ffd729c605286a8b2abe4733f147197b5142e60d6bc77ea44854c","benches/xid.rs":"7eb058c1140a253f7177af8868a95aabb1b92c52dc7eee5abbfeadb507d7845d","src/lib.rs":"9fdfc52c5695b4bdf6a777afe6705f18f50a9d05939f17fbede0e84aeef11c64","src/tables.rs":"9afe268f3a6087a96d4217aaf2fe9ebd5418f133bd733193009df7dccebf154b","tests/compare.rs":"f2311271aa1db7380e5bf153ef83ee99777e14579e4f28c2b1a3e21877ffe715","tests/fst/mod.rs":"69a3aaf59acd8bca962ecc6234be56be8c0934ab79b253162f10eb881523901f","tests/fst/xid_continue.fst":"b58be4f0c498253e7a5ac664046096f15f249da66131347d4b822097623549a2","tests/fst/xid_start.fst":"aec7eecdacfce308d2e6210f47d28ed3aad5c8b048efbfbc11fc22e60fbf435b","tests/roaring/mod.rs":"f5c6d55463a7f53e92a493cf046d717149250fbafc0e0fe94bdb531377bf8b11","tests/static_size.rs":"3a3467bad2bbb0cef8f0fe953967e16bc0b0a865ec52fd7eb34d620e9b3a11d4","tests/tables/mod.rs":"e6949172d10fc4b2431ce7546269bfd4f9146454c8c3e31faf5e5d80c16a8ab6","tests/tables/tables.rs":"302d87306100b6280f8db93e167dc70c47f724045cf1312b7354683656c3f36b","tests/trie/mod.rs":"d4acbb716bcbaf80660039797f45e138ed8bbd66749fa3b19b1a971574679cc9","tests/trie/trie.rs":"f7b9edc1e8a98e3be42b653bba27bb4eb5fc48a559d6d8d1c6a4db4b5425b0d5"},"package":"f63a545481291138910575129486daeaf8ac54aee4387fe7906919f7830c7d9d"}

data/ext/code_ownership/Cargo.toml

@@ -17,7 +17,7 @@ rb-sys = { version = "0.9.111", features = [
 magnus = { version = "0.8" }
 serde = { version = "1.0.219", features = ["derive"] }
 serde_magnus = "0.10"
-codeowners = { git = "https://github.com/rubyatscale/codeowners-rs.git", tag = "v0.3.0" }
+codeowners = { git = "https://github.com/rubyatscale/codeowners-rs.git", tag = "v0.3.2" }
 
 [dev-dependencies]
 rb-sys = { version = "0.9.117", features = [

data/ext/code_ownership/src/lib.rs

@@ -109,14 +109,14 @@ fn build_run_config() -> RunConfig {
         Ok(path) => path,
         _ => PathBuf::from("."),
     };
-    let codeowners_file_path = project_root.join(".github/CODEOWNERS");
     let config_path = project_root.join("config/code_ownership.yml");
 
     RunConfig {
         project_root,
-        codeowners_file_path,
+        codeowners_file_path: None,
         config_path,
         no_cache: false,
+        executable_name: Some("bin/codeownership validate".to_string()),
     }
 }
 

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