Content
42%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 generated command catalog: install steps are concrete and the per-subcommand layout is navigable, but it suffers from duplicated sections with conflicting flags, examples that add nothing beyond the command name, placeholders never shown with real values, and no workflow or validation guidance for destructive operations. It functions as a lookup table rather than an actionable skill.
Suggestions
Deduplicate the repeated sections: 'context-eng config', 'context-eng show', 'context-eng set', and 'context-eng list' each appear twice with different flag sets, which is confusing and incoherent.
Make examples real: show flags with plausible values (e.g., 'omniroute context-eng set --token-budget 8000 --reserve-pct 20') instead of bare command skeletons, and add one-line descriptions to every subcommand.
Split the full command reference into a separate reference file and keep SKILL.md as an overview with the key workflows (add/switch a context, add and test an RTK filter), including a confirm/verify step before destructive operations like 'remove', 'expire-all', and 'migrate'.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Every subcommand carries a boilerplate '**Example:**' block that merely restates the header command, flags are listed without values, and 'context-eng config/show/set/list' each appear twice (with conflicting flag sets) — noticeably redundant for the information actually delivered. | 2 / 5 |
Actionability | Commands are real and executable ('npm install -g omniroute', 'omniroute sessions list'), but examples are bare skeletons: placeholders like <id>, <name>, and <file> are never instantiated, no example ever shows a flag with a value, and most subcommands have no description of what they do. | 3 / 5 |
Workflow Clarity | Per-subcommand usage is discernible, but there is no task sequencing (e.g., add context then switch to it, add a filter then test it) and no validation or confirmation guidance for destructive/batch operations ('remove', 'expire-all', 'migrate'), which caps workflow clarity at 3 per the rubric. | 3 / 5 |
Progressive Disclosure | The body has section structure (per-subcommand headers, a quick-install section), but ~400 lines of command reference are inlined in SKILL.md that clearly belong in a separate reference file, and no bundle files or links to deeper material exist — matching the 'overview plus 200 lines of API reference that could be in a separate file' anchor. | 3 / 5 |
Total | 11 / 20 Passed |