CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

review-tests

Reviews Chrome DevTools frontend changes in target directories (\.test\.ts$, ^test/, ^front_end/testing/), strictly reflecting review standards, architectural invariants, and security boundaries derived from historical code reviews.

Quality

77%

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 ./.agents/skills/review-tests/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

92%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 an exemplary instruction-only review skill: nine rules each with concrete grep-able detection patterns, precise non-flag exceptions, severities, and an explicit citation instruction. The only weaknesses are minor redundancy (directory patterns stated twice) and slightly verbose 'Why' rationale.

DimensionReasoningScore

Conciseness

The body is dense and almost every clause earns its place ('Look for' and 'Do not flag' lists give concrete, non-redundant review heuristics), so it is well above anchor 3. It falls short of 5 due to minor trimmable content: the target-directory regexes are repeated in both the title and intro, and some 'Why' prose re-explains knowledge Claude already has (e.g., why wall-clock setTimeout is flaky on CI).

4 / 5

Actionability

Every rule supplies exact, searchable review patterns with real identifiers — 'createTarget', 'describeWithEnvironment', 'new Promise(r => setTimeout(r, ...))', 'assert.isNotNull(...)' on 'container?.querySelector / Map.get / Array.find', '(instance as any).privateMethod()', 'as unknown as T', 'window.*' without 'afterEach' restoration — plus explicit non-flag conditions per rule. As an instruction-only (review checklist) skill this is fully actionable with specific examples covering the common cases.

5 / 5

Workflow Clarity

This is a single-task skill (match changes against the 9 rules) and the action is unambiguous: the intro explicitly instructs 'cite the rule id in findings', and every rule carries a severity (ERROR/WARNING) that orders prioritization. It is not 4 because there are no sequencing gaps for the single-action task; no destructive/batch-operation validation cap applies since the skill only reviews code.

5 / 5

Progressive Disclosure

There are no bundle files (references/, scripts/, assets/ are absent), and the single-file structure is appropriate at this size: a short intro, one 'Path group' header, and one clearly titled subsection per rule with consistent Why/Look for/Do-not-flag organization. The single external reference (docs/uiengineering.md) is clearly signaled inline, and navigation is trivial — matching anchor 5's easy-navigation, appropriately-placed-content standard rather than anchor 4's 'minor organization gaps'.

5 / 5

Total

19

/

20

Passed

Description

53%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 clearly states what it does and defines a precise directory-scoped niche, but presents triggers as regex jargon instead of natural user phrases and lacks any explicit 'Use when' guidance. It is serviceable but below the quality of the good examples, which combine concrete actions with natural trigger clauses.

Suggestions

Add an explicit trigger clause, e.g., 'Use when reviewing changes to DevTools test files (.test.ts), the test/ directory, or front_end/testing/ helpers.'

Replace or supplement the raw regex patterns ('(\.test\.ts$, ^test/, ^front_end/testing/)') with natural phrases users would say, such as 'unit tests', 'test files', or 'Mocha tests'.

List one or two concrete review capabilities instead of meta-language, e.g., 'flags flaky setTimeout-based assertions, duplicated mock factories, and unsafe type casts in tests'.

DimensionReasoningScore

Specificity

Names the domain ('Chrome DevTools frontend changes in target directories (\.test\.ts$, ^test/, ^front_end/testing/)') and one concrete action ('Reviews'), but the rest ('strictly reflecting review standards, architectural invariants, and security boundaries') is meta-description rather than capability listing. It does not reach anchor 4 because it never enumerates several specific actions (e.g., flags flaky timers, enforces fixture reuse), and is clearly above anchor 2 since the domain and a real action are stated.

3 / 5

Completeness

The 'what' is clear ('Reviews Chrome DevTools frontend changes... reflecting review standards'), but there is no 'Use when...' clause; the 'when' is only weakly implied by the parenthesized directory patterns. Per the judging guidelines a missing explicit trigger clause caps completeness at 3, and it is not 4 because the when-condition never becomes explicit.

3 / 5

Trigger Term Quality

Relevant keywords exist ('Chrome DevTools', 'frontend', 'test', 'review') but triggers are expressed as regex path patterns ('(\.test\.ts$, ^test/, ^front_end/testing/)') rather than natural user phrasing, and common variations like 'test files', 'unit tests', or 'CL review' are absent. Not 4 because natural terms users would actually say are largely missing; above 2 because the domain keywords are specific and on-target.

3 / 5

Distinctiveness Conflict Risk

The directory-scoped trigger patterns carve out a clear niche (DevTools test-path changes only), giving strong distinctiveness. It stops short of 5 because 'strictly reflecting review standards... derived from historical code reviews' overlaps with a closely related general code-review skill rather than having fully distinct natural-language triggers.

4 / 5

Total

13

/

20

Passed

Validation

93%

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

Validation — 15 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

relative_links

Relative link issues: 1 suspicious

Warning

Total

15

/

16

Passed

Repository

ChromeDevTools/devtools-frontend

Path

.agents/skills/review-tests/SKILL.md

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.