Content
58%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 core QA workflow is excellent — well-sequenced phases with strong validation and feedback loops and concrete executable commands — but it is buried in a monolithic 1100+ line file where a third of the content is gstack framework boilerplate (persona, telemetry, prompting flows) and an inlined test-bootstrap sub-skill. Referenced detail files do not exist in the bundle, and a few placeholders ('$REPORT_DIR', Phase 8a) are not executable as written.
Suggestions
Move the framework boilerplate (Preamble, Voice, AskUserQuestion Format, Completeness Principle, Contributor Mode, Telemetry, Plan Status Footer) into a shared reference file or generated include, keeping SKILL.md focused on the QA workflow.
Extract the Test Framework Bootstrap (sections B2-B8) into its own reference file (e.g., references/test-bootstrap.md) and keep only a short trigger/decision pointer in the main body.
Define '$REPORT_DIR' explicitly (e.g., 'REPORT_DIR=.gstack/qa-reports') in Phase 1 and replace the Phase 8a comment hints with concrete Grep/Glob tool examples; also ensure 'qa/templates/qa-report-template.md' and 'qa/references/issue-taxonomy.md' actually exist in the bundle or inline their essential content.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Roughly the first third of the body (the ~45-line preamble script, 'Voice' persona, AskUserQuestion format, 'Boil the Lake' principle, Contributor Mode, Telemetry, Plan Status Footer) is framework boilerplate unrelated to QA testing, and the Test Framework Bootstrap (B2-B8) is itself a full sub-skill inlined here — several padded sections that cost tokens on every invocation. It is not 1 because the QA phases themselves are dense and concrete rather than explaining concepts Claude already knows, and not 3 because the volume of off-task padding is far more than 'some unnecessary explanation'. | 2 / 5 |
Actionability | Most guidance is copy-paste ready: exact browse commands ('$B goto', '$B snapshot -i -a -o', '$B console --errors'), commit message formats ('fix(qa): ISSUE-NNN — short description'), a weighted health-score rubric, and a runtime-to-framework table. It is not 5 because of real gaps: Phase 8a is comment-style pseudocode ('# Grep for error messages...') rather than commands, and '$REPORT_DIR' is used in Phases 3-5 without ever being defined. | 4 / 5 |
Workflow Clarity | Phases 0-11 are clearly sequenced with explicit validation checkpoints and feedback loops throughout: dirty-tree gate before starting, 'snapshot -D' before/after verification, re-test after every fix (8d), verified/best-effort/reverted classification (8e), 'git revert HEAD' on regression, a final QA pass that warns if the health score regresses (Phase 9), and self-regulation stops (WTF-likelihood, 50-fix cap). Batch fix operations have exactly the validation loops the rubric demands. | 5 / 5 |
Progressive Disclosure | The bundle contains no references/, scripts/, or assets/ directories, yet the body points to 'qa/templates/qa-report-template.md', 'qa/references/issue-taxonomy.md', and '$GSTACK_ROOT/ETHOS.md' — broken one-level references — while ~1120 lines of content that clearly belongs in separate files (preamble/telemetry boilerplate, persona, test-framework bootstrap) are inlined in SKILL.md. It is not 3 because the inlining is extensive and the referenced detail files are absent, going beyond 'could be better organized'; it is not 1 because the file does have consistent section headers and is navigable. | 2 / 5 |
Total | 13 / 20 Passed |