feat: fix benchzribe issues and add automated README updates

- Fix missing go.sum dependencies with go mod tidy
- Remove dead code in main.go by refactoring to use existing handlers
- Fix graph generation bug (missing fmt import and data["ops"] reference)
- Add new GitHub Actions workflow for automated README updates
- Update README to reflect working graph functionality
- Add test script for CI flow verification
- Generate real benchmark results and interactive graphs

Fixes:
- Graph generation now works correctly with proper data handling
- CI workflow automatically updates README with latest benchmark results
- All tests pass and code builds successfully

Author: tijo-thomaz

.github/workflows/bench.yml

@@ -42,6 +42,8 @@ jobs:
           cd mockapi
           go test ./internal/benchmarks -bench=. -benchmem > ../bench.out
           cd ..
+          echo "Generated benchmark results:"
+          cat bench.out
 
       - name: Store benchmark result
         uses: benchmark-action/github-action-benchmark@v1
@@ -63,10 +65,14 @@ jobs:
       - name: Update README
         run: |
           ./benchzribe run
+          echo "README updated, checking changes:"
+          git status
 
       - name: Generate graph
         run: |
           ./benchzribe graph
+          echo "Graph generated, checking files:"
+          ls -la *.html *.out
 
       - name: Upload artifacts
         uses: actions/upload-artifact@v4
@@ -77,7 +83,20 @@ jobs:
             benchmark-graph.html
           retention-days: 90
 
+      - name: Check for changes
+        id: verify-changed-files
+        run: |
+          if git diff --quiet; then
+            echo "No changes detected"
+            echo "has_changes=false" >> $GITHUB_OUTPUT
+          else
+            echo "Changes detected:"
+            git diff --name-only
+            echo "has_changes=true" >> $GITHUB_OUTPUT
+          fi
+
       - name: Commit changes
+        if: steps.verify-changed-files.outputs.has_changes == 'true'
         uses: stefanzweifel/git-auto-commit-action@v5
         with:
           commit_message: "chore: update benchmark results [skip ci]"

.github/workflows/update-benchmarks.yml

@@ -0,0 +1,90 @@
+name: Update Benchmark Results
+
+on:
+  push:
+    branches: [ main ]
+    paths:
+      - 'internal/**'
+      - 'cmd/**'
+      - 'mockapi/**'
+      - 'go.mod'
+      - 'go.sum'
+  workflow_dispatch:  # Allow manual triggering
+
+permissions:
+  contents: write
+
+jobs:
+  update-benchmarks:
+    name: Update README with Latest Benchmarks
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+          token: ${{ secrets.GITHUB_TOKEN }}
+
+      - name: Set up Go
+        uses: actions/setup-go@v5
+        with:
+          go-version: '1.21'
+          cache: true
+
+      - name: Install dependencies
+        run: |
+          go mod download
+          go mod tidy
+
+      - name: Build benchzribe
+        run: |
+          go build -v -o benchzribe ./cmd/benchzribe
+
+      - name: Run benchmarks
+        run: |
+          echo "Running benchmarks..."
+          cd mockapi
+          go test ./internal/benchmarks -bench=. -benchmem > ../bench.out
+          cd ..
+          echo "Benchmark results generated:"
+          cat bench.out
+
+      - name: Update README with benchmark results
+        run: |
+          echo "Updating README..."
+          ./benchzribe run
+          echo "README update completed"
+
+      - name: Generate benchmark graph
+        run: |
+          echo "Generating graph..."
+          ./benchzribe graph
+          echo "Graph generated: benchmark-graph.html"
+
+      - name: Check for changes
+        id: check_changes
+        run: |
+          git add -A
+          if git diff --staged --quiet; then
+            echo "No changes to commit"
+            echo "changes=false" >> $GITHUB_OUTPUT
+          else
+            echo "Changes detected:"
+            git diff --staged --name-only
+            echo "changes=true" >> $GITHUB_OUTPUT
+          fi
+
+      - name: Commit and push changes
+        if: steps.check_changes.outputs.changes == 'true'
+        run: |
+          git config --local user.email "action@github.com"
+          git config --local user.name "GitHub Action"
+          git commit -m "chore: update benchmark results [skip ci]
+
+          Auto-generated by benchzribe tool
+          - Updated README.md with latest benchmark results
+          - Generated new benchmark graph
+          
+          Triggered by: ${{ github.event_name }}
+          Commit: ${{ github.sha }}"
+          git push

