From 9ff101368d11e4707ff263e3b0202b268c9cd2d0 Mon Sep 17 00:00:00 2001 From: CleverThis Date: Tue, 12 May 2026 22:26:46 +0000 Subject: [PATCH] fix(context): resolve PR #10590 review blockers and compliance items MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - 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 --- src/cleveragents/application/services/acms_service.py | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/src/cleveragents/application/services/acms_service.py b/src/cleveragents/application/services/acms_service.py index 0f43dcba7..dd141ffbc 100644 --- a/src/cleveragents/application/services/acms_service.py +++ b/src/cleveragents/application/services/acms_service.py @@ -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. """