PR_26175_CHARLIE_009-system-health-current-r2-health

Author: ToolboxAid

admin/system-health.html

@@ -143,11 +143,12 @@ <h2 id="admin-system-health-title">System Health Tables</h2>
                                     </tr>
                                 </thead>
                                 <tbody>
-                                    <tr><td>Bucket</td><td data-admin-system-health-storage-value="bucket">Configured bucket placeholder</td><td data-health-status="PENDING" data-admin-system-health-storage-status="bucket" title="Reason: R2 bucket reader is intentionally not wired in this foundation PR." aria-label="PENDING: R2 bucket reader is intentionally not wired in this foundation PR.">PENDING</td></tr>
-                                    <tr><td>List</td><td data-admin-system-health-storage-value="list">Objects prefix</td><td data-health-status="PENDING" data-admin-system-health-storage-status="list" title="Reason: R2 list check is intentionally not wired in this foundation PR." aria-label="PENDING: R2 list check is intentionally not wired in this foundation PR.">PENDING</td></tr>
-                                    <tr><td>Read</td><td data-admin-system-health-storage-value="read">Health object</td><td data-health-status="PENDING" data-admin-system-health-storage-status="read" title="Reason: R2 read check is intentionally not wired in this foundation PR." aria-label="PENDING: R2 read check is intentionally not wired in this foundation PR.">PENDING</td></tr>
-                                    <tr><td>Write</td><td data-admin-system-health-storage-value="write">Health object</td><td data-health-status="PENDING" data-admin-system-health-storage-status="write" title="Reason: R2 write check is intentionally not wired in this foundation PR." aria-label="PENDING: R2 write check is intentionally not wired in this foundation PR.">PENDING</td></tr>
-                                    <tr><td>Delete</td><td data-admin-system-health-storage-value="delete">Health object</td><td data-health-status="PENDING" data-admin-system-health-storage-status="delete" title="Reason: R2 delete check is intentionally not wired in this foundation PR." aria-label="PENDING: R2 delete check is intentionally not wired in this foundation PR.">PENDING</td></tr>
+                                    <tr><td>Bucket connectivity</td><td data-admin-system-health-storage-value="bucket">Loading</td><td data-health-status="PENDING" data-admin-system-health-storage-status="bucket" title="Reason: R2 bucket connectivity has not loaded yet." aria-label="PENDING: R2 bucket connectivity has not loaded yet.">PENDING</td></tr>
+                                    <tr><td>List</td><td data-admin-system-health-storage-value="list">Loading</td><td data-health-status="PENDING" data-admin-system-health-storage-status="list" title="Reason: R2 list check has not loaded yet." aria-label="PENDING: R2 list check has not loaded yet.">PENDING</td></tr>
+                                    <tr><td>Upload</td><td data-admin-system-health-storage-value="upload">Loading</td><td data-health-status="PENDING" data-admin-system-health-storage-status="upload" title="Reason: R2 upload check has not loaded yet." aria-label="PENDING: R2 upload check has not loaded yet.">PENDING</td></tr>
+                                    <tr><td>Read</td><td data-admin-system-health-storage-value="read">Loading</td><td data-health-status="PENDING" data-admin-system-health-storage-status="read" title="Reason: R2 read check has not loaded yet." aria-label="PENDING: R2 read check has not loaded yet.">PENDING</td></tr>
+                                    <tr><td>Delete</td><td data-admin-system-health-storage-value="delete">Loading</td><td data-health-status="PENDING" data-admin-system-health-storage-status="delete" title="Reason: R2 delete check has not loaded yet." aria-label="PENDING: R2 delete check has not loaded yet.">PENDING</td></tr>
+                                    <tr><td>Last checked</td><td data-admin-system-health-storage-value="lastChecked">Loading</td><td data-health-status="PENDING" data-admin-system-health-storage-status="lastChecked" title="Reason: R2 health has not loaded yet." aria-label="PENDING: R2 health has not loaded yet.">PENDING</td></tr>
                                 </tbody>
                             </table>
                         </div>

assets/theme-v2/js/admin-system-health.js

@@ -9,8 +9,9 @@ import {
 } from "../../js/shared/status.js";
 
 const STORAGE_DIAGNOSTIC_ACTIONS = Object.freeze([
+    Object.freeze({ actionId: "storage-bucket-connectivity", key: "bucket" }),
     Object.freeze({ actionId: "storage-list", key: "list" }),
-    Object.freeze({ actionId: "storage-write-test-object", key: "write" }),
+    Object.freeze({ actionId: "storage-upload-test-object", key: "upload" }),
     Object.freeze({ actionId: "storage-read-test-object", key: "read" }),
     Object.freeze({ actionId: "storage-delete-test-object", key: "delete" }),
 ]);
