-
-
Notifications
You must be signed in to change notification settings - Fork 536
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
Panic in oxc-miette
, when linting file bigger that 64KB
#9738
Comments
I am unable to reproduce your issue: PS D:\shulaoda\oxlint-repo> pnpm oxlint -D all -D nursery --import-plugin --jsdoc-plugin --jest-plugin --vitest-plugin --jsx-a11y-plugin --nextjs-plugin --react-perf-plugin index.js --fix --fix-suggestions --fix-dangerously
⚠ File is too long to fit on the screen
help: "index.js" seems like a minified file
Found 0 warnings and 1 error.
Finished in 5ms on 1 file with 485 rules using 20 threads. BTW, I am using |
I cannot replicate either, and I can't figure out how it could panic in |
With this binary I can reproduce this each time - https://github.com/qarmin/Automated-Fuzzer/releases/download/Nightly/oxlint.7z This looks like - rust-lang/rust#102528 In CI, this broken between It may be related to overflow-checks that I enable by default in release builds width seems to have strange value |
Close as not planned until we can replicate this reliably. |
I was able to reproduce this with newest nightly rust compiler, by compiling oxlint with just
|
I found also rust commit with change - rust-lang/rust@fb9ce02 |
Ah ok, it's a rustc breaking change, |
Fixed by oxc-project/oxc-miette#20 |
File content(at the bottom should be attached raw, not formatted file - github removes some non-printable characters, so copying from here may not work):
(Truncated, because real file have size 64KB)
command
cause this
compressed.zip
The text was updated successfully, but these errors were encountered: