CtrlK
BlogDocsLog inGet started
Tessl Logo

lock-tests

Lock the full test inventory before any implementation code is written. Reads spec+plan+AC, writes ALL failing tests in a batch, emits a Test Inventory doc, and gates with user approval.

64

Quality

76%

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

Fix and improve this skill with Tessl

tessl review fix ./devflow-plugin/skills/lock-tests/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

81%Weight 40%Scale 1-5

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

The body is a well-sequenced, highly actionable test-locking workflow with strong validation checkpoints and a mandatory approval gate, fully executable in its operational steps. Its main weakness is conciseness — several prose passages repeat the wrapper-delegation explanation and could be tightened.

DimensionReasoningScore

Conciseness

The body is mostly efficient operational guidance but includes padding that could be tightened — notably the run-on Phase 2 paragraph and repeated explanations of 'devflow's wrapper around the upstream executing-plans skill' — so it sits at the mostly-efficient-but-could-tighten 3 anchor rather than the leaner 4.

3 / 5

Actionability

Provides fully executable bash (worktree recovery, branch-slug sanitizing, git status checks), exact file paths, concrete AskUserQuestion option strings, a literal regex, and a copy-paste Test Inventory markdown template covering the common cases, matching the score-5 anchor.

5 / 5

Workflow Clarity

Phases are explicitly sequenced with validation checkpoints — Phase 1.5 verifies each test fails for the right reason with acceptable/unacceptable failure modes, Phase 1.8 is a mandatory approval gate, Phase 0 checks git cleanliness, and the 'Important' section is a checklist — satisfying the batch/destructive feedback-loop requirement and the score-5 anchor.

5 / 5

Progressive Disclosure

The skill is a single self-contained file with no bundle references and clear phase-based section structure, avoiding nested references; it lands at 4 because organization is good and content is appropriately placed, though some inline material (the constraints table, the inventory template) could conceivably be split out, keeping it below the ideal one-level-deep reference structure of 5.

4 / 5

Total

17

/

20

Passed

Description

71%Weight 40%Scale 1-5

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

The description is specific and distinct, clearly conveying the test-locking workflow's concrete actions, but it omits any explicit 'Use when...' trigger guidance, which caps completeness at 3 and slightly weakens trigger discoverability. Adding a one-line trigger clause would raise both completeness and trigger-term quality.

Suggestions

Add an explicit 'Use when...' clause, e.g. 'Use when locking failing tests before implementation in the devflow new-feature pipeline' to satisfy the completeness trigger requirement.

Soften internal jargon ('spec+plan+AC') or pair it with plain-language synonyms ('specification, plan, acceptance criteria') so natural trigger terms match what users actually say.

Clarify the gating trigger by naming the interaction ('AskUserQuestion approval gate') to strengthen distinctiveness from generic TDD skills.

DimensionReasoningScore

Specificity

Lists multiple specific concrete actions — 'Lock the full test inventory', 'Reads spec+plan+AC', 'writes ALL failing tests in a batch', 'emits a Test Inventory doc', and 'gates with user approval' — giving comprehensive coverage of the workflow, matching the score-5 anchor rather than the score-4 anchor which expects minor gaps.

5 / 5

Completeness

Clearly states what the skill does but lacks an explicit 'Use when...' trigger clause; 'before any implementation code is written' is a sequencing hint, not a trigger, so per the missing-trigger cap it cannot exceed 3 even though the 'what' is strong.

3 / 5

Trigger Term Quality

Contains good natural keywords a pipeline user would say ('test inventory', 'failing tests', 'user approval', 'spec+plan+AC') but leans on internal jargon and misses common synonyms, placing it above the score-3 anchor yet short of the comprehensive synonym/extension coverage expected at 5.

4 / 5

Distinctiveness Conflict Risk

The 'lock-tests' / 'Test Inventory doc' / 'user-approval gate' framing carves a fairly distinct niche with low conflict risk, but it overlaps with broader TDD skills enough to land at 4 rather than the minimal-conflict 5 anchor.

4 / 5

Total

16

/

20

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.

Repository
AndreJorgeLopes/devflow
Reviewed

Table of Contents

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.