refactor: dynamic progress bar width based on terminal size

Author: fullstackjam

go.mod

@@ -1,6 +1,6 @@
 module github.com/openbootdotdev/openboot
 
-go 1.22
+go 1.24.0
 
 require (
 	github.com/charmbracelet/bubbles v0.20.0
@@ -32,6 +32,7 @@ require (
 	github.com/rivo/uniseg v0.4.7 // indirect
 	github.com/spf13/pflag v1.0.5 // indirect
 	golang.org/x/sync v0.8.0 // indirect
-	golang.org/x/sys v0.25.0 // indirect
+	golang.org/x/sys v0.40.0 // indirect
+	golang.org/x/term v0.39.0 // indirect
 	golang.org/x/text v0.18.0 // indirect
 )

go.sum

@@ -57,6 +57,10 @@ golang.org/x/sys v0.0.0-20210809222454-d867a43fc93e/go.mod h1:oPkhp1MJrh7nUepCBc
 golang.org/x/sys v0.6.0/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
 golang.org/x/sys v0.25.0 h1:r+8e+loiHxRqhXVl6ML1nO3l1+oFoWbnlu2Ehimmi34=
 golang.org/x/sys v0.25.0/go.mod h1:/VUhepiaJMQUp4+oa/7Zr1D23ma6VTLIYjOOTFZPUcA=
+golang.org/x/sys v0.40.0 h1:DBZZqJ2Rkml6QMQsZywtnjnnGvHza6BTfYFWY9kjEWQ=
+golang.org/x/sys v0.40.0/go.mod h1:OgkHotnGiDImocRcuBABYBEXf8A9a87e/uXjp9XT3ks=
+golang.org/x/term v0.39.0 h1:RclSuaJf32jOqZz74CkPA9qFuVTX7vhLlpfj/IGWlqY=
+golang.org/x/term v0.39.0/go.mod h1:yxzUCTP/U+FzoxfdKmLaA0RV1WgE0VY7hXBwKtY/4ww=
 golang.org/x/text v0.18.0 h1:XvMDiNzPAl0jr17s6W9lcaIhGUfUORdGCNsuLmPG224=
 golang.org/x/text v0.18.0/go.mod h1:BuEKDfySbSR4drPmRPG/7iBdf8hvFMuRexcpahXilzY=
 gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405 h1:yhCVgyC4o1eVCa2tZl7eS0r+SDo693bJlVdllGtEeKM=

internal/installer/installer.go

@@ -27,7 +27,7 @@ func Run(cfg *config.Config) error {
 
 func runInstall(cfg *config.Config) error {
 	fmt.Println()
-	ui.Header("OpenBoot Installer v0.9.3")
+	ui.Header("OpenBoot Installer v0.9.4")
 	fmt.Println()
 
 	if cfg.DryRun {

internal/ui/progress.go

@@ -2,11 +2,22 @@ package ui
 
 import (
 	"fmt"
+	"os"
 	"strings"
 	"sync"
 	"time"
 
 	"github.com/charmbracelet/lipgloss"
+	"golang.org/x/term"
+)
+
+const (
+	minBarWidth     = 20
+	defaultBarWidth = 40
+	minPkgWidth     = 10
+	defaultPkgWidth = 25
+	statusWidth     = 16 // " 48/48 (100%)"
+	etaWidth        = 8  // "~10m30s "
 )
 
 var (
@@ -30,7 +41,8 @@ type ProgressTracker struct {
 	total         int
 	completed     int
 	active        map[string]bool
-	width         int
+	barWidth      int
+	pkgWidth      int
 	startTime     time.Time
 	mu            sync.Mutex
 	spinnerIdx    int
@@ -40,10 +52,21 @@ type ProgressTracker struct {
 
 var spinnerFrames = []string{"⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧", "⠇", "⠏"}
 
+func getTerminalWidth() int {
+	if w, _, err := term.GetSize(int(os.Stdout.Fd())); err == nil && w > 0 {
+		return w
+	}
+	return 80
+}
+
 func NewProgressTracker(total int) *ProgressTracker {
+	termWidth := getTerminalWidth()
+	barWidth, pkgWidth := calculateWidths(termWidth)
+
 	p := &ProgressTracker{
 		total:       total,
-		width:       40,
+		barWidth:    barWidth,
+		pkgWidth:    pkgWidth,
 		startTime:   time.Now(),
 		active:      make(map[string]bool),
 		spinnerStop: make(chan bool),
@@ -70,6 +93,28 @@ func NewProgressTracker(total int) *ProgressTracker {
 	return p
 }
 
+func calculateWidths(termWidth int) (barWidth, pkgWidth int) {
+	fixed := statusWidth + etaWidth
+	available := termWidth - fixed - 2
+
+	if available < minBarWidth+minPkgWidth {
+		return minBarWidth, minPkgWidth
+	}
+
+	barWidth = available * 55 / 100
+	pkgWidth = available - barWidth
+
+	if barWidth > defaultBarWidth {
+		barWidth = defaultBarWidth
+		pkgWidth = available - barWidth
+	}
+	if pkgWidth > defaultPkgWidth+10 {
+		pkgWidth = defaultPkgWidth + 10
+	}
+
+	return barWidth, pkgWidth
+}
+
 func (p *ProgressTracker) SetCurrent(pkgName string) {
 	p.mu.Lock()
 	defer p.mu.Unlock()
@@ -88,10 +133,20 @@ func (p *ProgressTracker) Complete(pkgName string) {
 	p.render()
 }
 
+func truncate(s string, maxLen int) string {
+	if len(s) <= maxLen {
+		return s
+	}
+	if maxLen <= 3 {
+		return s[:maxLen]
+	}
+	return s[:maxLen-3] + "..."
+}
+
 func (p *ProgressTracker) render() {
 	percent := float64(p.completed) / float64(p.total)
-	filled := int(percent * float64(p.width))
-	empty := p.width - filled
+	filled := int(percent * float64(p.barWidth))
+	empty := p.barWidth - filled
 
 	bar := progressBarStyle.Render(strings.Repeat("█", filled)) +
 		progressBgStyle.Render(strings.Repeat("░", empty))
@@ -115,13 +170,13 @@ func (p *ProgressTracker) render() {
 				break
 			}
 		}
-		if len(activeDisplay) > 20 {
-			activeDisplay = activeDisplay[:17] + "..."
+		suffixLen := 0
+		if activeCount > 1 {
+			suffixLen = len(fmt.Sprintf(" +%d", activeCount-1))
 		}
+		activeDisplay = truncate(activeDisplay, p.pkgWidth-suffixLen)
 		if activeCount > 1 {
-			activeDisplay = fmt.Sprintf("%-20s +%d", activeDisplay, activeCount-1)
-		} else {
-			activeDisplay = fmt.Sprintf("%-20s", activeDisplay)
+			activeDisplay = fmt.Sprintf("%s +%d", activeDisplay, activeCount-1)
 		}
 	}