This pull request updates our webhook handling mechanism to replace GitHub webhook processing with GitLab equivalents. It introduces changes in the probot/gitlab_app.js to handle merge_request events instead of issue events and securely store the accessToken. Additionally, it adds a new Flask app in sweepai/webhook_handler.py to process GitLab merge_request webhooks, post comments on merge requests, and handle potential errors.
Summary
Modified probot/gitlab_app.js to:
Securely store the accessToken for GitLab API requests.
Change event handling from issue to merge_request, including posting a thank you note on merge requests.
[ ] 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.
PR Feedback (click)
Description
This pull request updates our webhook handling mechanism to replace GitHub webhook processing with GitLab equivalents. It introduces changes in the
probot/gitlab_app.js
to handlemerge_request
events instead ofissue
events and securely store theaccessToken
. Additionally, it adds a new Flask app insweepai/webhook_handler.py
to process GitLabmerge_request
webhooks, post comments on merge requests, and handle potential errors.Summary
probot/gitlab_app.js
to:accessToken
for GitLab API requests.issue
tomerge_request
, including posting a thank you note on merge requests.sweepai/webhook_handler.py
Flask app to:merge_request
webhook events.Fixes #81.
🎉 Latest improvements to Sweep:
💡 To get Sweep to edit this pull request, you can:
This is an automated message generated by Sweep AI.