ship
Devloop commit and PR creation with validation for completed plan work
What this skill does
# Ship - Commit and PR
Validate and ship changes with branch-aware workflow. **You do the work directly.**
**Bash hygiene**: prefer quiet flags to minimize output (`npm install --silent`, `git status -sb`, pipe long output through `| tail -n 20`).
## Phase 1: Pre-flight Check
```bash
git status && git diff --stat && git branch --show-current
```
Read `.devloop/local.md` for preferences: `git.pr-on-complete`, `commits.style`.
| Branch | Flow |
|--------|------|
| main/master | Direct commit (caution) |
| feat/*, fix/* | Feature branch → PR |
| Other | Ask user |
```yaml
AskUserQuestion:
questions:
- question: "How would you like to ship?"
header: "Mode"
multiSelect: false
options:
- label: "Full validation"
description: "Run tests, review, then commit/PR"
- label: "Quick commit"
description: "Skip validation, just commit"
- label: "PR only"
description: "Changes already committed, create PR"
- label: "Atomic commits"
description: "One commit per completed plan task"
```
## Phase 2: Validation (if Full mode)
**Run tests:** Detect from package.json/go.mod/requirements.txt/pom.xml.
**Code review:** If `review.before-commit: always`, use `devloop:engineer` in reviewer mode.
**Safety:** No secrets, no debug code, correct branch.
## Phase 3: Smart Commit
If plan exists:
1. Find tasks marked `[x]` since last commit
2. Offer: single commit vs atomic commits
**Conventional style:**
```
<type>(<scope>): <description>
- Task 1.1: Description
- Task 1.2: Description
```
| Task Contains | Type |
|---------------|------|
| Create, Add, Implement | feat |
| Fix, Resolve | fix |
| Refactor, Clean | refactor |
| Test, Spec | test |
| Doc, README | docs |
| Other | chore |
**Issue closing:** If plan has `issue:` in frontmatter:
- `github.auto_close: always` → Auto-add `Closes #N`
- `github.auto_close: ask` → Prompt
- `github.auto_close: never` → Skip
```bash
git add -A && git commit -m "$(cat <<'EOF'
feat(scope): description
- Task 1.1: Description
- Task 1.2: Description
Closes #42
EOF
)"
```
## Phase 4: PR Creation
| Condition | Action |
|-----------|--------|
| On main/master | Skip |
| pr-on-complete: never | Skip |
| pr-on-complete: always | Auto-create |
| pr-on-complete: ask | Ask user |
Generate PR from plan:
```markdown
## Summary
[Plan title and overview]
## Changes
- [x] Task 1.1: Description
- [x] Task 1.2: Description
## Testing
- [ ] Tests pass locally
- [ ] Code reviewed
```
```bash
gh pr create --title "[Plan Title]" --body "[Description]"
```
## Phase 5: Post-Ship
1. Add PR link to plan header
2. Add Progress Log entry
3. If all tasks complete, update status to `Complete`
```bash
"${CLAUDE_PLUGIN_ROOT}/scripts/check-plan-complete.sh" .devloop/plan.md
```
**If complete:**
```yaml
AskUserQuestion:
questions:
- question: "All tasks done! What next?"
header: "Next"
multiSelect: false
options:
- label: "Archive plan"
description: "Move to archive, start fresh"
- label: "Wait for review"
description: "PR created, wait for feedback"
- label: "Done"
description: "Keep plan for reference"
```
Archive: `"${CLAUDE_PLUGIN_ROOT}/scripts/archive-plan.sh" .devloop/plan.md`
**If not complete:** Offer continue/wait/break.
## Safety Checklist
Before commit: No secrets, no debug, correct branch, tests pass.
Before PR: Good messages, PR description, correct base branch.
## Quick Reference
| Mode | Flow |
|------|------|
| Full validation | Test → Review → Commit → PR |
| Quick commit | Commit only |
| PR only | Create PR from commits |
| Atomic commits | One commit per task |
Related in General
modeling-omnistudio-epc-catalog
IncludedSalesforce Industries CME EPC product-modeling skill for Product2-based catalog creation. Use when creating EPC products, configuring product attributes, building offer bundles with Product Child Items, or reviewing EPC DataPack JSON metadata for product catalog changes. TRIGGER when: user creates or updates Product2 EPC records, AttributeAssignment payloads, AttributeMetadata/AttributeDefaultValues, Offer bundles, or ProductChildItem relationships. DO NOT TRIGGER when: designing OmniScripts/FlexCards/Integration Procedures (use building-omnistudio-omniscript, building-omnistudio-flexcard, or building-omnistudio-integration-procedure), implementing Apex business logic (use generating-apex), or troubleshooting deployment pipelines (use deploying-metadata).
relationship-science-coach
IncludedUse this skill for direct, practical adult relationship coaching: couples conflict, repair, trust, marriage, dating, flirting, attachment patterns, emotional connection, sex, desire differences, eroticism, kink negotiation, affection, love languages, breakups, and long-term passion. Draw on Gottman, EFT and Hold Me Tight, attachment science, modern sex research, Perel, Nagoski, Kerner, Schnarch, Love and Stosny, and flexible love-language tools. Be concrete and low-hedge. Redirect only for imminent danger, abuse, coercive control, minors, non-consent, self-harm, stalking, or medical/legal/psychiatric decisions.
building-sf-integrations
IncludedSalesforce integration architecture and runtime plumbing with 120-point scoring. Use this skill to set up Named Credentials, External Credentials, External Services, REST/SOAP callout patterns, Platform Events, and Change Data Capture. TRIGGER when: user sets up Named Credentials, External Services, REST/SOAP callouts, Platform Events, CDC, or touches .namedCredential-meta.xml files. DO NOT TRIGGER when: Connected App/OAuth config (use configuring-connected-apps), Apex-only logic (use generating-apex), or data import/export (use handling-sf-data).
venue-templates
IncludedAccess comprehensive LaTeX templates, formatting requirements, and submission guidelines for major scientific publication venues (Nature, Science, PLOS, IEEE, ACM), academic conferences (NeurIPS, ICML, CVPR, CHI), research posters, and grant proposals (NSF, NIH, DOE, DARPA). This skill should be used when preparing manuscripts for journal submission, conference papers, research posters, or grant proposals and need venue-specific formatting requirements and templates.
let-fate-decide
IncludedDraws the 12 Houses of the Zodiac Tarot spread to inject entropy into planning when prompts are vague, ambiguous, or casually delegated. Interprets the spread to guide next steps. Use when the user says 'let fate decide', 'YOLO', 'whatever', 'idk', or other nonchalant phrases, makes Yu-Gi-Oh references, or when you are about to arbitrarily pick between multiple reasonable approaches. Prefer over ask-questions-if-underspecified when the user's tone is casual or playful rather than precision-seeking.
net-ops
IncludedCross-platform network troubleshooting (Windows, macOS, Linux) via local or remote shell. Use for: DNS broken, can't resolve hostnames, nslookup/dig works but apps fail, NRPT, WFP, scutil, /etc/resolver, systemd-resolved, /etc/resolv.conf, NetworkManager, VPN DNS leak residue (ProtonVPN/Mullvad/WireGuard/AnyConnect), AV/firewall blocking DNS or DoH, Tailscale DNS interaction, intermittent connectivity, remote diagnostics over SSH.