Inline scripts with `type` attribute don't minify

[teddybradford]

Initial checklist

• I read the support docs
• I read the contributing guide
• I agree to follow the code of conduct
• I searched issues and discussions and couldn’t find anything (or linked relevant results below)

Affected package

rehype-minify-javascript-script@5.0.1

Steps to reproduce

If a script tag has type attribute it doesn't seem to get minified.

<script type="module">
        console.log('hello, world');

        console.log('hello, world');
</script>

<script>
        console.log('hello, world');

        console.log('hello, world');
</script>

Actual behavior

<script type="module">
        console.log('hello, world');

        console.log('hello, world');
</script>

<script>console.log("hello, world"),console.log("hello, world")</script>

Expected behavior

<script type="module">console.log("hello, world"),console.log("hello, world")</script>

<script>console.log("hello, world"),console.log("hello, world")</script>

Runtime

No response

Package manager

No response

Operating system

No response

Build and bundle tools

No response

[wooorm]

Hi! Your issue title is about type but your examples are about a specific type=module. Perhaps you can spend some time investigating what is what. Is this about type? Or about type=module? Is it about other values for type which are synonyms for JS modules?

[teddybradford]

The script doesn't get minified if type has a value. For example:

This gets minified because type has an empty value:

<script type="">
        console.log('hello, world');

        console.log('hello, world');
</script>

This does not get minified because type has a value:

<script type="foobar">
        console.log('hello, world');

        console.log('hello, world');
</script>