Log messages that contains a semicolon are not replaced correctly. For example:
console.log("Ok;");
or
console.log(["Test", "Ok"].map(function(s){return s;}));
I've added two unit tests that are failing with the current code.
This PR updates the Regex inner expression to detect the log message from:
\([^;]*\)
to
\((?:[^)(]+|\((?:[^)(]+|\([^)(]*\))*\))*\)
The new expression is based on matching nested brackets. Note that this will only work up to two levels of nesting. To add levels, you could change the middle (second) [^()]* part to ([^()]|([^()]))*, but I believe supporting two levels should be sufficient for most cases.
Log messages that contains a semicolon are not replaced correctly. For example:
console.log("Ok;");orconsole.log(["Test", "Ok"].map(function(s){return s;}));I've added two unit tests that are failing with the current code.This PR updates the Regex inner expression to detect the log message from:
\([^;]*\)to\((?:[^)(]+|\((?:[^)(]+|\([^)(]*\))*\))*\)The new expression is based on matching nested brackets. Note that this will only work up to two levels of nesting. To add levels, you could change the middle (second) [^()]* part to ([^()]|([^()]))*, but I believe supporting two levels should be sufficient for most cases.