fix(backend): サーバーを始動する前に他のサービスが始動するように

CHANGELOG.md

@@ -14,7 +14,8 @@
 - Fix: ユーザーのプロフィール画面をアドレス入力などで直接表示した際に概要タブの描画に失敗する問題の修正( #15032 )
 - Fix: 起動前の疎通チェックが機能しなくなっていた問題を修正  
   (Cherry-picked from https://activitypub.software/TransFem-org/Sharkey/-/merge_requests/737)
-
+- Fix: チャートエンジン・キュープロセッサが起動する前にサーバーがリクエストを受け付ける可能性がある問題を修正  
+  (Cherry-picked from https://activitypub.software/TransFem-org/Sharkey/-/merge_requests/788)
 
 ## 2024.11.0
 

packages/backend/src/boot/common.ts

@@ -18,15 +18,16 @@ export async function server() {
 		logger: new NestLogger(),
 	});
 
-	const serverService = app.get(ServerService);
-	await serverService.launch();
-
 	if (process.env.NODE_ENV !== 'test') {
-		app.get(ChartManagementService).start();
-		app.get(QueueStatsService).start();
-		app.get(ServerStatsService).start();
+		await app.get(ChartManagementService).start();
+		await app.get(QueueStatsService).start();
+		await app.get(ServerStatsService).start();
 	}
 
+	// Start server last so the other services can register hooks first
+	const serverService = app.get(ServerService);
+	await serverService.launch();
+
 	return app;
 }
 
@@ -35,8 +36,8 @@ export async function jobQueue() {
 		logger: new NestLogger(),
 	});
 
-	jobQueue.get(QueueProcessorService).start();
-	jobQueue.get(ChartManagementService).start();
+	await jobQueue.get(QueueProcessorService).start();
+	await jobQueue.get(ChartManagementService).start();
 
 	return jobQueue;
 }

packages/backend/src/daemons/QueueStatsService.ts

@@ -33,7 +33,7 @@ export class QueueStatsService implements OnApplicationShutdown {
 	 * Report queue stats regularly
 	 */
 	@bindThis
-	public start(): void {
+	public async start(): Promise<void> {
 		const log = [] as any[];
 
 		ev.on('requestQueueStatsLog', x => {
@@ -82,7 +82,7 @@ export class QueueStatsService implements OnApplicationShutdown {
 			activeInboxJobs = 0;
 		};
 
-		tick();
+		await tick();
 
 		this.intervalId = setInterval(tick, interval);
 	}

packages/backend/src/daemons/ServerStatsService.ts

@@ -68,7 +68,7 @@ export class ServerStatsService implements OnApplicationShutdown {
 			if (log.length > 200) log.pop();
 		};
 
-		tick();
+		await tick();
 
 		this.intervalId = setInterval(tick, interval);
 	}