Content
63%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, policy-rich instruction skill with a clear four-step workflow, strong guardrails, explicit failure handling, and a mostly copy-ready output template. Its main weaknesses are repetition that inflates token cost and heavy inline content (especially the automation-offer material) that should live in reference files alongside the existing request-schema.yaml.
Suggestions
State the read-only guarantee once (e.g., in Rules) and reference it from the intro and Automation Offer Guard instead of repeating the near-identical sentence three times.
Move the Automation Offer Guard details and its prompt template into a reference file (e.g., references/automation-offer.md), keeping only a short when-to-offer summary in the body.
Consolidate the overlapping scope-resolution bullets in Section 1 ('do not assume an owner book', 'offer the same explicit choices', 'use owner scope only after the user explicitly selects it') into a single confirm-scope-before-proceeding rule.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense domain policy rather than explanation of known concepts, but the ~190 lines contain noticeable repetition that could be tightened: the read-only guarantee is stated nearly verbatim in the intro ('never changes forecast categories, close dates, CRM records, tasks, or messages'), again in the Automation Offer Guard ('must not change forecast categories, close dates, amounts, CRM records, tasks, or messages'), and again in Rules; Section 1 also restates the confirm-before-proceeding rule across several bullets. This fits 'mostly efficient but includes some unnecessary explanation or could be tightened' better than the clearly-verbose level 2 or the minor-trim level 4. | 3 / 5 |
Actionability | For an instruction-only skill the guidance is largely executable: a copy-ready output-format template, a concrete automation prompt block with named fields, an exact CRM filter example ('Sales_Coverage_Opp__c = 'Startups''), exact automation-record paths ('$CODEX_HOME/automations/*/automation.toml'), and explicit enrichment-mode enums. It falls short of 5 because some directives remain high-level — 'make a bounded candidate pass' and 'do only the minimum field or schema discovery' leave the exact actions to be inferred — matching 'mostly executable guidance with minor gaps'. | 4 / 5 |
Workflow Clarity | The four numbered steps (resolve scope → build truth set → bounded enrichment → evaluate posture) are clearly sequenced with explicit stop/ask checkpoints ('stop and ask for the smallest usable source', 'downgrade to a risk-only readout and say why'), plus a dedicated Failure Handling section and read-only guardrails. It does not reach 5 because some checkpoints are implicit rather than crisp validation steps — e.g., enrichment-mode escalation and the 'present the sourced default and ask the user to confirm it' rule are conditional prose rather than explicit verify-then-proceed steps — placing it at 'clear sequence with most checkpoints present; minor validation gaps'. | 4 / 5 |
Progressive Disclosure | Structure exists and the single bundle reference (references/request-schema.yaml, verified to exist) is well signaled with an explicit statement of when it matters ('Use ... when structured input, posture/category/amount enums, comparison inputs, or enrichment-mode normalization matters'). However, substantial content that plausibly belongs in a separate file is inlined — the entire Automation Offer Guard section including its multi-field prompt template (~25 lines) and the long output-format block — so the body carries material a lighter overview would offload. This matches 'some structure ... references present but ... content that should be separate is inline' better than the level-4 'most content is appropriately placed'. | 3 / 5 |
Total | 14 / 20 Passed |