🎨 Improve the structure of some internal packages

internal/build/view/annotations.go

@@ -0,0 +1,99 @@
+package view
+
+import (
+	"fmt"
+	"regexp"
+
+	"github.com/buildkite/go-buildkite/v4"
+	"github.com/charmbracelet/lipgloss"
+)
+
+const (
+	styleDefault = lipgloss.Color("#DDD")
+	styleInfo    = lipgloss.Color("#337AB7")
+	styleWarning = lipgloss.Color("#FF841C")
+	styleError   = lipgloss.Color("#F45756")
+	styleSuccess = lipgloss.Color("#2ECC40")
+
+	// Maximum length for annotation body preview
+	maxAnnotationLength = 120
+)
+
+func (v *BuildView) renderAnnotations() string {
+	var sections []string
+
+	title := lipgloss.NewStyle().Bold(true).Padding(0, 1).Underline(true).Render("Annotations")
+	sections = append(sections, title)
+
+	for _, annotation := range v.Annotations {
+		sections = append(sections, renderAnnotation(&annotation))
+	}
+
+	return lipgloss.JoinVertical(lipgloss.Top, sections...)
+}
+
+func renderAnnotation(annotation *buildkite.Annotation) string {
+	return lipgloss.JoinVertical(lipgloss.Top,
+		lipgloss.NewStyle().Align(lipgloss.Left).Padding(0, 1).Render(""),
+		lipgloss.NewStyle().
+			Padding(0, 1).
+			Border(lipgloss.RoundedBorder()).
+			BorderForeground(getAnnotationStateColor(annotation.Style)).
+			Render(fmt.Sprintf("%s\t%s",
+				getAnnotationIcon(annotation.Style),
+				truncateAndStripTags(annotation.BodyHTML),
+			)),
+	)
+}
+
+func getAnnotationStateColor(state string) lipgloss.Color {
+	switch state {
+	case "success":
+		return styleSuccess
+	case "error":
+		return styleError
+	case "warning":
+		return styleWarning
+	case "info":
+		return styleInfo
+	default:
+		return styleDefault
+	}
+}
+
+func getAnnotationIcon(state string) string {
+	switch state {
+	case "success":
+		return "✔"
+	case "error":
+		return "✖"
+	case "warning":
+		return "⚠"
+	case "info":
+		return "ℹ"
+	default:
+		return "🗒️"
+	}
+}
+
+func truncateAndStripTags(body string) string {
+	// Strip HTML tags while preserving newlines
+	cleaned := stripTags(body)
+
+	if len(cleaned) > maxAnnotationLength {
+		return cleaned[:maxAnnotationLength] + "...(more)"
+	}
+	return cleaned
+}
+
+func stripTags(body string) string {
+	// Remove closing tags first
+	re := regexp.MustCompile(`</[^>]+>`)
+	body = re.ReplaceAllString(body, "")
+
+	// Then remove opening tags
+	re = regexp.MustCompile(`<[^>]*>`)
+	body = re.ReplaceAllString(body, "")
+
+	return body
+}

internal/build/view/artifacts.go

@@ -0,0 +1,54 @@
+package view
+
+import (
+	"fmt"
+
+	"github.com/buildkite/go-buildkite/v4"
+	"github.com/charmbracelet/lipgloss"
+)
+
+func (v *BuildView) renderArtifacts() string {
+	var sections []string
+
+	title := lipgloss.NewStyle().Bold(true).Padding(0, 1).Underline(true).Render("Artifacts")
+	sections = append(sections, title)
+
+	for _, artifact := range v.Artifacts {
+		sections = append(sections, renderArtifact(&artifact))
+	}
+
+	return lipgloss.JoinVertical(lipgloss.Top, sections...)
+}
+
+func renderArtifact(artifact *buildkite.Artifact) string {
+	return lipgloss.JoinVertical(lipgloss.Top,
+		lipgloss.NewStyle().Align(lipgloss.Left).Padding(0, 1).Render(""),
+		lipgloss.JoinHorizontal(lipgloss.Left,
+			lipgloss.NewStyle().Width(38).Align(lipgloss.Left).Padding(0, 1).Render(artifact.ID),
+			lipgloss.NewStyle().Width(50).Align(lipgloss.Left).Padding(0, 1).Render(artifact.Path),
+			lipgloss.NewStyle().Align(lipgloss.Right).Padding(0, 1).Render(formatBytes(artifact.FileSize)),
+		),
+	)
+}
+
+func formatBytes(bytes int64) string {
+	const (
+		KB = 1024
+		MB = 1024 * KB
+		GB = 1024 * MB
+		TB = 1024 * GB
+	)
+
+	switch {
+	case bytes >= TB:
+		return fmt.Sprintf("%.1fTB", float64(bytes)/TB)
+	case bytes >= GB:
+		return fmt.Sprintf("%.1fGB", float64(bytes)/GB)
+	case bytes >= MB:
+		return fmt.Sprintf("%.1fMB", float64(bytes)/MB)
+	case bytes >= KB:
+		return fmt.Sprintf("%.1fKB", float64(bytes)/KB)
+	default:
+		return fmt.Sprintf("%dB", bytes)
+	}
+}