Content
50%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 a cleanly structured but mechanical CLI listing: commands and flags are cataloged, yet every example merely restates the usage signature, a duplicated "keys list" section is present, and destructive operations carry no validation or workflow guidance. The entire reference is inlined where a lean overview plus a separate reference file would serve better.
Suggestions
Replace placeholder examples with concrete executable invocations (e.g., "omniroute keys add openai sk-..." and "omniroute keys rotate key_abc123 --grace-period 3600000") and briefly state what each flag does.
Remove the duplicated "keys list" section (lines 40 and 148 define it with conflicting flag sets) and have examples show actual expected output or behavior.
Add validation guidance for destructive operations (remove/revoke/rotate), such as checking "omniroute keys show <id>" before revoking, and clarify when to use regenerate vs rotate vs revoke.
Trim the body to a concise overview plus quick-install and common workflows, moving the full flag-level reference into a references/ file linked from SKILL.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | There is no concept over-explanation, but every "Example" block just repeats the command signature verbatim (e.g., "omniroute keys add <provider> [apiKey]") adding no information, and the "keys list" section appears twice with different flags. This is mostly efficient with clear unnecessary repetition that could be tightened, matching the 3 anchor; it is not 4 because the redundant example blocks and duplicated section are real token waste across ~230 lines. | 3 / 5 |
Actionability | Real commands and flags are enumerated, but all examples are placeholder invocations (<id>, <provider>, [apiKey]) rather than executable, concrete usages, and no flag semantics, valid provider names, or expected outputs are given. This fits the 'some concrete guidance but incomplete; missing key details' anchor; it is not 4 because nothing shown is copy-paste executable. | 3 / 5 |
Workflow Clarity | The body only provides an install step followed by a flat command catalog: no sequencing for multi-step tasks and no validation or confirmation guidance for destructive operations (keys remove, keys revoke, keys rotate) despite their --yes flags. Per the scoring notes, destructive operations without validation cap workflow_clarity at 3; it is not 2 because the catalog itself is coherently organized and the install entry point is clear. | 3 / 5 |
Progressive Disclosure | The full ~230-line CLI API reference is inlined in SKILL.md with no separate reference files (no references/ bundle exists), matching the anchor of a couple hundred lines of API reference that belongs in a separate file. It is not 2 because the content is well-sectioned with clear per-subcommand headers, making navigation reasonable. | 3 / 5 |
Total | 12 / 20 Passed |