Skip to content

Refactor isRequire probe #220

Closed
Closed
@fraxken

Description

@fraxken

The idea/goal of the task is to refactor the isRequire who is becoming more complicated to read and maintain.

I think the part where we walk the require arguments need to be externalized to a dedicated class: https://github.com/NodeSecure/js-x-ray/blob/master/src/probes/isRequire.js#L142

However this can be a challenge because of how we currently walk the AST.

Metadata

Metadata

Assignees

Labels

help wantedExtra attention is needed

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions