go-generics — independently scanned and version-tracked by SaferSkills.
SaferSkills independently audited go-generics (Agent Skill) and scored it 100/100 (green). The audit ran 55 deterministic rules across Security, Supply Chain, Maintenance, Transparency, and Community; it found 0 high-severity and 0 lower-severity findings. The full rule-by-rule trace and per-finding evidence are below. Free, methodology-open.
Findings & checks · 0 flagged
Every scanned point with the score it earned and what moved between them.
First recorded scan — no prior version to compare against.
The primary manifest — the file an agent reads to learn what this artifact does.
Start with concrete types. Generalize only when a second type appears.
any and excessive type switching"Write code, don't design types." — Robert Griesemer and Ian Lance Taylor
Do multiple types share identical logic?
├─ No → Use concrete types
├─ Yes → Do they share a useful interface?
│ ├─ Yes → Use an interface
│ └─ No → Use genericsBad:
// Premature generics: only ever called with int
func Sum[T constraints.Integer | constraints.Float](vals []T) T {
var total T
for _, v := range vals {
total += v
}
return total
}Good:
func SumInts(vals []int) int {
var total int
for _, v := range vals {
total += v
}
return total
}| Name | Typical Use |
|---|---|
T | General type parameter |
K | Map key type |
V | Map value type |
E | Element/item type |
For complex constraints, a short descriptive name is acceptable:
func Marshal[Opts encoding.MarshalOptions](v any, opts Opts) ([]byte, error)Type aliases (type Old = new.Name) are rare — use only for package migration or gradual API refactoring.
Combine constraints with ~ (underlying type) and | (union):
type Numeric interface {
~int | ~int8 | ~int16 | ~int32 | ~int64 |
~float32 | ~float64
}
func Sum[T Numeric](vals []T) T {
var total T
for _, v := range vals {
total += v
}
return total
}Use the constraints package or cmp package (Go 1.21+) for standard constraints like cmp.Ordered instead of writing your own.
Read references/CONSTRAINTS.md when writing custom type constraints, composing constraints with ~ and |, or debugging type inference issues.
// Bad: generic wrapper adds complexity without value
type Set[T comparable] struct {
m map[T]struct{}
}
// Better: use map[T]struct{} directly when the usage is simple
seen := map[string]struct{}{}Generics justify their complexity when they eliminate duplication across multiple call sites. A single-use generic is just indirection.
// Bad: T is only used to satisfy an interface — just use the interface
func Process[T io.Reader](r T) error { ... }
// Good: accept the interface directly
func Process(r io.Reader) error { ... }// Bad: constraint is more restrictive than needed
func Contains[T interface{ ~int | ~string }](slice []T, target T) bool { ... }
// Good: comparable is sufficient
func Contains[T comparable](slice []T, target T) bool { ... }| Topic | Guidance | |
|---|---|---|
| When to use generics | Only when multiple types share identical logic and interfaces don't suffice | |
| Starting point | Write concrete code first; generalize later | |
| Naming | Single uppercase letter (T, K, V, E) | |
| Type aliases | Same type, alternate name; use only for migration | |
| Constraint composition | Use ~ for underlying types, ` | for unions; prefer cmp.Ordered` over custom |
| Common pitfall | Don't genericize single-use code or when interfaces suffice |
~30 seconds. Free. No account. Every finding cites a rule and a line of evidence.