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: executable commands and HCL at every step, a version/support decision tree, validation checkpoints for a batch operation, and real one-level-deep bundle references. The main improvement opportunities are deduplicating the provider-support instructions, removing the empty "Supported List Resources" stub, and offloading some inline example detail to reference files.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dominated by lean, high-value code blocks and commands with almost no explanation of concepts Claude already knows (no "what is Terraform" padding), matching "efficient; minor instances that could be trimmed". It is not a 5 because the provider-support check is repeated nearly verbatim in "Check Provider Support First" and "Prerequisites" (identical list_resources.sh invocations shown twice), and the "Supported List Resources" section is a near-empty stub that spends tokens saying only "varies by version, check the script"; not a 3 because the redundancy is limited rather than pervasive. | 4 / 5 |
Actionability | Fully executable throughout: copy-paste-ready bash (terraform query -generate-config-out=imported.tf, the jq schema-extraction command, terraform state list), concrete HCL query examples (tag filters, multi-region for_each, parameterized variables), and an end-to-end Complete Example. The only placeholders appear in the explicitly-labeled syntax template and are immediately instantiated by real examples, so the common cases are covered. | 5 / 5 |
Workflow Clarity | A clear decision tree (check support → check Terraform version → pick Search vs. Manual workflow), a numbered five-step workflow with review/plan/apply checkpoints, post-apply state verification, and a troubleshooting table giving issue→solution feedback. This is a batch operation and validation is present (review generated blocks, terraform plan, "Verifying Imported State" section), so the batch cap does not apply; it is not a 5 because the numbered workflow does not explicitly embed the verify-state step as a checkpoint, and the decision tree bullets have malformed formatting ("** If supported**:") that slightly muddies the sequence. | 4 / 5 |
Progressive Disclosure | Good structure with well-signaled, one-level-deep references that resolve to real bundle files: [references/MANUAL-IMPORT.md](references/MANUAL-IMPORT.md) for the fallback workflow and ./scripts/list_resources.sh for support discovery; MANUAL-IMPORT.md itself contains no nested references. It is not a 5 because the ~385-line body keeps substantial detail inline (multiple query examples, cleanup before/after, import-by-identity) that could be split into references to keep SKILL.md a leaner overview. | 4 / 5 |
Total | 17 / 20 Passed |