PR #11494: feat(agent): add z.ai/GLM-5 preserved thinking support

• Repository: NousResearch/hermes-agent
• Author: neuneu2k
• State: open | merged: False
• Link: https://github.com/NousResearch/hermes-agent/pull/11494

Description (problem / solution / changelog)

Enable z.ai/Zhipu GLM-5.x and GLM-4.7 preserved thinking mode for multi-turn agent loops.

Three changes in run_agent.py:

1. _is_zai_direct() helper — detects zai provider or known z.ai/bigmodel endpoint URLs (api.z.ai, open.bigmodel.cn).

2. _build_api_kwargs() — injects thinking parameter in extra_body for GLM-5/4.7 models:

   • Default: {type: enabled, compact_history: false} (preserved thinking)
   • reasoning_config.enabled=false → {type: disabled}
   • GLM-4.6/4.5 excluded (they auto-determine thinking)

3. Message sanitization — re-injects reasoning_content on assistant messages for z.ai so multi-turn reasoning continuity works with compact_history=false.

Response-side extraction was already handled by the generic _extract_reasoning() method (checks reasoning_content field).

Tests: 19 new tests covering detection, parameter injection, config gating, and multi-turn passthrough.

What does this PR do?

The GLM 5 family, and to a lesser degree the 4.7 line, has been trained on preserved interleaved thinking, It's supposed to improve chained tool calling by keeping the reasoning steps in context instead as a short term memory.

This PR enables preserved thinking mode on z.ai models if and only if they are served directly from their inference endpoints.

Related Issue

Fixes Preserved thinking for GLM models when the inference provider supports it.

Type of Change

• 🐛 Bug fix (non-breaking change that fixes an issue)
• ✨ New feature (non-breaking change that adds functionality)
• 🔒 Security fix
• 📝 Documentation update
• ✅ Tests (adding or improving test coverage)
• ♻️ Refactor (no behavior change)
• 🎯 New skill (bundled or hub)

Code

• I've read the Contributing Guide
• My commit messages follow Conventional Commits (fix(scope):, feat(scope):, etc.)
• I searched for existing PRs to make sure this isn't a duplicate
• My PR contains only changes related to this fix/feature (no unrelated commits)
• I've run pytest tests/ -q and all tests pass
• I've added tests for my changes (required for bug fixes, strongly encouraged for features)
• I've tested on my platform: Debian GNU/Linux 12 (bookworm)

Documentation & Housekeeping

• I've updated relevant documentation (README, docs/, docstrings) — or N/A
• I've updated cli-config.yaml.example if I added/changed config keys — or N/A
• I've updated CONTRIBUTING.md or AGENTS.md if I changed architecture or workflows — or N/A
• I've considered cross-platform impact (Windows, macOS) per the compatibility guide — or N/A
• I've updated tool descriptions/schemas if I changed tool behavior — or N/A

Changed files

• run_agent.py (modified, +45/-0)
• tests/run_agent/test_zai_thinking.py (added, +260/-0)