Content
73%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-engineered, highly actionable workflow skill: explicit sequencing, strong validation gates and feedback loops, honest negative-result handling, and a well-used one-level-deep reference bundle. Its main cost is token weight — the ownership and skill-context-priority rules are repeated many times, and the 693-line body could be tightened considerably without losing behavior.
Suggestions
State the ownership/never-edit-managed-files boundary once in a dedicated section (or references/TARGET-OWNERSHIP.md) and reference it with one line from Steps 3, 4, and 7.2 and Rules 7/12/13, cutting an estimated 30-50 lines of repetition.
Compress the 'How to apply evolve's own skill-context rules' block in Step 0.2 to the two essential rules (project-level override on conflict; enforcement check on all outputs) since the priority principle is restated in Step 3's priority warning and the body preamble.
Fix the empty '### Context Cleanup' heading before '## Artifact Ownership' (attach the /clear suggestion to it or remove it), and consider moving the large stale-rules report format and evolution-log template into a reference file to slim the main body.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body adds only project-specific process knowledge (no explanation of concepts Claude already knows), but is noticeably padded by repetition: the ownership/never-edit-managed-files boundary is restated roughly six times (the 'Critical: Respect Skill Ownership' section, Step 3 scope constraint, Step 4 'IMPORTANT', Step 7.2 item 5, and Rules 7/12/13), and the skill-context-priority principle is explained three times (Step 0.2 lines on precedence, the 'How to apply evolve's own skill-context rules' block, and the Step 3 priority warning). This fits 'mostly efficient but could be tightened' rather than 2, since the redundancy is a minority of a genuinely complex 8-step workflow. | 3 / 5 |
Actionability | Highly actionable throughout: exact paths ('.ai-factory/patches/*.md', 'patch-cursor.json'), a concrete cursor JSON shape, a Prevention Point Registry table format, complete report and skill-context file templates, bash commands (mkdir -p / rm / rmdir), AskUserQuestion batching sizes (3 and 4), and enumerated decision options. Minor gaps keep it below 5: 'Reuse the project's scenario mechanism when available' defers to an undefined external mechanism, and there is no worked example of an improvement that was validated and applied with observed results (the one example honestly records NOTRUN/postpone). | 4 / 5 |
Workflow Clarity | A clearly sequenced 8-step workflow with numbered sub-steps and explicit validation checkpoints: Step 4 gates Step 5 ('Do NOT proceed to Step 5 until all stale rule decisions are collected'), Step 6.5 validates every candidate before approval, 'Do NOT apply any changes until the user answers', post-output enforcement verification, and cursor-failure handling. Feedback loops are present (revalidate before apply, full rescan procedure, refine/postpone outcomes) alongside checklists (Rules 1-14, quality rules) — a strong match for the top anchor. | 5 / 5 |
Progressive Disclosure | Good bundle structure: three real reference files (TARGET-OWNERSHIP.md, SHARED-CONTEXT.md, CANDIDATE-VALIDATION.md), each clearly signaled and read at the right step, one level deep (they cross-link each other but contain the actual content — no pass-through nesting), with heavy procedural detail properly offloaded from the 693-line body. Minor gaps: the sizable stale-rules report format and evolution-log template are fully inline and could be split out, and the empty '### Context Cleanup' heading (immediately followed by '## Artifact Ownership') is a structural glitch. | 4 / 5 |
Total | 16 / 20 Passed |