Commit graph

141 commits

Author SHA1 Message Date
mrT23
4a0b12c036
Refactor reaction handling in GitHub provider and update help text in PR tools 2024-02-20 08:06:33 +02:00
mrT23
2f327c26e8
auto approve 2024-02-13 11:21:59 +02:00
mrT23
555151602f
rstrip() 2024-02-09 11:26:43 +02:00
mrT23
fa077dc516
formatting 2024-02-08 18:55:58 +02:00
Hussam.lawen
a3f4c44632
PR Review 2024-02-08 15:25:43 +02:00
mrT23
b190b1879e
auto approval 2024-02-06 09:09:07 +02:00
mrT23
39891e4ab1
feat: Improve error handling and code readability in pr_agent tools 2024-01-18 17:01:25 +02:00
mrT23
578d15c6fc
usage guide 2024-01-07 16:38:08 +02:00
mrT23
edcb666fbc
docs: Reorganize and enhance documentation for review and describe tools 2024-01-07 09:28:06 +02:00
KennyDizi
37b040b50a Use 'and' in lieu of '&' operator 2023-12-27 10:17:08 +07:00
KennyDizi
31168cd7de Get PR review enable help text from setting 2023-12-27 10:12:41 +07:00
mrT23
a6764c9058
feat: Update help text addition condition in pr_reviewer.py 2023-12-26 09:25:15 +02:00
mrT23
a854e1a408
feat: Refactor help text addition in pr_reviewer.py and update tool names in README.md 2023-12-26 09:18:38 +02:00
mrT23
5fb373b212
Refactor AI handler instantiation to use lazy initialization in PR tools 2023-12-17 16:52:03 +02:00
mrT23
246be6147f
Set LiteLLMAIHandler as default AI handler in all PR tools and simplify AI handler injection in PRAgent 2023-12-14 09:00:14 +02:00
Brian Pham
e37598fdca Merge remote-tracking branch 'upstream/main' into abstract-BaseAiHandler 2023-12-14 07:45:43 +08:00
Brian Pham
557b39ec87 Merge branch 'base-ai-handler' into abstract-BaseAiHandler 2023-12-14 07:44:13 +08:00
Brian Pham
8fb4a42ef1 Update AI handler instantiation in server files 2023-12-13 08:16:02 +08:00
Brian Pham
7eb2e769cf Move ai handlers to specific folder 2023-12-12 23:03:38 +08:00
Brian Pham
5239e1c3e9 Load default AI Handler from util function 2023-12-12 21:51:05 +08:00
Hussam.lawen
fdc776887d
Refactor labels 2023-12-11 16:47:38 +02:00
Brian Pham
b8021d7ca3 rename file 2023-12-11 16:57:23 +08:00
Brian Pham
523a896465 Rename AiHandler to LiteLLMAiHandler 2023-12-11 16:56:49 +08:00
Brian Pham
c0303ff9ec Merge remote-tracking branch 'upstream/main' into abstract-BaseAiHandler 2023-12-09 16:47:13 +00:00
Brian Pham
f2abe5c73e Abstract AiHandler to BaseAiHandler 2023-12-09 16:39:25 +00:00
Brian Pham
7e47baa9db Refactor AI handler classes 2023-12-10 00:25:25 +08:00
mrT23
7e029ead45
refactor: Improve label handling in pr_reviewer.py 2023-11-28 17:03:55 +02:00
mrT23
8f615e17a3
s 2023-11-26 16:42:41 +02:00
mrT23
dff4d1befc
Merge pull request #462 from Codium-ai/tr/more_protections
Enhancements in YAML Parsing and Error Handling
2023-11-19 07:40:06 -08:00
mrT23
83247cadec
protections 2023-11-19 17:30:57 +02:00
mrT23
eb977b4c24
gfm instructions 2023-11-19 11:02:11 +02:00
mrT23
762a6981e1
extra_labels 2023-11-15 14:12:59 +02:00
mrT23
b362c406bc
Merge remote-tracking branch 'origin/main' into tr/review_extra_labels 2023-11-15 14:07:44 +02:00
mrT23
7a342d3312
extra_labels 2023-11-15 14:07:32 +02:00
mrT23
2e95988741
extra_labels 2023-11-15 14:04:17 +02:00
mrT23
9478447141
extra_labels 2023-11-15 14:02:13 +02:00
mrT23
94cb6b9795
more feedback 2023-11-15 09:06:26 +02:00
mrT23
737792d83c
publish_persistent_comment 2023-11-09 15:24:55 +02:00
mrT23
7e5889061c
publish_persistent_comment 2023-11-09 15:20:31 +02:00
mrT23
14610d5375
persistent
s
2023-11-08 20:16:08 +02:00
mrT23
e352c98ce8
Merge pull request #431 from Codium-ai/hl/type_vs_labels
Refactoring PR Labels Handling and Display
2023-11-06 02:10:38 -08:00
mrT23
4c484f8e86
Merge pull request #423 from zmeir/zmeir-external-incremental_review_thresholds
Implementing Thresholds for Incremental PR Reviews
2023-11-06 01:07:01 -08:00
zmeir
8e3fa3926a Extract incremental review checks to separate method 2023-11-06 09:21:22 +02:00
mrT23
e37daf6987
link to change 2023-11-06 08:27:34 +02:00
Hussam.lawen
3548b88463
type and labels 2023-11-05 15:48:39 +02:00
zmeir
c623c3baf4 Added new configurations to prevent too frequent incremental commits on push trigger 2023-11-02 12:24:54 +02:00
KennyDizi
9fbf89670d Improve expression portion of f-strings 2023-11-01 19:11:52 +07:00
KennyDizi
ad1c51c536 Fix SyntaxError: f-string expression part cannot include a backslash 2023-11-01 19:06:29 +07:00
Zohar Meir
7b6a6c7164
Fix formatting when last commit message contains _ 2023-10-31 10:05:13 +02:00
Zohar Meir
816ddeeb9e
Fix call to _get_previous_review_comment
Hey @mrT23, I thinks there's a problem with moving this line to after `self.git_provider.publish_comment(pr_comment)`.

The reason I originally placed it here is because otherwise, if you run `/review --pr_reviewer.remove_previous_review_comment=true` it will publish your review and then immediately after delete it, because it will look for the previous review comment only after you published your new review - so it will take your new review as the previous one. In order to get the real "previous" review you must collect the comments list before publishing a review, so placing this method call first ensures that.

The method `self._get_previous_review_comment()` is a no-op if `pr_reviewer.remove_previous_review_comment=false` so I see no downside in keeping it before `self.git_provider.publish_comment(pr_comment)`

Additionally, the check for `if previous_review_comment:` is redundant because it's done internally in `self._remove_previous_review_comment`. I thought it looked cleaner without this extra nesting here, but if you think more verbosity is better I'll keep it.
2023-10-30 09:06:51 +02:00