From a1cd6f3e42b124231ed12f28bb5619ded42d889a Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 16 Aug 2025 02:50:53 +0000
Subject: [PATCH 01/12] Initial plan


From 032658cca4a40428a535302642638d1e76dd26bd Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 16 Aug 2025 02:56:21 +0000
Subject: [PATCH 02/12] Fix weekly-report jq error when API returns non-array
 responses

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>
---
 .github/actions/weekly-report/generate-report.sh | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/.github/actions/weekly-report/generate-report.sh b/.github/actions/weekly-report/generate-report.sh
index b76ba4a..b7e83c8 100755
--- a/.github/actions/weekly-report/generate-report.sh
+++ b/.github/actions/weekly-report/generate-report.sh
@@ -105,15 +105,15 @@ while IFS= read -r repo; do
     
     # Count opened issues in the last week
     opened_issues=$(api_call "/repos/${ORGANIZATION}/${repo}/issues" | \
-        jq --arg since "$WEEK_AGO" '[.[] | select(.created_at >= $since and .pull_request == null)] | length')
+        jq --arg since "$WEEK_AGO" 'if type == "array" then [.[] | select(.created_at >= $since and .pull_request == null)] | length else 0 end')
     
     # Count closed issues in the last week
     closed_issues=$(api_call "/repos/${ORGANIZATION}/${repo}/issues?state=closed" | \
-        jq --arg since "$WEEK_AGO" '[.[] | select(.closed_at >= $since and .pull_request == null)] | length')
+        jq --arg since "$WEEK_AGO" 'if type == "array" then [.[] | select(.closed_at != null and .closed_at >= $since and .pull_request == null)] | length else 0 end')
     
     # Count merged PRs in the last week
     merged_prs=$(api_call "/repos/${ORGANIZATION}/${repo}/pulls?state=closed" | \
-        jq --arg since "$WEEK_AGO" '[.[] | select(.merged_at != null and .merged_at >= $since)] | length')
+        jq --arg since "$WEEK_AGO" 'if type == "array" then [.[] | select(.merged_at != null and .merged_at >= $since)] | length else 0 end')
     
     # Handle null/empty values
     opened_issues=${opened_issues:-0}

From 7812c9c132dd9102ed17866aa46c34679722eb24 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 16 Aug 2025 03:39:30 +0000
Subject: [PATCH 03/12] Add rate limit handling and API delay options to
 weekly-report action

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>
---
 .github/actions/weekly-report/README.md       |  16 ++-
 .github/actions/weekly-report/action.yml      |   7 +-
 .../actions/weekly-report/generate-report.sh  | 117 ++++++++++++++++--
 3 files changed, 127 insertions(+), 13 deletions(-)

diff --git a/.github/actions/weekly-report/README.md b/.github/actions/weekly-report/README.md
index bf5f92f..a8e4062 100644
--- a/.github/actions/weekly-report/README.md
+++ b/.github/actions/weekly-report/README.md
@@ -14,6 +14,8 @@ This action generates a report containing:
 - **Smart repository filtering**: Uses GitHub Search API to identify repositories with recent activity (commits in the last 7 days) before checking for issues and PRs
 - **Fallback mechanism**: If no repositories are found with recent commits, falls back to checking all organization repositories to ensure complete coverage
 - **Activity-based reporting**: Only includes repositories with actual activity in the generated report
+- **Rate limit handling**: Automatically retries on rate limit errors with exponential backoff, and provides clear warnings when data is incomplete
+- **Configurable delays**: Optional delays between API calls to reduce rate limit pressure
 
 ## Usage
 
@@ -25,6 +27,7 @@ This action generates a report containing:
     organization: 'QuantEcon'
     output-format: 'markdown'
     exclude-repos: 'lecture-python.notebooks,auto-updated-repo'
