fix(integrations): Handle windows paths with no prefix or backslash prefix in RewriteFrames

[srubin]

Forward slashes are not allowed in Windows paths. This extends RewriteFrames to detect additional Windows-like paths and convert them to use forward slashes.

Before submitting a pull request, please take a look at our
Contributing guidelines and verify:

• If you've added code that should be tested, please add tests.
• Ensure your code lints and the test suite passes (yarn lint) & (yarn test).

[Lms24]

Hi @srubin thanks for opening this PR! On first glance this looks good to me. Gonna let CI run to check for tests. Just out of curiousity, is this something that you need for your use case? Under which circumstances do such frames occur? And furthermore, is there an open issue around this? (if not, that's alright, I just want to ensure we link things appropriately).

[srubin]

We were running into this problem with source maps generated on Windows in webpack. However, I found that I could fix this in our webpack config by standardizing the path separators in devtoolModuleFilenameTemplate.

There's not an open issue around this, and we're not blocked on the fix in this PR. For us it's cleaner to address it upstream (in our webpack config) anyway. So, no complaints if you close this PR!

[Lms24]

Thanks for the explanation!
Makes sense to me. I'm okay to merge this but it'd be great if you could look into the polynomial regex warning (just to confirm, can you see the CodeQL warning? Not sure about our repo settings here). Any chance that you could come up with a more efficient regex? Considering that this runs for every stack frame I'd like to avoid a potential performance bottleneck (stack trace parsing is already quite resource-intensive as-is).

[srubin]

Updated to use string.includes instead. I had to force push to change a commit message to use the fix(integrations): prefix. If the force push has messed something up here, let me know and I can start a clean branch for the final change.

[Lms24]

No worries, force push is not a problem (we do it all the time 😅). I restarted CI

[Lms24]

@srubin can you run yarn fix to fix the linting errors? The other test errors look like flakes to me so I think once the linter is happy, we can merge this.

[srubin]

Done.

[Lms24]

Perfect, then let's merge this. Thanks for contributing @srubin!