Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
This PR is being reviewed by Cursor Bugbot
Details
You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
| @@ -1,5 +1,3 @@ | |||
| { | |||
| "/page": "app/page.js", | |||
| "/progress/page": "app/progress/page.js", | |||
There was a problem hiding this comment.
Head shake navigation leads to removed /about route
Medium Severity
The build manifests are being modified to remove the /about route, but FaceGestureTracker in mediapipe-hand-tracker.tsx still calls router.push('/about') when a head shake gesture is detected. Users who enable face gestures and perform a head shake will be navigated to a non-existent route, resulting in a 404 error.
Additional Locations (1)
1 participant
Note
Low Risk
Low risk because this only updates generated
.nextmanifest outputs, effectively dropping route entries for pages that no longer exist. Main risk is accidental commit of build artifacts causing confusion across environments.Overview
The Next.js build manifests under
.nextwere updated to remove route entries for/progress/page,/about/page, and/gallery/page, leaving only the root/page(and/layout) in the compiled output.This change appears to reflect fewer built app routes being produced/packaged in the current build.
Written by Cursor Bugbot for commit e1329fc. This will update automatically on new commits. Configure here.