This pull request addresses the issue where the "sweep watch" command line interface would exit without starting the server. The changes involve refactoring the main function from cli.py to watch.py and ensuring it properly initializes with the necessary repository information.
Summary
Moved the main function logic from cli.py to watch.py to centralize event watching logic.
Added parameter validation in the main function in watch.py to ensure the repository object and name are correctly passed and utilized.
Updated cli.py to call the main function from watch.py with the appropriate arguments, ensuring the server starts as expected.
Removed redundant global variables from watch.py that were moved to the main function to prevent potential issues with uninitialized variables.
Ensured that the main function in watch.py is only executed with the correct environment variables, enhancing the robustness and reliability of the command.
Fixes #1.
🎉 Latest improvements to Sweep:
New dashboard launched for real-time tracking of Sweep issues, covering all stages from search to coding.
Integration of OpenAI's latest Assistant API for more efficient and reliable code planning and editing, improving speed by 3x.
[ ] Apply: We should use loguru for error logging. If the log is inside an exception, use logger.exception to add tracebacks, where logger is imported from loguru. Use f-strings for string formatting in logger calls (e.g. logger.info(f'Hello {name}') instead of logger.info('Hello {name}', name=name)).
[ ] Apply: There should be no debug log or print statements in production code.
[ ] Apply: All functions should have parameters and output annotated with type hints. Use list, tuple and dict instead of typing.List, typing.Tuple and typing.dict.
[ ] Apply: Leftover TODOs in the code should be handled.
[ ] Apply: All new business logic should have corresponding unit tests in the same directory. For example, sweepai/api_test.py tests sweepai/api.py. Use unittest and unittest.mock as required.
[ ] Apply: Any clearly inefficient or repeated code should be optimized or refactored.
[ ] Apply: Remove any comments before code that are obvious. For example # this prints hello world; print('hello world').
This is an automated message generated by Sweep AI.
Description
This pull request addresses the issue where the "sweep watch" command line interface would exit without starting the server. The changes involve refactoring the
main
function fromcli.py
towatch.py
and ensuring it properly initializes with the necessary repository information.Summary
main
function logic fromcli.py
towatch.py
to centralize event watching logic.main
function inwatch.py
to ensure the repository object and name are correctly passed and utilized.cli.py
to call themain
function fromwatch.py
with the appropriate arguments, ensuring the server starts as expected.watch.py
that were moved to themain
function to prevent potential issues with uninitialized variables.main
function inwatch.py
is only executed with the correct environment variables, enhancing the robustness and reliability of the command.Fixes #1.
🎉 Latest improvements to Sweep:
💡 To get Sweep to edit this pull request, you can:
This is an automated message generated by Sweep AI.