README.md

@@ -6,7 +6,7 @@ A command-line tool for parsing, visualizing, and managing Go benchmark results.
 
 - 📊 Parse Go benchmark results and generate formatted tables
 - 📝 Automatically update README with benchmark results
-- 📈 Generate visualizations of benchmark data (coming soon)
+- 📈 Generate interactive visualizations of benchmark data
 - ⏱️ Track performance changes over time
 - 🔄 Easy integration with CI/CD pipelines
 
@@ -28,21 +28,31 @@ go test -bench . -benchmem ./... > bench.out
 benchzribe run
 ```
 
-3. Generate performance graphs (coming soon):
+3. Generate interactive performance graphs:
 ```bash
 benchzribe graph
 ```
 
 ## 📊 Benchmark Results
 
 <!-- BENCHSCRIBE:START -->
+### 📊 Benchmark Results
+
+| Benchmark | ns/op | B/op | allocs/op |
+|-----------|-------|------|------------|
+| SimpleOperation-12 | 545 | 0 | 0 |
+| StringConcatenation-12 | 9766 | 21080 | 99 |
+| SliceOperations-12 | 619 | 0 | 0 |
+| TestHandler-12 | 3351 | 6139 | 19 |
+
+_Last updated: Fri, 27 Jun 2025 02:02:30 BST_
 
 <!-- BENCHSCRIBE:END -->
 
 ## 🛠️ Commands
 
 - `benchzribe run` - Parse benchmark results and update README
-- `benchzribe graph` - Generate performance visualization (coming soon)
+- `benchzribe graph` - Generate interactive performance visualization
 - `benchzribe readme` - Manually update README section
 
 ## 🤝 Contributing

benchmark-graph.html

@@ -0,0 +1,24 @@
+
+<!DOCTYPE html>
+<html>
+<head>
+    <meta charset="utf-8">
+    <title>Awesome go-echarts</title>
+    <script src="https://go-echarts.github.io/go-echarts-assets/assets/echarts.min.js"></script>
+</head>
+
+<body><div class="container">
+    <div class="item" id="ghGUkJAHvaxJ" style="width:900px;height:500px;"></div>
+</div><script type="text/javascript">
+    "use strict";
+    let goecharts_ghGUkJAHvaxJ = echarts.init(document.getElementById('ghGUkJAHvaxJ'), "white", { renderer: "canvas" });
+    let option_ghGUkJAHvaxJ = {"color":["#5470c6","#91cc75","#fac858","#ee6666","#73c0de","#3ba272","#fc8452","#9a60b4","#ea7ccc"],"legend":{"show":true},"series":[{"name":"ns/op","type":"line","data":[{"value":544.7},{"value":9766},{"value":619.1},{"value":3351}]},{"name":"B/op","type":"line","data":[{"value":0},{"value":21080},{"value":0},{"value":6139}]},{"name":"allocs/op","type":"line","data":[{"value":0},{"value":99},{"value":0},{"value":19}]}],"title":{"text":"Benchmark Results"},"toolbox":{},"tooltip":{"show":true,"trigger":"axis"},"xAxis":[{"data":["Benchmark 1","Benchmark 2","Benchmark 3","Benchmark 4"]}],"yAxis":[{}]}
+
+    goecharts_ghGUkJAHvaxJ.setOption(option_ghGUkJAHvaxJ);
+</script>
+<style>
+    .container {margin-top:30px; display: flex;justify-content: center;align-items: center;}
+    .item {margin: auto;}
+</style>
+</body>
+</html>

benchzribe

@@ -71,50 +71,13 @@ func main() {
 }
 
 func run() error {
-	// Read benchmark results
-	results, err := parser.Parse("bench.out")
-	if err != nil {
-		return fmt.Errorf("failed to parse benchmark results: %w", err)
-	}
-
-	// Format results for README
-	var readmeContent strings.Builder
-	for _, r := range results {
-		fmt.Fprintf(&readmeContent, "### %s\n", r.Name)
-		fmt.Fprintf(&readmeContent, "- ns/op: %.2f\n", float64(r.NsPerOp))
-		fmt.Fprintf(&readmeContent, "- B/op: %.2f\n", float64(r.BytesPerOp))
-		fmt.Fprintf(&readmeContent, "- allocs/op: %.2f\n", float64(r.AllocsPerOp))
-	}
-
-	// Update README
-	if err := readme.Update("README.md", readmeContent.String()); err != nil {
-		return fmt.Errorf("failed to update README: %w", err)
-	}
-
-	return nil
+	cfg := config.DefaultConfig()
+	return handleRun(cfg)
 }
 
 func generateGraph() error {
-	// Read benchmark results
-	results, err := parser.Parse("bench.out")
-	if err != nil {
-		return fmt.Errorf("failed to parse benchmark results: %w", err)
-	}
-
-	// Convert results to graph data format
-	data := make(map[string][]float64)
-	for _, r := range results {
-		data["ns/op"] = append(data["ns/op"], float64(r.NsPerOp))
-		data["B/op"] = append(data["B/op"], float64(r.BytesPerOp))
-		data["allocs/op"] = append(data["allocs/op"], float64(r.AllocsPerOp))
-	}
-
-	// Generate graph
-	if err := graph.GenerateGraph(data); err != nil {
-		return fmt.Errorf("failed to generate graph: %w", err)
-	}
-
-	return nil
+	cfg := config.DefaultConfig()
+	return handleGraph(cfg)
 }
 
 func handleRun(cfg config.Config) error {

cmd/benchzribe/main.go

@@ -1,6 +1,7 @@
 package graph
 
 import (
+	"fmt"
 	"os"
 
 	"github.com/go-echarts/go-echarts/v2/charts"
@@ -31,10 +32,15 @@ func GenerateGraph(data map[string][]float64) error {
 		}),
 	)
 
-	// Add X axis
+	// Add X axis - use the first data series to determine length
 	var xAxis []string
-	for i := 0; i < len(data["ops"]); i++ {
-		xAxis = append(xAxis, "Run "+string(rune('A'+i)))
+	var dataLen int
+	for _, values := range data {
+		dataLen = len(values)
+		break
+	}
+	for i := 0; i < dataLen; i++ {
+		xAxis = append(xAxis, fmt.Sprintf("Benchmark %d", i+1))
 	}
 	line.SetXAxis(xAxis)
 

internal/graph/graph.go

@@ -0,0 +1,32 @@
+#!/bin/bash
+
+echo "🔧 Testing CI Flow for BenchZribe"
+echo "=================================="
+
+echo "1. Building benchzribe..."
+go build -v -o benchzribe ./cmd/benchzribe
+
+echo "2. Running benchmarks..."
+cd mockapi
+go test ./internal/benchmarks -bench=. -benchmem > ../bench.out
+cd ..
+
+echo "3. Generated benchmark results:"
+cat bench.out
+
+echo "4. Updating README with benchzribe..."
+./benchzribe run
+
+echo "5. Generating graph..."
+./benchzribe graph
+
+echo "6. Checking git status..."
+git status
+
+echo "7. Files generated:"
+ls -la *.html *.out 2>/dev/null || echo "No graph/bench files found"
+
+echo ""
+echo "✅ CI Flow test completed!"
+echo "The README should now contain updated benchmark results."
+echo "The benchmark-graph.html should contain an interactive chart."