Skip to content
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

Add log statements #4105

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Add log statements #4105

wants to merge 3 commits into from

Conversation

kevinlu1248
Copy link
Member

No description provided.

Copy link

vercel bot commented Jun 23, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sweep-chat 🔄 Building (Inspect) Visit Preview 💬 Add feedback Jun 24, 2024 6:20pm
sweep-chat-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 24, 2024 6:20pm
sweep-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 24, 2024 6:20pm
sweep-pulley 🔄 Building (Inspect) Visit Preview 💬 Add feedback Jun 24, 2024 6:20pm

Copy link

sweep-fundrise bot commented Jul 10, 2024

Sweep: PR Review

Author: kevinlu1248
The Pull Request looks good! Sweep did not find any new issues.

Summary

This pull request added log statements and improved the handling of code suggestions in the autofix and validation processes.

The generate_code_suggestions function in sweepai/agents/modify.py was refactored to separate applied and queued code suggestions. This change allows for better tracking of the state of code modifications.

In sweepai/chat/api.py, the autofix function was updated to accept a new modify_files_dict parameter, enabling more complex autofix scenarios. The streaming response in this function now differentiates between applied and queued code suggestions, providing clearer information about the state of modifications.

The validate_pull function was also modified to accept a modify_files_dict parameter, allowing for additional data processing during pull request validation.

Logging statements were added to the write_message_to_disk and read_message_from_disk functions, providing information about the number of messages saved or loaded. This change enhances debugging and monitoring capabilities.

Lastly, the original_code_suggestions parameter and its related processing were removed from the write_message_to_disk function, simplifying the data persistence process.


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sweep Assigns Sweep to an issue or pull request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants