@quark-fw/plugin-threads 0.1.15 → 0.1.16

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@quark-fw/plugin-threads",
-	"version": "0.1.15",
+	"version": "0.1.16",
 	"main": "src/index.ts",
 	"types": "src/index.ts",
 	"type": "module",
@@ -21,16 +21,16 @@
 		"access": "public"
 	},
 	"dependencies": {
-		"@quark-fw/atom": "0.1.15",
-		"@quark-fw/plugin": "0.1.15",
-		"@quark-fw/plugin-env": "0.1.15",
-		"@quark-fw/plugin-redis": "0.1.15",
+		"@quark-fw/atom": "0.1.16",
+		"@quark-fw/plugin": "0.1.16",
+		"@quark-fw/plugin-env": "0.1.16",
+		"@quark-fw/plugin-redis": "0.1.16",
 		"chokidar": "^5.0.0"
 	},
 	"devDependencies": {
 		"ioredis": "^5.9.3"
 	},
 	"peerDependencies": {
-		"@quark-fw/core": "0.1.15"
+		"@quark-fw/core": "0.1.16"
 	}
 }

package/src/index.ts

@@ -38,6 +38,19 @@ await quark.registerPlugin({
 		const handles = new Map<string, WorkerHandle[]>();
 		// "file#instance" -> число рестартов подряд
 		const restarts = new Map<string, number>();
+		// "file#instance" -> отложенный рестарт упавшего инстанса. Упавший
+		// инстанс уже убран из handles, поэтому stop() его не видит — без
+		// отмены таймера остановленный (или удалённый) воркер воскресал, а
+		// stop+spawn при правке файла давали дубль инстанса.
+		const pendingRestarts = new Map<string, NodeJS.Timeout>();
+		const cancelRestarts = (file: string) => {
+			for (const [key, timer] of pendingRestarts) {
+				if (!key.startsWith(`${file}#`)) continue;
+				clearTimeout(timer);
+				pendingRestarts.delete(key);
+				restarts.delete(key);
+			}
+		};
 
 		const updateList = () => {
 			list.value = [...handles.values()].flat().map((h) => ({
@@ -104,11 +117,13 @@ await quark.registerPlugin({
 				restarts.set(key, n + 1);
 				const delay = Math.min(1000 * 2 ** n, RESTART_MAX_DELAY_MS);
 
-				setTimeout(() => {
+				const timer = setTimeout(() => {
+					pendingRestarts.delete(key);
 					if (destroyed) return;
 					quark.ee.value.emit("thread:restart", file, instance);
 					forkInstance(file, instance);
 				}, delay).unref();
+				pendingRestarts.set(key, timer);
 			});
 		};
 
@@ -117,11 +132,19 @@ await quark.registerPlugin({
 				handle.stopping = true;
 				if (handle.child.exitCode !== null) return resolve();
 
+				// SIGTERM после таймаута, SIGKILL — если и его проигнорировали:
+				// иначе stop() и выключение приложения висели бы вечно
+				let killTimer: NodeJS.Timeout | undefined;
 				const timeout = setTimeout(() => {
 					handle.child.kill();
+					killTimer = setTimeout(
+						() => handle.child.kill("SIGKILL"),
+						SHUTDOWN_TIMEOUT_MS,
+					);
 				}, SHUTDOWN_TIMEOUT_MS);
 				handle.child.once("exit", () => {
 					clearTimeout(timeout);
+					clearTimeout(killTimer);
 					resolve();
 				});
 				try {
@@ -135,11 +158,13 @@ await quark.registerPlugin({
 			list,
 			async spawn(file) {
 				const abs = path.resolve(file);
+				cancelRestarts(abs);
 				if ((handles.get(abs) ?? []).length) return;
 				forkInstance(abs, 0);
 			},
 			async stop(file) {
 				const abs = path.resolve(file);
+				cancelRestarts(abs);
 				const all = handles.get(abs) ?? [];
 				await Promise.all(all.map(stopHandle));
 				handles.delete(abs);