CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

review-check-architecture

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

Quality

79%

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

Quality

Content

90%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 content is an exemplar of lean, fully executable skill authoring: one runnable script covering 14 concrete checks with a defined output artifact. The only notable gap is that pass/fail interpretation for the count-based and semantically inverted checks is left implicit rather than stated as a rule.

DimensionReasoningScore

Conciseness

The body is a one-line objective, a scope line, a dense copy-paste bash block, and a table format — every token earns its place and nothing explains concepts Claude already knows. This matches anchor 5 ('Lean and efficient; assumes Claude's competence'); there is no padding that would pull it to anchor 4.

5 / 5

Actionability

All 14 checks are fully executable copy-paste bash commands (grep/find/wc pipelines) covering the common architecture conformance cases, plus an explicit output format. This matches anchor 5 ('Fully executable; copy-paste ready code or commands; specific examples cover the common cases').

5 / 5

Workflow Clarity

The sequence is clear — run the checks, produce the PASS/FAIL summary table, report X/14 — and the checks are read-only so no destructive-operation validation cap applies. It falls short of anchor 5 because PASS/FAIL semantics must be inferred: count-based checks (e.g., 'wc -l' for any-types and debug artifacts) emit no PASS/FAIL signal, and check names like 'Services without try/catch' invert the grep pattern's meaning without an explicit rule (occurrences > 0 = FAIL).

4 / 5

Progressive Disclosure

No bundle files exist and the skill is a single self-contained script (~65 lines) that does not need external references; the bash block and summary table are appropriately placed inline. It sits at anchor 4 ('Good structure; most content is appropriately placed; minor organization gaps') rather than 5 because the reporting/summary instructions lack a section header and the body slightly exceeds the under-50-lines simple-skill threshold.

4 / 5

Total

18

/

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 correctly uses third person, includes an explicit 'Use when' trigger clause, and anchors itself to a specific project document (ARCHITECTURE.md) for distinctiveness. Its main weakness is thin capability detail — 'runs automated conformance checks' says nothing about what kinds of patterns are verified — and limited keyword variation for triggering.

Suggestions

Enumerate 2-3 concrete check categories in the description, e.g., 'Checks error-handling placement, state-management boundaries, cross-module imports, and component size against ARCHITECTURE.md rules.'

Add natural trigger synonyms users might say, such as 'architecture review', 'conformance', 'standards compliance', or 'audit against the architecture doc'.

State the output explicitly (e.g., 'produces a PASS/FAIL summary table with an overall score') so the 'what' is as concrete as the 'when'.

DimensionReasoningScore

Specificity

The description names the domain and roughly two actions — "checking if code follows ARCHITECTURE.md patterns" and "runs automated conformance checks" — but never enumerates what the conformance checks actually cover (e.g., state management, error handling, module boundaries). This matches anchor 3 ('Names domain and 1-2 concrete actions, but not comprehensive'); it is above anchor 2 because the actions are concrete, but below anchor 4 because no specific check categories are listed.

3 / 5

Completeness

Both parts are explicit: the 'what' is "runs automated conformance checks" and the 'when' is "Use when checking if code follows ARCHITECTURE.md patterns". It does not reach anchor 5 because the 'what' is generic ('automated conformance checks' gives no concrete detail about what is checked), matching anchor 4 ('Has both what and when; when could be more explicit or specific').

4 / 5

Trigger Term Quality

Relevant keywords are present ("checking", "code follows", "ARCHITECTURE.md patterns", "conformance checks") and they mirror natural user phrasing, but common variations like "architecture review", "compliance", "standards", or "audit" are missing. This fits anchor 3 ('Some relevant keywords but missing common variations or synonyms') better than anchor 4's broader multi-term coverage.

3 / 5

Distinctiveness Conflict Risk

Naming the project-specific "ARCHITECTURE.md" file gives the skill a clear niche that distinguishes it from generic code-review or lint skills, leaving only minor overlap risk with closely related review skills. It is not anchor 5 because the phrasing 'checking if code follows patterns' alone would still overlap with general review triggers if the ARCHITECTURE.md reference were absent.

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/react/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.