@serwist/next - additionalPrecacheEntries prevents automatically precaching public dir files

[DuCanhGH]

Discussed in #138

^{Originally posted by jon-smith April 30, 2024}
I'm using @serwist/next with a statically exported Next.js app and it's working well, but I'm confused about some precaching options and I'm unsure if it's an issue or I'm using it incorrectly.

I noticed that the index file doesn't get precached by default (I want to ensure this is in sync with the precached js/css files), so I have set up my init as follows:

const withSerwist = withSerwistInit({
	swSrc: 'src/app/sw.ts',
	swDest: 'public/sw.js',
	additionalPrecacheEntries: [{ url: '/', revision }],
});

However this then prevents some files in my public directory from being precached. I noticed in the code (https://github.com/serwist/serwist/blob/main/packages/next/src/index.ts#L165), globPublicPatterns is not used if additionalPrecacheEntries have been set. Is there a reason for this?

The simple workaround is to add my public files to additionalPrecacheEntries but I feel like I might be just using it wrong.