Content
78%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, actionable skill body: lean tables, executable commands, documented fallbacks, and a genuine fail-closed validation note. The main gaps are the missing requirements.txt in the bundle, duplicated reference/command listings, and the absence of an explicit error-recovery step in the workflow.
Suggestions
Add requirements.txt to the bundle (or correct the install path) — the Workflow's 'pip install -r skills/macro-regime-detector/requirements.txt' references a file that is not present alongside SKILL.md, references/, and scripts/.
Consolidate the reference-file listings: the three references/*.md files appear in both the Workflow steps and the Resources section; keep the annotated listing in Resources and reference it from the Workflow to remove redundancy.
Extend the fail-closed note with an explicit recovery step, e.g., 'If no report is written or the exit is non-zero: verify FMP_API_KEY / network access, then re-run in yfinance-only mode' — turning a what-not-to-conclude note into a feedback loop.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is efficient and skill-specific — tables for components, regimes, and sibling-skill comparisons, with no explanations of concepts Claude already knows. Minor trimmable redundancy remains: the three reference files are listed in both the Workflow and a separate Resources section, and the Script Arguments section partially repeats the run command from the Workflow, fitting 'minor instances of over-explanation that could be trimmed' rather than the lean 5 anchor. | 4 / 5 |
Actionability | Concrete, copy-paste-ready commands are provided ('python3 -m pip install -r skills/macro-regime-detector/requirements.txt', 'uv run python3 skills/macro-regime-detector/scripts/macro_regime_detector.py --output-dir reports/') with documented options. Minor gaps: requirements.txt is referenced but absent from the bundle, and the Script Arguments block shows 'python3 macro_regime_detector.py' without the full path context used in the Workflow — 'concrete code or commands with minor gaps'. | 4 / 5 |
Workflow Clarity | The four-step sequence is clear with prerequisites, data-fetch fallback behavior (FMP → yfinance, SHY/TLT fallback), and an explicit fail-closed checkpoint ('The detector fails closed and writes no report when none of its six components has usable data. Do not treat a missing report or non-zero exit as a valid low-transition regime.'). It falls short of the 5 anchor because there is no explicit error-recovery step telling Claude what to do (e.g., check FMP_API_KEY, retry without key) when the script fails — the guidance only says what not to conclude. | 4 / 5 |
Progressive Disclosure | The SKILL.md is a clean overview with well-signaled, one-level-deep references, and every referenced file (references/regime_detection_methodology.md, references/indicator_interpretation_guide.md, references/historical_regimes.md) exists in the bundle. Detailed methodology and interpretation content is appropriately split into those reference files rather than inlined, and the Resources section annotates what each contains — matching the clear-overview 5 anchor. | 5 / 5 |
Total | 17 / 20 Passed |