CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

review-check-architecture

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

Quality

82%

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

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 body is a lean, fully executable skill: one copy-paste-ready script covering 12 concrete conformance checks with a defined summary-table output. The only gaps are minor — implicit pass/fail thresholds for the count-based checks and no failure-reporting guidance.

DimensionReasoningScore

Conciseness

The body is pure instruction: a one-line purpose ("Validate the entire project (or a specific module) against docs/ARCHITECTURE.md"), a scope line ("Scope: $ARGUMENTS (if empty, validate all src/modules/)"), one executable script, and a table spec. No concept explanations, no padding; nothing to trim, so it exceeds anchor 4.

5 / 5

Actionability

The 12-check bash block is copy-paste ready with concrete grep/find/wc pipelines, --include filters, 2>/dev/null guards, and PASS fallbacks, plus an explicit output format ("| Check | Status | Occurrences |" and "Overall score: X/12 checks passing"). Fully executable with no gaps.

5 / 5

Workflow Clarity

The sequence run checks → produce summary table → report "X/12 checks passing" is clear, and the checks embed their own pass/fail output. Not 5: PASS/FAIL criteria for the count-based checks (8: any types, 11: debug artifacts) are implicit, and there is no guidance on presenting failures or next steps. Not 3: the checks are read-only validations with an explicit reporting structure, so no destructive-step feedback loop is missing.

4 / 5

Progressive Disclosure

Everything lives in one "## Automated Checks" section with no external references, which is appropriate for a single-script skill and makes navigation trivial. Not 5: the body runs ~59 lines — just past the compact-skill threshold — with no per-check manifest or PASS/FAIL criteria organizing the 12 checks, though no bundle files are needed.

4 / 5

Total

18

/

20

Passed

Description

70%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 cleanly answers both what and when with a strong, literal trigger term (ARCHITECTURE.md), but it compresses the skill's capability into a single restated action. Expanding the what with the concrete outputs (12 automated checks, PASS/FAIL summary table) and adding trigger synonyms would lift specificity and completeness.

Suggestions

Enumerate concrete actions in the description, e.g. "Runs 12 automated checks (error handling, change detection, DI style, module boundaries, file size) and reports a PASS/FAIL summary table" to raise specificity beyond one restated action.

Add natural trigger variations users might say, such as "architecture review", "architecture compliance", or "validate against the architecture doc", to broaden trigger coverage.

Mention the scoping capability ("for the whole project or a single module") in the description so the what reflects what the skill actually does.

DimensionReasoningScore

Specificity

"checking if code follows ARCHITECTURE.md patterns" and "runs automated conformance checks" name the domain plus 1-2 actions, but they restate a single capability rather than listing several distinct ones (the 12 checks, summary table, and scoring appear only in the body). Fits anchor 3; not 4 because coverage of concrete actions is not several and comprehensive.

3 / 5

Completeness

Both elements are explicit: "Use when checking if code follows ARCHITECTURE.md patterns" answers when, and "runs automated conformance checks" answers what. Below 5 because the what is terse and the when offers only one trigger phrasing rather than concrete trigger phrases.

4 / 5

Trigger Term Quality

"ARCHITECTURE.md patterns" is a precise, literal trigger users would naturally say, and "conformance checks" adds a relevant keyword. Not 5 because common synonyms like "architecture review", "compliance", or "coding standards" are missing; better than anchor 3's single generic keyword.

4 / 5

Distinctiveness Conflict Risk

The description is anchored to a specific project artifact ("ARCHITECTURE.md patterns") and a niche activity ("conformance checks"), making it mostly distinct with only minor overlap risk against generic code-review skills. Not 5 because a plain "review my code" request could still plausibly trigger it.

4 / 5

Total

15

/

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