agentvibes 5.12.0-alpha.0 → 5.13.0

package/src/console/tabs/music-tab.js

@@ -11,16 +11,20 @@
 
 import fs from 'node:fs';
 import path from 'node:path';
-import os from 'node:os';
 import { fileURLToPath } from 'node:url';
 import { buildAudioEnv, spawnMp3Player } from '../audio-env.js';
+import { resolveMusicProvider, spawnMusicRemote } from '../music-preview.js';
+import { playBlingCue } from '../bling.js';
 import { t } from '../../i18n/strings.js';
 
+const _MUSIC_TAB_DIR = path.dirname(fileURLToPath(import.meta.url));
+
+// AgentVibes package/repo root — used to resolve bundled assets (tracks dir,
+// bling cue) and as the package fallback when resolving the active provider.
+const _PKG_ROOT = path.resolve(_MUSIC_TAB_DIR, '..', '..', '..');
+
 // Package-relative tracks dir — used as fallback when cwd has no .claude/audio/tracks/
-const _PKG_TRACKS_DIR = path.resolve(
-  path.dirname(fileURLToPath(import.meta.url)),
-  '..', '..', '..', '.claude', 'audio', 'tracks'
-);
+const _PKG_TRACKS_DIR = path.join(_PKG_ROOT, '.claude', 'audio', 'tracks');
 
 const IS_TEST = process.env.AGENTVIBES_TEST_MODE === 'true';
 
@@ -540,6 +544,10 @@ export function createMusicTab(screen, services) {
 
   let _playingProcess = null;
   let _playingTrackId = null;
+  // Remote playback is fire-and-forget (the local sender exits in ms while the
+  // receiver keeps playing), so the on/off toggle tracks the intended remote
+  // track here rather than relying on a live local process.
+  let _remotePlayingTrackId = null;
 
   function _killPlayingProcess() {
     if (_playingProcess) {
@@ -550,6 +558,44 @@ export function createMusicTab(screen, services) {
 
   const _spawnEnv = buildAudioEnv();
 
+  /**
+   * Spawn the SSH sender to play a music track on the receiver, or stop the
+   * current one. Fire-and-forget (the sender exits after handing the payload to
+   * SSH; the receiver plays/stops asynchronously). Returns true if the send was
+   * launched. On a play error the remote-playing state is cleared so the toggle
+   * doesn't get stuck "on".
+   * @param {{remote:boolean, projectDir:string}} mp
+   * @param {{track?:string, stop?:boolean}} opts
+   */
+  function _sendMusicRemote(mp, { track = null, stop = false } = {}) {
+    let rproc;
+    try {
+      rproc = spawnMusicRemote({ packageRoot: _PKG_ROOT, projectDir: mp.projectDir, env: _spawnEnv, track, stop });
+    } catch {
+      previewLine.setContent('{red-fg}Remote music preview failed{/red-fg}');
+      screen.render();
+      setTimeout(() => { previewLine.setContent(_listFocused ? _hintText() : ''); screen.render(); }, 4000);
+      return false;
+    }
+    let _rerr = '';
+    if (rproc.stderr) rproc.stderr.on('data', d => { _rerr += d.toString(); });
+    rproc.on('exit', (code) => {
+      if (stop || code === 0) return;
+      if (_remotePlayingTrackId === track) _remotePlayingTrackId = null;
+      const msg = _rerr.trim().split('\n').pop() || 'Remote preview failed';
+      previewLine.setContent(`{red-fg}♪ ${msg}{/red-fg}`);
+      screen.render();
+      setTimeout(() => { previewLine.setContent(_listFocused ? _hintText() : ''); screen.render(); }, 4000);
+    });
+    rproc.on('error', () => {
+      if (stop) return;
+      if (_remotePlayingTrackId === track) _remotePlayingTrackId = null;
+      previewLine.setContent('{red-fg}Remote music preview failed{/red-fg}');
+      screen.render();
+    });
+    return true;
+  }
+
   process.on('exit', () => { _killPlayingProcess(); });
 
   /**
@@ -566,6 +612,32 @@ export function createMusicTab(screen, services) {
       return;
     }
 
+    // Remote provider: forward to the receiver instead of local playback (which
+    // is silent on a headless box). The receiver auto-stops any prior track when
+    // a new one arrives; pressing Space on the currently-playing track sends an
+    // explicit stop (toggle off).
+    const _mp = resolveMusicProvider(_PKG_ROOT);
+    if (_mp.remote) {
+      if (_remotePlayingTrackId === trackId) {
+        _sendMusicRemote(_mp, { stop: true });
+        _remotePlayingTrackId = null;
+        previewLine.setContent(_listFocused ? _hintText() : '');
+        screen.render();
+        return;
+      }
+      // Bling first (fire-and-forget, plays locally) — same readiness cue as the
+      // voice preview — then forward the track to the receiver.
+      playBlingCue(_PKG_ROOT);
+      const rlabel = _allTracks.find(t => t.id === trackId)?.label ?? formatTrackLabel(trackId);
+      if (_sendMusicRemote(_mp, { track: trackId })) {
+        _remotePlayingTrackId = trackId;
+        previewLine.setContent(`{${COLORS.playingFg}-fg}♪ Playing on receiver: ${rlabel}  (Space to stop){/${COLORS.playingFg}-fg}`);
+      }
+      screen.render();
+      return;
+    }
+
+    // ── Local playback ──────────────────────────────────────────────────────
     // Toggle: second press on the same track → stop
     if (_playingTrackId === trackId) {
       _killPlayingProcess();
@@ -579,6 +651,10 @@ export function createMusicTab(screen, services) {
     _killPlayingProcess();
     _playingTrackId = null;
 
+    // Bling first (fire-and-forget) — same readiness cue as the voice preview —
+    // then start local playback.
+    playBlingCue(_PKG_ROOT);
+
     const proc = spawnMp3Player(trackPath, _spawnEnv);
     if (!proc) {
       const installHint = process.platform === 'win32'

package/src/console/tabs/placeholder-tab.js

@@ -38,7 +38,6 @@ export function createPlaceholderTab(contentArea, label) {
 /** Map of tabId → display label for all tabs */
 export const TAB_DISPLAY_LABELS = {
   settings:        'Settings',
-  voices:          'Voices',
   music:           'Music',
   agents:          'BMad',
   receiver:        'Receiver',
@@ -66,7 +65,6 @@ export function getTabLabel(id, lang = 'en') {
   const keyMap = {
     setup:           'tabSetup',
     settings:        'tabSettings',
-    voices:          'tabVoices',
     music:           'tabMusic',
     agents:          'tabBmad',
     receiver:        'tabReceiver',