Skip to content

fix: stop doctor flagging custom provider#15159

Closed
OnTheDotMediaLtd wants to merge 1 commit into
NousResearch:mainfrom
OnTheDotMediaLtd:fix/doctor-custom-provider
Closed

fix: stop doctor flagging custom provider#15159
OnTheDotMediaLtd wants to merge 1 commit into
NousResearch:mainfrom
OnTheDotMediaLtd:fix/doctor-custom-provider

Conversation

@OnTheDotMediaLtd

Copy link
Copy Markdown

Summary

  • stop hermes doctor from falsely flagging model.provider: custom as unrecognised
  • preserve validation for genuinely unknown providers
  • add regression coverage for builtin custom and alias ai-gateway providers

Verification

  • /home/andreborg/.hermes/hermes-agent/venv/bin/python -m pytest tests/hermes_cli/test_doctor.py -q
  • hermes doctor

@alt-glitch alt-glitch added type/bug Something isn't working P2 Medium — degraded but workaround exists comp/cli CLI entry point, hermes_cli/, setup wizard labels Apr 24, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Related to #15083 (merged) — similar false-positive provider validation fix, this one targets the builtin 'custom' and 'ai-gateway' provider names in hermes doctor.

@teknium1

Copy link
Copy Markdown
Contributor

Closing as superseded by #15823.

Triage notes (high confidence):
Merged PR #15823 'fix(doctor): accept bare custom provider' (2026-04-26) applied the same fix; origin/main hermes_cli/doctor.py:594 now does provider not in {'auto','custom'} so bare custom is no longer flagged.

Thanks for the contribution — the underlying problem this PR addresses has been resolved by the linked PR on current main. If you believe this was closed in error, please comment and we'll reopen.

(Bulk-closed during a CLI PR triage sweep.)

@teknium1 teknium1 closed this May 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/cli CLI entry point, hermes_cli/, setup wizard P2 Medium — degraded but workaround exists type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants