Create Pull Request

Purpose

Create well-structured pull requests with proper GitHub issue linking, comprehensive descriptions, quality metrics, and adherence to WescoBar project standards.

When to Use

• •After completing feature implementation and all quality gates pass
• •During Conductor workflow Phase 4 (PR Creation)
• •When ready to submit code for review
• •After all tests pass and audit score ≥ 8.0

Critical Requirements

✅ MUST Do Before PR Creation

1. •All tests passing - No failing tests allowed
2. •Audit score ≥ 8.0 - Quality threshold met
3. •Build successful - Production build completes
4. •Commits pushed - All commits on remote branch
5. •Branch up-to-date - Synced with base branch (development)

❌ NEVER Do

• •Create PR with failing tests
• •Skip quality gates
• •Use incorrect issue linking format
• •Create PR before all validation passes

Instructions

Step 1: Gather PR Metadata

Collect required information:

• •Issue number: From branch name or conductor context
• •Issue title: From GitHub issue
• •Files changed: Count from git diff
• •Test coverage: From test results
• •Audit score: From audit agent
• •Implementation summary: Key changes made

Step 2: Draft PR Body

Use this template:

## Summary
[Brief description of what was implemented]

## Changes
- [Key change 1]
- [Key change 2]
- [Key change 3]

## Architecture Review
- VSA compliance: ✅
- SOLID principles: ✅
- Layer boundaries: ✅

## Test Coverage
- Unit tests: [COVERAGE]%
- Integration tests: ✅ Passing
- UI tests: ✅ Passing
- E2E tests: ✅ Passing [if applicable]

## Quality Metrics
- Audit score: [SCORE]/10
- Build: ✅ Passing
- Lint: ✅ Clean
- TypeScript: ✅ No errors

## Files Changed
- Modified: [COUNT] files
- Created: [COUNT] files
- Deleted: [COUNT] files

## Issue Reference
Fixes #[ISSUE_NUMBER]

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Step 3: Validate Issue Linking Format

CRITICAL: Issue linking MUST use exact format Fixes #123

✅ CORRECT:

• •Fixes #123
• •Closes #456
• •Resolves #789

❌ WRONG (GitHub won't auto-close):

• •Fixes: #123 (colon breaks it)
• •**Fixes:** #123 (markdown breaks it)
• •Fix #123 (singular doesn't work)
• •fixes #123 (lowercase doesn't work in PR body)

Step 4: Create PR with gh CLI

# Create PR with proper base and head branches
gh pr create \
  --title "feat: [FEATURE_TITLE]" \
  --body "[PR_BODY_FROM_STEP_2]" \
  --base development \
  --head [BRANCH_NAME]

Step 5: Verify PR Creation

After creation:

1. •✅ PR URL returned
2. •✅ PR number assigned
3. •✅ Issue linked (check on GitHub)
4. •✅ All checks queued/running

PR Title Convention

Follow conventional commits:

feat: Add user dark mode toggle
fix: Resolve character portrait caching issue
refactor: Simplify WorldContext state management
test: Add integration tests for Gemini API
docs: Update README with new API patterns

Common Issues

Issue: PR creation fails with "No commits between base and head"

Solution: Ensure commits are pushed to remote branch:

git push -u origin [BRANCH_NAME]

Issue: Issue doesn't auto-link

Solution: Check issue linking format - must be Fixes #123 (exact format)

Issue: PR checks don't start

Solution: Verify GitHub Actions are enabled for repository

Integration with Conductor Workflow

The Conductor agent uses this skill in Phase 4:

**Phase 4, Step 3**: Create Pull Request

Using the `create-pull-request` skill:
- Gather all metrics from previous phases
- Draft comprehensive PR body
- Validate issue linking format
- Create PR with gh CLI
- Verify creation successful

Related Skills

• •commit-changes - Single atomic commit before PR
• •link-github-issue - Validate issue linking format
• •monitor-ci-checks - Monitor PR checks after creation

Additional Resources

See REFERENCE.md in this skill directory for:

• •Complete PR template examples
• •Issue linking format reference
• •GitHub CLI documentation
• •Troubleshooting guide