Content
60%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is a broad 'senior engineer' persona prompt with useful code patterns but heavy generic advice Claude already knows, malformed code tokens, and no progressive disclosure or bundle structure. It reads more like a system prompt than a focused, actionable skill. Tightening the guidance and fixing the broken code would materially raise quality.
Suggestions
Cut generic principles Claude already knows (SOLID/DRY/KISS/YAGNI, coverage thresholds, function-length rules) and keep only skill-specific conventions and examples.
Fix the broken tokens throughout (e.g. '$heavy-module' -> './heavy-module', '$auth$login' -> '/auth/login', 'swarm$coder$status' -> 'swarm/coder/status') so code is copy-paste executable.
Add an explicit validate-then-fix loop to the Implementation Process (e.g. 'Run lint/tests; if they fail, fix and re-run before handing off') and move large reference material (MCP tool examples, file-organization conventions) into separate files referenced one level deep.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is padded with concepts Claude already knows (SOLID/DRY/KISS/YAGNI, 'keep functions <20 lines', '>80% coverage'), but it also contains concrete code patterns, so it is mostly efficient yet includes unnecessary explanation rather than the lean anchor. | 2 / 3 |
Actionability | It provides real TypeScript code snippets, but several are non-executable or malformed — e.g. 'import(".$heavy-module")', 'api_endpoints: ["$auth$login", "$auth$logout"]', and 'swarm$coder$status' (slash paths rendered as '$') — so guidance is concrete but incomplete/broken. | 2 / 3 |
Workflow Clarity | The 'Implementation Process' lists sequenced steps (Understand, Design, TDD, Incremental), but there are no explicit validation checkpoints or feedback loops despite operations that warrant them (the 'post' hook runs lint, but the workflow body has no validate-then-fix loop). | 2 / 3 |
Progressive Disclosure | It is a monolithic single-file wall of text (~270 lines) with no references, no bundle files in references/scripts/assets, and no navigation to deeper material, matching the monolithic anchor. | 1 / 3 |
Total | 7 / 12 Passed |