+    api-delay: '1'  # Add 1 second delay between API calls to avoid rate limits
 ```
 
 ## Inputs
@@ -35,6 +38,7 @@ This action generates a report containing:
 | `organization` | GitHub organization name | No | `QuantEcon` |
 | `output-format` | Output format (`markdown` or `json`) | No | `markdown` |
 | `exclude-repos` | Comma-separated list of repository names to exclude from the report | No | `''` |
+| `api-delay` | Delay in seconds between API calls to avoid rate limits (0 = no delay) | No | `0` |
 
 ## Outputs
 
@@ -59,6 +63,16 @@ See the [weekly report workflow](../../workflows/weekly-report.yml) for a comple
 The generated markdown report includes:
 - A summary table showing activity by repository
 - Total counts across all repositories
+- Data completeness warnings if API calls failed due to rate limits or other errors
 - Report metadata (generation date, period covered)
 
-Only repositories with activity in the reporting period are included in the detailed table.
\ No newline at end of file
+Only repositories with activity in the reporting period are included in the detailed table.
+
+## Rate Limiting
+
+GitHub's API has rate limits (5000 requests/hour for authenticated requests). For large organizations:
+
+- **Monitor warnings**: The report will include warnings when rate limits are hit
+- **Add delays**: Use the `api-delay` parameter to add delays between requests (e.g., `api-delay: '1'` for 1 second delays)
+- **Run during off-peak**: Schedule reports during off-peak hours to avoid conflicts with other API usage
+- **Incomplete data**: When rate limited, the report will show `0` for affected repositories and include a warning
\ No newline at end of file
diff --git a/.github/actions/weekly-report/action.yml b/.github/actions/weekly-report/action.yml
index cbe3e2b..361a4f1 100644
--- a/.github/actions/weekly-report/action.yml
+++ b/.github/actions/weekly-report/action.yml
@@ -18,6 +18,10 @@ inputs:
     description: 'Comma-separated list of repository names to exclude from the report'
     required: false
     default: ''
+  api-delay:
+    description: 'Delay in seconds between API calls to avoid rate limits (0 = no delay)'
+    required: false
+    default: '0'
 
 outputs:
   report-content:
@@ -35,4 +39,5 @@ runs:
         INPUT_GITHUB_TOKEN: ${{ inputs.github-token }}
         INPUT_ORGANIZATION: ${{ inputs.organization }}
         INPUT_OUTPUT_FORMAT: ${{ inputs.output-format }}
-        INPUT_EXCLUDE_REPOS: ${{ inputs.exclude-repos }}
\ No newline at end of file
+        INPUT_EXCLUDE_REPOS: ${{ inputs.exclude-repos }}
+        INPUT_API_DELAY: ${{ inputs.api-delay }}
\ No newline at end of file
diff --git a/.github/actions/weekly-report/generate-report.sh b/.github/actions/weekly-report/generate-report.sh
index b7e83c8..9f870f2 100755
--- a/.github/actions/weekly-report/generate-report.sh
+++ b/.github/actions/weekly-report/generate-report.sh
@@ -6,6 +6,7 @@ GITHUB_TOKEN="${INPUT_GITHUB_TOKEN}"
 ORGANIZATION="${INPUT_ORGANIZATION:-QuantEcon}"
 OUTPUT_FORMAT="${INPUT_OUTPUT_FORMAT:-markdown}"
 EXCLUDE_REPOS="${INPUT_EXCLUDE_REPOS:-}"
+API_DELAY="${INPUT_API_DELAY:-0}"  # Optional delay between API calls in seconds
 
 # Date calculations for last week
 WEEK_AGO=$(date -d "7 days ago" -u +"%Y-%m-%dT%H:%M:%SZ")
@@ -14,13 +15,64 @@ NOW=$(date -u +"%Y-%m-%dT%H:%M:%SZ")
 echo "Generating weekly report for ${ORGANIZATION} organization"
 echo "Period: ${WEEK_AGO} to ${NOW}"
 
-# Function to make GitHub API calls
+# Function to make GitHub API calls with rate limit handling
 api_call() {
     local endpoint="$1"
     local page="${2:-1}"
-    curl -s -H "Authorization: token ${GITHUB_TOKEN}" \
-         -H "Accept: application/vnd.github.v3+json" \
-         "https://api.github.com${endpoint}?page=${page}&per_page=100"
+    local max_retries=3
+    local retry_count=0
+    local delay="${API_DELAY:-0}"
+    
+    # Add delay between requests if specified
+    if [ "$delay" -gt 0 ]; then
+        sleep "$delay"
+    fi
+    
+    while [ $retry_count -lt $max_retries ]; do
+        local response=$(curl -s -w "\n%{http_code}" -H "Authorization: token ${GITHUB_TOKEN}" \
+                            -H "Accept: application/vnd.github.v3+json" \
+                            "https://api.github.com${endpoint}?page=${page}&per_page=100")
+        
+        local http_code=$(echo "$response" | tail -n1)
+        local body=$(echo "$response" | head -n -1)
+        
+        case "$http_code" in
+            200)
+                echo "$body"
+                return 0
+                ;;
+            403)
+                # Check if it's a rate limit error
+                if echo "$body" | jq -e '.message' 2>/dev/null | grep -q "rate limit"; then
+                    retry_count=$((retry_count + 1))
+                    if [ $retry_count -lt $max_retries ]; then
+                        local wait_time=$((retry_count * retry_count * 60))  # Exponential backoff: 1min, 4min, 9min
+                        echo "Rate limit exceeded for $endpoint. Waiting ${wait_time}s before retry $retry_count/$max_retries..." >&2
+                        sleep "$wait_time"
+                        continue
+                    else
+                        echo "Rate limit exceeded for $endpoint after $max_retries retries. Data will be incomplete." >&2
+                        echo '{"error": "rate_limit_exceeded", "message": "API rate limit exceeded"}'
+                        return 1
+                    fi
+                else
+                    echo "Access forbidden for $endpoint: $body" >&2
+                    echo '{"error": "forbidden", "message": "Access forbidden"}'
+                    return 1
+                fi
+                ;;
+            404)
+                echo "Repository not found: $endpoint" >&2
+                echo '{"error": "not_found", "message": "Repository not found"}'
+                return 1
+                ;;
+            *)
+                echo "API call failed for $endpoint with status $http_code: $body" >&2
+                echo '{"error": "api_error", "message": "API call failed"}'
+                return 1
+                ;;
+        esac
+    done
 }
 
 # Get repositories with recent activity using GitHub Search API
@@ -86,6 +138,8 @@ fi
 total_opened_issues=0
 total_closed_issues=0
 total_merged_prs=0
+failed_repos=0
+rate_limited_repos=0
 report_content=""
 
 # Start building the report
@@ -104,16 +158,43 @@ while IFS= read -r repo; do
     echo "Processing repository: $repo"
     
     # Count opened issues in the last week
-    opened_issues=$(api_call "/repos/${ORGANIZATION}/${repo}/issues" | \
-        jq --arg since "$WEEK_AGO" 'if type == "array" then [.[] | select(.created_at >= $since and .pull_request == null)] | length else 0 end')
+    opened_response=$(api_call "/repos/${ORGANIZATION}/${repo}/issues")
+    if [ $? -eq 0 ]; then
+        opened_issues=$(echo "$opened_response" | jq --arg since "$WEEK_AGO" 'if type == "array" then [.[] | select(.created_at >= $since and .pull_request == null)] | length else 0 end')
+    else
+        opened_issues=0
+        if echo "$opened_response" | jq -e '.error' 2>/dev/null | grep -q "rate_limit"; then
+            rate_limited_repos=$((rate_limited_repos + 1))
+        else
+            failed_repos=$((failed_repos + 1))
+        fi
+    fi
     
     # Count closed issues in the last week
-    closed_issues=$(api_call "/repos/${ORGANIZATION}/${repo}/issues?state=closed" | \
-        jq --arg since "$WEEK_AGO" 'if type == "array" then [.[] | select(.closed_at != null and .closed_at >= $since and .pull_request == null)] | length else 0 end')
+    closed_response=$(api_call "/repos/${ORGANIZATION}/${repo}/issues?state=closed")
+    if [ $? -eq 0 ]; then
+        closed_issues=$(echo "$closed_response" | jq --arg since "$WEEK_AGO" 'if type == "array" then [.[] | select(.closed_at != null and .closed_at >= $since and .pull_request == null)] | length else 0 end')
+    else
+        closed_issues=0
+        if echo "$closed_response" | jq -e '.error' 2>/dev/null | grep -q "rate_limit"; then
+            rate_limited_repos=$((rate_limited_repos + 1))
+        else
+            failed_repos=$((failed_repos + 1))
+        fi
+    fi
     
     # Count merged PRs in the last week
-    merged_prs=$(api_call "/repos/${ORGANIZATION}/${repo}/pulls?state=closed" | \
-        jq --arg since "$WEEK_AGO" 'if type == "array" then [.[] | select(.merged_at != null and .merged_at >= $since)] | length else 0 end')
+    prs_response=$(api_call "/repos/${ORGANIZATION}/${repo}/pulls?state=closed")
+    if [ $? -eq 0 ]; then
+        merged_prs=$(echo "$prs_response" | jq --arg since "$WEEK_AGO" 'if type == "array" then [.[] | select(.merged_at != null and .merged_at >= $since)] | length else 0 end')
+    else
+        merged_prs=0
+        if echo "$prs_response" | jq -e '.error' 2>/dev/null | grep -q "rate_limit"; then
+            rate_limited_repos=$((rate_limited_repos + 1))
+        else
+            failed_repos=$((failed_repos + 1))
+        fi
+    fi
     
     # Handle null/empty values
     opened_issues=${opened_issues:-0}
@@ -141,7 +222,21 @@ if [ "$OUTPUT_FORMAT" = "markdown" ]; then
     report_content+="- **Total Repositories Checked:** $(echo "$repo_names" | wc -l)\n"
     report_content+="- **Total Issues Opened:** $total_opened_issues\n"
     report_content+="- **Total Issues Closed:** $total_closed_issues\n"
-    report_content+="- **Total PRs Merged:** $total_merged_prs\n\n"
+    report_content+="- **Total PRs Merged:** $total_merged_prs\n"
+    
+    # Add warnings about incomplete data if any API calls failed
+    if [ $rate_limited_repos -gt 0 ] || [ $failed_repos -gt 0 ]; then
+        report_content+="\n### ⚠️ Data Completeness Warnings\n\n"
+        if [ $rate_limited_repos -gt 0 ]; then
+            report_content+="- **Rate Limited:** $rate_limited_repos API calls hit rate limits. Data may be incomplete.\n"
+        fi
+        if [ $failed_repos -gt 0 ]; then
+            report_content+="- **Failed Requests:** $failed_repos API calls failed. Data may be incomplete.\n"
+        fi
+        report_content+="\n*Consider adding API delays or running during off-peak hours to avoid rate limits.*\n"
+    fi
+    
+    report_content+="\n"
     report_content+="*Report generated on $(date) by QuantEcon Weekly Report Action*\n"
 fi
 

From a9c7cd11df121fb9562c2f8c590d8745d308ec31 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 16 Aug 2025 05:08:45 +0000
Subject: [PATCH 04/12] Fix URL construction bug causing 422 errors for
 endpoints with query parameters

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>
---
 .github/actions/weekly-report/generate-report.sh | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/.github/actions/weekly-report/generate-report.sh b/.github/actions/weekly-report/generate-report.sh
index 9f870f2..371d061 100755
--- a/.github/actions/weekly-report/generate-report.sh
+++ b/.github/actions/weekly-report/generate-report.sh
@@ -29,9 +29,17 @@ api_call() {
     fi
     
     while [ $retry_count -lt $max_retries ]; do
+        # Construct URL with proper query parameter handling
+        local url="https://api.github.com${endpoint}"
+        if [[ "$endpoint" == *"?"* ]]; then
+            url="${url}&page=${page}&per_page=100"
+        else
+            url="${url}?page=${page}&per_page=100"
+        fi
+        
         local response=$(curl -s -w "\n%{http_code}" -H "Authorization: token ${GITHUB_TOKEN}" \
                             -H "Accept: application/vnd.github.v3+json" \
-                            "https://api.github.com${endpoint}?page=${page}&per_page=100")
+                            "$url")
         
         local http_code=$(echo "$response" | tail -n1)
         local body=$(echo "$response" | head -n -1)

From fa264a1733e8be6f47f18315f6ca33406e4f2b25 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 16 Aug 2025 05:18:30 +0000
Subject: [PATCH 05/12] Fix issue content empty bug in weekly-report workflow
 by properly escaping markdown content

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>
---
 .github/workflows/weekly-report.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/weekly-report.yml b/.github/workflows/weekly-report.yml
index 3e2fc15..8ca2296 100644
--- a/.github/workflows/weekly-report.yml
+++ b/.github/workflows/weekly-report.yml
@@ -33,8 +33,8 @@ jobs:
         uses: actions/github-script@v7
         with:
           script: |
-            const reportContent = `${{ steps.report.outputs.report-content }}`;
-            const summary = `${{ steps.report.outputs.report-summary }}`;
+            const reportContent = JSON.parse(${{ toJSON(steps.report.outputs.report-content) }});
+            const summary = ${{ toJSON(steps.report.outputs.report-summary) }};
             
             // Create issue title with current date
             const now = new Date();

From 37cd9ae475154cb88a18a1ba630ffc6786661ce0 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 16 Aug 2025 05:27:24 +0000
Subject: [PATCH 06/12] Fix issue content bug: remove incorrect JSON.parse()
 call in GitHub script action

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>
---
 .github/workflows/weekly-report.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/weekly-report.yml b/.github/workflows/weekly-report.yml
index 8ca2296..6c5282b 100644
--- a/.github/workflows/weekly-report.yml
+++ b/.github/workflows/weekly-report.yml
@@ -33,7 +33,7 @@ jobs:
         uses: actions/github-script@v7
         with:
           script: |
-            const reportContent = JSON.parse(${{ toJSON(steps.report.outputs.report-content) }});
+            const reportContent = ${{ toJSON(steps.report.outputs.report-content) }};
             const summary = ${{ toJSON(steps.report.outputs.report-summary) }};
             
             // Create issue title with current date

From 0cb6f30b42aad73513d28ff2a6bc5570dc69d901 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 16 Aug 2025 05:38:25 +0000
Subject: [PATCH 07/12] Fix bash string concatenation in weekly-report action
 for proper output handling

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>
---
 .../actions/weekly-report/generate-report.sh  | 55 ++++++++++++-------
 1 file changed, 35 insertions(+), 20 deletions(-)

diff --git a/.github/actions/weekly-report/generate-report.sh b/.github/actions/weekly-report/generate-report.sh
index 371d061..535b897 100755
--- a/.github/actions/weekly-report/generate-report.sh
+++ b/.github/actions/weekly-report/generate-report.sh
@@ -152,11 +152,14 @@ report_content=""
 
 # Start building the report
 if [ "$OUTPUT_FORMAT" = "markdown" ]; then
-    report_content="# QuantEcon Weekly Report\n\n"
-    report_content+="**Report Period:** $(date -d "$WEEK_AGO" '+%B %d, %Y') - $(date -d "$NOW" '+%B %d, %Y')\n\n"
-    report_content+="## Summary\n\n"
-    report_content+="| Repository | Opened Issues | Closed Issues | Merged PRs |\n"
-    report_content+="|------------|---------------|---------------|------------|\n"
+    report_content="# QuantEcon Weekly Report
+
+**Report Period:** $(date -d "$WEEK_AGO" '+%B %d, %Y') - $(date -d "$NOW" '+%B %d, %Y')
+
+## Summary
+
+| Repository | Opened Issues | Closed Issues | Merged PRs |
+|------------|---------------|---------------|------------|"
 fi
 
 # Process each repository
@@ -217,7 +220,8 @@ while IFS= read -r repo; do
     # Add to report if there's activity
     if [ $((opened_issues + closed_issues + merged_prs)) -gt 0 ]; then
         if [ "$OUTPUT_FORMAT" = "markdown" ]; then
-            report_content+="| $repo | $opened_issues | $closed_issues | $merged_prs |\n"
+            report_content="${report_content}
+| $repo | $opened_issues | $closed_issues | $merged_prs |"
         fi
     fi
     
@@ -225,38 +229,49 @@ done <<< "$repo_names"
 
 # Add summary to report
 if [ "$OUTPUT_FORMAT" = "markdown" ]; then
-    report_content+="|**Total**|**$total_opened_issues**|**$total_closed_issues**|**$total_merged_prs**|\n\n"
-    report_content+="## Details\n\n"
-    report_content+="- **Total Repositories Checked:** $(echo "$repo_names" | wc -l)\n"
-    report_content+="- **Total Issues Opened:** $total_opened_issues\n"
-    report_content+="- **Total Issues Closed:** $total_closed_issues\n"
-    report_content+="- **Total PRs Merged:** $total_merged_prs\n"
+    report_content="${report_content}
+|**Total**|**$total_opened_issues**|**$total_closed_issues**|**$total_merged_prs**|
+
+## Details
+
+- **Total Repositories Checked:** $(echo "$repo_names" | wc -l)
+- **Total Issues Opened:** $total_opened_issues
+- **Total Issues Closed:** $total_closed_issues
+- **Total PRs Merged:** $total_merged_prs"
     
     # Add warnings about incomplete data if any API calls failed
     if [ $rate_limited_repos -gt 0 ] || [ $failed_repos -gt 0 ]; then
-        report_content+="\n### ⚠️ Data Completeness Warnings\n\n"
+        report_content="${report_content}
+
+### ⚠️ Data Completeness Warnings
+"
         if [ $rate_limited_repos -gt 0 ]; then
-            report_content+="- **Rate Limited:** $rate_limited_repos API calls hit rate limits. Data may be incomplete.\n"
+            report_content="${report_content}
+- **Rate Limited:** $rate_limited_repos API calls hit rate limits. Data may be incomplete."
         fi
         if [ $failed_repos -gt 0 ]; then
-            report_content+="- **Failed Requests:** $failed_repos API calls failed. Data may be incomplete.\n"
+            report_content="${report_content}
+- **Failed Requests:** $failed_repos API calls failed. Data may be incomplete."
         fi
-        report_content+="\n*Consider adding API delays or running during off-peak hours to avoid rate limits.*\n"
+        report_content="${report_content}
+
+*Consider adding API delays or running during off-peak hours to avoid rate limits.*"
     fi
     
-    report_content+="\n"
-    report_content+="*Report generated on $(date) by QuantEcon Weekly Report Action*\n"
+    report_content="${report_content}
+
+*Report generated on $(date) by QuantEcon Weekly Report Action*"
 fi
 
 # Create summary
 summary="Week Summary: $total_opened_issues issues opened, $total_closed_issues issues closed, $total_merged_prs PRs merged"
 
 # Save report to file
-echo -e "$report_content" > weekly-report.md
+echo "$report_content" > weekly-report.md
 
 # Set outputs
 echo "report-content<<EOF" >> $GITHUB_OUTPUT
-echo -e "$report_content" >> $GITHUB_OUTPUT
+echo "$report_content" >> $GITHUB_OUTPUT
 echo "EOF" >> $GITHUB_OUTPUT
 
 echo "report-summary=$summary" >> $GITHUB_OUTPUT

From 7911f5e16f50a56b15d6a66d43bd2a98072a7259 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 16 Aug 2025 05:47:17 +0000
Subject: [PATCH 08/12] Add comprehensive debugging to weekly-report action to
 diagnose null output issue

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>
---
 .../actions/weekly-report/generate-report.sh  | 32 ++++++++++++++++---
 1 file changed, 27 insertions(+), 5 deletions(-)

diff --git a/.github/actions/weekly-report/generate-report.sh b/.github/actions/weekly-report/generate-report.sh
index 535b897..d6d11a2 100755
--- a/.github/actions/weekly-report/generate-report.sh
+++ b/.github/actions/weekly-report/generate-report.sh
@@ -1,6 +1,9 @@
 #!/bin/bash
 set -e
 
+echo "DEBUG: Starting weekly report generation"
+echo "DEBUG: Environment check - GITHUB_OUTPUT: ${GITHUB_OUTPUT:-NOT_SET}"
+
 # Get inputs
 GITHUB_TOKEN="${INPUT_GITHUB_TOKEN}"
 ORGANIZATION="${INPUT_ORGANIZATION:-QuantEcon}"
@@ -8,6 +11,8 @@ OUTPUT_FORMAT="${INPUT_OUTPUT_FORMAT:-markdown}"
 EXCLUDE_REPOS="${INPUT_EXCLUDE_REPOS:-}"
 API_DELAY="${INPUT_API_DELAY:-0}"  # Optional delay between API calls in seconds
 
+echo "DEBUG: Inputs - ORG: $ORGANIZATION, FORMAT: $OUTPUT_FORMAT, EXCLUDE: $EXCLUDE_REPOS"
+
 # Date calculations for last week
 WEEK_AGO=$(date -d "7 days ago" -u +"%Y-%m-%dT%H:%M:%SZ")
 NOW=$(date -u +"%Y-%m-%dT%H:%M:%SZ")
@@ -160,11 +165,14 @@ if [ "$OUTPUT_FORMAT" = "markdown" ]; then
 
 | Repository | Opened Issues | Closed Issues | Merged PRs |
 |------------|---------------|---------------|------------|"
+    echo "DEBUG: Initial report content set, length: ${#report_content}"
 fi
 
 # Process each repository
+repo_count=0
 while IFS= read -r repo; do
     [ -z "$repo" ] && continue
+    repo_count=$((repo_count + 1))
     
     echo "Processing repository: $repo"
     
@@ -227,6 +235,9 @@ while IFS= read -r repo; do
     
 done <<< "$repo_names"
 
+echo "DEBUG: Processed $repo_count repositories"
+echo "DEBUG: Final report content length: ${#report_content}"
+
 # Add summary to report
 if [ "$OUTPUT_FORMAT" = "markdown" ]; then
     report_content="${report_content}
@@ -269,12 +280,23 @@ summary="Week Summary: $total_opened_issues issues opened, $total_closed_issues
 # Save report to file
 echo "$report_content" > weekly-report.md
 
-# Set outputs
-echo "report-content<<EOF" >> $GITHUB_OUTPUT
-echo "$report_content" >> $GITHUB_OUTPUT
-echo "EOF" >> $GITHUB_OUTPUT
+# Debug: Check if GITHUB_OUTPUT is set and accessible
+echo "DEBUG: GITHUB_OUTPUT environment variable: ${GITHUB_OUTPUT:-NOT_SET}"
+echo "DEBUG: Report content length: ${#report_content}"
+echo "DEBUG: Summary: $summary"
 
-echo "report-summary=$summary" >> $GITHUB_OUTPUT
+# Set outputs
+if [ -n "$GITHUB_OUTPUT" ]; then
+    echo "DEBUG: Writing to GITHUB_OUTPUT file"
+    echo "report-content<<EOF" >> "$GITHUB_OUTPUT"
+    echo "$report_content" >> "$GITHUB_OUTPUT"
+    echo "EOF" >> "$GITHUB_OUTPUT"
+    
+    echo "report-summary=$summary" >> "$GITHUB_OUTPUT"
+    echo "DEBUG: Outputs written to GITHUB_OUTPUT"
+else
+    echo "ERROR: GITHUB_OUTPUT environment variable not set!"
+fi
 
 echo "Weekly report generated successfully!"
 echo "Summary: $summary"
\ No newline at end of file

From 3b77d9f4296638e5ff4c3d84f9c643dbcd6a06ad Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 16 Aug 2025 05:58:48 +0000
Subject: [PATCH 09/12] Fix GITHUB_OUTPUT multiline content handling with
 unique delimiter to prevent conflicts

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>
---
 .github/actions/weekly-report/generate-report.sh | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/.github/actions/weekly-report/generate-report.sh b/.github/actions/weekly-report/generate-report.sh
index d6d11a2..fc3ba04 100755
--- a/.github/actions/weekly-report/generate-report.sh
+++ b/.github/actions/weekly-report/generate-report.sh
@@ -288,12 +288,19 @@ echo "DEBUG: Summary: $summary"
 # Set outputs
 if [ -n "$GITHUB_OUTPUT" ]; then
     echo "DEBUG: Writing to GITHUB_OUTPUT file"
-    echo "report-content<<EOF" >> "$GITHUB_OUTPUT"
+    echo "DEBUG: Content preview (first 100 chars): ${report_content:0:100}"
+    echo "DEBUG: Summary preview: $summary"
+    
+    # Use a unique delimiter to avoid conflicts with content
+    delimiter="QUANTECON_REPORT_END_$(date +%s)"
+    echo "report-content<<${delimiter}" >> "$GITHUB_OUTPUT"
     echo "$report_content" >> "$GITHUB_OUTPUT"
-    echo "EOF" >> "$GITHUB_OUTPUT"
+    echo "${delimiter}" >> "$GITHUB_OUTPUT"
     
     echo "report-summary=$summary" >> "$GITHUB_OUTPUT"
+    
     echo "DEBUG: Outputs written to GITHUB_OUTPUT"
+    echo "DEBUG: GITHUB_OUTPUT file size: $(wc -c < "$GITHUB_OUTPUT")"
 else
     echo "ERROR: GITHUB_OUTPUT environment variable not set!"
 fi

From 679679aa5029c15edec2dbe79e5fa89c653b6fb6 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 16 Aug 2025 06:07:23 +0000
Subject: [PATCH 10/12] Fix issue content delivery by reading markdown file
 directly instead of using step outputs

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>
---
 .github/workflows/weekly-report.yml | 15 ++++++++++++---
 test-report.md                      |  8 ++++++++
 2 files changed, 20 insertions(+), 3 deletions(-)
 create mode 100644 test-report.md

diff --git a/.github/workflows/weekly-report.yml b/.github/workflows/weekly-report.yml
index 6c5282b..335276c 100644
--- a/.github/workflows/weekly-report.yml
+++ b/.github/workflows/weekly-report.yml
@@ -33,8 +33,17 @@ jobs:
         uses: actions/github-script@v7
         with:
           script: |
-            const reportContent = ${{ toJSON(steps.report.outputs.report-content) }};
-            const summary = ${{ toJSON(steps.report.outputs.report-summary) }};
+            const fs = require('fs');
+            
+            // Read the report content directly from the generated file
+            let reportContent = '';
+            try {
+              reportContent = fs.readFileSync('weekly-report.md', 'utf8');
+              console.log(`Successfully read report file, length: ${reportContent.length}`);
+            } catch (error) {
+              console.error('Failed to read weekly-report.md:', error);
+              reportContent = 'Error: Unable to load weekly report content.';
+            }
             
             // Create issue title with current date
             const now = new Date();
@@ -55,7 +64,7 @@ jobs:
             });
             
             console.log(`Created issue: ${title}`);
-            console.log(`Summary: ${summary}`);
+            console.log(`Report content length: ${reportContent.length}`);
       
       - name: Upload report as artifact
         uses: actions/upload-artifact@v4
diff --git a/test-report.md b/test-report.md
new file mode 100644
index 0000000..97d4caa
--- /dev/null
+++ b/test-report.md
@@ -0,0 +1,8 @@
+# Test Report
+This is a test with **markdown** formatting.
+
+## Summary
+- Item 1
+- Item 2
+
+*Generated on Sat Aug 16 2025 06:07:06 GMT+0000 (Coordinated Universal Time)*
\ No newline at end of file

From 7bd4506f3a21adb6806950fe5c1ea266625bcb3d Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 16 Aug 2025 06:07:43 +0000
Subject: [PATCH 11/12] Remove accidentally committed test file

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>
---
 test-report.md | 8 --------
 1 file changed, 8 deletions(-)
 delete mode 100644 test-report.md

diff --git a/test-report.md b/test-report.md
deleted file mode 100644
index 97d4caa..0000000
--- a/test-report.md
+++ /dev/null
@@ -1,8 +0,0 @@
-# Test Report
-This is a test with **markdown** formatting.
-
-## Summary
-- Item 1
-- Item 2
-
-*Generated on Sat Aug 16 2025 06:07:06 GMT+0000 (Coordinated Universal Time)*
\ No newline at end of file

From 2b6bb76f6e4ea696bdf3355ac8af333a0c332e3c Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 16 Aug 2025 06:18:13 +0000
Subject: [PATCH 12/12] Add total current issues column to weekly report table

Co-authored-by: mmcky <8263752+mmcky@users.noreply.github.com>
---
 .../actions/weekly-report/generate-report.sh  | 31 ++++++++++++++-----
 1 file changed, 24 insertions(+), 7 deletions(-)

diff --git a/.github/actions/weekly-report/generate-report.sh b/.github/actions/weekly-report/generate-report.sh
index fc3ba04..8cd7bb4 100755
--- a/.github/actions/weekly-report/generate-report.sh
+++ b/.github/actions/weekly-report/generate-report.sh
@@ -148,6 +148,7 @@ if [ -n "$EXCLUDE_REPOS" ]; then
 fi
 
 # Initialize report variables
+total_current_issues=0
 total_opened_issues=0
 total_closed_issues=0
 total_merged_prs=0
@@ -163,8 +164,8 @@ if [ "$OUTPUT_FORMAT" = "markdown" ]; then
 
 ## Summary
 
-| Repository | Opened Issues | Closed Issues | Merged PRs |
-|------------|---------------|---------------|------------|"
+| Repository | Total Current Issues | Opened Issues | Closed Issues | Merged PRs |
+|------------|---------------------|---------------|---------------|------------|"
     echo "DEBUG: Initial report content set, length: ${#report_content}"
 fi
 
@@ -176,6 +177,19 @@ while IFS= read -r repo; do
     
     echo "Processing repository: $repo"
     
+    # Count total current open issues
+    current_issues_response=$(api_call "/repos/${ORGANIZATION}/${repo}/issues?state=open")
+    if [ $? -eq 0 ]; then
+        current_issues=$(echo "$current_issues_response" | jq 'if type == "array" then [.[] | select(.pull_request == null)] | length else 0 end')
+    else
+        current_issues=0
+        if echo "$current_issues_response" | jq -e '.error' 2>/dev/null | grep -q "rate_limit"; then
+            rate_limited_repos=$((rate_limited_repos + 1))
+        else
+            failed_repos=$((failed_repos + 1))
+        fi
+    fi
+    
     # Count opened issues in the last week
     opened_response=$(api_call "/repos/${ORGANIZATION}/${repo}/issues")
     if [ $? -eq 0 ]; then
@@ -216,20 +230,22 @@ while IFS= read -r repo; do
     fi
     
     # Handle null/empty values
+    current_issues=${current_issues:-0}
     opened_issues=${opened_issues:-0}
     closed_issues=${closed_issues:-0}
     merged_prs=${merged_prs:-0}
     
     # Add to totals
+    total_current_issues=$((total_current_issues + current_issues))
     total_opened_issues=$((total_opened_issues + opened_issues))
     total_closed_issues=$((total_closed_issues + closed_issues))
     total_merged_prs=$((total_merged_prs + merged_prs))
     
-    # Add to report if there's activity
-    if [ $((opened_issues + closed_issues + merged_prs)) -gt 0 ]; then
+    # Add to report if there's activity or current open issues
+    if [ $((current_issues + opened_issues + closed_issues + merged_prs)) -gt 0 ]; then
         if [ "$OUTPUT_FORMAT" = "markdown" ]; then
             report_content="${report_content}
-| $repo | $opened_issues | $closed_issues | $merged_prs |"
+| $repo | $current_issues | $opened_issues | $closed_issues | $merged_prs |"
         fi
     fi
     
@@ -241,11 +257,12 @@ echo "DEBUG: Final report content length: ${#report_content}"
 # Add summary to report
 if [ "$OUTPUT_FORMAT" = "markdown" ]; then
     report_content="${report_content}
-|**Total**|**$total_opened_issues**|**$total_closed_issues**|**$total_merged_prs**|
+|**Total**|**$total_current_issues**|**$total_opened_issues**|**$total_closed_issues**|**$total_merged_prs**|
 
 ## Details
 
 - **Total Repositories Checked:** $(echo "$repo_names" | wc -l)
+- **Total Current Open Issues:** $total_current_issues
 - **Total Issues Opened:** $total_opened_issues
 - **Total Issues Closed:** $total_closed_issues
 - **Total PRs Merged:** $total_merged_prs"
@@ -275,7 +292,7 @@ if [ "$OUTPUT_FORMAT" = "markdown" ]; then
 fi
 
 # Create summary
-summary="Week Summary: $total_opened_issues issues opened, $total_closed_issues issues closed, $total_merged_prs PRs merged"
+summary="Week Summary: $total_current_issues current open issues, $total_opened_issues issues opened, $total_closed_issues issues closed, $total_merged_prs PRs merged"
 
 # Save report to file
 echo "$report_content" > weekly-report.md