@goodandready/dsh-clinebot 0.5.7 → 0.5.9

package/lib/index.js

@@ -230,6 +230,7 @@ export function apply(ctx, config) {
               const is429 = failure.status === 429 || failure.code === 429 || failure.code === 'rate_limit_exceeded'
               const isExceeded = isQuotaExceededError(`${failure.message || ''} ${failure.code || ''}`)
 
+              const isProxy = publicConfig(live()).proxyMode !== false
               if (isError) {
                 recordSessionRequest({
                   latencyMs,
@@ -239,15 +240,17 @@ export function apply(ctx, config) {
                   completionTokens,
                   model: options?.model,
                 })
-                recordUsage({
-                  model: options?.model,
-                  promptTokens,
-                  completionTokens,
-                  totalTokens: promptTokens + completionTokens,
-                  isError: true,
-                  is429,
-                  statsPath: publicConfig(live()).statsPath,
-                })
+                if (!isProxy) {
+                  recordUsage({
+                    model: options?.model,
+                    promptTokens,
+                    completionTokens,
+                    totalTokens: promptTokens + completionTokens,
+                    isError: true,
+                    is429,
+                    statsPath: publicConfig(live()).statsPath,
+                  })
+                }
                 if (is429 || isExceeded) {
                   handleStream429()
                 }
@@ -268,13 +271,15 @@ export function apply(ctx, config) {
                   completionTokens,
                   model: options?.model,
                 })
-                recordUsage({
-                  model: options?.model,
-                  promptTokens,
-                  completionTokens,
-                  totalTokens: promptTokens + completionTokens,
-                  statsPath: publicConfig(live()).statsPath,
-                })
+                if (!isProxy) {
+                  recordUsage({
+                    model: options?.model,
+                    promptTokens,
+                    completionTokens,
+                    totalTokens: promptTokens + completionTokens,
+                    statsPath: publicConfig(live()).statsPath,
+                  })
+                }
               }
             }
             yield chunk
@@ -287,13 +292,16 @@ export function apply(ctx, config) {
               completionTokens,
               model: options?.model,
             })
