From dd981edea980c8af3dc24265cac0ee3b2783a28d Mon Sep 17 00:00:00 2001 From: Claude Date: Thu, 2 Apr 2026 10:55:00 +0100 Subject: [PATCH] ax(ueps): remove prose inline comments that restate code intent AX Principle 2: comments must be usage examples, not prose descriptions. "Corrupt the last byte of the payload/frame" restates what the adjacent code does rather than demonstrating a call with concrete values. Co-Authored-By: Charon --- pkg/ueps/packet_test.go | 1 - pkg/ueps/reader_test.go | 1 - 2 files changed, 2 deletions(-) diff --git a/pkg/ueps/packet_test.go b/pkg/ueps/packet_test.go index 108dac5..8933786 100644 --- a/pkg/ueps/packet_test.go +++ b/pkg/ueps/packet_test.go @@ -134,7 +134,6 @@ func TestPacket_MarshalAndSign_Ugly(t *testing.T) { t.Fatalf("MarshalAndSign failed: %v", err) } - // Corrupt the last byte of the frame (part of the payload) corrupted := make([]byte, len(frame)) copy(corrupted, frame) corrupted[len(corrupted)-1] ^= 0xFF diff --git a/pkg/ueps/reader_test.go b/pkg/ueps/reader_test.go index 2e6cdfb..c53b17d 100644 --- a/pkg/ueps/reader_test.go +++ b/pkg/ueps/reader_test.go @@ -52,7 +52,6 @@ func TestReader_ReadAndVerify_Bad(t *testing.T) { t.Fatalf("MarshalAndSign failed: %v", err) } - // Corrupt the last byte of the payload corrupted := make([]byte, len(frame)) copy(corrupted, frame) corrupted[len(corrupted)-1] ^= 0xFF