krystophny/pr

Start New Bug Fix

Create new bug fix branches. Supports multiple in parallel using git worktrees.

Usage

/pr                      # Create from current changes vs main
/pr minloc-fix           # Create fix/minloc-fix branch
/pr fix1 fix2 fix3       # Multiple branches in parallel worktrees

MODE DETECTION (when no arguments)

# Check for uncommitted changes vs main
CHANGES=$(git diff main --stat)

if [ -n "$CHANGES" ]; then
  echo "Creating PR from changes vs main"
  BRANCH="fix/$(git rev-parse --abbrev-ref HEAD)-$(date +%H%M%S)"
else
  echo "No changes vs main - provide branch name"
  exit 1
fi

PARALLEL EXECUTION (multiple branches)

for NAME in $ARGUMENTS; do
  BRANCH="fix/$NAME"
  WORKTREE="/tmp/worktree-$NAME"

  # Create worktree with new branch from main
  git worktree add -b "$BRANCH" "$WORKTREE" origin/main

  # Spawn agent pipeline in background
done

# Wait for all agents, collect results

AGENT PIPELINE

1. georg-test-engineer: Create MRE test
2. sergei-perfectionist-coder: Implement fix
3. patrick-reviewer: Review before PR

SINGLE BRANCH WORKFLOW

1. SETUP BRANCH

cd $WORKTREE  # or current dir
git fetch origin
git checkout -b fix/$NAME origin/main

2. CREATE MRE TEST (spawn georg-test-engineer)

• Create minimal reproducing test
• Add to build system with labels
• MUST fail on main, pass after fix

3. VERIFY TEST FAILS ON MAIN (CAPTURE OUTPUT)

git stash
git checkout origin/main
# Run test - MUST FAIL - CAPTURE CONSOLE OUTPUT FOR PR DESCRIPTION
<test_command> 2>&1 | tee /tmp/test-fail-main.log
git checkout fix/$NAME
git stash pop

MANDATORY: Save this failure output for PR description evidence.

4. IMPLEMENT FIX (spawn sergei-perfectionist-coder)

• Search codebase first (reuse-first)
• Fix in earliest pipeline stage
• Minimal changes only

5. VERIFY AND PUSH

# Run test - MUST PASS
# Run full suite - ALL must pass (100%)
git add <specific-files>
git commit -m "fix: <description>"
git push -u origin fix/$NAME

6. REVIEW (spawn patrick-reviewer)

• Clean code compliance
• Tech debt check
• Test quality

7. CREATE PR WITH VERIFICATION EVIDENCE

gh pr create --draft --title "fix: <description>" --body-file /tmp/pr.md

PR body MUST include verification section:

## Verification

### Test fails on main
\`\`\`
$ git checkout upstream/main
$ <test_command>
[... actual console output from /tmp/test-fail-main.log ...]
\`\`\`

### Test passes after fix
\`\`\`
$ git checkout fix/$NAME
$ <test_command>
[... actual console output showing PASS ...]
\`\`\`

WORKTREE CLEANUP

for NAME in $ARGUMENTS; do
  git worktree remove "/tmp/worktree-$NAME" 2>/dev/null
done

TECHNICAL DEBT PREVENTION

| Check | Requirement | |-------|-------------| | Function size | < 100 lines | | Module size | < 1000 lines | | TODO/FIXME | Must have linked issue | | Commented code | Must be deleted |

AGENT RESPONSIBILITIES

georg-test-engineer

• Create minimal reproducing test
• Ensure test is strict (not shallow)
• Verify test fails on main

sergei-perfectionist-coder

• Search codebase before implementing
• Implement minimal fix
• Run full test suite (100% pass)

patrick-reviewer

• Verify clean code principles
• Detect technical debt
• Approve or handback