go-style-guide

Go style review identifies race conditions, goroutine lifecycle issues, and mutex misuse by analyzing code against concurrency patterns. The Skill checks for unsafe goroutine management, context handling failures, and synchronization problems that cause production failures.

Go style review validates error handling, API design, and testing patterns against established guidelines. It flags critical bugs in exception handling, poorly designed interfaces, and missing test coverage that harm long-term code quality.

Yes. The Skill applies phase-based review to completed work and pull requests, analyzing branches for critical and important issues across concurrency, errors, API design, and testing without requiring code to be merged.

The Skill focuses on production code and avoids flagging panics unnecessarily, balancing safety with pragmatism. It prioritizes critical bugs like race conditions and lifecycle errors over low-risk style inconsistencies.

Go style review consults API design reference materials to catch poorly scoped interfaces, inconsistent naming, and patterns that complicate consumption. It validates interfaces against Go conventions for clarity and usability.

The Skill leverages six reference documents covering concurrency, errors, style, API design, testing patterns, and a review checklist. These materials ensure consistent, comprehensive evaluation against Go best practices.