fix(cli): resolve lint errors and step name conflict in command bus
CI / push-validation (pull_request) Successful in 33s
CI / helm (pull_request) Successful in 36s
CI / lint (pull_request) Successful in 44s
CI / quality (pull_request) Successful in 51s
CI / build (pull_request) Successful in 47s
CI / typecheck (pull_request) Successful in 1m10s
CI / security (pull_request) Successful in 1m27s
CI / unit_tests (pull_request) Successful in 6m15s
CI / docker (pull_request) Successful in 1m46s
CI / integration_tests (pull_request) Successful in 9m40s
CI / coverage (pull_request) Successful in 8m29s
CI / status-check (pull_request) Successful in 3s

- Replace deprecated typing imports (Dict, Callable, Generic) with
  collections.abc.Callable and built-in dict
- Convert Command from ABC to plain base class (B024: no abstract methods)
- Convert CommandHandler to PEP 695 type parameter syntax (UP046)
- Remove unused PlanService/ProjectService imports from command_registry
- Fix ruff format: blank line before nested defs in command_registry and steps
- Rename @then('a ValueError should be raised') to domain-specific suffix
  to avoid AmbiguousStep collision with lsp_registry_steps.py

ISSUES CLOSED: #8880
This commit is contained in:
2026-06-04 10:18:40 -04:00
committed by Forgejo
parent a185069070
commit 7ed9c1279f
4 changed files with 8 additions and 18 deletions
+1 -1
View File
@@ -27,7 +27,7 @@ Feature: CLI Command Bus Decoupling
Scenario: Dispatch fails for unregistered command
Given a command bus with no handlers
When I dispatch a command without a registered handler
Then a ValueError should be raised
Then a ValueError should be raised for unregistered command dispatch
And the error message should mention the command type
Scenario: Handler check