CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

flows-design-review

Semi-automated design quality review for Flows apps. Runs concrete repo probes (grep, lint, build) to propose a draft 1–5 score for each of the official 10 quality-guidelines questions from docs.cognite.com/cdf/flows/guides/quality-guidelines, then asks the user to confirm or override each score. Still requires the user to walk their tasks end-to-end in the running app (Step 2) since navigation and clickability feel cannot be measured statically. Writes reviews/design-review/feedback-round-<N>/design-review-report.md with an overall average and prioritized fix lists. Use when the user asks to run a Flows design review, run the design quality assessment, or run flows-design-review. Must be run AFTER flows-code-review reaches 0 Must Fix and BEFORE flows-external-app-submit.

69

Quality

87%

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

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

Highly actionable and workflow-sound: concrete probes, explicit score-translation heuristics, validation checkpoints, and a machine-readable report format. The weaknesses are repeated hedging about human authority, minor explanations of concepts Claude already knows, and a monolithic single-file layout that inlines ~180 lines of rubric anchors that belong in a reference file.

Suggestions

Move the per-question 1–5 anchor rubrics (the '5 Excellent' through '1 Poor' lists) into references/quality-guidelines-rubric.md and link to it once — keeping SKILL.md as the probe/decision overview and cutting roughly half the file.

State the 'probes propose, the user confirms' authority rule once in Operating rules and delete the three or four later restatements ('starting points, not verdicts', 'the user always has the final say', 'user-assigned scores remain authoritative').

Trim asides that re-explain what Claude already knows, e.g. 'placeholder-as-label is an anti-pattern', '(often a smell)', and 'non-semantic, often missing keyboard support' — the probe line alone is sufficient.

DimensionReasoningScore

Conciseness

The bulk of the body is lean operational content Claude does not already know (probe commands, rubric anchors, report template), but the same point about human authority is restated four or more times — 'The user's job is to confirm or override the proposed score', 'They are starting points, not verdicts', 'The user always has the final say', 'user-assigned scores remain authoritative' — and phrases like 'placeholder-as-label is an anti-pattern' and 'horizontal scroll containers (often a smell)' explain concepts Claude already knows. Mostly efficient but includes unnecessary explanation that could be tightened; not 4 because the redundancy goes beyond minor instances, and not 2 because no section is heavily padded.

3 / 5

Actionability

Guidance is fully executable: exact grep/eslint/build commands per question, concrete score-translation thresholds (e.g. '0 hard-coded colors + 0 aura/no-overriding-styles warnings → 5'), a copy-paste-ready report template, and structured AskQuestion options. Matches the 5 anchor; not 4 because there are no meaningful gaps in executability.

5 / 5

Workflow Clarity

Steps 0 through 6 are clearly sequenced with explicit validation checkpoints: pre-scan before prompting, 'Do NOT proceed to scoring until the user confirms they walked every task', a refusal path (stub report and exit), feedback-round incrementing, and a re-run loop when the average is below the 3.8 gate. The destructive/batch cap does not apply; this matches the 5 anchor with feedback loops and checkpoints throughout.

5 / 5

Progressive Disclosure

No bundle files exist (references/, scripts/, assets/ are absent) and the entire skill is one ~355-line file, with roughly half of it being the official 1–5 anchor rubric for the 10 questions — content that clearly belongs in a one-level-deep reference file the agent loads at scoring time. Internal structure (headers, tables) is good, but content that should be separate is inlined with no references at all; not 4 because the rubric bulk is not 'appropriately placed', and not 2 because the inlined content is well-organized and clearly signaled within the file.

3 / 5

Total

16

/

20

Passed

Description

92%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: concrete capabilities, explicit trigger clause, and explicit ordering that disambiguates it from sibling skills in the certification pipeline. The only weakness is that trigger terms are essentially three spellings of the skill name rather than natural user phrasings.

DimensionReasoningScore

Specificity

The description lists multiple concrete actions: 'Runs concrete repo probes (grep, lint, build) to propose a draft 1–5 score for each of the official 10 quality-guidelines questions', 'asks the user to confirm or override each score', and 'Writes reviews/design-review/feedback-round-<N>/design-review-report.md with an overall average and prioritized fix lists'. Coverage is comprehensive, spanning what is automated, what stays manual, and the output artifact.

5 / 5

Completeness

Both 'what' (semi-automated review, probes, draft scores, user confirmation, report output) and 'when' (explicit 'Use when...' clause) are clearly answered, plus ordering constraints ('Must be run AFTER flows-code-review reaches 0 Must Fix and BEFORE flows-external-app-submit'). Matches the 5 anchor with concrete trigger phrases; not 4 because the 'when' clause is fully explicit rather than merely present.

5 / 5

Trigger Term Quality

Triggers are explicit — 'Use when the user asks to run a Flows design review, run the design quality assessment, or run flows-design-review' — but all three are variants of 'run <skill name>', missing natural phrasings users would actually say like 'check my app's design', 'review the UI', or 'score the design'. Good coverage with a few natural terms missing, matching the 4 anchor; not 5 because no synonyms or loose phrasings beyond the literal name are present.

4 / 5

Distinctiveness Conflict Risk

The niche is clear ('Flows apps', 'design quality review') and explicitly disambiguated from adjacent skills via the pipeline ordering after flows-code-review and before flows-external-app-submit, including the '0 Must Fix' precondition. Minimal conflict risk; not 4 because there is no meaningful overlap with sibling skills left unaddressed.

5 / 5

Total

19

/

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

allowed_tools_field

'allowed-tools' contains unusual tool name(s)

Warning

Total

15

/

16

Passed

Repository

cognitedata/builder-skills

Path

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