fix(guard): remove unused org-ownership functions blocking release

[lpcox]

Problem

The Release workflow (run #24740822133) failed in the docker job because the Rust guard build has -D warnings enabled and two functions trigger dead_code warnings:

error: function `get_cached_owner_is_org` is never used
  --> src/labels/backend.rs:63

error: function `is_repo_org_owned` is never used
  --> src/labels/backend.rs:282

Fix

Remove the two unused functions (get_cached_owner_is_org and is_repo_org_owned) and their associated tests. These were added for future org-ownership detection but are never called from production code.

The cache setter (set_cached_owner_is_org) is retained — it's called during is_repo_private lookups to populate the owner-type cache for when these functions are needed later.

Removed

• get_cached_owner_is_org() — private cache reader (line 63)
• is_repo_org_owned() — public API (line 282)
• test_is_repo_org_owned_uses_cache — test
• test_is_repo_org_owned_empty_args — test

Verification

make agent-finished passes (format, build, lint, all tests including Rust guard).

[Copilot]

Pull request overview

Removes unused org-ownership helper APIs from the Rust guard to unblock release builds that treat warnings as errors.

Changes:

• Deleted unused cache reader get_cached_owner_is_org and public API is_repo_org_owned that were triggering dead_code warnings.
• Removed the associated unit tests for is_repo_org_owned.
• Kept owner-type cache population during is_repo_private lookups.

Show a summary per file

File	Description
guards/github-guard/rust-guard/src/labels/backend.rs	Removes unused org-ownership functions/tests to eliminate dead_code warnings during release builds.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comments suppressed due to low confidence (2)

guards/github-guard/rust-guard/src/labels/backend.rs:66

• After removing the only read path (get_cached_owner_is_org / is_repo_org_owned), the owner-type cache is now write-only: set_cached_owner_is_org is called from is_repo_private_with_callback, but nothing ever reads repo_owner_type_cache. This adds extra work and can grow memory over the process lifetime without providing value. Consider removing the owner-type extraction/caching until there is a real consumer, or adding a bounded/evicting cache before keeping it.

This issue also appears on line 1287 of the same file.

fn set_cached_owner_is_org(repo_id: &str, is_org: bool) {
    if let Ok(mut cache) = repo_owner_type_cache().lock() {
        cache.insert(repo_id.to_string(), is_org);
    }

guards/github-guard/rust-guard/src/labels/backend.rs:1290

• With the is_repo_org_owned tests removed, the helper clear_owner_type_cache_for_tests (defined earlier in this module) is now unused. This can reintroduce a dead_code warning when compiling tests (and can fail CI if warnings are denied). Consider deleting the helper or annotating it to suppress dead_code if you want to keep it for future tests.

        });
        assert_eq!(extract_owner_is_org(&response, "myorg/myrepo"), None);
    }
}

• Files reviewed: 1/1 changed files
• Comments generated: 0