fix: wrap long project/branch names instead of truncating

Author: backstabslash

format.go

@@ -99,12 +99,22 @@ func shortProject(slug string) string {
 	if s == "" {
 		s = slug
 	}
-	if len(s) > 40 {
-		s = "..." + s[len(s)-37:]
-	}
 	return s
 }
 
+func wrapName(name string, chunkSize int) []string {
+	if name == "" {
+		return nil
+	}
+	var chunks []string
+	for len(name) > chunkSize {
+		chunks = append(chunks, name[:chunkSize])
+		name = name[chunkSize:]
+	}
+	chunks = append(chunks, name)
+	return chunks
+}
+
 type OutputOptions struct {
 	ShowDaily    bool
 	ShowMonthly  bool
@@ -547,23 +557,26 @@ func printProjectBreakdown(data *ParseResult, opts OutputOptions) {
 		}
 		sort.Slice(sorted, func(i, j int) bool { return sorted[i].bucket.Cost > sorted[j].bucket.Cost })
 
-		first := true
-		for _, m := range sorted {
-			b := m.bucket
+		names := wrapName(name, 30)
+		for i, m := range sorted {
 			n := ""
-			if first {
-				n = name
-				if len(n) > 35 {
-					n = n[:32] + "..."
-				}
+			if i < len(names) {
+				n = names[i]
 			}
+			b := m.bucket
 			fmt.Printf("  %-35s %s %7d %s\n",
 				n, cyan.Sprintf("%-16s", shortModel(m.name)),
 				b.Requests, colorCost(b.Cost, 10))
-			first = false
+		}
+		for i := len(sorted); i < len(names)-1; i++ {
+			fmt.Printf("  %s\n", names[i])
+		}
+		subtotalName := ""
+		if len(names) > len(sorted) {
+			subtotalName = names[len(names)-1]
 		}
 		fmt.Printf("  %-35s %-16s %7s %s\n",
-			"", "SUBTOTAL", "", colorCost(proj.total, 10))
+			subtotalName, "SUBTOTAL", "", colorCost(proj.total, 10))
 		fmt.Println()
 	}
 }
@@ -604,23 +617,26 @@ func printBranchBreakdown(data *ParseResult, opts OutputOptions) {
 			}
 			sort.Slice(sorted, func(i, j int) bool { return sorted[i].bucket.Cost > sorted[j].bucket.Cost })
 
-			firstBranch := true
-			for _, m := range sorted {
-				b := m.bucket
-				bn := ""
-				if firstBranch {
-					bn = br.branch
-					if len(bn) > 30 {
-						bn = bn[:27] + "..."
-					}
+			names := wrapName(br.branch, 25)
+			for i, m := range sorted {
+				n := ""
+				if i < len(names) {
+					n = names[i]
 				}
+				b := m.bucket
 				fmt.Printf("  %-30s %s %7d %s\n",
-					bn, cyan.Sprintf("%-16s", shortModel(m.name)),
+					n, cyan.Sprintf("%-16s", shortModel(m.name)),
 					b.Requests, colorCost(b.Cost, 10))
-				firstBranch = false
+			}
+			for i := len(sorted); i < len(names)-1; i++ {
+				fmt.Printf("  %s\n", names[i])
+			}
+			subtotalName := ""
+			if len(names) > len(sorted) {
+				subtotalName = names[len(names)-1]
 			}
 			fmt.Printf("  %-30s %-16s %7s %s\n",
-				"", "SUBTOTAL", "", colorCost(br.total, 10))
+				subtotalName, "SUBTOTAL", "", colorCost(br.total, 10))
 			fmt.Println()
 		}
 	}

format_test.go

@@ -56,8 +56,8 @@ func TestShortProject(t *testing.T) {
 		{"simple-project", "simple/project"},
 		// Double hyphens collapsed
 		{"a--b", "a/b"},
-		// Truncation at 40 chars
-		{"a-very-long-project-name-that-exceeds-the-forty-character-limit", "...hat-exceeds-the-forty-character-limit"},
+		// Long names preserved
+		{"a-very-long-project-name-that-exceeds-the-forty-character-limit", "a/very-long-project-name-that-exceeds-the-forty-character-limit"},
 		// Empty slug returns original
 		{"", ""},
 	}
@@ -69,6 +69,32 @@ func TestShortProject(t *testing.T) {
 	}
 }
 
+func TestWrapName(t *testing.T) {
+	tests := []struct {
+		name     string
+		chunk    int
+		expected []string
+	}{
+		{"short", 10, []string{"short"}},
+		{"abcdefghij", 10, []string{"abcdefghij"}},
+		{"abcdefghijklmnopqrst", 10, []string{"abcdefghij", "klmnopqrst"}},
+		{"abcdefghijklmnopqrstuvwxyz12345", 10, []string{"abcdefghij", "klmnopqrst", "uvwxyz1234", "5"}},
+		{"", 10, nil},
+	}
+	for _, tt := range tests {
+		got := wrapName(tt.name, tt.chunk)
+		if len(got) != len(tt.expected) {
+			t.Errorf("wrapName(%q, %d) len = %d, want %d", tt.name, tt.chunk, len(got), len(tt.expected))
+			continue
+		}
+		for i := range got {
+			if got[i] != tt.expected[i] {
+				t.Errorf("wrapName(%q, %d)[%d] = %q, want %q", tt.name, tt.chunk, i, got[i], tt.expected[i])
+			}
+		}
+	}
+}
+
 func TestFmtDuration(t *testing.T) {
 	tests := []struct {
 		input    time.Duration