Skip to content

feat(expert): send insights chats over MQTT - #9074

Open
andypalmi wants to merge 2 commits into
refactor/expert-chat-publisherfrom
feat/insights-chat-mqtt
Open

andypalmi wants to merge 2 commits into
refactor/expert-chat-publisherfrom
feat/insights-chat-mqtt

Conversation

@andypalmi

Copy link
Copy Markdown
Contributor

Description

  1. With expertInsightsMqtt on, insights chats go through an insights chat subscriber and publisher on their own connection and the insights channel, with the selected capabilities in the request context.
  2. Tool calls, reasoning and progress stream into the reply like support.
  3. Insights tracks its own stopped turns. A drop of the connection of the agent not on screen reconnects quietly, and instance notices still go only to support.
  4. HTTP stays the transport when the flag is off.

End to end it needs the flag (#9071), the broker ACLs and bridge (#9070) and the central broker topics (FlowFuse/engineering#589).

Related Issue(s)

Closes #9034

Checklist

  • I have read the contribution guidelines
  • Suitable unit/system level tests have been added and they pass
  • Documentation has been updated
    • Upgrade instructions
    • Configuration details
    • Concepts
  • Changes flowforge.yml?
    • Issue/PR raised on FlowFuse/helm to update ConfigMap Template
    • Issue/PR raised on FlowFuse/CloudProject to update values for Staging/Production
  • Link to Changelog Entry PR, or note why one is not needed.
    • Not needed yet: insights stays on HTTP until the rollout (FlowFuse/engineering#590).

Labels

  • Includes a DB migration? -> add the area:migration label

With expertInsightsMqtt on, insights chats go through an insights chat
subscriber and publisher on their own connection and the insights channel,
with the selected capabilities in the request context, and stream tool calls,
reasoning and progress like support. HTTP stays the transport when the flag is
off.

Insights tracks its own stopped turns. Connection notices only talk to the
chat on screen, so a drop of the other agent's connection reconnects quietly.
Instance notices still go only to the support agent.
@andypalmi andypalmi self-assigned this Oct 27, 2026
@andypalmi
andypalmi requested review from Steve-Mcl and cstns October 27, 2026 16:44
@andypalmi
andypalmi added this pull request to stack #9075 October 27, 2026 16:44
@codecov

codecov Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.62%. Comparing base (2848f75) to head (4f1dbcb).

Additional details and impacted files
@@                       Coverage Diff                       @@
##           refactor/expert-chat-publisher    #9074   +/-   ##
===============================================================
  Coverage                           79.62%   79.62%           
===============================================================
  Files                                 475      475           
  Lines                               26086    26086           
  Branches                             6954     6954           
===============================================================
  Hits                                20770    20770           
  Misses                               5316     5316           
Flag Coverage Δ
backend 79.62% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@andypalmi
andypalmi removed this pull request from stack #9075 October 27, 2026 17:46
@andypalmi
andypalmi added this pull request to stack #9076 October 27, 2026 17:46

This branch was successfully deployed

1 active deployment
staging — 4f1dbcb1 Deployed Oct 27, 2026 by andypalmi via Deploy application #12789
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.

Send insights chat over MQTT

1 participant