Content
92%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 instruction-only skill body: dense with non-obvious project constraints, a validated refactoring workflow with baseline checkpoints, and clean delegation of detail to existing project knowledge files. The only minor gap is that a few directives ('grep all YAML files first') could be made copy-paste executable.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Every line carries repo-specific knowledge Claude could not know (import-time MODELING_REGISTRY population, strict SP ordering 'pad → slice → FA kwargs → slice position_ids', position_ids == 0 segment boundaries, silent YAML config breakage). There is no padding and no explanation of concepts Claude already knows, matching anchor 5's 'every token earns its place'; anchor 4 would require trimmable over-explanation, which is absent. | 5 / 5 |
Actionability | Guidance is mostly executable and concrete: exact paths (veomni/data/data_collator.py, .agents/knowledge/testing.md), commands ('run pytest tests/', 'grep all YAML files first'), and a precise collator ordering rule. It falls short of anchor 5 because some direction stays at the hint level (e.g. 'grep all YAML files first' without the actual grep command) and the impact-analysis table says what to check but not how. | 4 / 5 |
Workflow Clarity | The Refactoring Safety Rules section is a clear sequence with explicit validation checkpoints and a feedback loop: 'Baseline first: run pytest tests/ before any change, record results', one change per commit with 'verify tests match baseline', and 'Check baseline again at the end — results must be identical'. This matches anchor 5's explicit validation with error-recovery guidance. | 5 / 5 |
Progressive Disclosure | This is a compact (~57-line) single-file skill with no bundle directories; all sections are tight and well-organized, and detail is correctly delegated one level deep to existing project files (docs/, configs/, .agents/knowledge/*.md) with a compact inline summary of the testing rules. Per the rubric's short-skill exception, well-organized sections with no need for external references warrant a 5. | 5 / 5 |
Total | 19 / 20 Passed |