feat(bot): add AI streaming with thinking steps#9
Merged
Conversation
- Upgrade slack-sdk to v3.40.0 for ChatStreamer support - Replace say() with client.chat_stream() for real-time streaming - Add thinking steps (PlanBlock, TaskCard) to show agent processing: - Analyzing question - Searching knowledge base - Generating response - Update all event handlers: mentions, DMs, slash command - Use task_display_mode=plan for structured task display
- Add docs/local-development.md with step-by-step local setup - Update README.md to highlight AI streaming feature - Clean up docs/setup.md formatting - Update getting-started.md with streaming preview
- Document all mise tasks for Docker services - Document local development commands - Document database migration commands - Document testing commands - Document Terraform IaC commands
- Add tests/test_streaming.py with 20 tests for streaming chunks - Fix import path: slack_sdk.models.messages.chunk (not models.blocks) - Update uv.lock to v3.42.0 for slack-sdk - Tests cover: imports, chunk creation, serialization, helper methods, status values, integration
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Contributor
Author
|
Build failures detected (backend/frontend build checks). The uv.lock was updated to reflect slack-sdk v3.42.0 which may need to be properly resolved in the Docker build. Task logged to track fix: "Fix wingman streaming PR build failures" To retry checks after fix: # Ensure dependencies are properly synced
cd backend && uv sync
git add uv.lock && git commit --amend --no-edit
git push --force-with-lease |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds Slack AI streaming support using the new
chat_stream()helper from slack-sdk v3.40.0.Changes
say()withclient.chat_stream()for real-time streamingslack_sdk.models.messages.chunk(notslack_sdk.models.blocks)Streaming Flow
Handlers Updated
app_mention(channel thread replies)message(direct messages)/wingmanslash commandTesting
20 streaming tests covering:
MarkdownTextChunk,TaskUpdateChunk,PlanUpdateChunk)Documentation Added/Updated
docs/local-development.mdREADME.mddocs/getting-started.mddocs/setup.mdCI checks show backend/frontend build failures. Likely due to
uv.lockupdate for slack-sdk v3.42.0 needing proper Docker rebuild. Please retry checks after approving, or runmise run up --buildlocally to verify.Quick Test
Docs