Content
66%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 skill is exceptionally actionable and workflow-clear with strong validation gates, but it suffers from severe verbosity and a monolithic structure that should be split into reference files. Both conciseness and progressive disclosure are weak points.
Suggestions
Extract the durable policy list, the dashboard/apply/review command specs, and the decision-classification matrix into separate reference files (e.g. references/policy.md, references/dashboard.md) and link to them one level deep from SKILL.md to cut the body to an overview.
De-duplicate rules repeated across command sections (e.g. 'must not patch apps/www', 'do not change lastSyncedCommit/lastPlannedCommit/lastPlan/partialSyncs') by stating them once in a shared constraints section and referencing it.
Remove boilerplate restating what Claude already knows (e.g. explanations of git diff/numstat behavior, status.json field meanings repeated per command) and keep only the sync-specific contract.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | At ~1200 lines the body is heavily padded with repeated restatements of the same rules (e.g. micro-overlap conditions, 'must not patch apps/www' appears across status/apply/dashboard/review/planning), definitions Claude could derive, and verbose enumeration of every state/edge — clearly more than the rubric's mid-point of verbosity. | 2 / 5 |
Actionability | Provides copy-paste-ready bash and node snippets (status checks, ref resolution, run-dir creation, inventory diffs, check-complete.mjs invocation) plus fully specified plan/output templates and decision-state contracts, covering the common cases concretely. | 5 / 5 |
Workflow Clarity | The flow is explicitly sequenced as numbered phases (1 Establish refs → 2 Run dir → 3 Classify → 4 Plan → 5 Stop for review → 6 Delegate via task → 7 Status updates) with validation checkpoints (merge-base ancestry proof, check-complete.mjs, focused verification, stop-before-implement boundary) and explicit error-recovery (blocked-ref handling, re-validate on failure). | 5 / 5 |
Progressive Disclosure | No bundle files exist (references/, scripts/, assets/ are absent) yet the body inlines an enormous monolithic policy/spec that clearly belongs in separate reference files (durable policy, dashboard spec, decision matrix), and references to external docs are buried in flat lists rather than clearly signaled one-level-deep navigation. | 2 / 5 |
Total | 14 / 20 Passed |