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

The body is a lean, highly executable validation script with a clear reporting workflow and no wasted tokens. Its main weaknesses are minor: ambiguous pass/fail output in several checks and no instruction for how to act on failures.

DimensionReasoningScore

Conciseness

The body is almost entirely an executable bash block ("grep -rn \"try {\" src/modules/*/services/...") plus a two-line intro, a scope line, and a compact output-table spec. There is no explanation of concepts Claude already knows and no padding; every token earns its place.

5 / 5

Actionability

Eighteen copy-paste-ready grep/find/ls commands cover the common conformance cases, matching anchor 4. It falls short of anchor 5 because several checks emit no clear pass signal (checks 3, 7, 8, 11, 14, 15 print nothing on pass; checks 10 and 13 print raw wc -l counts where 0 means pass), which complicates reliably filling the mandated PASS/FAIL table.

4 / 5

Workflow Clarity

The sequence is clear: resolve scope ($ARGUMENTS or all src/modules/), run all checks, produce a summary table, report X/18 passing. Checks are read-only so the destructive-operation cap does not apply, but there is no guidance on what to do with FAIL or WARN results (report only vs. suggest fixes) and the table format is only partially specified ('| ... | ... |'), keeping it at anchor 4 rather than 5.

4 / 5

Progressive Disclosure

Good structure with well-labeled numbered checks and clear sections, and the inline script is the core action rather than deferred reference material, fitting anchor 4. It is not anchor 5 because the 75-line bash block exceeds the simple short-skill case and could be externalized to a scripts/ file with SKILL.md acting as 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 well-formed with an explicit 'Use when' trigger clause and a distinct ARCHITECTURE.md niche, but it is thin on capability specifics and natural trigger variations. It reads as a single narrow trigger phrase plus a generic action statement rather than a comprehensive coverage of what the skill does.

Suggestions

Enumerate 2-3 concrete capabilities in the description (e.g., 'scans services, hooks, actions, and stores for architecture violations and reports a pass/fail summary per check') to raise specificity.

Add natural trigger variations users would actually say, such as 'architecture review', 'architecture compliance', 'architecture violations', or 'standards check', alongside the existing ARCHITECTURE.md reference.

Clarify the output promise (e.g., 'produces a summary table with an overall X/18 conformance score') so the 'what' is as concrete as the 'when'.

DimensionReasoningScore

Specificity

The description names the domain and 1-2 concrete actions ("checking if code follows ARCHITECTURE.md patterns", "runs automated conformance checks") but does not list several specific actions such as the individual conformance checks or reporting, so it matches anchor 3 rather than 4.

3 / 5

Completeness

It explicitly answers both 'what' ("runs automated conformance checks") and 'when' ("Use when checking if code follows ARCHITECTURE.md patterns"). It falls short of anchor 5 because the trigger guidance is a single narrow phrase with no synonym variations.

4 / 5

Trigger Term Quality

It includes some relevant keywords ("checking", "code", "ARCHITECTURE.md", "patterns", "conformance") but leans on one filename and the jargony "conformance", missing natural variations users would say like "architecture review", "compliance", "violations", or "standards".

3 / 5

Distinctiveness Conflict Risk

The specific "ARCHITECTURE.md" reference carves a clear niche with minor overlap risk against generic code-review or lint skills. It is not anchor 5 because "checking if code follows patterns" remains fairly broad without the filename anchor.

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