fix(qq): apply musicu.fcg upstream fixes + add playlist search

Per https://github.com/ZHANGTIANYAO1/teamspeak-music-bot/issues/61:
- Remove searchid param (its presence now causes empty results)
- Enforce num_per_page >= 10 (lower values return empty)
- Fix search_type: 2 for albums, 3 for playlists (8 was "user")

Now uses the fixed musicu.fcg as primary (supports songs + albums +
playlists), with client_search_cp as fallback.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

src/music/qq.ts

@@ -12,20 +12,21 @@ import type {
 } from "./provider.js";
 import { parseLyrics } from "./netease.js";
 
-// Primary search client: c.y.qq.com/soso/fcgi-bin/client_search_cp (classic
+// Primary search client: u.y.qq.com/cgi-bin/musicu.fcg (JSON sub-request
-// endpoint, currently working without a cookie).
+// batch). Was broken ca. 2026-05 due to two upstream API changes:
-const qqSearchApi = axios.create({
+//   1. searchid param must NOT be present (causes all lists to be empty)
-  baseURL: "https://c.y.qq.com",
+//   2. num_per_page must be >= 10 (lower values return empty)
+// Both fixes applied per https://github.com/ZHANGTIANYAO1/teamspeak-music-bot/issues/61
+const qqMusicuApi = axios.create({
+  baseURL: "https://u.y.qq.com",
   timeout: 10000,
   headers: { referer: "https://y.qq.com" },
 });
 
-// Fallback search client: u.y.qq.com/cgi-bin/musicu.fcg used JSON sub-requests
+// Fallback search client: c.y.qq.com/soso/fcgi-bin/client_search_cp (classic
-// but started returning is_filter=-9 (empty results) for unauthenticated
+// endpoint, song + album only, no playlist support).
-// requests ca. 2026-05. Kept as a redundancy in case the primary endpoint
+const qqSearchApi = axios.create({
-// goes down or changes its response format.
+  baseURL: "https://c.y.qq.com",
-const qqFallbackApi = axios.create({
-  baseURL: "https://u.y.qq.com",
   timeout: 10000,
   headers: { referer: "https://y.qq.com" },
 });
@@ -92,22 +93,89 @@ export class QQMusicProvider implements MusicProvider {
   }
 
   async search(query: string, limit = 20): Promise<SearchResult> {
-    // Primary: c.y.qq.com client_search_cp (currently working).
+    // Primary: u.y.qq.com/cgi-bin/musicu.fcg — supports songs + albums +
-    const primary = await this.searchViaClientSearchCp(query, limit);
+    // playlists. Fixed per https://github.com/ZHANGTIANYAO1/teamspeak-music-bot/issues/61
+    // (removed searchid, num_per_page >= 10, corrected search_type values).
+    const primary = await this.searchViaMusicuFcg(query, limit);
     if (primary) return primary;
 
-    // Fallback: u.y.qq.com musicu.fcg JSON sub-requests (may return empty
+    // Fallback: c.y.qq.com/soso/fcgi-bin/client_search_cp (song + album,
-    // when Tencent applies is_filter, but acts as a safety net in case the
+    // no playlist support). Kept as redundancy.
-    // primary endpoint format changes or goes down).
+    return this.searchViaClientSearchCp(query, limit);
-    return this.searchViaMusicuFcg(query, limit);
   }
 
-  /** Primary search via c.y.qq.com/soso/fcgi-bin/client_search_cp */
+  /** Primary search via u.y.qq.com/cgi-bin/musicu.fcg.
-  private async searchViaClientSearchCp(
+   *
+   * Two upstream API changes (2026-05) required fixes:
+   *   1. Omit `searchid` — its presence now causes all lists to be empty.
+   *   2. `num_per_page` >= 10 — lower values return empty.
+   *   3. `search_type: 2` for albums, `3` for playlists (8 was "user"). */
+  private async searchViaMusicuFcg(
     query: string,
     limit: number
   ): Promise<SearchResult | null> {
     try {
+      const numPerPage = Math.max(10, Math.min(limit, 50));
+      const reqData = JSON.stringify({
+        req_0: {
+          module: "music.search.SearchCgiService",
+          method: "DoSearchForQQMusicDesktop",
+          param: { query, num_per_page: numPerPage, search_type: 0 },
+        },
+        req_album: {
+          module: "music.search.SearchCgiService",
+          method: "DoSearchForQQMusicDesktop",
+          param: { query, num_per_page: 10, search_type: 2 },
+        },
+        req_playlist: {
+          module: "music.search.SearchCgiService",
+          method: "DoSearchForQQMusicDesktop",
+          param: { query, num_per_page: 10, search_type: 3 },
+        },
+      });
+      const res = await qqMusicuApi.get("/cgi-bin/musicu.fcg", {
+        params: { format: "json", data: reqData },
+      });
+
+      const songList: any[] =
+        res.data?.req_0?.data?.body?.song?.list ?? [];
+      if (songList.length === 0) return null;
+
+      const songs: Song[] = songList.map((s: any) => ({
+        id: String(s.mid ?? s.id),
+        name: s.title ?? s.name ?? "",
+        artist: (s.singer ?? []).map((a: any) => a.name).join(" / "),
+        album: s.album?.name ?? s.album?.title ?? "",
+        duration: s.interval ?? 0,
+        coverUrl: s.album?.mid
+          ? `https://y.gtimg.cn/music/photo_new/T002R300x300M000${s.album.mid}.jpg`
+          : "",
+        platform: "qq",
+      }));
+
+      const albumList: any[] = res.data?.req_album?.data?.body?.album?.list ?? [];
+      const albums = mapQqAlbums(albumList);
+
+      const playlistList: any[] = res.data?.req_playlist?.data?.body?.songlist?.list ?? [];
+      const playlists: Playlist[] = playlistList.map((p: any) => ({
+        id: String(p.dissid ?? p.id ?? ""),
+        name: p.dissname ?? p.title ?? "",
+        coverUrl: p.imgurl ?? p.logo ?? "",
+        songCount: p.songnum ?? p.song_count ?? 0,
+        platform: "qq" as const,
+      }));
+
+      return { songs, playlists, albums };
+    } catch {
+      return null;
+    }
+  }
+
+  /** Fallback search via c.y.qq.com/soso/fcgi-bin/client_search_cp */
+  private async searchViaClientSearchCp(
+    query: string,
+    limit: number
+  ): Promise<SearchResult> {
     const songParams = {
       w: query,
       format: "json",
@@ -135,10 +203,6 @@ export class QQMusicProvider implements MusicProvider {
         ? (songRes.value.data?.data?.song?.list ?? [])
         : [];
 
-      // Treat zero songs as "endpoint returned no useful data" so we fall
-      // through to the fallback path.
-      if (songList.length === 0) return null;
-
     const songs: Song[] = songList.map((s: any) => ({
       id: String(s.songmid ?? s.songid ?? ""),
       name: s.songname ?? s.name ?? "",
@@ -157,50 +221,6 @@ export class QQMusicProvider implements MusicProvider {
         : [];
     const albums = mapQqAlbums(albumList);
 
-      return { songs, playlists: [], albums };
-    } catch {
-      return null; // network error or unexpected response → fall back
-    }
-  }
-
-  /** Fallback search via u.y.qq.com/cgi-bin/musicu.fcg */
-  private async searchViaMusicuFcg(
-    query: string,
-    limit: number
-  ): Promise<SearchResult> {
-    const reqData = JSON.stringify({
-      req_0: {
-        module: "music.search.SearchCgiService",
-        method: "DoSearchForQQMusicDesktop",
-        param: { searchid: "1", query, num_per_page: Math.min(limit, 50), search_type: 0 },
-      },
-      req_album: {
-        module: "music.search.SearchCgiService",
-        method: "DoSearchForQQMusicDesktop",
-        param: { searchid: "1", query, num_per_page: 5, search_type: 8 },
-      },
-    });
-    const res = await qqFallbackApi.get("/cgi-bin/musicu.fcg", {
-      params: { format: "json", data: reqData },
-    });
-    const list: any[] =
-      res.data?.req_0?.data?.body?.song?.list ?? [];
-
-    const songs: Song[] = list.map((s: any) => ({
-      id: String(s.mid ?? s.id),
-      name: s.title ?? s.name ?? "",
-      artist: (s.singer ?? []).map((a: any) => a.name).join(" / "),
-      album: s.album?.name ?? s.album?.title ?? "",
-      duration: s.interval ?? 0,
-      coverUrl: s.album?.mid
-        ? `https://y.gtimg.cn/music/photo_new/T002R300x300M000${s.album.mid}.jpg`
-        : "",
-      platform: "qq",
-    }));
-
-    const albumList: any[] = res.data?.req_album?.data?.body?.album?.list ?? [];
-    const albums = mapQqAlbums(albumList);
-
     return { songs, playlists: [], albums };
   }