istanbul-instrumenter-loader doesn't support optional catch bindings

[scottohara]

webpack-contrib/istanbul-instrumenter-loader#96

• optional catch binding not supported until instanbul-lib-instrument@2.2.0
• version in instanbul-instrumenter-loader@3.0.1 is ^1.7.3
• for now, added (_e) to each affected catch block

If we can't commit to the offer to take over maintenance of the entire istanbul-instrumenter-loader project, another option would be to create our own fork that just bumps the dependency version.

[scottohara]

Also doesn't support optional chaining (foo?.bar)

Example: dataSync-controller_spec.ts, find specs:

scenarios.forEach((scenario: Scenario): void => {
  describe(scenario.type ?? "invalid sync type", (): void => {

Had to change to:

scenarios.forEach((scenario: Scenario): void => {
  describe(null === scenario.type ? "invalid sync type" : scenario.type, (): void => { 

[scottohara]

Investigate switching to https://github.com/JS-DevTools/coverage-istanbul-loader

The README indicates:

Coverage Istanbul Loader is based on Istanbul Instrumenter Loader, but uses a newer version of the Istanbul API and has better support for source maps.

The current loader is configured as:

{
	test: /\.(?:t|j)s$/u,
	loader: "istanbul-instrumenter-loader",
	options: {
		esModules: true
	},
	exclude: /node_modules/u,
	enforce: "post"
}

The alternate loader appears to support the same config, so it should be a drop-in replacement.

[scottohara]

DataDog appear to have confirmed this works with webpack 5:

DataDog/browser-sdk#1020