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.
An exemplary procedural skill body: version-pinned executable examples, a non-negotiable ten-step checklist with validation checkpoints, and clean one-level-deep disclosure into six existing reference files and five existing CLI scripts. The only notable drag is conciseness — the historical security-triage narrative, the citation-fetch protocol, and the dated URL list consume tokens that could be trimmed or relocated to a reference file.
Suggestions
Condense the Security triage section to its actionable rule (never name a project script after an imported package) and drop the historical phantom-finding inventory narrative.
Move the dated source URL list into a reference file and keep only the release/version pins inline, so stale-dated content does not sit in the main body.
Trim the citation-fetch protocol to a two-line pointer to preserve token budget for the survival workflow itself.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and procedural and assumes competence (no explanation of what survival analysis or Cox regression is), with nearly every token carrying non-obvious contract detail (metric input shapes, 0.28 API defects, leakage rules). However, administrative/history sections pad it — the Security triage recounting a "phantom analyzer finding" from a past inventory, the multi-paragraph citation-fetch protocol, and the dated source URL list — which is minor over-explanation that could be trimmed rather than 'mostly efficient with unnecessary explanation'. | 4 / 5 |
Actionability | Every section is copy-paste executable: Surv construction, a full leakage-safe ColumnTransformer pipeline, the metric-contract snippet with exact array shapes, the GridSearchCV wrapper with parameter prefixes, the competing-risk CIF call, pinned uv install commands, and concrete CLI invocations. Code is complete and covers the common cases, matching the anchor for fully executable, copy-paste-ready guidance rather than 'minor gaps'. | 5 / 5 |
Workflow Clarity | The "Non-negotiable workflow" is an explicit 10-step checklist with built-in validation checkpoints (validate outcomes, split before learned preprocessing, restrict evaluation times, match predictions to metrics), and the CLI section sequences validate → train → evaluate → report with stated failure behavior ("fails clearly on an unsupported holdout"). This matches the anchor for clear sequence, explicit validation steps, and a checklist for a complex process. | 5 / 5 |
Progressive Disclosure | The body is an overview that delegates depth to six real reference files (all present in references/) and five real scripts (all present in scripts/), with every inline pointer stating when to read it (e.g. "Use references/data-handling.md before loading custom or competing-risk data") plus a Reference files index section. Bundle structure is one level deep with only lateral cross-links between reference files, matching the anchor for a clear overview with well-signaled one-level-deep references. | 5 / 5 |
Total | 19 / 20 Passed |