pjandro/OmniRoute
1
0
Fork 0
mirror of https://github.com/diegosouzapw/OmniRoute.git synced 2026-07-26 09:52:11 +03:00
Code Issues Packages Projects Releases Wiki Activity

fix: use replaceAll for think tags to handle multiple occurrences

Browse Source
This commit is contained in:
AveryanAlex
2026-03-29 00:26:24 +03:00
parent 7723e46c26
commit fdaeccf1e5
3 changed files with 19 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
open-sse/transformer/responsesTransformer.ts
View File

@@ -377,7 +377,7 @@ export function createResponsesApiTransformStream(logger = null) {
if (content.includes("<think>")) {
state.inThinking = true;
content = content.replace("<think>", "");
content = content.replaceAll("<think>", "");
startReasoning(controller, idx);
}

2
open-sse/translator/response/openai-responses.ts
View File

@@ -75,7 +75,7 @@ export function openaiToOpenAIResponsesResponse(chunk, state) {
if (content.includes("<think>")) {
state.inThinking = true;
content = content.replace("<think>", "");
content = content.replaceAll("<think>", "");
startReasoning(state, emit, idx);
}

17
tests/unit/responses-translation-fixes.test.mjs
View File

@@ -403,3 +403,20 @@ test("Responses→Chat streaming: reasoning delta emits reasoning_content in Cha
assert.ok(result, "should return a chunk");
assert.equal(result.choices[0].delta.reasoning_content, "thinking step...");
});
test("Chat→Responses streaming: multiple <think> tags in one chunk handled", () => {
const state = initState(FORMATS.OPENAI_RESPONSES);
// Chunk with multiple think tags
const chunk = {
choices: [{ index: 0, delta: { content: "<think>first</think>middle<think>second</think>end" }, finish_reason: null }],
id: "c1",
};
const events = openaiToOpenAIResponsesResponse(chunk, state);
// Should not have literal <think> in any text delta
const textDeltas = events
.filter((e) => e.event === "response.output_text.delta")
.map((e) => e.data.delta);
const combined = textDeltas.join("");
assert.ok(!combined.includes("<think>"), `text should not contain <think> tag, got: ${combined}`);
});