Problem

Two related bugs in the flush_memories Codex path cause "Auxiliary memory flush failed" with HTTP 400 errors.

Bug 1 — temperature in fallback path

When flush_memories is in codex_responses mode and the auxiliary client fails for any reason, the fallback path explicitly sets codex_kwargs["temperature"] = _flush_temperature (0.3) before calling _run_codex_stream. The Codex Responses endpoint does not accept temperature and returns:

HTTP 400: Unsupported parameter: temperature

The error propagated to the outer except Exception as e: and surfaced as "Auxiliary memory flush failed" — misleadingly implicating temperature rather than the actual aux failure.

The bug was latent before 8a2506af (which widened except RuntimeError → except Exception as e:). After that commit the inner failure started triggering the fallback path more frequently, exposing this.

Bug 2 — unsupported message roles in adapter

_CodexCompletionsAdapter.create() passes all non-system message roles through to the Responses API unchanged. The Codex endpoint only accepts user and assistant in the input array. Sessions that include memory tool results (prior flush calls) contain role: 'tool' messages, causing:

HTTP 400: Invalid value: 'tool'. Supported values are: 'assistant', 'system', 'developer', and 'user'.

Root Cause

Bug 1 — run_agent.py, flush_memories Codex fallback:

if _flush_temperature is not None:
    codex_kwargs["temperature"] = _flush_temperature  # ← sends to Codex endpoint

Bug 2 — agent/auxiliary_client.py, _CodexCompletionsAdapter.create():

for msg in messages:
    role = msg.get("role", "user")
    # ...
    else:
        input_msgs.append({"role": role, ...})  # ← passes role='tool' through

Note: _CodexCompletionsAdapter already correctly omits temperature from resp_kwargs. The fallback path failed to do the same.

Fix

Bug 1: Strip temperature unconditionally from codex_kwargs before _run_codex_stream.

Bug 2: In _CodexCompletionsAdapter, skip role='tool' and role='function' messages; map role='developer' to instructions; silently skip unknown roles.

Tests

Existing test_codex_mode_no_aux_uses_responses_api in tests/run_agent/test_flush_memories_codex.py covers the fallback path.