Content
82%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 tightly written, highly actionable skill body: every section carries exact commands, file paths, and repo-specific guardrails, with a real verification step and error-recovery guidance. Its only weaknesses are minor — slight redundancy in the "Why" prose and no post-recreation sequentiality check.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes competence ("Check `git status --short --branch` first", "Never hand-edit `openmeter/ent/db/`") with no explanation of concepts Claude already knows. Minor trimmable padding remains: the "Why:" bullet list and "In practice, regenerating first resolves or clarifies most conflicts much faster..." restate the same point as the preceding instruction. Not a 3 since excess is limited to one section; not a 5 since that section is mildly redundant. | 4 / 5 |
Actionability | Every section gives copy-paste-ready, executable commands: `make generate`, `atlas migrate --env local hash`, `atlas migrate --env local diff <migration-name>`, `nix develop --impure .#ci -c <command>`, and `POSTGRES_HOST=127.0.0.1 go test -tags=dynamic ./openmeter/ledger/...`, plus exact file paths (`.up.sql`/`.down.sql` pairs, `tools/migrate/migrations/atlas.sum`, `openmeter/ent/schema/*.go`). The common cases are covered concretely with no pseudocode. | 5 / 5 |
Workflow Clarity | The standard flow and migration flow are numbered and ordered, and there is an explicit validation step ("run the smallest relevant test slice for the touched area") with error-triage guidance ("If there is a known pre-existing failure, call it out explicitly and separate it from any new regression"). Regeneration-first is a feedback loop for conflicts. Not a 5 because there is no explicit post-recreation check that migrations are sequential and no guidance on what to do if `atlas migrate hash` itself fails. | 4 / 5 |
Progressive Disclosure | The single-file body is well organized into clear sections ("Before rebasing", "Standard flow", "Migration conflicts", "atlas.sum conflicts", "Verification") with no nested or buried references, and nothing clearly belongs in a separate file. Not a 5 because at ~115 lines it exceeds the under-50-line simple-skill carve-out, and areas like the migration-recreation flow could be split into a reference file to keep SKILL.md a leaner overview. | 4 / 5 |
Total | 17 / 20 Passed |