Conversation
|
@kenny2077 is attempting to deploy a commit to the umn social coding Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Devin Review found 1 potential issue.
⚠️ 1 issue in files not directly in the diff
⚠️ Yarn installs depend on unrelated mirror
Yarn installs now fetch 31 platform packages from registry.npmmirror.com, including the Linux Next.js compiler. Yarn follows each resolved URL directly. An inaccessible mirror blocks frontend installation and deployment on affected platforms.
0499dcb to
a8b604d
Compare
|
Resolved the lockfile registry issue in a8b604d. I regenerated yarn.lock with Yarn 1.22.22 against https://registry.npmjs.org, confirmed that both committed lockfiles now reference only registry.npmjs.org, and reran a fresh-cache frozen Yarn install, TypeScript, ESLint, and the production build successfully. |
Summary
nextand@next/mdxfrom the previously locked 16.0.7 release to 16.3.4Security impact
Next.js 16.3.3 was a security release, and 16.3.4 is its current follow-up release.
Against the current
mainlockfile,npm audit --omit=dev --package-lock-onlyclassifiesnext@16.0.7as a direct critical vulnerability. Its advisory list includes unauthenticated remote-code-execution issues on Windows-hosted servers (GHSA-p293-qw3h-jr36) and in AVIF image optimization (GHSA-2xp9-vwfh-vxw4). After this upgrade, the production audit contains no advisory attributed tonext.Compatibility
The existing
>=18.xengine declaration was also looser than Next.js 16's actual>=20.9.0requirement, so this PR makes that contract explicit.Verification
npm ciinstall --frozen-lockfile --force --non-interactivenpx tsc --noEmitnpx eslint . --ext .js,.jsx,.ts,.tsx --ignore-pattern next-env.d.ts(0 errors; one existingreact-hooks/exhaustive-depswarning)npm run build/,/class/CSCI1933, and/api/search?q=CSCInpm audit --omit=dev --package-lock-only: production findings decrease from 18 to 14, critical findings decrease from 2 to 1, and thenextentry is removedScope
The remaining production audit findings include existing
sqlite3andlodashdependency paths and should be handled separately. React, unrelated dependency upgrades, and cleanup of the existing lint script are intentionally excluded from this PR.