Content
88%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.
An exemplary skill body: engine-selection routing is table-driven, the doctor step provides real validation with state-by-state recovery actions, the session loop is copy-paste ready, and every referenced bundle file exists and is clearly labeled. The two small costs are a couple of trimmable meta-passages and the owned-engine reference subtree adding a second level of nesting.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is terse, table-driven, and telegraphic ("Two identical failures mean change approach, not retry. A third identical attempt is a defect."), with no explanation of concepts Claude already knows. Two minor instances that could be trimmed: the guard meta-note "It is not a security boundary: code that imports the raw entry (`resolveOmowrightEntry()`) is not guarded..." and the emphasis restatement "A Chrome that is merely installed is not their browser" following "the browser the user actually uses". Anchor 4 (efficient, minor instances of over-explanation) fits better than anchor 5, whose "every token earns its place" is slightly undercut by those passages. | 4 / 5 |
Actionability | Fully executable throughout: `node "<skill-root>/scripts/browser-doctor.mjs" --json`, a complete session loop with real method calls (`connectBrowserSkill`, `bskSnapshot`, `click`, `fill`, `press`, `screenshot`, `stop()` in a `finally`), `session.requestHelp({ prompt, targets, timeoutMs })`, and install commands. The doctor state table maps every output (`ready`, `no-cli`, `choose-browser`, `no-browser-support`) to an exact next command. Copy-paste ready; matches anchor 5. | 5 / 5 |
Workflow Clarity | Clear sequence: Step 0 (engine gate keyed on `OMO_BROWSER_ENGINE`) → Step 1 (load + validate via browser-doctor, with a recovery loop: "relay it verbatim, wait, re-run the doctor") → the loop with a numbered rules checklist ("Read before every action", "Navigation and large DOM changes stale every ref") and mandatory `stop()` cleanup. Explicit validation step plus feedback loops for error recovery; anchor 5's checklist-and-recovery pattern matches. | 5 / 5 |
Progressive Disclosure | All referenced files exist (references/commands.md, install.md, remote.md, owned-engine/README.md, recipes/1password.md) and the "Where the rest lives" table labels each with its content, so navigation is clear. However, the owned-engine subtree (README.md → ladder.md, network.md, frames-and-humans.md) makes the reference hierarchy two levels deep, short of anchor 5's "one-level-deep references"; anchor 4's "minor organization gaps" is the best fit, and it is well above anchor 3 since nothing is buried or unlabeled. | 4 / 5 |
Total | 18 / 20 Passed |