@goodandready/dsh-tts 0.4.32 → 0.4.33

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## 0.4.33 (2026-10-11)
+
+- **SSE Stream Watchdog**: Added idle timeout watchdog (30s) in Web player (`lib/client-src/20-player.js`) to automatically trigger reconnect on stale/frozen EventSource connection (#206).
+- **Engine Manager Cleanup**: Added explicit `removeEventListener('abort', ...)` in `execute` finally block for model downloads in `lib/engines/manager.js` (#206).
+- **Updater Testing**: Added behavioral test capturing `child_process.spawn` arguments during `/api/dsh-tts/update` execution verifying no bypass of minimum release age (#206).
+- **Client Bundle**: Rebuilt `lib/client.js` from canonical client-src fragments with idle watchdog.
+
 ## 0.4.32
 
 ### Fixed

package/lib/client.js

@@ -527,17 +527,36 @@ window.__ModuleLoader__.load({
       return () => window.removeEventListener('dsh-voice:speaking', onSpeaking)
     }
 
-    // Unified SSE Realtime Stream Connection (#158)
+    // Unified SSE Realtime Stream Connection (#158, #206)
     let activeSse = null
     let activeSseAttempt = 0
     let activeSseTimer = null
+    let activeSseWatchdog = null
     let sseDisposed = false
+    const SSE_IDLE_TIMEOUT_MS = 30000
 
     function startSseStream() {
       if (typeof window === 'undefined' || typeof window.EventSource === 'undefined') return () => {}
       sseDisposed = false
 
+      const resetWatchdog = () => {
+        if (activeSseWatchdog) clearTimeout(activeSseWatchdog)
+        if (sseDisposed) return
+        activeSseWatchdog = setTimeout(() => {
+          if (activeSse && !sseDisposed) {
+            try {
+              if (typeof activeSse.onerror === 'function') {
+                activeSse.onerror(new Event('error'))
+              } else {
+                activeSse.close()
+              }
+            } catch (_err) { /* error trigger ignore */ }
+          }
+        }, SSE_IDLE_TIMEOUT_MS)
+      }
+
       const onItem = (e) => {
+        resetWatchdog()
         try {
           const item = JSON.parse(e.data)
           if (!item || !item.id) return
@@ -556,6 +575,7 @@ window.__ModuleLoader__.load({
       }
 
       const onChime = (e) => {
+        resetWatchdog()
         try {
           const item = JSON.parse(e.data)
           if (!item) return
@@ -567,6 +587,7 @@ window.__ModuleLoader__.load({
 
       const connect = () => {
         if (sseDisposed) return
+        if (activeSseWatchdog) { clearTimeout(activeSseWatchdog); activeSseWatchdog = null }
         if (activeSse) {
           try { activeSse.close() } catch (_err) { /* active sse already closed */ }
           activeSse = null
@@ -574,10 +595,15 @@ window.__ModuleLoader__.load({
         try {
           const es = new EventSource('/dsh-tts/stream')
           activeSse = es
-          es.onopen = () => { activeSseAttempt = 0 }
+          es.onopen = () => {
+            activeSseAttempt = 0
+            resetWatchdog()
+          }
+          es.onmessage = () => { resetWatchdog() }
           es.addEventListener('utterance', onItem)
           es.addEventListener('chime', onChime)
           es.onerror = () => {
+            if (activeSseWatchdog) { clearTimeout(activeSseWatchdog); activeSseWatchdog = null }
             if (activeSse === es) {
               try { es.close() } catch (_err) { /* eventsource already closed */ }
               activeSse = null
@@ -588,6 +614,7 @@ window.__ModuleLoader__.load({
             if (activeSseTimer) clearTimeout(activeSseTimer)
             activeSseTimer = setTimeout(connect, delay)
           }
+          resetWatchdog()
         } catch { /* EventSource failed */ }
       }
 
@@ -595,7 +622,8 @@ window.__ModuleLoader__.load({
 
       return () => {
         sseDisposed = true
-        if (activeSseTimer) clearTimeout(activeSseTimer)
+        if (activeSseTimer) { clearTimeout(activeSseTimer); activeSseTimer = null }
+        if (activeSseWatchdog) { clearTimeout(activeSseWatchdog); activeSseWatchdog = null }
         if (activeSse) {
           try { activeSse.close() } catch (_err) { /* active sse already closed on dispose */ }
           activeSse = null

package/lib/engines/manager.js

@@ -197,9 +197,13 @@ export function createModelManager({ root = process.env.DSH_HOME || path.join(os
     const dir = getModelPath(engineId)
 
     const controller = new AbortController()
+    let onAbortListener = null
     if (options.signal) {
       if (options.signal.aborted) controller.abort()
-      else options.signal.addEventListener('abort', () => controller.abort(), { once: true })
+      else {
+        onAbortListener = () => controller.abort()
+        options.signal.addEventListener('abort', onAbortListener, { once: true })
+      }
     }
 
     const downloadState = {
@@ -311,6 +315,9 @@ export function createModelManager({ root = process.env.DSH_HOME || path.join(os
         lastErrors.set(engineId, { message: sanitized, failedAt: Date.now() })
         throw err
       } finally {
+        if (options.signal && onAbortListener) {
+          try { options.signal.removeEventListener('abort', onAbortListener) } catch (_err) { /* listener already removed */ }
+        }
         if (activeDownloads.get(engineId) === downloadState) {
           activeDownloads.delete(engineId)
         }

package/lib/updater.js

@@ -125,7 +125,7 @@ function runtime() {
     : validProfileName(basename(profileDir))
       ? basename(profileDir)
       : 'web'
-  const cliEntry = findDshCliEntry()
+  const cliEntry = process.env.DSH_CLI_ENTRY ?? findDshCliEntry()
   return cliEntry === undefined ? { profileName, profileDir } : { profileName, profileDir, cliEntry }
 }
 
@@ -170,7 +170,10 @@ export function isNewerVersion(currentValue, candidateValue) {
   return comparePrerelease(candidate.prerelease, current.prerelease) > 0
 }
 
-async function latestVersion(packageName, registry) {
+async function latestVersion(packageName, registry, options) {
+  if (typeof options?.latestVersionImpl === 'function') {
+    return await options.latestVersionImpl(packageName, registry)
+  }
   if (
     latestCache?.packageName === packageName &&
     latestCache.registry === registry &&
@@ -202,7 +205,7 @@ async function currentVersion(manifestUrl) {
 
 export async function status(options, target) {
   const current = await currentVersion(options.manifestUrl)
-  const latest = await latestVersion(options.packageName, options.registry ?? 'https://registry.npmjs.org')
+  const latest = await latestVersion(options.packageName, options.registry ?? 'https://registry.npmjs.org', options)
   return {
     packageName: options.packageName,
     currentVersion: current,
@@ -217,7 +220,25 @@ export async function status(options, target) {
 async function installExact(target, packageSpec, options) {
   if (target.cliEntry === undefined) throw new Error('Automatic update is unavailable in this runtime.')
   await new Promise((resolvePromise, reject) => {
-    const child = spawn(
+    const spawnCall = options?.spawnImpl || spawn
+    const child = spawnCall === spawn ? spawn(
+      process.execPath,
+      [
+        target.cliEntry,
+        'plugin',
+        '--profile',
+        target.profileName,
+        'add',
+        packageSpec,
+        `--registry=${options.registry ?? 'https://registry.npmjs.org/'}`,
+      ],
+      {
+        cwd: target.profileDir,
+        windowsHide: true,
+        stdio: ['ignore', 'pipe', 'pipe'],
+        env: { ...process.env, NO_COLOR: '1' },
+      }
+    ) : spawnCall(
       process.execPath,
       [
         target.cliEntry,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goodandready/dsh-tts",
-  "version": "0.4.32",
+  "version": "0.4.33",
   "description": "Text-to-speech for DeepSeek Harness: speak agent replies in the Web UI with a provider fallback chain (OpenAI, ElevenLabs, Google, Azure, Groq, Deepgram, OpenRouter, Edge, Piper, eSpeak).",
   "license": "MIT",
   "type": "module",