From 64e70b20d7a7f49e6640007fc43ba0dad52abbb7 Mon Sep 17 00:00:00 2001
From: Matthew Valancy <matthew.valancy@gmail.com>
Date: Thu, 18 Jun 2026 20:51:26 -0700
Subject: [PATCH] report: render every check (each with its citable ref),
 sections collapsed
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Previously only warn/fail + media cases were listed, so a passing check had no
visible ref to cite. Now every case renders (with its ref); sections are <details>
collapsed by default (open if warn/fail) so the page stays scannable while nothing
is hidden — any check is citable.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 tests/lib/reporting/html.mjs | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/tests/lib/reporting/html.mjs b/tests/lib/reporting/html.mjs
index 7cb91840..b02341bb 100644
--- a/tests/lib/reporting/html.mjs
+++ b/tests/lib/reporting/html.mjs
@@ -31,11 +31,10 @@ function caseHtml(c) {
 function seqHtml(seq, i) {
   const c = seq.counts || {};
   const cases = (seq.cases || []);
-  // Show failed/warn cases expanded; collapse a long all-green list to just its attachments.
-  const notable = cases.filter((x) => x.status === 'failed' || x.status === 'warn');
-  const withAtts = cases.filter((x) => (x.attachments || []).length);
-  const shown = notable.length ? notable : withAtts.slice(0, 24);
-  return `<details class="seq ${esc(seq.status)}" ${seq.status === 'failed' ? 'open' : ''}>
+  // Render EVERY check (each carries a citable ref) so any of them can be cited;
+  // the section is a <details> collapsed by default (open if it failed), so the
+  // page stays scannable while nothing is hidden.
+  return `<details class="seq ${esc(seq.status)}" ${seq.status === 'failed' || seq.status === 'warn' ? 'open' : ''}>
     <summary>
       <span class="badge" style="background:${COLOR[seq.status] || '#888'}">${ICON[seq.status] || ''} ${esc(seq.status)}</span>
       ${seq.ref ? `<code class="sref" title="section ref">§${esc(seq.ref)}</code>` : ''}
@@ -45,7 +44,7 @@ function seqHtml(seq, i) {
     </summary>
     ${seq.command ? `<code class="cmd">${esc(seq.command)}</code>` : ''}
     ${seq.notes ? `<p class="notes">${esc(seq.notes)}</p>` : ''}
-    ${shown.map(caseHtml).join('') || '<p class="empty">no per-case detail</p>'}
+    ${cases.map(caseHtml).join('') || '<p class="empty">no per-case detail</p>'}
   </details>`;
 }