From df80dbddb58511e597627cd5f4b4a82363d2b525 Mon Sep 17 00:00:00 2001 From: Claude Date: Thu, 2 Apr 2026 09:32:37 +0100 Subject: [PATCH] ax(ueps): rename remaining to payloadBytes for semantic clarity MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit AX Principle 1: names should convey semantic meaning, not implementation perspective. `remaining` describes a buffer operation; `payloadBytes` describes what the data IS — the packet payload content. Co-Authored-By: Charon --- pkg/ueps/reader.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/ueps/reader.go b/pkg/ueps/reader.go index 289562b..f29b4f7 100644 --- a/pkg/ueps/reader.go +++ b/pkg/ueps/reader.go @@ -44,11 +44,11 @@ func ReadAndVerify(reader *bufio.Reader, sharedSecret []byte) (*ParsedPacket, er if tagByte == TagPayload { // Payload is length-prefixless; caller frames the stream. // HMAC covers signedData (header TLVs) + raw payload bytes, not the 0xFF tag. - remaining, err := io.ReadAll(reader) + payloadBytes, err := io.ReadAll(reader) if err != nil { return nil, err } - payload = remaining + payload = payloadBytes break }