Content
63%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 well-organized and broadly actionable with a clear multi-scene workflow and recovery guidance, but it is held back by repeated skill-type classification, a missing explicit retry loop, and references to bundle files that are not actually present.
Suggestions
Consolidate the command/workflow/judgment/reference-heavy classification into one canonical location and reference it from the other sections instead of restating it.
Add an explicit validate→fix→retry step to the Canonical workflow path (e.g. 'if validation fails, fix the reported issue and re-run step 6 before finalizing').
Either ship the referenced files (resources/ssl-lite-template.md, resources/validation-checklist.md, ../_shared/core/*.md) or remove/correct the References entries so navigation is not broken.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient (tables, numbered steps, terse guardrails), but the command/workflow/judgment/reference-heavy classification is repeated roughly four times (Expected inputs, Control-flow features, Entry, Canonical workflow path) and the Guardrails section restates constraints already covered elsewhere. | 3 / 5 |
Actionability | Provides concrete, executable guidance — named tools ("rg, find, awk, sed, git diff --check", "apply_patch"), specific paths, a numbered Canonical workflow path, and explicit guardrails — with only the minor gap that execution leans on a template file that is not present in the bundle. | 4 / 5 |
Workflow Clarity | Clear sequencing via Entry, Scenes (PREPARE→FINALIZE), and a Canonical workflow path, plus a VERIFY scene and a Failure and recovery table; however the canonical path's validation step (step 6) does not spell out an explicit validate→fix→retry loop. | 4 / 5 |
Progressive Disclosure | The in-body section structure (Scheduling / Structural Flow / Logical Operations / References) is clean, but the References section points to resources/ssl-lite-template.md, resources/validation-checklist.md, and ../_shared/core/*.md that do not exist in the bundle, so the signaled navigation is broken. | 3 / 5 |
Total | 14 / 20 Passed |