diff --git a/open-sse/services/errorClassifier.ts b/open-sse/services/errorClassifier.ts index d3765737d0..9aebe0531d 100644 --- a/open-sse/services/errorClassifier.ts +++ b/open-sse/services/errorClassifier.ts @@ -11,7 +11,7 @@ import { getProviderCategory, getRegistryEntry } from "../config/providerRegistr // NOT a silent "fake success" failure. Used to avoid rewriting a valid HTTP 200 // (e.g. a Claude Code `max_tokens: 1` connectivity ping) into a synthetic 502. const LEGIT_EMPTY_CLAUDE_STOP = new Set(["max_tokens", "tool_use"]); -const LEGIT_EMPTY_OPENAI_FINISH = new Set(["length", "tool_calls"]); +const LEGIT_EMPTY_OPENAI_FINISH = new Set(["length", "tool_calls", "content_filter"]); export function isEmptyContentResponse(responseBody: unknown): boolean { if (!responseBody || typeof responseBody !== "object") return false; diff --git a/tests/unit/empty-content-stopreason-3572.test.ts b/tests/unit/empty-content-stopreason-3572.test.ts index ea5fb4f8b1..b9676254c1 100644 --- a/tests/unit/empty-content-stopreason-3572.test.ts +++ b/tests/unit/empty-content-stopreason-3572.test.ts @@ -62,6 +62,24 @@ test("#3572 OpenAI: empty content + finish_reason=stop stays flagged (fake-succe ); }); +test("#3572 OpenAI: empty content + finish_reason=content_filter is NOT empty-failure (safety-filtered response)", () => { + assert.equal( + isEmptyContentResponse({ + choices: [{ index: 0, message: { content: "" }, finish_reason: "content_filter" }], + }), + false + ); +}); + +test("#3572 OpenAI: empty content + finish_reason=content_filter (stream chunk) is NOT empty-failure", () => { + assert.equal( + isEmptyContentResponse({ + choices: [{ index: 0, delta: { content: "" }, finish_reason: "content_filter" }], + }), + false + ); +}); + test("#3572 regression: non-empty content is never flagged", () => { assert.equal(isEmptyContentResponse({ content: [{ type: "text", text: "hi" }] }), false); assert.equal(