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.
The body is a well-organized, brief, instruction-only skill with a clear sequenced workflow including a verification gate and an explicit fallback path. Its main gap is actionability: the operational steps name intentions (detect, look for) without any concrete commands, config locations, or example invocations that would make the guidance directly executable.
Suggestions
Add concrete detection mechanics, e.g., an actual check such as `command -v code-notify` or the specific hook/config file paths to inspect per tool (Codex, Cursor, Claude, Gemini).
Include one copy-paste-ready example of wiring the notification (e.g., a sample hook config or end-of-task invocation) so the 'use it for end-of-task notifications' step is executable rather than descriptive.
Consolidate the verify-before-relying guidance, which currently appears in Cross-Platform Rules, Workflow step 2, and Output Guidance, into one place to tighten conciseness.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~30-line body is lean with no explanation of concepts Claude already knows and no padding, e.g., "Treat code-notify as an optional local dependency managed outside this repository" and "If missing, tell the user the integration is unavailable in the current shell and continue without blocking the main task". It misses 5 because the verify-before-relying guidance is repeated across three sections ("Verify the installed command, wrapper, hook, or config before using it", workflow step 2, and "Mention that code-notify is available only after verifying") and meta-advice like "Prefer concise setup notes over tool-specific prose duplication" could be trimmed, matching 'Efficient; minor instances of over-explanation that could be trimmed'. | 4 / 5 |
Actionability | The guidance is directive and conditionally complete ("use it for end-of-task or attention-needed notifications only", fallback behavior in step 4), but every operational step is an abstract intent with no concrete mechanism: "Detect whether the current tool already has a notification hook or wrapper configured" and "Look for a local code-notify command, alias, script, or documented install path" give no commands, config file paths, or example invocations. This matches 'Some concrete guidance but incomplete; missing key details' — above 2 because the workflow does specify what to do in each outcome, below 4 because nothing in the body is directly executable. | 3 / 5 |
Workflow Clarity | The 4-step workflow is a clear, ordered sequence with an explicit presence-verification gate and fallback branch: detect existing hook, "Look for a local code-notify command... before assuming invocation details", use it if present, otherwise "tell the user the integration is unavailable... and continue without blocking". This matches 'Clear sequence with most checkpoints present; minor validation gaps' — it does not reach 5 because the detection steps have no explicit method (no command or file to check), and exceeds 3 because the verify-before-use checkpoint and failure path are explicit rather than implicit. | 4 / 5 |
Progressive Disclosure | The skill is under 50 lines, self-contained, and needs no external references; the body is organized into five clear, well-scoped sections (Goal, Use This Skill When, Cross-Platform Rules, Workflow, Output Guidance) with no inlined material that belongs in separate files. Per the rubric's simple-skill guideline, this earns 5 with just well-organized sections and no bundle files to misplace. | 5 / 5 |
Total | 16 / 20 Passed |