tabby-agentdeck 1.2.3 → 1.2.4

package/hooks/agentdeck-mailbox.mjs

@@ -11,7 +11,12 @@ const { mailboxRoot } = await import('./agentdeck-runtime.cjs').then(m => m.defa
 const root = mailboxRoot()
 const tab = process.env.AGENTDECK_TAB || ''
 let identity
-const mode = process.argv[2]
+const mode = process.argv[2]
+// Bound the whole hook, including multiple requests and trickling responses.
+if (mode === '--hook') {
+    const deadline = Number(process.env.AGENTDECK_HOOK_DEADLINE) || Date.now() + 1000
+    setTimeout(() => process.exit(0), Math.max(1, Math.min(1000, deadline - Date.now()))).unref()
+}
 function credentials () {
     if (identity) return identity
     if (!/^[a-zA-Z0-9_-]+$/.test(tab)) throw new Error('Start this MCP server inside an AgentDeck session')

package/hooks/agentdeck-notify.ps1

@@ -446,9 +446,39 @@ Write-HookDiag 'tcp-sent'
 Write-AgentDeckHookTrace 'tcp_end'
 if ($mailContext) {
     Write-AgentDeckHookTrace 'mailbox_begin'
-    & node (Join-Path $PSScriptRoot 'agentdeck-mailbox.mjs') --hook $targetId ([string]$hook.hook_event_name) 2>$null
-    Write-HookDiag "mailbox-done exit=$LASTEXITCODE"
-    Write-AgentDeckHookTrace 'mailbox_end' @{ exit_code = $LASTEXITCODE }
+    $mailProcess = New-Object System.Diagnostics.Process
+    try {
+        $mailProcess.StartInfo.FileName = (Get-Command node -CommandType Application -ErrorAction Stop | Select-Object -First 1).Source
+        $mailProcess.StartInfo.Arguments = '"' + (Join-Path $PSScriptRoot 'agentdeck-mailbox.mjs') + '" --hook "' + $targetId + '" "' + [string]$hook.hook_event_name + '"'
+        $mailProcess.StartInfo.UseShellExecute = $false
+        $mailProcess.StartInfo.CreateNoWindow = $true
+        $mailProcess.StartInfo.RedirectStandardOutput = $true
+        $mailProcess.StartInfo.RedirectStandardError = $true
+        $mailProcess.StartInfo.StandardOutputEncoding = New-Object System.Text.UTF8Encoding($false)
+        $mailProcess.StartInfo.EnvironmentVariables['AGENTDECK_HOOK_DEADLINE'] = [string]([DateTimeOffset]::UtcNow.ToUnixTimeMilliseconds() + 1000)
+        $mailBudget = [System.Diagnostics.Stopwatch]::StartNew()
+        [void]$mailProcess.Start()
+        $mailOutput = $mailProcess.StandardOutput.ReadToEndAsync()
+        $mailError = $mailProcess.StandardError.ReadToEndAsync()
+        if (-not $mailProcess.WaitForExit([Math]::Max(0, 1500 - [int]$mailBudget.ElapsedMilliseconds))) {
+            try { $mailProcess.Kill() } catch { }
+            Write-HookDiag 'mailbox-timeout budget=1500ms'
+            Write-AgentDeckHookTrace 'mailbox_timeout' @{ budget_ms = 1500 }
+        } else {
+            # Process exit and redirected pipe completion are separate notifications.
+            # Drain the pipe within the remaining budget instead of dropping a valid reply.
+            $mailDrained = $mailOutput.Wait([Math]::Max(0, 1500 - [int]$mailBudget.ElapsedMilliseconds))
+            if ($mailProcess.ExitCode -eq 0 -and $mailDrained) {
+                $mailText = $mailOutput.Result
+                if ($mailText) { [Console]::Out.Write($mailText) }
+            }
+            Write-AgentDeckHookTrace 'mailbox_end' @{ exit_code = $mailProcess.ExitCode }
+        }
+    } catch {
+        Write-AgentDeckHookTrace 'mailbox_error' @{} $_
+    } finally {
+        $mailProcess.Dispose()
+    }
 }
 Write-HookDiag 'end'
 exit 0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tabby-agentdeck",
-  "version": "1.2.3",
+  "version": "1.2.4",
   "description": "Run AI coding agents side by side in Tabby with fixed tab numbers, cross-session messaging, live status, file previews, agent-friendly hotkeys and image paste",
   "keywords": [
     "tabby-plugin",
@@ -18,8 +18,8 @@
     "build:release": "node tools/sync-version.js && webpack --mode production --env release",
     "prepack": "npm run build:release && node tools/check-release.js",
     "watch": "webpack --progress --color --watch",
-    "test": "npm run test:existing && npm run test:codex && npm run test:accounts && node test/sidebarI18n.cjs && node test/accountSession.cjs && npm run test:sessions && npm run test:storage && npm run test:performance && npm run test:hook-trace",
-    "test:hook-trace": "node test/hookTrace.cjs",
+    "test": "npm run test:existing && npm run test:codex && npm run test:accounts && node test/sidebarI18n.cjs && node test/accountSession.cjs && node test/accountRefreshPolicy.cjs && node test/accountRefreshClient.cjs && npm run test:sessions && npm run test:storage && npm run test:performance && npm run test:hook-trace",
+    "test:hook-trace": "node test/hookTrace.cjs && node test/mailboxDeadline.cjs",
     "test:performance": "node test/performance.cjs",
     "test:storage": "tsc src/storagePaths.ts src/accounts.ts src/storageSettings.ts src/runtimeMigration.ts --outDir .tmp --module commonjs --target es2019 --skipLibCheck --esModuleInterop && node test/storagePaths.cjs && node test/storageSettings.cjs && node test/runtimePaths.cjs",
     "test:sessions": "tsc src/sessionSearch.ts src/sessionMailbox.ts src/sessionSlots.ts --outDir .tmp --module commonjs --target es2019 --skipLibCheck --esModuleInterop && node test/sessionSearch.cjs && node test/sessionCommunication.cjs && node test/mailboxBridge.cjs",