Add **/*.wasm?module as a default module rule

[GregBrimble]

What this PR solves / how to test:

Associated docs issues/PR:

• Document .wasm?module support cloudflare-docs#7750

Author has included the following, where applicable:

• Tests
• Changeset

Reviewer has performed the following, where applicable:

• Checked for inclusion of relevant tests
• Checked for inclusion of a relevant changeset
• Checked for creation of associated docs updates
• Manually pulled down the changes and spot-tested

Fixes # [insert issue number].

[changeset-bot]

🦋 Changeset detected

Latest commit: 34802e5

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
wrangler	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[github-actions]

A wrangler prerelease is available for testing. You can install this latest build in your project with:

npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/4252812557/npm-package-wrangler-2783

You can reference the automatically updated head of this PR with:

npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/prs/2783/npm-package-wrangler-2783

Or you can use npx with this latest build directly:

npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/4252812557/npm-package-wrangler-2783 dev path/to/script.js

Additional artifacts:

npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/4252812557/npm-package-cloudflare-pages-shared-2783

Note that these links will no longer work once the GitHub Actions artifact expires.

[petebacondarwin]

As discussed offline, we feel comfortable that adding this feature will not unduly constrain Wrangler in the future:

• it only adds an import that currently would cause a deployment to crash
• the semantic of this import is unlikely to change in the future
• if it does the user will be able to override it, initially, via a custom rule in the Worker config - and by this time, hopefully, Pages projects will also support being configured in the same way
• if the general meaning of this import changes in a way that we would want to change the default, we could do this using the standard deprecation and major version bump breaking change process that Wrangler already uses.

[codecov]

Codecov Report

Merging #2783 (34802e5) into main (26d8066) will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2783      +/-   ##
==========================================
+ Coverage   73.97%   74.00%   +0.02%     
==========================================
  Files         166      166              
  Lines       10163    10163              
  Branches     2705     2705              
==========================================
+ Hits         7518     7521       +3     
+ Misses       2645     2642       -3     

Impacted Files	Coverage Δ
packages/wrangler/src/module-collection.ts	97.36% <ø> (ø)
packages/wrangler/src/git-client.ts	43.75% <0.00%> (+4.16%)	⬆️
...ackages/wrangler/src/__tests__/helpers/mock-bin.ts	100.00% <0.00%> (+5.26%)	⬆️

[dario-piotrowicz]

I can see we have tests/fixtures for *.wasm, should we also add some for *.wasm?module?

(maybe not, just asking 🙂)

[dario-piotrowicz]

Looks good to me 😄 (based on my very limited knowledge of this code)