rust-guard: deduplicate repo fallback resolution and private-flag field checks

[Copilot]

response_paths.rs duplicated the same default-repo fallback logic across four tool handlers, and backend.rs repeated equivalent private field probing logic in three separate blocks. This PR consolidates both patterns into single reusable paths while preserving behavior.

• response_paths.rs: extract shared default repo resolver

  • Added default_repo_for_items(arg_repo_full: String, items: &[Value]) -> String.
  • Replaced 4 duplicated call-site blocks in:
    • list_pull_requests
    • list_issues
    • list_commits
    • list_releases

• backend.rs: collapse repeated private-field probes

  • Updated private_flag_from_repo_object to iterate candidate field names in order:
    • "private", "is_private", "isPrivate"
  • Kept existing visibility fallback logic unchanged.

• Resulting code shape (representative)

  fn default_repo_for_items(arg_repo_full: String, items: &[Value]) -> String {
      if !arg_repo_full.is_empty() {
          return arg_repo_full;
      }
      items.first().map(extract_repo_from_item).unwrap_or_default()
  }
  
  fn private_flag_from_repo_object(item: &Value) -> Option<bool> {
      for field in &["private", "is_private", "isPrivate"] {
          if let Some(is_private) = item.get(*field).and_then(|v| v.as_bool()) {
              return Some(is_private);
          }
      }
      // existing visibility fallback remains
      ...
  }

[Copilot]

Pull request overview

This PR refactors the Rust GitHub guard labeling code to remove duplicated “default repo” fallback logic in path-based response labeling, and to consolidate repeated “private” field probing in repo objects—keeping behavior consistent while reducing maintenance overhead.

Changes:

• Extracted default_repo_for_items(...) and reused it across the collection labelers that previously reimplemented the same fallback logic.
• Simplified private_flag_from_repo_object(...) to iterate through the known candidate boolean fields (private, is_private, isPrivate) before falling back to visibility handling.
• Updated the affected tool handler branches to use the shared helpers without changing labeling semantics.

Show a summary per file

File	Description
guards/github-guard/rust-guard/src/labels/response_paths.rs	Deduplicates default-repo fallback resolution used by multiple collection labeling paths.
guards/github-guard/rust-guard/src/labels/backend.rs	Consolidates private-flag detection by probing candidate field names in a single loop.

Copilot's findings

Tip

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

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