Content
82%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-structured body: executable commands with defaults and examples, a documented output schema, and a clear agent workflow with conditional fallbacks. Weaker spots are the absence of empty-result/failure handling in the workflow and a mostly monolithic layout where a references/ split would help.
Suggestions
Add a short failure/empty-result branch to the Agent 使用流程 (e.g., if no results, retry with alternate keywords or fall back to --engine web) to close the workflow-validation gap.
Trim the 示例对话 section, which repeats the search command examples already shown above, to tighten token usage.
Move the output-format field reference and .env configuration details into a references/ file (e.g., references/output-schema.md and references/configuration.md) and link them from SKILL.md to improve progressive disclosure.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Nearly all content is skill-specific operational detail Claude could not infer (CLI options table, engine behavior, output JSON schema, .env knobs) with no padding or explanations of known concepts. Minor trimming is possible — the "示例对话" example dialog largely repeats the earlier search command examples — placing it at anchor 4 rather than the fully lean anchor 5. | 4 / 5 |
Actionability | Commands are fully executable and copy-paste ready (bash and PowerShell invocations, options with defaults, worked examples for pan/quality/limit/engine variations), the output JSON structure is documented field-by-field, and the referenced scripts (film-search.sh/.ps1/.js, shared/deep-extract.js) actually exist and implement the documented search/hot/resolve subcommands. This matches anchor 5 ('specific examples cover the common cases'). | 5 / 5 |
Workflow Clarity | The "Agent 使用流程" section gives a clear 5-step sequence with a conditional checkpoint (use resolve when a result has pageUrl but no url; surface extractCode). However, there is no guidance for failure or empty results (e.g., retry with different keywords or fall back to --engine web), so it sits at anchor 4 ('most checkpoints present; minor validation gaps') rather than anchor 5's explicit feedback loops. | 4 / 5 |
Progressive Disclosure | Sections are well organized and the bundle is appropriately structured (scripts stay as executables invoked by path, verified to exist, rather than being inlined). But the ~145-line body keeps everything inline — the output-field reference and .env configuration could live in a references/ file — so it matches anchor 4 ('good structure; most content appropriately placed; minor organization gaps') rather than anchor 5's well-signaled separate references. | 4 / 5 |
Total | 17 / 20 Passed |