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 dense, well-structured pattern guide: executable code for every pattern, exact state shapes, explicit single-channel and failure-mode guidance, and useful Do/Don't checklists. Weaknesses are mild — some narrative and internals-level prose could be trimmed or moved to references, and a couple of domain-specific functions in examples are unshown.
Suggestions
Trim the 'Why' section to a single sentence and split the long single-paragraph sections (Settings page, selection state) into tighter bullet-style guidance to save tokens.
Move the Jitter Prevention internals (X-Request-Source header flow, requestSource filtering mechanics) into a references file, keeping only the useDbSync({ ignoreSource: TAB_ID }) pattern inline.
Show the signatures of fetchEmailList/fetchThread in the view-screen example or annotate them as domain-provided placeholders so the example is fully copy-paste runnable.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body assumes Claude's competence — no padding explaining basic concepts, and comments carry non-obvious rationale (e.g. "a hand-rolled random id silently breaks selection sync for hidden and background tabs"). Minor trimmable material remains: the 'Why' narrative ("feels like a collaborator rather than a disconnected chatbot") and long single-paragraph sections (Settings page, selection state) could be tightened, fitting anchor 4 rather than the fully lean anchor 5. | 4 / 5 |
Actionability | Four complete, copy-paste-ready TypeScript blocks (useNavigationState, readAppState, the full view-screen script, writeAppState navigate, useDbSync) plus exact state shapes and write payloads cover the common cases. Minor gaps keep it below anchor 5: domain fetchers (fetchEmailList, fetchThread) are referenced but unshown, and getCommandPath: (command: any) is loose. | 4 / 5 |
Workflow Clarity | Patterns are sequenced coherently (UI writes navigation → agent reads before acting → view-screen snapshot → one-shot navigate command), with explicit failure-mode reasoning ("competing consumers can make the browser URL change before React Router commits") and Do/Don't checklists. Not a numbered multi-step workflow with validation checkpoints, so anchor 5 doesn't fit, but the operations are not destructive or batch, so the workflow-clarity cap does not apply. | 4 / 5 |
Progressive Disclosure | No bundle files exist; the skill is standalone with clean one-level section headers and a clear 'Related Skills' index pointing to sibling skills. Some reference-grade internals (Jitter Prevention sync mechanics, Settings redirect behavior) are inlined where a separate reference file could carry them, which matches anchor 4's good structure with minor organization gaps. | 4 / 5 |
Total | 16 / 20 Passed |