Content
71%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 lean, operationally precise CLI-driver skill body with concrete invocations, explicit flag rules, and an unusually thorough error/fall-through taxonomy. Its main defects are the four dangling workflows/*.md links, which leave the per-mode instructions unreachable, and prose so compressed that key contracts are deferred to the script's --help rather than disclosed in the bundle.
Suggestions
Ship the four missing workflow files (workflows/review.md, workflows/challenge.md, workflows/audit.md, workflows/quick.md) or inline their step sequences — the workflow table currently links to targets that do not exist in the bundle.
Move the Error Handling and Configuration detail (envelope-field cause rules, room-check debugging, env-key inventory) into a references/ file like the existing multi-lane.md, keeping a short error-triage summary in the body.
Consolidate the repeated "Full contract: `second-opinion --help`" pointers into one stated location and inline the minimal option/exit-code table so the skill is actionable without a runtime help fetch.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body assumes Claude's competence throughout — no padding about what CLIs or reviews are — and every section delivers operational rules ("Execute all workflow sections in order", "Always pass `--cwd` with the absolute project root path"). Not 5 because pointers like "Full contract: `second-opinion --help`" recur across three sections and several sentences are so clause-dense (the 10-line Error Handling paragraph) that trimming would improve them; not 3-2 because there is no known-concept explanation and virtually no filler. | 4 / 5 |
Actionability | The invocation is concrete and executable — ".agents/skills/second-opinion/scripts/second-opinion <mode> [options]" plus a filled-in quick-mode example with real flags ("quick \"your question here\" --cwd /path --foreground") and unambiguous pass/omit rules for `--target`, `--timeout`, `--cwd`, and `--foreground`. Not 5 because the option surface, exit-handling contract, and identity rules are deferred to "second-opinion --help" rather than shown, leaving gaps Claude must fetch at runtime; not 3 because the given commands are copy-paste executable and the flag rules cover the common cases. | 4 / 5 |
Workflow Clarity | Execution is clearly sequenced: mode table → execution rules → wait-command handling ("Execute the exact printed wait command ... until terminal"), with explicit validation checkpoints in exit codes, the stderr JSON error object ("{\"error\": ...}\"), the CLI-failure record taxonomy, and recovery feedback ("For a timeout, suggest a larger `--timeout` or a narrower `--range`"). Not 5 because the per-mode step sequences live in the workflow files and the recovery rules are scattered through dense prose rather than a checklist; not 3 because validation signals and feedback loops are explicit and thorough. | 4 / 5 |
Progressive Disclosure | Structure and signaling are good — a workflow table with clear links, and "Lane resolution, merge rules ... [references/multi-lane.md]" is a well-signaled one-level-deep reference that exists in the bundle. But four of the five linked workflow files (workflows/review.md, challenge.md, audit.md, quick.md) do not exist in the bundle, breaking navigation to the core per-mode instructions, and the dense Error Handling and Configuration walls are inlined where a reference file would fit. Not 4 because dangling primary references are more than a minor organization gap; not 2 because references are clearly signaled, not buried, and multi-lane.md is properly split out. | 3 / 5 |
Total | 15 / 20 Passed |