[Lanre Shittu (Shizoqua)]

Description

Implements the P311-03 case for RegistryUpdateBehavior dispatch in ScorerEvaluator.run_evaluation_async, claimed and assigned in #3166.

The method checked SKIP_IF_EXISTS and NEVER_UPDATE with == and !=, so ALWAYS_UPDATE and any future member fell through implicitly. It now matches each member explicitly and adds a TYPE_CHECKING-only assert_never guard, following the existing pattern in ScorerEvaluator.from_scorer (#3924).

Behavior is unchanged, including the runtime fallback for values outside the enum (run the evaluation and write to the registry).

Tests and Documentation

Added a parametrized test in tests/unit/score/test_scorer_evaluator.py covering all three members and the out-of-enum fallback: whether existing metrics are checked and whether the registry is written. It passes on both the old and new code, confirming no behavior change.

No docs or notebooks changed, so JupyText was not needed.

[Copilot]

🟢 Approval recommended

No active review issues remain for the registry-update dispatch change.

No open issues 🎉

What changed in this PR

Makes registry-update dispatch in ScorerEvaluator.run_evaluation_async explicit and statically exhaustive for P311-03, following the existing from_scorer pattern while preserving runtime behavior.

Changes:

• Handles all three RegistryUpdateBehavior members explicitly, with a type-checking-only assert_never guard and the existing out-of-enum fallback.
• Adds parametrized coverage for existing-metrics checks and registry writes across all supported modes and the fallback.

File	Description
tests/​unit/​score/​test_scorer_evaluator.py	Adds registry-update behavior regression coverage.
pyrit/​score/​scorer_evaluation/​scorer_evaluator.py	Replaces implicit dispatch with exhaustive matching and explicit behavior flags.

_{Review effort: Balanced}