release-patterns

# Release Patterns Skill

## Purpose

Ensure consistent PR creation, CI/CD validation, and release coordination following rebase-first workflow.

## When This Skill Applies

Invoke this skill when:

- Creating pull requests
- Running pre-PR validation (`yarn ci:validate`)
- Checking CI/CD status
- Coordinating merge timing
- Verifying rebase status

## Stop-the-Line Conditions

### FORBIDDEN Patterns

```bash
# FORBIDDEN: Missing ticket reference
gh pr create --title "feat: add feature"  # Missing [{TICKET_PREFIX}-XXX]

# FORBIDDEN: Using squash/merge commits
gh pr merge --squash  # Breaks linear history
gh pr merge --merge   # Creates merge commit

# FORBIDDEN: Skipping CI validation
git push origin feature  # Without yarn ci:validate first

# FORBIDDEN: Pushing without rebase
git push origin feature  # When branch is behind dev
```

### CORRECT Patterns

```bash
# CORRECT: Ticket reference in title
gh pr create --title "feat(scope): description [{TICKET_PREFIX}-XXX]"

# CORRECT: Rebase merge only
gh pr merge --rebase --delete-branch

# CORRECT: CI validation before push
yarn ci:validate && git push --force-with-lease

# CORRECT: Always rebase first
git fetch origin && git rebase origin/dev
git push --force-with-lease origin {TICKET_PREFIX}-XXX-description
```

## Pre-PR Checklist (MANDATORY)

Before creating any PR:

- [ ] Branch name: `{TICKET_PREFIX}-{number}-{description}`
- [ ] Commits follow: `type(scope): description [{TICKET_PREFIX}-XXX]`
- [ ] Rebased on latest dev: `git fetch origin && git rebase origin/dev`
- [ ] CI passes locally: `yarn ci:validate`
- [ ] Linear history: No merge commits (`git log --oneline --graph -10`)

## CI/CD Validation Command

```bash
# MANDATORY before any PR
yarn ci:validate && echo "READY FOR PR" || echo "FIX ISSUES FIRST"
```

## PR Creation Template

````bash
gh pr create --title "feat(scope): description [{TICKET_PREFIX}-XXX]" --body "$(cat <<'EOF'
## Summary

Implements [feature/fix] as specified in Linear ticket {TICKET_PREFIX}-XXX.

**Linear Ticket**: https://linear.app/{LINEAR_WORKSPACE}/issue/{TICKET_PREFIX}-XXX

## Changes Made

- Change 1
- Change 2

## Testing

```bash
yarn ci:validate
# All checks passed
````

## Pre-merge Checklist

- [x] Rebased on latest dev
- [x] CI passes
- [x] Linear ticket referenced

🤖 Generated with [Claude Code](https://claude.com/claude-code)
EOF
)"

````

## Merge Strategy

**ONLY** use rebase merge:

```bash
# CORRECT
gh pr merge --rebase --delete-branch

# NEVER
gh pr merge --squash   # Loses commit history
gh pr merge --merge    # Creates merge commits
````

## QAS Gate (MANDATORY)

Before merging any PR, invoke QAS for independent review:

```text
Task tool: QAS subagent
Prompt: "Review PR #XXX for {TICKET_PREFIX}-YYY. Validate commit format, CI status, patterns."
```

## Authoritative References

- **PR Template**: `.github/pull_request_template.md`
- **Workflow Guide**: `CONTRIBUTING.md` (Pull Request Process section)
- **CI/CD Pipeline**: `docs/CI-CD-Pipeline-Guide.md`
- **Agent Workflow SOP**: `docs/sop/AGENT_WORKFLOW_SOP.md` (3-stage review chain)