@shawnstack/quickforge 1.3.16 → 1.3.18

package/node_modules/undici/lib/dispatcher/client.js

@@ -235,9 +235,13 @@ class Client extends DispatcherBase {
         ...(typeof autoSelectFamily === 'boolean' ? { autoSelectFamily, autoSelectFamilyAttemptTimeout } : undefined),
         ...connect
       })
-    } else if (socketPath != null) {
+    } else {
       const customConnect = connect
-      connect = (opts, callback) => customConnect({ ...opts, socketPath }, callback)
+      connect = (opts, callback) => customConnect({
+        ...opts,
+        ...(socketPath != null ? { socketPath } : null),
+        ...(allowH2 != null ? { allowH2 } : null)
+      }, callback)
     }
 
     this[kUrl] = util.parseOrigin(url)

package/node_modules/undici/lib/dispatcher/h2c-client.js

@@ -15,7 +15,7 @@ class H2CClient extends Client {
       )
     }
 
-    const { connect, maxConcurrentStreams, pipelining, ...opts } =
+    const { maxConcurrentStreams, pipelining, ...opts } =
             clientOpts ?? {}
     let defaultMaxConcurrentStreams = 100
     let defaultPipelining = 100

package/node_modules/undici/lib/mock/mock-call-history.js

@@ -3,14 +3,14 @@
 const { kMockCallHistoryAddLog } = require('./mock-symbols')
 const { InvalidArgumentError } = require('../core/errors')
 
