Content
77%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This is a well-structured validator skill with excellent actionability and workflow clarity. The 5-step process is clearly defined with concrete commands, specific code patterns to detect, and a precise JSON output schema. The main weakness is moderate verbosity—the explanatory notes ('Why HARD', 'Justification') and the inline canonical proverbs list add bulk that could be trimmed or externalized.
Suggestions
Trim the 'Why HARD' and 'Justification' explanations from each proverb entry—Claude can infer why data races are bad or why unwrapped errors are problematic.
Consider moving the canonical proverbs fallback list to a separate reference file to reduce the main skill's token footprint.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The skill is mostly efficient but includes some unnecessary explanation. The 'Why HARD' justifications and 'Justification' notes for SHOULD violations add moderate value but could be trimmed. The scope declaration section explaining what NOT to check is useful for a validator but somewhat verbose. The canonical proverbs fallback list adds bulk but serves a clear purpose. | 2 / 3 |
Actionability | The skill provides fully executable bash commands for getting changes, concrete code patterns to look for (e.g., `if err != nil { return err }`), specific suggestions (e.g., `fmt.Errorf("failed to process user: %w", err)`), and a complete JSON output schema with example values. Every step has concrete, actionable guidance. | 3 / 3 |
Workflow Clarity | The 5-step workflow is clearly sequenced with explicit ordering (get changes → get proverbs → read files → check → report). It includes fallback mechanisms (git diff ordering, offline proverbs list), batch processing guidance for large changesets, and a clear pass/fail criterion. The validation is inherent in the structured output format. | 3 / 3 |
Progressive Disclosure | The content is well-organized with clear sections and headers, but it's a long monolithic file (~150 lines of content) with no references to external files. The canonical proverbs list and detailed violation descriptions could be split into separate reference files. However, for a standalone skill with no bundle, keeping everything inline is reasonable. | 2 / 3 |
Total | 10 / 12 Passed |