Content
71%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 skill body with real executable code, a useful model-selection decision tree, and exemplary progressive disclosure to six existing one-level-deep reference files. Its weaknesses are moderate token redundancy (Overview background, a When-to-Use section duplicating the description, a Reference Files section duplicating inline pointers, and an imports block duplicating workflow imports) and workflows that lack explicit validation checkpoints.
Suggestions
Cut the 'Overview' background paragraph explaining what survival analysis is and delete the 'When to Use This Skill' section (already covered by the frontmatter description), saving ~20 lines.
Replace the 'Reference Files' section and the 'Quick Reference: Key Imports' block with a single pointer to the reference files — the inline 'See:' signals and workflow imports already cover both.
Add an explicit validation checkpoint to Workflow 1, e.g. after the split: verify censoring rates are similar across train/test and check the event-per-feature ratio before fitting.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The core sections (model selection tree, evaluation snippets, workflows, pitfalls) are dense and earn their tokens, but several sections are redundant or pad: the Overview explains what survival analysis is ('Survival analysis aims to establish connections between covariates and the time of an event...') — a concept Claude already knows; 'When to Use This Skill' restates the frontmatter description; the 'Reference Files' section re-lists the same six files already signaled inline with 'See: references/...'; and the 'Quick Reference: Key Imports' block repeats imports shown in the workflows. This fits 'mostly efficient but includes some unnecessary explanation or could be tightened' — it does not reach level 2's 'noticeably verbose; several padded sections' because the majority of the 390 lines are actionable rather than padded. | 3 / 5 |
Actionability | Nearly all guidance is executable: real imports, correct function signatures ('concordance_index_ipcw(y_train, y_test, risk_scores)', 'Surv.from_arrays(event=..., time=...)'), a concrete decision tree, and four runnable workflows. Minor gaps keep it below level 5: Workflow 2 calls 'np.where' without importing numpy and fits undefined 'X, y'; Workflow 3 uses 'as_concordance_index_ipcw_scorer' without showing its import; Workflow 4 references 'X_train_scaled' never created in that block. This matches 'mostly executable guidance; concrete code or commands with minor gaps'. | 4 / 5 |
Workflow Clarity | Each of the four workflows is a numbered, clearly sequenced pipeline (load → preprocess → fit → predict → evaluate) ending in an evaluation step, and 'Best Practices'/'Common Pitfalls' provide guardrails. The missing piece is explicit validation checkpoints and error-recovery loops — e.g., no step to verify censoring balance after the train-test split or to handle a failed PH-assumption check — matching 'clear sequence with most checkpoints present; minor validation gaps' rather than level 5's explicit validate/fix/retry structure. The destructive/batch cap does not apply since these are non-destructive modeling operations. | 4 / 5 |
Progressive Disclosure | The body is a genuine overview with six well-signaled references ('See: references/cox-models.md', etc.), all of which exist in the bundle, are one level deep (no reference points to another reference), and are indexed again in a 'Reference Files' section with a load-when-needed instruction. Detail is appropriately split — model-family deep dives live in the references while the body keeps only selection guidance and short snippets — matching the level-5 anchor of a clear overview with well-signaled one-level-deep references. | 5 / 5 |
Total | 16 / 20 Passed |