fix(home): MCP connect key field shows no focus ring (web-lint red)

Merged
LA
4d ago
+1-0

Base red, priority zero. The feat tip web-lint has been red since a34003dde261 (STATUS 9304): check:text-focus reports McpConnectCardParts.swift:125: a SwiftUI text field without .focusEffectDisabled(), from the direct push 385cb09a747 (MCP connect card).

The key field now has .focusEffectDisabled(), as AddMachineCard and the other SwiftUI fields do: a text field shows focus by its caret and its control's state, not a ring of its own.

Changelog

none

Observed / Not run

  • Observed: python3 scripts/cmux-next/check-text-focus-rings.py fails on the base and passes on the head; the pre-push gate passes.
  • Not run: a build or a capture of the card (one view modifier; the card is behind CMUX_MCP_CONNECT_PREVIEW=1).

Autofix with [code]smith
Open in [code]smith · Need help on this PR? Tag @codesmith-bot with what you need.


Summary by cubic

Fixes the MCP connect key field showing a focus ring, which was failing the check:text-focus lint. The key field now uses .focusEffectDisabled(), matching AddMachineCard and other SwiftUI fields: a text field shows focus through its caret and control state, not a ring of its own.