Fix duplicate test function declarations in logger package

[Copilot]

The logger package had duplicate test function declarations across rpc_logger_test.go and rpc_helpers_test.go, causing compilation failures. Go doesn't allow duplicate function names in the same package, even across test files.

Changes

• Removed duplicate tests from rpc_logger_test.go:

  • TestTruncateAndSanitize
  • TestExtractEssentialFields

  These functions test rpc_helpers.go and are properly tested in rpc_helpers_test.go.

• Fixed test assertions in rpc_helpers_test.go:

  • Updated sanitization expectations ("token: " → "token=" after normalization)
  • Corrected truncation length calculation (23 → 27 characters)

• Applied gofmt to fix trailing whitespace in validation_schema.go and rpc_helpers_test.go

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• this-host-does-not-exist-12345.com
  • Triggering command: /tmp/go-build3826922757/b282/mcp.test /tmp/go-build3826922757/b282/mcp.test -test.testlogfile=/tmp/go-build3826922757/b282/testlog.txt -test.paniconexit0 -test.timeout=10m0s -test.v=true submodules | head -n 10 -trimpath x_amd64/vet -p mime/quotedprintls-files -lang=go1.25 x_amd64/vet -uns�� anitize common_test.go x_amd64/vet go1.25.6 -c=4 -nolocalimports x_amd64/vet (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.