Content
62%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 admirably lean and well-sectioned with a genuinely useful safety constraint, but it stops at being a script catalog. It contains an outright error in one example command, omits key execution details (output format, obtaining a session_id), and never sequences the monitoring workflow the status legend implies.
Suggestions
Fix the erroneous example: `list_messages_of_session.py` should be invoked as "uv run python skills/opencode/list_messages_of_session.py {session_id}", not `list_sessions.py`.
Add a short monitoring workflow that uses the status legend, e.g. "1. Run list_sessions.py 2. For each WAITING (🟢) session, run list_messages_of_session.py with its session_id 3. Review the latest user message and respond — never kill the opencode process".
Fence the example commands as code blocks and state what each script outputs (e.g., the session_id field needed for the next command).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Every line is operational: the invocation pattern ("uv run python skills/opencode/script_name.py"), per-script commands, the critical constraint ("Do not kill the opencode process"), and a compact status legend. There is no concept explanation or padding of any kind, matching anchor 5 ("lean and efficient; every token earns its place"). | 5 / 5 |
Actionability | Commands are concrete, but the example for `list_messages_of_session.py` actually invokes `list_sessions.py` ("uv run python skills/opencode/list_sessions.py {session_id}"), which misdirects execution, and key details are missing (script output format, how to obtain a session_id). This exceeds "minor gaps" (anchor 4) and fits anchor 3 ("concrete guidance but incomplete; missing key details"). | 3 / 5 |
Workflow Clarity | The implied monitor loop (list sessions, read status, inspect messages, act) is never sequenced — the body is a script catalog plus a status legend with no steps or checkpoints (e.g., what to do for WAITING vs WORKING sessions). It sits between anchor 1 (steps missing) and anchor 2 (rough sequence with many gaps), leaning 2 only because the status legend hints at the intended flow. | 2 / 5 |
Progressive Disclosure | The skill is under 50 lines with no external files to reference, and sections (Helper Scripts / Scripts / Session Status Types) are clearly headed, qualifying for the simple-skill exception. Inconsistent formatting — unfenced example commands, sentence-fragment descriptions, trailing whitespace — keeps it just below anchor 5. | 4 / 5 |
Total | 14 / 20 Passed |