From 95939bbea569f9ad16fe547b9bde2687ab8a8db6 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Fri, 7 Nov 2025 16:34:42 +0000 Subject: [PATCH] Improve logging accuracy for worker restarts - Fixed log messages to accurately reflect worker state - Added proper logging after gRPC worker is confirmed ready - Improved HTTP worker restart logging with PID - Addresses code review feedback for better clarity Co-authored-by: whyour <22700758+whyour@users.noreply.github.com> --- back/app.ts | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/back/app.ts b/back/app.ts index 7f0d0141..15153a00 100644 --- a/back/app.ts +++ b/back/app.ts @@ -76,20 +76,21 @@ class Application { }). Restarting...`, ); // If gRPC worker died, restart it and wait for it to be ready - // before potentially needing to restart HTTP worker if (metadata.serviceType === 'grpc') { const newGrpcWorker = this.forkWorker('grpc'); - this.waitForWorkerReady(newGrpcWorker, 30000).catch((error) => { - Logger.error('Failed to restart gRPC worker:', error); - process.exit(1); - }); + this.waitForWorkerReady(newGrpcWorker, 30000) + .then(() => { + Logger.info('gRPC worker restarted and ready'); + }) + .catch((error) => { + Logger.error('Failed to restart gRPC worker:', error); + process.exit(1); + }); } else { // For HTTP worker, just restart it - this.forkWorker(metadata.serviceType); + const newWorker = this.forkWorker(metadata.serviceType); + Logger.info(`Restarted ${metadata.serviceType} worker (PID: ${newWorker.process.pid})`); } - Logger.info( - `Restarted ${metadata.serviceType} worker`, - ); } this.workerMetadataMap.delete(worker.id);