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.
The content is well-structured, executable, and uses progressive disclosure effectively by offloading detail to a cloned SDK repo via clear tables. Its main weakness is the absence of an explicit validation feedback loop in the build/test workflow.
Suggestions
Add an explicit validate→fix→retry loop after building (e.g., 'If basic-host fails to load the app, check the console errors, fix the registration/CSP config, and rebuild') to strengthen workflow_clarity.
Tighten the 'Core Concept: Tool + Resource' section to a one-line definition plus the data-flow line, trimming the explanatory framing.
For Advanced Patterns, add a single short inline code snippet for the highest-value pattern (e.g., polling or streaming input) rather than only pointing to docs/patterns.md.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and task-oriented with tables and code blocks rather than prose; a small amount of conceptual framing (the Tool+Resource core concept) is necessary context for a niche SDK but could be marginally tighter, so it sits just below the lean/efficient anchor. | 4 / 5 |
Actionability | Provides concrete, executable guidance (git clone, npm install commands, handler-registration TypeScript, sendLog and testing commands); the Advanced Patterns section is pointer-only rather than code, leaving minor gaps. | 4 / 5 |
Workflow Clarity | A clear sequence exists (clone repo → register tool/resource → register handlers before connect → build → test with basic-host) with checkpoints like 'Register ALL handlers BEFORE app.connect()' and a testing section, though an explicit validate→fix→retry feedback loop is only weakly present. | 4 / 5 |
Progressive Disclosure | SKILL.md is an overview pointing to well-signaled, one-level-deep references (template/API tables mapping files in the cloned /tmp/mcp-ext-apps repo, docs/patterns.md); no local bundle files exist, and organization is good with minor gaps in signaling some reference paths. | 4 / 5 |
Total | 16 / 20 Passed |