Conversation
There was a problem hiding this comment.
Greptile Summary
This PR claims to downgrade Next.js to fix Bun bundling compatibility issues, but actually upgrades Next.js from version ^15.3.2 to ^15.4.1. The changes add package overrides in both the root package.json and apps/sim/package.json to enforce specific versions of Next.js and @next/env across the entire dependency tree.
The key changes include:
- Moving Next.js version from
^15.3.2to^15.4.1in both package files - Adding
overridessections to force consistent versions across all workspace packages - Moving
@next/envfromdevDependenciesto theoverridessection in the root package.json
The overrides mechanism ensures all transitive dependencies use the specified versions, which can help resolve version conflicts. However, this contradicts the stated goal since upgrading to a newer Next.js version is unlikely to resolve compatibility issues with Bun that were present in the previous version.
PR Description Notes:
- The PR title says "downgrade nextjs" but the change is actually an upgrade
- The description claims incompatibility with "nextjs 15" but upgrades within the Next.js 15 series
- Testing section just repeats "downgrade nextjs" without specific test details
Confidence score: 1/5
- This PR has a fundamental contradiction between intent and implementation that could cause production issues
- Score reflects the mismatch between claimed downgrade and actual upgrade, which may not resolve the stated Bun compatibility issues
- Both package.json files require immediate attention to verify the correct Next.js version needed for Bun compatibility
2 files reviewed, 2 comments
| "mammoth": "^1.9.0", | ||
| "mysql2": "3.14.3", | ||
| "next": "^15.3.2", | ||
| "next": "^15.4.1", |
There was a problem hiding this comment.
logic: Version changed from ^15.3.2 to ^15.4.1 - this is an upgrade, not a downgrade as claimed in PR title/description
| "next": "^15.4.1", | ||
| "@next/env": "^15.4.1" |
There was a problem hiding this comment.
logic: This upgrades Next.js from 15.3.2 to 15.4.1, contradicting the PR title claiming a 'downgrade'. If Bun has compatibility issues with Next.js 15, upgrading to a newer 15.x version may not resolve the problem.
Summary
downgrade nextjs because bun bundling is incompatible with nextjs 15
Type of Change
Testing
downgrade nextjs
Checklist