Content
96%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.
An exceptionally tight, operational skill body: executable commands, explicit validation with feedback loops, and careful guardrails ('Never do these', --repo discipline, merge-vs-rebase policy) with no wasted tokens. The only structural note is that the dual-procedure content could be split into separate reference files if a bundle were introduced.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense, imperative, and free of concept explanation or padding — even implementation details like 'npm exec qm does not work in a source checkout; the workspace symlink points at cli/' are non-obvious operational facts Claude could not infer. Every token earns its place, matching the lean anchor. | 5 / 5 |
Actionability | Commands are copy-paste executable with clearly marked parameters: 'npm install --save-exact @yc-software/qm@<version>', 'node cli/bin/qm.ts check --config deploy/layers/<org>/qm.config.jsonc', full 'gh pr create --repo <source-fork> --base main ...' blocks, and the complete merge sequence. Not a 4 because no step relies on pseudocode or unstated detail. | 5 / 5 |
Workflow Clarity | Clear sequence (identify checkout → merge → resolve conflicts → verify → open PR) with explicit validation checkpoints ('check', 'doctor', 'plan', typecheck, lint, per-layer contract-major check) and error-recovery branches ('Already up to date' → report instead of an empty PR; unsupported contract major → adapt as part of the sync). Destructive risks (push, merge, landing) all have pre-flight validation. | 5 / 5 |
Progressive Disclosure | Sections are well-organized and appropriately scoped for a single-file skill with no bundle files provided, matching the good-structure anchor. It is not a 5 because the two distinct procedures (package deployment vs. source fork) are interleaved in one body, and pointers such as the README, AGENTS.md, and the 'upstream-pr' skill are external mentions rather than clearly signaled bundle references. | 4 / 5 |
Total | 19 / 20 Passed |