Content
96%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 exceptionally dense, actionable body: executable code for both engines, a doctor-driven validation loop with explicit state handling, a disciplined interaction loop with checklists, and safety rules with brief rationales. The only structural weakness is the owned-engine branch requiring two hops to reach ladder/network/frames content.
Suggestions
Add one-line pointers to references/owned-engine/ladder.md, network.md and frames-and-humans.md (or fold them into the 'Where the rest lives' table) so no content sits two hops from SKILL.md.
The Step 0 desktop section duplicates the engine-choice framing ('everything above works exactly as in the terminal... with the same table'); a single sentence plus the connectBuiltinBrowser block would trim it without losing information.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Nearly every token is operational: engine-selection tables, doctor-state tables with next actions, and copy-paste code with inline comments. It never explains concepts Claude already knows (no 'what a browser is', no library intros); even rule rationales are one clause ("It is the user's real profile; clearing it logs them out everywhere"). This fits the 'lean and efficient; every token earns its place' anchor rather than the score-4 'minor instances of over-explanation'. | 5 / 5 |
Actionability | Fully executable guidance throughout: the loadOmowright import, `node "<skill-root>/scripts/browser-doctor.mjs" --json`, a complete attached-session try/finally block (`connectBrowserSkill`, `bskSnapshot`, `click`, `fill`, `press`, `waitForNavigation`, `screenshot`, `stop`), the `connectBuiltinBrowser()` example, and `requestHelp({ prompt, targets, timeoutMs })`. Doctor states map directly to next actions. This matches the 'fully executable; copy-paste ready' anchor, not score 4's 'minor gaps'. | 5 / 5 |
Workflow Clarity | The sequence is explicit (engine table → Step 1 doctor → the loop → handoff → stop) with real validation checkpoints and feedback loops: the doctor's state table ends in 'relay it verbatim, wait, re-run the doctor'; the numbered loop rules ('Read before every action', 'Navigation and large DOM changes stale every ref', 'Two identical failures mean change approach') act as a checklist; and `stop()` is mandated 'on success and on failure'. Error recovery paths (BuiltinBrowserUnavailableError codes, requestHelp outcomes) are enumerated. This is the score-5 anchor, above score 4's 'minor validation gaps'. | 5 / 5 |
Progressive Disclosure | The body is a well-navigable overview with a clear 'Where the rest lives' table, and every referenced path (commands.md, install.md, remote.md, owned-engine/README.md, recipes/1password.md) plus all four scripts exist in the bundle. However, the owned-engine README is itself a hub linking three further files (ladder.md, network.md, frames-and-humans.md), making that branch a two-level reference chain, and it defers to an external authoritative skill. That is the 'good structure... minor organization gaps' of score 4 rather than the fully one-level-deep references of score 5. | 4 / 5 |
Total | 19 / 20 Passed |