Content
75%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 policy skill: every delegation surface has executable code, decision tables resolve the main ambiguities, and risky auto-submit operations get explicit guardrails. Its weakness is redundancy — the completeText and auto-submit rules each appear in two or three sections — and the absence of any failure-path guidance keeps workflow clarity at 4.
Suggestions
State the completeText() escape hatch once (in 'How' or 'Exception') and have the 'Don't' entry reference it in one line, cutting roughly two repeated sections of explanation.
Merge the auto-submit guidance ('Capture user input in the sidebar' and the 'Don't auto-submit a hardcoded prompt' bullet) into a single rule with its examples to remove the duplicated rationale.
Add a short failure-path note for background runs and sub-agent delegation (e.g. how to detect and retry a failed run alongside useAgentChatGenerating) to close the workflow-clarity gap.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient app-specific guidance Claude could not infer (sendToAgentChat semantics, the message/context boundary), but the completeText escape-hatch rule is stated three times ('Rule', 'Don't', 'Exception') and the auto-submit rule twice ('Capture user input', 'Don't'), so the document could be meaningfully tightened. It is not anchor 4's 'minor instances' — the redundancy spans whole sections. | 3 / 5 |
Actionability | Copy-paste-ready TypeScript for every surface (client, background, Node scripts, server transform, generating-state hook) plus two decision tables (message/context/images surfaces, submit true/false/omitted) and a concrete Button example. The common delegation cases are covered with executable code and specific option values. | 5 / 5 |
Workflow Clarity | Decision flows are explicit and sequenced: surface selection by caller type, a submit-vs-prefill decision table, a sub-agent briefing contract (Objective/Context/Output/Boundaries) with fan-out and synthesis discipline, and checkpoints on risky operations ('Never auto-submit a generic creative prompt', prefilling 'Delete all projects older than 30 days' for user review). Falls short of anchor 5 because there are no error-recovery loops (e.g. what to do when callAgent or a sub-agent run fails). | 4 / 5 |
Progressive Disclosure | No bundle files exist, and the body is well-sectioned with clear headers and one clearly signaled cross-skill reference ('See the a2a-protocol skill for the full pattern'). Minor gap: the ~50-line sub-agent teams section is a separable concern that could live in its own reference file. Not anchor 3 — structure and signaling are good, and inline placement of a 285-line policy document is defensible without a bundle. | 4 / 5 |
Total | 16 / 20 Passed |