CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

review-security

Review code changes with Security Review subagent.

Quality

80%

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

Quality

Content

100%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 tight, fully executable orchestration spec: exact launch parameters, a fill-in prompt template with per-field rules, explicit feedback loops for subagent failures and checkout blockers, and a precise output format including table columns and severity ordering. Nothing is padded and nothing Claude already knows is re-explained. Its only weakness is inherited from the frontmatter description, not the content itself.

DimensionReasoningScore

Conciseness

Every line encodes a non-obvious decision rule (diff ownership, base-branch defaults, checkout blockers, retry limits, output columns) with zero explanation of concepts Claude already knows and no filler. This matches anchor 5 (lean, assumes Claude's competence, every token earns its place) rather than anchor 4, which requires identifiable instances of over-explanation to trim — none are present.

5 / 5

Actionability

The skill provides the exact subagent launch parameters ("subagent_type: \"security-review\"", "run_in_background: false"), a copy-paste prompt template with per-line inclusion rules, exact output columns ("Severity, Location (file:line), Finding"), and a worked status-line example. Anchor 5 (fully executable, covers common cases) fits; the only placeholders are inherently variable fields whose fill rules are spelled out.

5 / 5

Workflow Clarity

The sequence is unambiguous and includes explicit error-recovery feedback loops: "inspect the failure text" → correct the invocation → "retry it once immediately" → "stop" and report; and the checkout-blocker path (explain blocker → ask → "Only stash after the user confirms" → retry switch). Not a 4 because no validation checkpoint is missing — the failure, checkout, and empty-diff cases all have explicit handling, and the simple-skill exception applies.

5 / 5

Progressive Disclosure

The skill is under 50 lines with no bundle files (references/, scripts/, assets/ are absent) and no need for external references, and the body is organized into clearly delineated sections (launch config, repository path rules, base branch rules, PR special case, prompt shape, failure handling, result summary). Per the rubric guideline, this simple-skill case scores 5 on well-organized sections alone.

5 / 5

Total

20

/

20

Passed

Description

50%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 concise and states a clear single capability, but it reads more like a title than a discovery description. It lacks a "Use when..." trigger clause, broader natural trigger terms, and any enumeration of what the review produces, capping most dimensions at the midpoint. It would rarely be selected over more descriptive competitors without relying on the skill name alone.

Suggestions

Add an explicit trigger clause, e.g. "Use when the user asks for a security review, to find vulnerabilities in a diff, or mentions securing code changes."

Broaden natural trigger terms to include synonyms users actually say: "vulnerabilities", "security audit", "threats", "review a PR/branch for security issues".

Name 1-2 concrete outcomes to sharpen the 'what', e.g. "Reviews code changes for security issues and reports severity-sorted findings with file:line locations."

DimensionReasoningScore

Specificity

"Review code changes with Security Review subagent" names the domain (code changes, security) and exactly one concrete action (review), matching the anchor for 1-2 concrete actions without comprehensive coverage. It is not a 4 because it lists no additional specific actions (e.g., producing severity-sorted findings, handling PRs), and not a 2 because "review code changes" is a concrete action rather than a generic domain label.

3 / 5

Completeness

It clearly answers "what" (review code changes with the Security Review subagent) but contains no "Use when..." clause or any trigger guidance; the judging guidelines explicitly cap completeness at 3 for a missing 'when'. It is above anchor 2 because the 'what' is concrete, not vague.

3 / 5

Trigger Term Quality

The description contains the natural core terms "review", "code changes", and "Security Review", but misses common variations and synonyms users would say such as "vulnerabilities", "security audit", "threats", or "PR". This matches anchor 3 (some relevant keywords, missing variations) rather than anchor 4, which expects noticeably fuller keyword coverage.

3 / 5

Distinctiveness Conflict Risk

"Security Review subagent" gives it a security niche, but "Review code changes" overlaps considerably with generic code-review skills that share the same trigger space. This matches anchor 3 (somewhat specific, could still overlap with similar skills) rather than anchor 4, since the overlap risk with closely related review skills is more than minor.

3 / 5

Total

12

/

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.

Validation — 16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository

saski/arnesto

Path

.cursor/skills-cursor/review-security/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.