Skip to content

Comments

feat(pkg-py): Use rich tool UI#86

Merged
gadenbuie merged 11 commits intomainfrom
py/feat/tool-ui
Sep 12, 2025
Merged

feat(pkg-py): Use rich tool UI#86
gadenbuie merged 11 commits intomainfrom
py/feat/tool-ui

Conversation

@gadenbuie
Copy link
Contributor

Closes #67
Closes #81

@gadenbuie gadenbuie requested a review from Copilot September 12, 2025 14:16
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements rich tool UI support for the Python querychat package, replacing basic text-based tool outputs with interactive buttons and enhanced visual feedback. The main changes include switching from Shiny's basic chat UI to shinychat's rich tool display system, and adding a new tool for resetting dashboard filters.

  • Adds rich tool UI with interactive buttons using shinychat and updated chatlas dependency
  • Introduces a reset dashboard tool for easy filter clearing
  • Refactors tool functions into a separate module with enhanced visual feedback

Reviewed Changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pyproject.toml Updates dependencies to include shinychat and chatlas version requirements
pkg-py/src/querychat/tools.py New module containing tool functions with rich UI support
pkg-py/src/querychat/static/js/querychat.js JavaScript for handling tool button interactions
pkg-py/src/querychat/querychat.py Updates to use shinychat and new tool system
pkg-py/src/querychat/prompt/prompt.md Updates tool names in system prompt
pkg-py/src/querychat/_utils.py Moves df_to_html utility function here
pkg-py/examples/app.py Updates example to use dynamic title display
pkg-py/CHANGELOG.md Documents the new features
Comments suppressed due to low confidence (1)

pyproject.toml:1

  • This line was removed from the ruff ignore list but there's no indication why this rule should now be enforced. The removal could cause linting failures if df is used as a variable name elsewhere in the codebase.
[build-system]

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@gadenbuie gadenbuie merged commit 7c0aeeb into main Sep 12, 2025
6 checks passed
@gadenbuie gadenbuie deleted the py/feat/tool-ui branch September 12, 2025 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Python] Add querychat_reset_dashboard() [Python]: Use rich tools

1 participant