MCP Memory Service

#!/bin/bash
# scripts/pr/amp_pr_review.sh - Complete PR review workflow using Amp CLI
#
# Usage: bash scripts/pr/amp_pr_review.sh <PR_NUMBER>
# Example: bash scripts/pr/amp_pr_review.sh 215

set -e

PR_NUMBER=$1

if [ -z "$PR_NUMBER" ]; then
    echo "Usage: $0 <PR_NUMBER>"
    exit 1
fi

echo "=================================================================="
echo "        Amp CLI Complete PR Review Workflow"
echo "        PR #${PR_NUMBER}"
echo "=================================================================="
echo ""

START_TIME=$(date +%s)
WORKFLOW_EXIT_CODE=0

# Step 1: Quality Gate Checks
echo "=== Step 1: Quality Gate Checks (Parallel) ==="
echo "Running complexity, security, and type hint analysis..."
echo ""

bash scripts/pr/amp_quality_gate.sh $PR_NUMBER

# Prompt user to run Amp tasks
echo ""
echo "⚠️  MANUAL STEP REQUIRED: Run the Amp commands shown above"
echo ""
read -p "Press ENTER after running all Amp quality gate commands... " -r
echo ""

# Collect quality gate results
quality_uuids=$(cat /tmp/amp_quality_gate_uuids_${PR_NUMBER}.txt 2>/dev/null || echo "")
if [ -n "$quality_uuids" ]; then
    bash scripts/pr/amp_collect_results.sh --timeout 300 --uuids "$quality_uuids"
    QUALITY_EXIT=$?

    if [ $QUALITY_EXIT -eq 2 ]; then
        echo ""
        echo "🔴 CRITICAL: Security vulnerabilities detected. Stopping workflow."
        echo "Fix security issues before continuing."
        exit 2
    elif [ $QUALITY_EXIT -eq 1 ]; then
        echo ""
        echo "⚠️  Quality gate warnings detected (non-blocking). Continuing..."
        WORKFLOW_EXIT_CODE=1
    fi
else
    echo "⚠️  Could not find quality gate UUIDs. Skipping collection."
fi

echo ""
echo "✅ Step 1 Complete: Quality Gate"
echo ""

# Step 2: Test Generation
echo "=== Step 2: Test Generation ==="
echo "Generating pytest tests for changed files..."
echo ""

bash scripts/pr/amp_generate_tests.sh $PR_NUMBER

echo ""
echo "⚠️  MANUAL STEP REQUIRED: Run the Amp test generation commands shown above"
echo ""
read -p "Press ENTER after running Amp test generation commands... " -r
echo ""

# Collect test generation results
test_uuids=$(cat /tmp/amp_test_generation_uuids_${PR_NUMBER}.txt 2>/dev/null || echo "")
if [ -n "$test_uuids" ]; then
    bash scripts/pr/amp_collect_results.sh --timeout 300 --uuids "$test_uuids"
    echo ""
    echo "✅ Tests generated. Review in .claude/amp/responses/consumed/"
else
    echo "⚠️  Could not find test generation UUIDs. Skipping collection."
fi

echo ""
echo "✅ Step 2 Complete: Test Generation"
echo ""

# Step 3: Breaking Change Detection
echo "=== Step 3: Breaking Change Detection ==="
echo "Analyzing API changes for breaking modifications..."
echo ""

head_branch=$(gh pr view $PR_NUMBER --json headRefName --jq '.headRefName' 2>/dev/null || echo "unknown")
bash scripts/pr/amp_detect_breaking_changes.sh main $head_branch

echo ""
echo "⚠️  MANUAL STEP REQUIRED: Run the Amp breaking change command shown above"
echo ""
read -p "Press ENTER after running Amp breaking change command... " -r
echo ""

