mutation-reviewlisted

Propose→execute mutation-review protocol for co-located unit tests — mock-boundary discipline, assertion-strength anti-patterns, the propose/execute split, the (AC-n) test-title convention, and the blocker-mutant emission contract. Runner commands and repo-specific blocker mutants come from the consumer config and extension files.
manoldonev/second-shift · ★ 1 · Code & Development · score 67

Install: claude install-skill manoldonev/second-shift

# Mutation review (propose → execute) This skill defines the **generic** mutation-review protocol shared by `unit-test-mutation-reviewer` (proposes) and the pipeline's mutation-gate sequencer (executes). It is repo-agnostic: the test-runner command comes from the consumer config, and any repo-specific blocker-mutant classes come from `.claude/second-shift/blocker-mutants.md`. Repo conventions (which test framework, co-location rules, fixture tricks) live in the consuming repo's own testing skill, not here. **Runner commands come from config.** The executor runs the repo's configured `test` command (from `<repo-root>/.claude/second-shift.config.json` `commands.<host>.test`, env override `SECOND_SHIFT_CONFIG`) scoped to the spec path — this skill never hardcodes a package manager or command. Where a step below says "run the spec", it means: invoke the configured `test` command against that spec file. ## Mock boundary | Layer | Prefer | | ---------------------- | ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | | Service business logic | Real collaborators where cheap; mock only I/O boundaries (the data-access handle, external HTTP, the queue/broker)