ghost 6.17.1 → 6.18.0

package/core/server/data/tinybird/scripts/benchmark-top-pages.sh

@@ -0,0 +1,186 @@
+#!/bin/bash
+
+# benchmark-top-pages.sh - Compare performance of api_top_pages vs api_top_pages_v3
+#
+# Usage: ./benchmark-top-pages.sh [iterations]
+#
+# Prerequisites: yarn dev:analytics
+
+set -e
+
+# Configuration
+TB_HOST="${TB_HOST:-http://localhost:7181}"
+ITERATIONS="${1:-5}"
+
+# Get site_uuid and token from Docker
+echo "============================================"
+echo "Tinybird Top Pages Benchmark"
+echo "============================================"
+
+# Get token
+TB_TOKEN=$(docker run --rm -v ghost-dev_shared-config:/config alpine cat /config/.env.tinybird 2>/dev/null | grep TINYBIRD_ADMIN_TOKEN | cut -d= -f2)
+if [ -z "$TB_TOKEN" ]; then
+    echo "Error: Could not find Tinybird token. Is yarn dev:analytics running?"
+    exit 1
+fi
+
+# Get site_uuid - first try from analytics data, fallback to Ghost database
+SITE_UUID=$(curl -s -H "Authorization: Bearer $TB_TOKEN" \
+    "${TB_HOST}/v0/sql?q=SELECT%20site_uuid%20FROM%20_mv_hits%20WHERE%20site_uuid%20!=%20'mock_site_uuid'%20LIMIT%201" | tr -d '\n')
+if [ -z "$SITE_UUID" ] || [ "$SITE_UUID" = "" ]; then
+    # Fallback to Ghost database
+    SITE_UUID=$(docker exec ghost-dev-mysql mysql -uroot -proot ghost_dev -N -e "SELECT value FROM settings WHERE \`key\`='db_hash'" 2>/dev/null | tr -d '\r\n')
+fi
+if [ -z "$SITE_UUID" ]; then
+    echo "Error: Could not get site_uuid"
+    exit 1
+fi
+
+# Calculate date range (last 30 days)
+DATE_TO=$(date +%Y-%m-%d)
+DATE_FROM=$(date -v-30d +%Y-%m-%d 2>/dev/null || date -d '30 days ago' +%Y-%m-%d)
+TIMEZONE="Etc/UTC"
+
+echo "Site UUID: $SITE_UUID"
+echo "Date Range: $DATE_FROM to $DATE_TO"
+echo "Iterations: $ITERATIONS"
+echo "============================================"
+echo ""
+
+# Check data volume
+echo "Checking data volume..."
+DATA_COUNT=$(curl -s -H "Authorization: Bearer $TB_TOKEN" \
+    "${TB_HOST}/v0/pipes/api_top_pages.json?site_uuid=${SITE_UUID}&date_from=${DATE_FROM}&date_to=${DATE_TO}&timezone=${TIMEZONE}&limit=1" \
+    | python3 -c "import sys,json; d=json.load(sys.stdin); print(d.get('statistics',{}).get('rows_read', 'unknown'))" 2>/dev/null || echo "unknown")
+echo "Rows in date range: ~${DATA_COUNT}"
+echo ""
+
+# Benchmark function
+benchmark() {
+    local version=$1
+    local label=$2
+    local endpoint="api_top_pages"
+
+    if [ -n "$version" ]; then
+        endpoint="${endpoint}_${version}"
+    fi
+
+    local url="${TB_HOST}/v0/pipes/${endpoint}.json?site_uuid=${SITE_UUID}&date_from=${DATE_FROM}&date_to=${DATE_TO}&timezone=${TIMEZONE}"
+
+    echo "Testing: $label"
+    echo "Endpoint: $endpoint"
+
+    local times=()
+    local total_time=0
+
+    for i in $(seq 1 $ITERATIONS); do
+        # Use curl timing
+        local result=$(curl -s -w "%{time_total}" -o /tmp/tb_response.json \
+            -H "Authorization: Bearer $TB_TOKEN" \
+            "$url")
+
+        local time_ms=$(python3 -c "print(int(float('$result') * 1000))")
+        times+=($time_ms)
+        total_time=$((total_time + time_ms))
+
+        # Check for errors
+        local error=$(python3 -c "import json; d=json.load(open('/tmp/tb_response.json')); print(d.get('error',''))" 2>/dev/null)
+        if [ -n "$error" ]; then
+            echo "  Iteration $i: ERROR - $error"
+        else
+            local stats=$(python3 -c "
+import json
+d=json.load(open('/tmp/tb_response.json'))
+rows=len(d.get('data',[]))
+stats=d.get('statistics',{})
+rows_read=stats.get('rows_read',0)
+print(f'{rows}|{rows_read}')" 2>/dev/null)
+            local rows=$(echo "$stats" | cut -d'|' -f1)
+            local rows_read=$(echo "$stats" | cut -d'|' -f2)
+            printf "  Iteration %d: %4dms | %d results | %s rows scanned\n" "$i" "$time_ms" "$rows" "$rows_read"
+        fi
+    done
+
+    if [ ${#times[@]} -gt 0 ]; then
+        local avg=$((total_time / ${#times[@]}))
+
+        # Calculate min/max
+        local min=${times[0]}
+        local max=${times[0]}
+        for t in "${times[@]}"; do
+            [ $t -lt $min ] && min=$t
+            [ $t -gt $max ] && max=$t
+        done
+
+        # Get final stats for summary
+        local final_rows_read=$(python3 -c "import json; print(json.load(open('/tmp/tb_response.json')).get('statistics',{}).get('rows_read',0))" 2>/dev/null)
+
+        echo "  ----------------------------------------"
+        printf "  Avg: %dms | Min: %dms | Max: %dms | Rows: %s\n" "$avg" "$min" "$max" "$final_rows_read"
+    fi
+    echo ""
+}
+
+# Store results for comparison
+V1_STATS=""
+V3_STATS=""
+
+run_benchmark() {
+    local version=$1
+    local label=$2
+    benchmark "$version" "$label"
+
+    # Capture stats from last run
+    local stats=$(python3 -c "
+import json
+d=json.load(open('/tmp/tb_response.json'))
+s=d.get('statistics',{})
+print(f\"{s.get('elapsed',0)*1000:.1f}|{s.get('rows_read',0)}\")" 2>/dev/null)
+
+    if [ "$version" = "" ]; then
+        V1_STATS="$stats"
+    else
+        V3_STATS="$stats"
+    fi
+}
+
+# Run benchmarks
+run_benchmark "" "api_top_pages (unversioned)"
+run_benchmark "v3" "api_top_pages_v3 (daily MV)"
+
+# Summary comparison
+echo "============================================"
+echo "Performance Comparison Summary"
+echo "============================================"
+
+V1_TIME=$(echo "$V1_STATS" | cut -d'|' -f1)
+V1_ROWS=$(echo "$V1_STATS" | cut -d'|' -f2)
+V3_TIME=$(echo "$V3_STATS" | cut -d'|' -f1)
+V3_ROWS=$(echo "$V3_STATS" | cut -d'|' -f2)
+
+python3 << PYTHON_SUMMARY
+v1_time = float("$V1_TIME")
+v1_rows = int("$V1_ROWS")
+v3_time = float("$V3_TIME")
+v3_rows = int("$V3_ROWS")
+
+print(f"api_top_pages:    {v1_time:>8.1f}ms | {v1_rows:>12,} rows scanned")
+print(f"api_top_pages_v3: {v3_time:>8.1f}ms | {v3_rows:>12,} rows scanned")
+print()
+
+if v1_time > 0 and v3_time > 0:
+    time_ratio = v1_time / v3_time
+    if time_ratio >= 1:
+        print(f"Speed:      v3 is {time_ratio:.1f}x faster")
+    else:
+        print(f"Speed:      v1 is {1/time_ratio:.1f}x faster")
+
+if v1_rows > 0 and v3_rows > 0:
+    rows_ratio = v1_rows / v3_rows
+    print(f"Efficiency: v3 scans {rows_ratio:.1f}x fewer rows")
+PYTHON_SUMMARY
+
+echo "============================================"
+
+# Cleanup
+rm -f /tmp/tb_response.json

package/core/server/data/tinybird/scripts/compare-top-pages.sh

@@ -0,0 +1,262 @@
+#!/bin/bash
+
+# compare-top-pages.sh - Validate api_top_pages vs api_top_pages_v3 results
+#
+# Usage: ./compare-top-pages.sh [-l limit] [-d days] [-t tolerance] [-v]
+#
+# Prerequisites: yarn dev:analytics
+#
+# Expected: v3 counts may be 1-2 lower for pages where sessions cross midnight
+# at date boundaries. This is correct behavior - v1 over-counts by including
+# page views outside the date range when sessions span multiple days.
+
+set -e
+
+# Default configuration
+TB_HOST="${TB_HOST:-http://localhost:7181}"
+LIMIT=50
+DAYS=30
+TOLERANCE=5
+VERBOSE=false
+
+# Parse arguments
+while [[ $# -gt 0 ]]; do
+    case $1 in
+        -l|--limit)
+            LIMIT="$2"
+            shift 2
+            ;;
+        -d|--days)
+            DAYS="$2"
+            shift 2
+            ;;
+        -t|--tolerance)
+            TOLERANCE="$2"
+            shift 2
+            ;;
+        -v|--verbose)
+            VERBOSE=true
+            shift
+            ;;
+        -h|--help)
+            head -20 "$0" | tail -17
+            exit 0
+            ;;
+        *)
+            echo "Unknown option: $1"
+            exit 1
+            ;;
+    esac
+done
+
+echo "============================================"
+echo "Tinybird Top Pages Comparison"
+echo "============================================"
+
+# Get token
+TB_TOKEN=$(docker run --rm -v ghost-dev_shared-config:/config alpine cat /config/.env.tinybird 2>/dev/null | grep TINYBIRD_ADMIN_TOKEN | cut -d= -f2)
+if [ -z "$TB_TOKEN" ]; then
+    echo "Error: Could not find Tinybird token. Is yarn dev:analytics running?"
+    exit 1
+fi
+
+# Get site_uuid from analytics data (check what's actually in the data)
+SITE_UUID=$(curl -s -H "Authorization: Bearer $TB_TOKEN" \
+    "${TB_HOST}/v0/sql?q=SELECT%20site_uuid%20FROM%20_mv_hits%20WHERE%20site_uuid%20!=%20'mock_site_uuid'%20LIMIT%201" | tr -d '\n')
+
+if [ -z "$SITE_UUID" ] || [ "$SITE_UUID" = "" ]; then
+    echo "Error: Could not get site_uuid from analytics data"
+    exit 1
+fi
+
+# Calculate date range
+DATE_TO=$(date +%Y-%m-%d)
+DATE_FROM=$(date -v-${DAYS}d +%Y-%m-%d 2>/dev/null || date -d "${DAYS} days ago" +%Y-%m-%d)
+TIMEZONE="Etc/UTC"
+
+echo "Site UUID: $SITE_UUID"
+echo "Date Range: $DATE_FROM to $DATE_TO"
+echo "Limit: $LIMIT results"
+echo "Tolerance: ${TOLERANCE}%"
+echo "============================================"
+echo ""
+
+# Create temp files
+V1_RESPONSE=$(mktemp)
+V3_RESPONSE=$(mktemp)
+
+cleanup() {
+    rm -f "$V1_RESPONSE" "$V3_RESPONSE"
+}
+trap cleanup EXIT
+
+# Fetch from both endpoints
+echo "Fetching from api_top_pages..."
+curl -s -H "Authorization: Bearer $TB_TOKEN" \
+    "${TB_HOST}/v0/pipes/api_top_pages.json?site_uuid=${SITE_UUID}&date_from=${DATE_FROM}&date_to=${DATE_TO}&timezone=${TIMEZONE}&limit=${LIMIT}" \
+    > "$V1_RESPONSE"
+
+echo "Fetching from api_top_pages_v3..."
+curl -s -H "Authorization: Bearer $TB_TOKEN" \
+    "${TB_HOST}/v0/pipes/api_top_pages_v3.json?site_uuid=${SITE_UUID}&date_from=${DATE_FROM}&date_to=${DATE_TO}&timezone=${TIMEZONE}&limit=${LIMIT}" \
+    > "$V3_RESPONSE"
+
+# Check for errors
+V1_ERROR=$(python3 -c "import json; d=json.load(open('$V1_RESPONSE')); print(d.get('error',''))" 2>/dev/null)
+V3_ERROR=$(python3 -c "import json; d=json.load(open('$V3_RESPONSE')); print(d.get('error',''))" 2>/dev/null)
+
+if [ -n "$V1_ERROR" ]; then
+    echo "Error from api_top_pages: $V1_ERROR"
+    exit 1
+fi
+
+if [ -n "$V3_ERROR" ]; then
+    echo "Error from api_top_pages_v3: $V3_ERROR"
+    exit 1
+fi
+
+echo ""
+
+# Compare results using Python
+export V1_RESPONSE V3_RESPONSE TOLERANCE VERBOSE
+python3 << 'PYTHON_SCRIPT'
+import json
+import sys
+import os
+
+v1_file = os.environ['V1_RESPONSE']
+v3_file = os.environ['V3_RESPONSE']
+tolerance = float(os.environ['TOLERANCE'])
+verbose = os.environ['VERBOSE'] == 'true'
+
+with open(v1_file) as f:
+    v1_data = json.load(f)
+
+with open(v3_file) as f:
+    v3_data = json.load(f)
+
+v1_results = {(r['pathname'], r['post_uuid']): r['visits'] for r in v1_data['data']}
+v3_results = {(r['pathname'], r['post_uuid']): r['visits'] for r in v3_data['data']}
+
+# Statistics
+v1_stats = v1_data.get('statistics', {})
+v3_stats = v3_data.get('statistics', {})
+
+print("=== Performance Statistics ===")
+print(f"api_top_pages:    {v1_stats.get('elapsed', 0)*1000:.2f}ms | {v1_stats.get('rows_read', 0):,} rows read")
+print(f"api_top_pages_v3: {v3_stats.get('elapsed', 0)*1000:.2f}ms | {v3_stats.get('rows_read', 0):,} rows read")
+print()
+
+# Combine all keys
+all_keys = set(v1_results.keys()) | set(v3_results.keys())
+
+differences = []
+matches = []
+only_in_v1 = []
+only_in_v3 = []
+
+for key in all_keys:
+    pathname, post_uuid = key
+    v1_val = v1_results.get(key)
+    v3_val = v3_results.get(key)
+
+    if v1_val is None:
+        only_in_v3.append((pathname, post_uuid, v3_val))
+    elif v3_val is None:
+        only_in_v1.append((pathname, post_uuid, v1_val))
+    else:
+        diff = abs(v1_val - v3_val)
+        diff_pct = (diff / max(v1_val, 1)) * 100
+
+        if diff > 0:
+            differences.append({
+                'pathname': pathname,
+                'post_uuid': post_uuid,
+                'v1': v1_val,
+                'v3': v3_val,
+                'diff': diff,
+                'diff_pct': diff_pct
+            })
+        else:
+            matches.append({
+                'pathname': pathname,
+                'post_uuid': post_uuid,
+                'visits': v1_val
+            })
+
+# Sort differences by absolute difference
+differences.sort(key=lambda x: x['diff'], reverse=True)
+
+print("=== Comparison Results ===")
+print(f"Total pages in v1: {len(v1_results)}")
+print(f"Total pages in v3: {len(v3_results)}")
+print(f"Exact matches: {len(matches)}")
+print(f"Differences: {len(differences)}")
+print(f"Only in v1: {len(only_in_v1)}")
+print(f"Only in v3: {len(only_in_v3)}")
+print()
+
+# Check for significant differences
+significant = [d for d in differences if d['diff_pct'] > tolerance]
+
+if differences:
+    print("=== Differences Found ===")
+    print(f"{'Pathname':<60} {'V1':>8} {'V3':>8} {'Diff':>8} {'Diff%':>8}")
+    print("-" * 96)
+
+    for d in differences[:20]:  # Show top 20 differences
+        pathname = d['pathname'][:58] if len(d['pathname']) > 58 else d['pathname']
+        marker = " *" if d['diff_pct'] > tolerance else ""
+        print(f"{pathname:<60} {d['v1']:>8} {d['v3']:>8} {d['diff']:>+8} {d['diff_pct']:>7.2f}%{marker}")
+
+    if len(differences) > 20:
+        print(f"... and {len(differences) - 20} more differences")
+    print()
+    print("(* = exceeds tolerance threshold)")
+
+if only_in_v1:
+    print()
+    print("=== Pages only in api_top_pages (v1) ===")
+    for pathname, post_uuid, visits in only_in_v1[:10]:
+        print(f"  {pathname:<60} visits: {visits}")
+    if len(only_in_v1) > 10:
+        print(f"  ... and {len(only_in_v1) - 10} more")
+
+if only_in_v3:
+    print()
+    print("=== Pages only in api_top_pages_v3 ===")
+    for pathname, post_uuid, visits in only_in_v3[:10]:
+        print(f"  {pathname:<60} visits: {visits}")
+    if len(only_in_v3) > 10:
+        print(f"  ... and {len(only_in_v3) - 10} more")
+
+if verbose and matches:
+    print()
+    print("=== Exact Matches ===")
+    for m in matches[:20]:
+        print(f"  {m['pathname']:<60} visits: {m['visits']}")
+    if len(matches) > 20:
+        print(f"  ... and {len(matches) - 20} more matches")
+
+print()
+print("============================================")
+
+# Check for unexpected differences (v3 should always be <= v1)
+unexpected = [d for d in differences if d['v3'] > d['v1']]
+
+# Exit status based on differences
+if unexpected:
+    print(f"FAIL: {len(unexpected)} pages have v3 > v1 (unexpected - v3 should never exceed v1)")
+    sys.exit(1)
+elif significant:
+    print(f"WARN: {len(significant)} pages exceed {tolerance}% tolerance (but v3 <= v1 as expected)")
+    print("      Small differences are expected due to cross-midnight session handling.")
+    sys.exit(0)
+elif differences:
+    print(f"OK: {len(differences)} pages have minor differences (within {tolerance}% tolerance)")
+    print("    v3 counts are lower as expected (more accurate date filtering).")
+    sys.exit(0)
+else:
+    print("PASS: All results match exactly!")
+    sys.exit(0)
+PYTHON_SCRIPT

package/core/server/data/tinybird/tests/api_top_pages_v3.yaml

@@ -0,0 +1,112 @@
+
+- name: Date range
+  description: All fixture data - uses pre-aggregated daily MV
+  parameters: site_uuid=mock_site_uuid&date_from=2100-01-01&date_to=2100-01-07&timezone=Etc/UTC
+  expected_result: |
+    {"post_uuid":"6b8635fb-292f-4422-9fe4-d76cfab2ba31","pathname":"\/blog\/hello-world\/","visits":9}
+    {"post_uuid":"06b1b0c9-fb53-4a15-a060-3db3fde7b1fc","pathname":"\/about\/","visits":8}
+    {"post_uuid":"","pathname":"\/","visits":7}
+    {"post_uuid":"06b1b0c9-fb53-4a15-a060-3db3fde7b1dd","pathname":"\/blog\/hello-world\/","visits":1}
+
+- name: Filtered by pathname - /about/
+  description: Filtered by pathname - /about/
+  parameters: site_uuid=mock_site_uuid&date_from=2100-01-01&date_to=2100-01-07&timezone=Etc/UTC&pathname=%2Fabout%2F
+  expected_result: |
+    {"post_uuid":"06b1b0c9-fb53-4a15-a060-3db3fde7b1fc","pathname":"\/about\/","visits":8}
+
+- name: Filtered by post_uuid - 06b1b0c9-fb53-4a15-a060-3db3fde7b1fc (/about/)
+  description: Filtered by post_uuid - 06b1b0c9-fb53-4a15-a060-3db3fde7b1fc (/about/)
+  parameters: site_uuid=mock_site_uuid&date_from=2100-01-01&date_to=2100-01-07&timezone=Etc/UTC&post_uuid=06b1b0c9-fb53-4a15-a060-3db3fde7b1fc
+  expected_result: |
+    {"post_uuid":"06b1b0c9-fb53-4a15-a060-3db3fde7b1fc","pathname":"\/about\/","visits":8}
+
+- name: Filtered by member status - paid
+  description: Filtered by member status - paid (includes comped)
+  parameters: site_uuid=mock_site_uuid&date_from=2100-01-01&date_to=2100-01-07&timezone=Etc/UTC&member_status=paid
+  expected_result: |
+    {"post_uuid":"06b1b0c9-fb53-4a15-a060-3db3fde7b1fc","pathname":"\/about\/","visits":3}
+    {"post_uuid":"6b8635fb-292f-4422-9fe4-d76cfab2ba31","pathname":"\/blog\/hello-world\/","visits":3}
+    {"post_uuid":"","pathname":"\/","visits":2}
+
+- name: Filtered by member status - free
+  description: Filtered by member status - free
+  parameters: site_uuid=mock_site_uuid&date_from=2100-01-01&date_to=2100-01-07&timezone=Etc/UTC&member_status=free
+  expected_result: |
+    {"post_uuid":"","pathname":"\/","visits":4}
+    {"post_uuid":"06b1b0c9-fb53-4a15-a060-3db3fde7b1fc","pathname":"\/about\/","visits":3}
+    {"post_uuid":"6b8635fb-292f-4422-9fe4-d76cfab2ba31","pathname":"\/blog\/hello-world\/","visits":2}
+
+- name: Filtered by member status - undefined
+  description: Filtered by member status - undefined
+  parameters: site_uuid=mock_site_uuid&date_from=2100-01-01&date_to=2100-01-07&timezone=Etc/UTC&member_status=undefined
+  expected_result: |
+    {"post_uuid":"6b8635fb-292f-4422-9fe4-d76cfab2ba31","pathname":"\/blog\/hello-world\/","visits":4}
+    {"post_uuid":"06b1b0c9-fb53-4a15-a060-3db3fde7b1fc","pathname":"\/about\/","visits":2}
+    {"post_uuid":"","pathname":"\/","visits":1}
+    {"post_uuid":"06b1b0c9-fb53-4a15-a060-3db3fde7b1dd","pathname":"\/blog\/hello-world\/","visits":1}
+
+- name: Filtered by timezone - America/Los_Angeles
+  description: Filtered by timezone - America/Los_Angeles (8 hours behind UTC)
+  parameters: site_uuid=mock_site_uuid&date_from=2100-01-01&date_to=2100-01-07&timezone=America/Los_Angeles
+  expected_result: |
+    {"post_uuid":"6b8635fb-292f-4422-9fe4-d76cfab2ba31","pathname":"\/blog\/hello-world\/","visits":9}
+    {"post_uuid":"06b1b0c9-fb53-4a15-a060-3db3fde7b1fc","pathname":"\/about\/","visits":8}
+    {"post_uuid":"","pathname":"\/","visits":7}
+    {"post_uuid":"06b1b0c9-fb53-4a15-a060-3db3fde7b1dd","pathname":"\/blog\/hello-world\/","visits":1}
+
+- name: Test with post_type - post
+  description: Test with post_type - post
+  parameters: site_uuid=mock_site_uuid&date_from=2100-01-01&date_to=2100-01-07&timezone=Etc/UTC&post_type=post
+  expected_result: |
+    {"post_uuid":"6b8635fb-292f-4422-9fe4-d76cfab2ba31","pathname":"\/blog\/hello-world\/","visits":9}
+    {"post_uuid":"06b1b0c9-fb53-4a15-a060-3db3fde7b1dd","pathname":"\/blog\/hello-world\/","visits":1}
+
+- name: Test with post_type - page
+  description: Test with post_type - page (includes empty post_type)
+  parameters: site_uuid=mock_site_uuid&date_from=2100-01-01&date_to=2100-01-07&timezone=Etc/UTC&post_type=page
+  expected_result: |
+    {"post_uuid":"06b1b0c9-fb53-4a15-a060-3db3fde7b1fc","pathname":"\/about\/","visits":8}
+    {"post_uuid":"","pathname":"\/","visits":7}
+
+- name: Test with multiple filters combined - member_status and pathname
+  description: Test combining member_status and pathname filters
+  parameters: site_uuid=mock_site_uuid&date_from=2100-01-01&date_to=2100-01-07&timezone=Etc/UTC&member_status=paid&pathname=%2Fabout%2F
+  expected_result: |
+    {"post_uuid":"06b1b0c9-fb53-4a15-a060-3db3fde7b1fc","pathname":"\/about\/","visits":3}
+
+- name: Test with multiple filters combined - post_type and member_status
+  description: Test combining post_type and member_status filters
+  parameters: site_uuid=mock_site_uuid&date_from=2100-01-01&date_to=2100-01-07&timezone=Etc/UTC&post_type=post&member_status=undefined
+  expected_result: |
+    {"post_uuid":"6b8635fb-292f-4422-9fe4-d76cfab2ba31","pathname":"\/blog\/hello-world\/","visits":4}
+    {"post_uuid":"06b1b0c9-fb53-4a15-a060-3db3fde7b1dd","pathname":"\/blog\/hello-world\/","visits":1}
+
+- name: Test pagination - skip
+  description: Test pagination with skip parameter
+  parameters: site_uuid=mock_site_uuid&date_from=2100-01-01&date_to=2100-01-07&timezone=Etc/UTC&skip=1&limit=2
+  expected_result: |
+    {"post_uuid":"06b1b0c9-fb53-4a15-a060-3db3fde7b1fc","pathname":"\/about\/","visits":8}
+    {"post_uuid":"","pathname":"\/","visits":7}
+
+- name: Test pagination - limit
+  description: Test pagination with limit parameter
+  parameters: site_uuid=mock_site_uuid&date_from=2100-01-01&date_to=2100-01-07&timezone=Etc/UTC&limit=2
+  expected_result: |
+    {"post_uuid":"6b8635fb-292f-4422-9fe4-d76cfab2ba31","pathname":"\/blog\/hello-world\/","visits":9}
+    {"post_uuid":"06b1b0c9-fb53-4a15-a060-3db3fde7b1fc","pathname":"\/about\/","visits":8}
+
+- name: Single day query
+  description: Test querying a single day
+  parameters: site_uuid=mock_site_uuid&date_from=2100-01-04&date_to=2100-01-04&timezone=Etc/UTC
+  expected_result: |
+    {"post_uuid":"6b8635fb-292f-4422-9fe4-d76cfab2ba31","pathname":"\/blog\/hello-world\/","visits":3}
+    {"post_uuid":"06b1b0c9-fb53-4a15-a060-3db3fde7b1fc","pathname":"\/about\/","visits":2}
+    {"post_uuid":"","pathname":"\/","visits":1}
+
+- name: Partial date range
+  description: Test querying a partial date range (first 3 days)
+  parameters: site_uuid=mock_site_uuid&date_from=2100-01-01&date_to=2100-01-03&timezone=Etc/UTC
+  expected_result: |
+    {"post_uuid":"","pathname":"\/","visits":5}
+    {"post_uuid":"06b1b0c9-fb53-4a15-a060-3db3fde7b1fc","pathname":"\/about\/","visits":3}
+    {"post_uuid":"6b8635fb-292f-4422-9fe4-d76cfab2ba31","pathname":"\/blog\/hello-world\/","visits":3}

package/core/server/models/automated-email.js

@@ -1,6 +1,10 @@
 const ghostBookshelf = require('./base');
+const logging = require('@tryghost/logging');
 const urlUtils = require('../../shared/url-utils');
 const lexicalLib = require('../lib/lexical');
+const {MEMBER_WELCOME_EMAIL_SLUGS} = require('../services/member-welcome-emails/constants');
+
+const MEMBER_WELCOME_EMAIL_SLUG_SET = new Set(Object.values(MEMBER_WELCOME_EMAIL_SLUGS));
 
 const AutomatedEmail = ghostBookshelf.Model.extend({
     tableName: 'automated_emails',
@@ -33,6 +37,36 @@ const AutomatedEmail = ghostBookshelf.Model.extend({
         }
 
         return attrs;
+    },
+
+    onSaved(model) {
+        if (!model?.id) {
+            return;
+        }
+
+        const slug = model.get('slug');
+
+        if (!MEMBER_WELCOME_EMAIL_SLUG_SET.has(slug)) {
+            return;
+        }
+
+        const previousStatus = model.previous('status');
+        const currentStatus = model.get('status');
+        const isNewModel = previousStatus === undefined;
+        const isEnableTransition = currentStatus === 'active' && (isNewModel || previousStatus === 'inactive');
+        const isDisableTransition = previousStatus === 'active' && currentStatus === 'inactive';
+
+        if (!isEnableTransition && !isDisableTransition) {
+            return;
+        }
+
+        logging.info({
+            system: {
+                event: isEnableTransition ? 'welcome_email.enabled' : 'welcome_email.disabled',
+                automated_email_id: model.id,
+                slug
+            }
+        }, isEnableTransition ? 'Welcome email enabled' : 'Welcome email disabled');
     }
 });
 

package/core/server/services/email-address/email-address-service-wrapper.js

@@ -33,6 +33,9 @@ class EmailAddressServiceWrapper {
             getFallbackEmail: () => {
                 return config.get('hostSettings:managedEmail:fallbackAddress') || null;
             },
+            getMembersSupportAddress: () => {
+                return settingsHelpers.getMembersSupportAddress();
+            },
             isValidEmailAddress: (emailAddress) => {
                 return validator.isEmail(emailAddress);
             }

package/core/server/services/email-address/email-address-service.js

@@ -12,12 +12,14 @@ class EmailAddressService {
     #getDefaultEmail;
     #getFallbackDomain;
     #getFallbackEmail;
+    #getMembersSupportAddress;
     #isValidEmailAddress;
     constructor(dependencies) {
         this.#getManagedEmailEnabled = dependencies.getManagedEmailEnabled;
         this.#getSendingDomain = dependencies.getSendingDomain;
         this.#getFallbackDomain = dependencies.getFallbackDomain;
         this.#getDefaultEmail = dependencies.getDefaultEmail;
+        this.#getMembersSupportAddress = dependencies.getMembersSupportAddress;
         this.#getFallbackEmail = () => {
             const fallbackAddress = dependencies.getFallbackEmail();
             if (!fallbackAddress) {
@@ -42,6 +44,15 @@ class EmailAddressService {
     get fallbackEmail() {
         return this.#getFallbackEmail();
     }
+    /**
+     * Get the actual sender address for member emails after DMARC transformation.
+     * On managed platforms, this may differ from the configured support address to ensure DMARC compliance.
+     */
+    getMembersSupportAddress() {
+        const configuredAddress = this.#getMembersSupportAddress();
+        const transformed = this.getAddressFromString(configuredAddress);
+        return transformed.from.address;
+    }
     getAddressFromString(from, replyTo) {
         const parsedFrom = email_address_parser_js_1.default.parse(from);
         const parsedReplyTo = replyTo ? email_address_parser_js_1.default.parse(replyTo) : undefined;