From da4dc3b604f6bc5ca31c35a3a71cdc4a3dde0057 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Tue, 3 Mar 2026 13:25:32 +0000
Subject: [PATCH 1/2] Initial plan


From 500683b11750f26dcfc6990e1d005dfad804b92c Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Tue, 3 Mar 2026 13:28:21 +0000
Subject: [PATCH 2/2] feat(review): change order of information display for a
 single goal

Co-authored-by: narthur <4655422+narthur@users.noreply.github.com>
---
 review.go | 24 ++++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/review.go b/review.go
index 7484912..a284c5c 100644
--- a/review.go
+++ b/review.go
@@ -224,9 +224,15 @@ func formatDueTime(deadlineOffset int) string {
 func formatGoalDetails(goal *Goal, config *Config, colorStyles map[string]lipgloss.Style) string {
 	var details string
 
-	// Display title only if not empty
-	if goal.Title != "" {
-		details += fmt.Sprintf("Title:       %s\n", goal.Title)
+	// Display current rate (n / unit)
+	if goal.Rate != nil && goal.Runits != "" {
+		rateStr := formatRate(*goal.Rate, goal.Runits, goal.Gunits)
+		details += fmt.Sprintf("Rate:        %s\n", rateStr)
+	}
+
+	// Display autoratchet only if set (not nil)
+	if goal.Autoratchet != nil {
+		details += fmt.Sprintf("Autoratchet: %.0f\n", *goal.Autoratchet)
 	}
 
 	// Display limsum with color coding based on urgency
@@ -254,10 +260,9 @@ func formatGoalDetails(goal *Goal, config *Config, colorStyles map[string]lipglo
 
 	details += fmt.Sprintf("Pledge:      $%.2f\n", goal.Pledge)
 
-	// Display current rate (n / unit)
-	if goal.Rate != nil && goal.Runits != "" {
-		rateStr := formatRate(*goal.Rate, goal.Runits, goal.Gunits)
-		details += fmt.Sprintf("Rate:        %s\n", rateStr)
+	// Display title only if not empty
+	if goal.Title != "" {
+		details += fmt.Sprintf("Title:       %s\n", goal.Title)
 	}
 
 	// Display autodata only if not empty
@@ -265,11 +270,6 @@ func formatGoalDetails(goal *Goal, config *Config, colorStyles map[string]lipglo
 		details += fmt.Sprintf("Autodata:    %s\n", goal.Autodata)
 	}
 
-	// Display autoratchet only if set (not nil)
-	if goal.Autoratchet != nil {
-		details += fmt.Sprintf("Autoratchet: %.0f\n", *goal.Autoratchet)
-	}
-
 	// Generate and display goal URL
 	baseURL := getBaseURL(config)
 	goalURL := fmt.Sprintf("%s/%s/%s", baseURL, url.PathEscape(config.Username), url.PathEscape(goal.Slug))