Content
50%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This skill provides highly actionable, executable guidance with clear tool commands and detection strategies. However, it suffers from severe verbosity - the inline report template and redundant explanations bloat the content significantly. The workflow is logical but lacks explicit validation gates, and content that should be in reference files is embedded inline.
Suggestions
Move the entire 'Generate Report' template section to a separate reference file (e.g., references/report-template.md) and link to it
Remove explanatory text about what dead code types are (e.g., 'Code that can never execute') - Claude knows this
Consolidate the 'Common False Positives' section with the 'Verify Findings' section to eliminate redundancy
Add explicit validation checkpoint: 'Before generating report, confirm at least 3 findings have been verified manually'
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Extremely verbose at ~400+ lines with significant redundancy. The report template alone is ~150 lines of example output. Explains concepts Claude knows (what unreachable code is, what unused imports are). The 'Common False Positives' section repeats information already covered in verification steps. | 1 / 3 |
Actionability | Provides fully executable bash commands and Python code snippets throughout. Tool installation, usage commands, and grep patterns are all copy-paste ready. The bundled scripts have clear invocation syntax with examples. | 3 / 3 |
Workflow Clarity | Steps are clearly numbered (1-6) with logical sequence, but lacks explicit validation checkpoints. The 'Verify Findings' step exists but doesn't have a clear feedback loop for when verification fails. No explicit 'stop and check' gates before proceeding to report generation. | 2 / 3 |
Progressive Disclosure | References external file (dead-code-patterns.md) appropriately, but the main skill file is monolithic with the massive report template inline. The report template and common false positives sections should be in separate reference files. | 2 / 3 |
Total | 8 / 12 Passed |