add option to send user message as part of request body in assistant …

[ethayu]

Add support for to send user message/query within assistant invocations, as referenced in #1302.

[ethayu]

@nsarrazin please let me know your thoughts

[nsarrazin]

Hi! Thanks for the contrib!

Do you think we could instead do something like {{url|post=https://example.com}} so that we can keep the old functionality as well ?

[ethayu]

@nsarrazin about keeping the old functionality:
my changes don't remove any existing functionality - it just makes it such that if a new env variable ENABLE_ASSISTANTS_POST is set to true, any call to {{url=https://example.com}} will be a POST request instead of a GET request, with the user prompt sent as the message body.

could you approve the workflow again? I had to resolve some merge conflicts.

[nsarrazin]

it just makes it such that if a new env variable ENABLE_ASSISTANTS_POST is set to true, any call to {{url=https://example.com}} will be a POST request instead of a GET

Yep, my point was that if we use a different keyword like url|post instead of setting it with an env variable then we can support both GET and POST in the same config 😄

[ethayu]

Yes, that actually makes more sense. I’ll do that.

[nsarrazin]

Hi @ethayu do you still want to take a look at this? Just curious no pressure 😄

[ethayu]

@nsarrazin sorry, I've been a little swamped with school recently! I'll get to it around thanksgiving break, if nobody else has taken care of it by then

[ethayu]

@nsarrazin done! please take a look 😄

[ethayu]

@nsarrazin lint issue fixed 😂

[ethayu]

@nsarrazin Could you kick off the checks please?