Fix Slackbot improvements (tasks 1-2): Progress message and uv extras #1189
Conversation
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
…as support - Update initial progress message to indicate the response may take time - Add instructions for using `uv run --extra` for optional dependencies - Update check_cli_command docstring with examples for extra dependencies This addresses issues #1188 tasks 1 and 2: 1. Users now know to wait when Marvin is processing 2. Bot can properly handle commands requiring optional dependencies 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR improves the Slackbot user experience by adding a progress message and enhancing LLM instructions for optional dependency handling. The changes address two specific tasks from issue #1188.
Key changes:
- Updated initial Slackbot response to inform users that processing may take a while
- Enhanced LLM system prompts and tool documentation to support
uv run --extrafor optional dependencies
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
examples/slackbot/src/slackbot/api.py |
Updates initial progress message to set user expectations about processing time |
examples/slackbot/src/slackbot/core.py |
Adds LLM instructions for using uv run --extra with optional dependencies |
examples/slackbot/src/slackbot/search.py |
Updates check_cli_command tool documentation to include optional dependency examples |
- Change from incorrect 'uv run --extra' to correct 'uv run --with prefect[extra]' - Update all examples and documentation to use proper syntax - Tested and verified: uv run --with 'prefect[docker]' works correctly 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
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
uv run --extrafor optional dependenciesRelated Issue
Addresses #1188 (tasks 1 and 2)
Changes
"🔄 Thinking... this may take a while"so users know to wait when Marvin is processingcheck_cli_commandtool to useuv run --extra <extra>when commands require optional dependencies (e.g., AWS, Docker, Kubernetes integrations)Test Plan
Note: Task 3 (handling message edits) will be addressed in a separate PR as it requires more investigation.
🤖 Generated with Claude Code