Site Health: Cache the full results and a timestamp in the status transient

src/js/_enqueues/admin/site-health.js

@@ -156,10 +156,14 @@ jQuery( function( $ ) {
 
 		count = SiteHealth.site_status.issues[ issue.status ];
 
-		// If no test name is supplied, append a placeholder for markup references.
-		if ( typeof issue.test === 'undefined' ) {
-			issue.test = issue.status + count;
-		}
+		/*
+		 * Collect the test name and status so they can be cached server-side. These
+		 * include the asynchronous tests that only run in the browser. Labels are left
+		 * out on purpose, as they are translated and the cache is shared across locales.
+		 */
+		SiteHealth.site_status.results[ issue.test ] = {
+			status: issue.status
+		};
 
 		if ( 'critical' === issue.status ) {
 			heading = sprintf(
@@ -250,6 +254,7 @@ jQuery( function( $ ) {
 		}
 
 		if ( isStatusTab ) {
+			// Refresh the lightweight counts first, so a large detailed payload can't block them.
 			$.post(
 				ajaxurl,
 				{
@@ -259,6 +264,18 @@ jQuery( function( $ ) {
 				}
 			);
 
+			// Send the per-test results separately, as a best-effort detailed cache update.
+			if ( Object.keys( SiteHealth.site_status.results ).length > 0 ) {
+				$.post(
+					ajaxurl,
+					{
+						'action': 'health-check-site-status-result',
+						'_wpnonce': SiteHealth.nonce.site_status_result,
+						'results': JSON.stringify( SiteHealth.site_status.results )
+					}
+				);
+			}
+
 			if ( 100 === val ) {
 				$( '.site-status-all-clear' ).removeClass( 'hide' );
 				$( '.site-status-has-issues' ).addClass( 'hide' );

src/wp-admin/includes/ajax-actions.php

@@ -5457,7 +5457,13 @@ function wp_ajax_health_check_loopback_requests() {
 /**
  * Handles site health check to update the result status via AJAX.
  *
+ * The aggregate counts and the full per-test results are sent as two independent
+ * requests, so a large results payload cannot prevent the lightweight counts from being
+ * refreshed. Each is handled on its own here, and either may be omitted.
+ *
  * @since 5.2.0
+ * @since 7.1.0 The submitted counts are validated, and the optional full per-test results
+ *              are sanitized and cached separately as the authoritative detailed results.
  */
 function wp_ajax_health_check_site_status_result() {
 	check_ajax_referer( 'health-check-site-status-result' );
@@ -5466,7 +5472,40 @@ function wp_ajax_health_check_site_status_result() {
 		wp_send_json_error();
 	}
 
-	set_transient( 'health-check-site-status-result', wp_json_encode( $_POST['counts'] ) );
+	if ( ! class_exists( 'WP_Site_Health' ) ) {
+		require_once ABSPATH . 'wp-admin/includes/class-wp-site-health.php';
+	}
+
+	$save_results = array();
+
+	// Refresh the lightweight, autoloaded aggregate counts used by the admin menu and Dashboard.
+	// TODO: We don't need to keep this anymore. We can just obtain the counts at runtime from the stored results.
+	if ( isset( $_POST['counts'] ) && is_array( $_POST['counts'] ) ) {
+		$counts         = wp_unslash( $_POST['counts'] );
+		$save_results[] = WP_Site_Health::set_site_status_counts( $counts );
+	}
+
+	/*
+	 * Cache the full per-test results as the authoritative detailed results. These include
+	 * the asynchronous tests that require JavaScript to run. The values are sanitized in
+	 * WP_Site_Health::update_site_status_detail().
+	 */
+	if ( isset( $_POST['results'] ) && is_string( $_POST['results'] ) ) {
+		$results = json_decode( wp_unslash( $_POST['results'] ), true );
+
+		if ( is_array( $results ) ) {
+			$save_results[] = WP_Site_Health::update_site_status_detail( $results );
+		}
+	}
+
+	if ( count( $save_results ) === 0 ) {
+		wp_send_json_error();
+	}
+
+	$errors = array_filter( $save_results, 'is_wp_error' );
+	if ( count( $errors ) > 0 ) {
+		wp_send_json_error( reset( $errors ) );
+	}
 
 	wp_send_json_success();
 }