28c5ad1d8b
nit
2024-01-08 13:06:03 +02:00
2bb5ae8c0d
Remove redundant condition (status 422 already means the same)
2024-01-08 13:05:10 +02:00
b0bffdec84
Refactor and add configuration toggle
2024-01-08 12:00:20 +02:00
22d17985a1
Less noisy fallback for publish_code_suggestions in case of invalid comments
...
As a first option, `publish_code_suggestions` will try to post all review comments in a single GitHub review. This is preferred because it will group all comments together in the GitHub UI under the same review, and will trigger just one notification for any viewers of the PR.
If just one of the comments is malformed, the entire API request will fail and none of the comments will be posted to the PR. In the current implementation, the fallback mechanism is to just post each comment separately with `try/except` and skip the invalid comments. This works, but potentially creates a lot of noise in the PR as each comment is posted as in a separate review, creating multiple notifications.
This suggested fallback is based on a similar idea, but without creating multiple review notifications. The it works is by iterating over the potential comments, and starting a PENDING review for the current comment. The review is not submitted and does not trigger a notification, but it is verified against the GitHub API, and so we can verify if the comment is valid. After checking all comments we then submit a single review with all the verified comments which is guaranteed to succeed.
The end result is having the exact same comments posted to the PR as with the current fallback method, but the downside is having twice as many API calls (for each comment we have 1 extra API call to delete the pending review).
2024-01-07 16:00:44 +02:00
faba5a224a
Merge pull request #575 from Codium-ai/tr/user_description
...
docs: Enhance 'improve' tool documentation and update 'DESCRIBE.md' w…
2024-01-07 01:30:44 -08:00
19f85f08b0
docs: Enhance 'improve' tool documentation and update 'DESCRIBE.md' with hyperlink
2024-01-07 11:29:07 +02:00
8512e9dffb
docs: Enhance 'improve' tool documentation and update 'DESCRIBE.md' with hyperlink
2024-01-07 11:26:24 +02:00
7c0be8ca44
docs: Enhance 'improve' tool documentation and update 'DESCRIBE.md' with hyperlink
2024-01-07 11:22:53 +02:00
d9872d7031
docs: Enhance 'improve' tool documentation and update 'DESCRIBE.md' with hyperlink
2024-01-07 11:19:47 +02:00
be4b9c4991
Merge pull request #574 from Codium-ai/tr/user_description
...
Enhance and reorganize documentation for review and describe tools
2024-01-07 00:01:30 -08:00
94172104f0
docs: Reorganize and enhance documentation for review and describe tools
2024-01-07 09:56:09 +02:00
8ec790d4f7
Merge remote-tracking branch 'origin/main' into tr/user_description
...
# Conflicts:
# docs/DESCRIBE.md
2024-01-07 09:49:43 +02:00
640972b00a
docs: Reorganize and enhance documentation for review and describe tools
2024-01-07 09:48:17 +02:00
7810ba7d76
docs: Reorganize and enhance documentation for review and describe tools
2024-01-07 09:40:24 +02:00
d91f1fce09
docs: Reorganize and enhance documentation for review and describe tools
2024-01-07 09:36:03 +02:00
edcb666fbc
docs: Reorganize and enhance documentation for review and describe tools
2024-01-07 09:28:06 +02:00
6d359fe1d8
Update DESCRIBE.md
2024-01-06 21:46:11 +02:00
19e38595f3
Merge pull request #573 from Codium-ai/tr/user_description
...
feat: Update adaptive collapsible file list logic in pr_description.py
2024-01-06 11:08:55 -08:00
355ef8c476
feat: Update adaptive collapsible file list logic in pr_description.py
2024-01-06 10:36:36 +02:00
f08d225360
Merge pull request #572 from Codium-ai/tr/user_description
...
describe docs
2024-01-06 00:15:36 -08:00
22b7dd9f2d
s
2024-01-06 10:15:04 +02:00
a85a328791
s
2024-01-06 10:08:29 +02:00
82d10f24f0
s
2024-01-06 10:04:48 +02:00
c9717f1d73
Update README.md
2024-01-05 23:29:59 +02:00
82b58d5b09
Update README.md
2024-01-05 21:37:45 +02:00
b6d41d6a91
Merge pull request #571 from Codium-ai/tr/user_description
...
Enhance Documentation and Configuration of 'describe' Tool
2024-01-05 11:30:41 -08:00
ac74fa8431
docs
2024-01-05 21:28:54 +02:00
42704d5781
Merge remote-tracking branch 'origin/main' into tr/user_description
2024-01-05 17:03:30 +02:00
3628786a61
feat: Clarify PRType label member usage in pr_description_prompts.toml
2024-01-05 17:03:14 +02:00
c69ede0138
Update REVIEW.md
2024-01-05 11:10:55 +02:00
4349156e97
Merge pull request #570 from wesvetter/patch-1
...
📚 Minor update to REVIEW.md for inclusive language
2024-01-04 22:48:36 -08:00
9f88105f72
📚 Minor update to REVIEW.md for inclusive language
...
Replaces "his [judgement]" with "their".
2024-01-04 10:32:53 -08:00
fe6b2065fb
Merge pull request #569 from zmeir/zmeir/enhance/auto_improve_extended_simple
...
Add toggle to automatically enable `/improve --extended`
2024-01-04 09:06:08 -08:00
c2b0891c0b
Simpler auto-extended toggle and keep the default as false
2024-01-04 18:53:45 +02:00
782f1ca1bd
Merge pull request #568 from Codium-ai/tr/user_description
...
Enhancement: Improved Extraction and Placement of User Descriptions in PRs
2024-01-04 08:35:34 -08:00
6d18a0c843
feat: Improve user description extraction in git_provider.py
2024-01-04 18:34:00 +02:00
e6093cd768
Merge pull request #567 from zmeir/zmeir/enhance/support_azure_in_langchain_ai_handler
...
Add support for Azure OpenAI in LangChainOpenAIHandler
2024-01-04 08:22:01 -08:00
aea0c4d45f
Merge pull request #566 from zmeir/zmeir/fix/config_command_missing_arg
...
Fix failing `/config` command
2024-01-04 08:14:29 -08:00
1c2bb2ef3d
feat: Update user description extraction and placement in PR description
2024-01-04 18:05:28 +02:00
7762bf59bf
feat: Update user description extraction and placement in PR description
2024-01-04 18:01:55 +02:00
3e29848cd0
Merge remote-tracking branch 'origin/main' into tr/user_description
...
# Conflicts:
# pr_agent/git_providers/git_provider.py
2024-01-04 17:49:10 +02:00
c3b5aaf8cc
feat: Update user description extraction and placement in PR description
2024-01-04 17:46:24 +02:00
ba3f22d81e
Move logging to a central location for all AI Handlers
2024-01-04 16:22:22 +02:00
ac7aaa0cd3
Add support for Azure OpenAI in LangChainOpenAIHandler
2024-01-04 16:22:22 +02:00
1ade09eaa3
Fix failing /config command
...
All commands need the `ai_handler` argument. The PRConfig class was missing it in the `__init__` method and so it failed with this error:
```
File "/home/vcap/app/pr_agent/agent/pr_agent.py", line 76, in handle_request
await command2class[action](pr_url, ai_handler=self.ai_handler, args=args).run()
TypeError: PRConfig.__init__() got an unexpected keyword argument 'ai_handler'
```
2024-01-04 14:49:34 +02:00
b7af45166a
Merge pull request #561 from zmeir/zmeir/fix/get_user_description
...
Fix `get_user_description`
2024-01-04 00:40:08 -08:00
92f89e6ca0
Merge pull request #565 from Codium-ai/tr/remove_old_walkthrough
...
Remove old 'enable_file_walkthrough' mode
2024-01-04 00:36:05 -08:00
ed78bfd946
use_collapsible_file_list
2024-01-04 10:27:07 +02:00
4204d78d7e
feat: Remove file walkthrough feature from PR agent
2024-01-04 09:59:44 +02:00
3c2ed8bbf1
feat: Remove file walkthrough feature from PR agent
2024-01-04 09:42:15 +02:00
