vibora 9.7.2 → 9.8.0

package/bin/vibora.js

@@ -45126,7 +45126,7 @@ function installUv() {
 var package_default = {
   name: "vibora",
   private: true,
-  version: "9.7.2",
+  version: "9.8.0",
   description: "Harness Attention. Orchestrate Agents. Ship.",
   license: "PolyForm-Perimeter-1.0.0",
   type: "module",
@@ -45659,7 +45659,7 @@ import { join as join4 } from "path";
 // plugins/vibora-opencode/index.ts
 var vibora_opencode_default = `import type { Plugin } from "@opencode-ai/plugin"
 import { appendFileSync } from "node:fs"
-import { execFile } from "node:child_process"
+import { spawn } from "node:child_process"
 import { tmpdir } from "node:os"
 import { join } from "node:path"
 
@@ -45681,20 +45681,72 @@ const log = (msg: string) => {
 }
 
 /**
- * Execute vibora command using execFile with shell option for proper PATH resolution.
- * Using execFile with explicit args array prevents shell injection while shell:true
+ * Execute vibora command using spawn with shell option for proper PATH resolution.
+ * Using spawn with explicit args array prevents shell injection while shell:true
  * ensures PATH is properly resolved (for NVM, fnm, etc. managed node installations).
+ * Includes 10 second timeout protection to prevent hanging.
  */
 async function runViboraCommand(args: string[]): Promise<{ exitCode: number; stdout: string; stderr: string }> {
   return new Promise((resolve) => {
-    execFile(VIBORA_CMD, args, { shell: true }, (error, stdout, stderr) => {
-      if (error) {
-        const execError = error as NodeJS.ErrnoException
-        resolve({ exitCode: execError.code ? 1 : 1, stdout: stdout || '', stderr: stderr || execError.message || '' })
-      } else {
-        resolve({ exitCode: 0, stdout: stdout || '', stderr: stderr || '' })
+    let stdout = ''
+    let stderr = ''
+    let resolved = false
+    let processExited = false
+    let killTimeoutId: ReturnType<typeof setTimeout> | null = null
+
+    const child = spawn(VIBORA_CMD, args, { shell: true })
+
+    const cleanup = () => {
+      processExited = true
+      if (killTimeoutId) {
+        clearTimeout(killTimeoutId)
+        killTimeoutId = null
+      }
+    }
+
+    child.stdout?.on('data', (data) => {
+      stdout += data.toString()
+    })
+
+    child.stderr?.on('data', (data) => {
+      stderr += data.toString()
+    })
+
+    child.on('close', (code) => {
+      cleanup()
+      if (!resolved) {
+        resolved = true
+        resolve({ exitCode: code || 0, stdout, stderr })
       }
     })
+
+    child.on('error', (err) => {
+      cleanup()
+      if (!resolved) {
+        resolved = true
+        resolve({ exitCode: 1, stdout, stderr: err.message || '' })
+      }
+    })
+
+    // Add timeout protection to prevent hanging
+    const timeoutId = setTimeout(() => {
+      if (!resolved) {
+        resolved = true
+        log(\`Command timeout: \${VIBORA_CMD} \${args.join(' ')}\`)
+        child.kill('SIGTERM')
+        // Schedule SIGKILL if process doesn't exit after SIGTERM
+        killTimeoutId = setTimeout(() => {
+          if (!processExited) {
+            log(\`Process didn't exit after SIGTERM, sending SIGKILL\`)
+            child.kill('SIGKILL')
+          }
+        }, 2000)
+        resolve({ exitCode: -1, stdout, stderr: \`Command timed out after \${VIBORA_COMMAND_TIMEOUT_MS}ms\` })
+      }
+    }, VIBORA_COMMAND_TIMEOUT_MS)
+
+    // Clear timeout if command completes
+    child.on('exit', () => clearTimeout(timeoutId))
   })
 }
 
@@ -45706,9 +45758,12 @@ let lastStatus: "in-progress" | "review" | "" = ""
 
 const VIBORA_CMD = "vibora"
 const IDLE_CONFIRMATION_DELAY_MS = 1500
+const VIBORA_COMMAND_TIMEOUT_MS = 10000
+const STATUS_CHANGE_DEBOUNCE_MS = 500
 
 let deferredContextCheck: Promise<boolean> | null = null
 let isViboraContext: boolean | null = null
+let pendingStatusCommand: Promise<{ exitCode: number; stdout: string; stderr: string }> | null = null
 
 export const ViboraPlugin: Plugin = async ({ $, directory }) => {
   log("Plugin initializing...")
@@ -45760,16 +45815,28 @@ export const ViboraPlugin: Plugin = async ({ $, directory }) => {
     if (status === lastStatus) return
 
     cancelPendingIdle()
+
+    if (pendingStatusCommand) {
+      log(\`Status change already in progress, will retry after \${STATUS_CHANGE_DEBOUNCE_MS}ms\`)
+      setTimeout(() => setStatus(status), STATUS_CHANGE_DEBOUNCE_MS)
+      return
+    }
+
     lastStatus = status
+
     ;(async () => {
       try {
         log(\`Setting status: \${status}\`)
-        const res = await runViboraCommand(['current-task', status, '--path', directory])
+        pendingStatusCommand = runViboraCommand(['current-task', status, '--path', directory])
+        const res = await pendingStatusCommand
+        pendingStatusCommand = null
+
         if (res.exitCode !== 0) {
           log(\`Status update failed: exitCode=\${res.exitCode}, stderr=\${res.stderr}\`)
         }
       } catch (e) {
         log(\`Status update error: \${e}\`)
+        pendingStatusCommand = null
       }
     })()
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vibora",
-  "version": "9.7.2",
+  "version": "9.8.0",
   "description": "Harness Attention. Orchestrate Agents. Ship.",
   "license": "PolyForm-Perimeter-1.0.0",
   "repository": {

package/server/index.js

@@ -4976,6 +4976,13 @@ function updateNotificationSettings(updates) {
   const changes = {};
   if (updates.enabled !== undefined && updates.enabled !== current.enabled) {
     changes.enabled = { from: current.enabled, to: updates.enabled };
+    if (updates.enabled === false) {
+      log2.settings.warn("Notifications being DISABLED", {
+        from: current.enabled,
+        to: updates.enabled,
+        stack: new Error().stack
+      });
+    }
   }
   if (updates.toast?.enabled !== undefined && updates.toast.enabled !== current.toast.enabled) {
     changes["toast.enabled"] = { from: current.toast.enabled, to: updates.toast.enabled };
@@ -5127,6 +5134,7 @@ function ensureLatestSettings() {
       parsed = JSON.parse(fs2.readFileSync(settingsPath, "utf-8"));
     } catch {}
   }
+  const originalNotificationsEnabled = parsed.notifications?.enabled;
   migrateSettings(parsed);
   const merged = deepMergeWithDefaults(parsed, DEFAULT_SETTINGS);
   if (!merged.notifications || typeof merged.notifications !== "object") {
@@ -5134,6 +5142,14 @@ function ensureLatestSettings() {
   } else {
     merged.notifications = deepMergeWithDefaults(merged.notifications, DEFAULT_NOTIFICATION_SETTINGS);
   }
+  const mergedNotificationsEnabled = merged.notifications?.enabled;
+  if (originalNotificationsEnabled !== mergedNotificationsEnabled) {
+    log2.settings.warn("Notification enabled state changed during settings normalization", {
+      from: originalNotificationsEnabled,
+      to: mergedNotificationsEnabled,
+      reason: originalNotificationsEnabled === undefined ? "missing key, using default" : "value changed during merge"
+    });
+  }
   if (!merged.zai || typeof merged.zai !== "object") {
     merged.zai = { ...DEFAULT_ZAI_SETTINGS };
   } else {