Skip to content

Fix model selection, cleanup dead schema, and rework Assist agent routing#18

Merged
techartdev merged 1 commit intomainfrom
papurha/safe-fixes-and-reworks
Apr 4, 2026
Merged

Fix model selection, cleanup dead schema, and rework Assist agent routing#18
techartdev merged 1 commit intomainfrom
papurha/safe-fixes-and-reworks

Conversation

@techartdev
Copy link
Copy Markdown
Owner

Summary

This PR bundles the low-risk fixes that are clearly ready plus a conservative rework of the useful parts of earlier proposals.

Included fixes

  • fix the Active Model select so its value is actually passed to chat-card and Assist requests
  • remove the dead attachments field from the openclaw.send_message schema/docs/translations
  • align chat-card cache-busting URLs with the current integration version
  • fix manifest documentation / issue tracker links so they point to the integration repo
  • move event bus subscriptions into async_added_to_hass for cleaner HA entity lifecycle behavior
  • extract duplicated recursive response-text parsing into a shared helper
  • add a warning when the API client has to fall back to creating its own aiohttp session
  • conservatively namespace Assist conversation IDs by agent ID to avoid cross-agent context bleed without introducing sticky persistent sessions

Issue / PR mapping

Fixes or addresses

Explicit rework (not merged verbatim)

Not included here

Why this shape

The goal was to land the clearly-correct fixes now and only rework the risky bits where there was a clean, low-blast-radius version worth keeping.

Manual test checklist

  • Select a model via the Active Model entity and verify chat-card requests use it
  • Select a model and verify Assist requests use it
  • Confirm different Assist agent IDs no longer share the same conversation ID namespace
  • Confirm openclaw.send_message no longer advertises unsupported attachments
  • Confirm event entities still fire correctly in the automation UI
  • Confirm the chat card reloads after update without stale cached JS

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.

1 participant