Content
92%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, highly actionable body: exact commands for both platforms, a doctor→apply→verify sequence with rollback and verification, and an appropriate one-reference split. The only weakness is mild over-explanation of script internals in the apply section.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is efficient — a routing table, exact commands, and no explanation of concepts Claude already knows — but minor over-explanation could be trimmed: the "apply" internals ("content-addressed backup", "atomic same-directory replace") describe script behavior Claude does not need in order to act, and the opener "without pretending every model supports one million tokens" is rhetorical. It is not 3 because there is no genuinely unnecessary explanation, and not 5 because those few sentences do not strictly earn their place. | 4 / 5 |
Actionability | Copy-paste-ready "uv run --no-project python "$SKILL_DIR/scripts/codex_context_window.py" doctor/apply/verify" blocks for both macOS/Linux and Windows PowerShell, an intent-to-action routing table, an explicit path-resolution procedure, and an exact report checklist. It is not 4 because the guidance fully covers the common cases with executable commands rather than hints. | 5 / 5 |
Workflow Clarity | The sequence is explicit with validation checkpoints and feedback loops: "doctor" runs first and is read-only, "Run only after `doctor` succeeds", "a failure restores the exact prior bytes" on strict-config diagnostics, and "verify" exits non-zero on mismatch so drift is caught. This is a config-modifying operation, but validation is present, so the destructive-operation cap does not apply. It is not 4 because every checkpoint — pre-check, rollback, post-write read-back, and re-verification — is explicit. | 5 / 5 |
Progressive Disclosure | The body is a clear overview (routing, commands, reporting, scope refusals) with one well-signaled, one-level-deep reference — "Read [references/context_window_contract.md] … when diagnosing model caps, the 95% usable-window display, or compaction semantics" — and both referenced bundle paths (references/context_window_contract.md, scripts/codex_context_window.py) exist as real files. Deep contract details and source links are appropriately split into the reference rather than inlined. It is not 4 because the split is clean and the reference is clearly signaled with its when-to-read condition. | 5 / 5 |
Total | 19 / 20 Passed |