Content
73%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 an exceptionally actionable and well-gated state-machine orchestrator — deterministic sequencing, fail-closed handoff gates, and validation feedback loops are all present — but it pays for that with poor token efficiency: heavy rule duplication and long inline policy sections that its own context-loading philosophy says belong in conditional reference files. The bundle structure is strong, though the in-body file tree no longer matches the actual bundle contents.
Suggestions
State the Generate opt-in consent rule once in a canonical location (e.g., the HARD RULE paragraph) and have the state-machine table rows and deterministic steps reference it, removing the three-to-four near-duplicate formulations.
Move the corrupted-state reconstruction heuristics (State Validation rule 2) into references/shared/schema-phase-status.md and the Contextual offers mechanics (~45 lines) into a dedicated reference, keeping one-line pointers in SKILL.md.
Sync the 'Files in This Skill' tree with the actual bundle — add the missing discover-openrouter-api.md, discover-anthropic-api.md, discover-preview.md, design-refs/security.md, generate-artifacts-report.md, the additional shared/ files, and the vendored/clarify/ directory — and clarify the duplicate ai-gemini-to-bedrock.md copies in design-refs/ vs vendored/ai/.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | At 507 lines (~61KB) loaded on every turn, the body is noticeably verbose: the Generate opt-in consent rule is stated at least four times (state machine generate/decide-complete/legacy-generate rows, the 'Generate is opt-in (HARD RULE)' paragraph, and deterministic step 6), the workshop-resume rule appears in both the table and step 2, and BigQuery guidance is repeated across Philosophy, the conditional table, and Workflow step 10. Substantial policy detail that the skill's own Context Loading Rules would place in conditional references is inlined instead — the ~45-line Contextual offers section, the ~30-line corrupted-state reconstruction heuristics, and the full hybrid-stack warning prompt — matching anchor 2 ('noticeably verbose; several unnecessary... padded sections') rather than the mostly-efficient anchor 3. | 2 / 5 |
Actionability | Guidance is fully executable: copy-paste bash probes ('uv --version 2>/dev/null || echo "UV_MISSING"'), a literal .phase-status.json schema, exact error strings to output ('Unrecognized phase: [value]. Valid phases: ...'), a deterministic state-transition table with JSON-path conditions, exact file paths per phase, and precise accuracy ranges (±5-10% infra, ±15-25% Bedrock). It matches anchor 5 — concrete commands and specific examples covering the common cases — with no vague-only direction. | 5 / 5 |
Workflow Clarity | The six-phase process has an exact mandated sequence (Workflow Execution steps 1-10), explicit validation checkpoints (HANDOFF_OK/GATE_FAIL fail-closed gates, 'Confirm all required output files exist with correct schema', validate-artifacts.md, validate-migration-report.md with exit-code handling, POLICY_OK policy gate), and error-recovery feedback loops (GATE_FAIL handling, corrupted-state reconstruction with user confirmation, re-entry reset rules). This matches anchor 5's explicit-validation-plus-feedback-loops profile; validation for the batch artifact-generation operations is thoroughly present, so no cap applies. | 5 / 5 |
Progressive Disclosure | The structure is genuinely good — one orchestrator with per-phase reference files, an explicit conditional-loading table keyed to JSON conditions, a stated ~800-line per-phase budget, a design-refs index, and a full file tree — but scored against the actual bundle it has real gaps: the 'Files in This Skill' tree is out of sync with disk (omits discover-openrouter-api.md, discover-anthropic-api.md, discover-preview.md, design-refs/security.md, generate-artifacts-report.md, several shared/ files, and the whole vendored/clarify/ directory), cross-bundle paths (../knowledge-base-for-startups/..., ../contextual-offers-for-startups/..., $PLUGIN_ROOT/...) are unresolvable from this bundle, and ai-gemini-to-bedrock.md exists in both design-refs/ and vendored/ai/. This is anchor 4 ('good structure... minor organization gaps') rather than the clean easy-navigation anchor 5. | 4 / 5 |
Total | 16 / 20 Passed |