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.
The body is highly executable with strong workflow sequencing and validation checkpoints, but progressive disclosure is weakened by two referenced files (MIGRATE_CRONS.md, REMOVE.md) that do not exist in the skill bundle, and the overview is long for a SKILL.md.
Suggestions
Ship the referenced MIGRATE_CRONS.md and REMOVE.md files (e.g. in references/ or the skill root) so the one-level-deep navigation links resolve to real content.
Move the detailed v2-architecture exposition into a reference file and keep SKILL.md as a lean phase-by-phase overview to reduce the ~530-line body.
Replace the 'other config' catch-all in the description with one or two concrete items (e.g. timezone, container timeout) to lift specificity.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Action-oriented and largely free of basic-concept padding, but the SKILL.md body runs ~530 lines with dense architectural exposition that could be trimmed or moved to a reference file. | 4 / 5 |
Actionability | Copy-paste-ready commands throughout (pnpm exec tsx, ncl users/members, onecli secrets create, cp/sed) with full flags and concrete placeholders covering the common migration cases. | 5 / 5 |
Workflow Clarity | A clear Phase 0-8 sequence with a migration-state.md checkpoint re-read each phase, explicit validation in Phase 8 (build + vitest run), sanity-checking in Phase 0, and a troubleshooting section providing feedback loops for destructive/batch operations. | 5 / 5 |
Progressive Disclosure | Phases are well organized and the scripts/ bundle is referenced correctly, but the body references ${CLAUDE_SKILL_DIR}/MIGRATE_CRONS.md and REMOVE.md which are not present in the bundle, breaking one-level-deep navigation. | 3 / 5 |
Total | 17 / 20 Passed |