Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bun does not replace RegExp #10508

Closed
teyfix opened this issue Apr 25, 2024 · 3 comments · Fixed by #11188
Closed

Bun does not replace RegExp #10508

teyfix opened this issue Apr 25, 2024 · 3 comments · Fixed by #11188
Labels
bug Something isn't working jsc Something related to JavaScriptCore

Comments

@teyfix
Copy link

teyfix commented Apr 25, 2024

What version of Bun is running?

1.1.4+fbe2fe0c3

What platform is your computer?

Darwin 23.2.0 arm64 arm

What steps can reproduce the bug?

Bun does not replace positive lookbehind RegExp /(?<=[,:] |TData = |<)(\w+(?:Error|Variables|Response))/g when Node.js does but it does match. Also when I run the tests, it fails to check if matched array is equal to expected. It succeeds when i explicitly check item by item.
You can reproduce the issue with this repository https://github.com/teyfik-on-it/bun-regex-match
Repository also has actions for node and bun that you can checkout to see the issue.

What is the expected behavior?

It should replace the expression

What do you see instead?

It does not replace the expression

Additional information

All the required information is available in the repository.

@teyfix teyfix added the bug Something isn't working label Apr 25, 2024
@accountsosoci
Copy link

Any updates?

@ufuck-on-it
Copy link

Still not solved?

@dylan-conway
Copy link
Collaborator

This is a bug in JSC. Filed a bug with a minified example https://bugs.webkit.org/show_bug.cgi?id=273254

@dylan-conway dylan-conway added the jsc Something related to JavaScriptCore label Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working jsc Something related to JavaScriptCore
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants