CtrlK
BlogDocsLog inGet started
Tessl Logo

tdg-personal/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.

57

Quality

57%

Does it follow best practices?

Impact

Pending

No eval scenarios have been run

SecuritybySnyk

Passed

No known issues

Overview
Quality
Evals
Security
Files

Quality

Discovery

54%

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 has strong trigger term coverage with excellent Flutter/Dart ecosystem keywords and is highly distinctive in its niche. However, it lacks an explicit 'Use when...' clause, which is a significant gap for skill selection, and the capabilities are described as topic areas rather than concrete actions Claude would perform.

Suggestions

Add an explicit 'Use when...' clause, e.g., 'Use when the user asks for a Flutter or Dart code review, requests feedback on widget structure, or wants state management advice.'

Replace the category listing with concrete actions, e.g., 'Reviews Flutter/Dart code for widget decomposition issues, improper state management patterns, performance anti-patterns, and accessibility gaps' instead of just listing topic areas.

DimensionReasoningScore

Specificity

Names the domain (Flutter/Dart code review) and lists several areas covered (widget best practices, state management patterns, performance, accessibility, security, clean architecture), but these are categories rather than concrete actions. It doesn't specify what actions are performed like 'flags missing const constructors' or 'checks for proper widget decomposition'.

2 / 3

Completeness

Describes what the skill covers (code review checklist for Flutter/Dart) but completely lacks a 'Use when...' clause or any explicit trigger guidance for when Claude should select this skill. Per the rubric, a missing 'Use when...' clause should cap completeness at 2, and since the 'what' is also somewhat vague (checklist covering topics rather than specific actions), this scores a 1.

1 / 3

Trigger Term Quality

Excellent coverage of natural keywords a user would say: 'Flutter', 'Dart', 'code review', 'BLoC', 'Riverpod', 'Provider', 'GetX', 'MobX', 'Signals', 'widget', 'state management', 'performance', 'accessibility', 'security', 'clean architecture'. These are all terms developers naturally use.

3 / 3

Distinctiveness Conflict Risk

The combination of Flutter/Dart focus, code review context, and the specific enumeration of state management libraries (BLoC, Riverpod, Provider, GetX, MobX, Signals) creates a very distinct niche that is unlikely to conflict with other skills.

3 / 3

Total

9

/

12

Passed

Implementation

42%

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

This is a thorough and well-organized Flutter/Dart code review checklist that covers an impressive breadth of topics with specific, knowledgeable guidance. Its main weaknesses are its monolithic length (no progressive disclosure to sub-files), limited executable code examples (only one code block for 15 sections), and lack of workflow guidance on how to actually conduct a review using the checklist. The content quality per-item is high, but the delivery format doesn't respect token efficiency or leverage file decomposition.

Suggestions

Split the monolithic checklist into separate files per major section (e.g., `state-management-review.md`, `performance-review.md`, `security-review.md`) and make SKILL.md a concise overview with links to each.

Add executable code examples to at least the most critical sections (Dart pitfalls, widget best practices, error handling) — the single sealed-class example is good but insufficient for 15 sections.

Add a brief workflow section at the top explaining how to conduct a review: e.g., '1. Run `flutter analyze` first, 2. Check state management patterns, 3. Review widget structure, 4. Verify tests exist for all state transitions'.

Remove the Sources section — Claude already knows these official Flutter/Dart documentation URLs and they consume tokens without adding actionable value.

DimensionReasoningScore

Conciseness

The skill is comprehensive but quite long (~500+ lines). While most items earn their place as checklist entries, the sheer volume is heavy for a SKILL.md. Some sections (e.g., explaining what sealed types are, the Sources section with basic Flutter doc links) add tokens Claude doesn't need. However, the checklist format is inherently efficient per-item.

2 / 3

Actionability

The checklist items are specific and concrete (e.g., 'use `MediaQuery.sizeOf(context)` instead of `MediaQuery.of(context)`'), and the state management code example is executable. However, the vast majority of the skill is checklist items without executable code examples — only one code block is provided across 15 sections. Many items describe what to check but don't show how to fix issues.

2 / 3

Workflow Clarity

This is a checklist-style skill rather than a multi-step workflow, so sequential ordering is less critical. The sections are logically organized and numbered. However, there's no guidance on how to actually conduct a code review using this checklist — no prioritization, no sequence for applying checks, and no validation steps (e.g., 'run flutter analyze first, then review manually').

2 / 3

Progressive Disclosure

The entire skill is a monolithic wall of checklist content with no references to external files. At 500+ lines covering 15 major sections, this would benefit enormously from splitting into separate files (e.g., state-management-review.md, performance-review.md) with the SKILL.md serving as an overview with links. The state management quick reference table alone could be a separate file.

1 / 3

Total

7

/

12

Passed

Validation

90%

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

Validation10 / 11 Passed

Validation for skill structure

CriteriaDescriptionResult

frontmatter_unknown_keys

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

Warning

Total

10

/

11

Passed

Reviewed

Table of Contents