Patch Changes
-
#2176
6b027c6Thanks @moygospadin! - fix(cli): parse model responses that break JSON around a quoted placeholderA source string of
"Attach '{title}'?"becomes„{title}" anhängen?in German:
the language's word order moves the ICU placeholder to the front of the sentence
and its typography wraps it in quotes. Models emit those quotes unescaped, so the
JSON string closes early and the placeholder's{lands where a value belongs;
some responses instead drop the value's opening quote entirely.jsonrepairgives
up on both (Unexpected character "{"andColon expected at position N) and the
whole file fails to translate. German took the brunt of it — the same run
translated every other locale fine, so it looked locale-specific, and it was
deterministic across retries.parseModelResponsegains a last-resort repair pass that escapes quotes sitting
inside a string rather than terminating it (a quote only closes the string when
the next non-whitespace character could legally follow one), and treats a
placeholder in a value position as the start of a string rather than an object.
Extraction now also prefers the first brace that actually opens an object, so a
preamble likeHier ist die Übersetzung für {title}:no longer becomes the start
of the response.The pass runs only after
jsonrepairitself has failed, so responses that
already parse — or thatjsonrepairrepairs — are untouched, and a genuine
nested object or a"Zeit: {time}"value is left alone. The nested
stringified-datapath goes through the same repair.