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

Broken Debugger #2386

Open
BrainyGenius opened this issue Nov 13, 2024 · 6 comments
Open

Broken Debugger #2386

BrainyGenius opened this issue Nov 13, 2024 · 6 comments

Comments

@BrainyGenius
Copy link

Bug Description

Regex Debugger displays outdated data.

Reproduction steps

  1. Make a complicated expression like this: /^\b(-|+)?((.|,)?\d+((.|,)?\d+|\d+((.|,)?)))\b/gi
  2. Type something in to be matched
  3. Open the debugger
  4. Change both the expression and the string to be matched

Expected Outcome

For the Debugger to show the new data matching steps

Browser

Include browser name and version
Google Chrome 130.0.6723.101 (Official Build) (64-bit)

OS

Include OS name and version
ChromeOS 130.0.6723.101 (Official Build) (64-bit)

@working-name
Copy link
Collaborator

Hi @BrainyGenius,

I have .117 but cannot reproduce this.

Also, not sure I follow the repro steps since you can't actually change the expression while the debugger is on, nor can you start the debugger when there's errors in the regex, like the |+)? part where + is treated as match 1 or more of the previous token, but the previous token is also a metacharacter which means OR.

Would you mind double checking the steps?

@BrainyGenius
Copy link
Author

@working-name
Even when I quit the debugger and change the string and expression that way, it still doesn't work right.
Screenshot 2024-11-15 3 28 46 PM
Screenshot 2024-11-15 3 28 38 PM

@BrainyGenius
Copy link
Author

I didn't mean to close the issue. That was an accident.

@working-name
Copy link
Collaborator

That is quite interesting. I've tried to recreate it but I cannot seem to. Even if I type the same regex and test string as you captured in your picture, it behaves normally - nothing matches because .117 is not at all like trhrthrthr.

While I don't have chromiumOS I doubt there's anything different about the browser.

@firasdib
Copy link
Owner

Are there any errors in the console? Can you record the issue?

@working-name
Copy link
Collaborator

@BrainyGenius just a quick ping to see if you had some time to look at the console and try to a screen recording.

Record your screen:

Press Shift + Ctrl + Show windows .
At the bottom, select Screen record .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants