Content
81%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 command-accurate, highly actionable operational guide with a well-sequenced preview-first workflow and explicit fail-fast validation loops. Its main weakness is redundancy: Avoid, Anti-Patterns, Constraints, Execution Boundaries, and Gotchas repeatedly restate the same rules (dry-run first, init --update warnings, environment.toml ownership, manual workflow deletion), inflating token cost.
Suggestions
Consolidate the overlapping rule sections (Avoid, Anti-Patterns, Constraints, Execution Boundaries, Gotchas) into one section — dry-run-before-mutation, the 'init --update is not the routine upgrade lane' warning, the environment.toml ownership rule, and the manual-workflow-deletion warning each appear 3-4 times across the body.
Move the CI-provider ownership matrix (CircleCI/CodeRabbit/Semgrep/GitHub Actions) and per-repo learnings detail out of the body and into references/contract.yaml, keeping SKILL.md to a pointer to reduce inline contract detail.
State each harness command caveat once (e.g., 'harness --help is focused first-contact help, not the full catalog' appears in both Current Command Shape and Gotchas) and cross-reference it from the other section.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes Claude's competence (no concept explanations), but several sections restate the same rules: dry-run-before-mutation appears in Philosophy, Current Project Contracts, Workflow, Constraints, and Execution Boundaries; the "init --update is not the routine upgrade lane" warning appears in Current Project Contracts, Gotchas, and Anti-Patterns; the environment.toml ownership rule and the ci-migrate/manual-deletion warning each appear three times. This matches 'mostly efficient but includes some unnecessary explanation or could be tightened' rather than the score-4 anchor, since whole duplicate sections could be consolidated. | 3 / 5 |
Actionability | Commands are copy-paste ready and cover the common cases with explicit variants: "harness next --json", "pnpm exec tsx src/cli.ts next --json" for the source repo, "harness ci-migrate prepare --provider circleci --dry-run", and gate commands like "pnpm check" / "pnpm test:deep". This matches the fully-executable anchor with common cases covered, rather than score 4 where concrete commands have minor gaps. | 5 / 5 |
Workflow Clarity | The 7-step Workflow is clearly sequenced with preview-before-mutation (step 4), applied-lane execution (step 5), and gates before reporting (step 6), plus an explicit feedback loop in Validation: "stop at first blocking gate, fix it, and rerun the exact failed check". Because validation and rollback (snapshot-backed ci-migrate abort) are present for the destructive/CI-migration lane, the destructive-cap rule does not apply and the top anchor fits better than the minor-gaps anchor. | 5 / 5 |
Progressive Disclosure | The Progressive Disclosure section signals real one-level-deep bundle files with purpose ("references/contract.yaml for the machine-readable contract", "references/evals.yaml for benchmark and quality gates", "references/task-profile.json for evaluator thresholds" — all verified present). It stays at score 4 rather than 5 because contract detail like the CI-provider ownership matrix and per-repo learnings is inlined in the body instead of a reference, and cross-skill "Infrastructure/references/..." paths cannot be verified within this bundle, leaving minor organization gaps. | 4 / 5 |
Total | 17 / 20 Passed |