From 21d205a0dc2acaa0fdd1ea06e0b1da8b129acef8 Mon Sep 17 00:00:00 2001 From: Claude Date: Thu, 2 Apr 2026 09:37:22 +0100 Subject: [PATCH] ax(node): rename pr to registry in NewPeerRegistryWithPath MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit `pr` abbreviates PeerRegistry — violates AX Principle 1 (predictable names over short names). Renamed to `registry` so the variable is self-describing without context. Co-Authored-By: Charon --- pkg/node/peer.go | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/pkg/node/peer.go b/pkg/node/peer.go index e4f68af..4ddac60 100644 --- a/pkg/node/peer.go +++ b/pkg/node/peer.go @@ -135,7 +135,7 @@ func NewPeerRegistry() (*PeerRegistry, error) { // registry, err := node.NewPeerRegistryWithPath("/tmp/test-peers.json") // used in tests to avoid xdg path caching func NewPeerRegistryWithPath(peersPath string) (*PeerRegistry, error) { - pr := &PeerRegistry{ + registry := &PeerRegistry{ peers: make(map[string]*Peer), path: peersPath, stopChan: make(chan struct{}), @@ -144,14 +144,14 @@ func NewPeerRegistryWithPath(peersPath string) (*PeerRegistry, error) { } // Try to load existing peers - if err := pr.load(); err != nil { + if err := registry.load(); err != nil { // No existing peers, that's ok - pr.rebuildKDTree() - return pr, nil + registry.rebuildKDTree() + return registry, nil } - pr.rebuildKDTree() - return pr, nil + registry.rebuildKDTree() + return registry, nil } // registry.SetAuthMode(PeerAuthAllowlist) // require pre-registration