feat: enable all validators and center SVG chart (#61)

* fix: create PR instead of direct push, test with tjs+schemasafe only

* fix: add ajv to test validators (required for README generation)

* fix: use PAT_TOKEN for PR creation to trigger required checks

* fix: use PAT for git push to trigger PR workflows

* fix: use peter-evans/create-pull-request to trigger PR workflows

* feat: enable all validators and center SVG chart

* fix: remove 'returning early' text from SVG

* fix: use head-to-head comparison in SVG chart

Update update-svg.ts to use the same comparison logic as the PR comment:
only tests where BOTH validators pass are compared (apples-to-apples).

This ensures the SVG chart shows consistent performance ratios with
the PR comment benchmarks.

* fix: use adjusted ops/sec in SVG based on head-to-head ratios

Calculate each validator's "adjusted" ops/sec from the head-to-head
ratio comparison. This ensures apples-to-apples comparison where each
bar represents performance on only the tests that both validators pass.

- tjs baseline is taken from the comparison with most shared tests
- Other validators' ops are calculated as: tjs_baseline_ops / ratio
- Maintains familiar ops/sec format while being truly comparable

Author: sberan

.github/workflows/update-benchmarks.yml

@@ -5,38 +5,16 @@ on:
     branches: [main]
   workflow_dispatch:
 
-# Runs full benchmarks on main and commits updated results
+# Runs full benchmarks on main and opens a PR with updated results
 
 permissions:
   contents: write
+  pull-requests: write
 
 jobs:
-  # Check if this is a benchmark commit to prevent infinite loop
-  check-skip:
-    runs-on: ubuntu-latest
-    outputs:
-      should_skip: ${{ steps.check.outputs.should_skip }}
-    steps:
-      - uses: actions/checkout@v4
-        with:
-          fetch-depth: 1
-      - name: Check if benchmark commit
-        id: check
-        run: |
-          COMMIT_MSG=$(git log -1 --format=%s)
-          if [[ "$COMMIT_MSG" == "chore: update benchmark results" ]]; then
-            echo "should_skip=true" >> $GITHUB_OUTPUT
-            echo "Skipping: benchmark commit detected"
-          else
-            echo "should_skip=false" >> $GITHUB_OUTPUT
-            echo "Running: not a benchmark commit"
-          fi
-
   # Run benchmarks for all validators (except jsonschema)
   benchmark:
     runs-on: ubuntu-latest
-    needs: check-skip
-    if: needs.check-skip.outputs.should_skip != 'true'
     strategy:
       matrix:
         draft: [draft4, draft6, draft7, draft2019-09, draft2020-12]
@@ -71,17 +49,15 @@ jobs:
           name: benchmark-${{ matrix.validator }}-${{ matrix.draft }}
           path: benchmarks/results/${{ matrix.validator }}-${{ matrix.draft }}.json
 
-  # Merge results, update files, and commit
+  # Merge results, update files, and open PR
   finalize:
     runs-on: ubuntu-latest
-    needs: [check-skip, benchmark]
-    if: needs.check-skip.outputs.should_skip != 'true'
+    needs: [benchmark]
 
     steps:
       - uses: actions/checkout@v4
         with:
           submodules: true
-          token: ${{ secrets.PAT_TOKEN }}
 
       - name: Use Node.js 22
         uses: actions/setup-node@v4
@@ -123,35 +99,28 @@ jobs:
           # Generate BENCHMARKS.md
           npx tsx benchmarks/generate-summary.ts
 
-          # Update README.md
-          npx tsx benchmarks/update-readme.ts
+          # Update README.md (requires ajv.json)
+          if [ -f benchmarks/results/ajv.json ]; then
+            npx tsx benchmarks/update-readme.ts
+          else
+            echo "Skipping README update (ajv.json not found)"
+          fi
 
           # Update SVG chart
           npx tsx benchmarks/update-svg.ts
 
-          # Generate detailed markdown reports for each validator
-          for v in ajv zod joi is-my-json-valid z-schema djv jsen schemasafe; do
+          # Generate detailed markdown reports for each validator that has results
+          for v in $(ls benchmarks/results/*.json 2>/dev/null | xargs -n1 basename | sed 's/.json//' | grep -v tjs); do
             UPPER_NAME=$(echo "$v" | tr '[:lower:]-' '[:upper:]_')
             npx tsx benchmarks/generate-report.ts "$v" -o "benchmarks/results/BENCHMARK_${UPPER_NAME}.md"
           done
 
-      - name: Check for changes
-        id: check_changes
-        run: |
-          git add -A
-          if git diff --staged --quiet; then
-            echo "has_changes=false" >> $GITHUB_OUTPUT
-            echo "No changes detected"
-          else
-            echo "has_changes=true" >> $GITHUB_OUTPUT
-            echo "Changes detected:"
-            git diff --staged --stat
-          fi
-
-      - name: Commit benchmark updates
-        if: steps.check_changes.outputs.has_changes == 'true'
-        run: |
-          git config user.name "github-actions[bot]"
-          git config user.email "github-actions[bot]@users.noreply.github.com"
-          git commit -m "chore: update benchmark results"
-          git push
+      - name: Create PR with benchmark updates
+        uses: peter-evans/create-pull-request@v7
+        with:
+          token: ${{ secrets.PAT_TOKEN }}
+          commit-message: "chore: update benchmark results"
+          title: "chore: update benchmark results"
+          body: "Automated benchmark update from main branch."
+          branch: chore/update-benchmarks
+          delete-branch: true