Content
67%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 body is a concrete, well-sequenced operational guide with real git commands, safety steps, and templates. Its main weaknesses are redundant restatement of the changeset rule, an internal inconsistency between the changeset instructions and the scratchpad template, and unresolved placeholders/dangling references.
Suggestions
State the 'do not keep the original changeset' rule once (in the Changesets section) and drop its repetition in numbered step 1 and Common failure modes.
Add the changeset line to the scratchpad template itself, since the text instructs 'Add changeset creation to the scratchpad template' but the template omits it.
Resolve the '<base>' placeholder with a concrete way to determine the base branch, and replace the dangling '.mastracode/commands/changeset.md' path with the actual CLI usage inline or a verified link.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly imperative and dense, but the 'do not keep the original changeset' rule is stated three times (Changesets intro, numbered step 1, and Common failure modes), which is noticeable redundancy rather than a minor trim. It fits the 'mostly efficient but could be tightened' anchor better than the score-4 'minor instances' anchor. | 3 / 5 |
Actionability | Concrete, executable commands appear throughout ('git branch backup/original-large-pr', 'git checkout backup/original-large-pr -- path/to/file', 'git restore -p --source backup/original-large-pr -- path/to/file', 'git range-diff', 'pnpm changeset -s -m "..." pkg-name') plus ready-to-use templates. It falls short of 5 because the '<base>' placeholder is never resolved and '.mastracode/commands/changeset.md' is a dangling path reference. | 4 / 5 |
Workflow Clarity | The 8-step required workflow is clearly sequenced, snapshots the original branch before touching history, and includes an explicit per-PR verification step with a scratchpad checklist. It is not 5 because step 7 (rebasing dependent PRs) never explicitly instructs re-running verification after the rebase, and the changeset instruction to extend the scratchpad template is not reflected in the template shown. | 4 / 5 |
Progressive Disclosure | Sections are well organized for a single-file workflow skill with templates placed inline where they belong, but the repo-specific changeset procedure (~24 lines) is inlined and the one external reference ('.mastracode/commands/changeset.md') is a bare, unverifiable relative path rather than a clearly signaled link. This matches 'good structure; minor organization gaps' rather than the clearly-signaled-references anchor at 5. | 4 / 5 |
Total | 15 / 20 Passed |