openclaw - 💡(How to fix) Fix [Bug]: SSE [DONE] terminator incorrectly passed to JSON parser — kills agentTurn cron jobs on stream end [1 comments, 2 participants]

Every agentTurn cron job that calls an LLM fails with a JSON parse error when the SSE stream ends. The SSE terminator data: [DONE] is incorrectly routed to the JSON parser instead of being handled as a terminal marker. Error: From cron runs: status=error, error=Could not parse Anthropic SSE event data: Unexpected token D, [DONE] is not valid JSON; data=[DONE], model=minimax/minimax-m2.5:free Same error across all 5+ jobs.

TL;DR

Check for the SSE terminator [DONE] before attempting to parse the data as JSON to prevent parsing errors.

Guidance

• Verify that the error message Could not parse Anthropic SSE event data: Unexpected token D, [DONE] is not valid JSON; data=[DONE] is consistent across all failed agentTurn cron jobs.
• Implement a check for the [DONE] terminator in the SSE data handler, as proposed in the issue, to prevent unnecessary JSON parsing.
• Review related issues, such as #52679, to ensure that similar edge cases are handled correctly.
• Test the updated SSE data handler with various input scenarios to ensure that it correctly handles both JSON data and the [DONE] terminator.

Example

const raw = line.trim();
if (raw === '[DONE]') { 
  return; 
}
const data = JSON.parse(raw);

Notes

The proposed fix assumes that the [DONE] terminator is always exactly [DONE] and that it is the only non-JSON data that will be encountered. Additional error handling or input validation may be necessary depending on the specific requirements of the application.

Recommendation

Apply the proposed workaround by checking for the [DONE] terminator before attempting to parse the data as JSON, as this directly addresses the root cause of the issue and prevents unnecessary parsing errors.