agentvibes 5.9.0 → 5.10.1

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

@@ -123,7 +123,7 @@ async function _ensureSopranoWebUI(onStatus, signal) {
   if (Date.now() - _sopranoSpawnedAt > 10_000) {
     _sopranoSpawnedAt = Date.now();
     try {
-      const p = spawn('soprano-webui', [], {
+      const p = spawn('soprano-webui', [], { // NOSONAR
         stdio: 'ignore', detached: true, windowsHide: true,
         shell: process.platform === 'win32',
       });
@@ -1159,7 +1159,7 @@ export function createSetupTab(screen, services) {
       const raw = fs.readFileSync(sshConfigPath, 'utf8');
       const seen = new Set();
       for (const line of raw.split('\n')) {
-        const m = line.match(/^\s*IdentityFile\s+(.+)\s*$/i);
+        const m = line.match(/^\s*IdentityFile\s+(.+)\s*$/i); // NOSONAR
         if (m) {
           const expanded = m[1].trim().replace(/^~/, os.homedir());
           if (!seen.has(expanded)) { seen.add(expanded); keys.push(expanded); }
@@ -1297,7 +1297,7 @@ export function createSetupTab(screen, services) {
     try {
       const lines = fs.readFileSync(path.join(os.homedir(), '.ssh', 'config'), 'utf8').split('\n');
       for (const line of lines) {
-        const m = line.match(/^\s*[Hh]ost\s+(.+)$/);
+        const m = line.match(/^\s*[Hh]ost\s+(.+)$/); // NOSONAR
         if (m) {
           for (const name of m[1].trim().split(/\s+/)) {
             if (!name.includes('*') && !name.includes('?') && !seen.has(name)) {
@@ -1608,10 +1608,10 @@ export function createSetupTab(screen, services) {
       const sampleText = 'This is how your audio settings sound right now.';
       const script = path.join(targetDir, '.claude', hooksSubdir, isWin ? 'play-tts.ps1' : 'play-tts.sh');
       const proc = isWin
-        ? spawn('powershell', ['-NoProfile', '-ExecutionPolicy', 'Bypass', '-File', script, sampleText], {
+        ? spawn('powershell', ['-NoProfile', '-ExecutionPolicy', 'Bypass', '-File', script, sampleText], { // NOSONAR
             stdio: 'ignore', windowsHide: true, env: { ...process.env, CLAUDE_PROJECT_DIR: targetDir },
           })
-        : spawn('bash', [script, sampleText], {
+        : spawn('bash', [script, sampleText], { // NOSONAR
             stdio: 'ignore', env: { ...process.env, CLAUDE_PROJECT_DIR: targetDir },
           });
       _previewProc = proc;
@@ -2360,13 +2360,13 @@ export function createSetupTab(screen, services) {
         let proc = null;
         try {
           if (engine === 'soprano') {
-            proc = spawn('soprano', [phrase], { stdio: 'ignore' });
+            proc = spawn('soprano', [phrase], { stdio: 'ignore' }); // NOSONAR
           } else if (engine === 'sapi') {
             const safePhrase = phrase.replace(/'/g, "''");
             const sapiScript = `Add-Type -AssemblyName System.Speech; (New-Object System.Speech.Synthesis.SpeechSynthesizer).Speak('${safePhrase}')`;
-            proc = spawn('powershell', ['-NoProfile', '-Command', sapiScript], { stdio: 'ignore', windowsHide: true });
+            proc = spawn('powershell', ['-NoProfile', '-Command', sapiScript], { stdio: 'ignore', windowsHide: true }); // NOSONAR
           } else if (engine === 'macos-say') {
-            proc = spawn('say', [phrase], { stdio: 'ignore' });
+            proc = spawn('say', [phrase], { stdio: 'ignore' }); // NOSONAR
           }
         } catch {}
         if (!proc) {
@@ -2519,13 +2519,13 @@ export function createSetupTab(screen, services) {
         if (_isWin) {
           const _playTts = path.join(_hooksBase, '.claude', 'hooks-windows', 'play-tts.ps1');
           const _llmArgs = llmKey ? ['-llm', llmKey] : [];
-          rProc = spawn('powershell', ['-NoProfile', '-ExecutionPolicy', 'Bypass', '-File', _playTts, phrase, voiceId, ..._llmArgs], {
+          rProc = spawn('powershell', ['-NoProfile', '-ExecutionPolicy', 'Bypass', '-File', _playTts, phrase, voiceId, ..._llmArgs], { // NOSONAR
             stdio: 'ignore', detached: false, windowsHide: true, env: _rEnv,
           });
         } else {
           const _playTts = path.join(_hooksBase, '.claude', 'hooks', 'play-tts.sh');
           const _llmArgs = llmKey ? ['--llm', llmKey] : [];
-          rProc = spawn('bash', [_playTts, phrase, voiceId, ..._llmArgs], {
+          rProc = spawn('bash', [_playTts, phrase, voiceId, ..._llmArgs], { // NOSONAR
             stdio: 'ignore', detached: true, env: _rEnv, cwd: targetDir,
           });
         }

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

@@ -936,12 +936,12 @@ export function createVoicesTab(screen, services) {
       let proc;
       if (isWindows) {
         const playTts = path.join(hooksBase, '.claude', 'hooks-windows', 'play-tts.ps1');
-        proc = spawn('powershell', ['-NoProfile', '-ExecutionPolicy', 'Bypass', '-File', playTts, phrase, voiceId], {
+        proc = spawn('powershell', ['-NoProfile', '-ExecutionPolicy', 'Bypass', '-File', playTts, phrase, voiceId], { // NOSONAR
           stdio: 'ignore', detached: false, windowsHide: true, env: _spawnEnv,
         });
       } else {
         const playTts = path.join(hooksBase, '.claude', 'hooks', 'play-tts.sh');
-        proc = spawn('bash', [playTts, phrase, voiceId], {
+        proc = spawn('bash', [playTts, phrase, voiceId], { // NOSONAR
           stdio: ['ignore', 'ignore', 'pipe'], detached: true, env: _spawnEnv,
           cwd: process.cwd(),
         });
@@ -1512,13 +1512,13 @@ export function createVoicesTab(screen, services) {
           Invoke-WebRequest -Uri '${configUrl}' -OutFile '${configFile.replace(/'/g, "''")}' -ErrorAction Stop
           Write-Output 'PHASE:done'
         `;
-        dlProc = spawn('powershell', ['-NoProfile', '-ExecutionPolicy', 'Bypass', '-Command', psScript], {
+        dlProc = spawn('powershell', ['-NoProfile', '-ExecutionPolicy', 'Bypass', '-Command', psScript], { // NOSONAR
           stdio: ['ignore', 'pipe', 'pipe'],
           env: _spawnEnv,
         });
       } else {
         const managerScript = path.resolve(packageRoot, '.claude', 'hooks', 'piper-voice-manager.sh');
-        dlProc = spawn('bash', ['-c', 'source "$1" && download_voice "$2"', '_', managerScript, modelToDownload], {
+        dlProc = spawn('bash', ['-c', 'source "$1" && download_voice "$2"', '_', managerScript, modelToDownload], { // NOSONAR
           stdio: ['ignore', 'pipe', 'pipe'],
           env: _spawnEnv,
         });

package/src/console/widgets/destroy-list.js

@@ -1,25 +1,25 @@
-/**
- * AgentVibes TUI — Shared Widget: Modal Destroy Helper
- *
- * Force-invalidates blessed's olines buffer after destroying a modal widget.
- * Without this, blessed skips repainting cells where lines==olines and the
- * terminal retains stale modal content as ghost artifacts.
- */
-
-/**
- * Destroy a blessed list/box widget and force full screen repaint.
- *
- * @param {object} widget  - blessed widget to destroy
- * @param {object} screen  - blessed screen instance
- * @param {Function} [onClose] - optional callback after destruction
- */
-export function destroyList(widget, screen, onClose) {
-  widget.destroy();
-  try {
-    for (let r = 0; r < screen.height; r++)
-      for (let c = 0; c < screen.width; c++)
-        if (screen.olines[r]?.[c]) screen.olines[r][c][0] = -1;
-  } catch {}
-  onClose?.();
-  screen.render();
-}
+/**
+ * AgentVibes TUI — Shared Widget: Modal Destroy Helper
+ *
+ * Force-invalidates blessed's olines buffer after destroying a modal widget.
+ * Without this, blessed skips repainting cells where lines==olines and the
+ * terminal retains stale modal content as ghost artifacts.
+ */
+
+/**
+ * Destroy a blessed list/box widget and force full screen repaint.
+ *
+ * @param {object} widget  - blessed widget to destroy
+ * @param {object} screen  - blessed screen instance
+ * @param {Function} [onClose] - optional callback after destruction
+ */
+export function destroyList(widget, screen, onClose) {
+  widget.destroy();
+  try {
+    for (let r = 0; r < screen.height; r++)
+      for (let c = 0; c < screen.width; c++)
+        if (screen.olines[r]?.[c]) screen.olines[r][c][0] = -1;
+  } catch {}
+  onClose?.();
+  screen.render();
+}

package/src/console/widgets/notice.js

@@ -1,55 +1,55 @@
-/**
- * AgentVibes TUI — Shared Widget: Notice Toast
- *
- * Displays a short auto-dismissing notice modal centred on screen.
- * Usable from any tab; no settings-specific state required.
- */
-
-import { destroyList } from './destroy-list.js';
-
-const IS_TEST = process.env.AGENTVIBES_TEST_MODE === 'true';
-let blessed;
-if (!IS_TEST) {
-  const { default: b } = await import('blessed');
-  blessed = b;
-}
-
-/**
- * Show a temporary notice that auto-dismisses after 2.5 seconds.
- *
- * @param {object} screen  - blessed screen instance
- * @param {string} message - text to display
- * @param {object} [opts]
- * @param {string} [opts.bg='#0a0e1a']       - background colour
- * @param {string} [opts.fg='#e3f2fd']       - foreground colour
- * @param {string} [opts.borderFg='bright-cyan'] - border colour
- * @param {number} [opts.durationMs=2500]    - auto-dismiss delay in ms
- */
-export function showNotice(screen, message, opts = {}) {
-  const bg         = opts.bg         ?? '#0a0e1a';
-  const fg         = opts.fg         ?? '#e3f2fd';
-  const borderFg   = opts.borderFg   ?? 'bright-cyan';
-  const durationMs = opts.durationMs ?? 2500;
-
-  const width = Math.max(28, message.length + 6);
-  const modal = blessed.box({
-    parent: screen,
-    top: 'center',
-    left: 'center',
-    width,
-    height: 3,
-    border: { type: 'line' },
-    tags: true,
-    content: `{center}${message}{/center}`,
-    style: {
-      fg,
-      bg,
-      border: { fg: borderFg },
-    },
-  });
-  screen.render();
-
-  setTimeout(() => {
-    destroyList(modal, screen);
-  }, durationMs);
-}
+/**
+ * AgentVibes TUI — Shared Widget: Notice Toast
+ *
+ * Displays a short auto-dismissing notice modal centred on screen.
+ * Usable from any tab; no settings-specific state required.
+ */
+
+import { destroyList } from './destroy-list.js';
+
+const IS_TEST = process.env.AGENTVIBES_TEST_MODE === 'true';
+let blessed;
+if (!IS_TEST) {
+  const { default: b } = await import('blessed');
+  blessed = b;
+}
+
+/**
+ * Show a temporary notice that auto-dismisses after 2.5 seconds.
+ *
+ * @param {object} screen  - blessed screen instance
+ * @param {string} message - text to display
+ * @param {object} [opts]
+ * @param {string} [opts.bg='#0a0e1a']       - background colour
+ * @param {string} [opts.fg='#e3f2fd']       - foreground colour
+ * @param {string} [opts.borderFg='bright-cyan'] - border colour
+ * @param {number} [opts.durationMs=2500]    - auto-dismiss delay in ms
+ */
+export function showNotice(screen, message, opts = {}) {
+  const bg         = opts.bg         ?? '#0a0e1a';
+  const fg         = opts.fg         ?? '#e3f2fd';
+  const borderFg   = opts.borderFg   ?? 'bright-cyan';
+  const durationMs = opts.durationMs ?? 2500;
+
+  const width = Math.max(28, message.length + 6);
+  const modal = blessed.box({
+    parent: screen,
+    top: 'center',
+    left: 'center',
+    width,
+    height: 3,
+    border: { type: 'line' },
+    tags: true,
+    content: `{center}${message}{/center}`,
+    style: {
+      fg,
+      bg,
+      border: { fg: borderFg },
+    },
+  });
+  screen.render();
+
+  setTimeout(() => {
+    destroyList(modal, screen);
+  }, durationMs);
+}

package/src/console/widgets/personality-picker.js

@@ -132,7 +132,7 @@ export function openPersonalityPicker(screen, currentPersonality, onSelect, onCl
       const _cwdScript = path.join(process.cwd(), '.claude', 'hooks-windows', 'play-tts.ps1');
       const _homeScript = path.join(os.homedir(), '.claude', 'hooks-windows', 'play-tts.ps1');
       const ttsScript = fs.existsSync(_cwdScript) ? _cwdScript : _homeScript;
-      _pickerTtsProc = spawn('powershell', ['-NoProfile', '-ExecutionPolicy', 'Bypass', '-File', ttsScript, phrase], {
+      _pickerTtsProc = spawn('powershell', ['-NoProfile', '-ExecutionPolicy', 'Bypass', '-File', ttsScript, phrase], { // NOSONAR
         stdio: 'ignore',
         env: _env,
       });
@@ -141,7 +141,7 @@ export function openPersonalityPicker(screen, currentPersonality, onSelect, onCl
       const remoteLlm = detectRemoteLlm();
       const ttsArgs = [ttsScript, phrase];
       if (remoteLlm) ttsArgs.push('--llm', remoteLlm);
-      _pickerTtsProc = spawn('bash', ttsArgs, {
+      _pickerTtsProc = spawn('bash', ttsArgs, { // NOSONAR
         stdio: 'ignore',
         detached: true,
         env: _env,

package/src/console/widgets/reverb-picker.js

@@ -78,7 +78,7 @@ export function openReverbPicker(screen, currentPreset, onSelect, onClose, opts
       const _isWin = process.platform === 'win32' && !process.env.WSL_DISTRO_NAME;
       if (!_isWin) {
         const effectsScript = path.join(process.cwd(), '.claude', 'hooks', 'effects-manager.sh');
-        spawnSync('bash', [effectsScript, 'set-reverb', selected.value, 'default'], {
+        spawnSync('bash', [effectsScript, 'set-reverb', selected.value, 'default'], { // NOSONAR
           stdio: 'ignore',
           timeout: 5000,
           env: { ...process.env },