Content
85%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-structured, command-driven skill body with an explicit lint-before-apply validation gate and recovery paths for destructive database operations, and appropriately split reference files. The only improvements are small: replace the `--dev-url "..."` placeholder, show the `atlas migrate hash` invocation, and trim the comparative pitch and repeated gate language.
Suggestions
Replace the `--dev-url "..."` placeholder in Step 6 with the concrete docker dev-url used in Steps 3-4 so the lint command is copy-paste ready.
Show the actual `atlas migrate hash` command invocation in Step 7 alongside the prose description.
Trim the Step 6 Flyway/Liquibase value-add sentence and the duplicated lint-gate instruction (stated in both 'How to use' and Step 5).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is command-first and dense with executable blocks, and tool-specific facts (dev-url scratch-DB semantics, hash tamper guard) earn their tokens. Minor trimmable padding remains: the Flyway/Liquibase value-add pitch in Step 6, the repeated lint-gate instruction in 'How to use' and Step 5, and re-explained NOT NULL default logic in the worked example — anchor 4 ("Efficient; minor instances of over-explanation that could be trimmed") rather than 5. | 4 / 5 |
Actionability | Nearly every step ships a copy-paste command with full flags (schema apply, migrate diff, migrate apply, CI YAML, worked example). Two minor gaps keep it from anchor 5: Step 6 uses a placeholder `--dev-url "..."` instead of a concrete value, and `atlas migrate hash` is described in Step 7 but never shown with its invocation. | 4 / 5 |
Workflow Clarity | Steps 1-9 are explicitly ordered with a hard validation gate: "run Step 6 lint first and assert it reports no destructive / data-loss findings; if it flags one, fix the migration ... and re-run lint until clean. Only then apply" — a full validate/fix/retry feedback loop for destructive database operations, plus error recovery for the hash-mismatch failure in Step 7 and the worked example. This matches anchor 5 (explicit validation steps, feedback loops) and clears the destructive-operation cap. | 5 / 5 |
Progressive Disclosure | The SKILL.md body is a workflow overview; HCL syntax detail and the anti-patterns/limitations catalogue are split into references/hcl-schema.md and references/atlas-caveats.md, both real files, clearly signaled inline with links and a summary of what each contains, and one level deep. This matches the anchor-5 pattern of a concise overview with well-signaled one-level-deep references. | 5 / 5 |
Total | 18 / 20 Passed |