Content
77%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 highly actionable and the workflow sequencing with validation checkpoints is excellent — concrete commands, stop-on-failure rules, and fix-and-recheck loops throughout. The weaknesses are token efficiency (security rules about path traversal, symlinks, and untrusted content are restated across multiple sections) and structure (a 340-line single file with an inline manifest-field reference that should be split into a references/ file).
Suggestions
Consolidate the repeated path-traversal / symlink / realpath-validation rules (currently restated in Extension Shape, Local Test Flow, and Before Handoff) into one 'Path Safety' section that each other section references in a single line.
Move the Extension Shape manifest-field reference (~90 lines covering `name` through `lspServers` and path-variable hydration) into a `references/manifest-fields.md` file, keeping only the field list and highest-risk warnings in SKILL.md.
Trim the duplicated untrusted-content guidance between Workflow steps 5-6 and the Local Test Flow preamble into a single stated posture with section-specific deltas.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly efficient domain knowledge Claude would not know (manifest field semantics, path-variable hydration rules), but the same security rules are restated repeatedly across sections: untrusted-content handling appears in Workflow steps 5-6, Local Test Flow, and Before Handoff, and 'reject absolute paths, `..` traversal, and symlinks that escape the root' is repeated four-plus times (Extension Shape for `contextFileName`/`hooks`/`lspServers`, Local Test Flow, Before Handoff). This fits 'mostly efficient but includes some unnecessary explanation or could be tightened' rather than the 4-anchor's 'minor instances'. | 3 / 5 |
Actionability | Every step carries a copy-paste-ready command: `qwen extensions new "$extension_path" "$template"`, `printf 'y\n' | qwen extensions link "$extension_path"`, `cd -- "$extension_path" && npm install --ignore-scripts`, `npm run build`, and the `node -e "JSON.parse(...)"` validation one-liner, including correct `--` argument guarding. This is fully executable guidance covering the common cases, matching the top anchor. | 5 / 5 |
Workflow Clarity | The main workflow is a numbered 11-step sequence with explicit validation checkpoints (Before Handoff checklist, After Linking verification, Linking Approval Procedure), explicit stop-on-failure rules ("If any command in the workflow fails, stop and report the error to the user... Do not proceed"), and fix-and-re-check feedback loops ("If any check fails, fix the issue and re-check before proceeding"). This matches the anchor for clear sequence with explicit validation, error-recovery loops, and a checklist. | 5 / 5 |
Progressive Disclosure | The file has clear section headers (Workflow, Template Selection, Extension Shape, Local Test Flow, Before Handoff) but is a single ~340-line monolith with no `references/`, `scripts/`, or `assets/` bundle and no external file pointers. The Extension Shape section is effectively a 90-line manifest-field API reference that belongs in a one-level-deep reference file. This fits 'Some structure but could be better organized; content that should be separate is inline' better than the 4-anchor, which requires most content to be appropriately placed in separate files. | 3 / 5 |
Total | 16 / 20 Passed |