shumengya/sproutclaw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(ai): handle unknown finish_reason in openai-completions gracefully

Browse Source 
Map unknown finish_reason values (e.g. "end" from Ollama/LM Studio) to
"stop" instead of throwing, since assistant content is already produced.

fixes #2142
This commit is contained in:
Mario Zechner
2026-03-14 03:59:55 +01:00
parent 9c0ba357ad
commit a79ca41190
2 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
packages/ai/CHANGELOG.md
View File

@@ -2,6 +2,10 @@
## [Unreleased] ## [Unreleased]
### Fixed
- Handle `finish_reason: "end"` from Ollama/LM Studio by mapping it to `"stop"` instead of throwing ([#2142](https://github.com/badlogic/pi-mono/issues/2142))
## [0.58.0] - 2026-03-14 ## [0.58.0] - 2026-03-14
### Added ### Added

5
packages/ai/src/providers/openai-completions.ts
View File

@@ -730,10 +730,11 @@ function parseChunkUsage(
return usage; return usage;
} }
function mapStopReason(reason: ChatCompletionChunk.Choice["finish_reason"]): StopReason { function mapStopReason(reason: ChatCompletionChunk.Choice["finish_reason"] | string): StopReason {
if (reason === null) return "stop"; if (reason === null) return "stop";
switch (reason) { switch (reason) {
case "stop": case "stop":
case "end":
return "stop"; return "stop";
case "length": case "length":
return "length"; return "length";
@@ -743,7 +744,7 @@ function mapStopReason(reason: ChatCompletionChunk.Choice["finish_reason"]): Sto
case "content_filter": case "content_filter":
return "error"; return "error";
default: { default: {
const _exhaustive: never = reason; const _exhaustive: never = reason as never;
throw new Error(`Unhandled stop reason: ${_exhaustive}`); throw new Error(`Unhandled stop reason: ${_exhaustive}`);
} }
} }