[KooshaPari]

Summary

tests/unit/glm-sse-transform-arity.test.ts fails on release/v3.8.51 base at af49d4972, independently of any PR. Every PR targeting this base inherits a red Unit Tests job.

Root cause

#12770 added an arity guard asserting the GLM call ends with suppressThinkClose. #12925 then legitimised the buffer-size argument (GLM_STREAM_BUFFER_BYTES), so the call now ends with three more positionals after suppressThinkClose. The second assertion is stale.

Fix

Update the assertion to expect GLM_STREAM_BUFFER_BYTES as the final argument while keeping the hardcoded 65536 dead-literal guard from #12770.

- assert.match(call, /suppressThinkClose\s*\)\s*$/);
+ assert.match(call, /GLM_STREAM_BUFFER_BYTES\s*\)\s*$/, "call should end with GLM_STREAM_BUFFER_BYTES, not a hardcoded literal");

Also updated the JSDoc to explain the change.

Tests

✔ createSSETransformStreamWithLogger has no highWaterMark slot
✔ GLM translateSseResponse does not pass a 16th positional to the stream helper

Related

Fixes #13319

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[diegosouzapw]

Thanks! The release branch already rewrote tests/unit/glm-sse-transform-arity.test.ts for #12925. It now asserts that the helper declares streamBufferBytes as its last parameter and that GLM fills that slot with the named constant. That covers exactly what this PR updates. Closing as covered.