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 high-quality, action-dense skill body: nearly everything is executable, the install-to-CI-gate sequence is clear, and structure is good. Weakest points are minor redundancy (repeated doc citations, duplicated result examples) and the absence of error-recovery guidance around failed or noisy runs.
Suggestions
Trim redundancy: cite ghz.sh/docs/usage once, and show either the summary-format output or the JSON schema highlights for the same run rather than both.
Add a short error-recovery loop for runs: what to check and retry when ghz reports connection errors or a large non-OK status distribution (e.g. verify TLS/insecure match, lower -c, re-run).
Move the CI workflow YAML and compare-ghz.py into a bundled scripts/ or references/ file and reference them from SKILL.md to keep the main file a lean overview.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is efficient and stays on ghz-specific knowledge Claude cannot be assumed to have (flags, JSON report schema, CI gating), but has minor trim opportunities: 'Per ghz.sh/docs/usage' is repeated three times, the summary block and the JSON schema highlights restate the same run's numbers, and the References section re-lists links already cited inline. This fits 'efficient; minor instances of over-explanation that could be trimmed' rather than the every-token-earns-its-place anchor. | 4 / 5 |
Actionability | Guidance is fully executable and copy-paste ready: brew/go install commands, a complete config.json, three concrete ghz invocations, a full GitHub Actions workflow, and a runnable compare-ghz.py regression gate covering the common cases (basic load, RPS pinning, CI gating). The only shorthand is the illustrative 'ghz -z 5m -c 100 ...' fragment, whose flags are fully documented in the adjacent table, so it does not constitute a gap in executable guidance. | 5 / 5 |
Workflow Clarity | The skill is clearly sequenced (Install → verify ghz --version → Configure config.json → Run → Parse results → CI gate) with most checkpoints present, including an explicit regression-gate failure path (sys.exit(1) when p99 exceeds +10%). It falls short of the top anchor because there is no error-recovery feedback loop for run-time failures (e.g. what to do on connection errors or a skewed non-OK status distribution mid-run). | 4 / 5 |
Progressive Disclosure | No bundle files exist, and the single SKILL.md is well-sectioned with all content inline; external references (ghz docs URLs, sibling skills grpc-streaming-test-author, k6-load-testing, jmeter-load-testing) are clearly signaled and one level deep in a References section. This matches 'good structure; most content appropriately placed; references mostly clear; minor organization gaps' — the CI workflow YAML and compare-ghz.py could arguably live in bundled scripts/ files, and at ~300 lines the under-50-line simple-skill exception does not apply, so it is not a 5. | 4 / 5 |
Total | 17 / 20 Passed |