regexp-tree does not optimize (\.(?!$)|$))$

[kurtextrem]

Coming from this SO post: https://stackoverflow.com/a/36760050
The regex

^((25[0-5]|(2[0-4]|1\d|[1-9]|)\d)(\.(?!$)|$)){4}$

got optimized (by hand) to:

^((25[0-5]|(2[0-4]|1\d|[1-9]|)\d)\.?\b){4}$

so, partbefore(\.(?!$)|$))$ can be expressed as (partbefore\.?\b)$, avoiding the negative look-ahead, making the regex smaller.

Maybe worth adding as optimization?

[kurtextrem]

cross-referencing this reply from eslint-plugin-regexp: ota-meshi/eslint-plugin-regexp#659 (comment)

The regex is actually not faster, for different reasons. I'm not sure what regexp-tree optimizes for (fastest runtime or smallest regex), so ymmv if or what action you'd want to take. In any case, I want to avoid duplicate work while investigation :)

[DmitrySoshnikov]

@kurtextrem thanks for the report. Yes, this would be nice to add - feel free to send a PR in case.