Skip to content

feat(homeassistant): split tool and platform env config#21676

Closed
StartupBros wants to merge 1 commit into
NousResearch:mainfrom
StartupBros:startupbros/homeassistant-tool-platform-env
Closed

feat(homeassistant): split tool and platform env config#21676
StartupBros wants to merge 1 commit into
NousResearch:mainfrom
StartupBros:startupbros/homeassistant-tool-platform-env

Conversation

@StartupBros

Copy link
Copy Markdown

Summary

  • Add HASS_TOOL_URL / HASS_TOOL_TOKEN for the Home Assistant tool surface.
  • Add HASS_PLATFORM_URL / HASS_PLATFORM_TOKEN for the Home Assistant websocket platform.
  • Add HASS_PLATFORM_DISABLE=1 to allow tool-only Home Assistant setups.
  • Preserve legacy HASS_URL / HASS_TOKEN as the shared fallback for both surfaces.
  • Clean up partially opened Home Assistant websocket resources when connect fails midway.

Why

Some deployments want Hermes tools to talk to a local Home Assistant proxy while not enabling the gateway websocket platform, or want the platform and tools to use different credentials/URLs. Today both surfaces are forced through the same HASS_URL / HASS_TOKEN values.

This keeps existing behavior by default while allowing safer tool-only/proxy deployments.

Test plan

uv run --extra dev pytest \
  tests/gateway/test_homeassistant.py \
  tests/tools/test_homeassistant_tool.py \
  tests/tools/test_send_message_missing_platforms.py -q

Result: 139 passed, 1 existing RuntimeWarning about _async_call_service.

@alt-glitch alt-glitch added type/feature New feature or request comp/gateway Gateway runner, session dispatch, delivery comp/tools Tool registry, model_tools, toolsets area/config Config system, migrations, profiles P3 Low — cosmetic, nice to have labels May 8, 2026
@StartupBros

Copy link
Copy Markdown
Author

Superseded by the split:

Closing in favor of #29199 + #29388.

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 comp/tools Tool registry, model_tools, toolsets P3 Low — cosmetic, nice to have type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants