From 983728e9138992485508487c22ad589fa6bb3acc Mon Sep 17 00:00:00 2001
From: Johann Schopplich <mail@johannschopplich.com>
Date: Thu, 30 Oct 2025 15:24:22 +0100
Subject: [PATCH] refactor: progress bar configuration

---
 benchmarks/src/constants.ts | 10 ----------
 benchmarks/src/report.ts    |  9 +++++++--
 2 files changed, 7 insertions(+), 12 deletions(-)

diff --git a/benchmarks/src/constants.ts b/benchmarks/src/constants.ts
index 5d0c7b1..adf5327 100644
--- a/benchmarks/src/constants.ts
+++ b/benchmarks/src/constants.ts
@@ -35,16 +35,6 @@ export const FORMATTER_DISPLAY_NAMES: Record<string, string> = {
   'yaml': 'YAML',
 } as const
 
-/**
- * Progress bar configuration
- */
-export const PROGRESS_BAR = {
-  /** Default width for progress bars */
-  defaultWidth: 25,
-  /** Compact width for inline displays */
-  compactWidth: 20,
-} as const
-
 /**
  * Enable dry run mode for quick testing with limited AI requests
  *
diff --git a/benchmarks/src/report.ts b/benchmarks/src/report.ts
index 41713c7..9ab61cc 100644
--- a/benchmarks/src/report.ts
+++ b/benchmarks/src/report.ts
@@ -54,6 +54,11 @@ export function generateMarkdownReport(
   const modelIds = models.map(m => m.modelId)
   const modelNames = modelIds.filter(id => results.some(r => r.model === id))
 
+  const maxDisplayNameWidth = Math.max(
+    ...Object.values(FORMATTER_DISPLAY_NAMES).map(name => name.length),
+  )
+  const progressBarWidth = 20
+
   const modelBreakdown = modelNames.map((modelName, i) => {
     const modelResults = formatResults.map((fr) => {
       const modelFormatResults = results.filter(r => r.model === modelName && r.format === fr.format)
@@ -70,12 +75,12 @@ export function generateMarkdownReport(
     }).sort((a, b) => b.accuracy - a.accuracy)
 
     const formatLines = modelResults.map((result) => {
-      const bar = createProgressBar(result.accuracy, 1, 20)
+      const bar = createProgressBar(result.accuracy, 1, progressBarWidth)
       const accuracyString = `${(result.accuracy * 100).toFixed(1)}%`.padStart(6)
       const countString = `(${result.correctCount}/${result.totalCount})`
       const prefix = result.format === 'toon' ? '→ ' : '  '
       const displayName = FORMATTER_DISPLAY_NAMES[result.format] || result.format
-      return `${prefix}${displayName.padEnd(12)}   ${bar}   ${accuracyString} ${countString}`
+      return `${prefix}${displayName.padEnd(maxDisplayNameWidth)}   ${bar}   ${accuracyString} ${countString}`
     }).join('\n')
 
     // Add blank line before model name, except for first model