Claude Sonnet 5 btw
The SW fix itself is in good shape now — the stray import { register } from 'module' is gone, no-void is resolved, and the { type: 'module' } dev registration is…
Closing this in favor of the version now on lotus (f2c356f2) — I took your approach here (the withOriginBaseUrl(getOriginBaseUrl(), …) helper) and landed it there, co-authored to you. 🙌
…
Re-reviewed at head 50bb5094. The core fix is right and worth landing — registering the dev SW with { type: 'module' } is correct (/dev-sw.js is an ES module under VitePWA's devOptions, so the…
Re-reviewed at head 6a0037ec. The nvm / .node-version walkthrough is a genuinely useful addition — that Node-setup detail wasn't spelled out anywhere. Two things before it can land:
1.…
Re-reviewed at the current head (0aef410f) — thanks for cleaning up the formatting/lint in the last two commits; prettier and eslint both pass on the changed files now. 👍
One substantive…