Content
57%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 dense, command-first reference that is highly executable but suffers from duplicated generated-vs-custom sections, unexplained flag lists, a destructive restore operation without verification, and monolithic inlining of admin material beyond the skill's serve scope. Splitting admin content into reference files and adding post-restore validation would address the weakest dimensions.
Suggestions
Add a verification step after 'omniroute restore' (e.g. run 'omniroute doctor' or 'omniroute status' and confirm providers/config before resuming traffic), since restore destructively overwrites config and the SQLite database.
Deduplicate the generated and custom sections: merge the two Install and two Errors sections, and either give the 'Subcommands' flags brief semantics or drop the bare flag lists in favor of the 'Admin lifecycle' coverage.
Move out-of-scope admin material (tunnels, backup/restore, provider setup, logs, autostart) into clearly signaled reference files, keeping SKILL.md focused on server start/stop/restart/daemon configuration per its name and description.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly lean (command blocks, tables, one-line error-to-fix mappings, no explanations of known concepts), but it is noticeably duplicated: two install sections, a generated 'Subcommands' flags list that repeats the 'Admin lifecycle' coverage, and two separate 'Errors' sections could be merged and tightened. | 3 / 5 |
Actionability | Nearly all guidance is copy-paste ready ('omniroute serve', 'omniroute doctor --json | jq ...', env-var exports, a fully specified non-interactive setup command), but the 'Subcommands' flags (--tui, --ready-timeout, --max-restarts, etc.) are listed with no semantics and the serve example is bare rather than demonstrating flag usage. | 4 / 5 |
Workflow Clarity | The quick-start sequence (start -> setup -> doctor) includes a validation checkpoint and reset-encrypted-columns shows dry-run-before-force, but 'omniroute restore' — a destructive overwrite of config and SQLite DB — has no validation/verification step, capping workflow clarity at 3 per the destructive-operation guideline. | 3 / 5 |
Progressive Disclosure | No bundle files exist and section headers are clear with an explicit table of external skill links, but this ~320-line body inlines admin material (tunnels, backup/restore, provider setup, logs, autostart) that belongs in separate reference files for a skill named 'cli-serve'. | 3 / 5 |
Total | 13 / 20 Passed |