CtrlK
BlogDocsLog inGet started
Tessl Logo

testland/test-removal-criteria

Decides which existing tests should stop existing, using five removal classes (duplicate, tautology, trivial, dead-signal, orphan) and a four-condition delete gate where every condition must hold or the verdict reverts to keep. Puts the burden of proof on removal: each deletion carries a written reason, redundant-coverage evidence from a per-test source map, and a named reviewer; a test that has merely never failed is treated as possibly load-bearing, not worthless. Use when a suite has outgrown its signal value and someone is opening a pruning change set that deletes test files; for choosing which tests to RUN per change (not delete) use regression-suite-selector, and to track accumulating low-value coverage over time use coverage-debt-tracker.

75

Quality

94%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide

SecuritybySnyk

Passed

No findings from the security scan

Overview
Quality
Evals
Security
Files

Quality

Content

85%

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

A thorough, well-structured judgment skill with strong actionability, an explicit validation gate over destructive batch deletes, and clean one-level progressive disclosure. Its one weakness is conciseness: persuasive rationale and repeated external citations add length beyond what the action requires.

Suggestions

Tighten conciseness by collapsing the five 'burden of proof' rules and the anti-patterns table's repeated Meszaros/Fowler quotes into a single citations block, keeping only the rule text inline.

Move the extended source quotations (e.g. the Meszaros Lost Test and Fowler TestCoverage passages) into a references file, leaving a one-line summary plus link in the body.

Trim rhetorical framing such as 'The asymmetry is the whole reason this skill is written conservatively' to a single declarative statement so every remaining token earns its place.

DimensionReasoningScore

Conciseness

The body is ~280 lines with substantial persuasive rationale and lengthy inline source quotes (e.g. Meszaros/Fowler citations restated at length in rules and the anti-patterns table); while it does not explain basic concepts Claude already knows, it could be tightened toward the level-3 'every token earns its place' anchor by trimming rhetorical framing and consolidating repeated citations.

2 / 3

Actionability

Concrete, executable guidance throughout — a per-class signal table, normalized signature definitions, an explicit four-condition gate, an executable TypeScript fold example, and a worked example with real rows — matching the level-3 'specific examples, copy-paste ready' anchor; it is not level 2 because guidance is complete rather than pseudocode or vague.

3 / 3

Workflow Clarity

A clear five-step sequence (Classify → apply class test → delete gate → choose action → record) with the conjunctive gate as an explicit validation checkpoint over destructive/batch deletes and a missing-input-as-failure feedback rule, satisfying the level-3 anchor; it is not level 2 because checkpoints and error-recovery handling are explicit, not implicit.

3 / 3

Progressive Disclosure

Overview body points to two real, clearly-signaled one-level-deep references (references/applying-the-removal-classes.md and references/removal-ledger-and-report-format.md, both verified to exist) with detailed material split out appropriately, matching the level-3 'clear overview with well-signaled one-level-deep references' anchor.

3 / 3

Total

11

/

12

Passed

Description

100%

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

A specific, well-scoped description that states concrete capabilities, gives an explicit Use-when trigger, and cleanly separates itself from neighboring selection/tracking skills. Voice is correctly third person throughout.

DimensionReasoningScore

Specificity

Names multiple concrete actions — 'five removal classes (duplicate, tautology, trivial, dead-signal, orphan)', 'a four-condition delete gate', 'a written reason, redundant-coverage evidence from a per-test source map, and a named reviewer' — matching the level-3 anchor of several specific concrete actions rather than a single domain/action at level 2.

3 / 3

Completeness

Explicitly answers both what ('Decides which existing tests should stop existing...') and when via an explicit 'Use when a suite has outgrown its signal value and someone is opening a pruning change set that deletes test files' clause, satisfying the level-3 anchor; it is not level 2 because the when-trigger is stated, not merely implied.

3 / 3

Trigger Term Quality

Natural trigger phrasing a user would say appears directly — 'opening a pruning change set that deletes test files' and 'which tests to RUN per change' — giving good coverage of how the need actually surfaces, not just technical jargon; it is not level 2 because common variations (delete/remove tests, prune a suite) are well represented.

3 / 3

Distinctiveness Conflict Risk

Carves a clear niche — 'owns the second question only' of removal vs selection — and explicitly routes adjacent needs elsewhere ('for choosing which tests to RUN per change use regression-suite-selector, and to track accumulating low-value coverage use coverage-debt-tracker'), making misfires unlikely; it is not level 2 because the boundary against sibling skills is drawn explicitly.

3 / 3

Total

12

/

12

Passed

Validation

100%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation16 / 16 Passed

Validation for skill structure

No warnings or errors.

Reviewed

Table of Contents