Content
86%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
An exemplary lean, repo-specific guideline skill: dense with non-obvious project knowledge, concrete commands and CSS mechanics, and a clear inspect-then-build-then-validate flow. The only gaps are minor: the primary validation instruction lacks example commands for selecting test/E2E targets, and there is no stated recovery step or recording location for validation outcomes.
Suggestions
Add a one-line example of how to run the focused component/unit target and the closest Playwright workflow (e.g. the exact pnpm test filter or Playwright command pattern), making the main validation step copy-paste executable.
State what to do when validation fails (fix and re-run) and where uncovered visual states should be recorded, closing the feedback loop in the Validation section.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Every line carries project-specific, non-obvious knowledge — e.g. "Dense rows use minimum dimensions rather than a fixed width: the text column needs `min-width: 0` plus ellipsis, while logos and trailing actions use `flex-shrink: 0`" and "Stateful collection loading, persistence, and cross-provider orchestration belong in `@iptvnator/portal/shared/data-access`, not UI or util". Nothing explains concepts Claude already knows, and nothing can be trimmed without losing actionable information. | 5 / 5 |
Actionability | Concrete and executable in most places: exact commands ("`pnpm run styles:icon-ligatures:validate` checks them", "also runs `pnpm run perf:compositing`"), specific CSS mechanics, and exact library/package paths to inspect. Not 5 because "Run the focused component/unit target and the closest Playwright workflow" gives no example of how to select or invoke those targets, and "Record any uncovered visual state" specifies no destination or format; not 3 because the guidance is largely copy-paste-ready and specific. | 4 / 5 |
Workflow Clarity | A clear implicit sequence — Inspect First → Design Contract → Validation — with explicit and even conditional checkpoints ("Electron CDP is a fallback for Electron-only gaps", "A change that adds `backdrop-filter`, `filter`, animations or a rounded scroller also runs `pnpm run perf:compositing` ... and quotes the numbers"). Not 5 because there is no error-recovery loop for failed validation and "record any uncovered visual state" lacks a checkpoint destination; not 3 because the checkpoints are explicit and the conditional validation rules are precise. | 4 / 5 |
Progressive Disclosure | A single-file skill under 50 lines with no bundle files, organized into three well-labeled sections. Its references are one level deep and clearly signaled — "Policy: `docs/architecture/iptvnator-ui-guidelines.md`", specific source paths, and "see the theme/style skill's Rendering Cost section" — so navigation is easy and nothing needs splitting. | 5 / 5 |
Total | 18 / 20 Passed |