CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

design-pattern-advisor

Select or assess the smallest justified software design pattern for a concrete design problem. Use for pattern choice, pattern misuse, and trade-off guidance. Do not use to refactor code, audit architecture, or perform broad code review.

Quality

83%

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

73%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 well-structured advisory skill body with an excellent sequenced decision workflow, explicit validation gates, and a concrete output contract. Its weaknesses are verbose policy/workflow boilerplate that could be trimmed, no inline worked example, and a handful of dangling `shared/` reference links plus an orphaned bundle script.

Suggestions

Remove or relocate the "Base Framework" policy block, "Optional Workflow Integration" section, and redundant "Routing Boundary" section (or fold routing into the existing "Purpose and activation" section) — these add ~25 lines of boilerplate unrelated to pattern selection and are the main conciseness drag.

Fix reference integrity: remove or correct the ten `shared/base/*.md` and `shared/workflow-contract.md` links that point to files absent from the bundle, since following them dead-ends the reader.

Add a brief inline worked example (e.g., a conditional that does NOT need Strategy vs. one that does) and mention `scripts/validate_scenarios.py` in the body so the bundle script is discoverable.

DimensionReasoningScore

Conciseness

The body is information-dense and avoids explaining concepts Claude already knows, but it carries real padding: the "Base Framework" policy block with nine packaged-policy links, the "Optional Workflow Integration" section, and a "Routing Boundary" section that largely restates the "Purpose and activation" section. Per the rubric this is "mostly efficient but includes some unnecessary explanation or could be tightened" rather than the minor-trim profile of a 4.

3 / 5

Actionability

For an instruction-only skill the guidance is concrete: a five-step decision workflow with explicit stopping rules, a copy-paste output contract skeleton, per-pattern test prescriptions ("strategy contract/selection, adapter translation/failure mapping..."), and a misuse checklist with required finding elements. It misses a 5 because worked common-case examples live entirely in a reference file and the body never demonstrates an end-to-end case.

4 / 5

Workflow Clarity

The five decision steps are clearly sequenced with explicit validation checkpoints and feedback loops: "If it meets the need, recommend it and stop", "validate behavior; migrate remaining variants; remove old branching only after callers, metrics, and rollback window permit", an evidence-gated fallback ("Unable to determine - evidence needed"), failure modes to avoid, and a closing quality checklist — matching the 5 anchor's sequence-plus-validation-plus-checklist profile.

5 / 5

Progressive Disclosure

The three real bundle references (pattern-catalog.md, language-and-distributed.md, implementation-and-examples.md) exist, are one level deep, and are well signaled with purpose-scoped guidance, and the catalog/examples content is appropriately split out. However, the body links ten `shared/base/*.md` and `shared/workflow-contract.md` paths that do not exist in the bundle, and `scripts/validate_scenarios.py` is present but never referenced — organization gaps that keep this below the 5 anchor's "easy navigation" standard.

4 / 5

Total

16

/

20

Passed

Description

87%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.

A strong description: clear third-person what-statement, an explicit "Use for" trigger clause, and an exclusion list that sharply separates it from sibling review/refactoring skills. The only room for improvement is broader natural-language trigger coverage (specific pattern names, "which pattern" phrasing).

DimensionReasoningScore

Specificity

The description lists several concrete actions — "Select or assess the smallest justified software design pattern", "pattern choice, pattern misuse, and trade-off guidance" — plus an explicit negative scope ("Do not use to refactor code, audit architecture, or perform broad code review"). It falls short of a 5 because the actions are stated at an advisory level without enumerating what the skill produces (e.g., candidate ranking, adoption plan, misuse findings).

4 / 5

Completeness

It explicitly answers both questions: the "what" ("Select or assess the smallest justified software design pattern for a concrete design problem") and the "when" via a concrete "Use for pattern choice, pattern misuse, and trade-off guidance" clause, further sharpened by an explicit "Do not use to..." exclusion list. This matches the 5 anchor's requirement of clear, explicit what-and-when with concrete trigger phrases; a 4 would require the when-clause to be less explicit, which it is not.

5 / 5

Trigger Term Quality

Natural phrases users would say are present: "design pattern", "pattern choice", "pattern misuse", "trade-off guidance". Coverage is good but misses common variations users actually utter, such as specific pattern names ("Strategy", "Factory", "Singleton") or phrases like "which pattern should I use", so it is not the comprehensive synonym-level coverage of the 5 anchor.

4 / 5

Distinctiveness Conflict Risk

The niche is clear (pattern selection/assessment for a concrete design problem) and the description actively disambiguates against adjacent skills ("Do not use to refactor code, audit architecture, or perform broad code review"), minimizing the risk of triggering for refactoring, architecture, or general review requests.

5 / 5

Total

18

/

20

Passed

Validation

93%

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

Validation — 15 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

relative_links

Relative link issues: 10 missing, 9 deeper-than-1-level

Warning

Total

15

/

16

Passed

Repository

Noorkhalel/saas-ai-skills

Path

skills/design-pattern-advisor/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.