CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

flutter-dart-code-review

Library-agnostic Flutter/Dart code review checklist covering widget best practices, state management patterns (BLoC, Riverpod, Provider, GetX, MobX, Signals), Dart idioms, performance, accessibility, security, and clean architecture.

Quality

76%

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 ./skills/flutter-dart-code-review/SKILL.md

The canonical home for this skill is flutter-dart-code-review in affaan-m/ECC

SKILL.md
Quality
Evals
Security

Quality

Content

72%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 highly actionable, information-dense checklist with concrete lint rules, APIs, thresholds, and a useful cross-library quick reference table, written in lean checklist prose that assumes reviewer competence. Its weaknesses are structural: no sequenced review workflow with validation checkpoints, everything inlined in one long file with no progressive disclosure, and some duplicated items across sections.

Suggestions

Add a short 'Review workflow' section sequencing the checks (e.g., 1. run flutter analyze / flutter test, 2. architecture & state management, 3. widgets & performance, 4. a11y/security/l10n) with a validation step confirming findings against analyzer output before reporting.

Split the deep-dive material (state management per-solution guidance, package evaluation criteria, the quick reference table) into references/ files (e.g., STATE_MGMT.md, PACKAGES.md) and keep SKILL.md as a lean overview with clearly signaled one-level-deep links.

Deduplicate repeated items — print()/logging (sections 1 and 2), strict analyzer settings (1 and 15), and deep-link sanitization (9 and 11) each appear twice — consolidating them into their primary section.

DimensionReasoningScore

Conciseness

The body is a dense, checklist-style reference that assumes Claude's competence — it never explains what Flutter is or how the libraries work — with terse one-line items like 'ValueKey used in lists/grids to preserve state across reorders'. Not a 5 because of noticeable redundancy: print() logging appears in sections 1 and 2, strict analyzer settings in sections 1 and 15, deep-link sanitization in sections 9 and 11, and rebuild/const guidance in sections 3 and 5, so some tokens could be trimmed.

4 / 5

Actionability

Nearly every item is directly checkable and cites concrete specifics: exact lint rules ('prefer_const_constructors', 'unawaited_futures', 'always_use_package_imports'), exact APIs ('MediaQuery.sizeOf(context)', 'ref.watch(p.select(...))', 'PlatformDispatcher.instance.onError'), concrete thresholds (48x48 targets, 4.5:1 contrast, 130+/160 pub points, 80% coverage, ~80-100 line build methods), plus an executable sealed-class code example and a library-mapping quick reference table. Per the scoring notes, absence of bulk code is not penalized in an instruction-only skill when guidance is this actionable — specific examples cover the common review cases.

5 / 5

Workflow Clarity

The content is organized into 15 well-labeled topical sections, but there is no sequenced process for conducting a review (e.g., 'run flutter analyze first, then check architecture, then widgets') and no validation checkpoints or feedback loops. It sits at the anchor 'sequence present but checkpoints missing or implicit' — the structure implies an order without stating one — and does not qualify for the simple-skill exception since it is a 430-line reference.

3 / 5

Progressive Disclosure

There are no bundle files (references/, scripts/, assets/ do not exist), so all content — including the long state management section, the package-review heuristics, and the quick reference table — is inlined in a single ~430-line SKILL.md. Section headers give it good internal structure, matching the anchor 'some structure but could be better organized; content that should be separate is inline', but it is too heavy for a single-file overview to reach level 4.

3 / 5

Total

15

/

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.

A strong, distinctive description with excellent natural trigger terms (framework, language, and all major state management library names) and concrete coverage areas. Its main weaknesses are the complete absence of an explicit 'Use when...' trigger clause — which caps completeness — and the lack of synonym/extension terms, and it enumerates topics rather than concrete actions.

Suggestions

Add an explicit trigger clause, e.g., 'Use when reviewing Flutter or Dart code, a pubspec.yaml, or when the user mentions BLoC, Riverpod, Provider, GetX, widget rebuilds, or Flutter performance.'

Include natural trigger variations and file artifacts users actually reference, such as '.dart', 'pubspec.yaml', 'Flutter app', or 'widget tree'.

Lead with the concrete action the skill performs (e.g., 'Reviews Flutter/Dart code against a checklist covering...') rather than the noun-phrase 'checklist covering', to sharpen the 'what'.

DimensionReasoningScore

Specificity

The description enumerates many concrete coverage areas — 'widget best practices, state management patterns (BLoC, Riverpod, Provider, GetX, MobX, Signals), Dart idioms, performance, accessibility, security, and clean architecture' — including six named libraries. It falls short of a 5 because it is a topical enumeration with a single action verb ('covering') rather than a list of multiple specific concrete actions like 'extract', 'fill', 'merge'.

4 / 5

Completeness

The 'what' is clear — a library-agnostic Flutter/Dart code review checklist with enumerated coverage areas — but there is no 'Use when...' clause or equivalent explicit trigger guidance, which per the judging guidelines caps completeness at 3. The 'when' is only weakly implied by the topic itself, fitting the anchor 'Has a clear what but when is missing or only weakly implied' rather than the level-4 anchor where an explicit 'when' is present.

3 / 5

Trigger Term Quality

Strong natural keyword coverage: 'Flutter/Dart code review', 'BLoC, Riverpod, Provider, GetX, MobX, Signals', 'widget', 'state management' are exactly the terms users say when requesting a Flutter review. Not a 5 because it lacks synonyms and file-extension triggers (e.g., '.dart', 'pubspec', 'Flutter app review') that the top anchor expects.

4 / 5

Distinctiveness Conflict Risk

The Flutter/Dart code-review niche with named state-management libraries is highly distinct; these triggers would almost never collide with another skill. It clearly matches the anchor 'Clear niche with distinct triggers; minimal conflict risk' and is not merely the level-4 'minor overlap risk with closely related skills'.

5 / 5

Total

16

/

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

frontmatter_unknown_keys

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

Warning

Total

15

/

16

Passed

Repository

valeriibeck-star/claude-skills

Path

skills/flutter-dart-code-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.