Skip to content

fix: sanitize chat payloads and provider precedence#1253

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

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

Conversation

@teknium1
Copy link
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
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