Content
47%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This skill defines a comprehensive infrastructure change execution orchestration workflow with excellent workflow clarity, including proper validation gates, user confirmation checkpoints, error recovery paths, and resume logic. However, it is excessively verbose — the inline agent context templates, repeated dispatch convention references, and detailed gate evaluation rules bloat the file significantly when much of this content is already referenced in external files. The actionability is moderate since it describes processes well but defers actual executable commands to reference files.
Suggestions
Move the inline agent context templates (Steps 2.2-2.7) to a reference file like `agent-invocation-templates.md` and reference it once, reducing the SKILL.md by ~40%.
Consolidate the repeated 'passing the model from `.arness/agent-models/infra.md`' boilerplate into a single note at the top of the dispatch loop section rather than repeating it four times.
Add concrete executable examples for the rollback checkpoint step (Step 2.1) — currently it says 'backup IaC state files (state pull/export)' without showing actual commands for any IaC tool.
Since dispatch-loop.md, gate-policies.md, and deploy-procedures.md are referenced but not provided in the bundle, ensure these files exist and contain the detailed logic currently duplicated inline.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The skill is extremely verbose at ~300+ lines. It includes extensive inline context templates that could be referenced from external files, repeats the agent model dispatch convention four times with identical boilerplate, and provides detailed structured text blocks that inflate token consumption significantly. Much of this content (e.g., the full agent invocation context templates) could be in reference files. | 1 / 3 |
Actionability | The skill provides a clear structured workflow with specific steps and agent invocations, but lacks executable code/commands. The deployment step defers to a reference file for actual commands, the IaC generation is described abstractly, and the context templates are illustrative rather than executable. Key details like how to actually backup state files or pull resource manifests are missing. | 2 / 3 |
Workflow Clarity | The 7-step dispatch loop is clearly sequenced with explicit validation checkpoints at each gate (security, cost, verification, review). There are well-defined feedback loops for error recovery (fix-and-retry, rollback, abort options), environment promotion gates requiring explicit user approval, and resume-from-interruption logic. The error handling section is comprehensive with specific recovery paths for each failure mode. | 3 / 3 |
Progressive Disclosure | The skill references several external files (dispatch-loop.md, gate-policies.md, deploy-procedures.md, report templates) which is good progressive disclosure design. However, no bundle files were provided to verify these references exist, and the main SKILL.md itself is a monolithic wall of text that inlines extensive context templates and agent invocation details that should be in the referenced files. The balance between what's inline vs. referenced is poor. | 2 / 3 |
Total | 8 / 12 Passed |