Skip to content

Commit 1419490

Browse files
lpcoxCopilot
andcommitted
fix(guard): remove unused org-ownership functions from backend.rs
Remove get_cached_owner_is_org and is_repo_org_owned which are never called outside of tests. These were added for future use but currently trigger -D warnings (dead_code) failures in CI release builds. The cache setter (set_cached_owner_is_org) is retained as it's called during is_repo_private lookups to populate the cache. Co-authored-by: Copilot <[email protected]>
1 parent 33edc25 commit 1419490

1 file changed

Lines changed: 0 additions & 39 deletions

File tree

  • guards/github-guard/rust-guard/src/labels

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

Lines changed: 0 additions & 39 deletions
Original file line numberDiff line numberDiff line change
@@ -60,13 +60,6 @@ fn set_cached_repo_visibility(repo_id: &str, is_private: bool) {
6060
}
6161
}
6262

63-
fn get_cached_owner_is_org(repo_id: &str) -> Option<bool> {
64-
repo_owner_type_cache()
65-
.lock()
66-
.ok()
67-
.and_then(|cache| cache.get(repo_id).copied())
68-
}
69-
7063
fn set_cached_owner_is_org(repo_id: &str, is_org: bool) {
7164
if let Ok(mut cache) = repo_owner_type_cache().lock() {
7265
cache.insert(repo_id.to_string(), is_org);
@@ -271,21 +264,6 @@ pub fn is_repo_private_with_callback(
271264
get_cached_repo_visibility(&repo_id)
272265
}
273266

274-
/// Check whether a repository is owned by an organization (vs a personal account).
275-
/// This is determined from the `owner.type` field in the search_repositories response,
276-
/// which is cached alongside repo visibility during `is_repo_private` calls.
277-
///
278-
/// Returns:
279-
/// - `Some(true)` if the owner is an Organization
280-
/// - `Some(false)` if the owner is a User (personal account)
281-
/// - `None` if the owner type could not be determined
282-
pub fn is_repo_org_owned(owner: &str, repo: &str) -> Option<bool> {
283-
if owner.is_empty() || repo.is_empty() {
284-
return None;
285-
}
286-
let repo_id = format!("{}/{}", owner, repo);
287-
get_cached_owner_is_org(&repo_id)
288-
}
289267

290268
/// Fetch pull request facts used for integrity derivation.
291269
pub fn get_pull_request_facts_with_callback(
@@ -1309,23 +1287,6 @@ mod tests {
13091287
});
13101288
assert_eq!(extract_owner_is_org(&response, "myorg/myrepo"), None);
13111289
}
1312-
1313-
#[test]
1314-
fn test_is_repo_org_owned_uses_cache() {
1315-
clear_owner_type_cache_for_tests();
1316-
set_cached_owner_is_org("cached-org/repo", true);
1317-
assert_eq!(is_repo_org_owned("cached-org", "repo"), Some(true));
1318-
1319-
set_cached_owner_is_org("cached-user/repo", false);
1320-
assert_eq!(is_repo_org_owned("cached-user", "repo"), Some(false));
1321-
clear_owner_type_cache_for_tests();
1322-
}
1323-
1324-
#[test]
1325-
fn test_is_repo_org_owned_empty_args() {
1326-
assert_eq!(is_repo_org_owned("", "repo"), None);
1327-
assert_eq!(is_repo_org_owned("owner", ""), None);
1328-
}
13291290
}
13301291

13311292
fn repo_visibility_from_items(value: &Value, repo_id: &str) -> Option<bool> {

0 commit comments

Comments
 (0)