Cache never expires about nw-builder HOT 3 CLOSED

@ayushmanchhabra this still seems to be a problem. I just updated my package.json to NW version 0.77.0-sdk and observed the following error:

[ ERROR ] 2023-06-20T14:33:26.951Z The specified version does not exist in the version manifest. Disable cache to redownload the version manifest. If you still get this error, that means the version you've specified is incorrect.

Deleting cache\manifest.json resolves the issue. Please fix this caching implementation...

My motivation for caching manifest.json was to allow builds without network (if the relevant nw binary is already downladed, that is).

You need to actually try to update the cache, though. Only use the existing manifest.json if a new download was unsuccessful. Otherwise, it should be re-written with the latest version.

from nw-builder.

would it be an acceptable fix to always download manifest.json? It's very lightweight and shouldn't be cached anyways. I can work on a PR if the fix sounds good to you @tharatau .

from nw-builder.

@williamwang96 My motivation for caching manifest.json was to allow builds without network (if the relevant nw binary is already downladed, that is). Ideally we should redownload manifest.json when connected to network and not redownload while not connected to network (so that builds don't fail).

Update: Feel free to open a PR!

from nw-builder.