[Repo Assist] refactor(rust-guard): add MinIntegrity::from_policy_str to unify duplicate parsers

[github-actions]

🤖 This PR was created by Repo Assist, an automated AI assistant.

Summary

Closes #7369

MinIntegrity already had as_str() to serialize an enum variant to a string, but had no reverse operation. As a result, integrity_level_rank and integrity_for_level both contained an identical 4-way case-insensitive if/else chain to parse a policy integrity level string. If a fifth integrity level is ever added, both places must be updated in sync with no compiler enforcement.

Changes

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

1. Add MinIntegrity::from_policy_str(level: &str) -> Option<Self> — parses a policy integrity level string case-insensitively; returns None for unrecognised values. This completes the MinIntegrity type as a proper bidirectional mapping.

2. Refactor integrity_level_rank — replace the 4-branch if/else chain with a match on from_policy_str().

3. Refactor integrity_for_level — same: replace the 4-branch if/else chain with a match on from_policy_str().

4. Add 14 new tests:

   • 6 tests for from_policy_str: known values, case-insensitivity, whitespace trimming, unknown inputs, and roundtrip (from_policy_str(as_str()) == Some(level))
   • 8 tests for extract_repo_from_item: all 5 lookup strategies (direct full_name, repository.full_name, base.repo.full_name, head.repo.full_name, URL fallback), plus empty case and priority ordering — this function previously had zero direct tests despite being called for every labelled item

Why This Matters

• The parsing logic is now defined once — adding a new integrity level requires only updating from_policy_str; the exhaustive match at each call site will fail to compile until updated.
• MinIntegrity becomes a complete, bidirectional type: serialize with as_str(), deserialize with from_policy_str().
• All existing tests (test_integrity_level_rank_* and test_integrity_for_level_*) continue to pass, confirming behavior is unchanged.

Test Status

• ✅ Rust tests: 479 passed, 0 failed (cargo test --lib)
• ⚠️ Go build/tests: skipped — proxy.golang.org is blocked in this environment (infrastructure constraint, not related to this change)

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

• proxy.golang.org

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "proxy.golang.org"

See Network Configuration for more information.

Generated by Repo Assist · 831.2 AIC · ⊞ 50K · ◷
_{Comment /repo-assist to run again}

Add this agentic workflows to your repo

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/repo-assist.md@851905c06e905bf362a9f6cc54f912e3df747d55

[Copilot]

Pull request overview

This PR refactors the Rust guard’s policy-integrity parsing by adding a single MinIntegrity::from_policy_str(&str) -> Option<Self> deserializer and reusing it in existing helpers, reducing duplicated parsing logic and improving maintainability.

Changes:

• Added MinIntegrity::from_policy_str() to parse policy integrity levels case-insensitively (with trimming) into MinIntegrity.
• Refactored integrity_level_rank and integrity_for_level to use from_policy_str() instead of duplicated if/else chains.
• Added new unit tests for from_policy_str and for extract_repo_from_item repository extraction strategies.

Show a summary per file

File	Description
guards/github-guard/rust-guard/src/labels/helpers.rs	Introduces MinIntegrity::from_policy_str, refactors integrity helpers to use it, and adds unit tests (including new coverage for extract_repo_from_item).

Copilot's findings

Tip

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

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