Content
65%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 lean, information-dense best-practice catalog that assumes Laravel competence and names specific APIs, but it provides no executable code examples and its 'How to Apply' workflow lacks validation checkpoints. Most critically, the 19 referenced rules/*.md files do not exist in the bundle, breaking the progressive-disclosure path the skill relies on.
Suggestions
Ship the referenced rule files (rules/db-performance.md, rules/security.md, etc.) or remove the '→ rules/*.md' pointers and the 'use a sub-agent to read rule files' instruction so the skill is self-contained.
Add at least one executable code snippet per major section, or keep the deferral but make it explicit that the body is a pattern checklist and the rules files hold syntax/examples.
Add a validation/verification checkpoint to the migration and queue workflows (e.g., test the down() rollback, verify retry_after > timeout) since these involve destructive or batch operations.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Quotes 'Eager load with `with()` to prevent N+1 queries' and 'retry_after must exceed job timeout; use exponential backoff [1, 5, 10]' show lean, dense rules that assume Laravel competence with no concept-padding. Matches the score-3 anchor; not a 2 because there is no unnecessary explanation or fluff to tighten — every bullet is a concrete best practice. | 3 / 3 |
Actionability | Quotes name concrete APIs ('`with()`', '`Cache::remember()`', '`ShouldBeUnique`', '`php artisan make:migration`', '`constrained()`'), but there are no complete executable code blocks and the body defers syntax ('For exact API syntax, verify with `search-docs`'). Matches score 2 (concrete guidance but incomplete, missing key details); not 3 because nothing is copy-paste ready, not 1 because real specific APIs are named rather than vague direction. | 2 / 3 |
Workflow Clarity | Quotes '1. Identify the file type and select relevant sections', '2. Check sibling files for existing patterns', '3. Verify API syntax with `search-docs`' give a present sequence, but there are no validation checkpoints despite destructive/batch operations in scope (migrations: 'Never modify migrations that have run in production'). Matches score 2 (steps listed but validation gaps, checkpoints implicit); not 3 because no feedback loops, not 1 because a clear 3-step sequence exists. | 2 / 3 |
Progressive Disclosure | Quotes '### 1. Database Performance → `rules/db-performance.md`' (repeated across 19 sections) and 'Always use a sub-agent to read rule files' show well-signaled one-level-deep references, but the referenced `rules/*.md` files are absent from the bundle (no rules/, references/, scripts/, or assets/ directories exist), so navigation leads nowhere and the overview carries all real content. Matches score 2 (some structure, references present but content that should be separate is effectively inline); not 3 because the detail files are missing and disclosure is broken, not 1 because it is not a monolithic wall and references are one level deep and clearly labeled. | 2 / 3 |
Total | 9 / 12 Passed |