Commit graph

955 commits

Author SHA1 Message Date
mrT23
7d2a35e32c
final commit 2023-12-07 10:27:19 +02:00
mrT23
e351428848
s 2023-12-07 10:24:36 +02:00
mrT23
4cd6649a44
feat: Enhance PR description formatting in pr_description.py
Improve the table structure for relevant files in PR description by adjusting the header and filename display. Add padding for filename and change summary, and move diff_plus_minus to a separate column. Refactor _insert_br_after_x_chars function to accept a variable length parameter.
2023-12-07 10:14:18 +02:00
mrT23
e62acef6d2
s1 2023-12-07 09:50:36 +02:00
mrT23
a043eb939b
feat: Apply repo settings on push trigger in github_app.py 2023-12-07 08:42:18 +02:00
mrT23
a61e492fe1
feat: Refactor PR files processing into separate function in pr_description.py 2023-12-06 17:01:21 +02:00
mrT23
93b6d31505
s 2023-12-06 16:36:27 +02:00
mrT23
429aed04b1
s 2023-12-06 16:32:53 +02:00
mrT23
eeb20b055a
feat: Add line count to file patch info and enhance PR description formatting 2023-12-06 15:29:45 +02:00
mrT23
4b073b32a5
feat: Enhance PR description with file label dictionary and update prompts in pr_description_prompts.toml 2023-12-06 12:30:51 +02:00
mrT23
f629755a9a
feat: Refine field descriptions in pr_description_prompts.toml for semantic file labels 2023-12-06 10:59:44 +02:00
mrT23
c1ed3ee511
feat: Refine field descriptions in pr_description_prompts.toml for semantic file labels 2023-12-06 08:08:01 +02:00
mrT23
0b70e07b8c
feat: Improve formatting in help.py command descriptions 2023-12-05 18:26:35 +02:00
mrT23
862c236076
s 2023-12-05 18:10:13 +02:00
mrT23
cf3401536a
feat: Remove 'Refactoring' label from custom labels and update related descriptions 2023-12-05 07:48:21 +02:00
mrT23
2feaee4306
feat: Update field descriptions in pr_description_prompts.toml for clarity 2023-12-04 21:45:22 +02:00
mrT23
863eb0105d
feat: Refactor semantic labels in PR description and improve clarity in pr_description.py and pr_description_prompts.toml 2023-12-04 21:23:59 +02:00
mrT23
21a7a0f136
feat: Enhance link generation for relevant lines and refactor code in git providers and PR description tools 2023-12-04 21:06:56 +02:00
Hussam.lawen
d2a129fe30
Add labeling files 2023-12-04 18:22:35 +02:00
mrT23
fe796245a3
Merge pull request #501 from Codium-ai/tr/prompt_tuning
Refactoring and Enhancement of PR Agent Prompts
2023-12-04 03:18:12 -08:00
Phill Zarfos
317fec0536 Throw descriptive error message if model is not in MAX_TOKENS array 2023-12-03 21:06:55 -05:00
mrT23
4dcbce41c8
feat: Refine prompts and improve formatting in pr_sort_code_suggestions_prompts.toml and pr_update_changelog_prompts.toml 2023-12-03 17:27:22 +02:00
mrT23
b3fa654446
feat: Refactor prompts and improve formatting in pr_questions_prompts.toml 2023-12-03 17:23:52 +02:00
mrT23
e09439fc1b
feat: Enhance formatting and clarity in pr_information_from_user_prompts.toml 2023-12-03 17:17:24 +02:00
mrT23
324e481ce7
feat: Improve formatting and clarity in pr_custom_labels.toml 2023-12-03 17:15:29 +02:00
mrT23
abfad088e3
feat: Refine prompts and instructions in pr_add_docs.toml 2023-12-03 17:10:33 +02:00
mrT23
f30789e6c8
feat: Refactor and enhance prompts in pr_code_suggestions_prompts.toml 2023-12-03 16:59:47 +02:00
mrT23
5c01f97f54
feat: Enhance PR description prompts in pr_description_prompts.toml 2023-12-03 16:48:26 +02:00
mrT23
2d726edbe4
feat: Improve formatting and clarity in pr_reviewer_prompts.toml 2023-12-03 16:40:06 +02:00
mrT23
37812dfede
feat: Update pr_url assignment in github_provider.py for GitHub Actions compatibility 2023-12-03 11:34:17 +02:00
mrT23
586785ffde
feat: Add pr_url attribute to git providers and final update message in PR description 2023-12-03 10:46:02 +02:00
mrT23
3af9c3bfb9
feat: Enhance code suggestion publishing with language-specific formatting in pr_code_suggestions.py 2023-12-01 12:12:49 +02:00
mrT23
c22084c7ac
feat: Add exception handling for missing previous review in github_provider.py 2023-12-01 11:56:03 +02:00
mrT23
b90dde48c0
Merge pull request #483 from tmokmss/add-bedrock-support
Add Amazon Bedrock support
2023-11-29 03:08:01 -08:00
Ori Kotek
0e54a13272
Improve error handling in settings retrieval
Fix bug where default values were not being used in GitHub Action runners when environmental variables are not set. Now, if an environmental variable cannot be found and no default is given, the default value will be used if one exists. This will prevent errors during setup on different environments and ensure system stability.
2023-11-29 11:52:02 +02:00
Ori Kotek
b67d06ae59
"Add fallback to environment variables in GitHub Action Runner"
A new function `get_setting_or_env` was implemented to facilitate fetching of either settings or environmental variables in the GitHub Action Runner. This was necessary to resolve an issue where a certain undefined behaviour occurs in GitHub Actions, leading to an attribute error. The new function also provides a default value parameter to ensure the return of a value even in failed attempts to fetch from either settings or environment variables.
2023-11-29 11:47:52 +02:00
tmokmss
5e642c10fa fallback to try_fix_yaml 2023-11-29 17:57:54 +09:00
Ori Kotek
580eede021
Add utility function to handle boolean type conversion
A utility function (`is_true`) was added to take care of validating and converting boolean values from string or boolean types. This function is used in three parts of the `run_action` method where automatic PR review, description, and improvement actions are triggered based on environment settings. This change makes the condition checks cleaner and prevents code duplication.
2023-11-29 10:33:12 +02:00
mrT23
ea56910a2f
Merge pull request #485 from Codium-ai/tr/bitbucket
Enhancement of Inline Comment Publishing in Bitbucket Provider and Logging Addition
2023-11-28 08:35:40 -08:00
mrT23
51e1278cd7
feat: Enhance inline comment publishing in Bitbucket provider and add logging for no suggestions in pr_code_suggestions.py 2023-11-28 18:29:35 +02:00
mrT23
7e029ead45
refactor: Improve label handling in pr_reviewer.py 2023-11-28 17:03:55 +02:00
tmokmss
f8f57419c4 Update ai_handler.py 2023-11-28 23:08:17 +09:00
tmokmss
917f4b6a01 hard code value 2023-11-28 20:59:21 +09:00
tmokmss
97d6fb999a set max_tokens_to_sample 2023-11-28 20:58:57 +09:00
tmokmss
1373ca23fc support Amazon Bedrock 2023-11-28 20:58:42 +09:00
mrT23
6264624c05
Merge branch 'main' into ok/fix_improve 2023-11-27 07:27:25 -08:00
Ori Kotek
2dcee63df5
fix improve 2023-11-27 12:32:53 +02:00
mrT23
b1c374808d
feat: Add line link generation in Bitbucket provider and improve markdown formatting in pr_code_suggestions.py and IMRPOVE.md 2023-11-26 17:12:02 +02:00
mrT23
20978402ea
s 2023-11-26 16:56:06 +02:00
mrT23
8f615e17a3
s 2023-11-26 16:42:41 +02:00
mrT23
5cbbaf44c9
feat: Add line link generation for GitLab and improve markdown formatting in pr_code_suggestions.py 2023-11-26 13:42:57 +02:00
mrT23
f96d4924e7
feat: Add line link generation in git providers and refactor code suggestions generation 2023-11-26 11:57:45 +02:00
mrT23
f36b672eaa
feat: Add option to summarize code suggestions in pr_code_suggestions.py 2023-11-26 11:22:14 +02:00
mrT23
668041c09f
Code suggestions guidelines: 2023-11-26 09:32:02 +02:00
mrT23
aa73eb2841
PR 2023-11-26 09:24:33 +02:00
mrT23
14d4ca8c74
PR 2023-11-26 09:22:19 +02:00
mrT23
690c113603
refactor: Improve clarity and consistency in pr_code_suggestions_prompts.toml and pr_reviewer_prompts.toml files 2023-11-26 09:17:42 +02:00
mrT23
1a28c77783
Previous description 2023-11-26 09:08:33 +02:00
mrT23
0326b7e4ac
refactor: Update PR prompts in toml files for clarity and consistency 2023-11-26 09:05:45 +02:00
mrT23
d8ae32fc55
language_extension_map 2023-11-26 08:52:55 +02:00
mrT23
8db2e3b2a0
feat: Enhance readability in toml files and add verbosity level logging in pr_generate_labels.py 2023-11-26 08:42:04 +02:00
mrT23
9465b7b577
refactor: Move clip_tokens function from pr_processing to utils module, and add tests 2023-11-26 08:29:47 +02:00
mrT23
d7df4287f8
feat: Update PR prompts in toml files to enhance readability and consistency 2023-11-26 08:17:16 +02:00
mrT23
b3238e90f2
s 2023-11-26 08:10:01 +02:00
mrT23
fdfd6247fb
Merge branch 'main' into tr/pydantic 2023-11-25 21:36:16 -08:00
mrT23
46d4d04e94
Merge pull request #455 from lukefx/bitbucket-server
Added BitBucket Server and Data Center support
2023-11-25 21:33:26 -08:00
Luca Simone
0f6564f42d feat: Added server and documentation 2023-11-25 17:37:44 +01:00
mrT23
e80a0ed9c8
glob 2023-11-23 09:16:50 +02:00
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
mrT23
c895657310
fixed review 2023-10-29 17:59:46 +02:00
mrT23
e3845283f8
release notes 2023-10-29 14:58:36 +02:00
mrT23
e97a03f522
Merge remote-tracking branch 'origin/main' into tr/final_fixes 2023-10-29 14:38:33 +02:00
Ori Kotek
61d3e1ebf4
Merge pull request #394 from zmeir/zmeir-external-push_trigger
Added support for automatic review on push event
2023-10-29 13:04:33 +02:00
mrT23
ae375c2ff0
final fixes 2023-10-29 13:01:55 +02:00
mrT23
c27dc436c4
final fixes 2023-10-29 12:29:14 +02:00
mrT23
7374243d0b
enable_custom_labels 2023-10-29 11:40:36 +02:00
mrT23
22c196cb3b
Merge remote-tracking branch 'origin/main' into tr/fix_custom_labels
# Conflicts:
#	pr_agent/git_providers/github_provider.py
2023-10-29 10:58:42 +02:00
mrT23
d772213cfc
fix labels 2023-10-29 08:58:12 +02:00
mrT23
638db96311
github action now also uses .pr_agent.toml 2023-10-28 13:34:32 +03:00
Hussam Lawen
4dffabf397
Merge pull request #396 from Codium-ai/hl/custom_labels
Implement Custom Labels for PRs
2023-10-28 01:37:54 +03:00
Hussam.lawen
6f2bbd3baa
Add documentation 2023-10-28 00:45:59 +03:00
Hussam.lawen
9e41f3780c
disable custom labels by default 2023-10-27 21:22:56 +03:00
Hussam.lawen
f53ec1d0cc
move enable custom labels to custom labels function 2023-10-27 21:12:58 +03:00
Hussam.lawen
a7cb59ca8b
small fix 2023-10-27 08:10:29 +03:00
Hussam.lawen
ca0ea77415
refactor 2023-10-27 07:58:42 +03:00
Hussam.lawen
0cf27e5fee
custom labels disabled by default 2023-10-27 07:54:59 +03:00
Hussam.lawen
f3bdbfc103
Add /generate_labels function + fix issues 2023-10-26 23:28:33 +03:00
Zohar Meir
e6bea76eee
Typo 2023-10-26 17:07:16 +03:00
zmeir
414f2b6767 Fix incremental review if there are no new commits (would have performed a full review instead) 2023-10-26 16:49:55 +03:00
zmeir
6541575a0e Refactor to use pull_request synchronize event 2023-10-26 16:49:54 +03:00
zmeir
02570ea797 Remove previous review comment on push event 2023-10-26 16:46:54 +03:00
Hussam.lawen
16777a5334
Add custom label description 2023-10-25 13:48:27 +03:00
zmeir
65bb70a1dd Added support for automatic review on push event
The new feature can be enabled via the new configuration `github_app.handle_push_event`. To avoid any unwanted side-effects, the current default of this configuration is set to `false`.

