Add green/red color indicators to test and coverage deltas

andimarek · claude · andimarek · commit 75d0a95ab36d · 2026-03-05T10:44:11.000+10:00
Test results: green for more total/passed, red for more failed/errors.
Coverage deltas: green for increase, red for decrease.

Co-Authored-By: Claude Opus 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/.github/workflows/pull_request.yml b/.github/workflows/pull_request.yml
@@ -179,14 +179,17 @@ jobs:
             }
 
             // --- Helpers ---
-            function delta(curr, prev) {
+            function delta(curr, prev, positiveIsGood) {
               const d = curr - prev;
               if (d === 0) return '±0';
-              return d > 0 ? `+${d}` : `${d}`;
+              const str = d > 0 ? `+${d}` : `${d}`;
+              if (positiveIsGood === undefined) return str;
+              const icon = (positiveIsGood ? d > 0 : d < 0) ? ' 🟢' : ' 🔴';
+              return str + icon;
             }
 
-            function fmtCell(curr, prev) {
-              return `${curr} (${delta(curr, prev)})`;
+            function fmtCell(curr, prev, positiveIsGood) {
+              return `${curr} (${delta(curr, prev, positiveIsGood)})`;
             }
 
             function pct(covered, missed) {
@@ -201,7 +204,9 @@ jobs:
             function fmtPctDelta(curr, prev) {
               const d = curr - prev;
               if (Math.abs(d) < 0.05) return '±0.0%';
-              return d > 0 ? `+${d.toFixed(1)}%` : `${d.toFixed(1)}%`;
+              const str = d > 0 ? `+${d.toFixed(1)}%` : `${d.toFixed(1)}%`;
+              const icon = d > 0 ? ' 🟢' : ' 🔴';
+              return str + icon;
             }
 
             // --- Build Test Results table ---
@@ -217,7 +222,7 @@ jobs:
               if (!current[v]) {
                 body += `| ${label} | - | - | - | - | - |\n`;
               } else {
-                body += `| ${label} | ${fmtCell(c.total, b.total)} | ${fmtCell(c.passed, b.passed)} | ${fmtCell(c.failed, b.failed)} | ${fmtCell(c.errors, b.errors)} | ${fmtCell(c.skipped, b.skipped)} |\n`;
+                body += `| ${label} | ${fmtCell(c.total, b.total, true)} | ${fmtCell(c.passed, b.passed, true)} | ${fmtCell(c.failed, b.failed, false)} | ${fmtCell(c.errors, b.errors, false)} | ${fmtCell(c.skipped, b.skipped)} |\n`;
               }
             }
 
@@ -235,7 +240,7 @@ jobs:
               }
             }
             if (hasAny) {
-              body += `| **Total** | **${fmtCell(totalCurr.total, totalBase.total)}** | **${fmtCell(totalCurr.passed, totalBase.passed)}** | **${fmtCell(totalCurr.failed, totalBase.failed)}** | **${fmtCell(totalCurr.errors, totalBase.errors)}** | **${fmtCell(totalCurr.skipped, totalBase.skipped)}** |\n`;
+              body += `| **Total** | **${fmtCell(totalCurr.total, totalBase.total, true)}** | **${fmtCell(totalCurr.passed, totalBase.passed, true)}** | **${fmtCell(totalCurr.failed, totalBase.failed, false)}** | **${fmtCell(totalCurr.errors, totalBase.errors, false)}** | **${fmtCell(totalCurr.skipped, totalBase.skipped)}** |\n`;
             }
 
             // --- Build Coverage table ---

Original file line number	Diff line number	Diff line change
`@@ -179,14 +179,17 @@ jobs:`
`179`	`179`	`}`
`180`	`180`
`181`	`181`	`// --- Helpers ---`
`182`		`- function delta(curr, prev) {`
	`182`	`+ function delta(curr, prev, positiveIsGood) {`
`183`	`183`	`const d = curr - prev;`
`184`	`184`	`if (d === 0) return '±0';`
`185`		- return d > 0 ? `+${d}` : `${d}`;
	`185`	+ const str = d > 0 ? `+${d}` : `${d}`;
	`186`	`+ if (positiveIsGood === undefined) return str;`
	`187`	`+ const icon = (positiveIsGood ? d > 0 : d < 0) ? ' 🟢' : ' 🔴';`
	`188`	`+ return str + icon;`
`186`	`189`	`}`
`187`	`190`
`188`		`- function fmtCell(curr, prev) {`
`189`		- return `${curr} (${delta(curr, prev)})`;
	`191`	`+ function fmtCell(curr, prev, positiveIsGood) {`
	`192`	+ return `${curr} (${delta(curr, prev, positiveIsGood)})`;
`190`	`193`	`}`
`191`	`194`
`192`	`195`	`function pct(covered, missed) {`
`@@ -201,7 +204,9 @@ jobs:`
`201`	`204`	`function fmtPctDelta(curr, prev) {`
`202`	`205`	`const d = curr - prev;`
`203`	`206`	`if (Math.abs(d) < 0.05) return '±0.0%';`
`204`		- return d > 0 ? `+${d.toFixed(1)}%` : `${d.toFixed(1)}%`;
	`207`	+ const str = d > 0 ? `+${d.toFixed(1)}%` : `${d.toFixed(1)}%`;
	`208`	`+ const icon = d > 0 ? ' 🟢' : ' 🔴';`
	`209`	`+ return str + icon;`
`205`	`210`	`}`
`206`	`211`
`207`	`212`	`// --- Build Test Results table ---`
`@@ -217,7 +222,7 @@ jobs:`
`217`	`222`	`if (!current[v]) {`
`218`	`223`	body += `\| ${label} \| - \| - \| - \| - \| - \|\n`;
`219`	`224`	`} else {`
`220`		- body += `\| ${label} \| ${fmtCell(c.total, b.total)} \| ${fmtCell(c.passed, b.passed)} \| ${fmtCell(c.failed, b.failed)} \| ${fmtCell(c.errors, b.errors)} \| ${fmtCell(c.skipped, b.skipped)} \|\n`;
	`225`	+ body += `\| ${label} \| ${fmtCell(c.total, b.total, true)} \| ${fmtCell(c.passed, b.passed, true)} \| ${fmtCell(c.failed, b.failed, false)} \| ${fmtCell(c.errors, b.errors, false)} \| ${fmtCell(c.skipped, b.skipped)} \|\n`;
`221`	`226`	`}`
`222`	`227`	`}`
`223`	`228`
`@@ -235,7 +240,7 @@ jobs:`
`235`	`240`	`}`
`236`	`241`	`}`
`237`	`242`	`if (hasAny) {`
`238`		- body += `\| Total \| ${fmtCell(totalCurr.total, totalBase.total)} \| ${fmtCell(totalCurr.passed, totalBase.passed)} \| ${fmtCell(totalCurr.failed, totalBase.failed)} \| ${fmtCell(totalCurr.errors, totalBase.errors)} \| ${fmtCell(totalCurr.skipped, totalBase.skipped)} \|\n`;
	`243`	+ body += `\| Total \| ${fmtCell(totalCurr.total, totalBase.total, true)} \| ${fmtCell(totalCurr.passed, totalBase.passed, true)} \| ${fmtCell(totalCurr.failed, totalBase.failed, false)} \| ${fmtCell(totalCurr.errors, totalBase.errors, false)} \| ${fmtCell(totalCurr.skipped, totalBase.skipped)} \|\n`;
`239`	`244`	`}`
`240`	`245`
`241`	`246`	`// --- Build Coverage table ---`