Content
90%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 exemplary repo-workflow skill: dense, non-obvious, copy-paste-ready knowledge with zero padding and a clear result-interpretation checkpoint for tests. The only structural lever is that the detailed testing recipes sit inline in an already-full SKILL.md and could move to a reference file if the skill grows.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Every line is repo-specific, non-inferable knowledge: exact commands, file paths, function names, and gotchas ("init_db.sql uses id as PK for sessions, not session_id"). There is no explanation of concepts Claude already knows, and the dense bullet style matches the 'lean and efficient; every token earns its place' anchor. Not 4 because no section could be trimmed without losing non-obvious information. | 5 / 5 |
Actionability | Commands are copy-paste ready ("pytest tests/ -q --basetemp .pytest_tmp", "python3 -m py_compile daemon/main.py cli/mcp_adapter.py"), and the testing patterns give exact idioms with real identifiers ("MagicMock(spec=Dependencies)", "app.dependency_overrides[get_dependencies] = lambda: deps", "asyncio.run(handler(req, deps=deps, _auth=None))"). This matches the fully-executable anchor with specific examples covering the common cases; no gaps found, so it does not fall to 4. | 5 / 5 |
Workflow Clarity | The Commands section sequences quick syntax check → full suite → targeted run, and includes a result-interpretation checkpoint ("'passed + skipped' = green" with expected ~7 skips), matching 'clear sequence with most checkpoints present'. Not 5 because the edit→verify loop is implicit rather than an explicit validate-and-recover workflow (no 'if tests fail, do X' guidance); not 3 because validation checkpoints are present, not absent, and no destructive/batch cap applies. | 4 / 5 |
Progressive Disclosure | The single SKILL.md is well organized into Commands / Architecture conventions / Gotchas / Testing patterns with no nested references at all (no references/, scripts/, or assets/ bundle exists), matching 'good structure; most content appropriately placed'. Not 5 because at ~70 body lines it exceeds the under-50-line simple-skill exception, and the detailed Testing patterns section is the one candidate that could split into a reference file; not 3 because there is no misplaced inline bulk and navigation is easy. | 4 / 5 |
Total | 18 / 20 Passed |