From bc9b2366d0bd7d989f0e3847d2e9bd8d838da9fe Mon Sep 17 00:00:00 2001
From: saopig1 <saopig@duck.com>
Date: Mon, 30 Mar 2026 20:45:24 +0800
Subject: [PATCH] feat: make frontend fully support multiple independent bots

Store queue and timing state per-bot instead of globally, move bot
selector from Home page into Navbar dropdown, show bot name badge
in the player bar, and route WebSocket stateChange queues per-bot.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 web/src/components/Navbar.vue       | 176 +++++++++++++++++++++++++++-
 web/src/components/Player.vue       |  22 +++-
 web/src/components/Queue.vue        |   9 +-
 web/src/composables/useWebSocket.ts |   6 +-
 web/src/stores/player.ts            | 163 ++++++++++++++++++--------
 web/src/views/Home.vue              |  35 ------
 6 files changed, 316 insertions(+), 95 deletions(-)

diff --git a/web/src/components/Navbar.vue b/web/src/components/Navbar.vue
index 7bd2c03..006a51e 100644
--- a/web/src/components/Navbar.vue
+++ b/web/src/components/Navbar.vue
@@ -9,9 +9,36 @@
     </div>
 
     <div class="nav-right">
-      <div v-if="activeBot" class="bot-status" :class="{ online: activeBot.connected }">
+      <!-- Multi-bot selector -->
+      <div v-if="store.bots.length > 1" class="bot-selector" ref="selectorRef">
+        <button class="bot-selector-btn" @click="dropdownOpen = !dropdownOpen">
+          <span class="bot-dot" :class="{ online: activeBot?.connected }" />
+          <span class="bot-selector-name">{{ activeBot?.name ?? '选择机器人' }}</span>
+          <Icon icon="mdi:chevron-down" class="bot-chevron" :class="{ rotated: dropdownOpen }" />
+        </button>
+        <div v-if="dropdownOpen" class="bot-dropdown">
+          <button
+            v-for="bot in store.bots"
+            :key="bot.id"
+            class="bot-dropdown-item"
+            :class="{ active: bot.id === store.activeBotId }"
+            @click="selectBot(bot.id)"
+          >
+            <span class="bot-dot" :class="{ online: bot.connected }" />
+            <span class="bot-dropdown-name">{{ bot.name }}</span>
+            <span v-if="bot.playing && !bot.paused" class="bot-playing-badge">播放中</span>
+            <span v-else-if="bot.paused" class="bot-paused-badge">已暂停</span>
+            <span v-else-if="bot.connected" class="bot-idle-badge">空闲</span>
+            <span v-else class="bot-offline-badge">离线</span>
+          </button>
+        </div>
+      </div>
+
+      <!-- Single bot status (original behavior) -->
+      <div v-else-if="activeBot" class="bot-status" :class="{ online: activeBot.connected }">
         {{ activeBot.name }} {{ activeBot.connected ? '在线' : '离线' }}
       </div>
+
       <RouterLink to="/settings" class="settings-btn">
         <Icon icon="mdi:cog" />
       </RouterLink>
@@ -20,12 +47,33 @@
 </template>
 
 <script setup lang="ts">
-import { computed } from 'vue';
+import { computed, ref, onMounted, onUnmounted } from 'vue';
 import { Icon } from '@iconify/vue';
 import { usePlayerStore } from '../stores/player.js';
 
 const store = usePlayerStore();
 const activeBot = computed(() => store.activeBot);
+const dropdownOpen = ref(false);
+const selectorRef = ref<HTMLElement | null>(null);
+
+function selectBot(id: string) {
+  store.setActiveBotId(id);
+  dropdownOpen.value = false;
+}
+
+function onClickOutside(e: MouseEvent) {
+  if (selectorRef.value && !selectorRef.value.contains(e.target as Node)) {
+    dropdownOpen.value = false;
+  }
+}
+
+onMounted(() => {
+  document.addEventListener('click', onClickOutside);
+});
+
+onUnmounted(() => {
+  document.removeEventListener('click', onClickOutside);
+});
 </script>
 
 <style lang="scss" scoped>
