don't ask me why I had to put %r{} around it, I just followed rubocop's instructions.

is there any documentaion online that i can use as a guide line for the review?

If you've never seen regexes before, they are a pain in the ass to review. They are kind of like a language in their own right, and the main way to understand regexes is to play around building them and testing them out on a site that shows you what the regex matches.
I've taken the regex here and prepared an example for you: https://regex101.com/r/RLd8UL/1
If you'd like to have a quick online meeting about how to understand this, let me know. I have time tomorrow

I have seen regex before, and it is almost unreadable. thank you for the resource

The best way to test the regex is probably to come up with valid HTML img tags that are not matched by the regex. The edge cases I could think of have been captured in this one.

I am cheating a little bit by using chatgpt, it has some suggestions and explaination, i will check it and comment it

this one also checks for space in between the elements and has better checking for the imageurl

you seem to have removed the first capturing group. chatgpt has that tendency because it doesn't see you using its result anywhere in your regex. However, we definitely use this capturing group to produce the new text. What I do like, is the use of \s; that's any whitespace right?

What exactly does the latter part improve? Because I don't think it really improves anything. Let me break my thoughts down:

• original: (.+?)\2( |>|/>) where \2 matches to the same value as the captured opening quote (either ', " or nothing at all). (.+?) lazily matches any character. That means that it will begin with src=' or src=" or src= and then it will continue to match any character until the first time it encounters that same quotation mark again. Subsequently, after the quotation mark, it needs to match one of the three options in ( |>|/>) which are a space, the > or />.
• your suggestion: ([^'">]+)\1(?=[/>])} where \1 matches the same value as the captured opening quote. Note that [^'">] is unnecessarily restrictive: I don't know exactly the specification of HTML, but I can image that something like src='lookatthisdoublequote"isntitamazing' is valid. Notice the " in the middle? Your regex suggestion would stop the src value at that middle quote, because it does not pass the [^'"] check. And the latter part, (?=[/>]) does not allow for any whitespace.

I do have some ideas based on your suggestion:

• use \s instead of spaces when I'm indicating whitespace
• I've reconsidered whether the last capturing group is necessary, but yes it is, because when we have src=somesource, we can only determine the end of the source value when we encounter a whitespace or / or >. But I can change it from ( |>|/>) to ( |>|/) which can be put simpler by writing [ >/]. However, if I want to match for \s instead of a space, I can't use the [] notation, I think

I've applied the new ideas I mentioned in the latest commit

This part was my bad "(?=[/>]) does not allow for any whitespace'' i asked chat to remove the whitespace.
I was unaware that this is a valid source "src='lookatthisdoublequote"isntitamazing'", but i can agree that it is to restictive