refactor: remove redundant bounds checks per code review
This commit is contained in:
@@ -305,11 +305,7 @@ func fixCLIToolResponse(input string) (string, error) {
|
|||||||
|
|
||||||
functionResponseContent := `{"parts":[],"role":"function"}`
|
functionResponseContent := `{"parts":[],"role":"function"}`
|
||||||
for ri, response := range groupResponses {
|
for ri, response := range groupResponses {
|
||||||
fallbackName := ""
|
partRaw := parseFunctionResponseRaw(response, group.CallNames[ri])
|
||||||
if ri < len(group.CallNames) {
|
|
||||||
fallbackName = group.CallNames[ri]
|
|
||||||
}
|
|
||||||
partRaw := parseFunctionResponseRaw(response, fallbackName)
|
|
||||||
if partRaw != "" {
|
if partRaw != "" {
|
||||||
functionResponseContent, _ = sjson.SetRaw(functionResponseContent, "parts.-1", partRaw)
|
functionResponseContent, _ = sjson.SetRaw(functionResponseContent, "parts.-1", partRaw)
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -261,10 +261,7 @@ func fixCLIToolResponse(input string) (string, error) {
|
|||||||
log.Warnf("failed to parse function response")
|
log.Warnf("failed to parse function response")
|
||||||
continue
|
continue
|
||||||
}
|
}
|
||||||
raw := response.Raw
|
raw := backfillFunctionResponseName(response.Raw, group.CallNames[ri])
|
||||||
if ri < len(group.CallNames) {
|
|
||||||
raw = backfillFunctionResponseName(raw, group.CallNames[ri])
|
|
||||||
}
|
|
||||||
functionResponseContent, _ = sjson.SetRaw(functionResponseContent, "parts.-1", raw)
|
functionResponseContent, _ = sjson.SetRaw(functionResponseContent, "parts.-1", raw)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user