feat: replace assistant handlers with agent argument methods#51
Conversation
| from logging import Logger | ||
|
|
||
| from slack_bolt import Say, SetSuggestedPrompts | ||
| from slack_bolt import BoltAgent, Say |
| agent: BoltAgent for making API calls | ||
| client: Slack WebClient for making API calls | ||
| context: Bolt context containing channel and thread information | ||
| logger: Logger instance for error tracking | ||
| payload: Event payload with message details (channel, user, text, etc.) | ||
| message: Dictionary with message information | ||
| say: Function to send messages to the thread | ||
| set_status: Function to update the assistant's status | ||
| """ |
There was a problem hiding this comment.
Thanks for showcases how the agent argument methods may look in the Assistant sample app! 🙇🏻
However, let's hold off on merging this. The agent is an under-developer, experimental feature that's unreleased. We don't want to be adding to our main assistant/agent sample app right now.
If this sounds good to you, we can close this PR without merging 📕
Edit: Just noticed that you're merging into feat-agent-argument. I suppose that's fine if you want to merge it, but I was going to delete that branch today. It was only intended to showcase a working example to test the argent argument PR.
mwbrooks
left a comment
mwbrooks
left a comment
There was a problem hiding this comment.
✅ Since this is going into a branch, I think we can merge it okay. Sorry for putting a halt on it with the change requested. At first, I thought this was going into main 🤦🏻 My bad!
|
@srtaalej @mwbrooks I appreciate the kind reviews both so much 👾 ✨ Let's get this merged toward upcoming developments adjacent! I realize these changes are in progress when I requested review too - apologies for possible confusions to this... If changes upstream happen before more review I'll be sure to follow up 🫡 |
3 participants
Type of change
Summary
This PR replaces
set_suggested_promptsandset_statusmethods from assistant handlers with the "agent" argument methods 🤖Tested with the following PRs built:
Requirements