- Move MockMCPTransport import to module level in counting_mcp_transport.py,
removing the noqa: PLC0415 suppression that caused RUF100 (unused noqa)
- Remove dead `if TYPE_CHECKING: pass` block and unused TYPE_CHECKING import
- Remove unused `from typing import Any` in race condition step definitions (F401)
- Fix excess blank lines (3 → 2) between imports and first section in step file
- Remove @tdd_expected_fail tag from tdd_mcp_client_start_race.feature; the
bug is fixed so the scenario now passes, and the file's own comment states
the tag must be removed when #10438 is resolved
ISSUES CLOSED: #10438
Add _state == McpClientState.STARTING guard inside the threading.RLock in
start() and _ensure_started() so that concurrent callers see the in-progress
state and return immediately, preventing double initialisation of the MCP
server connection.
- Move _CountingTransport test double from inline step file to dedicated
features/mocks/counting_mcp_transport.py per CONTRIBUTING.md mock placement rules
- Updated step definitions to import CountingMCPTransport from new location
- Added Race condition entry to CHANGELOG under [Unreleased] -> Fixed
- Added contributor credit for McpClient race condition fix
ISSUES CLOSED: #10438
HAL9000
freemo