Can anyone please review the PR for exposing event...
# proj-agent-canvas
m
Can anyone please review the PR for exposing event trigger details in the automation UI? https://github.com/OpenHands/agent-canvas/pull/1007
h
Awesome! Thanks @mammoth-pillow-18568. One thing that I think should be changed just from the screenshot is how Github is shown as a plugin. That may seem like the correct grouping but I think it is more clear to remove it since github is already in the event source field. Plugins right now don't have as clear of a definition inside of agent canvas as they will eventually. They follow the pattern shown in the extensions repo here https://github.com/OpenHands/extensions/tree/main/plugins and more generally documented in https://docs.openhands.dev/overview/plugins#plugins. Aside from that were you able to test this with an automation?
m
Thanks for the feedback! Quick clarification — should I remove "GitHub" from the plugins section only when it matches the event source (i.e. filter it out from the list), or should the plugins section not appear at all for event-triggered automations? Also, regarding testing — I tested with mock data using npm run dev:mock. I don't have a running automation service set up locally yet.
h
I'd say remove plugins entirely for now
m