fix(service-worker): bind fetch to globalThis

[sapphi-red]

This PR fixes #3176 correctly. It was not working with #3200.

I added a test but it's full of mocks and probably not much useful. Feel free to remove it 😅
Ideally an e2e test is needed.

The author should do the following, if applicable

• Add tests
• Run tests
• bun run format:fix && bun run lint:fix to format the code
• Add TSDoc/JSDoc to document the code

[yusukebe]

Hi @sapphi-red

Thanks! I'll check it later.

[yusukebe]

Hi @sapphi-red Thank you for the PR.

I added a test but it's full of mocks and probably not much useful. Feel free to remove it 😅

I think it's good to use the mocks.

Ideally an e2e test is needed.

Exactly. If we decide to add it, we can put it into the runtime-tests. But we don't have to hurry.

---

I've left one comment. Please check it!

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.58%. Comparing base (31b4cd4) to head (f79db99).
Report is 2 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3500   +/-   ##
=======================================
  Coverage   95.58%   95.58%           
=======================================
  Files         155      155           
  Lines        9357     9357           
  Branches     2730     2761   +31     
=======================================
  Hits         8944     8944           
  Misses        413      413           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[yusukebe]

LGTM!

[yusukebe]

@sapphi-red

Looks good to me! Thank you for your contribution. I'll merge this and release a new version.