Merge pull request #242 from CJackHwang/codex/fix-issues-in-pull-request-#240

fix: avoid double-decoding XML entity text in markup tool-call parsing
This commit is contained in:
CJACK.
2026-04-07 13:16:01 +08:00
committed by GitHub

View File

@@ -145,12 +145,12 @@ func parseSingleXMLToolCall(block string) (ParsedToolCall, bool) {
params[t.Name.Local] = strings.TrimSpace(v)
break
}
name = strings.TrimSpace(html.UnescapeString(v))
name = strings.TrimSpace(v)
}
case "input", "arguments", "argument", "args", "params":
var v string
if err := dec.DecodeElement(&v, &t); err == nil && strings.TrimSpace(v) != "" {
if parsed := parseToolCallInput(strings.TrimSpace(html.UnescapeString(v))); len(parsed) > 0 {
if parsed := parseToolCallInput(strings.TrimSpace(v)); len(parsed) > 0 {
for k, vv := range parsed {
params[k] = vv
}