Content
82%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.
A tight, example-driven reference whose executable code, wrong/right contrasts, and non-obvious construction-phase gotchas make it highly actionable. Its weaknesses are minor: some baseline Ginkgo-docs restatement, a duplicated warning, small typos, and everything inlined in one file rather than splitting deeper detail out.
Suggestions
Fix the typos on line 10 ('Perfer `DescrtibeTable`' → 'Prefer `DescribeTable`') and de-duplicate the BeforeSuite warning stated at both the start and end of the fixture-loading section.
Trim restatements of baseline Ginkgo behavior (basic DescribeTable mechanics, the EntryDescription format table) to just the non-obvious construction-time semantics.
Consider moving secondary topics (DescribeTableSubtree, shared-behavior closures) into a references/ file so SKILL.md stays a lean overview.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and Ginkgo-specific (the construction-time evaluation gotcha is genuinely non-obvious), but it restates some baseline Ginkgo documentation (DescribeTable mechanics, the EntryDescription table), repeats the BeforeSuite warning twice ('a loop reading a BeforeSuite-populated slice generates zero specs' and 'Don't load it in BeforeSuite — it's too late, and the loop generates no specs'), and contains typos ('Perfer `DescrtibeTable`'). Mostly efficient with minor trims available — the 4 anchor, not 5; not 3 because nearly all content is skill-specific rather than generic padding. | 4 / 5 |
Actionability | Every section carries complete, executable Go examples covering the common cases: the WRONG/RIGHT contrast for the nil-shelf gotcha, the four Entry description mechanisms in one runnable table, DescribeTableSubtree, and the TestXxx fixture-loading pattern with a pre-RUNSpecs assertion ('g.Expect(fixtureBooks).NotTo(BeEmpty())'). Copy-paste-ready throughout; matches the 5 anchor. | 5 / 5 |
Workflow Clarity | Although not a sequential process skill, each section states when to apply it ('When you want a whole subtree (multiple Its, their own setup) per entry, use DescribeTableSubtree'; 'Struct-per-row for many params'), the key failure mode is explicitly flagged ('THE gotcha', 'nil pointer!'), and error recovery is shown via the WRONG/RIGHT pair. Selection guidance is clear but there is no explicit decision ordering across patterns — the 4 anchor; not 3 because the guidance that exists is coherent and error paths are concretely illustrated. | 4 / 5 |
Progressive Disclosure | No bundle files exist (references/, scripts/, assets/ are all absent), so all ~150 lines are inline under well-organized section headers with external doc links (onsi.github.io/#table-specs etc.) and pointers to sibling skills (ginkgo:overview, ginkgo:decorators, ginkgo:filtering). Structure and signaling are good, but the body exceeds the under-50-lines condition for a 5 without external references, and all detail (DescribeTableSubtree, shared behaviors) is inlined rather than split out — the 4 anchor; not 3 because organization and navigation are genuinely solid. | 4 / 5 |
Total | 17 / 20 Passed |