chore: move the visual reporter to React Router 8

packages/visual-reporter/.npmignore

@@ -14,6 +14,8 @@ temp/
 out/
 public/
 build/static/report/
+# React Router 8 keeps the small server build that renders index.html; the CLI only serves build/client
+build/server/
 demo/
 app/
 scripts/

packages/visual-reporter/app/entry.server.tsx

@@ -6,7 +6,7 @@
 
 import { PassThrough } from 'node:stream'
 
-import type { AppLoadContext, EntryContext } from 'react-router'
+import type { EntryContext } from 'react-router'
 import { createReadableStreamFromReadable } from '@react-router/node'
 import { ServerRouter } from 'react-router'
 import { isbot } from 'isbot'
@@ -20,10 +20,6 @@ export default function handleRequest(
     responseStatusCode: number,
     responseHeaders: Headers,
     routerContext: EntryContext,
-    // This is ignored so we can keep it in the template for visibility.  Feel
-    // free to delete this parameter in your app if you're not using it!
-    // eslint-disable-next-line @typescript-eslint/no-unused-vars
-    loadContext: AppLoadContext
 ) {
     return isbot(request.headers.get('user-agent') || '')
         ? handleBotRequest(

packages/visual-reporter/package.json

@@ -35,8 +35,8 @@
     "sirv-cli": "^3.0.1"
   },
   "devDependencies": {
-    "@react-router/dev": "^7.18.4",
-    "@react-router/node": "^7.18.4",
+    "@react-router/dev": "^8.4.0",
+    "@react-router/node": "^8.4.0",
     "@types/react": "^19.3.0",
     "@types/react-dom": "^19.3.0",
     "autoprefixer": "^10.6.1",
@@ -45,7 +45,7 @@
     "react": "^19.3.0",
     "react-dom": "^19.3.0",
     "react-icons": "^5.7.0",
-    "react-router": "^7.18.4",
+    "react-router": "^8.4.0",
     "react-select": "^5.10.2",
     "tailwindcss": "^4.3.3",
     "typescript": "^6.0.3",