zmeir
2bb5ae8c0d
Remove redundant condition (status 422 already means the same)
2024-01-08 13:05:10 +02:00
zmeir
b0bffdec84
Refactor and add configuration toggle
2024-01-08 12:00:20 +02:00
zmeir
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
Tal
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
mrT23
19f85f08b0
docs: Enhance 'improve' tool documentation and update 'DESCRIBE.md' with hyperlink
2024-01-07 11:29:07 +02:00
mrT23
8512e9dffb
docs: Enhance 'improve' tool documentation and update 'DESCRIBE.md' with hyperlink
2024-01-07 11:26:24 +02:00
mrT23
7c0be8ca44
docs: Enhance 'improve' tool documentation and update 'DESCRIBE.md' with hyperlink
2024-01-07 11:22:53 +02:00
mrT23
d9872d7031
docs: Enhance 'improve' tool documentation and update 'DESCRIBE.md' with hyperlink
2024-01-07 11:19:47 +02:00
Tal
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
mrT23
94172104f0
docs: Reorganize and enhance documentation for review and describe tools
2024-01-07 09:56:09 +02:00
mrT23
8ec790d4f7
Merge remote-tracking branch 'origin/main' into tr/user_description
...
# Conflicts:
# docs/DESCRIBE.md
2024-01-07 09:49:43 +02:00
mrT23
640972b00a
docs: Reorganize and enhance documentation for review and describe tools
2024-01-07 09:48:17 +02:00
mrT23
7810ba7d76
docs: Reorganize and enhance documentation for review and describe tools
2024-01-07 09:40:24 +02:00
mrT23
d91f1fce09
docs: Reorganize and enhance documentation for review and describe tools
2024-01-07 09:36:03 +02:00
mrT23
edcb666fbc
docs: Reorganize and enhance documentation for review and describe tools
2024-01-07 09:28:06 +02:00
Tal
6d359fe1d8
Update DESCRIBE.md
2024-01-06 21:46:11 +02:00
Tal
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
mrT23
355ef8c476
feat: Update adaptive collapsible file list logic in pr_description.py
2024-01-06 10:36:36 +02:00
Tal
f08d225360
Merge pull request #572 from Codium-ai/tr/user_description
...
describe docs
2024-01-06 00:15:36 -08:00
mrT23
22b7dd9f2d
s
2024-01-06 10:15:04 +02:00
mrT23
a85a328791
s
2024-01-06 10:08:29 +02:00
mrT23
82d10f24f0
s
2024-01-06 10:04:48 +02:00
Tal
c9717f1d73
Update README.md
2024-01-05 23:29:59 +02:00
Tal
82b58d5b09
Update README.md
2024-01-05 21:37:45 +02:00
Tal
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
mrT23
ac74fa8431
docs
2024-01-05 21:28:54 +02:00
mrT23
42704d5781
Merge remote-tracking branch 'origin/main' into tr/user_description
2024-01-05 17:03:30 +02:00
mrT23
3628786a61
feat: Clarify PRType label member usage in pr_description_prompts.toml
2024-01-05 17:03:14 +02:00
Tal
c69ede0138
Update REVIEW.md
2024-01-05 11:10:55 +02:00
Tal
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
Wes Vetter
9f88105f72
📚 Minor update to REVIEW.md for inclusive language
...
Replaces "his [judgement]" with "their".
2024-01-04 10:32:53 -08:00
Tal
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
Zohar Meir
c2b0891c0b
Simpler auto-extended toggle and keep the default as false
2024-01-04 18:53:45 +02:00
Tal
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
mrT23
6d18a0c843
feat: Improve user description extraction in git_provider.py
2024-01-04 18:34:00 +02:00
Tal
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
Tal
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
mrT23
1c2bb2ef3d
feat: Update user description extraction and placement in PR description
2024-01-04 18:05:28 +02:00
mrT23
7762bf59bf
feat: Update user description extraction and placement in PR description
2024-01-04 18:01:55 +02:00
mrT23
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
mrT23
c3b5aaf8cc
feat: Update user description extraction and placement in PR description
2024-01-04 17:46:24 +02:00
zmeir
ba3f22d81e
Move logging to a central location for all AI Handlers
2024-01-04 16:22:22 +02:00
zmeir
ac7aaa0cd3
Add support for Azure OpenAI in LangChainOpenAIHandler
2024-01-04 16:22:22 +02:00
zmeir
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
Tal
b7af45166a
Merge pull request #561 from zmeir/zmeir/fix/get_user_description
...
Fix `get_user_description`
2024-01-04 00:40:08 -08:00
Tal
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
mrT23
ed78bfd946
use_collapsible_file_list
2024-01-04 10:27:07 +02:00
mrT23
4204d78d7e
feat: Remove file walkthrough feature from PR agent
2024-01-04 09:59:44 +02:00
mrT23
3c2ed8bbf1
feat: Remove file walkthrough feature from PR agent
2024-01-04 09:42:15 +02:00
Zohar Meir
8d2da74380
Find user description in a case-insensitive way
2024-01-04 09:41:55 +02:00
