Golang best practices skill
Production-ready Go code review skill for AI agents based on authoritative sources
npx -y skills add saifoelloh/golang-best-practices-skillAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
One thing to look at
- 16 stars16 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.
What its author says it does
Copied from the file, not written here
Comprehensive Go + GORM + PostgreSQL review meta-skill. Coordinates 9 specialized domain skills for fullstack Go application audits. Covers concurrency, architecture, idioms, queries, SQL performance, and migration safety.
The file declares its own license as MIT. That is the author’s claim about this one file, and it is not the same thing as the license GitHub reports for the repository, which is listed with the other numbers below.
SKILL.md
3.0 KB, as published. Nobody here has run it
Golang Fullstack Best Practices (Meta-Skill)
Comprehensive code review skill coordinating 9 specialized domain skills for Go backend services using GORM with PostgreSQL.
v3.0.0: The "Ultimate Merger" release. Unified 89 rules across Go language and Database layers.
Available Skills
Go Language Domains
- Concurrency Safety — Goroutines, channels, race conditions.
- Clean Architecture — Layered architecture, gRPC, dependency rules.
- Design Patterns — Code smells, GoF patterns, refactoring.
- Idiomatic Go — Go conventions, interface design, pointers.
Database Layer Domains
- GORM Query Patterns — ORM queries, transactions, context.
- PostgreSQL Syntax — Raw SQL, Postgres-specific functions, quoting.
- Query Performance — N+1 detection, indexing, connection pooling.
- Migration Safety — Zero-downtime schema changes, rollbacks.
Unified Core
- Error Handling — Combined Go error patterns + PostgreSQL error codes.
Rule Count by Priority
| Priority | Count | Focus |
|---|---|---|
| CRITICAL | 20 | Bugs, crashes, data loss, SQL injection |
| HIGH | 30 | Reliability, performance, architecture |
| MEDIUM | 34 | Quality, idioms, maintainability |
| ARCH | 5 | Clean Architecture compliance |
| TOTAL | 89 |
Unified Routing Logic
- Concurrency/Channels/Goroutines → concurrency-safety
- Layers/gRPC/Usecases/Dipendencies → clean-architecture
- Complex logic/Refactoring/Code smells → design-patterns
- Pointer usage/Interfaces/Idioms → idiomatic-go
- DB Errors/Postgres codes/errors.Is/As → error-handling
- GORM queries/db.Find/Transactions → gorm-query-patterns
- Raw SQL/Pg-specific functions/Syntax → postgresql-syntax
- Slow queries/Indexes/N+1/Pool → query-performance
- ALTER TABLE/AutoMigrate/Rollback → migration-safety
- Full Audit → all 9 skills
How to Use
For Full Audit
"Review this Go service for all language and database best practices"
For Targeted Reviews
"Audit this repository file for GORM query performance and N+1 issues" "Review these goroutines for race conditions and context leaks" "Is this database migration safe for production?"