Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,7 @@
"@types/react-dom": "^18",
"@types/react-syntax-highlighter": "^15.5.13",
"eslint": "^8",
"eslint-config-next": "14.2.5",
"eslint-config-next": "14.2.28",

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

critical

This PR proposes changing eslint-config-next from 14.2.5 to 14.2.28.

However, there are two major concerns:

  1. Apparent Downgrade: Based on the official release dates for Next.js (which eslint-config-next versions typically follow):

    • v14.2.5 (current version in your package.json) was released on May 20, 2024.
    • v14.2.28 (proposed version) was released on April 8, 2024.
      This means the proposed change is a downgrade to an older version, despite the patch number 28 being numerically higher than 5. Downgrading could reintroduce previously fixed bugs or remove improvements present in 14.2.5.
  2. Package Availability: Versions 14.2.5 and 14.2.28 of eslint-config-next do not appear to be published on the public npm registry. The latest version on npm is 14.2.4.

Could you please clarify:

  • How does your project resolve these specific Next.js package versions (e.g., private npm registry, direct Git dependencies)?
  • Assuming these versions are accessible, is there a specific, critical reason (e.g., a severe vulnerability in 14.2.5 not present in 14.2.28) that Snyk is recommending this downgrade? The Snyk PR title says "Upgrade", which seems contradictory here.

Unless there's a compelling justification for this downgrade, this change should likely be rejected. It's highly unusual to downgrade, especially when initiated by a tool like Snyk, without a clear explanation of a critical issue in the newer version.

"knip": "^5.41.1",
"micromatch": "^4.0.8",
"postcss": ">=8.4.31",
Expand Down