Content
75%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 highly actionable, well-sequenced Behave reference: complete executable code for features, steps, hooks, tags, and reporting, plus a useful anti-patterns table and honest limitations. Its main weaknesses are the Overview's padding with BDD philosophy Claude already knows and the absence of an explicit run-fail-fix feedback loop in the workflow.
Suggestions
Trim the Overview: drop the BDD collaboration quotes ('Behavior-driven development encourages collaboration...') and the Cucumber semi-official-tier aside, keeping only the doc link and a one-line orientation.
Add a brief feedback loop after Step 8 (Run): e.g. 'On failure, behave prints the failing step and line; fix the step or feature and re-run' to give the workflow an explicit validate-and-retry checkpoint.
Consider moving the pytest-bdd comparison (Step 9) and the extended anti-patterns table into a references/ file to reduce SKILL.md length and strengthen progressive disclosure.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient — the body is dominated by dense code and tables — but the Overview explains BDD concepts Claude already knows ("Behavior-driven development encourages collaboration between developers, QA and non-technical or business participants") and includes trivia like the Cucumber "semi-official tier" note. Not a 4: the philosophy quotes and repeated "Per [behave-docs]" attributions are unnecessary padding that could be trimmed; not a 2 because the bulk (Steps 1–9, anti-patterns, limitations) is lean and earned. | 3 / 5 |
Actionability | Fully executable throughout: "pip install behave", a complete Gherkin feature with Scenario Outline and Examples, copy-paste step implementations using real Behave decorators ("@given('the cart contains {qty:d} of \"{sku}\" at ${price:f}')"), full environment.py hooks, and exact CLI commands ("behave --tags=critical", "behave --junit --junit-directory reports/junit/"). The examples cover the common cases end-to-end. | 5 / 5 |
Workflow Clarity | Clear 9-step sequence from Install through Run, with the assert-based Then steps providing inherent validation. Not a 5: the Run step shows commands but no feedback loop for interpreting failures and iterating (the closest is an anti-pattern row: "Quick smoke check in before_all; fail fast"), which leaves minor checkpoint gaps. | 4 / 5 |
Progressive Disclosure | No bundle files exist (references/, scripts/, assets/ are absent), so this is a single-file skill with well-organized sections and a References section of clearly signaled one-level-deep links ([bd], sibling skills). Good structure with minor gaps: at ~245 lines, content like the pytest-bdd comparison table and extended anti-patterns could arguably live in a reference file; not a 3 because nothing is buried and navigation is easy, not a 5 because the whole skill is inlined rather than split (the 5-without-references exception applies only to skills under 50 lines). | 4 / 5 |
Total | 16 / 20 Passed |