-
Notifications
You must be signed in to change notification settings - Fork 0
Refactor/add uv #1
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.
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
base: main
Are you sure you want to change the base?
Conversation
…efactor/add_uv
There was a problem hiding this 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 modernizes the project by migrating from a CLI-based RAG system to a web-based UI, adds modern Python packaging, and updates documentation and environment settings.
- Replaced
simple_rag.pyCLI withsrc/templates/index.htmlweb chat interface and client-side logic - Added
pyproject.tomlfor project metadata, tooling, and packaging; bumped Python version to 3.11 - Updated
README.mdto use Ultraviolet (UV) for dependency management and revised run instructions
Reviewed Changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/templates/index.html | New HTML/CSS/JS template for interactive RAG chat UI |
| simple_rag.py | Removed legacy CLI-based RAG implementation |
| pyproject.toml | Added modern project configuration, dependencies, and build setup |
| README.md | Updated install and run instructions to use UV and new entrypoint |
| .python-version | Set project Python version to 3.11 |
Comments suppressed due to low confidence (2)
pyproject.toml:43
- The entry point
simple_rag.main:mainno longer exists after removingsimple_rag.py. Update this script mapping to point at the new web runner (e.g.,run_rag:maininsrc/run_rag.py) or remove it if unused.
simple-rag = "simple_rag.main:main"
src/templates/index.html:136
- [nitpick] New client-side logic for sending messages and error handling has been added but isn’t covered by automated tests. Consider adding UI or integration tests (e.g., with Playwright or Cypress) to validate core chat behaviors.
<script>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…demo into refactor/add_uv
… and remove ruff dependency from uv.lock
This pull request introduces significant updates to modernize the project, enhance its functionality, and improve user experience. Key changes include migrating to a new project structure with
pyproject.toml, replacing the command-line interface with a web-based UI, updating the Python version, and revising the documentation to reflect these changes.Project Modernization:
pyproject.tomlto define project metadata, dependencies, and development tools, enabling better dependency management and compatibility with modern Python tooling.3.11in.python-version.Migration to Web-Based UI:
simple_rag.pyscript with a web-based interface, including a new HTML template (src/templates/index.html) for the chat functionality. This provides a more user-friendly experience and supports real-time interactions. [1] [2]Documentation Updates:
README.mdto reflect the new setup process, including instructions for installing dependencies with UV and running the system using the new web-based interface. [1] [2]Codebase Simplification:
simple_rag.py, which implemented the old command-line-based retrieval-augmented generation (RAG) system, as it has been replaced by the new architecture.