agentvibes 5.3.0 → 5.4.0

package/src/installer.js

@@ -132,6 +132,36 @@ function isNativeWindows() {
   return process.platform === 'win32' && !process.env.WSL_DISTRO_NAME;
 }
 
+/**
+ * Wrap an ora spinner with safe fallbacks for all methods.
+ * Ensures compatibility across platforms (Windows, WSL, macOS, Linux) and ora versions.
+ * @param {Object} spinner - An ora spinner instance (or any spinner-like object)
+ * @returns {Object} A spinner proxy with guaranteed methods
+ */
+function createRobustSpinner(spinner) {
+  const safe = (method, fallback) => (...args) => {
+    if (typeof spinner[method] === 'function') {
+      try { spinner[method](...args); } catch { fallback?.(...args); }
+    } else {
+      fallback?.(...args);
+    }
+    return proxy;
+  };
+  const proxy = {
+    start:          safe('start'),
+    stop:           safe('stop'),
+    succeed:        safe('succeed', (t) => { if (t) process.stdout.write(`✓ ${t}\n`); }),
+    fail:           safe('fail',    (t) => { if (t) process.stderr.write(`✗ ${t}\n`); }),
+    warn:           safe('warn',    (t) => { if (t) process.stdout.write(`⚠ ${t}\n`); }),
+    info:           safe('info',    (t) => { if (t) process.stdout.write(`ℹ ${t}\n`); }),
+    stopAndPersist: safe('stopAndPersist'),
+    get text()      { return spinner.text ?? ''; },
+    set text(t)     { try { spinner.text = t; } catch { /* non-TTY */ } },
+    get isSpinning(){ return spinner.isSpinning ?? false; },
+  };
+  return proxy;
+}
+
 /**
  * Get the Piper provider name (always 'piper' on all platforms)
  * @returns {string} The piper provider identifier
@@ -4215,7 +4245,7 @@ async function checkAndInstallPiperWindows(targetDir, options) {
   }
   const piperDir = path.join(localAppData, 'Programs', 'Piper');
   const piperExe = path.join(piperDir, 'piper.exe');
-  const spinner = ora();
+  const spinner = createRobustSpinner(ora());
 
   if (fsSync.existsSync(piperExe)) {
     console.log(chalk.green('✓ Piper TTS is already installed at ' + piperDir + '\n'));
@@ -4844,7 +4874,14 @@ async function updatePersonalityFiles(targetDir, srcPersonalitiesDir) {
  * @returns {Object} Mock spinner object
  */
 function createSilentSpinner() {
-  const s = { start: () => s, succeed: () => s, info: () => s, fail: () => s, warn: () => s, stop: () => s };
+  const s = {
+    start: () => s, succeed: () => s, info: () => s,
+    fail: () => s, warn: () => s, stop: () => s,
+    stopAndPersist: () => s,
+    get text() { return ''; },
+    set text(_) {},
+    get isSpinning() { return false; },
+  };
   return s;
 }
 
@@ -5025,7 +5062,7 @@ function displayUpdateSummary(results) {
  * @param {Object} options - Update options
  */
 async function updateAgentVibes(targetDir, options) {
-  const spinner = ora('Updating AgentVibes...').start();
+  const spinner = createRobustSpinner(ora('Updating AgentVibes...').start());
 
   try {
     // Perform all update operations
@@ -5145,7 +5182,7 @@ async function install(options = {}) {
   const silentSpinner = createSilentSpinner();
 
   console.log('');
-  const spinner = ora('Installing AgentVibes...').start();
+  const spinner = createRobustSpinner(ora('Installing AgentVibes...').start());
 
   try {
     // Create .claude directory structure
@@ -5599,7 +5636,7 @@ program
       console.log(chalk.gray('✓ Auto-confirmed (--yes flag)\n'));
     }
 
-    const spinner = ora('Uninstalling AgentVibes...').start();
+    const spinner = createRobustSpinner(ora('Uninstalling AgentVibes...').start());
 
     try {
       let removedCount = 0;