Service Worker not defined while accessing PWA in development

[GirlBossRush]

As first reported in a PR by @cuining, this regression seems to be introduced in the move away from Parcel.

I believe this may be addressed in #3833, but some more information is still needed.

OS/Web Information

• Web Browser: Chrome
• Local OS: macOS
• Remote OS: macOS
• Remote Architecture: 64 bit
• code-server --version: 3.11.0

Steps to Reproduce

1. Start code-server via yarn watch
2. Open in Chrome, add a PWA shortcut
3. Refresh

Expected

No console errors should occur.

Actual

Service worker registration fails due to an invalid path. However, the file is available at an different path.

Screenshot

[jsjoeio]

Apologies @cuining for prematurely closing your PR and thank you for raising the issue! ♥️ (and @TeffenEllis for following up on this!)

3.11

I am not able to reproduce in 3.11, which is a good sign.

Local development

I am also not able to reproduce 🤔 (macOS + Brave)

[jsjoeio]

Wait, premature comment, I did not refresh

[jsjoeio]

Video

Screen.Recording.2021-07-21.at.2.25.43.PM.mov

Okay so actually I believe this is a different bug. I told @code-asher about this a while ago but couldn't reproduce consistently. This happened before the migration away from Parcel.

The workaround is to close the PWA and reopen.

But the fact that you're seeing the raw JS in the browser feels odd and I wonder if these are two separate bugs or the same, manifesting differently 🤔

[lsoica]

I have the same issue with 3.11. It looks for the serviceWorker.js at /node_modules/code-server/dist/serviceWorker.js but in fact the new location is /node_modules/code-server/out/browser/serviceWorker.js

[jsjoeio]

Ahhh, I must have forgot to change it in one place. Thank you for the tip @lsoica!

[jsjoeio]

Could this be a caching issue? i.e. it cached the old path. Would a hard refresh fix it?

[lsoica]

Cleaning app cache and browser file cache didn't help. Anything else that I've missed ?

[jsjoeio]

Nope! Thank you for trying. cc @code-asher in case he has any other ideas. We're going to look at this in the next cycle (starting next week 🤞)

[code-asher]

Ack we didn't update this path!

https://github.com/cdr/code-server/blob/7b8cd25c0c48b6a1799587ebbc9b09d6af7988f3/src/browser/register.ts#L8

We probably didn't notice during development because we had the old Parcel dist directory from previous builds.

[code-asher]

Looks like we should re-open that PR as it has the right fix.