CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

review-check-architecture

Use when checking if code follows ARCHITECTURE.md patterns - runs automated conformance checks.

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 ./packs/svelte/skills/review-check-architecture/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

82%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 lean, highly actionable skill whose single bash block covers 16 concrete conformance checks and defines a clear output format. The main improvements are small executable details (the find precedence quirk, interpretation of count-based checks) and grouping the checks for easier navigation.

DimensionReasoningScore

Conciseness

The body is entirely functional — a scope resolution line, 16 executable checks, and an output format spec — with no explanation of concepts Claude already knows and no padding. Every token earns its place, matching the 'Lean and efficient' anchor rather than score 4, which anticipates trimmable over-explanation.

5 / 5

Actionability

All 16 checks are concrete, executable bash commands that are essentially copy-paste ready. Not score 5 due to minor executable gaps: check 16's 'find ... -name "+page.ts" -o -name "+page.server.ts"' lacks parentheses (find's implicit -print binds only to the last predicate, silently skipping +page.ts files), and checks 10/13 output bare 'wc -l' counts with no guidance on interpreting them as pass/fail.

4 / 5

Workflow Clarity

The sequence is unambiguous — run the checks, resolve scope ('if empty, validate all src/lib/modules/'), fill in the summary table, compute the overall score. Not score 5 because there is no guidance on what to do with FAIL results or how to fill the table for count-based checks; the workflow cap of 3 does not apply since the skill is itself a read-only validation checker, not a destructive or unvalidated batch operation.

4 / 5

Progressive Disclosure

Good structure for a self-contained skill: a clear heading, explicit scope, a single cohesive check script, and a defined output format. Not score 5 because the body exceeds 50 lines and the 16 checks could be grouped (Svelte 4 migration vs SvelteKit 1 vs general hygiene) or moved to a scripts/ file to slim the overview.

4 / 5

Total

17

/

20

Passed

Description

62%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 functional and correctly structured with an explicit 'Use when...' trigger and a concrete stated action, but it undersells the skill's scope. Enriching it with the specific check areas and more natural trigger phrases would raise specificity and trigger-term quality.

Suggestions

List 2-3 concrete capabilities in the description (e.g., 'greps for Svelte 4 legacy syntax, cross-module imports, oversized components, and missing error handling') to raise specificity.

Add natural trigger variations users would actually say, such as 'architecture review', 'conformance check', or 'does this follow the architecture doc', to improve trigger-term coverage.

State the output artifact explicitly (e.g., 'produces a pass/fail summary table with a X/16 score') so the 'what' is more complete and distinctive.

DimensionReasoningScore

Specificity

Names the domain ('checking if code follows ARCHITECTURE.md patterns') and one concrete action ('runs automated conformance checks'), but does not enumerate what the conformance checks actually cover. It matches the anchor 'Names domain and 1-2 concrete actions, but not comprehensive' rather than score 4, which requires several specific actions.

3 / 5

Completeness

Both parts are explicit: 'what' via 'runs automated conformance checks' and 'when' via the 'Use when checking if code follows ARCHITECTURE.md patterns' clause. Not score 5 because the when-clause provides only a single trigger phrase without concrete variations or synonyms; not score 3 because neither part is missing or merely implied.

4 / 5

Trigger Term Quality

Relevant keywords like 'architecture', 'patterns', and 'conformance' are present, but common natural variations users would say ('architecture review', 'standards', 'does this follow the architecture doc') are missing. This fits 'Some relevant keywords but missing common variations or synonyms' rather than score 4's 'Good keyword coverage'.

3 / 5

Distinctiveness Conflict Risk

The description is anchored to a specific artifact ('ARCHITECTURE.md') and a distinct purpose (automated conformance checking), giving it a clear niche. Not score 5 because it could still overlap with generic code-review or architecture-review skills.

4 / 5

Total

14

/

20

Passed

Validation

87%

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

Validation — 14 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

allowed_tools_field

'allowed-tools' contains unusual tool name(s)

Warning

frontmatter_unknown_keys

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

Warning

Total

14

/

16

Passed

Repository

fernandezbaptiste/specialist-agent

Path

packs/svelte/skills/review-check-architecture/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.