Commit graph

737 commits

Author SHA1 Message Date
Ori Kotek
613ccb4c34
Add support for base_url in GitHub SDK 2023-11-21 16:48:36 +02:00
Ori Kotek
2add584fbc
Fix a bug in GitLab webhook 2023-11-21 16:28:01 +02:00
mrT23
b3129c7dd9
Merge pull request #464 from Codium-ai/tr/more_protections
Refactor YAML parsing for improved error handling
2023-11-20 02:28:57 -08:00
mrT23
1b600cd85f
Refactor YAML parsing for improved error handling 2023-11-20 10:30:59 +02:00
mrT23
26cc26129c
Merge pull request #463 from Codium-ai/tr/more_protections
minor fix
2023-11-19 07:45:39 -08:00
mrT23
d1d7903e39
minor fix 2023-11-19 17:44:11 +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
3504a64269
protections 2023-11-19 17:35:40 +02:00
mrT23
83247cadec
protections 2023-11-19 17:30:57 +02:00
mrT23
c7a681038d
gfm instructions 2023-11-19 11:11:11 +02:00
mrT23
eb977b4c24
gfm instructions 2023-11-19 11:02:11 +02:00
Luca Simone
b62e0967d5 fix: Revert back to exception since context.get will not throw KeyError 2023-11-17 10:08:40 +01:00
Luca Simone
26dc2e9d21
fix: raising exception instead of empty string
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
2023-11-16 11:19:46 +01:00
Luca Simone
d78a71184d
fix: Use checked exception KeyError for missing key
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
2023-11-16 10:59:01 +01:00
Luca Simone
eae30c32a2
fix: Use checked exception for ValueError
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
2023-11-16 10:58:35 +01:00
mrT23
bc28d657b2
Merge pull request #438 from koid/fix/remove-unnecessary-setup
Removal of Redundant Logger Setup
2023-11-15 10:35:13 -08:00
Luca Simone
3fae5cbd8d feat: Added BitBucket Server
Signed-off-by: Luca Simone <info@lucasimone.info>
2023-11-15 15:47:44 +01:00
mrT23
388cc740b6
Merge pull request #436 from rhyst/support-vertex-ai
Support Google's Vertex AI
2023-11-15 04:26:08 -08: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
Rhys Tyers
557ec72bfe Update documentation for Vertex AI 2023-11-15 10:27:48 +00:00
Rhys Tyers
6b4b16dcf9 Support Google's Vertex AI 2023-11-15 10:26:58 +00:00
mrT23
c4899a6c54
bitbucket 2023-11-15 12:11:02 +02:00
mrT23
24d82e65cb
gitlab 2023-11-15 09:45:10 +02:00
mrT23
2567a6cf27
gitlab 2023-11-15 09:40:45 +02:00
mrT23
94cb6b9795
more feedback 2023-11-15 09:06:26 +02:00
mrT23
0df0542958
prompt 2023-11-13 15:55:35 +02:00
Zohar Meir
7d89b82967
Fix get_user_description in case pr_description.enable_pr_type=false
Fixes an issue when getting the user description after a PR-Agent description was already generated, in case the configuration setting `pr_description.enable_pr_type` was `false`.
2023-11-13 14:41:14 +02:00
Hussam.lawen
a5e5a82952
s 2023-11-13 13:49:16 +02:00
Hussam.lawen
ccbb62b50a
remove the "one or more" for custom labels 2023-11-13 13:47:06 +02:00
mrT23
a8dddd1999
prompt 2023-11-13 12:14:18 +02:00
mrT23
f5c6dd55b8
triple quote 2023-11-13 12:04:58 +02:00
mrT23
0e932af2e3
multi line 2023-11-13 12:01:08 +02:00
mrT23
e9891fc530
s1 2023-11-12 16:37:53 +02:00
Ori Kotek
9e5e9afe92
Refactor CLI argument handling and request processing 2023-11-12 16:11:34 +02:00
mrT23
5e43c202dd
s1 2023-11-12 15:45:22 +02:00
mrT23
727eea2b62
s1 2023-11-12 15:00:06 +02:00
mrT23
8fdf174dec
fallback 2023-11-10 18:44:19 +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
755e04cf65
bitbucket finally works 2023-11-08 20:41:55 +02:00
mrT23
44d6c95714
response 2023-11-08 20:38:18 +02:00
mrT23
14610d5375
persistent
s
2023-11-08 20:16:08 +02:00
mrT23
c2bec614e5
s 2023-11-08 14:46:11 +02:00
mrT23
49725e92f2
s 2023-11-08 14:41:15 +02:00
mrT23
a1e32d8331
s 2023-11-08 14:36:59 +02:00
mrT23
0293412a42
s 2023-11-08 14:31:08 +02:00
mrT23
10ec0a1812
s 2023-11-08 14:21:03 +02:00
mrT23
69b68b78f5
s 2023-11-08 14:17:59 +02:00
mrT23
c5bc4b44ff
fix added files 2023-11-08 12:51:30 +02:00
mrT23
39e5102a2e
fix added files 2023-11-08 12:47:18 +02:00
koid
f0991526b5 remove unnecessary setup_logger 2023-11-08 16:56:44 +09:00
mrT23
54f41dd603
code 2023-11-07 14:41:15 +02:00
mrT23
094f641fb5
code 2023-11-07 14:38:37 +02:00
mrT23
a35a75eb34
get_max_tokens + added 'gpt-4-1106-preview' 2023-11-07 14:28:41 +02:00
Hussam.lawen
ff79776410
Keep user labels 2023-11-06 15:14:08 +02:00
Hussam.lawen
ec3f2fb485
Revert "generate labels keep user labels only"
This reverts commit 94a2a5e527.
2023-11-06 15:08:29 +02:00
Hussam.lawen
94a2a5e527
generate labels keep user labels only 2023-11-06 15:07:06 +02:00
Hussam.lawen
1eefd3365b
Merge commit 'e352c98ce83bfbd99078f62d8705eb938a6ba5b5' into hl/type_vs_labels 2023-11-06 14:24:33 +02:00
Hussam.lawen
db37ee819a
support git providers with no label support 2023-11-06 14:11:49 +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
Hussam.lawen
e96b03da57
add configuration enable_pr_type 2023-11-06 11:58:26 +02:00
Hussam.lawen
1d2aedf169
Don't Display pr labels in the text 2023-11-06 11:35:22 +02: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
mrT23
8a79114ed9
Merge pull request #430 from Codium-ai/tr/fix_prompt
Fix PR Description Prompt and Data Preparation
2023-11-06 01:06:26 -08:00
mrT23
6d6d864417
fix prompt 2023-11-06 09:44:59 +02:00
mrT23
7238c81f0c
fix prompt 2023-11-06 09:41:26 +02:00
mrT23
62412f8cd4
fix prompt 2023-11-06 09:38:39 +02:00
mrT23
5d2bdadb45
fix prompt 2023-11-06 09:33:10 +02:00
mrT23
06d030637c
fix prompt 2023-11-06 09:32:46 +02:00
zmeir
8e3fa3926a Extract incremental review checks to separate method 2023-11-06 09:21:22 +02:00
zmeir
92071fcf1c Stack all incremental parameters 2023-11-06 09:13:04 +02:00
mrT23
fed1c160eb
files walkthrough bullets 2023-11-06 08:43:15 +02:00
mrT23
e37daf6987
link to change 2023-11-06 08:27:34 +02:00
mrT23
8fc663911f
fixe bitbucket get_repo_settings bug 2023-11-06 08:15:43 +02:00
Hussam.lawen
3548b88463
type and labels 2023-11-05 15:48:39 +02:00
koid
e6ef123ce5 add middleware when initializing fastapi 2023-11-05 15:38:19 +09:00
Ori Kotek
fe3527de3c
Add exception handling for applying repo settings failure 2023-11-03 12:23:49 +02:00
Ori Kotek
b99c769b53
Merge pull request #415 from zmeir/zmeir-patch-2
Refactor Command Handling for Different Triggers
2023-11-02 18:32:42 +02:00
Ori Kotek
c0b3c76884
Merge remote-tracking branch 'origin/main' into ok/bitbucket_fix 2023-11-02 15:27:11 +02:00
Ori Kotek
e1370a8385
Update publish_inline_comments in bitbucket_provider.py to use 'position' instead of 'start_line' 2023-11-02 15:24:47 +02:00
zmeir
c623c3baf4 Added new configurations to prevent too frequent incremental commits on push trigger 2023-11-02 12:24:54 +02:00
Hussam Lawen
3ddc7e79d1
Update pr_reviewer_prompts.toml 2023-11-02 11:45:34 +02:00
Zohar Meir
15573e2286
Fix error in get_main_pr_languages when the diff is empty
This can happen for example when you have one commit add a line to a file and the next commit deletes that line. Then if those are the only 2 commits in the PR the diff will be empty.
2023-11-02 10:10: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
Hussam Lawen
9ab7ccd20d
Merge pull request #416 from zmeir/patch-1
Fix formatting when last commit message contains _
2023-11-01 13:13:31 +02:00
KennyDizi
29a8cf8357 fix typo for ollama 2023-10-31 20:38:27 +07:00
Zohar Meir
7b6a6c7164
Fix formatting when last commit message contains _ 2023-10-31 10:05:13 +02:00
Zohar Meir
cf4d007737
Fix commands list for push trigger 2023-10-31 00:00:48 +02:00
Ori Kotek
775ccb3f25
Refactor _perform_commands function in github_app.py to improve command handling 2023-10-30 20:14:25 +02:00
Ori Kotek
73bb70fef4
Update get_repo_settings in bitbucket_provider.py to fetch file via API request 2023-10-30 18:36:46 +02:00
Ori Kotek
4bda9dfe04
Update get_repo_settings to decode file from target branch in gitlab_provider.py 2023-10-30 17:01:49 +02:00
Ori Kotek
e74bb80668
Refactor get_repo_settings method in gitlab_provider.py to decode file contents 2023-10-30 16:45:47 +02:00
Ori Kotek
e06fb534d3
Merge remote-tracking branch 'origin/main' into ok/fix_gitlab_bug 2023-10-30 16:34:03 +02:00
Ori Kotek
71a341855e
Add log_context to handle_request calls in gitlab_webhook.py 2023-10-30 16:00:09 +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