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 highly actionable, well-structured skill body with an explicitly gated 11-step workflow, exact commands and MCP call sequences, and clean one-level reference separation verified against the actual bundle. The main issues are minor: a redundant full example report, an internal ordering contradiction between the workflow table (tflint before validate) and the anti-patterns section (validate before tflint), and unreferenced asset files.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and command-first throughout (workflow table, scripts table, terse command blocks) and does not explain concepts Claude already knows, so it sits above the midpoint. Not a 5 because of minor padding that could be trimmed: the ~45-line "Example Cross-Referenced Report" largely duplicates the template above it, the "External Documentation" link table adds little, and the opening paragraph restates the frontmatter description. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready guidance throughout: exact script invocations ("bash scripts/extract_tf_info_wrapper.sh <path>", "terraform init -backend=false && terraform validate"), a concrete provider-lookup workflow with exact MCP tool names and a ready-made WebSearch query string, a fill-in report template, and a scripted ask/install/rerun procedure for missing tools. This matches the top anchor; score 4 would require missing key details, and none are apparent. | 5 / 5 |
Workflow Clarity | The 11-step table has explicit Required/Recommended gates, an explicit ordering constraint (read reference files before security scans), and error-recovery paths (common_errors.md on errors, install-and-rerun loop for missing tools) — most checkpoints are present. Not a 5 because there is no explicit validate-fail→fix→re-validate loop, and the table contradicts the Anti-Patterns section: it sequences tflint (step 6) before terraform init/validate (steps 7–8), while the anti-pattern mandates validate before tflint. | 4 / 5 |
Progressive Disclosure | Good structure with well-signaled one-level-deep references: four reference files are each mapped to a read-when point in a table, scripts are listed with purpose and command, and all referenced paths exist and match their described contents. Not a 5 because the actual bundle contains files never referenced or navigable from SKILL.md (assets/sample.tf and assets/.terraform.lock.hcl), a minor organization gap against the 'easy navigation' top anchor. | 4 / 5 |
Total | 17 / 20 Passed |