Fixes #6273 Exclude JS module type from minification

[Tabrisrp]

Description

Exclude type="module" scripts from JS mifinication

Fixes #6273

Type of change

• Bug fix (non-breaking change which fixes an issue).

Is the solution different from the one proposed during the grooming?

XS issue, no grooming

Checklists

Generic development checklist

• My code follows the style guidelines of this project, with adapted comments and without new warnings.
• I have added unit and integration tests that prove my fix is effective or that my feature works.
• The CI passes locally with my changes (including unit tests, integration tests, linter).

Test summary

• I triggered all changed lines of code at least once without new errors/warnings/notices.

[CrochetFeve0251]

The code seems valid. However @piotrbak already created a PR to create a exclusion list maintained by the backend for that issue : https://github.com/wp-media/rucss-backend/issues/83

[piotrbak]

@CrochetFeve0251 @Tabrisrp Apologies, thought that the grooming will happen. I added this issue to the backen repository, so we could move this part there:

wp-rocket/inc/Engine/Optimization/Minify/JS/AbstractJSOptimization.php

Lines 100 to 104 in 3abb772

	false !== strpos( $tag[0], 'data-minify=' )
	||
	false !== strpos( $tag[0], 'data-no-minify=' )
	||
	preg_match( '/type=(?<quote>[\'"])module\g<quote>/', $tag[0] )