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.
The body is highly actionable with concrete commands and an explicit validation-first workflow including feedback loops. It is well-structured but carries a long inline rationale section and references shared implementation files that are not part of the provided bundle.
Suggestions
Confirm the referenced `_shared/check-session.ts` and `_shared/relay-sign.ts` exist in the bundle, or note them as out-of-scope implementation files rather than navigable references.
Consider moving the 背后的原理 rationale into a short reference file (e.g. REFERENCE.md) to keep SKILL.md as a lean overview, since the skill exceeds the simple <50-line threshold.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and operationally focused with concrete commands and tables rather than padding about concepts Claude already knows. The 背后的原理 section explains non-obvious operational rules (why headed, why no cookie import) but could be trimmed slightly, placing it at anchor 4 rather than 5. | 4 / 5 |
Actionability | Copy-paste-ready commands are given (`camoufox-cli --session <platform> --persistent --headed --json open "<URL>"`, `login-manager --platform <platform>`), with concrete login-page URLs, per-platform cookie fields, storage paths, and an exit-code→agent-action table covering the common cases, matching anchor 5. | 5 / 5 |
Workflow Clarity | A clearly sequenced multi-step workflow (Step 0 display rules → Step 1 open → Step 2 notify/wait → Step 3 export+verify) with explicit validation checkpoints (two-tier liveness, verify-before-commit) and feedback loops (exit 2 no-retry, retain session on failure for manual retry), matching anchor 5. | 5 / 5 |
Progressive Disclosure | Good section structure (支持的平台, 用法, 注意事项, 背后的原理) with content mostly well-placed. However the referenced `_shared/check-session.ts` and `_shared/relay-sign.ts` are mentioned inline as implementation notes and are not present in the bundle (only scripts/export-and-verify.ts exists), a minor organization gap keeping it at anchor 4 rather than 5. | 4 / 5 |
Total | 18 / 20 Passed |