[iamwavecut]

Problem

Gemma sometimes writes a tool call as text and closes a <|"|> string with a plain quote:

draw_image{caption:<|"|>Text",prompt:<|"|>A detailed prompt<|"|>}

The strict Gemma parser rejects this shape, so the fallback inline parser handles it. That parser read the caption up to the next <|"|>, which is the opening delimiter of prompt. The prompt came out right, but the caption under the image ended with ",prompt:.

Change

• In inline_arg_end, a <|"|> right after ,key: (a known inline argument key) opens the next argument and no longer closes the current value. The value then ends at the separator, like an unclosed string.
• When a <|"|>-opened value does not end at a <|"|>, an unmatched trailing " or ' is dropped. Balanced quotes inside the value stay.

Tests

New test gemma_string_closed_with_a_plain_quote_ends_before_the_next_argument covers the exact failing shape, a missing closing quote, a plain-quoted next argument, and a caption with quotes and a comma inside that must stay unchanged. The test failed before the change.

• cargo fmt --all -- --check
• cargo clippy --workspace --all-targets -- -D warnings
• cargo test -p openplotva-dialog (109 passed)
• cargo test -p openplotva-llm (221 passed, 2 ignored)
• cargo test -p openplotva-app dialog (354 passed, 1 ignored)

[github-actions]

	Messages
📖	Danger checked 1 changed file(s).

Generated by 🚫 dangerJS against 3fcd19a