fix: include plugin platforms in status - #44255
Conversation
Trigger plugin discovery before reading platform_registry.plugin_entries() in hermes status so third-party platform adapters registered by plugins appear in the Messaging Platforms section.\n\nAdd regression coverage with a plugin-registered test platform.\n\nCloses NousResearch#44119
✅ Verification: Clean ReviewReviewed this PR's diff. The fix correctly addresses the missing plugin platforms in
No issues found. |
|
Thanks for the focused regression fix. The premise remains present on current main: The proposed call matches the established enumeration pattern in No blocking correctness or design-fit issues found. The member discussion identifies #44124 as the earlier duplicate, but current main still needs this behavioral guarantee. Automated hermes-sweeper review. |
Summary
hermes statusreads plugin-registered platform entriesTest Plan
/home/rivuza/hermes-agent/.venv/bin/python -m pytest tests/hermes_cli/test_status.py::test_show_status_includes_plugin_registered_platform -q/home/rivuza/hermes-agent/.venv/bin/python -m pytest tests/hermes_cli/test_status.py -qCloses #44119