Skip to content

fix: recognize gateway launchd wrappers#25883

Closed
schnubb-web wants to merge 1 commit into
NousResearch:mainfrom
schnubb-web:fix/gateway-wrapper-doctor-checks
Closed

fix: recognize gateway launchd wrappers#25883
schnubb-web wants to merge 1 commit into
NousResearch:mainfrom
schnubb-web:fix/gateway-wrapper-doctor-checks

Conversation

@schnubb-web

Copy link
Copy Markdown

Summary

  • use Gemini's native x-goog-api-key header for doctor model connectivity checks
  • treat profile-local launchd wrapper scripts as current when they delegate to the active Hermes gateway run command
  • add regression coverage for accepted and rejected launchd wrapper scripts

Test Plan

  • python -m pytest tests/hermes_cli/test_update_gateway_restart.py tests/hermes_cli/test_doctor.py -q

- Use Gemini API-key auth for doctor model connectivity checks
- Treat profile-local launchd wrappers as current when they delegate to gateway run --replace
- Add regression coverage for accepted and rejected wrapper scripts
@alt-glitch alt-glitch added type/bug Something isn't working comp/cli CLI entry point, hermes_cli/, setup wizard comp/gateway Gateway runner, session dispatch, delivery provider/gemini Google Gemini (AI Studio, Cloud Code) P3 Low — cosmetic, nice to have labels May 14, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

The Gemini x-goog-api-key doctor fix is a duplicate of #20642 (5th+ PR with this fix). The launchd wrapper recognition is new value-add. Consider splitting or rebasing on #20642 once it lands.

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 comp/gateway Gateway runner, session dispatch, delivery P3 Low — cosmetic, nice to have provider/gemini Google Gemini (AI Studio, Cloud Code) type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants