fix(context): resolve PR #10590 review blockers and compliance items
CI / benchmark-publish (pull_request) Has been skipped
CI / benchmark-regression (pull_request) Failing after 58s
CI / unit_tests (pull_request) Has started running
CI / integration_tests (pull_request) Has started running
CI / e2e_tests (pull_request) Has started running
CI / helm (pull_request) Successful in 52s
CI / push-validation (pull_request) Successful in 2m39s
CI / build (pull_request) Successful in 2m56s
CI / lint (pull_request) Successful in 3m2s
CI / quality (pull_request) Successful in 3m22s
CI / typecheck (pull_request) Successful in 3m35s
CI / security (pull_request) Successful in 3m41s
CI / coverage (pull_request) Has been cancelled
CI / docker (pull_request) Has been cancelled
CI / status-check (pull_request) Has been cancelled
CI / benchmark-publish (pull_request) Has been skipped
CI / benchmark-regression (pull_request) Failing after 58s
CI / unit_tests (pull_request) Has started running
CI / integration_tests (pull_request) Has started running
CI / e2e_tests (pull_request) Has started running
CI / helm (pull_request) Successful in 52s
CI / push-validation (pull_request) Successful in 2m39s
CI / build (pull_request) Successful in 2m56s
CI / lint (pull_request) Successful in 3m2s
CI / quality (pull_request) Successful in 3m22s
CI / typecheck (pull_request) Successful in 3m35s
CI / security (pull_request) Successful in 3m41s
CI / coverage (pull_request) Has been cancelled
CI / docker (pull_request) Has been cancelled
CI / status-check (pull_request) Has been cancelled
- Remove unused TYPE_CHECKING imports (DomainContextStrategy, StrategyCapabilities) from acms_service.py — they are only referenced in docstrings, not type annotations, causing F401 lint errors. This resolves the lint failure that was introduced by the prior commit b5b08e25's attempt to move them into the TYPE_CHECKING block. - Fix line-length violation (E501) on StrategyCapabilities docstring — shortened :mod: directive reference to fit 88-char limit. ISSUES CLOSED: #10590
This commit is contained in:
@@ -32,10 +32,6 @@ import structlog
|
||||
|
||||
if TYPE_CHECKING:
|
||||
from cleveragents.config.settings import Settings
|
||||
from cleveragents.domain.models.acms.strategy import (
|
||||
ContextStrategy as DomainContextStrategy,
|
||||
StrategyCapabilities as DomainStrategyCapabilities,
|
||||
)
|
||||
from cleveragents.infrastructure.database.unit_of_work import UnitOfWork
|
||||
from cleveragents.infrastructure.plugins.manager import PluginManager
|
||||
|
||||
@@ -116,7 +112,7 @@ class StrategyCapabilities:
|
||||
"""Pipeline-specific capabilities declared by a context strategy.
|
||||
|
||||
This is a backward-compatible variant of the domain-model
|
||||
``DomainStrategyCapabilities`` (see :mod:`cleveragents.domain.models.acms.strategy`).
|
||||
``~cleveragents.domain.models.acms.strategy.DomainStrategyCapabilities``.
|
||||
For production strategies that implement the true spec protocol, use
|
||||
``DomainStrategyCapabilities`` instead.
|
||||
"""
|
||||
|
||||
Reference in New Issue
Block a user