luulam/9router
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(chatCore): add missing capsOverride + streamErrorPatterns to destructure

Browse Source 
* handleChatCore() referenced capsOverride (line 162) and
  streamErrorPatterns (line 475) but the destructured param list
  did not include them. Callers that did not pass these (e.g.
  open-sse/handlers/responsesHandler.js, unit callers, older
  client builds) would trigger 'capsOverride is not defined' /
  'streamErrorPatterns is not defined' ReferenceError mid-request.
* Both defaulted to null. capsOverride is read by capability merge
  (line 162, already guarded by '|| {}'). streamErrorPatterns is
  read in the early-peek hook (line 484) which was null-safe
  only because the variable happened to be in scope when chat.js
  spread it in; responsesHandler never passed it and would crash.
* Unblocks all callers regardless of which fields they pass.
This commit is contained in:
luulam
2026-09-07 15:23:48 +07:00
parent 59f17b3725
commit 6f52d7020c
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
open-sse/handlers/chatCore.js
View File

@@ -59,7 +59,7 @@ export function stripContinuityFields(body) {
return body; return body;
} }
export async function handleChatCore({ body, modelInfo, credentials, log, onCredentialsRefreshed, onRequestSuccess, onDisconnect, clientRawRequest, connectionId, userAgent, apiKey, ccFilterNaming, rtkEnabled, headroomEnabled, headroomUrl, headroomCompressUserMessages, headroomTimeoutMs, cavemanEnabled, cavemanLevel, ponytailEnabled, ponytailLevel, pxpipeEnabled, pxpipeMinChars, pxpipeTimeoutMs, pxpipeTransform, onPxpipeEvent, sourceFormatOverride, providerThinking }) { export async function handleChatCore({ body, modelInfo, credentials, log, onCredentialsRefreshed, onRequestSuccess, onDisconnect, clientRawRequest, connectionId, userAgent, apiKey, ccFilterNaming, rtkEnabled, headroomEnabled, headroomUrl, headroomCompressUserMessages, headroomTimeoutMs, cavemanEnabled, cavemanLevel, ponytailEnabled, ponytailLevel, pxpipeEnabled, pxpipeMinChars, pxpipeTimeoutMs, pxpipeTransform, onPxpipeEvent, sourceFormatOverride, providerThinking, capsOverride = null, streamErrorPatterns = null }) {
const { provider, model } = modelInfo; const { provider, model } = modelInfo;
const requestStartTime = Date.now(); const requestStartTime = Date.now();
// Stable per-session color so all lines of one CLI conversation share a tag // Stable per-session color so all lines of one CLI conversation share a tag