nil-safety
Flag unguarded dereferences of optional pointer fields reachable from a malformed CR, and goroutines without recover (CWE-476).
2.72k repo starsObserved in 1 repos
Testing & Quality
Testing & Quality skills
Browse reusable Agent Skills, each with a clear purpose and practical guidance.
nonsensical-decisions
Flag unnecessary indirection, mismatched abstractions, or confusing data flow that obscures what the code is doing.
2.72k repo starsObserved in 1 repos
Testing & Quality
over-commenting
Flag comments that explain what self-documenting code already says.
2.72k repo starsObserved in 1 repos
Testing & Quality
pointless-intermediate-variables
Flag redundant local variables that add noise without adding clarity.
2.72k repo starsObserved in 1 repos
Testing & Quality
push-guards-to-callees
Review that repeated guards at multiple call sites are moved inside the callee instead.
2.72k repo starsObserved in 1 repos
Testing & Quality
race-conditions
Review concurrent Go code for races, preferring fixes at the mutation site over adding snapshot fields.
2.72k repo starsObserved in 1 repos
Testing & Quality
reinvented-helpers
Flag logic that duplicates an existing util/helper function instead of reusing it.
2.72k repo starsObserved in 1 repos
Testing & Quality
resource-bounds-dos
Flag missing resource bounds / DoS resistance — uncapped loops or allocations, missing timeouts, reconciler-wedging input, unbounded metric cardinality (CWE-400, CWE-770).
2.72k repo starsObserved in 1 repos
Testing & Quality
scope-creep
Flag diffs that bundle a bugfix with an unrelated refactor, or generalize a change to all types when only one type needs it.
2.72k repo starsObserved in 1 repos
Testing & Quality
split-test-files
Review that new feature tests are split into their own file rather than added to an already large test file.
2.72k repo starsObserved in 1 repos
Testing & Quality