CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

design-spec-review

Warn when user-facing UI changes drift from DESIGN.md. Advisory only; findings never gate Warden clearance.

61

Quality

77%

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 ./.warden/skills/design-spec-review/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 body is a dense, highly actionable review checklist: every bullet names a concrete detection pattern tied to a numbered DESIGN.md rule, with explicit severity mapping, scope exclusions, and report format. Its only real weakness is that the workflow (read DESIGN.md → scope the diff → check categories → report) is implied by section order rather than stated as an explicit sequence.

DimensionReasoningScore

Conciseness

The body is telegraphic, checklist-style bullets with zero padding and no explanation of concepts Claude already knows — every line encodes repo-specific detection heuristics ("`border` + `box-shadow` on the same lifted surface", "durations > 300ms on UI controls", "compare document scrollWidth to measured clientWidth"). This clearly matches the lean/efficient anchor; not 4, as there are no over-explanations to trim.

5 / 5

Actionability

For an instruction-only skill the guidance is fully executable: exact paths to grep (`apps/app/src/components/ui/`), exact code patterns (`text-gray-400`, `#000`, `outline: none`, `CardDescription` under `CardTitle`), concrete thresholds (320px viewport, 305px usable, >300ms), an explicit severity mapping (medium/low, never high), and a defined report format. It is not 4 — there are no minor gaps in the concrete guidance.

5 / 5

Workflow Clarity

The sequence is discernible and stated where it matters ("Read `DESIGN.md` at the repository root first", then scope changed lines, check the category checklists, then report per the format section) with inline checkpoints ("Grep ... before reporting", scope exclusions, zero-findings-is-valid). It is not 5 because the workflow is never laid out as an explicit ordered sequence with validation checkpoints, leaving the overall flow implicit.

4 / 5

Progressive Disclosure

No bundle files exist, and the single file is well-sectioned (What to check / Do NOT report / Report format) with its one external reference (DESIGN.md) clearly signaled as the source of truth. It is not 5 because the ~123-line rule catalog is entirely inlined with no split of any kind, and not 3 because the structure and navigation are genuinely good.

4 / 5

Total

18

/

20

Passed

Description

53%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 concise and clearly states what the skill does and its advisory nature, with a distinct niche anchored by DESIGN.md and Warden clearance. Its main weakness is the absence of any explicit "Use when..." trigger guidance and natural trigger synonyms, which caps both completeness and trigger term quality at 3.

Suggestions

Add an explicit trigger clause, e.g. "Use when reviewing diffs or PRs that touch user-facing UI (apps/app/src, desktop renderer, den-web, MCP App views)."

Include natural trigger synonyms users would actually say — "design review", "design spec", "check UI against DESIGN.md", "UI consistency" — so the skill matches real phrasing.

Name one or two concrete actions (e.g. "cite numbered rule ids from DESIGN.md and report medium/low findings") to lift specificity from one implied action to a fuller capability list.

DimensionReasoningScore

Specificity

"Warn when user-facing UI changes drift from DESIGN.md" names the domain and one concrete action (warn on drift), but "Advisory only; findings never gate Warden clearance" is a constraint rather than a capability, so coverage of what the skill actually does (cite rule ids, report medium/low findings) is not comprehensive. This matches anchor 3 rather than 4, which requires several specific actions listed.

3 / 5

Completeness

The "what" is clear (warn when UI changes drift from DESIGN.md, advisory only), but there is no "Use when..." clause or equivalent explicit trigger guidance, capping completeness at 3 per the rubric guideline. It is not 2 because the "what" is concrete and the review context is weakly implied by "UI changes".

3 / 5

Trigger Term Quality

Relevant keywords like "user-facing UI", "DESIGN.md", and "drift" are present, but common natural variations users would say ("design review", "design spec", "check UI against the design system", "UI consistency") are missing. This sits at anchor 3 — some relevant keywords but missing common variations — and not 4, which requires good keyword coverage.

3 / 5

Distinctiveness Conflict Risk

References to "DESIGN.md" and "Warden clearance" pin the skill to a specific repo workflow, making it mostly distinct with only minor overlap risk against a generic design-review or code-review skill. Not 5, because "warn when user-facing UI changes drift" could still collide with a general design-review skill in the same ecosystem.

4 / 5

Total

13

/

20

Passed

Validation

100%

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

Validation — 16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository

different-ai/openwork

Path

.warden/skills/design-spec-review/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.