Content
88%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 thorough, highly actionable skill with excellent workflow sequencing and validation checkpoints for a security-sensitive task. The main weaknesses are repeated security mantras across sections and a large inlined reference catalog that could be split out for better progressive disclosure.
Suggestions
Consolidate the repeated path-traversal / untrusted-content rules (absolute paths, '..' traversal, '$'-prefixed env vars) into one stated rule in Extension Shape and reference it from Local Test Flow and Before Handoff to trim repetition.
Move the detailed Extension Shape field catalog into a references/ file (e.g. EXTENSION_SHAPE.md) and keep a short summary inline with a clearly signaled link, improving progressive disclosure for the ~350-line body.
Extract the Local Test Flow audit checklist into a concise bullet list at the top of that section so the per-field details read as a reference rather than narrative prose.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and mostly efficient with copy-paste commands, but the same path-traversal and untrusted-content mantras (reject absolute paths, '..' traversal, '$'-prefixed env vars) recur across Extension Shape, Local Test Flow, and Before Handoff, adding tokens that could be consolidated. | 4 / 5 |
Actionability | Fully executable guidance throughout: concrete commands (`qwen extensions new`, `printf 'y\n' | qwen extensions link`, `npm install --ignore-scripts`, `npm run build`, `node -e "JSON.parse(...)"`) and concrete field examples cover the common cases copy-paste ready. | 5 / 5 |
Workflow Clarity | Clear sequenced workflow with explicit validation checkpoints and feedback loops ('If any command fails, stop and report', 'fix the issue and re-check before proceeding'), a Linking Approval Procedure with approval gates, and a Before Handoff checklist for a complex, security-sensitive process. | 5 / 5 |
Progressive Disclosure | Well-organized into clearly signaled sections (Workflow, Template Selection, Extension Shape, Local Test Flow, After Linking, Iterating, Before Handoff) with easy header navigation, but the long inlined Extension Shape field reference (~40 lines of catalog) could live in a separate file given the body is ~350 lines. | 4 / 5 |
Total | 18 / 20 Passed |