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.
A sophisticated, well-architected orchestration skill: explicit routing, gates, state schema, and validation checkpoints, with an excellent one-level-deep bundle layout. The costs are density and length — the gate-semantics and contextual-offers sections read as compressed rulebooks that could partly live in reference files, and cross-skill dependencies assume siblings that are not vendored here.
Suggestions
Tighten conciseness by moving the ~55-line Contextual offers closing step into its own reference file (e.g. references/decision-refs/contextual-offers.md), leaving SKILL.md with a short trigger rule and pointer.
Rewrite the Gate 2 paragraph as a small bulleted decision table (entry point × migration_plan state → Gate 2 offered or not) so its nested conditionals are scannable instead of a single tangled sentence.
Condense the $PLUGIN fallback definition to the two-line rule (try ${CLAUDE_PLUGIN_ROOT}, else skill-relative) and cut the explanatory prose about where files live, which the Files table already conveys.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense operational state-machine prose rather than padded tutorials — no explaining of concepts Claude already knows — but several sections could be tightened: the "$PLUGIN" fallback paragraph, the Gate 2 paragraph's nested conditionals ("only when … or … on build_deploy only … OR when …"), and the ~55-line "Contextual offers" section with its long nested parentheticals. This matches anchor 3 ("mostly efficient but includes some unnecessary explanation or could be tightened") better than anchor 4, since more than minor trimming is possible. | 3 / 5 |
Actionability | Guidance is highly concrete and executable for an orchestration skill: exact file paths ("begin at references/phases/intake/intake.md"), a checkable prerequisite command ("uv --version"), a full .phase-status.json example with status values, precise gate conditions keyed to named state fields, and a read-merge-write update discipline. It sits just under anchor 5 because several instructions resolve only by chaining into phase files (e.g. "generate.md Step 5.5", "migration-plan.md Step -1") and into sibling-skill files outside this bundle, so the SKILL.md alone is not fully self-executing. | 4 / 5 |
Workflow Clarity | The backbone sequence is explicit (intake → … → complete), entry-point routing enumerates each path, and validation is prominent: the phase gate ("Do NOT load design.md … unless … phases.clarify == 'completed' AND phases.confirm == 'completed'"), the recommendation_reviewed precondition for all gates, schema/run_id validation of verification evidence, and resumability via persisting poc=in_progress before loading. It misses anchor 5 because the Gate 2 offer conditions in particular are a tangled multi-clause paragraph that a reader must unpack, and some checkpoint details live implicitly in the referenced phase files. | 4 / 5 |
Progressive Disclosure | Bundle structure is exemplary: a Files table mapping every reference to its purpose, phases split one per directory, per-topic decision-refs, per-runtime JSON profiles, and a vendored INTERPRETER.md as the single execution contract — and spot checks confirm all listed paths (phases/, decision-refs/, runtimes/, vendored/dsl/INTERPRETER.md, scripts/scoring.py, scripts/schemas/seed.json) exist. It falls short of anchor 5 because the 206-line body is itself a dense controller (the Contextual-offers closing step in particular is inline procedural content that belongs in its own reference file), and it depends on sibling-bundle files (../knowledge-base-for-startups/references/offers.md, ../contextual-offers-for-startups/SKILL.md, ../architect-for-startups/…) that are not part of this skill's bundle. | 4 / 5 |
Total | 15 / 20 Passed |