Content
82%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 expert-calibrated, highly actionable body: concrete commands for every provider shape, explicit three-level validation with feedback loops, and a grep-able diagnosis table. The main gaps are length that could be offloaded to reference files, an implicit rather than explicit step sequence, and two referenced files that are not present in the bundle.
Suggestions
Consolidate the workflow into an explicit ordered checklist (e.g., 1. compute_details read → 2. pick shape → 3. build → 4. weights → 5. validate → 6. record) near the top, so the sequence doesn't have to be inferred from section order.
Move the 14-row diagnosis table to a reference file (e.g., references/diagnosis.md) and keep only the top few universal rows inline, trimming the SKILL.md token cost.
Fix or qualify the dangling references to 'remote-compute-modal/env-setup.md' and 'remote-compute-<provider>/env-setup.md' — either ship them in the bundle or note that they live in the remote-compute skill's directory so agents don't chase a missing path.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Assumes Claude's competence (no explanations of what conda or Slurm are) and nearly every sentence carries a non-obvious gotcha (purge windows, APPTAINER_CACHEDIR, no-egress compute nodes). But the four provider-shape paragraphs and the 14-row diagnosis table run long with multi-clause sentences that could be tightened or split out. Efficient with minor trimming opportunities — anchor 4, not 5 because not every token earns its place. | 4 / 5 |
Actionability | Copy-paste-ready commands throughout: 'compute_details({provider, mode:"read"})', 'conda create -n <name> python=<X>', 'apptainer pull <name>.sif docker://<ref>', 'module use $HOME/modulefiles', the literal ENV_TABLE entry, 'build_env(name)', exact sed fixes, and 'OMP/MKL/OPENBLAS_NUM_THREADS=<tier.cpus>'. Fully executable guidance covering the common cases per provider shape — matches the 5 anchor. | 5 / 5 |
Workflow Clarity | The flow (read compute_details first → recognise the shape → build → place weights → validate → record back) is coherent, and validation is explicit at three levels with when-to-run guidance ('after any env rebuild... or doc edit — and before declaring the env ready') plus symptom→fix feedback loops in the diagnosis table. But the sequence is implied by section order rather than stated as explicit steps, and there is no consolidated build checklist. Anchor 4 ('clear sequence with most checkpoints; minor gaps'), not 5. | 4 / 5 |
Progressive Disclosure | Good structure with a well-signaled, one-level-deep, real reference ('references/envs_reference.md — the Claude Science envs as worked examples'). However 'remote-compute-modal/env-setup.md' and 'remote-compute-<provider>/env-setup.md' are referenced but absent from the bundle, and the large inline diagnosis table arguably belongs in a reference file. Anchor 4 ('good structure... minor organization gaps'), not 5 because of the dangling/unverifiable references. | 4 / 5 |
Total | 17 / 20 Passed |