From aa36675cfb6d8962210dd4be059949fedd34424c Mon Sep 17 00:00:00 2001 From: Claude Date: Thu, 2 Apr 2026 13:33:02 +0100 Subject: [PATCH] ax(node): rename receiver h to handler in ResponseHandler (AX Principle 1) Single-letter receiver `h` requires mental mapping to understand it refers to *ResponseHandler. Renamed to `handler` for self-documenting, predictable naming. Co-Authored-By: Charon --- pkg/node/protocol.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/pkg/node/protocol.go b/pkg/node/protocol.go index a951da9..8e985f1 100644 --- a/pkg/node/protocol.go +++ b/pkg/node/protocol.go @@ -19,7 +19,7 @@ func (e *ProtocolError) Error() string { type ResponseHandler struct{} // if err := handler.ValidateResponse(resp, MsgPong); err != nil { return 0, err } -func (h *ResponseHandler) ValidateResponse(resp *Message, expectedType MessageType) error { +func (handler *ResponseHandler) ValidateResponse(resp *Message, expectedType MessageType) error { if resp == nil { return &ProtocolError{Code: ErrCodeUnknown, Message: "nil response"} } @@ -43,8 +43,8 @@ func (h *ResponseHandler) ValidateResponse(resp *Message, expectedType MessageTy // var stats StatsPayload // if err := handler.ParseResponse(resp, MsgStats, &stats); err != nil { return nil, err } -func (h *ResponseHandler) ParseResponse(resp *Message, expectedType MessageType, target interface{}) error { - if err := h.ValidateResponse(resp, expectedType); err != nil { +func (handler *ResponseHandler) ParseResponse(resp *Message, expectedType MessageType, target interface{}) error { + if err := handler.ValidateResponse(resp, expectedType); err != nil { return err }