-
Notifications
You must be signed in to change notification settings - Fork 27.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
parser.js from next-config-eslint is missing metadata #72833
Comments
We could not detect a valid reproduction link. Make sure to follow the bug report template carefully. Why was this issue closed?To be able to investigate, we need access to a reproduction to identify what triggered the issue. We need a link to a public GitHub repository (template for App Router, template for Pages Router), but you can also use these templates: CodeSandbox: App Router or CodeSandbox: Pages Router. The bug template that you filled out has a section called "Link to the code that reproduces this issue", which is where you should provide the link to the reproduction.
What should I do?Depending on the reason the issue was closed, you can do the following:
In general, assume that we should not go through a lengthy onboarding process at your company code only to be able to verify an issue. My repository is private and cannot make it publicIn most cases, a private repo will not be a sufficient minimal reproduction, as this codebase might contain a lot of unrelated parts that would make our investigation take longer. Please do not make it public. Instead, create a new repository using the templates above, adding the relevant code to reproduce the issue. Common things to look out for:
I did not open this issue, but it is relevant to me, what can I do to help?Anyone experiencing the same issue is welcome to provide a minimal reproduction following the above steps by opening a new issue. I think my reproduction is good enough, why aren't you looking into it quickly?We look into every Next.js issue and constantly monitor open issues for new comments. However, sometimes we might miss one or two due to the popularity/high traffic of the repository. We apologize, and kindly ask you to refrain from tagging core maintainers, as that will usually not result in increased priority. Upvoting issues to show your interest will help us prioritize and address them as quickly as possible. That said, every issue is important to us, and if an issue gets closed by accident, we encourage you to open a new one linking to the old issue and we will look into it. Useful Resources |
Link to the code that reproduces this issue
https://github.com/vercel/next.js/blob/canary/packages/eslint-config-next/parser.js
To Reproduce
Compare the exports in parser.js vs. the eslint specs for custom parsers.
Parser.js:
https://github.com/vercel/next.js/blob/canary/packages/eslint-config-next/parser.js
Eslint documentation that calls for including the meta section as part of a custom parser's exports:
https://eslint.org/docs/latest/extend/custom-parsers#meta-data-in-custom-parsers
Current vs. Expected behavior
Current behavior:
Next.js defines an eslint parser that doesn't include meta information
Expected behavior:
Next.js defines an eslint parser that follows spec and includes meta information
I see at least 2 possible fixes:
Add meta to the list of things from @babel/eslint-parser that parser.js imports and re-exports
or
Use @babel/eslint-parser directly without a wrapper (similar to how the typescript-eslint parser is included)
Provide environment information
Operating System: Platform: win32 Arch: x64 Version: Windows 11 Enterprise Available memory (MB): 47769 Available CPU cores: 12 Binaries: Node: 20.18.0 npm: 10.8.2 Yarn: N/A pnpm: 9.13.0 Relevant Packages: next: 15.0.3 // Latest available version is detected (15.0.3). eslint-config-next: 15.0.3 react: 18.3.1 react-dom: 18.3.1 typescript: 5.6.3 Next.js Config: output: export
Which area(s) are affected? (Select all that apply)
Developer Experience
Which stage(s) are affected? (Select all that apply)
next build (local)
Additional context
Had some issues initially when upgrading from eslint 8 -> 9 / nextjs 14 -> 15.
Noticed this issue in the eslint repo that discussed the same error I was seeing and noted that the eslint developers indicated the root cause was how Nextjs was setting up the babel parser:
eslint/eslint#19080
The text was updated successfully, but these errors were encountered: