Skip to content

fix(api-server): pass fallback_model to AIAgent#4976

Closed
bugkill3r wants to merge 1 commit into
NousResearch:mainfrom
bugkill3r:fix/api-server-fallback-4954
Closed

fix(api-server): pass fallback_model to AIAgent#4976
bugkill3r wants to merge 1 commit into
NousResearch:mainfrom
bugkill3r:fix/api-server-fallback-4954

Conversation

@bugkill3r

Copy link
Copy Markdown
Contributor

Summary

Test plan

  • Configure fallback_providers in config.yaml
  • Send a request via the API server while the primary provider is rate-limited
  • Verify fallback activates instead of failing

The API server platform never passed fallback_model to AIAgent(),
so the fallback provider chain was always empty for requests through
the OpenAI-compatible endpoint. Load it via GatewayApp._load_fallback_model()
to match the behavior of Telegram/Discord/Slack platforms.
teknium1 added a commit that referenced this pull request Apr 5, 2026
- Fix GatewayApp → GatewayRunner import in api_server.py (PR #4976)
- Update launchd test assertions for new bootstrap/bootout/kickstart commands (PR #4892)
- Add nonlocal message declaration in run_sync() to fix UnboundLocalError (pre-existing scoping bug)
teknium1 added a commit that referenced this pull request Apr 5, 2026
- Fix GatewayApp → GatewayRunner import in api_server.py (PR #4976)
- Update launchd test assertions for new bootstrap/bootout/kickstart commands (PR #4892)
- Add nonlocal message declaration in run_sync() to fix UnboundLocalError (pre-existing scoping bug)
@teknium1

teknium1 commented Apr 5, 2026

Copy link
Copy Markdown
Contributor

Merged via PR #5288 (consolidated bugfix salvage). Your commit(s) were cherry-picked onto current main with your authorship preserved in git log. Thanks @bugkill3r for the fix!

@teknium1 teknium1 closed this Apr 5, 2026
Tommyeds pushed a commit to Tommyeds/hermes-agent that referenced this pull request Apr 12, 2026
- Fix GatewayApp → GatewayRunner import in api_server.py (PR NousResearch#4976)
- Update launchd test assertions for new bootstrap/bootout/kickstart commands (PR NousResearch#4892)
- Add nonlocal message declaration in run_sync() to fix UnboundLocalError (pre-existing scoping bug)
angelburgosrosado pushed a commit to angelburgosrosado/hermes-agent that referenced this pull request Apr 27, 2026
- Fix GatewayApp → GatewayRunner import in api_server.py (PR NousResearch#4976)
- Update launchd test assertions for new bootstrap/bootout/kickstart commands (PR NousResearch#4892)
- Add nonlocal message declaration in run_sync() to fix UnboundLocalError (pre-existing scoping bug)
angelburgosrosado pushed a commit to angelburgosrosado/hermes-agent that referenced this pull request Apr 28, 2026
- Fix GatewayApp → GatewayRunner import in api_server.py (PR NousResearch#4976)
- Update launchd test assertions for new bootstrap/bootout/kickstart commands (PR NousResearch#4892)
- Add nonlocal message declaration in run_sync() to fix UnboundLocalError (pre-existing scoping bug)
02356abc pushed a commit to 02356abc/hermes-agent that referenced this pull request May 14, 2026
- Fix GatewayApp → GatewayRunner import in api_server.py (PR NousResearch#4976)
- Update launchd test assertions for new bootstrap/bootout/kickstart commands (PR NousResearch#4892)
- Add nonlocal message declaration in run_sync() to fix UnboundLocalError (pre-existing scoping bug)
olympus-terminal pushed a commit to olympus-terminal/hermes-agent that referenced this pull request May 16, 2026
- Fix GatewayApp → GatewayRunner import in api_server.py (PR NousResearch#4976)
- Update launchd test assertions for new bootstrap/bootout/kickstart commands (PR NousResearch#4892)
- Add nonlocal message declaration in run_sync() to fix UnboundLocalError (pre-existing scoping bug)
gweeteve pushed a commit to gweeteve/hermes-agent that referenced this pull request Jun 2, 2026
- Fix GatewayApp → GatewayRunner import in api_server.py (PR NousResearch#4976)
- Update launchd test assertions for new bootstrap/bootout/kickstart commands (PR NousResearch#4892)
- Add nonlocal message declaration in run_sync() to fix UnboundLocalError (pre-existing scoping bug)
Egavasyug pushed a commit to Egavasyug/hermes-agent that referenced this pull request Jun 10, 2026
- Fix GatewayApp → GatewayRunner import in api_server.py (PR NousResearch#4976)
- Update launchd test assertions for new bootstrap/bootout/kickstart commands (PR NousResearch#4892)
- Add nonlocal message declaration in run_sync() to fix UnboundLocalError (pre-existing scoping bug)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: API server platform ignores fallback_providers config

2 participants