From 5379dc7c7c28a42ca078afccc8a342ee7a16d7ac Mon Sep 17 00:00:00 2001 From: Reese Norris Date: Sun, 12 Jul 2026 20:00:31 -0400 Subject: [PATCH] fix: address remaining AGENTS.md nits --- AGENTS.md | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/AGENTS.md b/AGENTS.md index 3d06d42..7f9f053 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -132,10 +132,10 @@ Checklist: ## 7. PR checklist (all PRs) - [ ] `go test -race ./...` +- [ ] `gofmt -l .` clean (CI enforces) - [ ] `golangci-lint run` (advisory in early PRs; hard fail later) - [ ] Coverage floors for this milestone (see §8); report via `scripts/coverage.sh` -- [ ] No `panic(` in non-test `pkg/` / `internal/` (`scripts/check-hygiene.sh`) -- [ ] No `fmt.Print` in library `pkg/` / `internal/` (use `slog`) +- [ ] Hygiene clean: no `panic(`, no `fmt.Print*` / `log.Print*` / `log.Fatal*` / `log.Panic*` in non-test `pkg/` / `internal/` (`scripts/check-hygiene.sh`; see Hygiene rules table) - [ ] No `reflect` in `pkg/protocol` - [ ] If web: boring-web checklist (§6) - [ ] If protocol/handler after **PR11**: e2e green @@ -229,6 +229,7 @@ golangci-lint run ``` Lint is **advisory** in CI (`continue-on-error: true`) until a dedicated cleanup PR removes legacy findings and hard-fails the gate. + ### E2E (after PR11) ```bash