Improve pro-tip output

Author: Thomas Stromberg

cmd/prcost/main.go

@@ -595,12 +595,12 @@ func printMergeTimeModelingCallout(breakdown *cost.Breakdown, cfg cost.Config) {
 		fmt.Println("  ┌─────────────────────────────────────────────────────────────┐")
 		fmt.Printf("  │ %-60s│\n", "MERGE TIME MODELING")
 		fmt.Println("  └─────────────────────────────────────────────────────────────┘")
-		fmt.Printf("  If you lowered your average merge time to %s, you would save\n", formatTimeUnit(targetHours))
-		fmt.Printf("  ~$%s/yr in engineering overhead", formatWithCommas(annualSavings))
 		if efficiencyDelta > 0 {
-			fmt.Printf(" (+%.1f%% throughput).\n", efficiencyDelta)
+			fmt.Printf("  Reduce merge time to %s to boost team throughput by %.1f%%\n", formatTimeUnit(targetHours), efficiencyDelta)
+			fmt.Printf("  and save ~$%s/yr in engineering overhead.\n", formatWithCommas(annualSavings))
 		} else {
-			fmt.Println(".")
+			fmt.Printf("  If you lowered your average merge time to %s, you would save\n", formatTimeUnit(targetHours))
+			fmt.Printf("  ~$%s/yr in engineering overhead.\n", formatWithCommas(annualSavings))
 		}
 		fmt.Println()
 	}

cmd/prcost/repository.go

@@ -738,12 +738,12 @@ func printExtrapolatedMergeTimeModelingCallout(ext *cost.ExtrapolatedBreakdown,
 		fmt.Println("  ┌─────────────────────────────────────────────────────────────┐")
 		fmt.Printf("  │ %-60s│\n", "MERGE TIME MODELING")
 		fmt.Println("  └─────────────────────────────────────────────────────────────┘")
-		fmt.Printf("  If you lowered your average merge time to %s, you would save\n", formatTimeUnit(targetHours))
-		fmt.Printf("  ~$%s/yr in engineering overhead", formatWithCommas(annualSavings))
 		if efficiencyDelta > 0 {
-			fmt.Printf(" (+%.1f%% throughput).\n", efficiencyDelta)
+			fmt.Printf("  Reduce merge time to %s to boost team throughput by %.1f%%\n", formatTimeUnit(targetHours), efficiencyDelta)
+			fmt.Printf("  and save ~$%s/yr in engineering overhead.\n", formatWithCommas(annualSavings))
 		} else {
-			fmt.Println(".")
+			fmt.Printf("  If you lowered your average merge time to %s, you would save\n", formatTimeUnit(targetHours))
+			fmt.Printf("  ~$%s/yr in engineering overhead.\n", formatWithCommas(annualSavings))
 		}
 		fmt.Println()
 	}

internal/server/static/formatR2RCallout.js

@@ -16,16 +16,12 @@ function formatR2RCallout(avgOpenHours, r2rSavings, currentEfficiency, modeledEf
     }
 
     const efficiencyDelta = modeledEfficiency - currentEfficiency;
-    let throughputText = '';
-    if (efficiencyDelta > 0) {
-        throughputText = ' (+' + efficiencyDelta.toFixed(1) + '% throughput)';
-    }
 
     // Format target merge time
     let targetText = targetMergeHours.toFixed(1) + 'h';
 
-    let html = '<div style="margin: 24px 0; padding: 12px 20px; background: linear-gradient(135deg, #e6f9f0 0%, #ffffff 100%); border: 1px solid #00c853; border-radius: 8px; font-size: 14px; color: #1d1d1f; line-height: 1.6;">';
-    html += 'Pro-Tip: Save <strong>' + savingsText + '/yr</strong> in lost development effort by reducing merge times to &lt;' + targetText + ' with ';
+    let html = '<div style="margin: 24px 0; padding: 12px 20px; background: linear-gradient(135deg, #e6f9f0 0%, #ffffff 100%); border: 1px solid #00c853; border-radius: 8px; font-size: 16px; color: #1d1d1f; line-height: 1.6;">';
+    html += '💡 <strong>Pro-Tip:</strong> Boost team throughput by <strong>' + efficiencyDelta.toFixed(1) + '%</strong> and save <strong>' + savingsText + '/yr</strong> by reducing merge times to &lt;' + targetText + ' with ';
     html += '<a href="https://codegroove.dev/products/ready-to-review/" target="_blank" rel="noopener" style="color: #00c853; font-weight: 600; text-decoration: none;">Ready to Review</a>. ';
     html += 'Free for open-source repositories, $6/user/org for private repos.';
     html += '</div>';

internal/server/static/formatR2RCallout.test.js

@@ -32,10 +32,13 @@ test('Renders callout for PRs with avgOpenHours > 1.5 (default)', () => {
     assert(result.length > 0, 'Should return non-empty HTML');
 });
 
-// Test 3: Should contain "Pro-Tip:" text
-test('Contains "Pro-Tip:" text', () => {
+// Test 3: Should contain "Pro-Tip:" text and throughput boost
+test('Contains "Pro-Tip:" text and throughput boost', () => {
     const result = formatR2RCallout(10, 50000, 60, 70);
+    assert(result.includes('💡'), 'Should contain lightbulb emoji');
     assert(result.includes('Pro-Tip:'), 'Should contain "Pro-Tip:"');
+    assert(result.includes('Boost team throughput by'), 'Should contain throughput boost message');
+    assert(result.includes('10.0%'), 'Should show efficiency delta of 10% (70 - 60)');
 });
 
 // Test 4: Should contain "Ready to Review" link

internal/server/static/index.html

@@ -1469,16 +1469,12 @@ <h3>Why calculate PR costs?</h3>
             }
 
             const efficiencyDelta = modeledEfficiency - currentEfficiency;
-            let throughputText = '';
-            if (efficiencyDelta > 0) {
-                throughputText = ' (+' + efficiencyDelta.toFixed(1) + '% throughput)';
-            }
 
             // Format target merge time
             let targetText = targetMergeHours.toFixed(1) + 'h';
 
-            let html = '<div style="margin: 24px 0; padding: 12px 20px; background: linear-gradient(135deg, #e6f9f0 0%, #ffffff 100%); border: 1px solid #00c853; border-radius: 8px; font-size: 14px; color: #1d1d1f; line-height: 1.6;">';
-            html += 'Pro-Tip: Save <strong>' + savingsText + '/yr</strong> in lost development effort by reducing merge times to &lt;' + targetText + ' with ';
+            let html = '<div style="margin: 24px 0; padding: 12px 20px; background: linear-gradient(135deg, #e6f9f0 0%, #ffffff 100%); border: 1px solid #00c853; border-radius: 8px; font-size: 16px; color: #1d1d1f; line-height: 1.6;">';
+            html += '💡 <strong>Pro-Tip:</strong> Boost team throughput by <strong>' + efficiencyDelta.toFixed(1) + '%</strong> and save <strong>' + savingsText + '/yr</strong> by reducing merge times to &lt;' + targetText + ' with ';
             html += '<a href="https://codegroove.dev/products/ready-to-review/" target="_blank" rel="noopener" style="color: #00c853; font-weight: 600; text-decoration: none;">Ready to Review</a>. ';
             html += 'Free for open-source repositories, $6/user/org for private repos.';
             html += '</div>';