Skip to content

Add core controller for discovery#3378

Open
marcelveldt-traveling wants to merge 2 commits intomusic-assistant:devfrom
marcelveldt-traveling:add-discovery-controller
Open

Add core controller for discovery#3378
marcelveldt-traveling wants to merge 2 commits intomusic-assistant:devfrom
marcelveldt-traveling:add-discovery-controller

Conversation

@marcelveldt-traveling
Copy link

This PR moves network discovery out of mass and individual providers into a shared discovery core controller.

  • adds upnp_discovery to ProviderManifest, similar to existing mdns_discovery
  • centralizes shared Zeroconf browsing, SSDP discovery, and _mass._tcp server advertisement
  • routes SSDP discovery results back into providers through a common callback
  • moves the UPnP broadcast/network-scan option to the discovery core config
  • updates the existing DLNA and Roku providers to use the new discovery flow
  • narrows DLNA SSDP matching to the MediaRenderer device target instead of ssdp:all
  • adds a README and tests for the new controller

The result is a cleaner discovery model that new UPnP/DLNA-based providers can subscribe to without re-implementing their own discovery loop.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 13, 2026

🔒 Dependency Security Report

✅ No dependency changes detected in this PR.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants