[diegosouzapw]

Closes #3612

TDD regression test added (failing→passing). Lint + typecheck:core clean. Part of the /review-issues v3.8.22 round.

[gemini-code-assist]

Code Review

This pull request updates the stream readiness detection logic in open-sse/utils/streamReadiness.ts to support tool-call-only chunks (which may only carry a numeric index in the first chunk) and chunks lacking an object or type field from OpenAI-compatible backends. It also adds corresponding unit tests in tests/unit/stream-readiness.test.ts to verify these behaviors and updates the changelog. No review comments were provided, and the implementation is clean and well-tested.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[kilo-code-bot]

WARNING: index should be validated as a non-negative integer

A numeric index is enough to mark a tool-call start, but malformed values like -1 or 0.5 would also pass. Use Number.isInteger(item.index) && item.index >= 0 so only valid OpenAI chunk indices are accepted.

Suggested change

	isRecord(item) && (hasNonEmptyString(item.id) || typeof item.index === "number");
	isRecord(item) && (hasNonEmptyString(item.id) || (Number.isInteger(item.index) && item.index >= 0));

[kilo-code-bot]

WARNING: Non-string object/type values are treated as absent

This guard only rejects string mismatches, so object: null or type: 123 would still be accepted if the choices look chat-like. Consider rejecting any non-chat.completion.chunk value for either field.

[kilo-code-bot]

Code Review Summary

Status: 2 Issues Found | Recommendation: Address before merge

Overview

Severity	Count
CRITICAL	0
WARNING	2
SUGGESTION	0

Issue Details (click to expand)

CRITICAL

File	Line	Issue

WARNING

File	Line	Issue
open-sse/utils/streamReadiness.ts	111	index is accepted for any numeric value, including malformed negative or fractional indices; validate with Number.isInteger(item.index) && item.index >= 0.
open-sse/utils/streamReadiness.ts	127	Non-string object/type values are treated as absent, so malformed payloads like object: null or type: 123 can still pass the chat-chunk guard.

SUGGESTION

File	Line	Issue

Other Observations (not in diff)

No issues found outside the diff.

Files Reviewed (3 files)

• CHANGELOG.md - 0 issues
• open-sse/utils/streamReadiness.ts - 2 issues
• tests/unit/stream-readiness.test.ts - 0 issues

Fix these issues in Kilo Cloud

---

_{Reviewed by nex-n2-pro:free · 861,891 tokens}