# Collect breaking change results
breaking_uuid=$(cat /tmp/amp_breaking_changes_uuid.txt 2>/dev/null || echo "")
if [ -n "$breaking_uuid" ]; then
    bash scripts/pr/amp_collect_results.sh --timeout 120 --uuids "$breaking_uuid"
    BREAKING_EXIT=$?

    if [ $BREAKING_EXIT -ne 0 ]; then
        echo ""
        echo "⚠️  Potential breaking changes detected. Review carefully."
        if [ $WORKFLOW_EXIT_CODE -eq 0 ]; then
            WORKFLOW_EXIT_CODE=1
        fi
    fi
else
    echo "⚠️  Could not find breaking change UUID. Skipping collection."
fi

echo ""
echo "✅ Step 3 Complete: Breaking Change Detection"
echo ""

# Step 4: Fix Suggestions (Optional)
echo "=== Step 4: Fix Suggestions (Optional) ==="
echo "Do you want to generate fix suggestions based on review comments?"
read -p "Generate fix suggestions? (y/N): " -r GENERATE_FIXES
echo ""

if [[ "$GENERATE_FIXES" =~ ^[Yy]$ ]]; then
    bash scripts/pr/amp_suggest_fixes.sh $PR_NUMBER

    echo ""
    echo "⚠️  MANUAL STEP REQUIRED: Run the Amp fix suggestions command shown above"
    echo ""
    read -p "Press ENTER after running Amp fix suggestions command... " -r
    echo ""

    # Collect fix suggestions
    fixes_uuid=$(cat /tmp/amp_fix_suggestions_uuid_${PR_NUMBER}.txt 2>/dev/null || echo "")
    if [ -n "$fixes_uuid" ]; then
        bash scripts/pr/amp_collect_results.sh --timeout 180 --uuids "$fixes_uuid"
        echo ""
        echo "✅ Fix suggestions available in .claude/amp/responses/consumed/"
    else
        echo "⚠️  Could not find fix suggestions UUID. Skipping collection."
    fi
else
    echo "Skipping fix suggestions."
fi

echo ""
echo "✅ Step 4 Complete: Fix Suggestions"
echo ""

# Final Summary
END_TIME=$(date +%s)
TOTAL_TIME=$((END_TIME - START_TIME))

echo "=================================================================="
echo "        Amp CLI PR Review Workflow Complete"
echo "=================================================================="
echo ""
echo "Total Time: ${TOTAL_TIME}s"
echo ""
echo "Results Summary:"
echo "- Quality Gate: $([ -f /tmp/amp_quality_results.json ] && echo "✅ Complete" || echo "⚠️  Incomplete")"
echo "- Test Generation: $([ -n "$test_uuids" ] && echo "✅ Complete" || echo "⚠️  Skipped")"
echo "- Breaking Changes: $([ -n "$breaking_uuid" ] && echo "✅ Complete" || echo "⚠️  Skipped")"
echo "- Fix Suggestions: $([ -n "$fixes_uuid" ] && echo "✅ Complete" || echo "⚠️  Skipped")"
echo ""

if [ $WORKFLOW_EXIT_CODE -eq 0 ]; then
    echo "🎉 PR #${PR_NUMBER} passed all Amp CLI checks!"
    echo ""
    echo "Next Steps:"
    echo "1. Review generated tests in .claude/amp/responses/consumed/"
    echo "2. Apply fix suggestions if applicable"
    echo "3. Run full test suite: pytest tests/"
    echo "4. Optional: Run gemini-pr-automator for automated review loop"
    echo "   bash scripts/pr/auto_review.sh ${PR_NUMBER} 5 true"
else
    echo "⚠️  PR #${PR_NUMBER} has warnings or issues requiring attention"
    echo ""
    echo "Next Steps:"
    echo "1. Review quality gate results: /tmp/amp_quality_results.json"
    echo "2. Address warnings before requesting review"
    echo "3. Re-run workflow after fixes: bash scripts/pr/amp_pr_review.sh ${PR_NUMBER}"
fi

echo ""
echo "All results saved to:"
echo "- /tmp/amp_quality_results.json"
echo "- .claude/amp/responses/consumed/"
echo ""

exit $WORKFLOW_EXIT_CODE