test-guard

SKILL.md

Test Guard

You are reviewing generated or changed test code before it ships. Enforce the rules below after the first test-writing pass and before the tests are presented, committed, or merged. Be a sharp reviewer, not a pedantic one: flag what wastes maintenance effort or hides real bugs, ignore cosmetic preferences.

These rules exist because coding agents over-generate tests. The common failure modes: mock-heavy unit tests that assert implementation details, near-duplicate test bodies that differ by one value, and tests that re-verify the framework instead of the project's logic. Each looks productive in a diff and costs maintenance forever.

When this skill activates

A coding agent has just written new test functions or test files, in any language

You are editing existing tests

You are reviewing a diff that contains test changes

The user asks you to write, add, or review tests

Adapt to the project first

These rules are universal, but their application is not. Before reviewing: