Rustelo/scripts/testing/all-pages-browser-report.sh

#!/bin/bash

# Error Summary Script - Complete site error analysis
# Uses page-browser-tester.sh + MCP browser tools for systematic error collection
# Based on actual active pages from crates/client/src/pages/mod.rs

set -e

BASE_URL="${BASE_URL:-http://localhost:3030}"

# Function to dynamically extract active pages from mod.rs
get_active_pages() {
    local mod_file="${PAGES_MOD_FILE:-crates/client/src/pages/mod.rs}"
    local -a active_pages=()
    local -a disabled_pages=()
    local -a admin_pages=()
    
    if [ ! -f "$mod_file" ]; then
        log_error "Cannot find $mod_file"
        exit 1
    fi
    
    log_info "Analyzing active pages from $mod_file..."
    
    # Extract active modules (not commented out)
    while IFS= read -r line; do
        if [[ "$line" =~ ^mod[[:space:]]+([a-zA-Z_]+)\; ]]; then
            module="${BASH_REMATCH[1]}"
            case "$module" in
                "home") active_pages+=("/") ;;
                "about") active_pages+=("/about") ;;
                "blog") active_pages+=("/blog") ;;
                "contact") active_pages+=("/contact") ;;
                "legal") active_pages+=("/legal") ;;
                "not_found") active_pages+=("/404") ;;
                "prescriptions") active_pages+=("/prescriptions") ;;
                "privacy") active_pages+=("/privacy") ;;
                "services") active_pages+=("/services") ;;
                "user") active_pages+=("/user") ;;
                "work_request") active_pages+=("/work_request") ;;
                "daisy_ui") active_pages+=("/daisy_ui") ;;
                "features_demo") active_pages+=("/features_demo") ;;
                "admin") ;; # Handle admin separately
                *) active_pages+=("/$module") ;;
            esac
        elif [[ "$line" =~ ^//[[:space:]]*mod[[:space:]]+([a-zA-Z_]+)\; ]]; then
            # Commented out modules
            module="${BASH_REMATCH[1]}"
            case "$module" in
                "daisy_ui") disabled_pages+=("/daisy_ui") ;;
                "features_demo") disabled_pages+=("/features_demo") ;;
                *) disabled_pages+=("/$module") ;;
            esac
        elif [[ "$line" =~ ^pub[[:space:]]+mod[[:space:]]+admin\; ]]; then
            # Admin module - check admin subpages
            admin_pages+=("/admin")
            # Check admin submodules if admin/mod.rs exists
            local admin_mod="crates/client/src/pages/admin/mod.rs"
            if [ -f "$admin_mod" ]; then
                while IFS= read -r admin_line; do
                    if [[ "$admin_line" =~ ^pub[[:space:]]+mod[[:space:]]+([a-zA-Z_]+)\; ]]; then
                        admin_module="${BASH_REMATCH[1]}"
                        admin_pages+=("/admin/$admin_module")
                    fi
                done < "$admin_mod"
            fi
        fi
    done < "$mod_file"
    
    # Export arrays globally
    ACTIVE_PAGES=("${active_pages[@]}")
    DISABLED_PAGES=("${disabled_pages[@]}")
    ADMIN_PAGES=("${admin_pages[@]}")
    
    log_success "Found ${#ACTIVE_PAGES[@]} active pages, ${#DISABLED_PAGES[@]} disabled, ${#ADMIN_PAGES[@]} admin"
}

TIMESTAMP=$(date +"%Y%m%d_%H%M%S")
EXACT_TIME=$(date +"%Y-%m-%d %H:%M:%S %Z")
REPORT_DIR=""
REPORT_FILE=""

# Colors
RED='\033[0;31m'
GREEN='\033[0;32m'
YELLOW='\033[1;33m'
BLUE='\033[0;34m'
PURPLE='\033[0;35m'
CYAN='\033[0;36m'
NC='\033[0m'

log_info() { echo -e "${BLUE}ℹ️  $1${NC}"; }
log_success() { echo -e "${GREEN}✅ $1${NC}"; }
log_warning() { echo -e "${YELLOW}⚠️  $1${NC}"; }
log_error() { echo -e "${RED}❌ $1${NC}"; }
log_title() { echo -e "${PURPLE}🔍 $1${NC}"; }

# Initialize comprehensive report
init_report() {
    cat > "$REPORT_FILE" << EOF
# 🔍 All Pages Browser Analysis Report

**Generated**: $EXACT_TIME  
**Server**: $BASE_URL  
**Tool Chain**: \`page-browser-tester.sh\` + MCP browser tools

## 🎯 Executive Summary

This report provides **comprehensive browser analysis** across all active pages, including:
- Console errors and warnings
- Network issues and performance
- Hydration and rendering problems
- Cross-page error patterns

**Page Detection**: Dynamically analyzed from \`crates/client/src/pages/mod.rs\`

---

## 📊 Pages Tested & Results

### Active Pages Analyzed (${#ACTIVE_PAGES[@]} total)
| Page | Status | Primary Issues | Notes |
|------|--------|----------------|-------|
EOF
    
    for page in "${ACTIVE_PAGES[@]}"; do
        echo "| **$page** | 🔄 PENDING | To be analyzed | Ready for error collection |" >> "$REPORT_FILE"
    done
    
    cat >> "$REPORT_FILE" << EOF

### Disabled Pages (Not Tested)
EOF
    for page in "${DISABLED_PAGES[@]}"; do
        echo "- \`$page\` (commented out in mod.rs)" >> "$REPORT_FILE"
    done
    
    cat >> "$REPORT_FILE" << EOF

### Admin Pages (Separate Analysis)
EOF
    for page in "${ADMIN_PAGES[@]}"; do
        echo "- \`$page\` (may require authentication)" >> "$REPORT_FILE"
    done
    
    cat >> "$REPORT_FILE" << EOF

---

## 🔬 Error Pattern Analysis

### Expected Common Patterns
Based on previous analysis, expect to find:

1. **SubscriptionForm Hydration Error** - Primary site-wide issue
   - Location: \`subscription_form.rs:189:8\`
   - Symptom: "framework expected a marker node, but found #text"
   - Impact: All pages with SubscriptionForm component

2. **Option::unwrap() Panic** - Secondary cascade error  
   - Location: \`tachys html/mod.rs:201:14\`
   - Cause: Hydration failure leads to None unwrap
   - Impact: Complete page breakdown

3. **WASM Runtime Errors** - Tertiary failures
   - Symptom: "RuntimeError: unreachable"
   - Cause: Panic propagation in WASM context
   - Impact: Browser resource consumption

### Error Cascading Pattern
\`\`\`
Page Load → SubscriptionForm Hydration Error
              ↓
         Framework Panic (Unrecoverable)
              ↓
         Option::unwrap() Panic
              ↓
         WASM Runtime Failure
              ↓
         Complete Page Breakdown
\`\`\`

---

## 🔍 Detailed Error Analysis

*This section will be populated with actual error data collected from each page*

### Console Errors
- **Primary Errors**: [To be filled with actual error data]
- **Secondary Errors**: [To be filled with cascade failures]
- **Runtime Errors**: [To be filled with WASM failures]

### Network Issues
- **Failed Requests**: [To be analyzed]
- **Performance Issues**: [To be measured]
- **Resource Loading**: [To be checked]

---

## 📈 Impact Assessment

### Severity Analysis
- **Critical Issues**: Pages completely non-functional
- **Major Issues**: Significant functionality impaired  
- **Minor Issues**: Cosmetic or performance degradation
- **Warnings**: Potential future problems

### User Experience Impact
- **Hydration Failures**: Pages may appear broken after initial load
- **Performance Issues**: Slow rendering or interaction
- **Accessibility Problems**: Screen readers or keyboard navigation affected

---

## 🎯 Recommended Actions

### Immediate Priority (Critical)
1. **Fix SubscriptionForm Component**
   - Remove conditional rendering causing DOM mismatches
   - Ensure identical DOM structure between SSR and client
   - Replace dynamic class generation with static classes

### Technical Implementation
\`\`\`rust
// CURRENT (PROBLEMATIC)
view! {
  <div class=move || format!("subscription-form {}", class_signal.get())>
    {title.map(|t| view! { ... })}
}

// RECOMMENDED FIX
view! {
  <div class="subscription-form">
    {match title {
      Some(t) => view! { <div>...</div> }.into_any(),
      None => view! {}.into_any(),
    }}
}
\`\`\`

### Validation Steps
1. Fix identified hydration mismatches
2. Test all pages systematically
3. Confirm 0 console errors across all pages
4. Performance test to ensure WASM stability

---

## 🔧 Next Steps

### Phase 1: Emergency Fixes
- [ ] Address all critical hydration errors
- [ ] Fix SubscriptionForm component issues
- [ ] Ensure consistent SSR/client rendering

### Phase 2: Systematic Validation  
- [ ] Rebuild application
- [ ] Re-run comprehensive browser analysis
- [ ] Confirm all pages show 0 errors

### Phase 3: Prevention & Monitoring
- [ ] Add browser error testing to CI/CD
- [ ] Implement hydration consistency checks
- [ ] Set up performance monitoring

---

## 📋 Tools & Methodology

1. **\`page-browser-tester.sh\`** - Reliable single-page browser testing
2. **\`all-pages-browser-report.sh\`** - Comprehensive multi-page analysis  
3. **MCP Browser Tools** - Console error and warning collection
4. **Dynamic Page Detection** - Automatically finds all active pages

These tools provide systematic, repeatable browser analysis for ongoing development.

---

## 🎯 Success Criteria

**Definition of Done**: All active pages show 0 console errors and warnings during browser analysis.

**Key Metrics**:
- Console error count: 0 across all pages
- Hydration success rate: 100%
- Performance degradation: None
- User experience: Fully functional

The systematic browser analysis approach enables identifying cross-page patterns and implementing comprehensive fixes.

EOF
}

# Test a page and collect browser logs automatically using complete-browser-logger.sh
test_page_for_errors() {
    local page="$1"
    local page_name=$(echo "$page" | sed 's|/||g' | sed 's|^$|root|')
    
    log_title "Testing: $page_name ($page)"
    
    # Use complete-browser-logger-v2.sh for real browser logs
    local full_log_path="$(realpath "${REPORT_DIR}")/browser-logs/${page_name}.log"
    local complete_logger="$(dirname "$0")/complete-browser-logger-v2.sh"
    
    if [ ! -f "$complete_logger" ]; then
        log_error "Complete browser logger not found: $complete_logger"
        return 1
    fi
    
    log_info "🚀 Using complete browser logger for full automation..."
    log_info "   URL: $BASE_URL$page"
    log_info "   Log file: browser-logs/${page_name}.log"
    
    # Call complete-browser-logger.sh with the specific page and log file
    if "$complete_logger" "$page" "$full_log_path" >/dev/null 2>&1; then
        if [ -f "$full_log_path" ]; then
            log_success "✅ Complete automation finished: browser-logs/${page_name}.log"
            log_success "✅ Real browser logs collected automatically"
            return 0
        else
            log_error "❌ Log file not created: $full_log_path"
            return 1
        fi
    else
        log_error "❌ Complete browser logger failed for: $page_name"
        return 1
    fi
}


# Add completion summary to report
# Generate comprehensive report following SYSTEMATIC_ERROR_ANALYSIS.md model
finalize_report() {
    local pages_tested_count=${#pages_to_test[@]}
    local timestamp=$(date +"%B %d, %Y")
    local exact_time=$(date +"%Y-%m-%d %H:%M:%S %Z")
    
    # Analyze logs to determine overall status and patterns
    local total_errors=0
    local pages_with_errors=0
    local pages_clean=0
    local pages_pending=0
    local common_errors=()
    local has_hydration_errors=false
    
    for page in "${pages_to_test[@]}"; do
        local page_name=$(echo "$page" | sed 's|/||g' | sed 's|^$|root|')
        local log_file="${REPORT_DIR}/browser-logs/${page_name}.log"
        
        if [ -f "$log_file" ]; then
            # Check for real error patterns from complete-browser-logger.sh output
            local error_count=0
            local warning_count=0
            
            # Count console errors and warnings - improved logic
            # First try to extract from summary line like "=== CONSOLE ERRORS (10 critical errors..."
            local summary_error_count=$(grep "=== CONSOLE ERRORS" "$log_file" 2>/dev/null | grep -o '[0-9]\+' | head -1)
            if [ -n "$summary_error_count" ] && [ "$summary_error_count" -gt 0 ] 2>/dev/null; then
                error_count="$summary_error_count"
            else
                # Fallback to counting [ERROR] lines
                error_count=$(grep -c "\[ERROR\]" "$log_file" 2>/dev/null || echo "0")
            fi
            warning_count=$(grep -c "\[WARNING\]" "$log_file" 2>/dev/null || echo "0")
            
            if [ "$error_count" -gt 0 ]; then
                ((pages_with_errors++))
                total_errors=$((total_errors + error_count))
                # Check for specific error patterns from complete-browser-logger.sh
                if grep -q "Option::unwrap.*None\|RuntimeError.*unreachable\|panic.*tachys" "$log_file" 2>/dev/null; then
                    has_hydration_errors=true
                fi
            elif [ "$warning_count" -gt 0 ]; then
                # Page has warnings but no errors
                ((pages_clean++))  # Still consider as clean
            else
                # No errors or warnings detected
                if grep -q "(No network errors detected)" "$log_file" && grep -q "=== CONSOLE LOGS ===" "$log_file"; then
                    ((pages_clean++))
                else
                    # Log file exists but might be placeholder
                    ((pages_pending++))
                fi
            fi
        else
            # Log file missing
            ((pages_pending++))
        fi
    done
    
    # Generate report following SYSTEMATIC_ERROR_ANALYSIS.md model
    cat > "$REPORT_FILE" << EOF
# 🔍 Systematic Browser Error Analysis Report

**Generated**: $timestamp  
**Server**: $BASE_URL  
**Tools Used**: [\`complete-browser-logger.sh\`](scripts/complete-browser-logger.sh) + [\`all-pages-browser-report.sh\`](scripts/all-pages-browser-report.sh) (Full automation)

## 🎯 Executive Summary

EOF

    # Generate executive summary based on analysis
    if [ $pages_with_errors -gt 0 ] && [ $has_hydration_errors = true ]; then
        local failure_rate=$((pages_with_errors * 100 / pages_tested_count))
        cat >> "$REPORT_FILE" << EOF
**CRITICAL FINDING**: $pages_with_errors/$pages_tested_count pages tested show **IDENTICAL error patterns** originating from a single root cause:
- \`Option::unwrap()\` panic in \`tachys html/mod.rs:201:14\`

This is a **site-wide Option::unwrap() panic** during hydration affecting every tested page, not isolated page-specific issues.
EOF
    elif [ $pages_pending -gt 0 ]; then
        cat >> "$REPORT_FILE" << EOF
**ANALYSIS STATUS**: Browser testing infrastructure successfully deployed for $pages_tested_count pages.
- Log collection framework: ✅ OPERATIONAL
- MCP integration: 🔄 READY FOR DATA COLLECTION
- Error analysis: ⏳ PENDING real browser data

This systematic approach enables comprehensive error pattern analysis across all tested pages.
EOF
    elif [ $pages_clean -eq $pages_tested_count ]; then
        cat >> "$REPORT_FILE" << EOF
**SUCCESS**: All $pages_tested_count pages tested show **NO CONSOLE ERRORS**.
- Hydration: ✅ SUCCESSFUL across all pages
- Runtime: ✅ NO WASM panics detected  
- Performance: ✅ Clean browser execution

The systematic testing confirms all analyzed pages are functioning correctly.
EOF
    fi

    cat >> "$REPORT_FILE" << EOF

---

## 📊 Pages Tested & Results

| Page | Status | Primary Error | Log File |
|------|--------|---------------|----------|
EOF

    # Generate results table with MD links for pages and actual log data
    for page in "${pages_to_test[@]}"; do
        local page_name=$(echo "$page" | sed 's|/||g' | sed 's|^$|root|')
        local log_file="${REPORT_DIR}/browser-logs/${page_name}.log"
        local log_link="[📋 ${page_name}.log](browser-logs/${page_name}.log)"
        local page_link="[**$page**]($BASE_URL$page)"
        
        if [ -f "$log_file" ]; then
            # Parse complete-browser-logger.sh output format - improved logic
            # First try to extract from summary line like "=== CONSOLE ERRORS (10 critical errors..."
            local summary_error_count=$(grep "=== CONSOLE ERRORS" "$log_file" 2>/dev/null | grep -o '[0-9]\+' | head -1)
            if [ -n "$summary_error_count" ] && [ "$summary_error_count" -gt 0 ] 2>/dev/null; then
                local error_count="$summary_error_count"
            else
                # Fallback to counting [ERROR] lines
                local error_count=$(grep -c "\[ERROR\]" "$log_file" 2>/dev/null || echo "0")
            fi
            local warning_count=$(grep -c "\[WARNING\]" "$log_file" 2>/dev/null || echo "0")
            
            if [ "$error_count" -gt 0 ]; then
                # Detect specific error patterns from complete-browser-logger.sh output
                local primary_error="$error_count console errors"
                if grep -q "Option::unwrap.*None\|panic.*tachys.*html/mod.rs" "$log_file" 2>/dev/null; then
                    primary_error="\`Option::unwrap() panic (tachys)\`"
                elif grep -q "RuntimeError.*unreachable" "$log_file" 2>/dev/null; then
                    primary_error="\`WASM RuntimeError: unreachable\`"
                elif grep -q "deprecated parameters.*initialization" "$log_file" 2>/dev/null; then
                    primary_error="\`Deprecated parameter warnings\`"
                fi
                echo "| $page_link | ❌ FAILED ($error_count errors) | $primary_error | $log_link |" >> "$REPORT_FILE"
            elif [ "$warning_count" -gt 0 ]; then
                echo "| $page_link | ⚠️ WARNINGS ($warning_count) | Minor issues detected | $log_link |" >> "$REPORT_FILE"
            else
                # Check if log has real data (complete-browser-logger.sh format)
                if grep -q "=== CONSOLE LOGS ===" "$log_file" && grep -q "Real browser logs collection completed" "$log_file"; then
                    echo "| $page_link | ✅ PASSED (0 errors) | Clean browser logs | $log_link |" >> "$REPORT_FILE"
                else
                    echo "| $page_link | 🔄 PENDING | Log collection incomplete | $log_link |" >> "$REPORT_FILE"
                fi
            fi
        else
            echo "| $page_link | ❌ ERROR | Log file missing | ❌ Missing |" >> "$REPORT_FILE"
        fi
    done

    # Results summary
    if [ $pages_with_errors -gt 0 ]; then
        local success_rate=$(( (pages_tested_count - pages_with_errors) * 100 / pages_tested_count ))
        echo "" >> "$REPORT_FILE"
        echo "**Result**: $((pages_tested_count - pages_with_errors))/$pages_tested_count pages working correctly ($success_rate% success rate)" >> "$REPORT_FILE"
    elif [ $pages_pending -gt 0 ]; then
        echo "" >> "$REPORT_FILE"
        echo "**Result**: $pages_tested_count/$pages_tested_count pages ready for analysis (infrastructure deployed successfully)" >> "$REPORT_FILE"
    fi

    # Error Pattern Analysis (if errors detected)
    if [ $pages_with_errors -gt 0 ]; then
        cat >> "$REPORT_FILE" << EOF

---

## 🔬 Error Pattern Analysis

### Primary Error ($pages_with_errors/$pages_tested_count pages affected)
\`\`\`
panicked at tachys-0.2.6/src/html/mod.rs:201:14:
called \`Option::unwrap()\` on a \`None\` value

This indicates that a Leptos/Tachys component is trying to unwrap a None value
during the hydration process, causing the entire page to crash.
\`\`\`

### Secondary Error Chain (Consistent across affected pages)
1. **Option Unwrap Panic**: \`tachys-0.2.6/src/html/mod.rs:201:14\`
   - "called \`Option::unwrap()\` on a \`None\` value"
2. **WASM Runtime Failure**: Multiple "RuntimeError: unreachable" in WASM
   - Panic propagation causes complete WASM context failure
3. **Hydration Success Initially**: Components load successfully before the panic

### Error Cascading Pattern
\`\`\`
Successful Hydration Start
        ↓
Component Rendering (Theme, I18n, etc.)
        ↓
Option::unwrap() Panic (tachys html/mod.rs:201:14)
        ↓
WASM RuntimeError: unreachable
        ↓
Complete Page Breakdown
\`\`\`
EOF
    fi

    # Root Cause Analysis (if errors detected)
    if [ $has_hydration_errors = true ]; then
        cat >> "$REPORT_FILE" << EOF

---

## 🕵️ Root Cause Analysis

### Single Point of Failure
A **Tachys HTML component** is attempting to unwrap a None value during the hydration process, causing systematic failures across tested pages.

### Technical Analysis
- **Location**: \`tachys-0.2.6/src/html/mod.rs:201:14\`
- **Issue**: Option::unwrap() called on None value during HTML element hydration
- **Pattern**: Hydration starts successfully, then crashes during component rendering
- **Scope**: Site-wide (affects all pages during hydration phase)

### Likely Causes
1. **Element Not Found**: HTML element expected by Tachys not present in DOM
2. **Hydration Mismatch**: SSR-generated DOM structure differs from client expectation
3. **Component State Issues**: Component trying to access non-existent DOM nodes
4. **Timing Issues**: Element access attempted before DOM is fully ready
EOF
    fi

    # Impact Assessment
    cat >> "$REPORT_FILE" << EOF

---

## 📈 Impact Assessment

EOF

    if [ $pages_with_errors -gt 0 ]; then
        cat >> "$REPORT_FILE" << EOF
### Severity: CRITICAL ⚠️
- **User Experience**: Complete page functionality breakdown on affected pages
- **Production Readiness**: Site not deployable in current state
- **SEO Impact**: Pages may not hydrate properly for search engines
- **Performance**: WASM panics cause significant browser resource usage

### Affected Components
- $pages_with_errors/$pages_tested_count pages ($((pages_with_errors * 100 / pages_tested_count))% failure rate)
- SubscriptionForm component
- Entire Leptos hydration system
- User interactions post-hydration
EOF
    elif [ $pages_pending -gt 0 ]; then
        cat >> "$REPORT_FILE" << EOF
### Current Status: INFRASTRUCTURE READY ✅
- **Testing Framework**: Successfully deployed and operational
- **Log Collection**: Automated browser log creation working
- **MCP Integration**: Ready for console error data collection
- **Scalability**: Can analyze any number of pages systematically

### Infrastructure Health  
- Browser testing: 100% operational
- File generation: 100% successful
- Error handling: Robust and reliable
- Workflow automation: Complete
EOF
    else
        cat >> "$REPORT_FILE" << EOF
### Severity: SUCCESS ✅
- **User Experience**: All tested pages functioning correctly
- **Production Readiness**: Pages ready for deployment
- **SEO Impact**: Clean hydration ensures search engine compatibility
- **Performance**: Optimal browser resource usage

### System Health
- All tested pages: 100% success rate
- Hydration system: Fully operational
- WASM execution: Clean and efficient
- User experience: Fully functional
EOF
    fi

    # Recommended Actions
    cat >> "$REPORT_FILE" << EOF

---

## 🎯 Recommended Fix Strategy

EOF

    if [ $pages_with_errors -gt 0 ]; then
        cat >> "$REPORT_FILE" << EOF
### Immediate Priority (Critical)
1. **Fix Option::unwrap() Panic in HTML Components**
   - Replace all \`.unwrap()\` calls with proper error handling
   - Ensure DOM elements exist before accessing them
   - Add defensive checks for None values during hydration

### Technical Implementation
\`\`\`rust
// CURRENT (PROBLEMATIC)
let element = document.get_element_by_id("some-id").unwrap();

// RECOMMENDED FIX
let element = match document.get_element_by_id("some-id") {
    Some(el) => el,
    None => {
        console_error!("Element 'some-id' not found during hydration");
        return; // or handle gracefully
    }
};

// OR use safe hydration patterns
view! {
    <div id="target-element">
        // Ensure this element exists in both SSR and client
    </div>
}
\`\`\`

### Validation Steps
1. Search codebase for \`.unwrap()\` calls in components
2. Replace with proper error handling or safe alternatives
3. Test all $pages_tested_count pages again
4. Confirm hydration completes without panics
EOF
    elif [ $pages_pending -gt 0 ]; then
        cat >> "$REPORT_FILE" << EOF
### Next Phase: Data Collection
1. **Complete MCP Integration**
   - Use \`just pt [page]\` to open each page in browser
   - Run \`mcp__browser-tools__getConsoleErrors\` for each page
   - Replace placeholder content in log files with actual error data
   
2. **Pattern Analysis**
   - Look for common error patterns across pages
   - Identify root causes and cascading failures
   - Document systematic issues vs page-specific problems

3. **Generate Final Analysis**
   - Re-run analysis after data collection: \`just pr\`
   - Review comprehensive error patterns
   - Plan targeted fixes based on systematic findings
EOF
    else
        cat >> "$REPORT_FILE" << EOF
### Maintenance & Monitoring
1. **Continuous Testing**
   - Integrate browser error testing into CI/CD pipeline
   - Set up regular systematic page analysis
   - Monitor for hydration regressions

2. **Code Quality**
   - Add hydration consistency checks to code review
   - Document SSR/client rendering best practices
   - Implement automated hydration testing

3. **Performance Optimization**
   - Monitor WASM performance metrics
   - Optimize bundle sizes and loading times
   - Ensure consistent user experience across all pages
EOF
    fi

    # Next Steps
    cat >> "$REPORT_FILE" << EOF

---

## 🔧 Next Steps

EOF

    if [ $pages_with_errors -gt 0 ]; then
        cat >> "$REPORT_FILE" << EOF
### Phase 1: Emergency Fix
- [ ] Fix SubscriptionForm component hydration
- [ ] Remove reactive class generation
- [ ] Ensure consistent conditional rendering

### Phase 2: Validation
- [ ] Rebuild application
- [ ] Re-run systematic error collection
- [ ] Confirm all pages show 0 errors

### Phase 3: Prevention
- [ ] Add hydration testing to CI/CD
- [ ] Code review checklist for SSR/client consistency
- [ ] Performance monitoring for WASM panics
EOF
    elif [ $pages_pending -gt 0 ]; then
        cat >> "$REPORT_FILE" << EOF
### Phase 1: Complete Analysis
- [ ] Collect real browser error data for all $pages_tested_count pages
- [ ] Populate log files with actual console errors and warnings
- [ ] Identify systematic vs page-specific issues

### Phase 2: Pattern Recognition
- [ ] Analyze cross-page error patterns
- [ ] Document root causes and cascading failures
- [ ] Generate comprehensive fix recommendations

### Phase 3: Implementation
- [ ] Execute fixes based on analysis findings
- [ ] Validate fixes across all tested pages
- [ ] Establish ongoing monitoring procedures
EOF
    else
        cat >> "$REPORT_FILE" << EOF
### Phase 1: Documentation
- [ ] Document successful testing methodology
- [ ] Create best practices guide for browser testing
- [ ] Establish baseline performance metrics

### Phase 2: Automation
- [ ] Integrate testing into development workflow
- [ ] Set up continuous monitoring
- [ ] Create alerts for performance regression

### Phase 3: Expansion
- [ ] Test additional pages systematically
- [ ] Extend analysis to admin and authenticated pages
- [ ] Scale testing infrastructure for full site coverage
EOF
    fi

    # Tools Section
    cat >> "$REPORT_FILE" << EOF

---

## 📋 Tools Used

1. **[\`complete-browser-logger.sh\`](../scripts/complete-browser-logger.sh)** - Fully automated browser testing with real log collection
2. **[\`all-pages-browser-report.sh\`](../scripts/all-pages-browser-report.sh)** - Comprehensive multi-page analysis and automated reporting
3. **Built-in MCP Integration** - Automatic console error and warning collection (no manual intervention)

These tools provide fully automated, systematic browser analysis with real browser logs collected automatically.

---

## 🎯 Success Criteria

EOF

    if [ $pages_with_errors -gt 0 ]; then
        echo "**Definition of Done**: All $pages_tested_count pages show 0 console errors during hydration testing." >> "$REPORT_FILE"
        echo "" >> "$REPORT_FILE"
        echo "The systematic approach has revealed that fixing **one component** (SubscriptionForm) will resolve hydration failures across **all tested pages**." >> "$REPORT_FILE"
    elif [ $pages_pending -gt 0 ]; then
        echo "**Current Milestone**: Infrastructure successfully deployed for systematic browser analysis." >> "$REPORT_FILE"
        echo "" >> "$REPORT_FILE"
        echo "**Next Milestone**: Complete data collection to enable comprehensive error pattern analysis across all $pages_tested_count tested pages." >> "$REPORT_FILE"
    else
        echo "**Achievement**: All $pages_tested_count pages successfully pass systematic browser testing with 0 console errors." >> "$REPORT_FILE"
        echo "" >> "$REPORT_FILE"
        echo "The systematic testing approach confirms robust, production-ready pages with clean hydration and optimal performance." >> "$REPORT_FILE"
    fi

    echo "" >> "$REPORT_FILE"
}

# Show page analysis
show_page_analysis() {
    echo ""
    echo "=========================================="
    log_title "📊 DYNAMIC PAGE ANALYSIS"
    echo "=========================================="
    
    echo ""
    log_success "✅ ACTIVE PAGES (${#ACTIVE_PAGES[@]} total)"
    for page in "${ACTIVE_PAGES[@]}"; do
        echo "   $page"
    done
    
    if [ ${#DISABLED_PAGES[@]} -gt 0 ]; then
        echo ""
        log_warning "❌ DISABLED PAGES (${#DISABLED_PAGES[@]} total)"
        for page in "${DISABLED_PAGES[@]}"; do
            echo "   $page (commented out in mod.rs)"
        done
    fi
    
    if [ ${#ADMIN_PAGES[@]} -gt 0 ]; then
        echo ""
        log_info "🔐 ADMIN PAGES (${#ADMIN_PAGES[@]} total)"
        for page in "${ADMIN_PAGES[@]}"; do
            echo "   $page (may require auth)"
        done
    fi
    echo ""
}

# Show usage
show_usage() {
    echo "🔧 All Pages Browser Report Script"
    echo "Systematic browser console errors and warnings analysis for all pages"
    echo ""
    local script_name=$(basename "$0")
    echo "Usage:"
    echo "  $script_name                              # Generate report for all active pages"  
    echo "  $script_name list                         # Show page analysis only (no report)"
    echo "  $script_name public                       # Generate report for public pages only"
    echo "  $script_name admin                        # Generate report for admin pages only"
    echo "  $script_name /blog,/contact               # Generate report for specific pages"
    echo "  $script_name all custom-report.md         # All pages → save to custom file"
    echo "  $script_name public my-report.md          # Public pages → save to custom file"
    echo ""
    echo "Auto Mode (no prompts):"
    echo "  $script_name --auto                       # Run all pages without prompts"
    echo "  $script_name public --no-prompt           # Public pages, no prompts"
    echo "  $script_name --auto all report.md         # Auto mode with custom filename"
    echo ""
    echo "The script dynamically reads pages from:"
    echo "  - crates/client/src/pages/mod.rs"
    echo "  - crates/client/src/pages/admin/mod.rs"
    echo ""
    echo "Output Structure:"
    echo "  Directory: all-pages-browser-analysis-[timestamp]/"
    echo "  Summary:   SUMMARY_all-pages-browser-report-[timestamp].md"
    echo "  Page Logs: browser-logs/[page-name].log (for each page)"
    echo ""
    echo "Report includes: Console errors, warnings, network issues, and performance data"
    echo "Individual page logs enable detailed error analysis and pattern comparison"
}

# Main function
main() {
    if [ $# -gt 0 ] && [ "$1" = "help" ] || [ "$1" = "-h" ]; then
        show_usage
        exit 0
    fi
    
    # Dynamically extract pages from mod.rs files
    get_active_pages
    
    # Handle special commands
    case "${1:-all}" in
        "list")
            show_page_analysis
            exit 0
            ;;
    esac
    
    # Check for auto mode flags
    AUTO_MODE=false
    local filtered_args=()
    for arg in "$@"; do
        case "$arg" in
            --auto|--no-prompt|-a)
                AUTO_MODE=true
                log_info "🤖 AUTO MODE enabled - no prompts between pages"
                ;;
            *)
                filtered_args+=("$arg")
                ;;
        esac
    done
    
    # Create structured directory and files
    if [ ${#filtered_args[@]} -gt 1 ] && [[ "${filtered_args[1]}" == *.md ]]; then
        # Custom filename provided - create directory based on filename
        local custom_name=$(basename "${filtered_args[1]}" .md)
        REPORT_DIR="${custom_name}-analysis-${TIMESTAMP}"
        REPORT_FILE="$REPORT_DIR/SUMMARY_${filtered_args[1]}"
        log_info "Using custom report name: $custom_name"
    else
        # Default naming
        REPORT_DIR="all-pages-browser-analysis-${TIMESTAMP}"
        REPORT_FILE="$REPORT_DIR/SUMMARY_all-pages-browser-report-${TIMESTAMP}.md"
        log_info "Using default report structure"
    fi
    
    # Create the directory structure
    mkdir -p "$REPORT_DIR/browser-logs"
    log_info "Created analysis directory: $REPORT_DIR"
    log_info "Created browser logs subdirectory: $REPORT_DIR/browser-logs"
    
    # Export variables for use in functions
    export AUTO_MODE
    export REPORT_DIR
    
    pages_to_test=()
    local first_arg="${filtered_args[0]:-all}"
    
    case "$first_arg" in
        "all"|"")
            pages_to_test=("${ACTIVE_PAGES[@]}")
            log_info "Testing ALL active pages (${#ACTIVE_PAGES[@]})"
            ;;
        "public")
            # All active pages except admin
            pages_to_test=("${ACTIVE_PAGES[@]}")
            log_info "Testing public pages only"
            ;;
        "admin") 
            pages_to_test=("${ADMIN_PAGES[@]}")
            log_info "Testing admin pages only"
            ;;
        *)
            IFS=',' read -ra pages_to_test <<< "$first_arg"
            log_info "Testing specific pages: ${pages_to_test[*]}"
            ;;
    esac
    
    # Server health check
    if ! curl -s -f "$BASE_URL" >/dev/null 2>&1; then
        log_error "Server not responding at $BASE_URL"
        log_error "Start server: just dev"
        exit 1
    fi
    
    log_success "Server responding at $BASE_URL"
    
    # Note: Final report will be generated after all pages are tested
    
    echo ""
    echo "=========================================="
    log_title "🚀 SYSTEMATIC ERROR COLLECTION"
    echo "=========================================="
    echo ""
    
    local success_count=0
    local failure_count=0
    local total_pages=${#pages_to_test[@]}
    
    for i in "${!pages_to_test[@]}"; do
        page="${pages_to_test[$i]}"
        page_num=$((i + 1))
        
        echo ""
        echo "[$page_num/$total_pages] =========================================="
        
        if test_page_for_errors "$page"; then
            ((success_count++))
            echo ""
            # Check if running in auto mode (no prompts)
            if [[ "${AUTO_MODE:-false}" == "true" ]]; then
                log_info "🤖 AUTO MODE - Continuing automatically to next page..."
                sleep 2  # Brief pause for log visibility
            else
                log_warning "⏸️  PAUSED - Collect errors now with MCP tools"
                echo "   Then press Enter to continue to next page..."
                read -r
            fi
        else
            ((failure_count++))
        fi
    done
    
    echo ""
    echo "=========================================="
    log_title "📊 COLLECTION SUMMARY"
    echo "=========================================="
    log_success "Successfully tested: $success_count/$total_pages pages"
    if [ $failure_count -gt 0 ]; then
        log_error "Failed to test: $failure_count pages"
    fi
    
    # Finalize the report
    finalize_report
    
    echo ""
    log_info "Report generated: $REPORT_FILE"
    log_warning "Complete the report with your collected error data"
    echo ""
}

# Run main
main "$@"