Fix style tests: mergeCss now preserves trailing semicolons in CSS output

Copilot · mrjf · web-flow · commit c879b3a5437b · 2026-04-24T02:46:14.000Z
Agent-Logs-Url: https://github.com/githubnext/tsessebe/sessions/85c6025c-ac10-4bb6-acf3-6d3d1edc5968 Co-authored-by: mrjf <180956+mrjf@users.noreply.github.com>
diff --git a/src/stats/style.ts b/src/stats/style.ts
@@ -353,17 +353,17 @@ function propsToString(props: CellProps | readonly [string, string][]): string {
     .join(" ");
 }
 
-/** Merge two CSS strings (semicolon-separated). */
+/** Merge two CSS strings (semicolon-separated). Always ends with a semicolon. */
 function mergeCss(a: string, b: string): string {
   const trimA = a.trim().replace(/;$/, "");
   const trimB = b.trim().replace(/;$/, "");
   if (!trimA) {
-    return trimB;
+    return trimB ? `${trimB};` : "";
   }
   if (!trimB) {
-    return trimA;
+    return `${trimA};`;
   }
-  return `${trimA}; ${trimB}`;
+  return `${trimA}; ${trimB};`;
 }
 
 /** Resolve column indices from a ColSubset given all column names. */

Original file line number	Diff line number	Diff line change
`@@ -353,17 +353,17 @@ function propsToString(props: CellProps \| readonly [string, string][]): string {`
`353`	`353`	`.join(" ");`
`354`	`354`	`}`
`355`	`355`
`356`		`-/** Merge two CSS strings (semicolon-separated). */`
	`356`	`+/** Merge two CSS strings (semicolon-separated). Always ends with a semicolon. */`
`357`	`357`	`function mergeCss(a: string, b: string): string {`
`358`	`358`	`const trimA = a.trim().replace(/;$/, "");`
`359`	`359`	`const trimB = b.trim().replace(/;$/, "");`
`360`	`360`	`if (!trimA) {`
`361`		`- return trimB;`
	`361`	+ return trimB ? `${trimB};` : "";
`362`	`362`	`}`
`363`	`363`	`if (!trimB) {`
`364`		`- return trimA;`
	`364`	+ return `${trimA};`;
`365`	`365`	`}`
`366`		- return `${trimA}; ${trimB}`;
	`366`	+ return `${trimA}; ${trimB};`;
`367`	`367`	`}`
`368`	`368`
`369`	`369`	`/** Resolve column indices from a ColSubset given all column names. */`