PR #17401: fix(auxiliary_client): preserve /anthropic/v1 path for MiniMax provider

• Repository: NousResearch/hermes-agent
• Author: lxing1988
• State: closed | merged: False
• Link: https://github.com/NousResearch/hermes-agent/pull/17401

Description (problem / solution / changelog)

Bug: MiniMax /anthropic/v1 URL incorrectly rewritten to /v1

Problem

MiniMax 的 Anthropic-compatible endpoint 是 /anthropic/v1/messages（固定两段式路径）。

但 _to_openai_base_url() 函数错误地将其重写：

base_url: https://api.minimaxi.com/anthropic
        ↓
_rewrite: 去掉 /anthropic → https://api.minimaxi.com/v1
        ↓
实际请求: https://api.minimaxi.com/v1/messages → 404
正确路径:  https://api.minimaxi.com/anthropic/v1/messages → 200

Root Cause

auxiliary_client.py 第 322 行：

if url.endswith("/anthropic"):
    rewritten = url[: -len("/anthropic")] + "/v1"

把 /anthropic 错误地重写为 /v1，但 MiniMax 需要保留 /anthropic/v1 完整路径。

Fix

只在 URL 恰好以 /anthropic 结尾（且不含 /v1 后缀）时才重写：

if url.endswith("/anthropic") and not url.endswith("/anthropic/v1"):

Testing

修复后 MiniMax provider 正常工作，auxiliary client 请求发到正确的 /anthropic/v1/messages 端点。

Changed files

• agent/auxiliary_client.py (modified, +10/-1)

PR #17467: fix(auxiliary): pass raw base_url to _maybe_wrap_anthropic for correct transport detection

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

Description (problem / solution / changelog)

Summary

Fixes HTTP 404 errors when using Anthropic-compatible providers (Kimi Coding, MiniMax) for auxiliary tasks (title generation, etc.).

Root Cause

_to_openai_base_url() rewrites Anthropic-protocol URLs to OpenAI-compatible format:

• /anthropic → /v1
• /coding → /coding/v1

However, _maybe_wrap_anthropic() needs the original URL to correctly detect whether the endpoint speaks Anthropic Messages protocol via _endpoint_speaks_anthropic_messages().

Before this fix:

• MiniMax: /anthropic rewritten to /v1, then _endpoint_speaks_anthropic_messages(/v1) returned False → no Anthropic wrapping → 404 on /v1/chat/completions
• Kimi Coding: /coding rewritten to /coding/v1, but _endpoint_speaks_anthropic_messages still matched due to /coding in URL, causing double /v1 confusion

After this fix:

• OpenAI client uses the rewritten URL (correct for chat.completions)
• _maybe_wrap_anthropic uses the raw URL for transport detection (correct for protocol selection)

Changes

In agent/auxiliary_client.py, 4 locations now preserve raw_base_url:

1. _resolve_api_key_provider() — pool path
2. _resolve_api_key_provider() — API key path
3. resolve_provider_client() — named custom provider
4. resolve_provider_client() — API-key providers

Test Plan

• Verify title generation works with Kimi Coding provider
• Verify auxiliary calls work with MiniMax provider
• Ensure no regression for OpenAI-format providers

Related

Fixes the "⚠ Auxiliary title generation failed: HTTP 404" error reported when using api.kimi.com/coding or MiniMax as the auxiliary provider.

Changed files

• agent/auxiliary_client.py (modified, +19/-13)

PR #17528: fix(agent): use /anthropic endpoint for MiniMax-CN auxiliary tasks

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

Description (problem / solution / changelog)

Closes #17387.

Summary

auxiliary_client._to_openai_base_url() unconditionally rewrites /anthropic → /v1 so the OpenAI SDK can hit chat.completions. For most providers that's correct — MiniMax exposes both surfaces and the OpenAI-compatible one lives at /v1. But MiniMax-CN's /v1 endpoint does not support auxiliary tasks (title generation, context compression, summarization), so every aux request 404s.

MiniMax-CN only serves those endpoints via the /anthropic surface, and we already have an AnthropicAuxiliaryClient wrapper (_maybe_wrap_anthropic) ready to consume that shape.

Fix

Thread a provider= kwarg into _to_openai_base_url. When the provider is in _ANTHROPIC_COMPAT_PROVIDERS (currently {"minimax", "minimax-cn"}), skip the /anthropic → /v1 rewrite so _maybe_wrap_anthropic can detect and route through the Anthropic Messages API instead.

-def _to_openai_base_url(base_url: str) -> str:
+def _to_openai_base_url(base_url: str, *, provider: str = "") -> str:
     url = str(base_url or "").strip().rstrip("/")
+    if provider and provider in _ANTHROPIC_COMPAT_PROVIDERS:
+        return url  # preserve /anthropic — _maybe_wrap_anthropic will handle it
     if url.endswith("/anthropic"):
         rewritten = url[: -len("/anthropic")] + "/v1"
         …

All 6 call sites in agent/auxiliary_client.py updated to pass the provider they already have in local scope. Behavior unchanged for every non-MiniMax provider; the existing rewrite still fires when provider="", "openai", "custom", etc.

Tests

Added TestToOpenaiBaseUrl in tests/agent/test_auxiliary_client.py with 7 regression cases:

• default rewrite behavior (preserved)
• provider-gated preservation for minimax and minimax-cn
• empty-provider fallback (preserved)
• trailing-slash normalization
• unrelated provider (openai) still gets the rewrite

Local inline verification of the logic: 6/6 pass. (Full pytest run requires Python 3.10+; my local env is 3.9 so CI will cover the test suite.)

Verification path

With this PR, an aux request on MiniMax-CN ends up on /anthropic/messages via AnthropicAuxiliaryClient instead of /v1/chat/completions (404). Compression / title generation / summarization on minimax-cn should recover.

Files

• agent/auxiliary_client.py: +10 / -3 (function signature + 6 call sites)
• tests/agent/test_auxiliary_client.py: +87

Changed files

• agent/auxiliary_client.py (modified, +22/-7)
• tests/agent/test_auxiliary_client.py (modified, +75/-0)