fix: update vite + nextjs config to ignore rescript build artifacts

[illusionalsagacity]

Ignoring these directories prevents HMR reloads when the rescript compiler runs, ensuring HMR is only triggered when the output files are actually changed by the compiler.

https://forum.rescript-lang.org/t/compiler-warnings-can-cause-hmr-de-optimizations/7020?u=illusionalsagacity

[nojaf]

Thanks, this looks good.
@cknitt or @fhammerschmidt could you give your blessings on the Next changes. I don't know how that works.

[fhammerschmidt]

yeah it looks weird to me at least, why isn't it just like in vite?

e.g.:

config.watchOptions = {
  ...config.watchOptions,
  ignored = [
    "**/lib/bs/**",
    "**/lib/ocaml/**",
    "**/lib/rescript.lock"
  ]
}

And I think you should never use path.join where glob patterns are expected, cause they normalize to \-paths on Windows, which is too early and then it's not interpreted as a path separator anymore. Webpack handles path normalization itself.

[illusionalsagacity]

I updated the next.js webpack watch options to use the globs like vite

then it's not interpreted as a path separator anymore. Webpack handles path normalization itself.

Ah, I'd always used path.join in my webpack configs before, I hadn't run into this before.

[fhammerschmidt]

Thanks a lot!