The high level flow (assuming the configuration is enabled):
1. receive push event from GitHub
2. extract branch and commits from event
3. find PR url for branch (currently does not support PRs from forks)
4. perform configured commands (e.g. `/describe`, `/review -i`)

The push event flow is guarded by a backlog queue so that multiple push events on the same branch won't trigger multiple duplicate runs of the PR-Agent commands.
Example timeline:
1. push 1 - start handling event
2. push 2 - waiting to be handled while push 1 event is still running
3. push 3 - event is dropped since handling it and handling push 2 is the same, so it is redundant
4. push 1 finished being handled
5. push 2 awakens from wait and continues handling (potentially reviewing the commits of both push 2 and push 3)

All of these options are configurable and can be enabled/disabled as per the user's desire.

Additional minor changes in this PR:
1. Created `DefaultDictWithTimeout` utility class to avoid too much boilerplate code in managing caches for outdated triggers.
2. Guard against running increment review when there are no new commits.
3. Minor styling changes for incremented review text.
2023-10-25 11:15:23 +03:00
Hussam.lawen
1a89c7eadf
refactor + add description options 2023-10-24 22:28:57 +03:00
Hussam.lawen
07617eab5a
add custom labels 2023-10-24 22:06:27 +03:00
Krystal Boozel
f9e4c2b098
Update configuration.toml 2023-10-23 21:34:12 -04:00
Hussam.lawen
fa24413201
Custom Labels 2023-10-23 16:29:33 +03:00
mrT23
b6cabda586
quick fix 2023-10-19 17:24:37 +03:00
mrT23
abbce60f18
Merge remote-tracking branch 'origin/main' 2023-10-19 17:10:30 +03:00
mrT23
5daaaf2c1d
quick fix 2023-10-19 17:10:21 +03:00
mrT23
e8f207691e
Merge pull request #391 from Codium-ai/tr/readme
Update and Enhance DESCRIBE.md Documentation
2023-10-19 02:03:50 -07:00
mrT23
b0dce4ceae
describe 2023-10-19 12:02:12 +03:00
Ori Kotek
fc494296d7
Merge pull request #387 from Codium-ai/ok/json_logging_in_bitbucket
Enhancing Logging in Bitbucket, GitLab, and Google Cloud Storage Secret Provider
2023-10-19 11:59:26 +03:00
Hussam Lawen
1b6fb3ea53
Merge pull request #385 from Codium-ai/hl/fix_add_docs_in_scripts
Add Blacklist for Non-Editable File Extensions in Documentation
2023-10-19 11:21:36 +03:00
mrT23
a2c3db463a
use_bullet_points 2023-10-19 10:45:42 +03:00
mrT23
21a1cc970e
- update readme
- minor prompts change
2023-10-19 09:16:20 +03:00
Ori Kotek
1314898cbf
Enhance logging in bitbucket_app, gitlab_webhook, and google_cloud_storage_secret_provider with JSON format and additional context 2023-10-18 16:44:03 +03:00
Ori Kotek
ff04d459d7
Update Bitbucket Pipeline instructions in INSTALL.md, remove redundant functionality 2023-10-18 15:46:43 +03:00
Hussam Lawen
88ca501c0c
Merge pull request #377 from zmeir/zmeir-review_incremental_detect_header
Get previous incremental review
2023-10-18 00:30:42 +03:00
mrT23
d41fe0cf79
comment 2023-10-17 19:45:04 +03:00
Hussam.lawen
3673924fe9
Add docs editable blacklist of file extensions like sql, yaml... 2023-10-17 18:50:39 +03:00
mrT23
d5c098de73
another protection 2023-10-17 10:21:05 +03:00
mrT23
9f5c0daa8e
protection 2023-10-17 09:43:48 +03:00
Marius Anderie
e6f1e0520a remove azure.com url restriction 2023-10-16 20:38:14 +02:00
Zohar Meir
d8de89ae33
Get previous incremental review
When getting the last commit in `/review -i` consider also the last __incremental__ review, not just the last __full__ review

