CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

code-review-web

Review web application code for bugs, security issues, performance problems, and stack-specific anti-patterns. Use this skill whenever the user wants to review code, debug a production issue, investigate a build failure, audit security, or check a PR before merging. Triggers on code review, review my code, debug, build error, broken, not working, why is X failing, check this code, security check, PR review, audit code, refactor. Also triggers when investigating 4xx or 5xx errors, deploy failures, environment variable issues, and CMS integration problems.

Quality

84%

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

SKILL.md
Quality
Evals
Security

Quality

Content

71%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.

A well-structured, navigable skill body with excellent progressive disclosure into verified reference files and two clearly sequenced workflows with useful checkpoints. The main cost is token weight: the generic Maintainability/Security checklists pay tokens for knowledge Claude already has instead of reserving the body for the genuinely non-obvious patterns (bug patterns, stack references).

Suggestions

Trim the Maintainability dimension to a one-line pointer of project-specific bars — naming, comment, and magic-constant principles are knowledge Claude already has and each bullet spends tokens restating it.

Compress Common bug pattern entries to pattern + fix (e.g., "CDN serving old asset under same filename → always use a new filename or cache-bust query string"), cutting the trailing rationale sentences.

Reserve the body's checklist budget for what is distinctive to this skill (the stack-agnostic bug patterns and stack reference routing) and move the generic security/performance checklist items into a reference file.

DimensionReasoningScore

Conciseness

The body is dense and unpadded, but sections like Maintainability ("Naming. Functions and variables named for what they do", "Comments. Explain why, not what", "No literal `60000` in code. Use named constants") and much of the Security dimension restate principles Claude already knows. This fits anchor 3 ("mostly efficient but includes some unnecessary explanation") better than anchor 4, where over-explanation would be only a minor instance rather than recurring sections.

3 / 5

Actionability

Concrete instruction-only guidance throughout: specific checks ("Session cookies have `Secure`, `HttpOnly`, `SameSite` attributes set"), a runnable command ("`git log --oneline`"), and an explicit output artifact ("a markdown document at `code-review-[date].md`" with a six-section structure). Minor gaps keep it below anchor 5 — e.g., few executable commands for verifying findings in the review workflow.

4 / 5

Workflow Clarity

Two clearly numbered sequences with several checkpoints ("Reproduce locally. Confirms it's a code issue", "Check cache state... before concluding it's a code bug", "Verify in production. Check the actual fix worked"). Not capped at 3 since this is a review skill, not a destructive/batch operation; it stays at 4 rather than 5 because there is no explicit fix-and-retry feedback loop and the review workflow lacks a verify-findings step.

4 / 5

Progressive Disclosure

The ~200-line body is a genuine overview: stack-specific detail is split into three real, one-level-deep reference files (verified on disk; they contain no references beyond a back-link to SKILL.md), clearly signaled both inline ("Use the template in references/review-template.md") and in a dedicated "Reference files" section. This matches anchor 5's clear overview with well-signaled one-level-deep references.

5 / 5

Total

16

/

20

Passed

Description

91%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.

A strong description: third-person voice, explicit what-and-when clauses, and an unusually rich set of natural trigger phrases including symptom-level ones (4xx/5xx, deploy failures, env var issues). The only weakness is minor overlap risk from broad triggers like "refactor" and generic "review code".

DimensionReasoningScore

Specificity

"Review web application code for bugs, security issues, performance problems, and stack-specific anti-patterns" lists several specific actions across the review domain. It stops short of anchor 5 because the actions are review categories rather than a comprehensive enumeration (e.g., no mention of deliverables like review reports or postmortems).

4 / 5

Completeness

It explicitly answers what ("Review web application code for bugs, security issues, performance problems, and stack-specific anti-patterns") and when ("Use this skill whenever the user wants to review code, debug a production issue, investigate a build failure, audit security, or check a PR before merging") with concrete trigger phrases. Anchor 4 would apply only if the 'when' were less explicit.

5 / 5

Trigger Term Quality

"code review, review my code, debug, build error, broken, not working, why is X failing, check this code, security check, PR review, audit code, refactor" plus "4xx or 5xx errors, deploy failures, environment variable issues, and CMS integration problems" gives comprehensive coverage of natural user phrasings and synonyms. This matches the anchor-5 example's breadth of variations.

5 / 5

Distinctiveness Conflict Risk

Distinctive triggers like 4xx/5xx errors, deploy failures, and CMS integration problems carve a clear web-review niche, but generic terms ("review code", "debug", "refactor") create minor overlap with closely related skills such as general code review or refactoring skills. It is mostly distinct, matching anchor 4 rather than anchor 5.

4 / 5

Total

18

/

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

frontmatter_unknown_keys

Unknown frontmatter key(s) found; consider removing or moving to metadata

Warning

Total

15

/

16

Passed

Repository

rampstackco/claude-skills

Path

skills/code-review-web/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.