Cairn
NousResearch/hermes-agent

Removing empty tool_calls in repair_message_sequence

Cluster 77921 · agent. Data as of

Nothing open on GitHub. Fix merged in #86654. 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 #86654. Check it covers your case. No release data is tracked.
Tracker
The maintainers' tracker marks this resolved, deciding PR #86654. That is separate from the fix event.
Still open
Nothing open on GitHub.

These items address a bug where empty tool_calls arrays persist in message sequences after repair_message_sequence runs. Fixes and a follow-up reproduction report are included.

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.

  • #86654 fix(agent): close the empty tool_calls 400 class at all three chokepoints (#83312, #77921)

    merged to main

    files: agent/agent_runtime_helpers.py, agent/transports/chat_completions.py, tests/agent/transports/test_chat_completions_empty_tool_calls.py, tests/run_agent/test_message_sequence_repair.py

  • #97167 fix(agent): tool-pair sanitizers — whitespace ids pair, stale empty tool_calls drop, orphan logic unified (salvage #55845 + #78063 + #59434)

    merged to main

    files: agent/agent_runtime_helpers.py, agent/context_compressor.py, tests/agent/test_context_compressor.py, tests/run_agent/test_message_sequence_repair.py

2 PRs closed without merging

From the close comments: merged via another change 1 · superseded by another PR 1

  • #77944 fix(session): drop empty tool_calls in repair_message_sequence (#77921) · superseded by another PR → #78063, #77944
  • #78063 fix(agent): drop stale empty tool_calls on repair_message_sequence merge (#77921) · merged via another change → f316f7d, #97167, #77921, #78071
Closed issues · 1
  • Closed#77921Still reproducing in v0.19.1: empty tool_calls after repair_message_sequence (follow-up to #58755)
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
#86654, #97167
Closed issues
1
Attempts
2 PRs closed without merging
Fix link checked
28 Aug 2026

For your agent

The same facts, typed.

cluster/77921.json
cluster/77921.md