agentvibes 4.2.0 → 4.4.0

package/src/services/navigation-service.js

@@ -1,123 +1,123 @@
-/**
- * AgentVibes TUI Console — Navigation Service
- * Story 6.2: Tab Bar & Global Keyboard Navigation
- *
- * Manages tab state, cycling, modal overlay state, and focus stack.
- * Used by navigation.js (key bindings) and app.js (wiring).
- */
-
-/** Ordered list of all tab IDs — used for cycling and validation */
-export const TAB_ORDER = ['install', 'settings', 'voices', 'music', 'agents', 'receiver', 'readme', 'help'];
-
-export class NavigationService {
-  /**
-   * @param {string} [initialTab='settings'] - Tab to activate on launch
-   */
-  constructor(initialTab = 'settings') {
-    this._activeTab = TAB_ORDER.includes(initialTab) ? initialTab : 'settings';
-    this._switchCallbacks = [];
-    this._focusStack = [];
-    this._modalOpen = false;
-  }
-
-  // ---------------------------------------------------------------------------
-  // Tab navigation
-
-  /** Returns the currently active tab ID */
-  getActiveTab() {
-    return this._activeTab;
-  }
-
-  /**
-   * Switch to the given tab.
-   * Ignores invalid tab IDs. Fires all registered onSwitch callbacks.
-   * @param {string} tabId
-   */
-  switchTab(tabId) {
-    if (!TAB_ORDER.includes(tabId)) return;
-    if (tabId === this._activeTab) return; // no-op: already on this tab
-    this._activeTab = tabId;
-    this._switchCallbacks.forEach(cb => cb(tabId));
-  }
-
-  /**
-   * Activate a tab unconditionally, bypassing the same-tab no-op guard.
-   * Used for initial UI setup: the constructor pre-sets _activeTab but
-   * onSwitch callbacks must still fire to render the initial state.
-   * @param {string} tabId
-   */
-  forceActivate(tabId) {
-    if (!TAB_ORDER.includes(tabId)) return;
-    this._activeTab = tabId;
-    this._switchCallbacks.forEach(cb => cb(tabId));
-  }
-
-  /**
-   * Cycle to the next tab in TAB_ORDER, wrapping from last back to first.
-   */
-  cycleTab() {
-    const idx = TAB_ORDER.indexOf(this._activeTab);
-    const nextIdx = (idx + 1) % TAB_ORDER.length;
-    this.switchTab(TAB_ORDER[nextIdx]);
-  }
-
-  /**
-   * Cycle to the previous tab in TAB_ORDER, wrapping from first back to last.
-   */
-  cycleTabPrev() {
-    const idx = TAB_ORDER.indexOf(this._activeTab);
-    const prevIdx = (idx - 1 + TAB_ORDER.length) % TAB_ORDER.length;
-    this.switchTab(TAB_ORDER[prevIdx]);
-  }
-
-  /**
-   * Register a callback fired whenever the active tab changes.
-   * @param {(tabId: string) => void} callback
-   */
-  onSwitch(callback) {
-    this._switchCallbacks.push(callback);
-  }
-
-  // ---------------------------------------------------------------------------
-  // Modal state (story 6.4 will expand this)
-
-  /** Returns true if a modal is currently open */
-  isModalOpen() {
-    return this._modalOpen;
-  }
-
-  /**
-   * Open a modal. Sets modal-open state and calls the factory fn if provided.
-   * @param {Function|null} fn - Optional factory/callback invoked immediately
-   */
-  openModal(fn) {
-    this._modalOpen = true;
-    fn?.();
-  }
-
-  /** Close the current modal, restoring modal-closed state */
-  closeModal() {
-    this._modalOpen = false;
-  }
-
-
-  // ---------------------------------------------------------------------------
-  // Focus stack (story 7.6 will use this for button-level focus)
-
-  /**
-   * Push a Blessed element onto the focus stack
-   * @param {object} element - Blessed widget
-   */
-  pushFocus(element) {
-    this._focusStack.push(element);
-  }
-
-  /**
-   * Pop the last element from the focus stack.
-   * Returns undefined if the stack is empty.
-   * @returns {object|undefined}
-   */
-  popFocus() {
-    return this._focusStack.pop();
-  }
-}
+/**
+ * AgentVibes TUI Console — Navigation Service
+ * Story 6.2: Tab Bar & Global Keyboard Navigation
+ *
+ * Manages tab state, cycling, modal overlay state, and focus stack.
+ * Used by navigation.js (key bindings) and app.js (wiring).
+ */
+
+/** Ordered list of all tab IDs — used for cycling and validation */
+export const TAB_ORDER = ['install', 'settings', 'voices', 'music', 'agents', 'receiver', 'readme', 'help'];
+
+export class NavigationService {
+  /**
+   * @param {string} [initialTab='settings'] - Tab to activate on launch
+   */
+  constructor(initialTab = 'settings') {
+    this._activeTab = TAB_ORDER.includes(initialTab) ? initialTab : 'settings';
+    this._switchCallbacks = [];
+    this._focusStack = [];
+    this._modalOpen = false;
+  }
+
+  // ---------------------------------------------------------------------------
+  // Tab navigation
+
+  /** Returns the currently active tab ID */
+  getActiveTab() {
+    return this._activeTab;
+  }
+
+  /**
+   * Switch to the given tab.
+   * Ignores invalid tab IDs. Fires all registered onSwitch callbacks.
+   * @param {string} tabId
+   */
+  switchTab(tabId) {
+    if (!TAB_ORDER.includes(tabId)) return;
+    if (tabId === this._activeTab) return; // no-op: already on this tab
+    this._activeTab = tabId;
+    this._switchCallbacks.forEach(cb => cb(tabId));
+  }
+
+  /**
+   * Activate a tab unconditionally, bypassing the same-tab no-op guard.
+   * Used for initial UI setup: the constructor pre-sets _activeTab but
+   * onSwitch callbacks must still fire to render the initial state.
+   * @param {string} tabId
+   */
+  forceActivate(tabId) {
+    if (!TAB_ORDER.includes(tabId)) return;
+    this._activeTab = tabId;
+    this._switchCallbacks.forEach(cb => cb(tabId));
+  }
+
+  /**
+   * Cycle to the next tab in TAB_ORDER, wrapping from last back to first.
+   */
+  cycleTab() {
+    const idx = TAB_ORDER.indexOf(this._activeTab);
+    const nextIdx = (idx + 1) % TAB_ORDER.length;
+    this.switchTab(TAB_ORDER[nextIdx]);
+  }
+
+  /**
+   * Cycle to the previous tab in TAB_ORDER, wrapping from first back to last.
+   */
+  cycleTabPrev() {
+    const idx = TAB_ORDER.indexOf(this._activeTab);
+    const prevIdx = (idx - 1 + TAB_ORDER.length) % TAB_ORDER.length;
+    this.switchTab(TAB_ORDER[prevIdx]);
+  }
+
+  /**
+   * Register a callback fired whenever the active tab changes.
+   * @param {(tabId: string) => void} callback
+   */
+  onSwitch(callback) {
+    this._switchCallbacks.push(callback);
+  }
+
+  // ---------------------------------------------------------------------------
+  // Modal state (story 6.4 will expand this)
+
+  /** Returns true if a modal is currently open */
+  isModalOpen() {
+    return this._modalOpen;
+  }
+
+  /**
+   * Open a modal. Sets modal-open state and calls the factory fn if provided.
+   * @param {Function|null} fn - Optional factory/callback invoked immediately
+   */
+  openModal(fn) {
+    this._modalOpen = true;
+    fn?.();
+  }
+
+  /** Close the current modal, restoring modal-closed state */
+  closeModal() {
+    this._modalOpen = false;
+  }
+
+
+  // ---------------------------------------------------------------------------
+  // Focus stack (story 7.6 will use this for button-level focus)
+
+  /**
+   * Push a Blessed element onto the focus stack
+   * @param {object} element - Blessed widget
+   */
+  pushFocus(element) {
+    this._focusStack.push(element);
+  }
+
+  /**
+   * Pop the last element from the focus stack.
+   * Returns undefined if the stack is empty.
+   * @returns {object|undefined}
+   */
+  popFocus() {
+    return this._focusStack.pop();
+  }
+}

