Content
85%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 highly actionable, well-sequenced operational guide with executable commands, explicit decision logic, and genuine error-recovery loops. Its one real defect is bundle hygiene: the Maintenance, docs/, .env.example, and LICENSE/NOTICE references point at files that are not shipped, and the vertical-domain guidance is duplicated between the Trigger and Decision Flow sections.
Suggestions
Ship or remove the dangling references: either include MAINTENANCE.md, docs/anysearch-skill.md, .env.example, LICENSE, and NOTICE in the bundle, or drop the Maintenance section and the `cp <skill_dir>/.env.example` command (replaced with 'create .env containing ANYSEARCH_API_KEY=...').
Deduplicate the vertical-domain guidance: keep the DEFAULT/HYBRID/exception rule in one place (Trigger) and let the Decision Flow diagram be purely graphical.
Trim platform-justification prose (the two-CLI 'smaller review/maintenance surface' rationale and the base/latest/full Docker image inventory) — it explains decisions Claude doesn't need to execute them.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and nearly filler-free: no explanations of concepts Claude already knows, and every section carries operational content (cheat sheet, decision flow, key priority chain, runtime steps). Minor trim opportunities keep it at 4 rather than 5: the vertical-domain rule is stated twice (Trigger section and Decision Flow), the two-CLI rationale ('on purpose (smaller review/maintenance surface)') and the base/latest/full Docker image detail are justification prose, and the Required-params rule duplicates what `doc` renders. | 4 / 5 |
Actionability | Fully executable, copy-paste-ready commands cover the common cases: the cheat sheet shows concrete invocations for search (general, --tag, --domain/--sdp), get_sub_domains, batch_search (flag and JSON forms), and extract; the `<cmd>` placeholder is explicitly defined ('Replace <cmd> with the command from runtime.conf (for example, python3 <skill_dir>/scripts/anysearch_cli.py)'); runtime detection/persistence and .env setup are given as runnable commands; and failure paths are concrete ('run <cmd> <subcommand> --help rather than full doc') with invalid invocations enumerated ('extract --format markdown ... Invalid'). | 5 / 5 |
Workflow Clarity | A clear decision flow with explicit branches (Path 1 / HYBRID / Path 2), a mandatory get_sub_domains-before-vertical-search precondition, and a runtime bootstrap that validates before persisting (detect → entry-test `doc` → write runtime.conf, with 'replace it instead of appending'). Error-recovery feedback loops are present for the failure modes that matter: argument-schema failure → `--help`, service/quota/network failure → the Fallback rule ('inform the user and MAY fall back to other available search methods if the user approves'), which satisfies the batch/validation cap guidance. This is not 4 because checkpoints are explicit rather than implied at every branch. | 5 / 5 |
Progressive Disclosure | Structure is good (Overview → Trigger → Cheat Sheet → Decision Flow → Key Config → Runtime → Security → Maintenance) and the full API reference is correctly pushed one level deep via the `doc` command (rendered from scripts/shared/doc_spec.md, which exists in the bundle). But scored against the actual bundle, several referenced files do not exist: 'See MAINTENANCE.md in this skill directory and docs/anysearch-skill.md', 'cp <skill_dir>/.env.example <skill_dir>/.env' (a command that fails — no .env.example is shipped), and 'see LICENSE and NOTICE in this skill directory' are all dangling references. Not 2, because the core content is well placed and the primary deep-reference mechanism (`doc`) works; not 4, because broken navigation is more than a minor organization gap. | 3 / 5 |
Total | 17 / 20 Passed |