-function handleFilterCallsWithOptions (criteria, options, handler, store) {
+function handleFilterCallsWithOptions (criteria, options, handler, store, allLogs) {
   switch (options.operator) {
     case 'OR':
-      store.push(...handler(criteria))
+      store.push(...handler(criteria, allLogs))
 
       return store
     case 'AND':
-      return handler.call({ logs: store }, criteria)
+      return handler(criteria, store)
     default:
       // guard -- should never happens because buildAndValidateFilterCallsOptions is called before
       throw new InvalidArgumentError('options.operator must to be a case insensitive string equal to \'OR\' or \'AND\'')
@@ -35,14 +35,14 @@ function buildAndValidateFilterCallsOptions (options = {}) {
 }
 
 function makeFilterCalls (parameterName) {
-  return (parameterValue) => {
+  return (parameterValue, logs) => {
     if (typeof parameterValue === 'string' || parameterValue == null) {
-      return this.logs.filter((log) => {
+      return logs.filter((log) => {
         return log[parameterName] === parameterValue
       })
     }
     if (parameterValue instanceof RegExp) {
-      return this.logs.filter((log) => {
+      return logs.filter((log) => {
         return parameterValue.test(log[parameterName])
       })
     }
@@ -175,30 +175,30 @@ class MockCallHistory {
 
       const finalOptions = { operator: 'OR', ...buildAndValidateFilterCallsOptions(options) }
 
-      let maybeDuplicatedLogsFiltered = []
+      let maybeDuplicatedLogsFiltered = finalOptions.operator === 'AND' ? this.logs : []
       if ('protocol' in criteria) {
-        maybeDuplicatedLogsFiltered = handleFilterCallsWithOptions(criteria.protocol, finalOptions, this.filterCallsByProtocol, maybeDuplicatedLogsFiltered)
+        maybeDuplicatedLogsFiltered = handleFilterCallsWithOptions(criteria.protocol, finalOptions, this.filterCallsByProtocol, maybeDuplicatedLogsFiltered, this.logs)
       }
       if ('host' in criteria) {
-        maybeDuplicatedLogsFiltered = handleFilterCallsWithOptions(criteria.host, finalOptions, this.filterCallsByHost, maybeDuplicatedLogsFiltered)
+        maybeDuplicatedLogsFiltered = handleFilterCallsWithOptions(criteria.host, finalOptions, this.filterCallsByHost, maybeDuplicatedLogsFiltered, this.logs)
       }
       if ('port' in criteria) {
-        maybeDuplicatedLogsFiltered = handleFilterCallsWithOptions(criteria.port, finalOptions, this.filterCallsByPort, maybeDuplicatedLogsFiltered)
+        maybeDuplicatedLogsFiltered = handleFilterCallsWithOptions(criteria.port, finalOptions, this.filterCallsByPort, maybeDuplicatedLogsFiltered, this.logs)
       }
       if ('origin' in criteria) {
-        maybeDuplicatedLogsFiltered = handleFilterCallsWithOptions(criteria.origin, finalOptions, this.filterCallsByOrigin, maybeDuplicatedLogsFiltered)
+        maybeDuplicatedLogsFiltered = handleFilterCallsWithOptions(criteria.origin, finalOptions, this.filterCallsByOrigin, maybeDuplicatedLogsFiltered, this.logs)
       }
       if ('path' in criteria) {
-        maybeDuplicatedLogsFiltered = handleFilterCallsWithOptions(criteria.path, finalOptions, this.filterCallsByPath, maybeDuplicatedLogsFiltered)
+        maybeDuplicatedLogsFiltered = handleFilterCallsWithOptions(criteria.path, finalOptions, this.filterCallsByPath, maybeDuplicatedLogsFiltered, this.logs)
       }
       if ('hash' in criteria) {
-        maybeDuplicatedLogsFiltered = handleFilterCallsWithOptions(criteria.hash, finalOptions, this.filterCallsByHash, maybeDuplicatedLogsFiltered)
+        maybeDuplicatedLogsFiltered = handleFilterCallsWithOptions(criteria.hash, finalOptions, this.filterCallsByHash, maybeDuplicatedLogsFiltered, this.logs)
       }
       if ('fullUrl' in criteria) {
-        maybeDuplicatedLogsFiltered = handleFilterCallsWithOptions(criteria.fullUrl, finalOptions, this.filterCallsByFullUrl, maybeDuplicatedLogsFiltered)
+        maybeDuplicatedLogsFiltered = handleFilterCallsWithOptions(criteria.fullUrl, finalOptions, this.filterCallsByFullUrl, maybeDuplicatedLogsFiltered, this.logs)
       }
       if ('method' in criteria) {
-        maybeDuplicatedLogsFiltered = handleFilterCallsWithOptions(criteria.method, finalOptions, this.filterCallsByMethod, maybeDuplicatedLogsFiltered)
+        maybeDuplicatedLogsFiltered = handleFilterCallsWithOptions(criteria.method, finalOptions, this.filterCallsByMethod, maybeDuplicatedLogsFiltered, this.logs)
       }
 
       const uniqLogsFiltered = [...new Set(maybeDuplicatedLogsFiltered)]

package/node_modules/undici/lib/util/cache.js

@@ -18,7 +18,7 @@ function makeCacheKey (opts) {
 
   let fullPath = opts.path || '/'
 
-  if (opts.query && !pathHasQueryOrFragment(opts.path)) {
+  if (opts.query && !pathHasQueryOrFragment(fullPath)) {
     fullPath = serializePathWithQuery(fullPath, opts.query)
   }
 
@@ -374,9 +374,11 @@ function assertCacheMethods (methods, name = 'CacheMethods') {
  * @returns {string}
  */
 function makeDeduplicationKey (cacheKey, excludeHeaders) {
-  // Create a deterministic string key from the cache key
-  // Include origin, method, path, and sorted headers
-  let key = `${cacheKey.origin}:${cacheKey.method}:${cacheKey.path}`
+  // Use JSON.stringify to produce a collision-resistant key.
+  // Previous format used `:` and `=` delimiters without escaping, which
+  // allowed different header sets to produce identical keys (e.g.
+  // {a:"x:b=y"} vs {a:"x", b:"y"}). See: https://github.com/nodejs/undici/issues/5012
+  const headers = {}
 
   if (cacheKey.headers) {
     const sortedHeaders = Object.keys(cacheKey.headers).sort()
@@ -385,12 +387,11 @@ function makeDeduplicationKey (cacheKey, excludeHeaders) {
       if (excludeHeaders?.has(header.toLowerCase())) {
         continue
       }
-      const value = cacheKey.headers[header]
-      key += `:${header}=${Array.isArray(value) ? value.join(',') : value}`
+      headers[header] = cacheKey.headers[header]
     }
   }
 
-  return key
+  return JSON.stringify([cacheKey.origin, cacheKey.method, cacheKey.path, headers])
 }
 
 module.exports = {

package/node_modules/undici/lib/web/fetch/formdata-parser.js

@@ -204,7 +204,7 @@ function multipartFormDataParser (input, mimeType) {
  * Parses content-disposition attributes (e.g., name="value" or filename*=utf-8''encoded)
  * @param {Buffer} input
  * @param {{ position: number }} position
- * @returns {{ name: string, value: string }}
+ * @returns {{ name: string, value: string, extended: boolean } | null}
  */
 function parseContentDispositionAttribute (input, position) {
   // Skip leading semicolon and whitespace
@@ -304,7 +304,7 @@ function parseContentDispositionAttribute (input, position) {
     value = decoder.decode(tokenValue)
   }
 
-  return { name: attrNameStr, value }
+  return { name: attrNameStr, value, extended: isExtended }
 }
 
 /**
@@ -368,6 +368,9 @@ function parseMultipartFormDataHeaders (input, position) {
     switch (bufferToLowerCasedHeaderName(headerName)) {
       case 'content-disposition': {
         name = filename = null
+        // Track whether filename was set from the extended (RFC 5987) form so
+        // a subsequent legacy `filename` attribute does not override it.
+        let filenameIsExtended = false
 
         // Collect the disposition type (should be "form-data")
         const dispositionType = collectASequenceOfBytes(
@@ -383,8 +386,8 @@ function parseMultipartFormDataHeaders (input, position) {
         // Parse attributes recursively until CRLF
         while (
           position.position < input.length &&
-          input[position.position] !== 0x0d &&
-          input[position.position + 1] !== 0x0a
+          (input[position.position] !== 0x0d ||
+          input[position.position + 1] !== 0x0a)
         ) {
           const attribute = parseContentDispositionAttribute(input, position)
 
@@ -395,7 +398,15 @@ function parseMultipartFormDataHeaders (input, position) {
           if (attribute.name === 'name') {
             name = attribute.value
           } else if (attribute.name === 'filename') {
-            filename = attribute.value
+            // Per RFC 5987 §4.1, when both legacy and extended forms of the
+            // same parameter are present, the extended (filename*) form takes
+            // precedence regardless of the order they appear in.
+            if (attribute.extended) {
+              filename = attribute.value
+              filenameIsExtended = true
+            } else if (!filenameIsExtended) {
+              filename = attribute.value
+            }
           }
         }
 
@@ -448,7 +459,7 @@ function parseMultipartFormDataHeaders (input, position) {
 
     // 2.9. If position does not point to a sequence of bytes starting with 0x0D 0x0A
     //      (CR LF), return failure. Otherwise, advance position by 2 (past the newline).
-    if (input[position.position] !== 0x0d && input[position.position + 1] !== 0x0a) {
+    if (input[position.position] !== 0x0d || input[position.position + 1] !== 0x0a) {
       throw parsingError('expected CRLF')
     } else {
       position.position += 2

package/node_modules/undici/lib/web/fetch/index.js

@@ -1030,7 +1030,7 @@ function fetchFinale (fetchParams, response) {
       let responseStatus = 0
 
       // 7. If fetchParams’s request’s mode is not "navigate" or response’s has-cross-origin-redirects is false:
-      if (fetchParams.request.mode !== 'navigator' || !response.hasCrossOriginRedirects) {
+      if (fetchParams.request.mode !== 'navigate' || !response.hasCrossOriginRedirects) {
         // 1. Set responseStatus to response’s status.
         responseStatus = response.status
 
@@ -1433,7 +1433,10 @@ async function httpNetworkOrCacheFetch (
   //    8. If contentLengthHeaderValue is non-null, then append
   //    `Content-Length`/contentLengthHeaderValue to httpRequest’s header
   //    list.
-  if (contentLengthHeaderValue != null) {
+  if (
+    contentLengthHeaderValue != null &&
+    !httpRequest.headersList.contains('content-length', true)
+  ) {
     httpRequest.headersList.append('content-length', contentLengthHeaderValue, true)
   }
 

package/node_modules/undici/lib/web/webidl/index.js

@@ -190,10 +190,10 @@ webidl.util.ConvertToInt = function (V, bitLength, signedness, flags) {
   } else {
     // 3. Otherwise:
 
-    // 1. Let lowerBound be -2^bitLength − 1.
-    lowerBound = Math.pow(-2, bitLength) - 1
+    // 1. Let lowerBound be -2^(bitLength − 1).
+    lowerBound = -Math.pow(2, bitLength - 1)
 
-    // 2. Let upperBound be 2^bitLength − 1 − 1.
+    // 2. Let upperBound be 2^(bitLength − 1) − 1.
     upperBound = Math.pow(2, bitLength - 1) - 1
   }
 
@@ -272,9 +272,9 @@ webidl.util.ConvertToInt = function (V, bitLength, signedness, flags) {
   // 10. Set x to x modulo 2^bitLength.
   x = x % Math.pow(2, bitLength)
 
-  // 11. If signedness is "signed" and x ≥ 2^bitLength − 1,
+  // 11. If signedness is "signed" and x ≥ 2^(bitLength − 1),
   //    then return x − 2^bitLength.
-  if (signedness === 'signed' && x >= Math.pow(2, bitLength) - 1) {
+  if (signedness === 'signed' && x >= Math.pow(2, bitLength - 1)) {
     return x - Math.pow(2, bitLength)
   }
 

package/node_modules/undici/lib/web/websocket/stream/websocketstream.js

@@ -284,12 +284,6 @@ class WebSocketStream {
       start: (controller) => {
         this.#readableStreamController = controller
       },
-      pull (controller) {
-        let chunk
-        while (controller.desiredSize > 0 && (chunk = response.socket.read()) !== null) {
-          controller.enqueue(chunk)
-        }
-      },
       cancel: (reason) => this.#cancel(reason)
     })
 
@@ -338,7 +332,7 @@ class WebSocketStream {
       try {
         chunk = utf8Decode(data)
       } catch {
-        failWebsocketConnection(this.#handler, 'Received invalid UTF-8 in text frame.')
+        failWebsocketConnection(this.#handler, 1007, 'Received invalid UTF-8 in text frame.')
         return
       }
     } else if (type === opcodes.BINARY) {

package/node_modules/undici/package.json

@@ -1,6 +1,6 @@
 {
   "name": "undici",
-  "version": "7.25.0",
+  "version": "7.26.0",
   "description": "An HTTP/1.1 client, written from scratch for Node.js",
   "homepage": "https://undici.nodejs.org",
   "bugs": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@shawnstack/quickforge",
-  "version": "1.3.16",
+  "version": "1.3.18",
   "description": "AI chat application with YOLO-mode local workspace tools. React + Vite + Tailwind CSS frontend, local Node.js storage server.",
   "keywords": [
     "ai",

package/server/agent-manager.mjs

@@ -360,6 +360,7 @@ function updateSessionMessages(session, messages) {
 function resetSessionCompaction(session) {
   session.contextCompaction = null
   session.lastAutoCompactAt = null
+  session.lastAutoCompactRejected = null
   session.lastTransformedContextMessages = null
   session.autoCompacting = false
 }
@@ -642,7 +643,7 @@ function compactedContextMessages(messages) {
 }
 
 async function transformSessionContext(session, messages, signal) {
-  await maybeAutoCompactSession({
+  const autoCompactResult = await maybeAutoCompactSession({
     session,
     messages,
     signal,
@@ -651,6 +652,13 @@ async function transformSessionContext(session, messages, signal) {
     logger,
     confirmAutoCompact: createAutoCompactApprovalPromise,
   })
+  if (!autoCompactResult.compacted && autoCompactResult.usage && autoCompactResult.reason && autoCompactResult.reason !== 'below_threshold') {
+    logger.info(`Auto compact skipped for session ${session.sessionId}: ${autoCompactResult.reason}`, {
+      sessionId: session.sessionId,
+      reason: autoCompactResult.reason,
+      usage: autoCompactResult.usage,
+    })
+  }
   const transformedMessages = buildAutoCompactLoopMessages(session, messages)
   session.lastTransformedContextMessages = transformedMessages
   return applyActiveCommandPrompt(compactedContextMessages(transformedMessages), session?.activeCommandPrompt)
@@ -837,6 +845,7 @@ export async function createAgent(sessionId, config = {}) {
     lastTransformedContextMessages: null,
     autoCompacting: false,
     lastAutoCompactAt: null,
+    lastAutoCompactRejected: null,
     /** Track active SSE connections. Only one SSE stream allowed per session to prevent
      *  connection-pool exhaustion when two browser tabs load the same session. */
     sseConnected: false,

package/server/auto-compaction.mjs

@@ -12,6 +12,7 @@ export const DEFAULT_AUTO_COMPACT_SETTINGS = {
 }
 
 const AUTO_COMPACT_MIN_INTERVAL_MS = 30_000
+const AUTO_COMPACT_REJECTION_SUPPRESS_MS = 10 * 60_000
 
 function clampNumber(value, fallback, min, max) {
   const parsed = Number(value)
@@ -85,16 +86,48 @@ function estimateMessagesChars(messages) {
   }, 0)
 }
 
-export function estimateContextUsage({ systemPrompt, messages, tools, model }) {
+function messageTimestampMs(message) {
+  const timestamp = message?.timestamp
+  if (typeof timestamp === 'number') return timestamp
+  if (typeof timestamp === 'string') {
+    const parsed = Date.parse(timestamp)
+    return Number.isNaN(parsed) ? 0 : parsed
+  }
+  return 0
+}
+
+function latestCompactTimestampMs(session) {
+  return messageTimestampMs(session?.contextCompaction?.summaryMessage)
+}
+
+function latestKnownInputTokens(messages, sinceTimestamp = 0) {
+  let latestTimestamp = -1
+  let latestInput = 0
+  for (const message of Array.isArray(messages) ? messages : []) {
+    if (message?.role !== 'assistant' || !message.usage) continue
+    const timestamp = messageTimestampMs(message)
+    if (sinceTimestamp > 0 && timestamp <= sinceTimestamp) continue
+    if (timestamp < latestTimestamp) continue
+    const input = Math.max(0, Number(message.usage.input ?? message.usage.totalTokens) || 0)
+    if (input <= 0) continue
+    latestTimestamp = timestamp
+    latestInput = input
+  }
+  return latestInput
+}
+
+export function estimateContextUsage({ systemPrompt, messages, tools, model, knownInputTokens = 0 }) {
   const contextWindow = Number(model?.contextWindow) || 0
   const reservedOutputTokens = Math.max(0, Number(model?.maxTokens) || 4096)
-  const inputTokens =
+  const estimatedInputTokens =
     estimateTextTokens(systemPrompt) +
     estimateMessagesTokens(messages) +
     estimateTextTokens(safeJson(tools))
+  const knownInput = Math.max(0, Number(knownInputTokens) || 0)
+  const inputTokens = Math.max(estimatedInputTokens, knownInput)
   const totalTokens = inputTokens + reservedOutputTokens
   const percent = contextWindow > 0 ? Math.round((totalTokens / contextWindow) * 1000) / 10 : 0
-  return { inputTokens, reservedOutputTokens, totalTokens, contextWindow, percent }
+  return { inputTokens, estimatedInputTokens, knownInputTokens: knownInput, reservedOutputTokens, totalTokens, contextWindow, percent }
 }
 
 function isUserMessage(message) {
@@ -112,6 +145,34 @@ function tailStartForRecentTurns(messages, keepRecentTurns) {
   return 0
 }
 
+function shouldSuppressAfterRejection(session, messages, usage) {
+  const rejection = session?.lastAutoCompactRejected
+  if (!rejection) return false
+  const now = Date.now()
+  const rejectedAt = Number(rejection.rejectedAt) || 0
+  if (rejectedAt <= 0 || now - rejectedAt > AUTO_COMPACT_REJECTION_SUPPRESS_MS) return false
+
+  const rejectedMessageCount = Number(rejection.messageCount) || 0
+  const currentMessageCount = Array.isArray(messages) ? messages.length : 0
+  if (currentMessageCount >= rejectedMessageCount + 3) return false
+
+  const rejectedPercent = Number(rejection.percent) || 0
+  const currentPercent = Number(usage?.percent) || 0
+  return currentPercent <= rejectedPercent + 5
+}
+
+function markAutoCompactRejected(session, messages, usage) {
+  session.lastAutoCompactRejected = {
+    rejectedAt: Date.now(),
+    messageCount: Array.isArray(messages) ? messages.length : 0,
+    percent: Number(usage?.percent) || 0,
+  }
+}
+
+function clearAutoCompactRejected(session) {
+  session.lastAutoCompactRejected = null
+}
+
 function compactSummaryText(message) {
   const content = message?.content
   const text = typeof content === 'string'
@@ -150,37 +211,29 @@ function buildCompactionSourceMessages(session, messages, tailStart) {
 export function buildAutoCompactLoopMessages(session, messages) {
   const summaryMessage = session?.contextCompaction?.summaryMessage
   if (!summaryMessage) return messages
-  const keepRecentTurns = session.contextCompaction.keepRecentTurns || DEFAULT_AUTO_COMPACT_SETTINGS.keepRecentTurns
-  const tailStart = tailStartForRecentTurns(messages, keepRecentTurns)
-  return [summaryMessage, ...messages.slice(tailStart)]
+  const source = Array.isArray(messages) ? messages : []
+  const compactedUpToIndex = Math.min(source.length, Math.max(0, Number(session.contextCompaction?.compactedUpToIndex) || 0))
+  return [summaryMessage, ...source.slice(compactedUpToIndex)]
 }
 
 export async function maybeAutoCompactSession({ session, messages, signal, emitSessionEvent, persistSession, logger, confirmAutoCompact }) {
   if (!session || session.autoCompacting) return { compacted: false }
   const settings = await readAutoCompactSettings()
-  if (!settings.enabled) return { compacted: false }
-  if (signal?.aborted) return { compacted: false }
+  if (!settings.enabled) return { compacted: false, reason: 'disabled' }
+  if (signal?.aborted) return { compacted: false, reason: 'aborted' }
 
   const loopMessages = buildAutoCompactLoopMessages(session, messages)
+  const knownInputTokens = latestKnownInputTokens(messages, latestCompactTimestampMs(session))
   const usage = estimateContextUsage({
     systemPrompt: session.agent.state.systemPrompt,
     messages: loopMessages,
     tools: session.agent.state.tools,
     model: session.model,
+    knownInputTokens,
   })
-  if (!usage.contextWindow || usage.percent < settings.thresholdPercent) return { compacted: false, usage }
-
-  emitSessionEvent?.(session, {
-    type: 'auto_compact_threshold_reached',
-    usage,
-    thresholdPercent: settings.thresholdPercent,
-    requireConfirmation: settings.requireConfirmation,
-  })
-
-  if (settings.requireConfirmation) {
-    const approved = await confirmAutoCompact?.(session, { usage, settings })
-    if (!approved || signal?.aborted) return { compacted: false, usage, reason: approved === false ? 'user_rejected' : 'confirmation_unavailable' }
-  }
+  if (!usage.contextWindow) return { compacted: false, usage, reason: 'missing_context_window' }
+  if (usage.percent < settings.thresholdPercent) return { compacted: false, usage, reason: 'below_threshold' }
+  if (shouldSuppressAfterRejection(session, messages, usage)) return { compacted: false, usage, reason: 'user_rejected_recently' }
 
   const now = Date.now()
   if (session.lastAutoCompactAt && now - session.lastAutoCompactAt < AUTO_COMPACT_MIN_INTERVAL_MS) {
@@ -196,6 +249,21 @@ export async function maybeAutoCompactSession({ session, messages, signal, emitS
     return { compacted: false, usage, reason: 'not_enough_history' }
   }
 
+  emitSessionEvent?.(session, {
+    type: 'auto_compact_threshold_reached',
+    usage,
+    thresholdPercent: settings.thresholdPercent,
+    requireConfirmation: settings.requireConfirmation,
+  })
+
+  if (settings.requireConfirmation) {
+    const approved = await confirmAutoCompact?.(session, { usage, settings })
+    if (!approved || signal?.aborted) {
+      if (approved === false) markAutoCompactRejected(session, messages, usage)
+      return { compacted: false, usage, reason: approved === false ? 'user_rejected' : 'confirmation_unavailable' }
+    }
+  }
+
   session.autoCompacting = true
   try {
     const result = await compactConversation({
@@ -225,6 +293,7 @@ export async function maybeAutoCompactSession({ session, messages, signal, emitS
       usageBefore: usage,
       thresholdPercent: settings.thresholdPercent,
     }
+    clearAutoCompactRejected(session)
     session.lastAutoCompactAt = now
     await persistSession(session)
     emitSessionEvent(session, {
@@ -242,6 +311,12 @@ export async function maybeAutoCompactSession({ session, messages, signal, emitS
     return { compacted: true, usage }
   } catch (error) {
     logger?.warn?.(`Auto compact failed for session ${session.sessionId}:`, error?.message || error, { sessionId: session.sessionId })
+    emitSessionEvent?.(session, {
+      type: 'auto_compact_failed',
+      usage,
+      thresholdPercent: settings.thresholdPercent,
+      error: error?.message || String(error),
+    })
     return { compacted: false, usage, reason: 'error', error }
   } finally {
     session.autoCompacting = false

package/server/tools/definitions.mjs

@@ -68,6 +68,8 @@ export const workspaceTools = [
     description: 'Run a shell command in the project bound to this chat. Use this for lint, build, tests, git status, and diagnostics.',
     parameters: Type.Object({
       command: Type.String({ description: 'Command to execute in the workspace.' }),
+      timeoutMs: Type.Optional(Type.Number({ description: 'Command timeout in milliseconds. Defaults to 30 minutes and is clamped to the supported range.', default: 1800000 })),
+      description: Type.Optional(Type.String({ description: 'Short explanation of why this command is being run.' })),
     }),
     executionMode: 'sequential',
   },