improve text

Author: tstromberg

Makefile

@@ -1,7 +1,15 @@
 .PHONY: test
-test:
+test: test-go test-js
+
+.PHONY: test-go
+test-go:
 	go test -race -cover ./...
 
+.PHONY: test-js
+test-js:
+	@echo "Running JavaScript tests..."
+	@node internal/server/static/formatR2RCallout.test.js
+
 # BEGIN: lint-install .
 # http://github.com/codeGROOVE-dev/lint-install
 

internal/server/server.go

@@ -2348,9 +2348,8 @@ func (s *Server) processOrgSampleWithProgress(ctx context.Context, req *OrgSampl
 // processPRsInParallel processes PRs in parallel and sends progress updates via SSE.
 //
 //nolint:revive // line-length/use-waitgroup-go: long function signature acceptable, standard wg pattern
-func (s *Server) processPRsInParallel(workCtx, reqCtx context.Context, samples []github.PRSummary, defaultOwner, defaultRepo, token string, cfg cost.Config, writer http.ResponseWriter) ([]cost.Breakdown, map[string]int) {
-	var breakdowns []cost.Breakdown
-	aggregatedSeconds := make(map[string]int)
+func (s *Server) processPRsInParallel(workCtx, reqCtx context.Context, samples []github.PRSummary, defaultOwner, defaultRepo, token string, cfg cost.Config, writer http.ResponseWriter) (breakdowns []cost.Breakdown, aggregatedSeconds map[string]int) {
+	aggregatedSeconds = make(map[string]int)
 	var mu sync.Mutex
 	var sseMu sync.Mutex // Protects SSE writes to prevent corrupted chunked encoding
 

internal/server/static/index.html

@@ -1480,9 +1480,9 @@ <h3>Why calculate PR costs?</h3>
             }
 
             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 += '✓ You\'re losing <strong>' + savingsText + '/yr</strong>' + throughputText + ' to code review lag. ';
-            html += '<a href="https://codegroove.dev/" target="_blank" rel="noopener" style="color: #00c853; font-weight: 600; text-decoration: none;">Ready-to-Review</a> fixes it: <strong>&lt;40min merges</strong>, free for OSS. ';
-            html += '<a href="mailto:go-faster@codeGROOVE.dev" style="color: #00c853; text-decoration: none;">go-faster@codeGROOVE.dev</a>';
+            html += 'Pro-Tip: Save <strong>' + savingsText + '/yr</strong> in lost development effort by reducing merge times to &lt;1h with ';
+            html += '<a href="https://codegroove.dev/" target="_blank" rel="noopener" style="color: #00c853; font-weight: 600; text-decoration: none;">Ready-to-Review</a>. ';
+            html += 'Free for OSS, cheap for everyone else.';
             html += '</div>';
             return html;
         }

pkg/cost/cost_test.go

@@ -187,6 +187,8 @@ func TestCalculateWithRealPRData(t *testing.T) {
 		} `json:"events"`
 		PullRequest struct {
 			CreatedAt         string `json:"created_at"`
+			ClosedAt          string `json:"closed_at"`
+			MergedAt          string `json:"merged_at"`
 			Author            string `json:"author"`
 			Additions         int    `json:"additions"`
 			AuthorWriteAccess int    `json:"author_write_access"`
@@ -219,11 +221,20 @@ func TestCalculateWithRealPRData(t *testing.T) {
 		t.Fatalf("Failed to parse created_at: %v", err)
 	}
 
+	var closedAt time.Time
+	if prxData.PullRequest.ClosedAt != "" {
+		closedAt, err = time.Parse(time.RFC3339, prxData.PullRequest.ClosedAt)
+		if err != nil {
+			t.Fatalf("Failed to parse closed_at: %v", err)
+		}
+	}
+
 	prData := PRData{
 		LinesAdded: prxData.PullRequest.Additions,
 		Author:     prxData.PullRequest.Author,
 		Events:     events,
 		CreatedAt:  createdAt,
+		ClosedAt:   closedAt,
 	}
 
 	cfg := DefaultConfig()