@@ -89,6 +137,130 @@ const activeBot = computed(() => store.activeBot);
   }
 }
 
+/* Bot selector dropdown */
+.bot-selector {
+  position: relative;
+}
+
+.bot-selector-btn {
+  display: flex;
+  align-items: center;
+  gap: 8px;
+  padding: 6px 14px;
+  background: var(--hover-bg);
+  border-radius: var(--radius-sm);
+  font-size: 13px;
+  font-weight: 500;
+  transition: background var(--transition-fast);
+  cursor: pointer;
+
+  &:hover {
+    background: var(--bg-card);
+  }
+}
+
+.bot-chevron {
+  font-size: 16px;
+  opacity: 0.5;
+  transition: transform 0.2s ease;
+
+  &.rotated {
+    transform: rotate(180deg);
+  }
+}
+
+.bot-dot {
+  width: 8px;
+  height: 8px;
+  border-radius: 50%;
+  background: var(--text-tertiary);
+  flex-shrink: 0;
+
+  &.online {
+    background: #22c55e;
+  }
+}
+
+.bot-selector-name {
+  max-width: 120px;
+  overflow: hidden;
+  text-overflow: ellipsis;
+  white-space: nowrap;
+}
+
+.bot-dropdown {
+  position: absolute;
+  top: calc(100% + 6px);
+  right: 0;
+  min-width: 200px;
+  background: var(--bg-secondary);
+  border: 1px solid var(--border-color);
+  border-radius: var(--radius-md);
+  padding: 4px;
+  box-shadow: 0 8px 24px rgba(0, 0, 0, 0.2);
+  z-index: 200;
+}
+
+.bot-dropdown-item {
+  display: flex;
+  align-items: center;
+  gap: 8px;
+  width: 100%;
+  padding: 8px 12px;
+  border-radius: var(--radius-sm);
+  font-size: 13px;
+  cursor: pointer;
+  transition: background var(--transition-fast);
+
+  &:hover {
+    background: var(--hover-bg);
+  }
+
+  &.active {
+    background: rgba(51, 94, 234, 0.12);
+    color: var(--color-primary);
+  }
+}
+
+.bot-dropdown-name {
+  flex: 1;
+  min-width: 0;
+  overflow: hidden;
+  text-overflow: ellipsis;
+  white-space: nowrap;
+}
+
+.bot-playing-badge,
+.bot-paused-badge,
+.bot-idle-badge,
+.bot-offline-badge {
+  font-size: 11px;
+  padding: 1px 6px;
+  border-radius: 4px;
+  font-weight: 500;
+  flex-shrink: 0;
+}
+
+.bot-playing-badge {
+  background: rgba(34, 197, 94, 0.15);
+  color: #22c55e;
+}
+
+.bot-paused-badge {
+  background: rgba(234, 179, 8, 0.15);
+  color: #eab308;
+}
+
+.bot-idle-badge {
+  background: rgba(51, 94, 234, 0.12);
+  color: var(--color-primary);
+}
+
+.bot-offline-badge {
+  background: var(--hover-bg);
+  color: var(--text-tertiary);
+}
+
 .settings-btn {
   font-size: 20px;
   opacity: 0.6;
diff --git a/web/src/components/Player.vue b/web/src/components/Player.vue
index f021704..e77b56c 100644
--- a/web/src/components/Player.vue
+++ b/web/src/components/Player.vue
@@ -28,7 +28,10 @@
         <CoverArt :url="currentSong.coverUrl" :size="40" />
         <div class="song-info">
           <div class="song-name">{{ currentSong.name }}</div>
-          <div class="song-artist">{{ currentSong.artist }}</div>
+          <div class="song-artist">
+            <span v-if="showBotBadge" class="bot-badge">{{ activeBot?.name }}</span>
+            {{ currentSong.artist }}
+          </div>
         </div>
       </div>
 
@@ -86,6 +89,7 @@ const showQueue = ref(false);
 const store = usePlayerStore();
 const activeBot = computed(() => store.activeBot);
 const currentSong = computed(() => store.currentSong);
+const showBotBadge = computed(() => store.bots.length > 1);
 
 function toggleLyrics() {
   if (route.path === '/lyrics') {
@@ -315,6 +319,22 @@ function cycleMode() {
 .song-artist {
   font-size: 11px;
   color: var(--text-secondary);
+  display: flex;
+  align-items: center;
+  gap: 4px;
+}
+
+.bot-badge {
+  display: inline-block;
+  font-size: 10px;
+  font-weight: 600;
+  padding: 0 5px;
+  background: rgba(51, 94, 234, 0.15);
+  color: var(--color-primary);
+  border-radius: 3px;
+  line-height: 16px;
+  white-space: nowrap;
+  flex-shrink: 0;
 }
 
 .player-center {
diff --git a/web/src/components/Queue.vue b/web/src/components/Queue.vue
index c7d2fec..83356b1 100644
--- a/web/src/components/Queue.vue
+++ b/web/src/components/Queue.vue
@@ -2,19 +2,19 @@
   <div class="queue-panel" :class="{ open }">
     <div class="queue-header">
       <h3 class="queue-title">播放队列</h3>
-      <span class="queue-count">{{ store.queue.length }} 首</span>
+      <span class="queue-count">{{ botQueue.length }} 首</span>
       <button class="close-btn" @click="$emit('close')">
         <Icon icon="mdi:close" />
       </button>
     </div>
 
-    <div v-if="store.queue.length === 0" class="queue-empty">
+    <div v-if="botQueue.length === 0" class="queue-empty">
       队列为空
     </div>
 
     <div v-else class="queue-list">
       <div
-        v-for="(song, i) in store.queue"
+        v-for="(song, i) in botQueue"
         :key="`${song.id}-${i}`"
         class="queue-item"
         :class="{ active: store.currentSong?.id === song.id }"
@@ -34,7 +34,7 @@
 </template>
 
 <script setup lang="ts">
-import { watch } from 'vue';
+import { watch, computed } from 'vue';
 import { Icon } from '@iconify/vue';
 import axios from 'axios';
 import { usePlayerStore } from '../stores/player.js';
@@ -49,6 +49,7 @@ defineEmits<{
 }>();
 
 const store = usePlayerStore();
+const botQueue = computed(() => store.queue);
 
 // Fetch queue when panel opens
 watch(() => props.open, (isOpen) => {
diff --git a/web/src/composables/useWebSocket.ts b/web/src/composables/useWebSocket.ts
index 0c2f2d7..b800363 100644
--- a/web/src/composables/useWebSocket.ts
+++ b/web/src/composables/useWebSocket.ts
@@ -30,10 +30,10 @@ export function useWebSocket() {
         case 'stateChange':
           store.updateBotStatus(data.botId, data.status);
           if (data.queue) {
-            store.setQueue(data.queue);
+            store.setQueue(data.botId, data.queue);
           } else {
-            // Queue not included in event; refresh it
-            store.fetchQueue();
+            // Queue not included in event; refresh for this specific bot
+            store.fetchQueueForBot(data.botId);
           }
           break;
         case 'botConnected':
diff --git a/web/src/stores/player.ts b/web/src/stores/player.ts
index d9479f4..b350da1 100644
--- a/web/src/stores/player.ts
+++ b/web/src/stores/player.ts
@@ -32,20 +32,28 @@ export interface PlaylistItem {
   platform: string;
 }
 
+interface TimingState {
+  serverElapsed: number;
+  serverSyncTime: number;
+  wasPlaying: boolean;
+}
+
 const HOME_CACHE_TTL = 5 * 60 * 1000;
 
+function defaultTiming(): TimingState {
+  return { serverElapsed: 0, serverSyncTime: 0, wasPlaying: false };
+}
+
 export const usePlayerStore = defineStore('player', {
   state: () => ({
     bots: [] as BotStatus[],
     activeBotId: null as string | null,
-    queue: [] as Song[],
+    /** Per-bot queues keyed by botId */
+    queues: {} as Record<string, Song[]>,
+    /** Per-bot timing state keyed by botId */
+    timings: {} as Record<string, TimingState>,
     theme: 'dark' as 'dark' | 'light',
 
-    // Server-synced elapsed time (ground truth)
-    serverElapsed: 0,       // last known server elapsed (seconds)
-    serverSyncTime: 0,      // Date.now() when we got serverElapsed
-    wasPlaying: false,       // was playing at last sync
-
     // Home page cache
     recommendPlaylists: [] as PlaylistItem[],
     dailySongs: [] as Song[],
@@ -67,19 +75,47 @@ export const usePlayerStore = defineStore('player', {
     isPaused(): boolean {
       return this.activeBot?.paused ?? false;
     },
-    /** Interpolated elapsed: serverElapsed + time since last sync (if playing) */
+    /** Queue for the currently active bot */
+    queue(): Song[] {
+      const botId = this.activeBotId ?? this.bots[0]?.id;
+      if (!botId) return [];
+      return this.queues[botId] ?? [];
+    },
+    /** Interpolated elapsed for the active bot */
     elapsed(): number {
-      if (!this.activeBot?.currentSong) return 0;
+      const botId = this.activeBotId ?? this.bots[0]?.id;
+      if (!botId || !this.activeBot?.currentSong) return 0;
+      const timing = this.timings[botId] ?? defaultTiming();
       const maxDuration = this.activeBot.currentSong.duration || Infinity;
-      if (!this.wasPlaying || this.serverSyncTime === 0) return Math.min(this.serverElapsed, maxDuration);
-      if (this.isPaused) return Math.min(this.serverElapsed, maxDuration);
-      return Math.min(this.serverElapsed + (Date.now() - this.serverSyncTime) / 1000, maxDuration);
+      if (!timing.wasPlaying || timing.serverSyncTime === 0) return Math.min(timing.serverElapsed, maxDuration);
+      if (this.isPaused) return Math.min(timing.serverElapsed, maxDuration);
+      return Math.min(timing.serverElapsed + (Date.now() - timing.serverSyncTime) / 1000, maxDuration);
     },
   },
 
   actions: {
+    _getTiming(botId: string): TimingState {
+      if (!this.timings[botId]) {
+        this.timings[botId] = defaultTiming();
+      }
+      return this.timings[botId];
+    },
+
+    _setTiming(botId: string, partial: Partial<TimingState>) {
+      const current = this._getTiming(botId);
+      this.timings[botId] = { ...current, ...partial };
+    },
+
+    getQueueForBot(botId: string): Song[] {
+      return this.queues[botId] ?? [];
+    },
+
     setActiveBotId(id: string) {
       this.activeBotId = id;
+      // Fetch queue for newly active bot if we don't have it yet
+      if (!this.queues[id]) {
+        this.fetchQueue();
+      }
     },
 
     updateBotStatus(botId: string, status: BotStatus) {
@@ -93,29 +129,33 @@ export const usePlayerStore = defineStore('player', {
         this.bots.push(status);
       }
 
-      if (botId !== (this.activeBotId ?? this.bots[0]?.id)) return;
-
-      // Sync elapsed from server status
+      // Sync elapsed from server status — always per-bot
       if (status.elapsed !== undefined) {
-        this.serverElapsed = status.elapsed;
-        this.serverSyncTime = Date.now();
-        this.wasPlaying = status.playing && !status.paused;
+        this._setTiming(botId, {
+          serverElapsed: status.elapsed,
+          serverSyncTime: Date.now(),
+          wasPlaying: status.playing && !status.paused,
+        });
       }
 
-      // Song changed — reset
+      // Song changed — reset timing for this bot
       if (status.currentSong?.id !== prevSongId) {
-        this.serverElapsed = status.elapsed ?? 0;
-        this.serverSyncTime = Date.now();
-        this.wasPlaying = status.playing && !status.paused;
+        this._setTiming(botId, {
+          serverElapsed: status.elapsed ?? 0,
+          serverSyncTime: Date.now(),
+          wasPlaying: status.playing && !status.paused,
+        });
       }
     },
 
     removeBotStatus(botId: string) {
       this.bots = this.bots.filter((b) => b.id !== botId);
+      delete this.queues[botId];
+      delete this.timings[botId];
     },
 
-    setQueue(queue: Song[]) {
-      this.queue = queue;
+    setQueue(botId: string, queue: Song[]) {
+      this.queues[botId] = queue;
     },
 
     toggleTheme() {
@@ -134,23 +174,28 @@ export const usePlayerStore = defineStore('player', {
       if (!this.activeBotId && this.bots.length > 0) {
         this.activeBotId = this.bots[0].id;
       }
-      // Sync elapsed from first status
-      const bot = this.activeBot;
-      if (bot?.elapsed !== undefined) {
-        this.serverElapsed = bot.elapsed;
-        this.serverSyncTime = Date.now();
-        this.wasPlaying = bot.playing && !bot.paused;
+      // Sync elapsed from each bot's status
+      for (const bot of this.bots) {
+        if (bot.elapsed !== undefined) {
+          this._setTiming(bot.id, {
+            serverElapsed: bot.elapsed,
+            serverSyncTime: Date.now(),
+            wasPlaying: bot.playing && !bot.paused,
+          });
+        }
       }
     },
 
-    /** Poll server for real elapsed time */
+    /** Poll server for real elapsed time for active bot */
     async syncElapsed() {
       if (!this.activeBotId || !this.isPlaying) return;
       try {
         const res = await axios.get(`/api/player/${this.activeBotId}/elapsed`);
-        this.serverElapsed = res.data.elapsed;
-        this.serverSyncTime = Date.now();
-        this.wasPlaying = true;
+        this._setTiming(this.activeBotId, {
+          serverElapsed: res.data.elapsed,
+          serverSyncTime: Date.now(),
+          wasPlaying: true,
+        });
       } catch {
         // ignore
       }
@@ -160,15 +205,27 @@ export const usePlayerStore = defineStore('player', {
       if (!this.activeBotId) return;
       try {
         const res = await axios.get(`/api/player/${this.activeBotId}/queue`);
-        this.queue = res.data.queue ?? [];
+        this.queues[this.activeBotId] = res.data.queue ?? [];
+      } catch {
+        // ignore
+      }
+    },
+
+    async fetchQueueForBot(botId: string) {
+      try {
+        const res = await axios.get(`/api/player/${botId}/queue`);
+        this.queues[botId] = res.data.queue ?? [];
       } catch {
         // ignore
       }
     },
 
     _syncAfterAction() {
-      this.serverSyncTime = Date.now();
-      this.wasPlaying = true;
+      if (!this.activeBotId) return;
+      this._setTiming(this.activeBotId, {
+        serverSyncTime: Date.now(),
+        wasPlaying: true,
+      });
       // Sync from server after a short delay for accuracy
       setTimeout(() => this.syncElapsed(), 500);
     },
@@ -176,21 +233,21 @@ export const usePlayerStore = defineStore('player', {
     async playAtIndex(index: number) {
       if (!this.activeBotId) return;
       await axios.post(`/api/player/${this.activeBotId}/play-at`, { index });
-      this.serverElapsed = 0;
+      this._setTiming(this.activeBotId, { serverElapsed: 0 });
       this._syncAfterAction();
     },
 
     async play(query: string, platform = 'netease') {
       if (!this.activeBotId) return;
       await axios.post(`/api/player/${this.activeBotId}/play`, { query, platform });
-      this.serverElapsed = 0;
+      this._setTiming(this.activeBotId, { serverElapsed: 0 });
       this._syncAfterAction();
     },
 
     async playById(songId: string, platform = 'netease') {
       if (!this.activeBotId) return;
       await axios.post(`/api/player/${this.activeBotId}/play-by-id`, { songId, platform });
-      this.serverElapsed = 0;
+      this._setTiming(this.activeBotId, { serverElapsed: 0 });
       this._syncAfterAction();
     },
 
@@ -207,52 +264,58 @@ export const usePlayerStore = defineStore('player', {
     async playPlaylist(playlistId: string, platform = 'netease') {
       if (!this.activeBotId) return;
       await axios.post(`/api/player/${this.activeBotId}/play-playlist`, { playlistId, platform });
-      this.serverElapsed = 0;
+      this._setTiming(this.activeBotId, { serverElapsed: 0 });
       this._syncAfterAction();
     },
 
     async pause() {
       if (!this.activeBotId) return;
       // Freeze elapsed at current interpolated value
-      this.serverElapsed = this.elapsed;
-      this.wasPlaying = false;
+      this._setTiming(this.activeBotId, {
+        serverElapsed: this.elapsed,
+        wasPlaying: false,
+      });
       await axios.post(`/api/player/${this.activeBotId}/pause`);
     },
 
     async resume() {
       if (!this.activeBotId) return;
       await axios.post(`/api/player/${this.activeBotId}/resume`);
-      this.serverSyncTime = Date.now();
-      this.wasPlaying = true;
+      this._setTiming(this.activeBotId, {
+        serverSyncTime: Date.now(),
+        wasPlaying: true,
+      });
       setTimeout(() => this.syncElapsed(), 300);
     },
 
     async next() {
       if (!this.activeBotId) return;
       await axios.post(`/api/player/${this.activeBotId}/next`);
-      this.serverElapsed = 0;
+      this._setTiming(this.activeBotId, { serverElapsed: 0 });
       this._syncAfterAction();
     },
 
     async prev() {
       if (!this.activeBotId) return;
       await axios.post(`/api/player/${this.activeBotId}/prev`);
-      this.serverElapsed = 0;
+      this._setTiming(this.activeBotId, { serverElapsed: 0 });
       this._syncAfterAction();
     },
 
     async stop() {
       if (!this.activeBotId) return;
       await axios.post(`/api/player/${this.activeBotId}/stop`);
-      this.serverElapsed = 0;
-      this.serverSyncTime = 0;
-      this.wasPlaying = false;
+      this._setTiming(this.activeBotId, {
+        serverElapsed: 0,
+        serverSyncTime: 0,
+        wasPlaying: false,
+      });
     },
 
     async seek(position: number) {
       if (!this.activeBotId) return;
       await axios.post(`/api/player/${this.activeBotId}/seek`, { position });
-      this.serverElapsed = position;
+      this._setTiming(this.activeBotId, { serverElapsed: position });
       this._syncAfterAction();
     },
 
diff --git a/web/src/views/Home.vue b/web/src/views/Home.vue
index 7e78d50..1f8aaf8 100644
--- a/web/src/views/Home.vue
+++ b/web/src/views/Home.vue
@@ -1,18 +1,5 @@
 <template>
   <div class="home">
-    <!-- Bot Instance Selector -->
-    <div class="bot-selector" v-if="store.bots.length > 1">
-      <button
-        v-for="bot in store.bots"
-        :key="bot.id"
-        class="bot-tab"
-        :class="{ active: bot.id === store.activeBotId }"
-        @click="store.setActiveBotId(bot.id)"
-      >
-        {{ bot.name }}
-      </button>
-    </div>
-
     <!-- Search Bar -->
     <div class="search-bar" @click="$router.push('/search')">
       <Icon icon="mdi:magnify" class="search-icon" />
@@ -166,28 +153,6 @@ onMounted(() => {
 </script>
 
 <style lang="scss" scoped>
-.bot-selector {
-  display: flex;
-  gap: 12px;
-  margin-bottom: 28px;
-}
-
-.bot-tab {
-  padding: 8px 20px;
-  border-radius: var(--radius-md);
-  font-size: 13px;
-  font-weight: 600;
-  background: var(--hover-bg);
-  opacity: 0.6;
-  transition: all var(--transition-fast);
-
-  &.active {
-    background: var(--color-primary);
-    color: white;
-    opacity: 1;
-  }
-}
-
 .search-bar {
   display: flex;
   align-items: center;