Content
75%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-organized, largely actionable skill body with a clear multi-step workflow and sensible reference splitting. The recurring gap is the orphaned bundled scanner script (scripts/ship_gate_scanner.py) that the body never invokes, plus light inlining of detection logic that already lives in the references.
Suggestions
Reference and prefer the bundled scanner in Step 2 (e.g., 'Run `python scripts/ship_gate_scanner.py .` for automated checks; fall back to the grep patterns in references/patterns.md only if the script is unavailable') to lift actionability and use the bundle you ship.
Move the inlined Step 1 stack-detection rule block into references/patterns.md (or a references/stack-detection.md) and summarize it in the body, reducing duplication with the patterns file and improving conciseness.
Add an explicit validation/verification checkpoint in Step 2 or Step 4 for triaging likely false positives before issuing a CRITICAL verdict, strengthening the feedback loop for this blocking gate.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Lean body that assumes Claude's competence (no basic concept explanations), but the inlined stack-detection rule block (Step 1) duplicates material that parallels references/patterns.md and the full report example is somewhat long — minor instances that could be trimmed. | 4 / 5 |
Actionability | Provides concrete, copy-pasteable detection rules ('next in dependencies -> Next.js') and points Claude to grep patterns in references/patterns.md, but never references the bundled scripts/ship_gate_scanner.py, which would be the most directly executable path. | 4 / 5 |
Workflow Clarity | Four clearly sequenced steps (Detect Stack -> Automated Checks -> Manual Confirmation -> Verdict) with progress reporting and a re-run feedback loop ('Fix critical items and re-run'), but verification of findings (false-positive handling) is only implicit and the script-backed check is not wired in. | 4 / 5 |
Progressive Disclosure | Good overview structure with one-level-deep, clearly signaled references to references/checks.md and references/patterns.md, but the bundled scripts/ship_gate_scanner.py is never referenced from the body and some stack-detection content is inlined rather than split out. | 4 / 5 |
Total | 16 / 20 Passed |