From 4fdc53eed7063c5c55a11c33e4e17d12c488d417 Mon Sep 17 00:00:00 2001
From: Faisal Ahammad <faisalahammad24@gmail.com>
Date: Thu, 12 Feb 2026 23:09:06 +0600
Subject: [PATCH] Add error count summary to plugin check UI

- Add countResults() function to count total errors and warnings
- Update renderResultsMessage() to display detailed counts
- Show messages like '3 errors and 2 warnings found' instead of generic 'Errors were found'
- Implement proper pluralization (1 error vs 2 errors)
- Maintain fallback to default message if no counts available

Fixes #1113
---
 assets/js/plugin-check-admin.js | 76 +++++++++++++++++++++++++++++++--
 1 file changed, 73 insertions(+), 3 deletions(-)

diff --git a/assets/js/plugin-check-admin.js b/assets/js/plugin-check-admin.js
index f067d600e..17b56d26f 100644
--- a/assets/js/plugin-check-admin.js
+++ b/assets/js/plugin-check-admin.js
@@ -218,6 +218,46 @@
 		return false;
 	}
 
+	/**
+	 * Counts the total number of errors and warnings in the aggregated results.
+	 *
+	 * @since 1.0.0
+	 *
+	 * @return {Object} Object with errorCount and warningCount properties.
+	 */
+	function countResults() {
+		let errorCount = 0;
+		let warningCount = 0;
+
+		// Count errors.
+		for ( const file of Object.keys( aggregatedResults.errors ) ) {
+			const lines = aggregatedResults.errors[ file ] || {};
+
+			for ( const line of Object.keys( lines ) ) {
+				const columns = lines[ line ] || {};
+
+				for ( const column of Object.keys( columns ) ) {
+					errorCount += ( columns[ column ] || [] ).length;
+				}
+			}
+		}
+
+		// Count warnings.
+		for ( const file of Object.keys( aggregatedResults.warnings ) ) {
+			const lines = aggregatedResults.warnings[ file ] || {};
+
+			for ( const line of Object.keys( lines ) ) {
+				const columns = lines[ line ] || {};
+
+				for ( const column of Object.keys( columns ) ) {
+					warningCount += ( columns[ column ] || [] ).length;
+				}
+			}
+		}
+
+		return { errorCount, warningCount };
+	}
+
 	function defaultString( key ) {
 		if (
 			pluginCheck.strings &&
@@ -545,9 +585,39 @@
 	 */
 	function renderResultsMessage( isSuccessMessage ) {
 		const messageType = isSuccessMessage ? 'success' : 'error';
-		const messageText = isSuccessMessage
-			? pluginCheck.successMessage
-			: pluginCheck.errorMessage;
+		let messageText;
+
+		if ( isSuccessMessage ) {
+			messageText = pluginCheck.successMessage;
+		} else {
+			// Count errors and warnings.
+			const { errorCount, warningCount } = countResults();
+
+			// Build the message with counts.
+			const errorPart =
+				errorCount > 0
+					? errorCount === 1
+						? '1 error'
+						: errorCount + ' errors'
+					: '';
+			const warningPart =
+				warningCount > 0
+					? warningCount === 1
+						? '1 warning'
+						: warningCount + ' warnings'
+					: '';
+
+			if ( errorPart && warningPart ) {
+				messageText = errorPart + ' and ' + warningPart + ' found.';
+			} else if ( errorPart ) {
+				messageText = errorPart + ' found.';
+			} else if ( warningPart ) {
+				messageText = warningPart + ' found.';
+			} else {
+				// Fallback to default message if somehow no errors/warnings.
+				messageText = pluginCheck.errorMessage;
+			}
+		}
 
 		resultsContainer.innerHTML =
 			renderTemplate( 'plugin-check-results-complete', {