Content
77%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This is a well-structured, highly actionable skill for a complex infrastructure cleanup workflow. Its greatest strengths are the clear 7-step workflow with explicit confirmation gates, comprehensive error handling, and concrete executable commands. The main weakness is length — some content (IaC tool tables, detailed error scenarios) could be extracted to reference files for better progressive disclosure and token efficiency.
Suggestions
Extract the IaC destroy command table (12 tools) into a separate reference file like `destroy-commands.md` and reference it from the main skill to reduce inline bulk.
Consider moving the detailed error handling section to a reference file, keeping only the most critical error cases (config missing, destroy failure) inline.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The skill is fairly long (~250 lines) but most content is necessary for a complex multi-step, multi-source workflow. However, there's some verbosity: the IaC tool destroy command table includes many tools that could be referenced externally, the /loop section repeats the 'never auto-destroy' point multiple times, and some explanatory text could be tightened (e.g., 'The TTL registry is a fallback for environments without issue tracking or for resources created outside the normal deployment flow'). | 2 / 3 |
Actionability | The skill provides concrete, executable commands throughout: specific `gh issue list` commands with exact flags, `tofu destroy -target=` patterns with var-file paths, `gh issue close` commands, and clear table formats for the TTL registry. Commands are copy-paste ready and tool-specific variations are enumerated. | 3 / 3 |
Workflow Clarity | The 7-step workflow is clearly sequenced with explicit validation checkpoints: confirmation before any destruction (Step 3), error handling for partial failures, state lock detection, and a feedback loop for failed destroys (retry or skip). The 'NEVER auto-destroy' constraint is prominently stated. The error handling section is comprehensive with specific recovery actions for each failure mode. | 3 / 3 |
Progressive Disclosure | The skill references external files (experience-derivation.md, rollback-patterns.md, providers.md, environments.md) appropriately, but the main SKILL.md itself is quite long with inline content that could be split out — particularly the IaC destroy command table (12 tools) and the detailed error handling section. No bundle files are provided to verify referenced paths, and the large inline tables make the skill harder to scan. | 2 / 3 |
Total | 10 / 12 Passed |