|
This regex replacement is not 100% safe because transforming JavaScript requires an actual parser.
However, parsing (e.g. via babel) comes with its own problems because now the parser needs to
be aware of syntax extensions which might not be supported by the parser, but the underlying
JavaScript engine. In fact, rewire used to have babel in place but required an extra
transform for the object spread operator (check out commit d9a81c0cdacf6995b24d205b4a2068adbd8b34ff
or see https://github.com/jhnns/rewire/pull/128). It was also notable slower
(see https://github.com/jhnns/rewire/issues/132).
There is another issue: replacing const with let is not safe because of their different behavior.
That's why we also have ESLint in place which tries to identify this error case.
There is one edge case though: when a new syntax is used *and* a const re-assignment happens,
rewire would compile happily in this situation but the actual code wouldn't work.
However, since most projects have a seperate linting step which catches these const re-assignment
errors anyway, it's probably still a reasonable trade-off.
Fixes https://github.com/jhnns/rewire/issues/132
|
