fix(runtime): pass model through to the LLM API (+ UTF-8 JSON escaping) #53

Merged

will.anderson merged 3 commits from fix/llm-model-and-utf8 into stage 2026-06-10 22:01:52 +00:00

Conversation 1 Commits 3 Files Changed 1

+43 -14

3 Commits

Author	SHA1	Message	Date
Tim Lingo	dbf2c659d9	fix(runtime): pass model through to the LLM API instead of dropping it ... llm_call_system / llm_call accepted a model argument and discarded it: they called llm_chain_call(system, user) with no model, and the legacy ANTHROPIC_API_KEY fallback passed NULL to llm_provider_request, so every non-agentic chat was pinned to LLM_DEFAULT_MODEL (claude-sonnet-4-5) regardless of the caller's selection. Thread model_pref through llm_chain_call: provider-chain entries still honor their own NEURON_LLM_N_MODEL override and fall back to the requested model otherwise; the legacy Anthropic path now uses the requested model. NULL/empty preserves prior default behavior. Effect: the soul's model selection (state soul_model / SOUL_LLM_MODEL, e.g. claude-opus-4-8) now reaches api.anthropic.com. Previously the chat response echoed the selected model in its label while the request billed Sonnet 4.5. Not built locally (no elc/cc toolchain on this checkout); needs stage CI. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>	2026-06-09 08:03:56 -05:00
Tim Lingo	2b8062c55f	fix(runtime): handle multi-byte UTF-8 in JSON string escaping ... Validate UTF-8 continuation bytes in jb_emit_escaped; pass valid sequences through and escape orphaned/invalid start bytes as \u00xx. Pre-existing change found uncommitted in the working tree; committed here so it is reviewable rather than lost. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>	2026-06-09 08:02:46 -05:00
will.anderson	2ed6b26dde	Merge pull request 'promote: stage → main (all elb linker fixes + ci-base rebuild)' (#42) from stage into main ... promote: stage → main (all elb linker fixes + ci-base rebuild)	2026-05-07 14:25:37 +00:00