agentvibes 5.0.0 → 5.1.0

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

@@ -75,6 +75,9 @@ const TRACK_DISPLAY = Object.freeze({
   'agent_vibes_japanese_city_pop_v1_loop.mp3':         '🌆 Japanese City Pop',
   'agent_vibes_salsa_v2_loop.mp3':                     '💃 Salsa',
   'agent_vibes_tabla_dream_pop_v1_loop.mp3':           '🥁 Tabla Dream Pop',
+  'Late Night Hip Hop Groove.mp3':                     '🎤 Late Night Hip Hop Groove',
+  'Drifting Down the Hall.mp3':                        '🌃 Drifting Down the Hall',
+  'Midnight Charleston Stomp.mp3':                     '🎩 Midnight Charleston Stomp',
 });
 
 const BUILT_IN_TRACK_CATALOG = Object.freeze([
@@ -93,6 +96,9 @@ const BUILT_IN_TRACK_CATALOG = Object.freeze([
   { id: 'agent_vibes_japanese_city_pop_v1_loop.mp3',         label: '🌆 Japanese City Pop' },
   { id: 'agent_vibes_salsa_v2_loop.mp3',                     label: '💃 Salsa' },
   { id: 'agent_vibes_tabla_dream_pop_v1_loop.mp3',           label: '🥁 Tabla Dream Pop' },
+  { id: 'Late Night Hip Hop Groove.mp3',                     label: '🎤 Late Night Hip Hop Groove' },
+  { id: 'Drifting Down the Hall.mp3',                        label: '🌃 Drifting Down the Hall' },
+  { id: 'Midnight Charleston Stomp.mp3',                     label: '🎩 Midnight Charleston Stomp' },
 ]);
 
 // ---------------------------------------------------------------------------
@@ -180,10 +186,13 @@ export function scanTracks() {
   try {
     const files = fs.readdirSync(tracksDir);
     const builtInIds = new Set(BUILT_IN_TRACK_CATALOG.map(t => t.id));
+    // Sort by the alphabetic part of the label (skip leading emoji/symbols)
+    // so the order reflects the track NAME, not the emoji codepoint.
+    const _sortKey = (s) => s.replace(/^[^a-zA-Z]+/, '');
     return files
       .filter(f => /\.mp3$/i.test(f))
-      .sort()
-      .map(f => ({ id: f, label: formatTrackLabel(f), isBuiltIn: builtInIds.has(f) }));
+      .map(f => ({ id: f, label: formatTrackLabel(f), isBuiltIn: builtInIds.has(f) }))
+      .sort((a, b) => _sortKey(a.label).localeCompare(_sortKey(b.label), undefined, { sensitivity: 'base' }));
   } catch {
     // Directory not found or unreadable — use the static catalog
     return BUILT_IN_TRACK_CATALOG.map(t => ({ ...t, isBuiltIn: true }));
@@ -470,24 +479,36 @@ export function createMusicTab(screen, services) {
   let _hintBase = '';   // content of items[_hintIdx] before hint was appended
   let _refreshing = false;
 
-  // Known limitation: _updateHint and _tlTick (blink) can interleave,
-  // causing brief visual glitches. The _refreshing guard prevents the worst
-  // cases but is not a complete fix. Acceptable for a TUI animation.
+  // Decoration helpers — strip blink cursor and (optionally) hint text from
+  // a list item's content. We strip hint by anchoring on the EXACT text from
+  // _rowHint() rather than a generic regex, so we cannot accidentally erase
+  // unrelated content that happens to contain similar tags.
+  const _BLINK_RE = / █/g;
+  function _stripBlink(raw) {
+    return (raw ?? '').replace(_BLINK_RE, '');
+  }
+  function _stripHint(raw) {
+    const hint = _rowHint();
+    if (!hint) return raw;
+    const noBlink = (raw ?? '').replace(_BLINK_RE, '');
+    if (noBlink.endsWith(hint)) return noBlink.slice(0, -hint.length);
+    return noBlink;
+  }
+  function _stripDecorations(raw) {
+    return _stripHint(_stripBlink(raw));
+  }
+
   function _updateHint(idx) {
     const items = trackList.items;
     // Restore previously hinted row — pad with spaces to overwrite ghost hint text
     const _pad = ' '.repeat(60);
     if (_hintIdx >= 0 && _hintIdx !== idx && items[_hintIdx]) {
-      const hadBlink = (items[_hintIdx].content ?? '').endsWith(' █');
-      items[_hintIdx].setContent((hadBlink ? _hintBase + ' █' : _hintBase) + _pad);
+      items[_hintIdx].setContent(_hintBase + _pad);
     }
     // Add hint to the new row, saving its clean base first
     if (idx >= 0 && items[idx]) {
-      let c = items[idx].content ?? '';
-      const hasBlink = c.endsWith(' █');
-      if (hasBlink) c = c.slice(0, -2);
-      _hintBase = c;
-      items[idx].setContent(c + _rowHint() + (hasBlink ? ' █' : ''));
+      _hintBase = _stripDecorations(items[idx].content);
+      items[idx].setContent(_hintBase + _rowHint());
     } else {
       _hintBase = '';
     }
@@ -909,13 +930,14 @@ export function createMusicTab(screen, services) {
     _tlBlink.on = !_tlBlink.on;
     const items = trackList.items;
     const cur = trackList.selected ?? 0;
+    // Clean old row — only strip blink (hint, if any, is preserved)
     if (_tlBlink.sel !== cur && _tlBlink.sel >= 0 && items[_tlBlink.sel]) {
-      items[_tlBlink.sel].setContent((items[_tlBlink.sel].content ?? '').replace(/ █$/, ''));
+      items[_tlBlink.sel].setContent(_stripBlink(items[_tlBlink.sel].content));
     }
     _tlBlink.sel = cur;
     if (items[cur]) {
-      const base = (items[cur].content ?? '').replace(/ █$/, '');
-      items[cur].setContent(_tlBlink.on ? `${base} █` : base);
+      const raw = _stripBlink(items[cur].content);
+      items[cur].setContent(_tlBlink.on ? `${raw} █` : raw);
     }
     screen.render();
   }
@@ -927,7 +949,11 @@ export function createMusicTab(screen, services) {
     _hintBase = '';
     _updateHint(_tlBlink.sel);
     const items = trackList.items;
-    if (items[_tlBlink.sel]) items[_tlBlink.sel].setContent((items[_tlBlink.sel].content ?? '') + ' █');
+    // Append blink cursor — content already has hint from _updateHint
+    if (items[_tlBlink.sel]) {
+      const raw = _stripBlink(items[_tlBlink.sel].content);
+      items[_tlBlink.sel].setContent(raw + ' █');
+    }
     if (!_playingTrackId) previewLine.setContent(_hintText());
     screen.render();
     _tlBlink.interval = setInterval(_tlTick, 500);
@@ -937,13 +963,17 @@ export function createMusicTab(screen, services) {
     if (!_playingTrackId) previewLine.setContent('');
     if (_tlBlink.interval) { clearInterval(_tlBlink.interval); _tlBlink.interval = null; }
     const items = trackList.items;
+    // Strip blink from selected row, strip both from the hinted row
     const sel = trackList.selected ?? 0;
     if (items[sel]) {
-      // Restore the hinted item to its clean base; for non-hinted items just strip █
-      items[sel].setContent(sel === _hintIdx ? _hintBase : (items[sel].content ?? '').replace(/ █$/, ''));
+      if (sel === _hintIdx) {
+        items[sel].setContent(_stripDecorations(items[sel].content));
+      } else {
+        items[sel].setContent(_stripBlink(items[sel].content));
+      }
     }
     if (_hintIdx >= 0 && _hintIdx !== sel && items[_hintIdx]) {
-      items[_hintIdx].setContent(_hintBase);
+      items[_hintIdx].setContent(_stripDecorations(items[_hintIdx].content));
     }
     _hintIdx = -1;
     _hintBase = '';

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

@@ -20,7 +20,7 @@ import os from 'node:os';
 import crypto from 'node:crypto';
 import { spawn } from 'node:child_process';
 import {
-  scanInstalledVoices, getVoiceMeta, PIPER_VOICES_DIR, SAMPLE_PHRASES, parseMultiSpeaker,
+  scanInstalledVoices, getVoiceMeta, genderIconTag, PIPER_VOICES_DIR, SAMPLE_PHRASES, parseMultiSpeaker,
 } from './voices-tab.js';
 import { LanguageService } from '../../services/language-service.js';
 import { SUPPORTED_LANGUAGES, t } from '../../i18n/strings.js';
@@ -525,7 +525,6 @@ export function createSettingsTab(screen, services) {
     navigationService?.openModal();
 
     let _allVoices = [];
-    let _filterText = '';
     let _previewProc = null;
     let _previewVoiceId = null;
     let _vpClosed = false;
@@ -567,26 +566,16 @@ export function createSettingsTab(screen, services) {
     });
     vpModal.setFront();
 
+    const COL_N = 30;
+    const COL_G = 4;
     blessed.text({
-      parent: vpModal, top: 1, left: 2,
-      content: 'Search:', style: { fg: COLORS.labelFg, bg: COLORS.contentBg },
-    });
-    const vpSearch = blessed.textbox({
-      parent: vpModal, top: 1, left: 11, width: 40, height: 1,
-      inputOnFocus: true, keys: true,
-      style: { fg: COLORS.valueFg, bg: 'blue', focus: { bg: 'cyan' } },
-    });
-
-    const COL_N = 28;
-    const COL_G = 10;
-    blessed.text({
-      parent: vpModal, top: 2, left: 6, tags: true,
-      content: `{cyan-fg}${'Name'.padEnd(COL_N)}${'Gender'.padEnd(COL_G)}Provider{/cyan-fg}`,
+      parent: vpModal, top: 1, left: 6, tags: true,
+      content: `{cyan-fg}${'Name'.padEnd(COL_N)}{/cyan-fg}{magenta-fg}♀{/magenta-fg}/{bright-cyan-fg}♂{/bright-cyan-fg} {cyan-fg}Provider{/cyan-fg}`,
       style: { bg: COLORS.contentBg },
     });
 
     const vpList = blessed.list({
-      parent: vpModal, top: 3, left: 2, right: 2, bottom: 5,
+      parent: vpModal, top: 2, left: 2, right: 2, bottom: 5,
       keys: true, vi: true, mouse: true,
       border: { type: 'line' },
       scrollbar: { ch: '|', style: { fg: 'cyan' } },
@@ -606,16 +595,10 @@ export function createSettingsTab(screen, services) {
 
     blessed.text({
       parent: vpModal, bottom: 2, left: 2, right: 2, tags: true,
-      content: '{white-fg}[↑↓/jk] Navigate  [Enter] Select  [Space] Preview  [/] Search  [Esc] Cancel{/white-fg}',
+      content: '{white-fg}[↑↓] Nav  [PgUp/PgDn] Page  [Home/End]  [a-z] Jump  [Enter] Select  [Space] Preview  [Esc] Cancel{/white-fg}',
       style: { bg: COLORS.contentBg },
     });
 
-    function _getFiltered() {
-      if (!_filterText) return _allVoices;
-      const f = _filterText.toLowerCase();
-      return _allVoices.filter(v => v.toLowerCase().includes(f));
-    }
-
     function _buildItems(voices) {
       const currentVoice = providerService?.getActiveVoiceId() ?? '';
       return voices.map(v => {
@@ -626,7 +609,8 @@ export function createSettingsTab(screen, services) {
         const name = meta.displayName.length > COL_N
           ? meta.displayName.slice(0, COL_N - 1) + '…'
           : meta.displayName.padEnd(COL_N);
-        return ` ${dot} ${name}${meta.gender.padEnd(COL_G)}${meta.provider}`;
+        // genderIconTag has invisible color tags — pad with literal spaces (1 visible char + 3 spaces = 4)
+        return ` ${dot} ${name}${genderIconTag(meta.gender)}   ${meta.provider}`;
       });
     }
 
@@ -635,8 +619,10 @@ export function createSettingsTab(screen, services) {
       const savedIdx = vpList.selected ?? 0;
       const savedScroll = vpList.childBase ?? 0;
       _allVoices = scanInstalledVoices();
-      const filtered = _getFiltered();
-      const items = _buildItems(filtered);
+      // Sort by display name so the first-letter quick jump is intuitive
+      _allVoices.sort((a, b) => getVoiceMeta(a).displayName.localeCompare(
+        getVoiceMeta(b).displayName, undefined, { sensitivity: 'base' }));
+      const items = _buildItems(_allVoices);
       vpList.setItems(items.length > 0 ? items : [' (no voices found)']);
       vpList.select(Math.min(savedIdx, items.length - 1));
       vpList.childBase = Math.min(savedScroll, Math.max(0, items.length - (vpList.height - 2)));
@@ -704,14 +690,8 @@ export function createSettingsTab(screen, services) {
       piper.on('error', () => { _previewProc = null; _previewVoiceId = null; });
     }
 
-    vpSearch.on('keypress', () => {
-      setTimeout(() => { _filterText = vpSearch.getValue().trim(); _refreshVP(); }, 0);
-    });
-    vpSearch.key(['escape'], () => { vpList.focus(); screen.render(); });
-    vpList.key(['/'], () => { vpSearch.clearValue(); vpSearch.focus(); screen.render(); });
     vpList.key(['enter'], () => {
-      const filtered = _getFiltered();
-      const sel = filtered[vpList.selected];
+      const sel = _allVoices[vpList.selected];
       if (sel) {
         if (providerService) providerService.setActiveVoice(sel);
         else configService.set('voice', sel);
@@ -719,15 +699,37 @@ export function createSettingsTab(screen, services) {
       _closeVP();
     });
     vpList.key(['space'], () => {
-      const filtered = _getFiltered();
-      const sel = filtered[vpList.selected];
+      const sel = _allVoices[vpList.selected];
       if (sel) _previewVoice(sel);
     });
     vpList.key(['escape', 'q'], _closeVP);
 
+    // PageUp / PageDown / Home / End navigation
+    const _pageSize = () => Math.max(1, (vpList.height ?? 10) - 2);
+    vpList.key(['pageup'],   () => { vpList.up(_pageSize());   screen.render(); });
+    vpList.key(['pagedown'], () => { vpList.down(_pageSize()); screen.render(); });
+    vpList.key(['home'],     () => { vpList.select(0); screen.render(); });
+    vpList.key(['end'],      () => { vpList.select(Math.max(0, _allVoices.length - 1)); screen.render(); });
+
+    // First-letter quick jump: typing 'a' jumps to the first voice starting
+    // with A. Block keys reserved by the list widget (vi nav, cancel) so
+    // they don't get swallowed: q (cancel), j/k/g/h/l (vi navigation).
+    const _vpJumpBlocked = new Set(['j', 'k', 'g', 'h', 'l', 'q']);
+    vpList.on('keypress', (ch, key) => {
+      if (!ch || key?.ctrl || key?.meta) return;
+      if (!/^[a-zA-Z]$/.test(ch)) return;
+      const target = ch.toLowerCase();
+      if (_vpJumpBlocked.has(target)) return;
+      const idx = _allVoices.findIndex(v => {
+        const name = getVoiceMeta(v).displayName.toLowerCase();
+        return name.startsWith(target);
+      });
+      if (idx >= 0) { vpList.select(idx); screen.render(); }
+    });
+
     _refreshVP();
     const currentVoice = providerService?.getActiveVoiceId() ?? '';
-    const activeIdx = _getFiltered().indexOf(currentVoice);
+    const activeIdx = _allVoices.indexOf(currentVoice);
     if (activeIdx >= 0) vpList.select(activeIdx);
     vpList.focus();
     screen.render();