Update code from service worker to module worker format

[vicb]

Summary

Update code from service worker to module worker format

Documentation checklist

• The documentation style guide has been adhered to.

Question:

I have added only the used parameters

async fetch(request): Promise<Response>

The full signature is

async fetch(request, env, ctx): Promise<Response>

Specifying only the used parameters is valid syntax.
Are there guidelines to prefer one over the other syntax?

My plan is to convert most example to using the new syntax as the old syntax is deprecated. I'll send more PR after this one gets in.

[hyperlint-ai]

4 files reviewed, 3 total issue(s) found.

---

Note: We resolved prior Hyperlint review comments because:

We updated our inline suggestion AI.

We do this to avoid keeping outdated or irrelevant comments around. We'll leave a new review with current comments below.

[vicb]

Let's use async fetch(request, env, ctx): Promise<Response>

[vicb]

As discuss we Brendan, I updated the PR to always use

async fetch(request, env, ctx): Promise<Response>

even if env and ctx are unused (because not supported in the legacy service worker)

[vicb]

@ToriLindsay @dcpena - can we get ✅ to merge this?

[ToriLindsay]

@vicb Approved. Thanks!

[vicb]

Thanks for the review and merge!