webserver: gh, better callout

Author: tstromberg

README.md

@@ -95,13 +95,17 @@ Analyze all PRs across an entire organization:
 prcost --org chainguard-dev
 
 # Custom sampling parameters
-prcost --org myorg --samples 100 --days 60
+prcost --org myorg --samples 50 --days 60
 ```
 
 ### Sampling Strategy
 
 Repository and organization modes use time-bucket sampling to ensure even distribution across the time period. This provides more representative estimates than random sampling by avoiding temporal clustering.
 
+**Sample Size Options:**
+- **25 samples** (default): Fast analysis with ±20% confidence interval
+- **50 samples**: Slower but more accurate with ±14% confidence interval (1.4× better accuracy)
+
 ## Library Usage
 
 All functionality is available as reusable library packages:

cmd/prcost/main.go

@@ -30,7 +30,7 @@ func main() {
 	// Org/Repo sampling flags
 	org := flag.String("org", "", "GitHub organization to analyze (optionally with --repo for single repo)")
 	repo := flag.String("repo", "", "GitHub repository to analyze (requires --org)")
-	samples := flag.Int("samples", 20, "Number of PRs to sample for extrapolation")
+	samples := flag.Int("samples", 25, "Number of PRs to sample for extrapolation (25=fast/±20%, 50=slower/±14%)")
 	days := flag.Int("days", 90, "Number of days to look back for PR modifications")
 
 	flag.Usage = func() {
@@ -52,7 +52,7 @@ func main() {
 		fmt.Fprintf(os.Stderr, "    %s --org myorg --repo myrepo --samples 50 --days 30\n\n", os.Args[0])
 		fmt.Fprint(os.Stderr, "  Organization-wide analysis:\n")
 		fmt.Fprintf(os.Stderr, "    %s --org chainguard-dev\n", os.Args[0])
-		fmt.Fprintf(os.Stderr, "    %s --org myorg --samples 100 --days 60\n", os.Args[0])
+		fmt.Fprintf(os.Stderr, "    %s --org myorg --samples 50 --days 60\n", os.Args[0])
 	}
 
 	flag.Parse()

internal/server/server.go

@@ -13,6 +13,7 @@ import (
 	"net/http"
 	"net/url"
 	"os"
+	"os/exec"
 	"regexp"
 	"strings"
 	"sync"
@@ -678,6 +679,23 @@ func (s *Server) token(ctx context.Context) string {
 		return token
 	}
 
+	// Try gh auth token if gh is in PATH
+	if ghPath, err := exec.LookPath("gh"); err == nil {
+		s.logger.InfoContext(ctx, "Found gh CLI in PATH", "path", ghPath)
+		cmd := exec.CommandContext(ctx, "gh", "auth", "token")
+		output, err := cmd.Output()
+		if err == nil {
+			token := strings.TrimSpace(string(output))
+			if token != "" {
+				s.logger.InfoContext(ctx, "Using GITHUB_TOKEN from gh auth token")
+				s.fallbackToken = token
+				return token
+			}
+		} else {
+			s.logger.WarnContext(ctx, "Failed to get token from gh auth token", errorKey, err)
+		}
+	}
+
 	// Try Google Secret Manager for GITHUB_TOKEN
 	token, err := gsm.Fetch(ctx, "GITHUB_TOKEN")
 	if err != nil {
@@ -691,7 +709,7 @@ func (s *Server) token(ctx context.Context) string {
 		return token
 	}
 
-	s.logger.WarnContext(ctx, "No fallback GitHub token found (tried GITHUB_TOKEN env and GITHUB_TOKEN GSM)")
+	s.logger.WarnContext(ctx, "No fallback GitHub token found (tried GITHUB_TOKEN env, gh auth token, and GITHUB_TOKEN GSM)")
 	return ""
 }
 

internal/server/static/index.html

@@ -988,10 +988,9 @@ <h1>PR Cost Calculator</h1>
                             id="repoSampleSize"
                             value="25"
                             min="1"
-                            max="25"
-                            disabled
+                            max="50"
                         >
-                        <div class="help-text">Fixed at 25 PRs</div>
+                        <div class="help-text">25 (fast, ±20% accuracy) or 50 (slower, ±14% accuracy)</div>
                     </div>
                     <div class="form-group">
                         <label for="repoDays">Days Back</label>
@@ -1029,10 +1028,9 @@ <h1>PR Cost Calculator</h1>
                             id="orgSampleSize"
                             value="25"
                             min="1"
-                            max="25"
-                            disabled
+                            max="50"
                         >
-                        <div class="help-text">Fixed at 25 PRs</div>
+                        <div class="help-text">25 (fast, ±20% accuracy) or 50 (slower, ±14% accuracy)</div>
                     </div>
                     <div class="form-group">
                         <label for="orgDays">Days Back</label>
@@ -1301,33 +1299,27 @@ <h3>Why calculate PR costs?</h3>
                 savings -= pricingCost;
 
                 if (savings > 0) {
-                    // Format savings without decimals
-                    const savingsRounded = Math.round(savings);
-
-                    // Format savings in millions or thousands
-                    let savingsFormatted;
-                    if (savingsRounded >= 1000000) {
-                        const millions = (savingsRounded / 1000000).toFixed(1);
-                        savingsFormatted = '$' + millions + 'M';
-                    } else if (savingsRounded >= 1000) {
-                        const thousands = Math.round(savingsRounded / 1000);
-                        savingsFormatted = '$' + thousands + 'K';
-                    } else {
-                        savingsFormatted = '$' + savingsRounded.toLocaleString('en-US');
-                    }
+                    // Calculate average waste per PR
+                    const avgWasteHoursPerPR = extPreventableHours / e.total_prs;
+                    const avgWasteCostPerPR = extPreventableCost / e.total_prs;
+
+                    // Estimate PRs per author: total PRs in period / unique users in sample
+                    const daysInPeriod = parseInt(days);
+                    const prsPerPersonInPeriod = e.total_prs / userCount;
+                    const prsPerPersonPerYear = prsPerPersonInPeriod * (365 / daysInPeriod);
 
-                    // Calculate headcount savings (fully-loaded salary)
-                    const fullyLoadedSalary = salary * benefitsMultiplier;
-                    const headcountSavings = Math.round(savings / fullyLoadedSalary);
+                    // Calculate annual waste per author
+                    const annualWasteHoursPerAuthor = avgWasteHoursPerPR * prsPerPersonPerYear;
+                    const annualWasteCostPerAuthor = avgWasteCostPerPR * prsPerPersonPerYear;
 
-                    // Use sourceName if available, otherwise default to "you"
-                    const targetName = sourceName || 'you';
+                    // Weekly waste per author
+                    const weeklyWastePerAuthor = annualWasteHoursPerAuthor / 52;
+                    const hoursPerWeek = weeklyWastePerAuthor.toFixed(1);
+                    const costPerPersonFormatted = '$' + Math.round(annualWasteCostPerAuthor).toLocaleString('en-US');
 
                     html += '<div class="efficiency-callout">';
-                    html += "<p style=\"font-size: 20px; font-weight: 600; margin-bottom: 20px; line-height: 1.4;\">Your eng team isn't slow. Your process is.</p>";
-                    html += `<p style="font-size: 17px; margin-bottom: 16px;">codeGROOVE can save <strong>${targetName}</strong> <strong>${savingsFormatted}+ annually</strong>—that's <strong>${headcountSavings} senior engineers</strong> worth of productivity back on your team.</p>`;
-                    html += '<p style="font-size: 17px; margin-bottom: 20px;">We collapse review cycles so your team ships faster. <strong>ROI shows week one.</strong> Free forever for open-source projects.</p>';
-                    html += '<p style="margin: 0;"><a href="mailto:go-faster@codeGROOVE.dev" style="font-size: 17px;">go-faster@codeGROOVE.dev</a></p>';
+                    html += `<p style="font-size: 16px; margin-bottom: 16px; line-height: 1.5;">Your devs lose <strong>${hoursPerWeek} hours/week</strong> waiting on code reviews. That's <strong>${costPerPersonFormatted}/year per person</strong>. Ready-to-Review fixes this.</p>`;
+                    html += '<p style="font-size: 16px; margin-bottom: 0; line-height: 1.5;">Free for OSS, $5/dev/month for private. Beta begins November 3rd: <a href="mailto:beta@codeGROOVE.dev">beta@codeGROOVE.dev</a></p>';
                     html += '</div>';
                 }
             }