'disable' option not disabling pwa feature in development mode.

[david-buck]

Summary

My next.config.mjs file has next-pwa configured like so:

export default withPWA({
  pwa: {
    dest: "public",
    register: false,
    skipWaiting: false,
    dynamicStartUrl: false,
    runtimeCaching,
    disable: process.env.NODE_ENV === "development",
  }, ...

The disable option seems to be ignored, even if set to disable: true which leads to all the quirks that come with running next-pwa in development mode.

Versions

• next-pwa: 5.5.5
• next: 12.2.3

How To Reproduce

Steps to reproduce the behavior:

1. add disable: true option to next.config.js
2. yarn dev
3. see warnings in console:

warn  - GenerateSW has been called multiple times, perhaps due to running webpack in --watch mode. The precache manifest generated after the first call may be inaccurate! Please see https://github.com/GoogleChrome/workbox/issues/1790 for more information.

[phil-madpaws]

Using this as a workaround for now:

module.exports = prod ? withPWA(nextConfig) : nextConfig;

[piro0919]

I think it works fine with v5.5.4.
I'm sorry, but I haven't investigated the cause yet.
I think maybe the flag handling is reversed.

[ymchun]

seems related to this line:

next-pwa/index.js

Line 338 in 9d28c3b

delete pwaNextConfig.pwa

.pwa config is deleted too early so that webpack function can't read any user config.

next-pwa/index.js

Line 56 in 9d28c3b

} = pwa

the value of pwa variable is {}

[faisalamin001]

Using this as a workaround for now:

module.exports = prod ? withPWA(nextConfig) : nextConfig;

it worked for me as well, thanks!
@phil-madpaws