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 overview for a broad toolkit: every capability section carries a concrete code sketch and a pointer to a real, one-level-deep reference file, and the analysis-mode rule is crisply stated. Weaknesses are modest — some duplicated example code between 'Core Capabilities' and 'Common Workflows', placeholder variables in section snippets, and absent validation/error-recovery checkpoints in the processing pipelines.
Suggestions
Deduplicate: the 'Common Workflows' Quick Start and Multi-Modal examples repeat code already shown in Core Capabilities sections 1 and 11 — replace them with workflows not already covered (e.g., a full EDA SCR analysis or an event-related EEG pipeline) or fold the runnable ecg_simulate Quick Start into section 1 and drop the rest.
Add a brief validation checkpoint to the main pipelines — e.g., after ecg_process show how to check signal quality (nk.ecg_quality / info['ECG_R_Peaks'] sanity check on detected peak count) and what to do when detection fails, which would lift workflow_clarity to 5.
Fix the small progressive-disclosure gap: ppg.md is listed in the References section but never linked from the PPG-related content (section 1 covers ECG/PPG but only links ecg_cardiac.md) — either link it inline where PPG analysis is described or merge its coverage explicitly.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is efficient: no tutorial padding, no explanations of what ECG/EEG are, and each section is a compact capabilities list plus a short code block. It stops short of 5 due to redundant material — the 'Common Workflows' Quick Start ECG example repeats section 1's ecg_process/hrv calls, the Multi-Modal workflow repeats section 11's bio_process call, and the References section re-lists files already linked inline (including ppg.md, which no section links). | 4 / 5 |
Actionability | Mostly executable, copy-paste-shaped code: real function names with realistic parameters (e.g., nk.ecg_process(ecg, sampling_rate=1000), nk.hrv(info['ECG_R_Peaks'], ...), and a fully runnable Quick Start using nk.ecg_simulate). Not a 5 because many per-section snippets depend on undefined placeholder variables (ecg_signal, peaks, trigger_channel) without showing their derivation, and parameters like nk.hrv(peaks) assume context the reader must get from the reference files. | 4 / 5 |
Workflow Clarity | Sequences are clearly laid out — the ECG pipeline 'cleaning → R-peak detection → delineation → quality assessment' and the event-related vs interval-related mode-selection rule (< 10s / ≥ 10s) are explicit and actionable. Validation is mostly implicit (quality assessment is mentioned as a pipeline stage but no checkpoint shows how to act on it, and no error-recovery guidance exists for failed peak detection or bad channels), which keeps it below 5; this is analysis rather than destructive/batch work, so the destructive-cap does not apply. | 4 / 5 |
Progressive Disclosure | This is the anchor 5 pattern: an overview SKILL.md where each of the 11 capability sections links to its own one-level-deep reference file (all 12 referenced files — ecg_cardiac.md, hrv.md, eeg.md, eda.md, rsp.md, ppg.md, emg.md, eog.md, signal_processing.md, complexity.md, epochs_events.md, bio_module.md — exist in ./references/), plus a consolidated References list and an explicit 'Load specific reference files as needed using the Read tool' navigation instruction. No nesting beyond one level, no orphan chains. | 5 / 5 |
Total | 17 / 20 Passed |