Allow Client to Handle Reload

The current client implementation handles reloading by calling window.reload.

https://github.com/pikapkg/esm-hmr/blob/07bf62c2ff783a4ea3319c0b2baa3351b65c073c/src/client.ts#L16-L18

This doesn’t make sense for WebExtensions where reloading is done by calling runtime.reload. I’m sure there are other contexts where you’d want to customize the reload behavior also (electron?).

It would be great if there was an API to customize the implementation of this function.

Maybe something along the lines of

import.meta.hot.registerReloadHandler(() => {
  // do your reloading here
  browser.runtime.reload()
});

The default registered reload handler could be the location one to preserve existing behavior.

[FredKSchott]

Oh, I also wonder if this line is returning false in your electron environment: const isWindowDefined = typeof window !== 'undefined';

Which would then cause our normal reload function to do nothing instead of reloading:

function reload() {
  if (!isWindowDefined) {
    return;
  }
  location.reload(true);
}

[FredKSchott]

Thank you for the kind words!