style: fix ruff format violations in subplan_service and tdd steps
CI / benchmark-publish (pull_request) Has been skipped
CI / push-validation (pull_request) Successful in 33s
CI / helm (pull_request) Successful in 42s
CI / build (pull_request) Successful in 57s
CI / lint (pull_request) Successful in 1m14s
CI / quality (pull_request) Successful in 1m37s
CI / security (pull_request) Successful in 1m44s
CI / typecheck (pull_request) Successful in 1m51s
CI / e2e_tests (pull_request) Successful in 4m21s
CI / integration_tests (pull_request) Failing after 4m58s
CI / unit_tests (pull_request) Successful in 5m52s
CI / docker (pull_request) Successful in 1m39s
CI / coverage (pull_request) Successful in 12m25s
CI / status-check (pull_request) Failing after 3s
CI / benchmark-publish (pull_request) Has been skipped
CI / push-validation (pull_request) Successful in 33s
CI / helm (pull_request) Successful in 42s
CI / build (pull_request) Successful in 57s
CI / lint (pull_request) Successful in 1m14s
CI / quality (pull_request) Successful in 1m37s
CI / security (pull_request) Successful in 1m44s
CI / typecheck (pull_request) Successful in 1m51s
CI / e2e_tests (pull_request) Successful in 4m21s
CI / integration_tests (pull_request) Failing after 4m58s
CI / unit_tests (pull_request) Successful in 5m52s
CI / docker (pull_request) Successful in 1m39s
CI / coverage (pull_request) Successful in 12m25s
CI / status-check (pull_request) Failing after 3s
Apply ruff auto-format to resolve CI lint failure (format check step). No logic changes — pure whitespace/line-length reformatting.
This commit is contained in:
@@ -226,11 +226,9 @@ def step_child_has_invariant_decisions(context: Context) -> None:
|
||||
child_plan_id: str = child_plan.identity.plan_id
|
||||
|
||||
# Query the decision service for invariant_enforced decisions on the child plan
|
||||
child_invariant_decisions: list[Decision] = (
|
||||
context.decision_service.list_by_type(
|
||||
child_plan_id,
|
||||
DecisionType.INVARIANT_ENFORCED,
|
||||
)
|
||||
child_invariant_decisions: list[Decision] = context.decision_service.list_by_type(
|
||||
child_plan_id,
|
||||
DecisionType.INVARIANT_ENFORCED,
|
||||
)
|
||||
|
||||
assert len(child_invariant_decisions) > 0, (
|
||||
@@ -256,11 +254,9 @@ def step_child_has_n_invariant_decisions(context: Context, count: int) -> None:
|
||||
child_plan: Plan = result.child_plans[0]
|
||||
child_plan_id: str = child_plan.identity.plan_id
|
||||
|
||||
child_invariant_decisions: list[Decision] = (
|
||||
context.decision_service.list_by_type(
|
||||
child_plan_id,
|
||||
DecisionType.INVARIANT_ENFORCED,
|
||||
)
|
||||
child_invariant_decisions: list[Decision] = context.decision_service.list_by_type(
|
||||
child_plan_id,
|
||||
DecisionType.INVARIANT_ENFORCED,
|
||||
)
|
||||
|
||||
assert len(child_invariant_decisions) == count, (
|
||||
@@ -287,11 +283,9 @@ def step_child_has_non_overridable_invariant(context: Context) -> None:
|
||||
child_plan: Plan = result.child_plans[0]
|
||||
child_plan_id: str = child_plan.identity.plan_id
|
||||
|
||||
child_invariant_decisions: list[Decision] = (
|
||||
context.decision_service.list_by_type(
|
||||
child_plan_id,
|
||||
DecisionType.INVARIANT_ENFORCED,
|
||||
)
|
||||
child_invariant_decisions: list[Decision] = context.decision_service.list_by_type(
|
||||
child_plan_id,
|
||||
DecisionType.INVARIANT_ENFORCED,
|
||||
)
|
||||
|
||||
assert len(child_invariant_decisions) > 0, (
|
||||
@@ -301,9 +295,7 @@ def step_child_has_non_overridable_invariant(context: Context) -> None:
|
||||
|
||||
# Verify the non_overridable invariant text was propagated
|
||||
expected_option: str = context.non_overridable_chosen_option
|
||||
propagated_options: list[str] = [
|
||||
d.chosen_option for d in child_invariant_decisions
|
||||
]
|
||||
propagated_options: list[str] = [d.chosen_option for d in child_invariant_decisions]
|
||||
assert expected_option in propagated_options, (
|
||||
f"Non-overridable invariant chosen_option {expected_option!r} not found "
|
||||
f"in child plan decisions: {propagated_options}. "
|
||||
@@ -311,9 +303,7 @@ def step_child_has_non_overridable_invariant(context: Context) -> None:
|
||||
)
|
||||
|
||||
|
||||
@then(
|
||||
"the spawn should succeed with no invariant decisions recorded for the child"
|
||||
)
|
||||
@then("the spawn should succeed with no invariant decisions recorded for the child")
|
||||
def step_spawn_succeeds_no_invariants(context: Context) -> None:
|
||||
"""Assert that spawn succeeds cleanly when parent has no invariant decisions."""
|
||||
assert context.spawn_error is None, (
|
||||
@@ -330,11 +320,9 @@ def step_spawn_succeeds_no_invariants(context: Context) -> None:
|
||||
child_plan: Plan = result.child_plans[0]
|
||||
child_plan_id: str = child_plan.identity.plan_id
|
||||
|
||||
child_invariant_decisions: list[Decision] = (
|
||||
context.decision_service.list_by_type(
|
||||
child_plan_id,
|
||||
DecisionType.INVARIANT_ENFORCED,
|
||||
)
|
||||
child_invariant_decisions: list[Decision] = context.decision_service.list_by_type(
|
||||
child_plan_id,
|
||||
DecisionType.INVARIANT_ENFORCED,
|
||||
)
|
||||
|
||||
assert len(child_invariant_decisions) == 0, (
|
||||
|
||||
@@ -375,9 +375,7 @@ class SubplanService:
|
||||
rationale=parent_decision.rationale,
|
||||
context_snapshot=parent_decision.context_snapshot,
|
||||
confidence_score=parent_decision.confidence_score,
|
||||
alternatives_considered=list(
|
||||
parent_decision.alternatives_considered
|
||||
),
|
||||
alternatives_considered=list(parent_decision.alternatives_considered),
|
||||
)
|
||||
|
||||
# ------------------------------------------------------------------
|
||||
|
||||
Reference in New Issue
Block a user