Full disclosure I'm not really sure the `/review -i` feature work very well - I might be wrong but it seemed like the actual review in fact addressed all the changes in the PR, and not just the ones from the last review (even though it adds a link to the commit of the last review).  
I think the commit list gathered in `/review -i` doesn't propagate the actual list the reviewer uses. Again, I might be wrong, just took a brief glance at it.
2023-10-16 16:37:10 +03:00
Ori Kotek
83e670c5df
Enhance logging context in github_app server with server type 2023-10-16 16:13:09 +03:00
Ori Kotek
c324d88be3
Refactor logging system to use custom logger across the codebase 2023-10-16 14:56:00 +03:00
Ori Kotek
e7258e732b
Refactor repo-specific settings application into a utility function, fix merge bug 2023-10-14 01:39:05 +03:00
mrT23
e2417ebe88
Merge pull request #363 from Codium-ai/tr/pr_suggestions
publish each suggestion seperatly only on gitlab
2023-10-08 17:01:44 +03:00
mrT23
da98fd712f
note 2023-10-08 16:58:22 +03:00
mrT23
e6548f4fe1
simpler solution 2023-10-08 16:57:22 +03:00
mrT23
e66fed2468
Merge pull request #362 from zmeir/zmeir-fix_help_message_for_bot
Fixed help message for bot user
2023-10-08 16:52:18 +03:00
mrT23
1b3fb49f9c
publish each suggestion seperatly only on gitlab 2023-10-08 16:50:25 +03:00
Zohar Meir
8f9f09ecbf
Fixed help message for bot user
This changes the help message to display properly when running a custom deployment of the PR-Agent app (i.e. not via GitHub Actions, and with the setting `github_app.override_deployment_type=false`)
2023-10-08 16:19:11 +03:00
jamesrom
92e9012fb6 Error handling 2023-10-07 09:39:53 +11:00
jamesrom
43dc648b05 Simplify filter 2023-10-06 22:44:29 +11:00
jamesrom
baa0e95227 Code comments for ignore.toml 2023-10-06 21:53:10 +11:00
jamesrom
b27f57d05d Update settings, documentation 2023-10-06 21:03:36 +11:00
mrT23
fd8c90041c
azure 2023-10-06 08:31:31 +03:00
mrT23
ea6253e2e8
revert azure 2023-10-06 08:12:11 +03:00
mrT23
2945c36899
source_branch 2023-10-05 18:21:52 +03:00
mrT23
1bab26f1c5
gfm_supported 2023-10-05 18:08:02 +03:00
mrT23
72eecbbf61
add line number 2023-10-05 17:59:08 +03:00
mrT23
989c56220b
add line number 2023-10-05 17:48:36 +03:00
jamesrom
e387086890 Add support for ignoring files
Add ignore.toml, configuration for ignoring files
Add file_filter.py, for matching files against glob/regex patterns
Update relevant code to use file filter
+Tests
2023-10-06 01:43:35 +11:00
mrT23
088f256415
stable 2023-10-05 17:03:10 +03:00
mrT23
44239f1a79
Patch Extra Lines 2023-10-05 08:38:43 +03:00
mrT23
428e6382bd
prompts minor update 2023-10-05 08:17:37 +03:00
mrT23
02ea2c7a40
Merge pull request #345 from brianpham93/azure-devops-get-item-error
Azure Devops: Set file content as empty string when azure_devops_client.get_item fails
2023-10-02 10:57:08 +03:00
Brian Pham
589d329a3c Add error logs for troubleshooting 2023-10-02 05:37:27 +00:00
Hussam.lawen
e941fa9ec0
Add to user tools guide 2023-10-01 19:51:15 +03:00
Brian Pham
4479c5f11b Azure Devops: Set file content as empty string when error 2023-10-01 16:22:37 +00:00
Hussam.lawen
b2369c66d8
Merge commit '8d075b76ae081d0d38813f789478e4fa0f404cd8' into hl/add_docs
# Conflicts:
#	README.md
2023-10-01 13:55:50 +03:00
Hussam.lawen
ab5ac8ffa8
rename vars + Add to README.md 2023-10-01 13:52:00 +03:00
Hussam.lawen
ccc7f1e10a
rename vars 2023-10-01 13:07:08 +03:00
mrT23
8d075b76ae
Merge pull request #344 from brianpham93/azure-devops-ignore-tree
Ignore change item with `gitObjectType == 'tree'`
2023-10-01 10:39:03 +03:00
BrianPham
32a8b0e9bc Ignore change item with gitObjectType == 'tree' 2023-10-01 07:34:01 +00:00
mrT23
e07412c098
prompt and readme updates 2023-10-01 09:00:58 +03:00
mrT23
813fa8571e
tools guide 2023-09-29 11:44:03 +03:00
Stefan Petrushevski
cae0f627e2
Add lockb to bad extensions 2023-09-28 22:26:25 +02:00
Hussam.lawen
7fbdc3aead
rstrip 2023-09-28 22:47:26 +03:00