fix/ws-reconnect-on-dev-shut-down

[PatrykKuniczak]

* denotes required fields

Priority*

• High: This PR needs to be merged first, before other tasks.
• Medium: This PR should be merged quickly to prevent conflicts due to common changes. (default)
• Low: This PR does not affect other tasks, so it can be merged later.

Purpose of the PR*

I want to avoid displaying error message of ws from #477 issue

Changes*

I've created func which edit IS_DEV var on dist/background.js file on node process termination, for avoid ws reconnecting after WSS server were stopped because of script interruption

How to check the feature

1. Run pnpm dev or pnpm dev:firefox
2. Wait for the end of building process
3. Interrupt with e.g CTRL + C in terminal

[Jonghakseo]

1. pnpm dev doesn't run because SIGKILL signal is not available on POSIX-compliant (Unix, Linux, macOS)
2. Websocket server doesn't shut down with pnpm dev process
3. This solution doesn't seem to work for other content scripts, new-tab, etc. except background.

[PatrykKuniczak]

@Jonghakseo Now it should work everywhere.

I've tested it and it's shutting down with pnpm dev all processes on 8081 are down after i've closed dev

[Jonghakseo]

[Jonghakseo]

This appears to be a timing issue. I was wondering if dist/background might not be present when the hmr module is running.

[PatrykKuniczak]

@Jonghakseo I can't reproduce it.
Have you tried pnpm clean:install before?

If yes, then give me steps to reproduce.