-
-
Notifications
You must be signed in to change notification settings - Fork 126
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
RangeError: Maximum call stack size exceeded #281
Comments
No, doesn't ring a 🔔 unfortunately. Without a reproduction there isn't much I can see/do. Most likely an infinite recursion and/or cyclic dependency. |
A variation of this error was caused in my project by a circular dependency. I was able to setup I wonder if there is anything useful that could be logged for debugging this scenario? |
I've released v2.31.0 which might fix it (probably not, though). Otherwise I'll need a reproduction. Using |
Closing this due to inactivity. Feel free to discuss further or open a new issue. Note that I'd need a reproduction. |
I have the same issue on the newest version ( now can't really give a repro, since it's a company project. I'll try to find out if there's a circular dependency, as mentioned above |
Knip v4 is about to mitigate some of these issues: https://knip.dev/blog/slim-down-to-speed-up (you can install and try v4 canary today). Recently added --isolate-workspaces (in v3) which may also help in certain edge cases. |
Hi there!
I work on a very big monorepo, and I encountered an issue when I run these commands with the latest version (2.30.1):
npx knip --no-gitignore
As we have multiple workspaces also tried just in one workspace (around 1000 files to analyze):
npx knip --no-gitignore --workspace apps/fr-app --exports
But I got a Heap of memory then I tried
NODE_OPTIONS=--max-old-space-size=5120 npx knip --no-gitignore --workspace apps/fr-app --exports
Also tried without
npx
with script in thepackage.json
But in any case, I always get this error:
Additional information: I didn't setup a knip config file, I used the default config only.
Does it ring a bell? Tell me if you need more information 🙂
The text was updated successfully, but these errors were encountered: