Content
67%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 CLI skill body with real external references, clear workflows, and strong error-handling guidance. Its two weaknesses are redundant command repetition and an implementation-checklist/footer that pad the token budget, and multiple references to a tests/ directory that does not exist in the bundle, which undermines both executability and navigation.
Suggestions
Remove the 'Implementation Checklist' and the 'Last updated / Version' footer, and consolidate the near-duplicate analyze/forest-plot invocations from Usage, Examples, and Testing into one canonical set of examples (pointing to references/cli-guide.md for further variants).
Fix or remove the broken tests/ references: either ship 'tests/data/sample_clinical_survival_data.csv', 'tests/run_smoke_test.R', and 'tests/run_smoke_test.sh' in the bundle, or rewrite the Testing and Basic Analysis sections to use only files that exist.
Add an explicit checkpoint between workflow steps 3 and 4 — verify that 'table/prognosis_uni_cox_results.xlsx' / 'table/prognosis_multi_cox_results.xlsx' were produced before running the forest-plot commands — to close the workflow's main validation gap.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient — the body is dominated by terse tables and commands with no concept explanations Claude already knows — but it carries avoidable weight: the same analyze/forest-plot invocations appear near-identically in Usage, Examples, and Testing; the 'Implementation Checklist' ('CLI parsing with optparse', 'set.seed() for reproducibility') documents implementation facts rather than instructing the agent; and the 'Last updated: 2026-04-16 | Version: 1.1.0' footer is time-sensitive filler. These could be trimmed or pushed to references/cli-guide.md without losing operational value. | 3 / 5 |
Actionability | The core guidance is fully executable — copy-paste 'Rscript scripts/main.R analyze ...' commands with a complete argument table, defaults, concrete input CSV format with sample rows, and documented output files. The gap: the smoke-test and basic-example commands point at 'tests/data/sample_clinical_survival_data.csv', 'tests/run_smoke_test.R', and 'tests/run_smoke_test.sh', none of which exist in the bundle, so those specific examples fail if run verbatim. | 4 / 5 |
Workflow Clarity | The Workflow section gives a clear four-step sequence with an explicit decision rule ('significant univariate features with p < 0.05', 'if fewer than 3 significant features... fall back to all requested features'), Step 1 validates inputs, input requirements state hard thresholds (at least 10 complete samples, 2 events), and the Error Handling table plus the 'IF error persists, READ: references/troubleshooting.md' escalation forms a recovery loop. Minor gap: no checkpoint telling Claude to confirm the analyze output table exists before invoking the plot commands. | 4 / 5 |
Progressive Disclosure | Good structure: a 'When to Read External Files' table maps each situation to references/algorithm.md, scripts/main.R, references/troubleshooting.md, and references/cli-guide.md — all real files, referenced one level deep with no nested references, and scripts/ is genuinely modular. Gaps: the inlined argument tables overlap content that also lives in cli-guide.md, and the body points to a 'tests/data/' directory that is absent from the bundle, leaving a broken navigation pointer. | 4 / 5 |
Total | 15 / 20 Passed |