go-code-review

SKILL.md

| Missing error checks, wrapped errors | references/error-handling.md | | Race conditions, channel misuse | references/concurrency.md | | Interface pollution, naming | references/interfaces.md | | Resource leaks, defer misuse | references/common-mistakes.md |

These patterns are acceptable and should NOT be flagged as issues:

| Missing error check | Error return is actionable (can retry, log, or propagate) | | Goroutine leak | No context cancellation path exists for the goroutine | | Missing defer | Resource isn't explicitly closed before next acquisition or return | | Interface pollution | Interface has > 1 method AND only one consumer exists |

Reviews Go code for idiomatic patterns, error handling, concurrency safety, and common mistakes. Use when reviewing .go files, checking error handling, goroutine usage, or interface design. Source: existential-birds/beagle.

View raw