- Truncate response body in error messages to 256 bytes to prevent
sensitive data leakage from upstream APIs (client.go)
- Replace unchecked strings.Split index with safe repoShortName helper
to prevent potential panic (cmd/monitor)
- Fix test assertions to match coreerr.E error format after prior refactor
Co-Authored-By: Virgil <virgil@lethean.io>
Test assertions were out of sync with the coreerr.E() error format
introduced in 9a24df3 — updated 8 assertions from "prefix 404" to
"prefix: HTTP 404". Added round-trip tests for UpdateRecord,
EnsureRecord, ClearACMEChallenge, CreateLoadBalancer, DeleteLoadBalancer,
CreateSnapshot, GetLoadBalancer, ListLoadBalancers, GetServer, and unit
tests for HostsByRole/AppServers. Coverage: 62.5% → 80.9%. Updated
CLAUDE.md to reflect go-log/go-io dependencies and coding standards.
Co-Authored-By: Virgil <virgil@lethean.io>
Replace all remaining fmt.Errorf and errors.New calls in production code
with structured coreerr.E(op, msg, err) from forge.lthn.ai/core/go-log.
Covers 10 files across the infra package and cmd/prod and cmd/monitor.
Co-Authored-By: Virgil <virgil@lethean.io>
Resolve stale forge.lthn.ai/core/cli v0.1.0 references (tag never existed,
earliest is v0.0.1) and regenerate go.sum via workspace-aware go mod tidy.
Co-Authored-By: Virgil <virgil@lethean.io>
Production infrastructure and security monitoring commands now
live alongside the infra library they depend on.
Co-Authored-By: Virgil <virgil@lethean.io>
