Experience-Monks / Experience-Monks/babel-plugin-static-fs
Remove `path` if unused in scope?
- Dominant language
- JavaScript
- Stars
- 35
- Forks
- 11
- PR merge metrics
- No merged PRs in 30d
Description
Not sure if this is dangerous. The `path` module does not have any side-effects that I know of, so we could remove it entirely from the program when the only usage has been stripped out. This would significantly reduce bundle size in browserify if you aren't already requiring path/process/etc. However, it might introduce an unexpected change for the end user.
Contributor guide
Research direction
Start by tracing how the plugin handles the Node `path` module and how its transformed output is consumed by browserify. Determine whether removing an unused `path` dependency changes end-user behavior or bundle output, and define the compatibility checks needed before considering the issue done.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- build-system
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100