CtrlK
BlogDocsLog inGet started
Tessl Logo

architect-review

Master software architect specializing in modern architecture patterns, clean architecture, microservices, event-driven systems, and DDD. Reviews system designs and code changes for architectural integrity, scalability, and maintainability. Use PROACTIVELY for architectural decisions.

32

Quality

28%

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 ./.agent/skills/architect-review/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

7%

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

This skill is essentially a persona description and a table of contents with no actionable content. It tells Claude to be an 'elite software architect' but provides zero concrete guidance on how to actually perform architectural reviews—no evaluation frameworks, no example outputs, no checklists, no specific patterns to apply. The sub-skill references suggest structure but the main file adds almost nothing Claude doesn't already know.

Suggestions

Replace the vague 4-step workflow with a concrete architectural review checklist (e.g., specific questions to ask about coupling, data flow, failure modes, with example outputs for each).

Add at least one concrete example showing an input (e.g., a system design description) and the expected architectural review output format with specific findings, risk ratings, and recommendations.

Remove the 'Expert Purpose' section entirely—it restates the description and adds no actionable information Claude can use.

Add validation steps to the workflow, such as 'Verify the architecture satisfies these quality attributes: [list]' with specific criteria for pass/fail.

DimensionReasoningScore

Conciseness

The content is verbose and explains concepts Claude already knows well (SOLID principles, microservices, DDD, clean architecture). The 'Expert Purpose' section largely restates the description. Phrases like 'Elite software architect' and 'Masters modern architecture patterns' are filler that add no actionable value.

1 / 3

Actionability

The instructions are entirely abstract ('Gather system context', 'Evaluate architecture decisions', 'Recommend improvements') with no concrete examples, specific commands, code snippets, checklists, or templates. There is nothing executable or copy-paste ready—it describes rather than instructs.

1 / 3

Workflow Clarity

The four-step workflow is vague and lacks any validation checkpoints, specific criteria for evaluation, or feedback loops. Steps like 'Evaluate architecture decisions and identify risks' provide no guidance on how to actually perform the evaluation or what constitutes a risk.

1 / 3

Progressive Disclosure

The skill does reference 10 sub-skill files with clear one-level-deep links, which is good structure. However, no bundle files were provided to verify these references exist, and the main SKILL.md itself contains very little substantive content—it's mostly a table of contents with no useful overview content to anchor the references.

2 / 3

Total

5

/

12

Passed

Description

50%

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 identifies a clear domain (software architecture) and lists relevant architectural paradigms, but relies heavily on buzzwords rather than concrete actions. The trigger guidance is too vague ('Use PROACTIVELY for architectural decisions') to reliably distinguish when this skill should be selected. The use of 'Master software architect' in first/third person framing is more of a role declaration than a capability description.

Suggestions

Replace the role declaration ('Master software architect') with specific actions like 'Evaluates service boundaries, designs API contracts, defines domain models, and reviews code for architectural pattern compliance'.

Expand the 'Use when...' clause with concrete trigger scenarios, e.g., 'Use when the user asks about service decomposition, domain modeling, event-driven design, system design review, or choosing between architectural patterns'.

Add natural user-facing terms like 'system design', 'service boundaries', 'API design', 'monolith', 'hexagonal architecture' to improve trigger term coverage.

DimensionReasoningScore

Specificity

Names the domain (software architecture) and some actions ('reviews system designs and code changes'), but the capabilities are mostly listed as buzzword categories (clean architecture, microservices, DDD) rather than concrete actions the skill performs.

2 / 3

Completeness

The 'what' is partially addressed (reviews designs for architectural integrity), and there is a 'when' clause ('Use PROACTIVELY for architectural decisions'), but the trigger guidance is vague and non-specific—'architectural decisions' is too broad to serve as a clear trigger.

2 / 3

Trigger Term Quality

Includes some relevant keywords like 'microservices', 'event-driven systems', 'DDD', 'scalability', and 'architectural decisions', but these are more technical jargon than natural user phrases. Missing common variations like 'system design', 'service boundaries', 'API design', 'monolith to microservices'.

2 / 3

Distinctiveness Conflict Risk

While it targets architecture specifically, terms like 'code changes', 'scalability', and 'maintainability' could easily overlap with general code review skills. The broad scope (microservices, DDD, clean architecture, event-driven) makes it somewhat distinctive but still risks conflicting with more focused design or review skills.

2 / 3

Total

8

/

12

Passed

Validation

81%

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

Validation9 / 11 Passed

Validation for skill structure

CriteriaDescriptionResult

metadata_version

'metadata.version' is missing

Warning

frontmatter_unknown_keys

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

Warning

Total

9

/

11

Passed

Repository
Dokhacgiakhoa/antigravity-ide
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.