[PR #699] [MERGED] chore(benchmarks): remove Vite 7 runner, update Next.js to 16.2.1 #782

New issue

Closed

opened 2026-05-06 13:10:04 +02:00 by BreizhHardware · 0 comments

BreizhHardware commented

2026-05-06 13:10:04 +02:00

Owner

📋 Pull Request Information

Original PR: https://github.com/cloudflare/vinext/pull/699
Author: @southpolesteve
Created: 3/27/2026
Status: ✅ Merged
Merged: 3/28/2026
Merged by: @southpolesteve

Base: main ← Head: benchmarks/remove-vite7-update-nextjs

📝 Commits (2)

035a69b chore(benchmarks): remove Vite 7 runner, update Next.js to 16.2.1
72ffcc4 fix: remove vinext-rolldown from pnpm-workspace.yaml, fix migration comment

📊 Changes

18 files changed (+37 additions, -193 deletions)

View changed files

📝 AGENTS.md (+1 -1)
📝 README.md (+2 -2)
📝 benchmarks/.gitignore (+0 -4)
📝 benchmarks/generate-app.mjs (+1 -1)
📝 benchmarks/nextjs/package.json (+1 -1)
📝 benchmarks/run.mjs (+15 -119)
➖ benchmarks/vinext-rolldown/package.json (+0 -22)
➖ benchmarks/vinext-rolldown/tsconfig.json (+0 -20)
➖ benchmarks/vinext-rolldown/vite.config.ts (+0 -6)
📝 examples/benchmarks/README.md (+1 -1)
📝 examples/benchmarks/app/components/dashboard.tsx (+5 -5)
📝 examples/benchmarks/app/components/format.ts (+2 -4)
📝 examples/benchmarks/app/page.tsx (+2 -2)
📝 examples/benchmarks/app/styles.css (+0 -1)
📝 examples/benchmarks/migrations/0001_init.sql (+2 -2)
➕ examples/benchmarks/migrations/0002_remove_rolldown_runner.sql (+5 -0)
📝 examples/benchmarks/worker/index.ts (+0 -1)
📝 pnpm-workspace.yaml (+0 -1)

📄 Description

Summary

Remove the vinext-rolldown benchmark runner, which was identical to the vinext runner (both resolved to the same Vite version via catalog:), causing duplicate results on the dashboard
Simplify to a two-runner comparison: Next.js (Turbopack) vs vinext (Vite 8)
Bump Next.js benchmark dependency from 16.1.7 to 16.2.1

Changes

Benchmark runner (benchmarks/):

Delete vinext-rolldown/ directory
Remove all rolldown logic from run.mjs (--skip-rolldown, hasRolldown, vinextRolldown results)
Remove from generate-app.mjs copy targets
Clean up .gitignore

Dashboard (examples/benchmarks/):

Update runner labels: "Next.js (Turbopack)" and "vinext (Vite 8)"
Remove vinext_rolldown from worker RUNNER_MAP
Update DB CHECK constraint in init migration
Add 0002_remove_rolldown_runner.sql migration to clean existing data
Remove rolldown CSS theme color
Update CommitList to show vinext data instead of rolldown

Docs:

Update README.md benchmark description and Vite version notes
Update AGENTS.md Vite 8 defaults section

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/cloudflare/vinext/pull/699 **Author:** [@southpolesteve](https://github.com/southpolesteve) **Created:** 3/27/2026 **Status:** ✅ Merged **Merged:** 3/28/2026 **Merged by:** [@southpolesteve](https://github.com/southpolesteve) **Base:** `main` ← **Head:** `benchmarks/remove-vite7-update-nextjs` --- ### 📝 Commits (2) - [`035a69b`](https://github.com/cloudflare/vinext/commit/035a69b860a2a41ad0b1e917b186fc741ed21449) chore(benchmarks): remove Vite 7 runner, update Next.js to 16.2.1 - [`72ffcc4`](https://github.com/cloudflare/vinext/commit/72ffcc47c30673e7431022954c147ff77ad5b1bc) fix: remove vinext-rolldown from pnpm-workspace.yaml, fix migration comment ### 📊 Changes **18 files changed** (+37 additions, -193 deletions) <details> <summary>View changed files</summary> 📝 `AGENTS.md` (+1 -1) 📝 `README.md` (+2 -2) 📝 `benchmarks/.gitignore` (+0 -4) 📝 `benchmarks/generate-app.mjs` (+1 -1) 📝 `benchmarks/nextjs/package.json` (+1 -1) 📝 `benchmarks/run.mjs` (+15 -119) ➖ `benchmarks/vinext-rolldown/package.json` (+0 -22) ➖ `benchmarks/vinext-rolldown/tsconfig.json` (+0 -20) ➖ `benchmarks/vinext-rolldown/vite.config.ts` (+0 -6) 📝 `examples/benchmarks/README.md` (+1 -1) 📝 `examples/benchmarks/app/components/dashboard.tsx` (+5 -5) 📝 `examples/benchmarks/app/components/format.ts` (+2 -4) 📝 `examples/benchmarks/app/page.tsx` (+2 -2) 📝 `examples/benchmarks/app/styles.css` (+0 -1) 📝 `examples/benchmarks/migrations/0001_init.sql` (+2 -2) ➕ `examples/benchmarks/migrations/0002_remove_rolldown_runner.sql` (+5 -0) 📝 `examples/benchmarks/worker/index.ts` (+0 -1) 📝 `pnpm-workspace.yaml` (+0 -1) </details> ### 📄 Description ## Summary - Remove the `vinext-rolldown` benchmark runner, which was identical to the `vinext` runner (both resolved to the same Vite version via `catalog:`), causing duplicate results on the dashboard - Simplify to a two-runner comparison: **Next.js (Turbopack)** vs **vinext (Vite 8)** - Bump Next.js benchmark dependency from 16.1.7 to 16.2.1 ## Changes **Benchmark runner (`benchmarks/`):** - Delete `vinext-rolldown/` directory - Remove all rolldown logic from `run.mjs` (`--skip-rolldown`, `hasRolldown`, `vinextRolldown` results) - Remove from `generate-app.mjs` copy targets - Clean up `.gitignore` **Dashboard (`examples/benchmarks/`):** - Update runner labels: "Next.js (Turbopack)" and "vinext (Vite 8)" - Remove `vinext_rolldown` from worker `RUNNER_MAP` - Update DB `CHECK` constraint in init migration - Add `0002_remove_rolldown_runner.sql` migration to clean existing data - Remove rolldown CSS theme color - Update CommitList to show vinext data instead of rolldown **Docs:** - Update `README.md` benchmark description and Vite version notes - Update `AGENTS.md` Vite 8 defaults section --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>