@@ -127,7 +128,7 @@ class AdminSystemHealthController {
     }
 
     renderStoragePending(reason) {
-        ["bucket", "list", "read", "write", "delete"].forEach((key) => {
+        ["bucket", "list", "upload", "read", "delete", "lastChecked"].forEach((key) => {
             this.setStorageStatus(key, "PENDING", reason);
         });
         this.renderRuntimePending(reason);
@@ -153,8 +154,10 @@ class AdminSystemHealthController {
 
     renderStorageStatus(storageStatus = {}) {
         const reason = storageStatus.message || "Cloudflare R2 configuration status returned by the safe Admin System Health API.";
-        this.setStorageValue("bucket", storageStatus.bucket, "not configured");
+        this.setStorageValue("bucket", storageStatus.environmentFolder ? `${storageStatus.bucket || "not configured"} ${storageStatus.environmentFolder}` : storageStatus.bucket, "not configured");
         this.setStorageStatus("bucket", storageStatus.bucketStatus || storageStatus.status, reason);
+        this.setStorageValue("lastChecked", storageStatus.lastChecked, "not available");
+        this.setStorageStatus("lastChecked", storageStatus.lastChecked ? "PASS" : "WARN", reason);
     }
 
     renderStartupPending(reason) {
@@ -198,9 +201,12 @@ class AdminSystemHealthController {
 
     storageResultTarget(result = {}) {
         if (typeof result.keysListed === "number" && result.actionId === "storage-list") {
-            return `${result.keysListed} object(s) under ${asText(result.projectsPrefix, "configured prefix")}`;
+            return `${result.keysListed} object(s) under ${asText(result.environmentFolder, "current environment folder")}`;
         }
-        return result.testObjectKey || result.projectsPrefix || result.message || "diagnostic target unavailable";
+        if (result.actionId === "storage-bucket-connectivity") {
+            return result.environmentFolder || result.message || "bucket connectivity target unavailable";
+        }
+        return result.testObjectKey || result.environmentFolder || result.message || "diagnostic target unavailable";
     }
 
     renderStorageResult(key, result = {}) {
@@ -210,6 +216,10 @@ class AdminSystemHealthController {
         }
         this.setStorageValue(key, this.storageResultTarget(result));
         this.setStorageStatus(key, result.status, result.message || "R2 diagnostic returned without a message.");
+        if (result.lastChecked) {
+            this.setStorageValue("lastChecked", result.lastChecked);
+            this.setStorageStatus("lastChecked", "PASS", "Most recent current-environment R2 health check timestamp.");
+        }
     }
 
     runStorageDiagnostics() {

docs_build/dev/reports/PR_26175_CHARLIE_009-system-health-current-r2-health.md

@@ -0,0 +1,41 @@
+# PR_26175_CHARLIE_009 System Health Current R2 Health
+
+## Scope
+
+Team: Charlie
+
+Purpose: Add current-environment Cloudflare R2 folder health only to Admin System Health.
+
+## Changes
+
+- Replaced the System Health Storage Health rows with current-environment R2 checks:
+  - bucket connectivity
+  - list
+  - upload
+  - read
+  - delete
+  - last checked
+- System Health storage diagnostics run against the current environment folder only:
+  - Local: `/local`
+  - DEV: `/dev`
+  - IST: `/ist`
+  - UAT: `/uat`
+  - PRD: `/prd`
+- Preserved the separate Admin Infrastructure storage diagnostic path that uses the configured project prefix.
+- Updated focused API and Playwright tests.
+
+## Architecture Constraint
+
+PASS. R2 health checks target only the current deployment environment folder. No peer environment R2 folders are actively checked by System Health.
+
+## Validation
+
+- PASS: `node --check src/dev-runtime/server/local-api-router.mjs`
+- PASS: `node --check assets/theme-v2/js/admin-system-health.js`
+- PASS: `git diff --check`
+- PASS: `node --test tests/dev-runtime/AdminHealthOperations.test.mjs`
+- PASS: `npx playwright test tests/playwright/tools/AdminHealthOperationsPage.spec.mjs --workers=1 --reporter=line`
+
+## Artifact
+
+- `tmp/PR_26175_CHARLIE_009-system-health-current-r2-health_delta.zip`

docs_build/dev/reports/coverage_changed_js_guardrail.txt

@@ -7,7 +7,7 @@ Source: Playwright/Chromium built-in V8 coverage from the active Playwright run.
 
 Changed runtime JS files considered:
 (0%) src/dev-runtime/server/local-api-router.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
-(89%) assets/theme-v2/js/admin-system-health.js - executed lines 284/284; executed functions 33/37
+(89%) assets/theme-v2/js/admin-system-health.js - executed lines 294/294; executed functions 33/37
 
 Guardrail warnings:
 (0%) src/dev-runtime/server/local-api-router.mjs - WARNING: changed runtime JS file missing from coverage; advisory only

docs_build/dev/reports/playwright_v8_coverage_report.txt

@@ -18,7 +18,7 @@ Exercised tool entry points detected:
 
 Changed runtime JS files covered:
 (0%) src/dev-runtime/server/local-api-router.mjs - WARNING: changed runtime JS file was not collected by Playwright V8 coverage; advisory only
-(89%) assets/theme-v2/js/admin-system-health.js - executed lines 284/284; executed functions 33/37
+(89%) assets/theme-v2/js/admin-system-health.js - executed lines 294/294; executed functions 33/37
 
 Files with executed line/function counts where available:
 (36%) src/api/server-api-client.js - executed lines 167/167; executed functions 5/14
@@ -28,7 +28,7 @@ Files with executed line/function counts where available:
 (74%) assets/theme-v2/js/gamefoundry-partials.js - executed lines 1001/1001; executed functions 69/93
 (80%) src/api/admin-owner-navigation.js - executed lines 42/42; executed functions 4/5
 (83%) assets/js/shared/status.js - executed lines 37/37; executed functions 5/6
-(89%) assets/theme-v2/js/admin-system-health.js - executed lines 284/284; executed functions 33/37
+(89%) assets/theme-v2/js/admin-system-health.js - executed lines 294/294; executed functions 33/37
 (91%) assets/theme-v2/js/admin-owner-navigation.js - executed lines 58/58; executed functions 10/11
 (100%) src/api/admin-system-health-api-client.js - executed lines 19/19; executed functions 3/3