Skip to content

feat: set loading states and stream responses #11

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
srtaalej merged 14 commits into from
Sep 24, 2025
Merged

feat: set loading states and stream responses #11

Changes from 2 commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
a00e5c1
docs: clarify that listeners/events is educational only
srtaalej Sep 18, 2025
df8b620
docs: shorten the listeners/events clarification
srtaalej Sep 18, 2025
475139f
refactor: move assistant logic into dedicated folder
srtaalej Sep 19, 2025
e1bc98a
docs: align wording with docs.slack.dev
srtaalej Sep 19, 2025
291ef2d
fix: update imports after file structure refactor
srtaalej Sep 19, 2025
a8389b1
build: black reformat
srtaalej Sep 19, 2025
42cf39f
docs: align wording with docs.slack.dev
srtaalej Sep 22, 2025
2acdaf4
feat: ai app streaming and loading states
srtaalej Sep 22, 2025
5bf3bf4
build: add python sdk beta release to requirements.txt
srtaalej Sep 23, 2025
bfd4649
fix: address pr feedback
srtaalej Sep 23, 2025
530052f
docs: update README title
srtaalej Sep 23, 2025
905f4fb
fix: address pr feedback
srtaalej Sep 23, 2025
0a97ef9
build: update slack_sdk version
srtaalej Sep 24, 2025
b797fdd
build: remove slack_cli hooks from requirements.txt
srtaalej Sep 24, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 9 additions & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,15 @@ black .

### `/listeners`

Every incoming request is routed to a "listener". Inside this directory, we group each listener based on the Slack Platform feature used, so `/listeners/events` handles incoming events, `/listeners/shortcuts` would handle incoming [Shortcuts](https://api.slack.com/interactivity/shortcuts) requests, and so on.
Every incoming request is routed to a "listener". Inside this directory, we group each listener based on the Slack Platform feature used, so `/listeners/events` handles incoming events like messages sent to the app.
Comment thread
srtaalej marked this conversation as resolved.
Outdated

**Note**: The `listeners/events` folder is purely educational and demonstrates alternative implementation approaches. These listeners are **not registered** and are not used in the actual application. For the working implementation, refer to `listeners/assistant.py`.

Comment on lines +76 to +79
Copy link
Copy Markdown
Contributor

@lukegalbraithrussell lukegalbraithrussell Sep 19, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
**Note**: The `listeners/events` folder is purely educational and demonstrates alternative implementation approaches. These listeners are **not registered** and are not used in the actual application. For the working implementation, refer to `listeners/assistant.py`.
:::info[The `listeners/events` folder is purely educational and demonstrates alternative approaches to implementation]
These listeners are **not registered** and are not used in the actual application. For the working implementation, refer to `listeners/assistant.py`.
:::

this makes it look like a little callout card on the docs! anytime i think about using "note:" i just replace it with a callout card.

Copy link
Copy Markdown
Member

@zimeg zimeg Sep 23, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📝 note: Added in 42cf39f.

Copy link
Copy Markdown
Member

@zimeg zimeg Sep 23, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🗣️ thought: I'm not sure that this renders as expected with GitHub markdown?

🔗 https://github.com/srtaalej/bolt-python-assistant-template/tree/main?tab=readme-ov-file#listeners

We might want to revert this or use different syntax?

🔗 https://github.com/orgs/community/discussions/16925

Copy link
Copy Markdown
Contributor

@lukegalbraithrussell lukegalbraithrussell Sep 25, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah idk why i suggested to use docs syntax in a README i think i got confused where i was oof

**`/listeners/assistant`**
Comment thread
zimeg marked this conversation as resolved.

Configures the new Slack Assistant features, providing a dedicated side panel UI for users to interact with the AI chatbot. This includes:
* `@assistant.thread_started` - Manages when users start new assistant threads.
Comment thread
srtaalej marked this conversation as resolved.
Outdated
* `@assistant.user_message` - Processes user messages in assistant threads and app DMs. **Replaces traditional DM handling as seen in** `/listeners/events/user_message.py`
Comment thread
srtaalej marked this conversation as resolved.
Outdated

## App Distribution / OAuth

Expand Down