Fix ty type check: make ScenarioResult identifier params optional

objective_target_identifier and objective_scorer_identifier may be None
when deserializing from dicts. The printer bases already handle None.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: rlundeen2

pyrit/models/scenario_result.py

@@ -95,9 +95,9 @@ def __init__(
         self,
         *,
         scenario_identifier: ScenarioIdentifier,
-        objective_target_identifier: ComponentIdentifier,
+        objective_target_identifier: ComponentIdentifier | None,
         attack_results: dict[str, list[AttackResult]],
-        objective_scorer_identifier: ComponentIdentifier,
+        objective_scorer_identifier: ComponentIdentifier | None,
         scenario_run_state: ScenarioRunState = "CREATED",
         labels: dict[str, str] | None = None,
         creation_time: datetime | None = None,