// Create task for each review category
const categories = ["Code Quality", "Architecture", "Design System", "Testing", "Security"];

categories.forEach(category => {
  TaskCreate({
    subject: `Review ${category} for PR #${PR_NUMBER}`,
    description: `Analyze PR changes for ${category} issues:
      - Check all changed files
      - Identify issues by severity (critical/important/suggestion)
      - Document findings with file:line references`,
    activeForm: `Reviewing ${category}`
  })
})

// Update task status as each category is reviewed
TaskUpdate({
  taskId: task.id,
  status: "completed"
})

# Get PR details
gh pr view $PR_NUMBER --json title,body,files,commits,reviews,state

# Get changed files
gh pr diff $PR_NUMBER

# Get CI status
gh pr checks $PR_NUMBER

### Code Quality Review

#### Critical
- **ProfileView.swift:45** - Force unwrap could crash
  ```swift
  let user = users.first!  // Could crash if empty

### Architecture

```markdown
### Architecture Review

#### Important
- **NotificationManager.swift** - Direct UIApplication access in Domain
  - Domain layer should not import UIKit
  - Move to Infrastructure layer

#### Positive
- Clean separation between ViewModel and View
- Use cases follow single responsibility

### Design System Review

#### Suggestion
- **NewFeatureView.swift:67** - Hardcoded padding
  ```swift
  .padding(16)  // Use Spacing.md

### Testing

```markdown
### Testing Review

#### Important
- **New files without tests:**
  - CalculateStrainScoreUseCase.swift (business logic)
  - NotificationSettingsViewModel.swift (state management)

#### Suggestion
- Add edge case tests for TRIMP calculation:
  - Zero duration workout
  - Maximum heart rate scenarios

## PR Review: #22

### Summary
**Title**: feat(notifications): add notification settings and TRIMP calculation
**Author**: @tanujsutaria
**Branch**: dev/mac-session-12.3-2026-01-28 → main
**Files**: 37 changed (+2971, -74)
**CI Status**: Passing

### Review Summary
| Category | Critical | Important | Suggestions |
|----------|----------|-----------|-------------|
| Code Quality | 0 | 2 | 3 |
| Architecture | 0 | 1 | 0 |
| Design System | 0 | 0 | 2 |
| Testing | 0 | 2 | 1 |
| **Total** | **0** | **5** | **6** |

### Verdict: APPROVE (with suggestions)

No critical issues. Address important items before merge.

---

### Detailed Findings

#### Code Quality (2 important, 3 suggestions)
[detailed findings...]

#### Architecture (1 important)
[detailed findings...]

#### Design System (2 suggestions)
[detailed findings...]

#### Testing (2 important, 1 suggestion)
[detailed findings...]

---

### Recommended Actions

**Before merge:**
1. Add error handling to NotificationSettingsViewModel async calls
2. Move UIApplication access to Infrastructure layer

**After merge (follow-up):**
1. Add unit tests for TRIMP calculation
2. Replace hardcoded padding values

## PR #22 Quick Review

**Ready to merge** with minor suggestions

**Stats**: 37 files, +2971/-74 lines
**CI**: Passing
**Issues**: 0 critical, 5 important, 6 suggestions

**Top 3 items:**
1. Add error handling in ViewModel async calls
2. Add tests for new use cases
3. Move UIApplication to Infrastructure

[Run with --verbose for full review]

PR_NUMBER="$1"
BASE_DIR=$(dirname "$(pwd)")
WORKTREE_PATH="$BASE_DIR/VitalArc-review-PR$PR_NUMBER"

# Create worktree for PR review
git worktree add "$WORKTREE_PATH" -b "review/pr-$PR_NUMBER" main

# Checkout the PR in the worktree
cd "$WORKTREE_PATH"
gh pr checkout $PR_NUMBER

echo "PR #$PR_NUMBER checked out in: $WORKTREE_PATH"
echo ""
echo "To test locally:"
echo "  cd $WORKTREE_PATH"
echo "  xcodebuild -scheme VitalArc ..."
echo ""
echo "When done reviewing:"
echo "  /worktree-manager remove review-PR$PR_NUMBER"

## PR Review Environment Created

**PR**: #123 - feat(nutrition): add meal planning
**Worktree**: /Users/user/Development/VitalArc-review-PR123
**Branch**: review/pr-123

### Testing Checklist
- [ ] Build passes locally
- [ ] Run tests: `xcodebuild test -scheme VitalArc ...`
- [ ] Manual testing of new feature
- [ ] Check design system compliance

### After Review
```bash
# Return to main worktree
cd /Users/user/Development/VitalArc

# Clean up review worktree
/worktree-manager remove review-PR123

## Error Handling

### PR Not Found

```markdown
## PR Not Found

Could not find PR #999

Check:
- PR number is correct
- You have access to the repository
- `gh auth status` shows logged in

## Authentication Required

Cannot access PR. Run:
```bash
gh auth login

### Large PR Warning

```markdown
## Large PR Detected

This PR has 100+ files changed. Full review may be slow.

**Options:**
1. Run `--quick` for summary only
2. Run `--focus=testing` for specific category
3. Continue with full review