-
Notifications
You must be signed in to change notification settings - Fork 29
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
Node.js? #67
Comments
Yes, JavaScript regular expressions are supported too. |
I see, how much does the file scanning support JS? I don't see anything in the readme about how it would scan JS files for regexes or what the limitations would be on dynamically constructed regexes like |
Regex extraction and regex analysis are separate problems. Regex extractionThe extraction relies on simple static analysis. It only finds regexes declared PRs to improve the static extraction are welcome. I'm not sure if the static analysis framework I used for JS handles dataflow analysis. Regex analysisThe super-linear regex detection ensemble exposed by this project is much more powerful than safe-regex's heuristic. |
I saw in your fork of
safe-regex
that you recommend this project instead. But is it designed to work on Node regular expressions? And in any case, maybe we should make a Node.js version to use with tools like ESLint?The text was updated successfully, but these errors were encountered: