红尘
d73ff9b0fb
Deploy Site / deploy-vercel (push) Has been cancelled
Deploy Site / deploy-docs (push) Has been cancelled
Docker / shell lint / Lint Dockerfile (hadolint) (push) Has been cancelled
Docker / shell lint / Lint docker/ shell scripts (shellcheck) (push) Has been cancelled
Docker Build and Publish / build-amd64 (push) Has been cancelled
Docker Build and Publish / build-arm64 (push) Has been cancelled
Lint (ruff + ty) / ruff + ty diff (push) Has been cancelled
Lint (ruff + ty) / ruff enforcement (blocking) (push) Has been cancelled
Lint (ruff + ty) / Windows footguns (blocking) (push) Has been cancelled
Nix Lockfile Fix / auto-fix-main (push) Has been cancelled
Nix Lockfile Fix / fix (push) Has been cancelled
Nix / nix (macos-latest) (push) Has been cancelled
Nix / nix (ubuntu-latest) (push) Has been cancelled
OSV-Scanner / Scan lockfiles (push) Has been cancelled
Build Skills Index / build-index (push) Has been cancelled
Tests / test (1) (push) Has been cancelled
Tests / test (2) (push) Has been cancelled
Tests / test (3) (push) Has been cancelled
Tests / test (4) (push) Has been cancelled
Tests / test (5) (push) Has been cancelled
Tests / test (6) (push) Has been cancelled
Tests / e2e (push) Has been cancelled
uv.lock check / uv lock --check (push) Has been cancelled
Docker Build and Publish / merge (push) Has been cancelled
Build Skills Index / trigger-deploy (push) Has been cancelled
Tests / save-durations (push) Has been cancelled
51 lines
1.9 KiB
Python
51 lines
1.9 KiB
Python
"""Regression: hermes doctor must not run a generic Bearer-auth health
|
|
check for providers that already have a dedicated check (Anthropic,
|
|
OpenRouter, Bedrock).
|
|
|
|
Anthropic's native API requires `x-api-key` + `anthropic-version` headers;
|
|
the generic loop sends `Authorization: Bearer ...` which Anthropic answers
|
|
with HTTP 404. The dedicated check at hermes_cli/doctor.py already covers
|
|
Anthropic with the right headers, so the pluggable profile must be
|
|
skipped by `_build_apikey_providers_list()`.
|
|
|
|
See: NousResearch/hermes-agent#22346
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
|
|
def test_build_apikey_providers_list_skips_dedicated_check_providers():
|
|
from hermes_cli import doctor
|
|
|
|
# Force a rebuild — the module caches the list on first call.
|
|
doctor._APIKEY_PROVIDERS_CACHE = None
|
|
entries = doctor._build_apikey_providers_list()
|
|
|
|
# Tuple shape: (display_name, env_vars, default_url, base_env, supports_health_check)
|
|
names = {entry[0].lower() for entry in entries}
|
|
assert not any("anthropic" in name for name in names), (
|
|
f"Anthropic provider profile leaked into generic Bearer-auth health "
|
|
f"check loop. Dedicated check above already covers it with "
|
|
f"x-api-key headers. Got entries: {sorted(names)}"
|
|
)
|
|
assert not any("openrouter" in name for name in names), (
|
|
f"OpenRouter has a dedicated check; generic loop must skip it. "
|
|
f"Got: {sorted(names)}"
|
|
)
|
|
assert not any("bedrock" in name for name in names), (
|
|
f"Bedrock uses AWS SDK creds, not Bearer auth; generic loop must skip. "
|
|
f"Got: {sorted(names)}"
|
|
)
|
|
|
|
|
|
def test_build_apikey_providers_list_includes_non_dedicated_providers():
|
|
"""Sanity guard: the skip-set must not strip every provider."""
|
|
from hermes_cli import doctor
|
|
|
|
doctor._APIKEY_PROVIDERS_CACHE = None
|
|
entries = doctor._build_apikey_providers_list()
|
|
|
|
names = {entry[0] for entry in entries}
|
|
assert "DeepSeek" in names
|
|
assert "Z.AI / GLM" in names
|