From 6128a137cc3ab1495f7239ccc7cab05773fb2313 Mon Sep 17 00:00:00 2001
From: Josh Wright <josh@thewrightserver.net>
Date: Fri, 9 Oct 2026 09:58:06 -0400
Subject: [PATCH] Format provisioning check values with units (Refs #90)

---
 src/lib/ui/components/ProvisioningProgress.svelte |  6 +++---
 src/lib/ui/provisioning-presentation.ts           | 10 +++++++++-
 2 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/src/lib/ui/components/ProvisioningProgress.svelte b/src/lib/ui/components/ProvisioningProgress.svelte
index fb461f3..198706d 100644
--- a/src/lib/ui/components/ProvisioningProgress.svelte
+++ b/src/lib/ui/components/ProvisioningProgress.svelte
@@ -1,7 +1,7 @@
 <script lang="ts">
     import RelativeTime from '$ui/components/RelativeTime.svelte';
     import type { ProvisioningDetail } from '$lib/provisioning-detail.js';
-    import { provisioningGroups, progressIcons, progressStateLabels } from '$ui/provisioning-presentation.js';
+    import { provisioningCheckValue, provisioningGroups, progressIcons, progressStateLabels } from '$ui/provisioning-presentation.js';
     import Badge from './primitives/Badge.svelte';
 
     let { detail }: { detail: ProvisioningDetail } = $props();
@@ -66,8 +66,8 @@
                     <article class="check">
                         <div class="check-heading"><strong>{check.label}</strong><Badge variant={check.result === 'pass' ? 'online' : check.result === 'fail' ? 'danger' : check.result === 'waiting' ? 'warning' : 'neutral'}>{evidenceLabels[check.result]}</Badge></div>
                         {#if check.expected !== null || check.actual !== null}<dl>
-                            {#if check.expected !== null}<div><dt>Expected</dt><dd>{check.expected}</dd></div>{/if}
-                            <div><dt>Observed</dt><dd>{check.actual ?? 'Unknown'}</dd></div>
+                            {#if check.expected !== null}<div><dt>Expected</dt><dd>{provisioningCheckValue(check.key, check.expected)}</dd></div>{/if}
+                            <div><dt>Observed</dt><dd>{provisioningCheckValue(check.key, check.actual)}</dd></div>
                         </dl>{/if}
                         {#if check.message}<p>{check.message}</p>{/if}
                         <div class="evidence-meta"><span>{check.source ? sources[check.source] : 'Source not recorded'}</span><span>Checked <span class="time"><RelativeTime value={check.checkedAt} /></span></span>{#if check.receivedAt}<span>Evidence received <span class="time"><RelativeTime value={check.receivedAt} /></span></span>{/if}</div>
diff --git a/src/lib/ui/provisioning-presentation.ts b/src/lib/ui/provisioning-presentation.ts
index 5e77fcd..3e9b6d4 100644
--- a/src/lib/ui/provisioning-presentation.ts
+++ b/src/lib/ui/provisioning-presentation.ts
@@ -1,6 +1,14 @@
-import { provisioningPhases, type ProvisioningDeployment, type ProvisioningDetail, type ProvisioningEvent, type ProvisioningEventPage, type ProvisioningPhase } from '$lib/provisioning-detail.js';
+import { provisioningPhases, type ProvisioningCheck, type ProvisioningDeployment, type ProvisioningDetail, type ProvisioningEvent, type ProvisioningEventPage, type ProvisioningPhase } from '$lib/provisioning-detail.js';
 import { actionCompleted, actionLabels, provisioningActions, type ProvisioningAction } from '$lib/provisioning.js';
 import { systemUsername } from '$lib/auth.js';
+import { countLabel } from './plural.js';
+
+export function provisioningCheckValue(key: ProvisioningCheck['key'], value: ProvisioningCheck['actual']): string {
+    if (typeof value !== 'number') return value ?? 'Unknown';
+    if (key === 'game') return countLabel(value, 'player');
+    if (key === 'plutonium-release' || key === 'plutonium-version') return `r${value}`;
+    return String(value);
+}
 
 export function provisioningRequester(operation: Pick<ProvisioningDeployment, 'requested_by' | 'triggered_by'>): string {
     return operation.requested_by ?? (operation.triggered_by === 'ui' ? 'Not recorded' : systemUsername);