Cursor rule

1---
2description: Run the pre-launch checklist via parallel fan-out to specialist
3  personas, then synthesize a go/no-go decision
4alwaysApply: true
5---
6 
7Invoke the agent-skills:shipping-and-launch skill.
8 
9`/ship` is a **fan-out orchestrator**. It runs three specialist personas in parallel against the current change, then merges their reports into a single go/no-go decision with a rollback plan. The personas operate independently — no shared state, no ordering — which is what makes parallel execution safe and useful here.
10 
11## Phase A — Parallel fan-out
12 
13Spawn three subagents concurrently using the Agent tool. **Issue all three Agent tool calls in a single assistant turn so they execute in parallel** — sequential calls defeat the purpose of this command.
14 
15In Claude Code, each call passes `subagent_type` matching the persona's `name` field:
16 
171. **`code-reviewer`** — Run a five-axis review (correctness, readability, architecture, security, performance) on the staged changes or recent commits. Output the standard review template.
182. **`security-auditor`** — Run a vulnerability and threat-model pass. Check OWASP Top 10, secrets handling, auth/authz, dependency CVEs. Output the standard audit report.
193. **`test-engineer`** — Analyze test coverage for the change. Identify gaps in happy path, edge cases, error paths, and concurrency scenarios. Output the standard coverage analysis.
20 
21In other harnesses without an Agent tool, invoke each persona's system prompt sequentially and treat their outputs as if returned in parallel — the merge phase still works.
22 
23Constraints (from Claude Code's subagent model):
24- Subagents cannot spawn other subagents — do not let one persona delegate to another.
25- Each subagent gets its own context window and returns only its report to this main session.
26- If you need teammates that talk to each other instead of just reporting back, use Claude Code Agent Teams and reference these personas as teammate types (see `references/orchestration-patterns.md`).
27 
28**Persona resolution.** If you've defined your own `code-reviewer`, `security-auditor`, or `test-engineer` in `.claude/agents/` or `~/.claude/agents/`, those take precedence over this plugin's versions — `/ship` picks up your customizations automatically. This is intentional: plugin subagents sit at the bottom of Claude Code's scope priority table, so user-level definitions win by design.
29 
30## Phase B — Merge in main context
31 
32Once all three reports are back, the main agent (not a sub-persona) synthesizes them:
33 
341. **Code Quality** — Aggregate Critical/Important findings from `code-reviewer` and any failing tests, lint, or build output. Resolve duplicates between reviewers.
352. **Security** — Promote any Critical/High `security-auditor` findings to launch blockers. Cross-reference with `code-reviewer`'s security axis.
363. **Performance** — Pull from `code-reviewer`'s performance axis; cross-check Core Web Vitals if applicable.
374. **Accessibility** — Verify keyboard nav, screen reader support, contrast (not covered by the three personas — handle directly here, or invoke the accessibility checklist).
385. **Infrastructure** — Env vars, migrations, monitoring, feature flags. Verify directly.
396. **Documentation** — README, ADRs, changelog. Verify directly.
40 
41## Phase C — Decision and rollback
42 
43Produce a single output:
44 
45```markdown
46## Ship Decision: GO | NO-GO
47 
48### Blockers (must fix before ship)
49- [Source persona: Critical finding + file:line]
50 
51### Recommended fixes (should fix before ship)
52- [Source persona: Important finding + file:line]
53 
54### Acknowledged risks (shipping anyway)
55- [Risk + mitigation]
56 
57### Rollback plan
58- Trigger conditions: [what signals would prompt rollback]
59- Rollback procedure: [exact steps]
60- Recovery time objective: [target]
61 
62### Specialist reports (full)
63- [code-reviewer report]
64- [security-auditor report]
65- [test-engineer report]
66```
67 
68## Rules
69 
701. The three Phase A personas run in parallel — never sequentially.
712. Personas do not call each other. The main agent merges in Phase B.
723. The rollback plan is mandatory before any GO decision.
734. If any persona returns a Critical finding, the default verdict is NO-GO unless the user explicitly accepts the risk.
745. **Skip the fan-out only if all of the following are true:** the change touches 2 files or fewer, the diff is under 50 lines, and it does not touch auth, payments, data access, or config/env. Otherwise, default to fan-out. `/ship` is designed for production-bound changes — when the blast radius is non-trivial, run the parallel review even if the diff looks small.
75