From 1021a61eadb55e1eb292444a35a2ab5493b3d67d Mon Sep 17 00:00:00 2001 From: tkrmagid Date: Fri, 18 Sep 2026 20:51:44 +0900 Subject: [PATCH] =?UTF-8?q?perf(search):=20=EC=9C=A0=ED=8A=9C=EB=B8=8C?= =?UTF-8?q?=EB=AE=A4=EC=A7=81=20=EA=B2=80=EC=83=89=EA=B3=BC=20ytsearch=20?= =?UTF-8?q?=ED=8F=B4=EB=B0=B1=EC=9D=84=20=EB=B3=91=EB=A0=AC=20=EC=8B=A4?= =?UTF-8?q?=ED=96=89?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 측정으로 확인한 낭비: YoutubeMusic.getSearchUrl 적중률이 6/10(평균 548ms)이라, 빗나가는 40% 에서는 YTM 을 끝까지 기다린 뒤 다시 ytsearch(~0.5초)를 도는 순차 대기가 통째로 버려지고 있었다. 변경: resolveQuery 기본 경로에서 ytsearch 폴백 검색을 먼저 띄워두고 YTM 결과를 기다린다. - YTM 적중 -> 폴백 결과는 버리고 기존과 동일하게 동작(Lavalink 검색 1회가 헛돌 뿐) - YTM 빗나감 -> 이미 진행 중인 폴백 결과를 그대로 사용(추가 대기 0) - 버려지는 폴백 promise 에 catch 를 달아 unhandledRejection 방지 - " Topic" 중복 부가 방지 조건 추가 URL 입력/`-p`(스포티파이 우선) 경로는 기존과 동일하게 동작한다. 실측 A/B(YTM 이 빗나가는 3곡, 검색+해석+리졸브 구간): 기존 1500 / 1688 / 1391ms -> 신규 1224 / 1180 / 1399ms 재생은 3곡 모두 성공, 관찰 중 멈춤 0. tsc 통과. --- bot/src/classes/LavalinkManager.ts | 19 ++++++++++++++++--- 1 file changed, 16 insertions(+), 3 deletions(-) diff --git a/bot/src/classes/LavalinkManager.ts b/bot/src/classes/LavalinkManager.ts index c5fde02..c840da5 100644 --- a/bot/src/classes/LavalinkManager.ts +++ b/bot/src/classes/LavalinkManager.ts @@ -116,10 +116,22 @@ export class LavalinkManager { // 기본: 유튜브뮤직(YoutubeMusic.getSearchUrl) 우선 → 결과가 없거나 애매(커버/방송/무관)하면 // getSearchUrl이 null을 주고, 공식 오디오를 잘 잡는 `ytsearch:... Topic`로 폴백한다. // ('-p' 플래그를 붙이면 스포티파이 우선 검색) + // 기본 경로는 YTM 검색과 ytsearch 폴백을 '동시에' 출발시킨다. + // YTM 적중률이 약 60%라, 빗나가는 40%에서는 YTM(~0.5초)을 끝까지 기다린 뒤 다시 + // ytsearch(~0.5초)를 도는 순차 대기가 통째로 낭비된다. 미리 같이 돌려두면 빗나가도 + // 추가 대기가 없다. (적중 시엔 폴백 결과를 버리므로 Lavalink 검색 1회가 헛돌지만 비용이 작다.) let searchText: string; + let pendingFallback: ReturnType | null = null; if (isUrl) searchText = text; else if (flags.has('p')) searchText = (await Spotify.getSearchUrl(text)) ?? (await YoutubeMusic.getSearchUrl(text)) ?? `ytsearch:${text}`; - else searchText = (await YoutubeMusic.getSearchUrl(text)) ?? `ytsearch:${text}`; + else { + const fallbackText = `ytsearch:${text}` + (flags.has('o') ? "" : " Topic"); + pendingFallback = node.rest.resolve(fallbackText); + pendingFallback.catch(() => {}); // 적중해서 버려질 때 unhandledRejection 방지 + const ytm = await YoutubeMusic.getSearchUrl(text).catch(() => null); + searchText = ytm ?? fallbackText; + if (ytm) pendingFallback = null; // YTM 적중 → 폴백 결과는 사용하지 않음 + } // 스포티파이 트랙 URL은 Lavalink가 유튜브 '영상(뮤비)'으로 폴백한다(유튜브뮤직 연결돼 있어도). // 대시보드에서 고른 스포티파이 곡이나 `-p` 결과가 트랙 URL이면, 메타데이터(제목+아티스트)로 // 유튜브뮤직 '오디오'를 직접 찾아 대체한다(텍스트 검색 경로와 동일하게 원곡 오디오 우선). @@ -131,8 +143,9 @@ export class LavalinkManager { searchText = (await YoutubeMusic.getSearchUrl(q)) ?? `ytsearch:${q}`; } } - if (searchText.startsWith("ytsearch") && !flags.has('o')) searchText += " Topic"; - const result = await node.rest.resolve(searchText); + if (searchText.startsWith("ytsearch") && !flags.has('o') && !searchText.endsWith(" Topic")) searchText += " Topic"; + // 위에서 미리 띄워둔 폴백 검색이 있으면 그 결과를 그대로 쓴다(중복 요청 방지). + const result = pendingFallback ? await pendingFallback : await node.rest.resolve(searchText); if (!result || result.loadType === LoadType.EMPTY || result.loadType === LoadType.ERROR) { if (result?.loadType === LoadType.ERROR) Logger.error(`[LavalinkManager] loadtype ERROR: ${result.data.message}`); return null;