Skip to content

fix: sanitize chat payloads and provider precedence#1253

Merged
teknium1 merged 4 commits into
mainfrom
hermes/hermes-9b4d5070
Mar 14, 2026
Merged

fix: sanitize chat payloads and provider precedence#1253
teknium1 merged 4 commits into
mainfrom
hermes/hermes-9b4d5070

Conversation

@teknium1

Copy link
Copy Markdown
Contributor

Summary

  • salvage the substantive fixes from fix: sanitize chat payloads and provider precedence #903 onto current main
  • preserve Adavya Sharma's authored commits for the payload sanitization + test additions
  • keep the provider precedence fix so saved config provider wins over stale env overrides
  • keep the checkpoint test lock swap for sandboxed CI
  • omit the stale top-level skills-config rename because current main already uses hermes skills config
  • add a tiny follow-up docstring clarification for the updated provider precedence

Contributor credit

This PR salvages the substantive work from #903 by @adavyas and preserves their authored commits where applicable.

Test plan

  • source /home/teknium/.hermes/hermes-agent/.venv/bin/activate
  • python -m pytest tests/test_provider_parity.py tests/test_runtime_provider_resolution.py tests/test_cli_provider_resolution.py tests/test_batch_runner_checkpoint.py -n0 -q
  • python -m hermes_cli.main --help
  • python -m hermes_cli.main skills config --help

Notes

@teknium1 teknium1 merged commit 29176f3 into main Mar 14, 2026
1 check passed
@teknium1 teknium1 deleted the hermes/hermes-9b4d5070 branch March 14, 2026 07:09
angelburgosrosado pushed a commit to angelburgosrosado/hermes-agent that referenced this pull request Apr 27, 2026
fix: sanitize chat payloads and provider precedence
02356abc pushed a commit to 02356abc/hermes-agent that referenced this pull request May 14, 2026
fix: sanitize chat payloads and provider precedence
olympus-terminal pushed a commit to olympus-terminal/hermes-agent that referenced this pull request May 16, 2026
fix: sanitize chat payloads and provider precedence
Egavasyug pushed a commit to Egavasyug/hermes-agent that referenced this pull request Jun 10, 2026
fix: sanitize chat payloads and provider precedence
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Strict OpenAI-compatible chat completions endpoints reject replayed tool_calls.call_id / response_item_id fields

2 participants