Repository navigation
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces a new utility file open-sse/utils/stream/openaiChunks.ts with the getOpenAIIntermediateChunks function. The review feedback correctly identifies unused imports (convertOpenAIToResponsesToolCall and uuidv4) in the newly added file and suggests removing them to maintain clean code.
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.
| import { convertOpenAIToResponsesToolCall } from "../handlers/responseTranslator.ts"; | ||
| import { v4 as uuidv4 } from "uuid"; | ||
|
|
||
| import { JsonRecord } from "./types.ts"; |
There was a problem hiding this comment.
The imports convertOpenAIToResponsesToolCall and uuidv4 are unused in this file and should be removed to maintain clean code.
| import { convertOpenAIToResponsesToolCall } from "../handlers/responseTranslator.ts"; | |
| import { v4 as uuidv4 } from "uuid"; | |
| import { JsonRecord } from "./types.ts"; | |
| import { JsonRecord } from "./types.ts"; |
Part of Issue diegosouzapw#3594 modularization. Extracted OpenAI intermediate chunk handling from the monolithic stream.ts. Exports: - getOpenAIIntermediateChunks
ddddbc7 to
f2b1bd4
Compare
|
Thanks, @oyi77 🙏. These stream extractions are currently additive — the new modules under |
Part of modularization effort (Issue #3594).
Extracts OpenAI intermediate chunk handling from the monolithic
open-sse/utils/stream.tsinto a dedicated module.Changes:
open-sse/utils/stream/openaiChunks.ts(9 lines)getOpenAIIntermediateChunksTesting: No behavior change. Existing stream tests pass.
Follow-up PRs will extract: types (#3917), utils (#3918), errors (#3919), textualToolCalls (#3920), sseFormatters (#3923), claudeLifecycle, responsesLifecycle, and streamCore.