package/src/services/provider-service.js

@@ -1,132 +1,132 @@
-/**
- * AgentVibes Provider Service
- * Story 7.1: Provider & Voice Settings Group
- *
- * Detects installed TTS providers, reads/writes active provider and voice
- * through ConfigService. Gracefully degrades when detection fails.
- */
-
-import { execFileSync } from 'node:child_process';
-import fs from 'node:fs';
-import path from 'node:path';
-import os from 'node:os';
-
-export class ProviderService {
-  /**
-   * @param {import('./config-service.js').ConfigService} configService
-   */
-  constructor(configService) {
-    this._config = configService;
-    this._installedProviders = null; // cached after first detection
-  }
-
-  // ---------------------------------------------------------------------------
-  // Provider
-
-  /**
-   * Returns the currently active TTS provider from config.
-   * Defaults to 'piper' if not configured.
-   * @returns {string}
-   */
-  getActiveProvider() {
-    return this._config.getConfig().provider ?? 'piper';
-  }
-
-  /**
-   * Sets the active TTS provider in config AND syncs to .claude/tts-provider.txt
-   * so the shell hooks (play-tts.sh → provider-manager.sh) pick up the change.
-   * @param {string} provider
-   */
-  setActiveProvider(provider) {
-    this._config.set('provider', provider);
-    this._syncProviderFile(provider);
-  }
-
-  /**
-   * Write provider to .claude/tts-provider.txt so shell hooks stay in sync.
-   * Writes to projectRoot/.claude/tts-provider.txt if .claude/ exists there,
-   * otherwise falls back to ~/.claude/tts-provider.txt.
-   * @param {string} provider
-   */
-  _syncProviderFile(provider) {
-    try {
-      const projectClaudeDir = path.resolve(this._config.getProjectRoot(), '.claude');
-      const targetDir = fs.existsSync(projectClaudeDir)
-        ? projectClaudeDir
-        : path.resolve(os.homedir(), '.claude');
-      const targetFile = path.resolve(targetDir, 'tts-provider.txt');
-      // Verify resolved path stays within targetDir (path traversal guard)
-      if (!targetFile.startsWith(targetDir + path.sep) && targetFile !== targetDir) return;
-      fs.mkdirSync(targetDir, { recursive: true });
-      fs.writeFileSync(targetFile, provider, 'utf8');
-    } catch {
-      // Non-fatal — config.json is the authoritative source
-    }
-  }
-
-  /**
-   * Returns an array of installed/available TTS providers.
-   * Detection uses `which` binary check. Always returns at least ['piper']
-   * as graceful degradation (piper is the primary supported provider).
-   * @returns {string[]}
-   */
-  getInstalledProviders() {
-    if (this._installedProviders) return this._installedProviders;
-
-    const providers = [];
-
-    if (this._isAvailable('piper')) providers.push('piper');
-    if (this._isAvailable('soprano')) providers.push('soprano');
-
-    // macOS Say (darwin only)
-    if (process.platform === 'darwin' && this._isAvailable('say')) {
-      providers.push('macos');
-    }
-
-    // Graceful degradation: always return at least piper
-    if (providers.length === 0) providers.push('piper');
-
-    this._installedProviders = providers;
-    return providers;
-  }
-
-  // ---------------------------------------------------------------------------
-  // Voice
-
-  /**
-   * Returns the currently active voice ID from config.
-   * Defaults to 'en_US-amy-medium' if not configured.
-   * @returns {string}
-   */
-  getActiveVoiceId() {
-    return this._config.getConfig().voice ?? 'en_US-amy-medium';
-  }
-
-  /**
-   * Sets the active voice ID in config.
-   * @param {string} voiceId
-   */
-  setActiveVoice(voiceId) {
-    this._config.set('voice', voiceId);
-  }
-
-  // ---------------------------------------------------------------------------
-  // Private
-
-  /**
-   * Check if a binary is available in PATH using `which`.
-   * Binary names are all hardcoded (not user input) — safe from injection.
-   * @param {string} binary - hardcoded binary name ('piper', 'soprano', 'say')
-   * @returns {boolean}
-   */
-  _isAvailable(binary) {
-    try {
-      execFileSync('which', [binary], { stdio: 'ignore', timeout: 2000 });
-      return true;
-    } catch {
-      return false;
-    }
-  }
-}
-
-export default ProviderService;
+/**
+ * AgentVibes Provider Service
+ * Story 7.1: Provider & Voice Settings Group
+ *
+ * Detects installed TTS providers, reads/writes active provider and voice
+ * through ConfigService. Gracefully degrades when detection fails.
+ */
+
+import { execFileSync } from 'node:child_process';
+import fs from 'node:fs';
+import path from 'node:path';
+import os from 'node:os';
+
+export class ProviderService {
+  /**
+   * @param {import('./config-service.js').ConfigService} configService
+   */
+  constructor(configService) {
+    this._config = configService;
+    this._installedProviders = null; // cached after first detection
+  }
+
+  // ---------------------------------------------------------------------------
+  // Provider
+
+  /**
+   * Returns the currently active TTS provider from config.
+   * Defaults to 'piper' if not configured.
+   * @returns {string}
+   */
+  getActiveProvider() {
+    return this._config.getConfig().provider ?? 'piper';
+  }
+
+  /**
+   * Sets the active TTS provider in config AND syncs to .claude/tts-provider.txt
+   * so the shell hooks (play-tts.sh → provider-manager.sh) pick up the change.
+   * @param {string} provider
+   */
+  setActiveProvider(provider) {
+    this._config.set('provider', provider);
+    this._syncProviderFile(provider);
+  }
+
+  /**
+   * Write provider to .claude/tts-provider.txt so shell hooks stay in sync.
+   * Writes to projectRoot/.claude/tts-provider.txt if .claude/ exists there,
+   * otherwise falls back to ~/.claude/tts-provider.txt.
+   * @param {string} provider
+   */
+  _syncProviderFile(provider) {
+    try {
+      const projectClaudeDir = path.resolve(this._config.getProjectRoot(), '.claude');
+      const targetDir = fs.existsSync(projectClaudeDir)
+        ? projectClaudeDir
+        : path.resolve(os.homedir(), '.claude');
+      const targetFile = path.resolve(targetDir, 'tts-provider.txt');
+      // Verify resolved path stays within targetDir (path traversal guard)
+      if (!targetFile.startsWith(targetDir + path.sep) && targetFile !== targetDir) return;
+      fs.mkdirSync(targetDir, { recursive: true });
+      fs.writeFileSync(targetFile, provider, 'utf8');
+    } catch {
+      // Non-fatal — config.json is the authoritative source
+    }
+  }
+
+  /**
+   * Returns an array of installed/available TTS providers.
+   * Detection uses `which` binary check. Always returns at least ['piper']
+   * as graceful degradation (piper is the primary supported provider).
+   * @returns {string[]}
+   */
+  getInstalledProviders() {
+    if (this._installedProviders) return this._installedProviders;
+
+    const providers = [];
+
+    if (this._isAvailable('piper')) providers.push('piper');
+    if (this._isAvailable('soprano')) providers.push('soprano');
+
+    // macOS Say (darwin only)
+    if (process.platform === 'darwin' && this._isAvailable('say')) {
+      providers.push('macos');
+    }
+
+    // Graceful degradation: always return at least piper
+    if (providers.length === 0) providers.push('piper');
+
+    this._installedProviders = providers;
+    return providers;
+  }
+
+  // ---------------------------------------------------------------------------
+  // Voice
+
+  /**
+   * Returns the currently active voice ID from config.
+   * Defaults to 'en_US-amy-medium' if not configured.
+   * @returns {string}
+   */
+  getActiveVoiceId() {
+    return this._config.getConfig().voice ?? 'en_US-amy-medium';
+  }
+
+  /**
+   * Sets the active voice ID in config.
+   * @param {string} voiceId
+   */
+  setActiveVoice(voiceId) {
+    this._config.set('voice', voiceId);
+  }
+
+  // ---------------------------------------------------------------------------
+  // Private
+
+  /**
+   * Check if a binary is available in PATH using `which`.
+   * Binary names are all hardcoded (not user input) — safe from injection.
+   * @param {string} binary - hardcoded binary name ('piper', 'soprano', 'say')
+   * @returns {boolean}
+   */
+  _isAvailable(binary) {
+    try {
+      execFileSync('which', [binary], { stdio: 'ignore', timeout: 2000 });
+      return true;
+    } catch {
+      return false;
+    }
+  }
+}
+
+export default ProviderService;