Content
93%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.
The body is a lean, dense instruction sheet: unambiguous operation-selection rules, exact patch_text syntax, hard size limits, and a failure-handling section with a stale-edit retry loop. The single weakness is workflow validation — destructive host writes lack a proactive post-edit verification step, relying instead on the tool's freshness rejection.
Suggestions
Add a post-edit verification step to the Editing Flow, e.g. 'After a write or patch, reread the affected lines to confirm the change landed as intended' — this would give the destructive write path a proactive feedback loop.
State what happens when multiple patch_text hunks in one request partially succeed (all-or-nothing vs. partial application), so failure recovery is unambiguous for multi-hunk edits.
Add one tiny worked patch_text example (2-3 lines showing @@ anchor, -old, +new) to make the hunk syntax rules instantly executable without re-deriving them from the rules list.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is ~45 lines of lean imperative bullets with zero padding — no explanation of what files or patching are, no throat-clearing, just operational rules like "Treat reads as bounded text previews (2,000 lines / 256 KiB)" and "Every non-header content line must begin with exactly one prefix". Every token earns its place and the skill assumes Claude's competence throughout. | 5 / 5 |
Actionability | For an instruction-only skill the guidance is fully actionable: concrete operation selection ("Prefer patch with old_text and new_text for simple exact replacements", "Use patch_text for context-anchored edits"), exact hunk syntax with @@ anchors and +/- prefixes, hard numeric limits (2,000 lines / 256 KiB), and concrete failure responses (F3 for blocked writes, reread-and-retry on stale rejection). The scoring notes state code absence is not penalized when guidance is this actionable. | 5 / 5 |
Workflow Clarity | The Editing Flow section gives a clear decision sequence (read first, then choose write/patch/patch_text by case) and includes an explicit feedback loop ("If freshness-aware line patching rejects an edit as stale, reread the file and retry with updated ranges") plus a dedicated Failure Handling section. It falls short of 5 because host file writes are destructive yet there is no post-write verification step (e.g., reread to confirm the edit landed) — validation is reactive to tool rejection rather than proactive — keeping it at 'most checkpoints present'. | 4 / 5 |
Progressive Disclosure | The skill is under 50 lines, has no bundle files, and needs none: content is split into five clearly-scoped sections (Boundary, Access Modes, Editing Flow, Patch Text Rules, Failure Handling) that are each concise enough to belong inline. Per the rubric guideline, a sub-50-line skill with no need for external references scores 5 with well-organized sections, which this matches. | 5 / 5 |
Total | 19 / 20 Passed |