Content
81%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 well-structured, lean instruction skill: clear sequenced workflow with explicit fail-fast validation and error-recovery paths, and a properly signaled reference layer with real bundle files. Main weaknesses are mild redundancy of the worktree/preserve-field rules across sections and one unresolvable deferred-context path.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is a lean, directive bullet list with no padding or explanation of known concepts — every bullet is an instruction. Not 5 because the worktree/suggested-mode rule and preserve-fields rule each repeat across Workflow, Constraints, Execution Boundaries, and Gotchas, which could be consolidated. | 4 / 5 |
Actionability | Concrete, executable directives for an instruction-only skill: "Resolve existing automations first by id, name, prompt, and local TOML evidence", the heartbeat-vs-cron routing rule, and "Use suggested create/update when proposing worktree automations with a non-null local environment setup config". Not 5 because no tool request shapes are shown inline; the concrete examples are delegated to references/tool-examples.md rather than covering common cases in the main file. | 4 / 5 |
Workflow Clarity | The Workflow section sequences goal confirmation, existing-automation resolution, field mapping, preflight/rollback definition, then validation, and the Validation section supplies an explicit feedback loop: "Run the smallest command or test that exercises the changed behavior" and "Fail fast: stop at first failed gate; do not proceed until it is fixed and rerun". The Failure Mode section adds error-recovery paths, matching the top anchor. | 5 / 5 |
Progressive Disclosure | The Progressive Disclosure section clearly signals four one-level-deep references (contract.yaml, tool-examples.md, evals.yaml, task-profile.json), each labeled with its purpose, and all four exist in the bundle. Not 5 because the fifth pointer, "Infrastructure/references/deferred-skill-context/agent-ops-codex-automation-architect/", does not resolve within the skill bundle, leaving a dangling external path. | 4 / 5 |
Total | 17 / 20 Passed |