Content
81%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.
The router body is an exemplary operational protocol: fully executable script commands, a six-step workflow with validation checkpoints, fallback feedback loops, and a pre-delivery checklist, with only minor duplicated red-line text to trim. Its critical defect is progressive disclosure: the manifest and static core/fragment layer it declares mandatory are absent from the bundle, so the router's central reference chain cannot actually be followed.
Suggestions
Ship the referenced manifest.yaml and static/core/ plus static/fragments/paper_type/ files in the bundle (or inline the essential core principles/workflow/output contract into SKILL.md) so the mandatory step-1 load path resolves.
De-duplicate the script prohibitions: state them once, either in step 2 or in the 'Script red lines' section, keeping only the additional red line about not resolving scripts relative to the current working directory.
Add a brief pointer to what the 16 card sections look like (e.g., a two-line sample or an anchor into references/card-schema.md) so the router remains navigable even if the static layer is unavailable.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and operational with no explanations of concepts Claude already knows, but the 'Script red lines' section duplicates step 2's prohibitions nearly verbatim ('Never write inline Python, a temporary extraction script, or a replacement script' vs 'Do not write or execute inline Python as a substitute for either bundled script. Do not create extract_pdf.py, parse_paper.py...'). That minor trimmable duplication fits the 4 anchor; it is not 5, and far above the padding seen in the 2-3 anchors. | 4 / 5 |
Actionability | Provides copy-paste-ready invocations for both bundled scripts with explicit path resolution ('Resolve SKILL_DIR as the directory containing this loaded SKILL.md... Run exactly the bundled script by its resolved path'), argument templates, the --render-dir variant, and per-mode auditor commands covering the common cases. Fully executable guidance, matching the 5 anchor; the 4 anchor's 'minor gaps' do not apply. | 5 / 5 |
Workflow Clarity | A six-step sequenced protocol with explicit validation checkpoints ('Inspect the script exit code and the bundle validation block before drafting', 'Treat audit errors as blockers'), feedback loops for error recovery ('If preparation fails, record the failure... Then enter the strongest supported fallback mode'; 'If the auditor itself cannot run, state that failure and manually apply only its documented checks'), and a nine-item pre-delivery checklist. This matches the 5 anchor including validation and error recovery. | 5 / 5 |
Progressive Disclosure | Scored against the actual bundle: the three references/ files and two scripts/ files exist, are one level deep, and are clearly signaled with load timing ('Read [references/card-schema.md] when drafting the final Markdown'). However, the mandatory entry point is dead — 'Read [manifest.yaml](manifest.yaml), then read every file under always_load' plus 'static/core/' and 'static/fragments/paper_type/' do not exist in the bundle, so the disclosure chain breaks at step 1 and the core content is unreachable. This navigation failure is worse than the 3 anchor's 'not clearly signaled' references, though the well-signaled surviving references keep it above the monolithic/no-structure level of 1. | 2 / 5 |
Total | 16 / 20 Passed |