Content
53%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 body is a dense, mostly executable REST reference with clear endpoints and query syntax, but it is undermined by a bundle that omits every file it references — the five referenced reference/script files do not exist — and by an ID mapping batch workflow with no validation steps. There is also moderate redundancy across sections that could be tightened.
Suggestions
Ship the referenced bundle files (references/api_fields.md, references/id_mapping_databases.md, references/query_syntax.md, references/api_examples.md, scripts/uniprot_client.py) or remove the pointers and inline the essential content — currently all five references are dead, so progressive disclosure and the Python helper section are unusable.
Add validation checkpoints to the ID mapping workflow: poll the status endpoint until the job completes, handle FAILED/errored jobs, and verify the returned mappings before use, since this is a batch operation on up to 100,000 IDs.
Deduplicate the content: merge the 'Query Syntax Examples' section with the search patterns in 'Searching for Proteins', and consolidate the Resources file listing with the inline reference citations to cut repeated tokens.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is information-dense with no padded concept explanations, but there is real repetition: query examples ("gene:BRCA1", "accession:P12345", "length:[100 TO 500]") appear in both 'Searching for Proteins' and 'Query Syntax Examples', the 'When to Use This Skill' section restates the description's capability list, and the Resources section re-lists reference files already cited inline. This matches 'mostly efficient but could be tightened' rather than 'minor instances that could be trimmed' (4). | 3 / 5 |
Actionability | Concrete, copy-pasteable guidance dominates: full endpoints ("https://rest.uniprot.org/uniprotkb/search?query={query}&format={format}"), executable query strings, a worked example URL, a field list, and a 3-step ID mapping workflow with real URLs. Not 5 because the 'Python Implementation' section directs Claude to a helper script ("use the provided helper script scripts/uniprot_client.py") that does not exist in the bundle, and no inline runnable code (curl/requests) covers the common cases. | 4 / 5 |
Workflow Clarity | The ID mapping workflow has a clear sequence ("1. Submit mapping job... 2. Check job status... 3. Retrieve results") but no validation checkpoints — no polling-until-complete guidance, no handling of FAILED jobs, and no verification of mapped IDs. Since ID mapping is a batch operation (the skill itself notes 'Maximum 100,000 IDs per job'), the rubric's cap of 3 for batch workflows without validation applies. Not 4 because the checkpoints aren't merely implicit — they are absent. | 3 / 5 |
Progressive Disclosure | The body is well-sectioned and clearly signals one-level-deep references ("See /references/query_syntax.md for comprehensive syntax documentation"), but scored against the actual bundle structure: none of the five referenced files (references/api_fields.md, id_mapping_databases.md, query_syntax.md, api_examples.md, scripts/uniprot_client.py) exist in the bundle — every pointer is dead, so the disclosure structure fails in practice. Not 3+ because functioning references are the core of this dimension; not 1 because the SKILL.md itself is well-organized and navigable, not a monolith or nested chain. | 2 / 5 |
Total | 12 / 20 Passed |