Content
75%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.
A well-structured, actionable workflow for a batch/destructive task that appropriately includes verification and feedback loops and correctly externalizes its discovery script. Main improvements would be generalizing the hardcoded cargo verify command and tightening the repeated 'don't delete before done' emphasis.
Suggestions
Generalize the verification step: present `cargo insta test --accept` as one example of a project-standard verification command rather than the literal command to run, so the skill applies outside Rust projects.
Consolidate the 'never remove a FIXME before the work is done' guidance into one authoritative callout instead of restating it in step 4, the blockquote, and the Notes.
Make the script reference an explicit navigation signal (e.g., a labeled 'See scripts/find-fixme.sh for discovery behavior' pointer) alongside the inline invocation to strengthen progressive disclosure.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Lean, imperative body with no padding of concepts Claude already knows; the only minor trim opportunity is the 'do not remove a FIXME before finishing' rule, which is restated in step 4, the critical-rule blockquote, and the Notes. | 4 / 5 |
Actionability | Provides concrete, executable commands (`bash .forge/skills/resolve-fixme/scripts/find-fixme.sh [PATH]`, `cargo insta test --accept`) backed by a real bundle script, with specific behavioral specs; the only gap is the hardcoded `cargo insta` verify command, which assumes a Rust project. | 4 / 5 |
Workflow Clarity | Clear five-step sequence (discover → expand → consolidate → implement → verify) with an explicit validation checkpoint and feedback loop in step 5 that satisfies the batch/destructive validation requirement; the cargo-specific verify command and slight tension between 'project's standard verification step' and the hardcoded example are minor gaps. | 4 / 5 |
Progressive Disclosure | Well-organized with clear section headers and one correctly split-out bundle file (the discovery script, which exists at scripts/find-fixme.sh) referenced one level deep; the script is invoked inline rather than presented via an explicit 'see X for details' navigation signal, leaving minor organization room. | 4 / 5 |
Total | 16 / 20 Passed |