-            recordUsage({
-              model: options?.model,
-              promptTokens,
-              completionTokens,
-              totalTokens: promptTokens + completionTokens,
-              statsPath: publicConfig(live()).statsPath,
-            })
+            const isProxy = publicConfig(live()).proxyMode !== false
+            if (!isProxy) {
+              recordUsage({
+                model: options?.model,
+                promptTokens,
+                completionTokens,
+                totalTokens: promptTokens + completionTokens,
+                statsPath: publicConfig(live()).statsPath,
+              })
+            }
           }
         } catch (err) {
           if (isCline) {
@@ -308,15 +316,18 @@ export function apply(ctx, config) {
               completionTokens,
               model: options?.model,
             })
-            recordUsage({
-              model: options?.model,
-              promptTokens,
-              completionTokens,
-              totalTokens: promptTokens + completionTokens,
-              isError: true,
-              is429: status === 429,
-              statsPath: publicConfig(live()).statsPath,
-            })
+            const isProxy = publicConfig(live()).proxyMode !== false
+            if (!isProxy) {
+              recordUsage({
+                model: options?.model,
+                promptTokens,
+                completionTokens,
+                totalTokens: promptTokens + completionTokens,
+                isError: true,
+                is429: status === 429,
+                statsPath: publicConfig(live()).statsPath,
+              })
+            }
             if (status === 429 || isQuotaExceededError(msg)) {
               handleStream429()
             }

package/lib/models.js

@@ -422,10 +422,10 @@ export function getAllModels(dynamicModels = [], modelContextOverrides = [], cus
           description: cm.description || `Custom model ${cm.name || cm.id}`,
           contextLength: cm.contextLength || 128000,
           maxTokens: cm.maxTokens || 8192,
-          input: ['text'],
+          input: Array.isArray(cm.input) && cm.input.length ? cm.input : ['text'],
           category: cm.category || 'general',
           isCustom: true,
-          reasoningEfforts: cm.isReasoning ? ['low', 'medium', 'high', 'max'] : undefined,
+          reasoningEfforts: cm.reasoningEfforts !== undefined ? cm.reasoningEfforts : (cm.isReasoning ? ['low', 'medium', 'high', 'max'] : undefined),
           unverified: false,
         }
         baseModels.push(applyOverride(fullCustom))
@@ -437,19 +437,19 @@ export function getAllModels(dynamicModels = [], modelContextOverrides = [], cus
   return baseModels
 }
 
-function findModel(id, dynamicModels = [], modelContextOverrides = [], customModels = []) {
-  const all = getAllModels(dynamicModels, modelContextOverrides)
+export function findModel(id, dynamicModels = [], modelContextOverrides = [], customModels = []) {
+  const all = getAllModels(dynamicModels, modelContextOverrides, customModels)
   const found = all.find((m) => m.id === id)
   if (found) return found
   return CLINE_MODELS.find((m) => m.id === id) || null
 }
 
 export function isSupportedModel(id, dynamicModels = [], modelContextOverrides = [], customModels = []) {
-  return findModel(id, dynamicModels, modelContextOverrides) !== null
+  return findModel(id, dynamicModels, modelContextOverrides, customModels) !== null
 }
 
-function getDefaultModelIds(dynamicModels = [], modelContextOverrides = [], customModels = []) {
-  return getAllModels(dynamicModels, modelContextOverrides).map((m) => m.id)
+export function getDefaultModelIds(dynamicModels = [], modelContextOverrides = [], customModels = []) {
+  return getAllModels(dynamicModels, modelContextOverrides, customModels).map((m) => m.id)
 }
 
 /**
@@ -501,8 +501,8 @@ export function formatModelDescription(model) {
 /**
  * Check if a model supports vision input.
  */
-export function isVisionModel(id, dynamicModels = [], modelContextOverrides = []) {
-  const m = findModel(id, dynamicModels, modelContextOverrides)
+export function isVisionModel(id, dynamicModels = [], modelContextOverrides = [], customModels = []) {
+  const m = findModel(id, dynamicModels, modelContextOverrides, customModels)
   if (!m) return false
   return Boolean(m.input?.includes('image') || m.input?.includes('vision'))
 }
@@ -548,3 +548,29 @@ export async function loadModelsDiskCache(cachePath) {
     return null
   }
 }
+
+/**
+ * Detects whether the authoritative dynamic models catalog differs from the current catalog
+ * in model count, IDs, or metadata (e.g. contextLength, maxTokens, category, etc.).
+ */
+export function isCatalogDifferent(currentList = [], authoritativeList = []) {
+  if (!Array.isArray(currentList) || !Array.isArray(authoritativeList)) return true
+  if (currentList.length !== authoritativeList.length) return true
+  const currentMap = new Map(currentList.map((m) => [m.id, m]))
+  for (const auth of authoritativeList) {
+    const cur = currentMap.get(auth.id)
+    if (!cur) return true
+    if (
+      cur.name !== auth.name ||
+      cur.contextLength !== auth.contextLength ||
+      cur.maxTokens !== auth.maxTokens ||
+      cur.category !== auth.category ||
+      cur.description !== auth.description ||
+      JSON.stringify(cur.input || []) !== JSON.stringify(auth.input || []) ||
+      JSON.stringify(cur.reasoningEfforts || []) !== JSON.stringify(auth.reasoningEfforts || [])
+    ) {
+      return true
+    }
+  }
+  return false
+}

package/lib/provider-sync.js

@@ -1,3 +1,4 @@
+import { getStatsSummary } from './stats-storage.js'
 import os from 'node:os'
 import path from 'node:path'
 import {
@@ -8,6 +9,7 @@ import {
   loadModelsDiskCache,
   OBSOLETE_MODEL_ID_MAP,
   migrateModelEntry,
+  isCatalogDifferent,
 } from './models.js'
 import {
   probeHealth,
@@ -121,12 +123,16 @@ export async function buildStatus(ctx, cfg) {
       if (fiveHour.percentUsed >= 95) {
         quotaWarning = {
           level: 'exhausted',
+          code: 'quota.exhausted_5h',
+          params: { percentUsed: fiveHour.percentUsed },
           message: `5-hour rolling limit is almost exhausted (${fiveHour.percentUsed}%). New requests may be rejected until quota reset.`,
           resetsAt: fiveHour.resetsAt,
         }
       } else if (fiveHour.percentUsed >= 90) {
         quotaWarning = {
           level: 'warning',
+          code: 'quota.warning_5h',
+          params: { percentUsed: fiveHour.percentUsed },
           message: `Low quota warning: ${fiveHour.percentUsed}% of 5-hour limit consumed (<10% remaining).`,
           resetsAt: fiveHour.resetsAt,
         }
@@ -134,17 +140,23 @@ export async function buildStatus(ctx, cfg) {
     }
 
     if (!quotaWarning && weekly && typeof weekly.percentUsed === 'number' && weekly.percentUsed >= 90) {
+      const remainingPercent = weekly.remainingPercent ?? (100 - weekly.percentUsed)
       quotaWarning = {
         level: weekly.percentUsed >= 99 ? 'exhausted' : 'warning',
-        message: `Low weekly quota warning: ${weekly.percentUsed}% consumed (${weekly.remainingPercent ?? (100 - weekly.percentUsed)}% remaining).`,
+        code: 'quota.warning_weekly',
+        params: { percentUsed: weekly.percentUsed, remainingPercent },
+        message: `Low weekly quota warning: ${weekly.percentUsed}% consumed (${remainingPercent}% remaining).`,
         resetsAt: weekly.resetsAt,
       }
     }
 
     if (!quotaWarning && monthly && typeof monthly.percentUsed === 'number' && monthly.percentUsed >= 90) {
+      const remainingPercent = monthly.remainingPercent ?? (100 - monthly.percentUsed)
       quotaWarning = {
         level: monthly.percentUsed >= 99 ? 'exhausted' : 'warning',
-        message: `Low monthly quota warning: ${monthly.percentUsed}% consumed (${monthly.remainingPercent ?? (100 - monthly.percentUsed)}% remaining).`,
+        code: 'quota.warning_monthly',
+        params: { percentUsed: monthly.percentUsed, remainingPercent },
+        message: `Low monthly quota warning: ${monthly.percentUsed}% consumed (${remainingPercent}% remaining).`,
         resetsAt: monthly.resetsAt,
       }
     }
@@ -176,6 +188,7 @@ export async function buildStatus(ctx, cfg) {
     usage: publicUsage(usage),
     quotaWarning,
     sessionStats: { ...sessionStats },
+    statsSummary: getStatsSummary(),
     lastRotation: getLastRotation(),
     isRegistered,
     availableModels: allModels,
@@ -213,9 +226,10 @@ export async function upsertPiAiProvider(ctx, cfg, activeModelIds) {
   const providerObj = buildPiAiProvider({
     baseUrl: effectiveBaseUrl,
     apiKeyEnv: isProxy ? LOCAL_PROXY_KEY_ENV : (activeAcc?.apiKeyEnv || pub.apiKeyEnv),
-    apiKey: localToken,
     models: modelsToRegister.length ? modelsToRegister : allModels,
-    customModels: [...(pub.dynamicModels || []), ...(pub.customModels || [])],
+    dynamicModels: pub.dynamicModels,
+    modelContextOverrides: pub.modelContextOverrides,
+    customModels: pub.customModels,
     displayName: PROVIDER_DISPLAY_NAME,
   })
 
@@ -301,12 +315,11 @@ export async function autoDiscoverPlanModels(ctx, { live, getSettingsApi, syncPr
       bypassCache: true,
     })
 
-    if (usageData?.ok && Array.isArray(usageData.dynamicModels) && usageData.dynamicModels.length > 0) {
+    if (usageData?.ok && Array.isArray(usageData.dynamicModels)) {
       const existingDynamic = pub.dynamicModels || []
-      const existingIds = new Set(existingDynamic.map((m) => m.id))
-      const hasNew = usageData.dynamicModels.some((m) => !existingIds.has(m.id))
+      const isDifferent = isCatalogDifferent(existingDynamic, usageData.dynamicModels)
 
-      if (hasNew && settingsApi?.replace) {
+      if (isDifferent && settingsApi?.replace) {
         const now = Date.now()
         const next = plainConfig(Config({
           ...plainConfig(live()),

package/lib/routes/models.js

@@ -33,7 +33,7 @@ export function registerModelsRoutes(ctx, { live, getSettingsApi, syncProviderSt
         }
 
         const dynamicModels = Array.isArray(usageData.dynamicModels) ? usageData.dynamicModels : []
-        const allModels = getAllModels(dynamicModels, pub.modelContextOverrides)
+        const allModels = getAllModels(dynamicModels, pub.modelContextOverrides, pub.customModels)
         const settingsApi = getSettingsApi()
         if (!settingsApi || typeof settingsApi.replace !== 'function') {
           return writeJson(res, 503, { ok: false, error: 'Settings service unavailable or read-only' })
@@ -87,7 +87,7 @@ export function registerModelsRoutes(ctx, { live, getSettingsApi, syncProviderSt
         if (Array.isArray(body.disabledModels)) {
           patch.disabledModels = body.disabledModels
         } else if (Array.isArray(body.enabledModels)) {
-          const allModels = getAllModels(pub.dynamicModels, pub.modelContextOverrides)
+          const allModels = getAllModels(pub.dynamicModels, pub.modelContextOverrides, pub.customModels)
           const enabledSet = new Set(body.enabledModels)
           patch.disabledModels = allModels.map((m) => m.id).filter((id) => !enabledSet.has(id))
         }
@@ -120,7 +120,7 @@ export function registerModelsRoutes(ctx, { live, getSettingsApi, syncProviderSt
         }
 
         const pub = publicConfig(live())
-        const allModels = getAllModels(pub.dynamicModels, [])
+        const allModels = getAllModels(pub.dynamicModels, [], pub.customModels)
         let currentOverrides = Array.isArray(pub.modelContextOverrides) ? [...pub.modelContextOverrides] : []
 
         if (body.mode === 'all-original') {
@@ -171,7 +171,7 @@ export function registerModelsRoutes(ctx, { live, getSettingsApi, syncProviderSt
         await settingsApi.replace(next)
         await syncProviderState(next)
 
-        const updatedModels = getAllModels(pub.dynamicModels, currentOverrides)
+        const updatedModels = getAllModels(pub.dynamicModels, currentOverrides, pub.customModels)
         return writeJson(res, 200, {
           ok: true,
           modelContextOverrides: next.modelContextOverrides,

package/lib/routes/proxy.js

@@ -154,6 +154,8 @@ export function registerProxyRoutes(ctx, { live, getSettingsApi, syncProviderSta
       let attempt = 0
       let lastError = null
       let hadTimeout = false
+      let lastTargetAccount = null
+      let lastRetryAfter = null
 
       while (attempt < maxRetries) {
         attempt += 1
@@ -165,6 +167,7 @@ export function registerProxyRoutes(ctx, { live, getSettingsApi, syncProviderSta
 
         const routeResult = resolveSessionAccount(sessionId, pool, pub.activeAccount)
         const targetAccount = routeResult.account
+        lastTargetAccount = targetAccount
 
         if (!targetAccount || !targetAccount.value) {
           cleanupClientListeners()
@@ -230,13 +233,23 @@ export function registerProxyRoutes(ctx, { live, getSettingsApi, syncProviderSta
           const accountId = targetAccount.apiKeyEnv || targetAccount.id || targetAccount.label
           markAccountCooldown(accountId, 60000, null, 'upstream_429')
           if (targetAccount.id && targetAccount.id !== accountId) markAccountCooldown(targetAccount.id, 60000, null, 'upstream_429')
-          recordUsage({
-            model: body.model,
-            accountId,
-            is429: true,
-            isError: true,
-            statsPath: pub.statsPath
-          })
+          lastError = new Error(`Upstream rejected with HTTP ${upstreamRes.status}`)
+          lastError.status = upstreamRes.status
+          const retryAfter = upstreamRes.headers?.get?.('retry-after')
+          if (retryAfter) {
+            lastRetryAfter = retryAfter
+          }
+          if (upstreamRes.body) {
+            try {
+              if (typeof upstreamRes.body.cancel === 'function') {
+                upstreamRes.body.cancel().catch(() => {})
+              } else if (typeof upstreamRes.body.destroy === 'function') {
+                upstreamRes.body.destroy()
+              }
+            } catch {
+              /* ignore cancel error */
+            }
+          }
           ctx?.logger?.warn?.(`[dsh-clinebot:proxy] Account "${accountId}" hit 429, attempting failover (attempt ${attempt}/${maxRetries})`)
           continue
         }
@@ -419,21 +432,47 @@ export function registerProxyRoutes(ctx, { live, getSettingsApi, syncProviderSta
 
       cleanupClientListeners()
       // Exhausted all retries
+      const is429Exhausted = !hadTimeout && (lastError?.status === 429 || lastError?.status === 402)
+      recordUsage({
+        model: body.model,
+        accountId: lastTargetAccount?.id || lastTargetAccount?.label || 'default',
+        isError: true,
+        is429: is429Exhausted,
+        statsPath: pub.statsPath
+      })
       if (hadTimeout) {
         return writeJson(res, 504, {
           error: {
             message: 'Gateway Timeout: upstream provider timed out across all available accounts',
             type: 'gateway_timeout',
-            code: 'upstream_timeout'
+            code: 'upstream_timeout',
+            details: lastError?.message || null
+          }
+        })
+      }
+
+      if (is429Exhausted) {
+        if (lastRetryAfter) {
+          res.setHeader('Retry-After', lastRetryAfter)
+        }
+        return writeJson(res, 429, {
+          error: {
+            message: 'All configured accounts in pool exceeded rate limits or quota',
+            type: 'rate_limit_exceeded',
+            code: 'rate_limit_exceeded',
+            details: lastError?.message || null
           }
         })
       }
 
-      writeJson(res, 429, {
+      const status = typeof lastError?.status === 'number' && lastError.status >= 500 && lastError.status <= 599
+        ? lastError.status
+        : 502
+      return writeJson(res, status, {
         error: {
-          message: 'All configured accounts in pool exceeded rate limits or quota',
-          type: 'rate_limit_exceeded',
-          code: 'rate_limit_exceeded',
+          message: `Bad Gateway: upstream network error or unavailable: ${lastError?.message || 'unknown error'}`,
+          type: 'bad_gateway',
+          code: 'upstream_unavailable',
           details: lastError?.message || null
         }
       })

package/lib/slash-command.js

@@ -16,6 +16,7 @@ import {
   clearUsageCache,
   clearProbeCache,
   usageCache,
+  buildUsageCacheKey,
   sessionStats,
 } from './cline-client.js'
 
@@ -33,7 +34,7 @@ export function registerSlashCommand(ctx, { live, getSettingsApi, syncProviderSt
 
         // 1. Subcommand /cline models
         if (subcmd === 'models') {
-          const allModels = getAllModels(pub.dynamicModels, pub.modelContextOverrides)
+          const allModels = getAllModels(pub.dynamicModels, pub.modelContextOverrides, pub.customModels)
           const disabledSet = new Set(pub.disabledModels || [])
           const activeCount = allModels.filter((m) => !disabledSet.has(m.id)).length
           const syncStatus = pub.planSynced
@@ -76,7 +77,7 @@ export function registerSlashCommand(ctx, { live, getSettingsApi, syncProviderSt
           for (const acc of pool) {
             const pinStr = acc.isPinned ? ' [Pinned]' : ''
             const statusStr = acc.present ? 'Configured' : 'Missing Key'
-            const cacheKey = `cline:usage:${(acc.value || '').slice(-8)}`
+            const cacheKey = buildUsageCacheKey(pub.baseUrl, acc.value)
             const cached = usageCache.get(cacheKey)?.data
             const usageInfo = cached?.windows?.fiveHour ? ` · 5h ${cached.windows.fiveHour.percentUsed}% used` : ''
             lines.push(`  ${acc.label} (${acc.apiKeyEnv})`)
@@ -156,7 +157,7 @@ export function registerSlashCommand(ctx, { live, getSettingsApi, syncProviderSt
 
         // 7. Subcommand /cline test [model] / smoke
         if (subcmd === 'test' || subcmd === 'smoke') {
-          if (param && !isSupportedModel(param, pub.dynamicModels)) {
+          if (param && !isSupportedModel(param, pub.dynamicModels, pub.modelContextOverrides, pub.customModels)) {
             return `ClineBot: Model "${param}" is not recognized. Use /cline models to see available models.`
           }
           const activeKey = await resolveActiveAccountKey(ctx, live())

package/lib/updater.js

@@ -219,10 +219,32 @@ export async function checkUpdateStatus(options, target = detectRuntime()) {
   }
 }
 
-async function installExact(target, packageSpec, options) {
+export function cleanStaleLock(profileDir) {
+  if (!profileDir) return false
+  const lockFile = resolve(profileDir, 'package.json.lock')
+  if (existsSync(lockFile)) {
+    try {
+      const pid = parseInt(readFileSync(lockFile, 'utf8').trim(), 10)
+      if (!isPidAlive(pid)) {
+        unlinkSync(lockFile)
+        return true
+      }
+    } catch (err) {
+      /* best-effort stale lock unlink */
+    }
+  }
+  return false
+}
+
+export async function installExact(target, packageSpec, options = {}) {
   if (target.cliEntry === undefined) throw new Error('Automatic update is unavailable in this runtime.')
+  const timeoutMs = typeof options.timeoutMs === 'number' ? options.timeoutMs : UPDATE_TIMEOUT_MS
+  const sigtermGraceMs = typeof options.sigtermGraceMs === 'number' ? options.sigtermGraceMs : 500
+  const sigkillGraceMs = typeof options.sigkillGraceMs === 'number' ? options.sigkillGraceMs : 500
+  const spawnFn = typeof options.spawn === 'function' ? options.spawn : spawn
+
   await new Promise((resolvePromise, reject) => {
-    const child = spawn(process.execPath, [
+    const child = spawnFn(process.execPath, [
       target.cliEntry, 'plugin', '--profile', target.profileName, 'add',
       packageSpec,
       `--registry=${options.registry || 'https://registry.npmjs.org/'}`,
@@ -235,40 +257,71 @@ async function installExact(target, packageSpec, options) {
     let detail = ''
     child.stdout?.on('data', (chunk) => { detail = (detail + String(chunk)).slice(-4000) })
     child.stderr?.on('data', (chunk) => { detail = (detail + String(chunk)).slice(-4000) })
-    const timer = setTimeout(() => {
-      child.kill('SIGTERM')
-      setTimeout(() => {
-        if (!child.killed) {
+
+    let exited = false
+    let isTimedOut = false
+    let sigtermTimer = null
+    let sigkillTimer = null
+
+    function cleanupTimers() {
+      clearTimeout(mainTimer)
+      if (sigtermTimer) clearTimeout(sigtermTimer)
+      if (sigkillTimer) clearTimeout(sigkillTimer)
+    }
+
+    function onProcessExit(code, signal) {
+      if (exited) return
+      exited = true
+      cleanupTimers()
+
+      if (isTimedOut) {
+        cleanStaleLock(target.profileDir)
+        reject(new Error('Update timed out.'))
+      } else if (code === 0) {
+        resolvePromise()
+      } else {
+        reject(new Error(detail.trim() || `Update exited with code ${String(code)}.`))
+      }
+    }
+
+    child.once('error', (error) => {
+      cleanupTimers()
+      reject(error)
+    })
+
+    child.once('exit', onProcessExit)
+    child.once('close', onProcessExit)
+
+    const mainTimer = setTimeout(() => {
+      isTimedOut = true
+      // Step 1: Send SIGTERM
+      try {
+        child.kill('SIGTERM')
+      } catch (err) {
+        /* ignore if process already terminated */
+      }
+
+      // Step 2: Wait for grace period, if process has not exited escalate to SIGKILL (#172)
+      sigtermTimer = setTimeout(() => {
+        const hasExited = exited || (child.exitCode !== null && child.exitCode !== undefined) || (child.signalCode !== null && child.signalCode !== undefined)
+        if (!hasExited) {
           try {
             child.kill('SIGKILL')
           } catch (err) {
             /* ignore if process already terminated */
           }
-        }
-        setTimeout(() => {
-          if (target.profileDir) {
-            const lockFile = resolve(target.profileDir, 'package.json.lock')
-            if (existsSync(lockFile)) {
-              try {
-                const pid = parseInt(readFileSync(lockFile, 'utf8').trim(), 10)
-                if (!isPidAlive(pid)) {
-                  unlinkSync(lockFile)
-                }
-              } catch (err) {
-                /* best-effort stale lock unlink */
-              }
+
+          // Step 3: Hard deadline for exit notification
+          sigkillTimer = setTimeout(() => {
+            if (!exited) {
+              exited = true
+              cleanStaleLock(target.profileDir)
+              reject(new Error('Update timed out.'))
             }
-          }
-        }, 100)
-      }, 500)
-      reject(new Error('Update timed out.'))
-    }, UPDATE_TIMEOUT_MS)
-    child.once('error', (error) => { clearTimeout(timer); reject(error) })
-    child.once('exit', (code) => {
-      clearTimeout(timer)
-      if (code === 0) resolvePromise()
-      else reject(new Error(detail.trim() || `Update exited with code ${String(code)}.`))
-    })
+          }, sigkillGraceMs)
+        }
+      }, sigtermGraceMs)
+    }, timeoutMs)
   })
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goodandready/dsh-clinebot",
-  "version": "0.5.7",
+  "version": "0.5.9",
   "description": "DeepSeek Harness companion for ClineBot / ClinePass: dynamic subscription models sync, quota exhaustion warnings, session metrics, dedicated settings page, live usage limits, and /cline slash-command.",
   "license": "MIT",
   "type": "module",