From 2358a052f1cc7f28f67a2fee19b844ac4d297582 Mon Sep 17 00:00:00 2001
From: John McLear <john@mclear.co.uk>
Date: Tue, 31 Mar 2026 11:15:41 +0100
Subject: [PATCH] Improve update-plugins workflow resilience and add summary
 (#7407)

Continue processing remaining plugins when one fails instead of
crashing. Add summary at the end showing succeeded/failed/skipped
counts and plugin names.

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 .github/workflows/update-plugins.yml | 26 +++++++++++++++++++++-----
 1 file changed, 21 insertions(+), 5 deletions(-)

diff --git a/.github/workflows/update-plugins.yml b/.github/workflows/update-plugins.yml
index 7c57eea02..fdccc8e1f 100644
--- a/.github/workflows/update-plugins.yml
+++ b/.github/workflows/update-plugins.yml
@@ -43,6 +43,10 @@ jobs:
           # List all ep_* repos from ether org
           plugins=$(gh repo list ether --limit 200 --json name --jq '.[] | select(.name | startswith("ep_")) | .name')
 
+          failed=""
+          succeeded=""
+          skipped=""
+
           for plugin in $plugins; do
             echo "============================================================"
             echo "Processing $plugin"
@@ -50,14 +54,26 @@ jobs:
 
             # Clone if not present
             if [ ! -d "$plugin" ]; then
-              git clone "https://github.com/ether/${plugin}.git" "$plugin" || { echo "SKIP: clone failed"; continue; }
+              git clone "https://github.com/ether/${plugin}.git" "$plugin" || { echo "SKIP: clone failed"; skipped="$skipped $plugin"; continue; }
             fi
 
             # Pull latest
-            (cd "$plugin" && git pull --ff-only) || { echo "SKIP: pull failed"; continue; }
+            (cd "$plugin" && git pull --ff-only) || { echo "SKIP: pull failed"; skipped="$skipped $plugin"; continue; }
 
-            # Run checkPlugin with autopush
-            cd etherpad-lite/bin
-            pnpm run checkPlugin "$plugin" autopush 2>&1 | tail -20 || echo "WARN: checkPlugin failed for $plugin"
+            # Run checkPlugin with autopush — continue on failure
+            if cd etherpad-lite/bin && pnpm run checkPlugin "$plugin" autopush 2>&1; then
+              succeeded="$succeeded $plugin"
+            else
+              echo "WARN: checkPlugin failed for $plugin"
+              failed="$failed $plugin"
+            fi
             cd ../..
           done
+
+          echo ""
+          echo "============================================================"
+          echo "SUMMARY"
+          echo "============================================================"
+          echo "Succeeded:$(echo $succeeded | wc -w) -$succeeded"
+          echo "Failed:$(echo $failed | wc -w) -$failed"
+          echo "Skipped:$(echo $skipped | wc -w) -$skipped"