mirror of
https://github.com/thedotmack/claude-mem.git
synced 2026-09-20 04:23:02 +08:00
Alex Newman
135ac08a9b
All three server generation providers already accept maxOutputTokens, but nothing ever populated it, so every request went out with the constructor default of 4096. On a model that answers at length the reply is truncated mid-structure, the observation parser rejects it, and the job settles as a non-retryable parse_error, so the observation is lost silently. Unset keeps 4096, so behaviour is unchanged unless the variable is set. Non-integer and non-positive values are ignored with a warning rather than sent as NaN. Co-authored-by: escooterclinic <dvla5@escooterclinic.co.uk>