fix(CI): resolve remaining lint/format violations in PR #8722
CI / lint (push) Successful in 1m54s
CI / build (push) Successful in 1m29s
CI / quality (push) Successful in 2m1s
CI / typecheck (push) Successful in 2m7s
CI / security (push) Successful in 2m20s
CI / helm (push) Successful in 29s
CI / integration_tests (push) Successful in 4m17s
CI / push-validation (push) Successful in 20s
CI / e2e_tests (push) Successful in 4m21s
CI / unit_tests (push) Successful in 5m42s
CI / benchmark-regression (push) Has been skipped
CI / docker (push) Successful in 2m1s
CI / coverage (push) Successful in 13m48s
CI / status-check (push) Successful in 4s
CI / benchmark-publish (push) Successful in 1h17m24s
CI / benchmark-publish (pull_request) Has been skipped
CI / build (pull_request) Successful in 51s
CI / helm (pull_request) Successful in 32s
CI / lint (pull_request) Successful in 1m4s
CI / typecheck (pull_request) Successful in 1m18s
CI / quality (pull_request) Successful in 1m19s
CI / security (pull_request) Successful in 1m25s
CI / push-validation (pull_request) Successful in 20s
CI / integration_tests (pull_request) Successful in 3m31s
CI / benchmark-regression (pull_request) Failing after 53s
CI / e2e_tests (pull_request) Successful in 5m13s
CI / unit_tests (pull_request) Successful in 6m43s
CI / coverage (pull_request) Has been cancelled
CI / docker (pull_request) Has been cancelled
CI / status-check (pull_request) Has been cancelled

- Apply ruff format to features/steps/strategize_decision_recording_steps.py:
  expanded single-line set literal to multi-line canonical form (PEP 87)
  This was the sole remaining CI lint blocker per review cycle 14.

- Remove unused imports from tests/actor/test_registry_builtin_yaml.py:
  removed 'Settings' and 'ProviderRegistry' to fix F401 errors.
  These were pre-existing violations caught by ruff check.

ISSUES CLOSED: #8522

Closes: #8722
This commit was merged in pull request #8722.
This commit is contained in:
2026-05-08 05:15:25 +00:00
committed by Forgejo
parent e7fb7168b4
commit 883ec872e2
2 changed files with 7 additions and 3 deletions
@@ -320,7 +320,12 @@ def step_then_decision_phase(context, phase):
"""
assert context.last_decision is not None
# Strategize-phase decision types accepted by the hook
strategize_types = {"strategy_choice", "resource_selection", "subplan_spawn", "invariant_enforced"}
strategize_types = {
"strategy_choice",
"resource_selection",
"subplan_spawn",
"invariant_enforced",
}
assert context.last_decision.decision_type.value in strategize_types, (
f"Expected a Strategize-phase decision type, got {context.last_decision.decision_type.value!r}"
)
+1 -2
View File
@@ -17,13 +17,12 @@ import yaml
from cleveragents.actor.registry import ActorRegistry
from cleveragents.actor.schema import ActorConfigSchema, ActorType, is_v3_yaml
from cleveragents.config.settings import ProviderDefaults, Settings
from cleveragents.config.settings import ProviderDefaults
from cleveragents.core.exceptions import NotFoundError
from cleveragents.domain.models.core.actor import Actor
from cleveragents.providers.registry import (
ProviderCapabilities,
ProviderInfo,
ProviderRegistry,
ProviderType,
)