Skip to content

fix(gateway): load nested gateway.platforms config - #34565

Closed
Que0x wants to merge 2 commits into
NousResearch:mainfrom
Que0x:fix/gateway-nested-platforms-config-loader
Closed

fix(gateway): load nested gateway.platforms config#34565
Que0x wants to merge 2 commits into
NousResearch:mainfrom
Que0x:fix/gateway-nested-platforms-config-loader

Conversation

@Que0x

@Que0x Que0x commented May 29, 2026

Copy link
Copy Markdown
Contributor

Summary

Fix gateway config loading so platform settings defined under gateway.platforms.<name> are honored at runtime.

Previously, load_gateway_config() only merged the top-level platforms: block, while CLI messaging and documentation directed users to configure bundled gateway platforms under gateway.platforms.<name>. As a result, fields like enabled, extra, and home_channel could be silently ignored when written to the documented path.

This change adds support for nested gateway.platforms loading while preserving existing precedence: top-level platforms: still overrides gateway.platforms: when both are present.

Why

This aligns runtime behavior with the documented configuration path and avoids a silent misconfiguration footgun in gateway setup.

Testing

  • tests/gateway/test_config.py50 passed in 2.84s
  • tests/test_gateway_streaming_nested_config.py3 passed in 0.29s

@alt-glitch alt-glitch added type/bug Something isn't working comp/gateway Gateway runner, session dispatch, delivery area/config Config system, migrations, profiles P2 Medium — degraded but workaround exists labels May 29, 2026
@teknium1

Copy link
Copy Markdown
Contributor

Merged via #35329 (#35329). Your commit was cherry-picked onto current main with authorship preserved (0bfe19b). Combined with #34557 (same subsystem). Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/config Config system, migrations, profiles comp/gateway Gateway runner, session dispatch, delivery 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