From d400946352989961fe4142bf03d2d53fb8461f41 Mon Sep 17 00:00:00 2001 From: Claude Date: Thu, 2 Apr 2026 11:26:37 +0100 Subject: [PATCH] ax(node): rename u to peerURL in Transport.Connect Single-letter variable name violates AX Principle 1 (predictable names over short names). peerURL is self-describing without requiring context. Co-Authored-By: Charon --- pkg/node/transport.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/node/transport.go b/pkg/node/transport.go index de895b3..3613fcd 100644 --- a/pkg/node/transport.go +++ b/pkg/node/transport.go @@ -319,13 +319,13 @@ func (t *Transport) Connect(peer *Peer) (*PeerConnection, error) { if t.config.TLSCertPath != "" { scheme = "wss" } - u := url.URL{Scheme: scheme, Host: peer.Address, Path: t.config.WSPath} + peerURL := url.URL{Scheme: scheme, Host: peer.Address, Path: t.config.WSPath} // Dial the peer with timeout to prevent hanging on unresponsive peers dialer := websocket.Dialer{ HandshakeTimeout: 10 * time.Second, } - conn, _, err := dialer.Dial(u.String(), nil) + conn, _, err := dialer.Dial(peerURL.String(), nil) if err != nil { return nil, fmt.Errorf("failed to connect to peer: %w", err) }