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 hasRequested or requestCount?

    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 userAttributes that is an object with all 2 or 3.

    Something to consider in a follow-up.

Edited by Doug Stull