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-written router: lean, concrete about runtime parameters and scope values, with quality guards stated inline. Its main defect is structural — the routing protocol's core layer (manifest.yaml and static/core/*) is absent from the bundle, so the mandated first load step cannot be followed as written.
Suggestions
Ship manifest.yaml and the static/core/ files (principles.md, workflow.md, chinese-mode.md) referenced in routing steps 1-3, or rewrite those steps to point at files that actually exist in the bundle.
Add a one-line minimal invocation of scripts/nature_citation.py (e.g., the --text/--scope/--outdir form) in the body so the primary path is executable before opening references/script-usage.md.
If static/core files are intentionally not bundled, inline the seven-step workflow summary in the body so the skill remains followable when the referenced files are unavailable.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The 45-line router body is lean and assumes Claude's competence: every line is a routing instruction, parameter, or a scope/quality guard (e.g., "if there are more than ~10 segments, switch to the batched long-article strategy in references/script-usage.md"), with no explanation of concepts Claude already knows. Not 4 because no section is over-explanation that could be trimmed. | 5 / 5 |
Actionability | Guidance is mostly executable: it names "Prefer scripts/nature_citation.py", the concrete scope values (`Nature系列` / `CNS` / `CNS及子刊` / flagship-only) passed "as --scope", the ~10-segment batching threshold, and on-demand triggers per topic. Not 5 because the body contains no copy-paste-ready invocation (that lives in references/script-usage.md); not 3 because the directives are specific and runnable, not high-level hints or pseudocode. | 4 / 5 |
Workflow Clarity | The four-step routing sequence is clear (load core, confirm scope/language inline, run the seven-step workflow, open references on demand) with several checkpoints: "State the detected scope and date limits in one short line before searching", "never cite a metadata-only candidate without checking the abstract or publisher page", and the PMID-refetch fallback for missing given names. Not 5 because a full validate→fix→retry loop is not spelled out in the body itself (the seven-step workflow and batching validation live in files that must be loaded). | 4 / 5 |
Progressive Disclosure | The four references/ files are well signaled and one level deep (each maps to an explicit on-demand trigger), but step 1's always_load layer is broken against the actual bundle: "Read [manifest.yaml](manifest.yaml)" and "static/core/principles.md", "static/core/workflow.md", "static/core/chinese-mode.md" do not exist in the skill directory, so the primary navigation path fails. Not 4 because missing always_load files are more than a minor organization gap; not 2 because the references that do exist are clearly signaled rather than buried or inlined. | 3 / 5 |
Total | 16 / 20 Passed |