Content
63%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 well-sequenced, highly actionable skill body whose verbatim templates and guardrails make it executable in practice, held back by monolithic inline seed exemplars (three full language companions plus a machine contract) that belong in reference files and by prose that could be tightened considerably without losing information.
Suggestions
Move the seed companions (A/B en, zh-Hans, zh-Hant) and the document-language contract JSON into references/ files (e.g. references/seeds.md, references/language-contract.md), keeping SKILL.md as an overview with one-level-deep, clearly signaled links.
Provide the shipyard-audit.mjs seed content (or a starter implementation in scripts/) since the skill instructs running it without ever defining what to seed into it.
Tighten the longest compound sentences (the legacy CONTEXT.md migration rules and CLAUDE.md-rewrite paragraphs) into short checkable rules, and state the first-use trigger once per surface instead of repeating it in both the scaffold table and the governance-loop section.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is information-dense with no explanations of concepts Claude already knows, but at ~365 lines it contains long compound sentences (e.g. the legacy CONTEXT.md migration paragraph), repeated first-use-trigger statements across sections 3 and 4, and three full inline language companions of which any single invocation renders at most one. | 3 / 5 |
Actionability | Guidance is mostly executable: verbatim seed templates with exact filenames, an ordered language-resolution procedure, and 'node scripts/shipyard-audit.mjs [repoRoot]' with documented exit codes. Minor gaps: the audit script's content is never provided (it must be authored at first use), and the guardrail hook command is a '<confirm-before-destructive-git>' placeholder. | 4 / 5 |
Workflow Clarity | A clear five-step sequence (Detect → Resolve language → Scaffold → Wire governance loop → Report) with checkpoints such as 'Report the map first, then act', 'stop before writing localized artifacts', and --check's exit-code contract; migration renames carry guardrails (git mv, byte-for-byte preservation, never-clobber). Not 5 because there is no explicit post-scaffold verification step. | 4 / 5 |
Progressive Disclosure | Section structure and navigation are good (When to Use, When Not to Use, numbered workflow, --check mode), but there are no bundle files at all: the document-language contract JSON and the three seed companions clearly belong in references/ files, leaving ~365 lines monolithic where a well-signaled one-level reference split is warranted. | 3 / 5 |
Total | 14 / 20 Passed |