After completing your implementation, review your own changes against each criterion below. These are quality checks you perform by inspecting your work — not commands to run (linting and tests already ran via validationCommand). Report any unmet criterion as a blocker.
- Exported types, functions, and interfaces have doc comments
- Error handling uses
github.com/bborbe/errorswith context wrapping - No debug output (print statements, fmt.Printf) — use structured logging
- Factory functions are pure composition — no conditionals, no I/O, no
context.Background() - Follow Interface → Constructor → Struct → Method pattern
- New code has good test coverage (target >= 80%)
- Changes to existing code have tests covering at least the changed behavior
- Tests use Ginkgo v2 / Gomega with Counterfeiter mocks
- CHANGELOG.md has an entry under
## Unreleased