Use native RegExp Match indices

[aslakhellesoy]

Summary

Use RegExpExecArray#indices to get the position of capture groups.

Details

This PR removes the dependency on the becke-ch--regex--s0-0-v1--base--pl--lib library and uses RegExpExecArray#indices instead.

RegExp Match Indices were introduced in v8 v9.0.259. The first Node.js release to use a v8 newer than that is 16.4.0.

Since we cannot drop support for Node.js < 16.4.0 until 2023/2024, the implementation currently uses a polyfill.

Motivation and Context

The main motivation for this change is to stop depending on http://www--s0-v1.becke.ch/tool/becke-ch--regex--s0-v1/becke-ch--regex--s0-0-v1--homepage--pl--client/, which seems to be a liability to depend on because it doesn't seem to have a public Git repo, except for this fork.

I would also expect this to be slightly more performant since it's based on a native implementation, but I doubt it will be significant for end users.

Also see #300

How Has This Been Tested?

The tests pass on Node v16.4.2, but I doubt it will pass on Node < 16.4.0 as explained above.

Types of changes

• Bug fix (non-breaking change which fixes an issue).
• New feature (non-breaking change which adds functionality).
• Breaking change (fix or feature that would cause existing functionality to not work as expected).

Checklist:

• The change has been ported to Java.
• The change has been ported to Ruby.
• The change has been ported to JavaScript.
• The change has been ported to Go.
• The change has been ported to .NET.
• I've added tests for my code.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have updated the CHANGELOG accordingly.

[aslakhellesoy]

Update: I added the polyfill so we don't have to wait until 2023/2024 to merge this.

[aurelien-reeves]

👍

[mattwynne]

👍

If we're using the polyfill let's edit the text in the PR to reflect that and remove the warnings.

[aslakhellesoy]

PR text updated