Cairn
NousResearch/hermes-agent

Remove empty tool_calls key after deduplication

Cluster 70126 · agent. Data as of

Nothing open on GitHub. Fix merged in #90330. Check it covers your case.

The short answer

0open issues
0open PRs
5items grouped
Known?
Yes. 5 related items grouped here, 1 issue already closed.
Fix event
A fix was merged to main in PR #90330. Check it covers your case. No release data is tracked.
Tracker
The maintainers' tracker marks this unresolved. That is separate from the fix event.
Still open
Nothing open on GitHub.

A bug where sanitize_api_messages leaves an empty tool_calls array after deduplication, breaking tool-calling sessions for certain models like Qwen. Fixes involve dropping the tool_calls key entirely when all entries are deduplicated.

Open issues0

No open issues.

Open PRs0

No open pull requests.

Fixes on record

Merged to main. A stamp counts the issues a fix is linked to.

  • #90330 fix: reasoning effort 'ultra' no longer 400s on non-Anthropic wires (#89503 class, salvage #89509)

    merged to main

    files: agent/auxiliary_client.py, agent/transports/chat_completions.py, agent/transports/codex.py, plugins/model-providers/copilot/__init__.py, plugins/model-providers/custom/__init__.py, tests/agent/test_reasoning_effort_wire_translation.py, tests/agent/transports/test_reasoning_effort_sibling_sites.py

3 PRs closed without merging

From the close comments: closed without comment 1 · duplicate 1 · superseded by another PR 1

  • #70176 fix(agent): drop tool_calls key when all tool_calls are deduplicated (#70126) · duplicate → #64345
  • #70181 fix(agent): drop tool_calls key when all tool_calls are deduplicated (#70126) · closed without comment
  • #70182 fix(transports+agent): GLM reasoning_effort ultra + drop empty tool_calls key (#70058, #70126) · superseded by another PR → #90330
Closed issues · 1
  • Closed#70126[Bug]: sanitize_api_messages dedup writes tool_calls: [], breaking Qwen-series tool-calling sessions on v0.18.2+
How this cluster was grouped
  • Link evidence (structural links throughout): every member is connected by structural evidence: shared changed files, or a closing/superseded reference.
  • Pairwise link density 0.40: the share of item pairs that carry a link, counting a structural link in full and a relevance-only link at 0.36. 1.00 means every pair is linked.

Pick up a Lab task

Check whether a fix covers the open issues here, with your own agent. Bring back evidence for maintainers.

Open the Lab

At a glance

Subsystem
agent
Merged PRs
#90330
Closed issues
1
Attempts
3 PRs closed without merging
Fix link checked
19 Aug 2026

For your agent

The same facts, typed.

cluster/70126.json
cluster/70126.md