Content
81%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 well-structured router skill with an explicit, well-validated workflow and fully executable commands, delegating domain detail to version-matched embedded guides and conditionally-loaded reference files. Its main weakness is token efficiency: the step-7 archive/batch policy and auth-state paragraphs are dense inline walls that would be tighter as reference files or trimmed rules.
Suggestions
Move the conversation-archive and batch-reconciliation policy in step 7 of section 4 (the 'For an authorized conversation archive...' and 'For batches...' blocks) into a dedicated reference file, keeping only a one-line trigger condition and the 'partial' reporting rule inline.
Condense the needs_refresh/auth-denial paragraphs into a short decision list (state → permitted action → forbidden action), cutting the repetitive 'Do not...' sentences that restate the same restriction.
Trim the embedded-read fallback rules in section 3 to the two concrete commands plus a single fallback sentence; the SHA-256 byte-identity check and creative-design exception can live in a reference file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is almost entirely imperative domain rules with no padding explaining concepts Claude already knows, but step 7 of section 4 ("For an authorized conversation archive that includes linked materials, derive the requested set from every captured root...") and the multi-paragraph auth/needs_refresh block are dense run-on policy walls that could be tightened or moved to a reference file. This fits 'mostly efficient but could be tightened' better than anchor 4, since the verbosity is substantial rather than minor. | 3 / 5 |
Actionability | Gives copy-paste-ready commands throughout: `lark-cli --version`, `lark-cli skills list --json`, `lark-cli skills read lark-doc/references/lark-doc-fetch.md`, `lark-cli --profile "<authorized-profile>" auth status --json`, plus concrete env vars (LARK_CLI_NO_PROXY=1). Specific behavior thresholds (exit 10, `ok == true`, `confirmation_required`) cover the common cases; this matches the fully-executable anchor. | 5 / 5 |
Workflow Clarity | A clear four-step sequence (confirm runtime → select embedded guide → load instructions → inspect/execute/verify) with explicit validation checkpoints: dry-run previews, an exit-10 human-confirmation gate for high-risk writes, batch reconciliation with a worked 'partial: 1/3' example, and an auth-error recovery branch loading `lark-shared`. Validation for destructive/batch operations is present, so the workflow-clarity cap does not apply; this matches the anchor with feedback loops and error recovery. | 5 / 5 |
Progressive Disclosure | Both referenced bundle files exist (references/recording-lookup.md, references/document-layout.md) and are clearly signaled with when-to-read conditions ("read [recording lookup](references/recording-lookup.md) before searching"), and domain detail is delegated one level deep to embedded CLI guides. Not a 5 because substantial policy content (batch reconciliation, conversation-archive derivation, needs_refresh handling) is inlined in the body where a reference file would improve navigation. | 4 / 5 |
Total | 17 / 20 Passed |