Update logging and code structure for duo agent widget in sidebar
The following discussions from !204072 (merged) should be addressed:
-
@dstull started a discussion: suggestion (non-blocking):
In a follow-up, it seems like this method isn't 'common' and should move to the 1 presenter that uses it.
-
@dstull started a discussion: suggestion (non-blocking):
In a follow-up, we should think about where this message is going... out into the void right now as we aren't even consuming it in frontend, which is fine...
But maybe we should do a logging of error here like https://gitlab.com/gitlab-org/gitlab/blob/fddbd599d38894eb1bd717db2bcb10ef44c50e5b/ee/app/services/ai/agents/update_platform_service.rb#L25-25
-
@dstull started a discussion: question (non-blocking):
Just for my own info, how does javascript do with us sometimes not having
hasRequestedorrequestCount?This is I think one of those reasons, even if handled gracefully, it would be probably an opportunity to always send an object. Perhaps this could be something like
userAttributesthat is an object with all 2 or 3.Something to consider in a follow-up.