Pad perf trace summary columns for stable markdown tables

- define thread/title column widths as constants - pad thread IDs and titles so large-thread render rows align consistently

Julius Marminge committed 9 months ago

Commit a2b1f8e1 parent a4941ec1 file · +5 −2Browse files

Showing the changes to scripts (1 of 1 file). Show all

Unified · Split

 }
 function createMarkdownSummary({ tracePath, donePayload, summary, thresholds }) {
+  const THREAD_COL_WIDTH = 8;
+  const TITLE_COL_WIDTH = 15;
+  const pad = (value, width) => String(value).padEnd(width, " ");
   const largeThreadRenderStats = Array.isArray(donePayload.interactions?.largeThreadRenderStats)
     ? donePayload.interactions.largeThreadRenderStats
     : [];
   const formatThreadRenderRow = (stat) => {
     const threadShort =
-      typeof stat.threadId === "string" ? stat.threadId.slice(0, 8) : "n/a";
+      typeof stat.threadId === "string" ? stat.threadId.slice(0, THREAD_COL_WIDTH) : "n/a";
     const titleShort =
       typeof stat.threadTitleShort === "string"
         ? stat.threadTitleShort
       typeof followUpMinMs === "number" && typeof followUpMaxMs === "number"
         ? `${followUpMinMs}-${followUpMaxMs} (${followUpSampleCount}x)`
         : "n/a";
-    return `| ${threadShort} | ${titleShort} | ${stat.messageCount} | ${firstRenderMs} | ${followUpRenderMs} | ${followUpRange} | ${deltaMs} | ${deltaPct}% |`;
+    return `| ${pad(threadShort, THREAD_COL_WIDTH)} | ${pad(titleShort, TITLE_COL_WIDTH)} | ${stat.messageCount} | ${firstRenderMs} | ${followUpRenderMs} | ${followUpRange} | ${deltaMs} | ${deltaPct}% |`;
   };
   const seedSource =
     donePayload.seed?.source === "file"

Keyboard shortcuts