Content
92%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.
An exemplary instruction-only skill body: lean, codebase-specific directives with a clear scope-then-review-then-report workflow, explicit verification steps, a re-review feedback loop, and well-signaled one-level-deep references to shared repo docs. The single gap is the absence of concrete commands or examples for the diff/metadata mechanics the workflow relies on.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with domain-specific directives and zero padding — e.g. 'Check feature ownership, dependency direction, renderer/main isolation, and that cross-process calls use the typed preload and ORPC/IPC surface' — with no explanations of concepts Claude already knows. Every token earns its place, matching anchor 5 rather than anchor 4, which reserves room for 'minor instances of over-explanation' that are absent here. | 5 / 5 |
Actionability | Guidance is concrete and grounded in named surfaces — 'verify the actual base and head from its metadata before calculating the changed paths and diff. Include any relevant uncommitted work separately; never guess either ref', 'For SQLite, Drizzle, credentials and backups, inspect privacy, atomicity, recovery and migration behavior' — fitting anchor 4's 'mostly executable guidance with minor gaps'. It is not anchor 5 because no commands or worked examples are given for the mechanics (e.g. how to fetch PR metadata or compute the diff), and a few directives remain abstract ('trace safe continuation and final reporting to their owners'). | 4 / 5 |
Workflow Clarity | The scope section is a numbered sequence with explicit verification checkpoints ('verify the actual base and head from its metadata', 'never guess either ref') and a genuine feedback loop ('If the PR base, head or merge base changed after earlier review, recalculate the diff and review the newly affected paths'), followed by a structured review checklist and a defined reporting protocol — matching anchor 5's 'clear sequence with explicit validation steps; feedback loops; checklists'. This is a read-only review skill, so the destructive-operation cap does not apply. | 5 / 5 |
Progressive Disclosure | The ~30-line body is a well-organized overview (scope → behavior checklist → reporting) with clearly signaled, one-level-deep references placed exactly where needed — '[failure ownership](../../../docs/error-handling.md)', '[test reliability](../../../docs/test-reliability.md)', '[Agent lessons](../../lessons.md)', '[upgrade instructions](../../../docs/upgrade-guides/README.md)' — and no bundle files exist that should have been split out, matching anchor 5's clear overview with well-signaled references. It is not anchor 4, since navigation and signaling leave no notable organization gaps. | 5 / 5 |
Total | 19 / 20 Passed |